Re: [Qemu-devel] [PATCH] virtio-balloon: don't hardcode config size value

2014-01-15 Thread Luiz Capitulino
On Wed, 15 Jan 2014 20:10:31 +0400 Michael Tokarev wrote: > 09.01.2014 18:58, Luiz Capitulino wrote: > > Use sizeof(strucy virtio_balloon_config) instead. > > Thanks, applied to the trivial patches queue (with the spelling fix). Thanks Michael.

Re: [Qemu-devel] [PATCH] virtio-balloon: don't hardcode config size value

2014-01-15 Thread Michael Tokarev
09.01.2014 18:58, Luiz Capitulino wrote: > Use sizeof(strucy virtio_balloon_config) instead. Thanks, applied to the trivial patches queue (with the spelling fix). /mjt

Re: [Qemu-devel] [PATCH] virtio-balloon: don't hardcode config size value

2014-01-14 Thread Michael S. Tsirkin
On Tue, Jan 14, 2014 at 09:05:31PM +0400, Michael Tokarev wrote: > 09.01.2014 18:58, Luiz Capitulino wrote: > > Use sizeof(strucy virtio_balloon_config) instead. > > > > --- a/hw/virtio/virtio-balloon.c > > +++ b/hw/virtio/virtio-balloon.c > > @@ -263,7 +263,7 @@ static void virtio_balloon_get_con

Re: [Qemu-devel] [PATCH] virtio-balloon: don't hardcode config size value

2014-01-14 Thread Luiz Capitulino
On Tue, 14 Jan 2014 21:05:31 +0400 Michael Tokarev wrote: > 09.01.2014 18:58, Luiz Capitulino wrote: > > Use sizeof(strucy virtio_balloon_config) instead. > > > > --- a/hw/virtio/virtio-balloon.c > > +++ b/hw/virtio/virtio-balloon.c > > @@ -263,7 +263,7 @@ static void virtio_balloon_get_config(V

Re: [Qemu-devel] [PATCH] virtio-balloon: don't hardcode config size value

2014-01-14 Thread Michael Tokarev
09.01.2014 18:58, Luiz Capitulino wrote: > Use sizeof(strucy virtio_balloon_config) instead. > > --- a/hw/virtio/virtio-balloon.c > +++ b/hw/virtio/virtio-balloon.c > @@ -263,7 +263,7 @@ static void virtio_balloon_get_config(VirtIODevice *vdev, > uint8_t *config_data) > config.num_pages = cp

Re: [Qemu-devel] [PATCH] virtio-balloon: don't hardcode config size value

2014-01-09 Thread Peter Crosthwaite
On Fri, Jan 10, 2014 at 12:58 AM, Luiz Capitulino wrote: > Use sizeof(strucy virtio_balloon_config) instead. > "struct". > Signed-off-by: Luiz capitulino Otherwise: Reviewed-by: Peter Crosthwaite > --- > > I have no idea which queue this should go through... > Trivial? Regards, Peter >