> -----Original Message----- > From: Andrew Cooper [mailto:andrew.coop...@citrix.com] > Sent: Thursday, February 18, 2016 3:52 AM > To: Xen-devel <xen-devel@lists.xen.org> > Cc: Andrew Cooper <andrew.coop...@citrix.com>; Jan Beulich > <jbeul...@suse.com>; Tim Deegan <t...@xen.org>; Ian Campbell > <ian.campb...@citrix.com>; Tian, Kevin <kevin.t...@intel.com>; Wu, Feng > <feng...@intel.com> > Subject: [PATCH] xen: Avoid left shifting into a sign bit > > Clang 3.8 notices, and objects because it is undefined behaviour. > > "error: shifting a negative signed value is undefined > [-Werror,-Wshift-negative- > value]" > > Use unsigned constants rather than signed ones. > > Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com> > --- > CC: Jan Beulich <jbeul...@suse.com> > CC: Tim Deegan <t...@xen.org> > CC: Ian Campbell <ian.campb...@citrix.com> > CC: Kevin Tian <kevin.t...@intel.com> > CC: Feng Wu <feng...@intel.com> > --- > xen/common/page_alloc.c | 2 +- > xen/drivers/passthrough/vtd/x86/ats.c | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-)
Acked-by: Feng Wu <feng...@intel.com> for the VT-d part. Thanks, Feng _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
