diff options
Diffstat (limited to 'results/classifier/108/graphic/1824528')
| -rw-r--r-- | results/classifier/108/graphic/1824528 | 76 |
1 files changed, 76 insertions, 0 deletions
diff --git a/results/classifier/108/graphic/1824528 b/results/classifier/108/graphic/1824528 new file mode 100644 index 000000000..f5cc3f57d --- /dev/null +++ b/results/classifier/108/graphic/1824528 @@ -0,0 +1,76 @@ +graphic: 0.971 +performance: 0.965 +socket: 0.954 +files: 0.948 +network: 0.941 +PID: 0.932 +permissions: 0.926 +device: 0.919 +debug: 0.913 +other: 0.912 +semantic: 0.901 +vnc: 0.900 +KVM: 0.895 +boot: 0.785 + +qemu fails to compile on gcc 9 `error: taking address of packed member of ‘struct <anonymous>’ may result in an unaligned pointer value [-Werror=address-of-packed-member]` + +Qemu compilation fails with below error on ppc64le host with gcc 9(9.0.1 20190328) +repo: https://github.com/qemu/qemu.git +branch: master +commit e1be98540ee672ef93292b65a986055512237c35 + + + CC net/dump.o +hw/usb/dev-mtp.c: In function ‘usb_mtp_write_metadata’: +hw/usb/dev-mtp.c:1708:36: error: taking address of packed member of ‘struct <anonymous>’ may result in an unaligned pointer value [-Werror=address-of-packed-member] + 1708 | dataset->filename); + | ~~~~~~~^~~~~~~~~~ +cc1: all warnings being treated as errors + CC net/eth.o +make: *** [/home/kvmci/qemu-main/rules.mak:69: hw/usb/dev-mtp.o] Error 1 +make: *** Waiting for unfinished jobs.... + CC net/announce.o + +Tried to patch as below and it compiles fine, not sure if this is right fix though, + + +# git diff +diff --git a/hw/usb/dev-mtp.c b/hw/usb/dev-mtp.c +index ebf210f..7d512e5 100644 +--- a/hw/usb/dev-mtp.c ++++ b/hw/usb/dev-mtp.c +@@ -231,7 +231,7 @@ typedef struct { + char date_modified[0]; /*unused*/ + char keywords[0]; /*unused*/ + /* string and other data follows */ +-} QEMU_PACKED ObjectInfo; ++} ObjectInfo; + + #define TYPE_USB_MTP "usb-mtp" + #define USB_MTP(obj) OBJECT_CHECK(MTPState, (obj), TYPE_USB_MTP) + + +This struct represents the MTP protocol wire format so *must* be marked packed. + +There unfortunately quite a few flaws in this MTP code area, so fixing the gcc warning is not straightforward. + +https://lists.gnu.org/archive/html/qemu-devel/2019-03/msg07763.html + +As a workaround, the simplest thing is to configure with --disable-werror, which will reduce these from errors to warnings. (This is the default if you're building from one of our release tarballs, but for builds from git we default to making all warnings into errors so we catch and fix them quickly.) + + +Sure, Thanks, using the workaround to proceed. + +Good if each commit goes through a automated build test across different platforms vs gcc versions. + +Regards, +-Satheesh. + +Fixed in + +https://lists.gnu.org/archive/html/qemu-devel/2019-04/msg02524.html + +The fixes for this issue in dev-mtp.c are now in QEMU git master and will be in the 4.1 release. + + |