0f796dd004
Update to v4.1.0. Also includes other major packaging changes as follows: There is a new package maintenance workflow - see README.PACKAGING for details. The sibling packages qemu-linux-user and qemu-testsuite are now created with the Build Service's MultiBuild feature. This also necessitates combining the qemu-linux-user changelog content back into qemu's. Luckily the delta there is quite small. Note that the qemu spec file is now that much busier, but added section markers should help reduce the confusion. Also qemu is being enabled for RISCV host compatibility, so some changes are related to that as well. OBS-URL: https://build.opensuse.org/request/show/730437 OBS-URL: https://build.opensuse.org/package/show/Virtualization/qemu?expand=0&rev=487
54 lines
2.2 KiB
Diff
54 lines
2.2 KiB
Diff
From: Valentine Barshak <gvaxon@gmail.com>
|
|
Date: Mon, 22 Jul 2019 10:47:50 +0100
|
|
Subject: Workaround compilation error with gcc 9.1
|
|
|
|
References: bsc#1121464
|
|
|
|
Compiling with gcc 9.1 generates lots of "taking address of packed
|
|
member of ... may result in an unaligned pointer value" warnings.
|
|
|
|
Some of these warnings are genuine, and indicate correctly that parts
|
|
of iPXE currently require the CPU (or runtime environment) to support
|
|
unaligned accesses. For example: the TCP/IP receive data path will
|
|
attempt to access 32-bit fields that may not be aligned to a 32-bit
|
|
boundary.
|
|
|
|
Other warnings are either spurious (such as when the pointer is to a
|
|
variable-length byte array, which can have no alignment requirement
|
|
anyway) or unhelpful (such as when the pointer is used solely to
|
|
provide a debug colour value for the DBGC() macro).
|
|
|
|
There appears to be no easy way to silence the spurious warnings.
|
|
Since the ability to perform unaligned accesses is already a
|
|
requirement for iPXE, work around the problem by silencing this class
|
|
of warnings.
|
|
|
|
Signed-off-by: Valentine Barshak <gvaxon@gmail.com>
|
|
Modified-by: Michael Brown <mcb30@ipxe.org>
|
|
Signed-off-by: Michael Brown <mcb30@ipxe.org>
|
|
Signed-off-by: Bruce Rogers <brogers@suse.com>
|
|
---
|
|
src/Makefile.housekeeping | 9 +++++++++
|
|
1 file changed, 9 insertions(+)
|
|
|
|
diff --git a/roms/ipxe/src/Makefile.housekeeping b/roms/ipxe/src/Makefile.housekeeping
|
|
index 97fa325bb52314e05192d0414436..e5f6927de889167d286ccfcdda92 100644
|
|
--- a/roms/ipxe/src/Makefile.housekeeping
|
|
+++ b/roms/ipxe/src/Makefile.housekeeping
|
|
@@ -185,6 +185,15 @@ WNST_TEST = $(CC) -Wstringop-truncation -x c -c /dev/null -o /dev/null \
|
|
>/dev/null 2>&1
|
|
WNST_FLAGS := $(shell $(WNST_TEST) && $(ECHO) '-Wno-stringop-truncation')
|
|
WORKAROUND_CFLAGS += $(WNST_FLAGS)
|
|
+
|
|
+# gcc 9.1 generates warnings for taking address of packed member which
|
|
+# may result in an unaligned pointer value. Inhibit the warnings.
|
|
+#
|
|
+WNAPM_TEST = $(CC) -Wno-address-of-packed-member -x c -c /dev/null \
|
|
+ -o /dev/null >/dev/null 2>&1
|
|
+WNAPM_FLAGS := $(shell $(WNAPM_TEST) && \
|
|
+ $(ECHO) '-Wno-address-of-packed-member')
|
|
+WORKAROUND_CFLAGS += $(WNAPM_FLAGS)
|
|
endif
|
|
|
|
# Some versions of gas choke on division operators, treating them as
|