[PATCH] Makefile: Build with -Wno-address-of-packed-member

[Kevin O'Connor]

Building with gcc v9 causes lots of warnings about pointers to packed
variables.  However, SeaBIOS is limited to x86 where unaligned
reads/writes are supported by the cpu.  So, disable that warning.

Signed-off-by: Kevin O'Connor <[e-mail]>
---
 Makefile | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Makefile b/Makefile
index d16d1ae..5f7d537 100644
--- a/Makefile
+++ b/Makefile
@@ -68,6 +68,7 @@ COMMONCFLAGS += $(call cc-option,$(CC),-fno-pie,)
 COMMONCFLAGS += $(call cc-option,$(CC),-fno-stack-protector,)
 COMMONCFLAGS += $(call cc-option,$(CC),-fno-stack-protector-all,)
 COMMONCFLAGS += $(call cc-option,$(CC),-fstack-check=no,)
+COMMONCFLAGS += $(call cc-option,$(CC),-Wno-address-of-packed-member,)
 COMMA := ,
 
 CFLAGS32FLAT := $(COMMONCFLAGS) -DMODE16=0 -DMODESEGMENT=0
-- 
2.21.0

[Philippe Mathieu-Daudé]

On 8/21/19 3:21 PM, Kevin O'Connor wrote:
> Building with gcc v9 causes lots of warnings about pointers to packed
> variables.  However, SeaBIOS is limited to x86 where unaligned
> reads/writes are supported by the cpu.  So, disable that warning.
>
> Signed-off-by: Kevin O'Connor <[e-mail]>

Reviewed-by: Philippe Mathieu-Daudé <[e-mail]>
> ---
>  Makefile | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/Makefile b/Makefile
> index d16d1ae..5f7d537 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -68,6 +68,7 @@ COMMONCFLAGS += $(call cc-option,$(CC),-fno-pie,)
>  COMMONCFLAGS += $(call cc-option,$(CC),-fno-stack-protector,)
>  COMMONCFLAGS += $(call cc-option,$(CC),-fno-stack-protector-all,)
>  COMMONCFLAGS += $(call cc-option,$(CC),-fstack-check=no,)
> +COMMONCFLAGS += $(call cc-option,$(CC),-Wno-address-of-packed-member,)
>  COMMA := ,
>
> CFLAGS32FLAT := $(COMMONCFLAGS) -DMODE16=0 -DMODESEGMENT=0

[Kevin O'Connor]

On Wed, Aug 21, 2019 at 03:55:16PM +0200, Philippe Mathieu-Daudé wrote:
> On 8/21/19 3:21 PM, Kevin O'Connor wrote:
> > Building with gcc v9 causes lots of warnings about pointers to packed
> > variables.  However, SeaBIOS is limited to x86 where unaligned
> > reads/writes are supported by the cpu.  So, disable that warning.
> >
> > Signed-off-by: Kevin O'Connor <[e-mail]>
>
> Reviewed-by: Philippe Mathieu-Daudé <[e-mail]>

Thanks.  I committed this patch.

-Kevin