From: "Luis R. Rodriguez" <mcgrof@do-not-panic.com>
To: xen-devel@lists.xensource.com, ian.campbell@citrix.com,
JBeulich@suse.com
Cc: samuel.thibault@ens-lyon.org,
"Luis R. Rodriguez" <mcgrof@suse.com>,
pryorm09@gmail.com, cfergeau@redhat.com
Subject: [PATCH 0/9] xen: build fixes with gcc5 and binutils 2.25.0
Date: Fri, 20 Nov 2015 09:47:43 -0800 [thread overview]
Message-ID: <1448041672-3986-1-git-send-email-mcgrof@do-not-panic.com> (raw)
From: "Luis R. Rodriguez" <mcgrof@suse.com>
Here's a slew of build fixes as well as build warning fixes
required when using the latest build tools, at least gcc 5 and
binutils 2.25.0. I ran into this while doing development on OpenSUSE
factory. Other rolling distros seem to be having similar issues based
on inspection of recent patches.
After fixing the issue I looked into the respective upstream
projects where are appropriate and provided annotations for
respective upstream fixes. The patches that have a respective upstream
project but do not have upstream annotations lack the annotations are
upstream code has already changed considerably and the patches do not
apply or the code has already been fixed but the respective upstream
atomic fix does not apply to our code branch. That said then, there is
no need to send anything to our usptream. My changes actually do have
a bit better annotations for some fixes than upstream, specifically
where I see a change fixed an issue I explain the actual issue that
should have been caused without the patch. That should make it easier
to evaluate integration of the patches into a stable branch or not.
Since some patches are part of upstream you could just consider
bumping upstream, or pulling the respective patch from upstream,
some of my annotations however have a bit better explanation for
why we wnat the changes. For stable branches these could be considere
as backport fixes. Up to you to decide how you want to manage these
fixes.
I've build tested this on both the xen master branch and the
xen stable-4.6 branch, so feel free to consider some of these into
stable-4.6 if its desirable to build Xen with the latest and greatest gcc
and binutils. I've only run time tested the master branch of Xen
on dom0 with these changes.
Changes are required on a slew of different trees. The order of the
patches match the order in which I found the issues and fixed them,
the summary of the changes for each different tree is below.
These following patches just fix warnings and as such are not required to
complete proper building, but they seem worthy enough at least for
consideration on unstable:
hw/usb-net.c: fix state check
qemu-xen-dir: virtio-rng: fix check for period_ms validity
mini-os: fix linker warning with app.lds
vtpm: fix vtpmblk.c compilation warning
vtpm: guard against redefining TPM_VENDOR_COMMAND
The rest are definitely needed for proper building.
The gnutls fix goes with a Coccinelle rule file which should enable
proper transformation of the needed changes in other projects, so
long as the code matches in form. The Coccinelle rule file changes
can also enable the same patch to be easily rebased should the code
change in between the patch being considered for merging which would
otherwise cause a legacy patch conflict.
------------------------------------------------------------------------------
tools/firmware/seabios-dir-remote/
Luis R. Rodriguez (1):
Revert "Use the extra stack for 16bit USB and PS2 keyboard/mouse commands."
src/kbd.c | 6 +++---
src/mouse.c | 6 +++---
2 files changed, 6 insertions(+), 6 deletions(-)
------------------------------------------------------------------------------
tools/qemu-xen-traditional-dir-remote/
Luis R. Rodriguez (2):
Use gnutls_priority_set_direct() to deprecate gnutls_*_set()
hw/usb-net.c: fix state check
hw/usb-net.c | 4 ++--
vnc.c | 21 +--------------------
2 files changed, 3 insertions(+), 22 deletions(-)
------------------------------------------------------------------------------
tools/qemu-xen-dir-remote/
Luis R. Rodriguez (2):
qemu-xen-dir: avoid using spice-experimental.h
qemu-xen-dir: virtio-rng: fix check for period_ms validity
hw/virtio/virtio-rng.c | 2 +-
spice-qemu-char.c | 1 -
ui/spice-core.c | 1 -
3 files changed, 1 insertion(+), 3 deletions(-)
------------------------------------------------------------------------------
extras/mini-os-remote/
Luis R. Rodriguez (1):
mini-os: fix linker warning with app.lds
Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
------------------------------------------------------------------------------
Top level xen git tree on stubdom/
Luis R. Rodriguez (3):
stubdom: fix unfound libgmp library issues
vtpm: fix vtpmblk.c compilation warning
vtpm: guard against redefining TPM_VENDOR_COMMAND
stubdom/Makefile | 5 +++++
stubdom/vtpm/vtpmblk.c | 1 +
stubdom/vtpmmgr/vtpm_manager.h | 3 +++
3 files changed, 9 insertions(+)
--
2.6.2
next reply other threads:[~2015-11-20 17:47 UTC|newest]
Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-20 17:47 Luis R. Rodriguez [this message]
2015-11-20 17:47 ` [PATCH 1/9] Revert "Use the extra stack for 16bit USB and PS2 keyboard/mouse commands." Luis R. Rodriguez
2015-11-20 17:47 ` [PATCH 2/9] Use gnutls_priority_set_direct() to deprecate gnutls_*_set() Luis R. Rodriguez
2015-11-25 14:53 ` Konrad Rzeszutek Wilk
2015-11-25 19:36 ` [Xen-devel] " Luis R. Rodriguez
2015-11-25 20:44 ` Konrad Rzeszutek Wilk
2016-02-06 1:44 ` [Xen-devel] " Luis R. Rodriguez
2016-02-06 3:45 ` Konrad Rzeszutek Wilk
2016-02-06 4:03 ` Luis R. Rodriguez
2016-02-06 4:12 ` Konrad Rzeszutek Wilk
2016-02-06 4:36 ` Luis R. Rodriguez
2016-02-06 20:18 ` Doug Goldstein
2016-02-09 10:54 ` Jan Beulich
2016-02-09 15:46 ` [Xen-devel] " Luis R. Rodriguez
2016-02-09 16:06 ` Jan Beulich
2016-02-09 16:36 ` Olaf Hering
2016-02-09 16:42 ` [Xen-devel] " Luis R. Rodriguez
2016-02-09 18:23 ` Olaf Hering
2016-02-10 3:04 ` [Xen-devel] " Luis R. Rodriguez
2016-02-10 9:46 ` Ian Campbell
2016-02-10 15:41 ` [Xen-devel] " Luis R. Rodriguez
2016-02-10 15:44 ` Olaf Hering
2016-02-10 15:55 ` [Xen-devel] " Luis R. Rodriguez
2015-11-20 17:47 ` [PATCH 3/9] hw/usb-net.c: fix state check Luis R. Rodriguez
2015-11-20 17:47 ` [PATCH 4/9] qemu-xen-dir: spice: remove spice-experimental.h include Luis R. Rodriguez
2015-11-20 17:47 ` [PATCH 5/9] qemu-xen-dir: virtio-rng: fix check for period_ms validity Luis R. Rodriguez
2015-11-20 17:47 ` [PATCH 6/9] mini-os: fix linker warning with app.lds Luis R. Rodriguez
2015-11-20 17:47 ` [PATCH 7/9] stubdom: fix unfound libgmp library issues Luis R. Rodriguez
2015-11-20 17:47 ` [PATCH 8/9] vtpm: fix vtpmblk.c compilation warning Luis R. Rodriguez
2015-11-20 17:47 ` [PATCH 9/9] vtpm: guard against redefining TPM_VENDOR_COMMAND Luis R. Rodriguez
2016-02-06 1:48 ` [PATCH 0/9] xen: build fixes with gcc5 and binutils 2.25.0 Luis R. Rodriguez
2016-02-06 3:52 ` Konrad Rzeszutek Wilk
2016-02-06 4:07 ` Luis R. Rodriguez
2016-02-06 20:22 ` Doug Goldstein
2016-02-07 5:34 ` Fengguang Wu
2016-02-08 9:58 ` Ian Campbell
2016-02-09 15:56 ` Luis R. Rodriguez
2016-02-09 16:26 ` Ian Campbell
2016-02-09 16:39 ` Luis R. Rodriguez
2016-02-10 9:34 ` Ian Campbell
2016-02-09 8:08 ` Jan Beulich
2016-02-09 16:09 ` Luis R. Rodriguez
2016-02-09 16:22 ` Ian Campbell
2016-02-09 16:30 ` Luis R. Rodriguez
2016-02-09 16:39 ` Ian Campbell
2016-02-09 16:53 ` Luis R. Rodriguez
2016-02-10 5:21 ` Luis R. Rodriguez
2016-02-10 9:38 ` Ian Campbell
2016-02-10 5:44 ` Luis R. Rodriguez
2016-02-10 9:47 ` Ian Campbell
2016-02-11 8:08 ` Olaf Hering
2016-02-10 0:24 ` Dario Faggioli
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1448041672-3986-1-git-send-email-mcgrof@do-not-panic.com \
--to=mcgrof@do-not-panic.com \
--cc=JBeulich@suse.com \
--cc=cfergeau@redhat.com \
--cc=ian.campbell@citrix.com \
--cc=mcgrof@suse.com \
--cc=pryorm09@gmail.com \
--cc=samuel.thibault@ens-lyon.org \
--cc=xen-devel@lists.xensource.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).