Xen devel - Jun 2009 - [PATCH] Fix variadic formatting in blktap2 breaking xen-unstable building on x86_32

Hi,

Building the tip of xen-unstable breaks, at least on x86_32 due to
warnings being treated as errors with gcc 4.1

The problem is variadic formatting, uint64_t is unsigned long long, yet
in several places in blktap2 it was formatted as unsigned long.

The attached patch fixes it.

I''m not sure where this was introduced, I just pulled the xen-unstable
tip 15 minutes ago and noticed it.

Cheers,
--Tim



_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

On Saturday, 27 June 2009 at 01:19, Tim Post wrote:
> Hi,
> 
> Building the tip of xen-unstable breaks, at least on x86_32 due to
> warnings being treated as errors with gcc 4.1
> 
> The problem is variadic formatting, uint64_t is unsigned long long, yet
> in several places in blktap2 it was formatted as unsigned long.
> 
> The attached patch fixes it.
> 
> I''m not sure where this was introduced, I just pulled the
xen-unstable
> tip 15 minutes ago and noticed it.
> 
> diff -r 442fb5f1e0a2 tools/blktap2/drivers/block-ram.c
> +++ b/tools/blktap2/drivers/block-ram.c	Sat Jun 27 01:12:22 2009 +0800
> @@ -101,7 +101,7 @@ static int get_image_info(int fd, td_dis
>  	disksector_size = info->sector_size;
>  	disksize        = info->size;
>  	diskinfo        = info->info;
> -	DPRINTF("Image sector_size: \n\t[%lu]\n",
> +	DPRINTF("Image sector_size: \n\t[%llu]\n",
>  		info->sector_size);
This just ping-pongs between breaking 32-bit and 64-bit. I think you
want to use PRIu64 in all these cases.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

On Fri, 2009-06-26 at 11:24 -0700, Brendan Cully wrote:
> This just ping-pongs between breaking 32-bit and 64-bit. I think you
> want to use PRIu64 in all these cases.
Yes, sorry for sending without really looking, was leaving my office at
the time. Several macros also need fixed, I''ll re-send the patch
tomorrow morning (its 2:39 AM my time now) .. unless someone else does
it first.

Cheers,
--Tim




_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

Hi Tim,

On Sat, Jun 27, 2009 at 02:40:48AM +0800, Tim Post
wrote:
> On Fri, 2009-06-26 at 11:24 -0700, Brendan Cully wrote:
> 
> > This just ping-pongs between breaking 32-bit and 64-bit. I think you
> > want to use PRIu64 in all these cases.
> 
> Yes, sorry for sending without really looking, was leaving my office at
> the time. Several macros also need fixed, I''ll re-send the patch
> tomorrow morning (its 2:39 AM my time now) .. unless someone else does
> it first.
I believe that this was filed by c/s 19826 

cheers,
Ryan
-- 
 Ryan O''Connor <rjo@cs.ubc.ca>

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel