[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] NBD cleanup
From: |
Carlo Marcelo Arenas Belon |
Subject: |
Re: [Qemu-devel] [PATCH] NBD cleanup |
Date: |
Wed, 18 Jun 2008 02:08:37 -0500 |
User-agent: |
Mutt/1.4.1i |
On Thu, Jun 12, 2008 at 12:09:02PM -0500, Anthony Liguori wrote:
> Laurent Vivier wrote:
> >This patch removes include of malloc.h from qemu-nbd.c to make it
> >compile on OS X. (as reported by C.W. Betts and commented by Daniel P.
> >Berrange)
> >It adjusts a printf format in nbd.c.
this last snippet avoids the following warning on amd64 linux (and other LP64
platforms) :
nbd.c: In function `nbd_trip':
nbd.c:391: warning: unsigned int format, different type arg (arg 7)
but will trigger something similar in x86 linux (and other ILP32 platforms)
where sizeof returns unsigned instead.
to prevent this line to keep getting changed back and forth (as shown by r4676
and the two currently proposed patches that touch nbd) to avoid this warning;
casting it to unsigned long as suggested in the attached patch (to be applied
instead of this one) should be enough, if probably not that elegant.
> Reviewed-by: Anthony Liguori <address@hidden>
Reviewed-by: Carlo Marcelo Arenas Belon <address@hidden>
Carlo
qemu-nbd-cleanup.patch
Description: Text document