xen ftbfs in disco
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
xen (Ubuntu) |
Fix Released
|
High
|
Stefan Bader | ||
Cosmic |
Won't Fix
|
High
|
Unassigned | ||
Disco |
Fix Released
|
High
|
Stefan Bader | ||
Eoan |
Fix Released
|
High
|
Stefan Bader |
Bug Description
[Impact]
Warning message causes FTBFS when configured to treat warnings as errors (edit: there are actually multiple failures due to new gcc-8 and gcc-9 warnings which are treated as errors). This bug report is used to track all the needed changes to allow compiling again in Disco and later.
IMPORTANT NOTE(!): Albeit compiling, there is still an unresolved problem when compiling with gcc-9 (and possibly newer) which produces a Xen hypervisor which is unbootable (the host immediately reboots after start (no message) as if hitting a triple fault.
So as a quick solution to get a working Xen hypervisor which works around bug #1851091, the only option is to do the upload to Disco and copy forward with binaries.
[Test Case]
$ apt-get source xen
(...)
$ cd xen*
$ sudo apt-get build-dep xen
(...)
$ ./configure
(...)
$ make
(...)
/usr/include/
return __builtin_
__bos (__s), __fmt, __va_arg_pack ());
cc1: all warnings being treated as errors
make[4]: *** [/home/
make[4]: Leaving directory '/home/
$ echo $?
2
[Regression Potential]
Changes string formatting of an error message, which should be
transparent to users, but things to watch for would relate to xen's
error message logs.
[Fix]
A simple tweak to snprintf() formatting from upstream would prevent
the theoretical buffer overflow being warned about. This needs applied
to disco and eoan; bionic and earlier are not affected.
[Discussion]
[Original Report]
https:/
gcc -m64 -DBUILD_ID -fno-strict-
xen-lowmemd.c: In function ‘handle_low_mem’:
xen-lowmemd.
In file included from /usr/include/
/usr/include/
return __builtin_
__bos (__s), __fmt, __va_arg_pack ());
cc1: all warnings being treated as errors
make[5]: *** [/<<PKGBUILDDIR
make[5]: Leaving directory '/<<PKGBUILDDIR
make[4]: *** [/<<PKGBUILDDIR
make[4]: Leaving directory '/<<PKGBUILDDIR
make[3]: *** [/<<PKGBUILDDIR
make[3]: Leaving directory '/<<PKGBUILDDIR
make[2]: *** [debian/
make[2]: Leaving directory '/<<PKGBUILDDIR>>'
make[1]: *** [debian/
make[1]: Leaving directory '/<<PKGBUILDDIR>>'
Changed in xen (Ubuntu): | |
importance: | Undecided → High |
tags: | added: ftbfs rls-dd-incoming |
Changed in xen (Ubuntu Cosmic): | |
status: | New → Triaged |
Changed in xen (Ubuntu): | |
status: | New → Triaged |
Changed in xen (Ubuntu Cosmic): | |
importance: | Undecided → High |
Changed in xen (Ubuntu Disco): | |
status: | New → Triaged |
importance: | Undecided → High |
tags: | added: patch |
tags: |
added: server-next removed: patch |
tags: | added: patch |
description: | updated |
Changed in xen (Ubuntu Eoan): | |
assignee: | nobody → Bryce Harrington (bryce) |
Changed in xen (Ubuntu Disco): | |
assignee: | nobody → Bryce Harrington (bryce) |
Changed in xen (Ubuntu Eoan): | |
assignee: | Bryce Harrington (bryce) → nobody |
Changed in xen (Ubuntu Disco): | |
assignee: | Bryce Harrington (bryce) → nobody |
Changed in xen (Ubuntu Eoan): | |
assignee: | nobody → Stefan Bader (smb) |
tags: |
added: update-excuse removed: server-next |
Changed in xen (Ubuntu Disco): | |
assignee: | nobody → Stefan Bader (smb) |
Changed in xen (Ubuntu Disco): | |
status: | Triaged → Fix Committed |
Changed in xen (Ubuntu Disco): | |
status: | Fix Committed → In Progress |
Changed in xen (Ubuntu Eoan): | |
status: | Triaged → In Progress |
description: | updated |
tags: |
added: verification-done verification-done-disco verification-done-eoan removed: verification-needed verification-needed-disco |
also ftbfs in cosmic: /launchpadlibra rian.net/ 418304930/ buildlog_ ubuntu- cosmic- amd64.xen_ 4.9.2-0ubuntu2_ BUILDING. txt.gz
https:/