From: pedro <marzo.pe...@gmail.com>
Date: Thu, 26 Feb 2015 09:25:41 +0100

> From: pmarzo <marzo.pe...@gmail.com>
> 
> offset and size are of type uint16_t so the %lu gives a warning
> A %u specifier, the same used in size makes gcc happy
> Not sure if a %x would be more correct
> 
> Signed-off-by: Pedro Marzo Perez <marzo.pe...@gmail.com>

This patch actually adds a warning on my machine, and your analysis
of the types is therefore probably incorrect:

drivers/net/xen-netback/netback.c: In function ‘xenvif_tx_build_gops’:
drivers/net/xen-netback/netback.c:1259:8: warning: format ‘%u’ expects argument 
of type ‘unsigned int’, but argument 5 has type ‘long unsigned int’ [-Wformat=]

The issue is probably "~PAGE_MASK" and I think the type of that
propagates into the type of the overall calculation.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to