Re: [Qemu-devel] [PATCH] device_tree: load_device_tree(): Allow NULL sizep

2012-08-20 Thread Stefan Hajnoczi
On Thu, Aug 16, 2012 at 3:14 AM, David Gibson wrote: > On Wed, Aug 15, 2012 at 02:41:56PM +0100, Stefan Hajnoczi wrote: >> On Sat, Aug 11, 2012 at 01:33:42PM +0100, Stefan Hajnoczi wrote: >> > On Sat, Aug 11, 2012 at 12:11 AM, Peter Crosthwaite >> > wrote: >> > > On Fri, Aug 10, 2012 at 11:42 PM,

Re: [Qemu-devel] [PATCH] device_tree: load_device_tree(): Allow NULL sizep

2012-08-15 Thread David Gibson
On Wed, Aug 15, 2012 at 02:41:56PM +0100, Stefan Hajnoczi wrote: > On Sat, Aug 11, 2012 at 01:33:42PM +0100, Stefan Hajnoczi wrote: > > On Sat, Aug 11, 2012 at 12:11 AM, Peter Crosthwaite > > wrote: > > > On Fri, Aug 10, 2012 at 11:42 PM, Stefan Hajnoczi > > > wrote: > > >> On Fri, Aug 10, 2012

Re: [Qemu-devel] [PATCH] device_tree: load_device_tree(): Allow NULL sizep

2012-08-15 Thread Stefan Hajnoczi
On Sat, Aug 11, 2012 at 01:33:42PM +0100, Stefan Hajnoczi wrote: > On Sat, Aug 11, 2012 at 12:11 AM, Peter Crosthwaite > wrote: > > On Fri, Aug 10, 2012 at 11:42 PM, Stefan Hajnoczi > > wrote: > >> On Fri, Aug 10, 2012 at 01:54:26PM +1000, Peter A. G. Crosthwaite wrote: > >>> The sizep arg is po

Re: [Qemu-devel] [PATCH] device_tree: load_device_tree(): Allow NULL sizep

2012-08-11 Thread Stefan Hajnoczi
On Sat, Aug 11, 2012 at 12:11 AM, Peter Crosthwaite wrote: > On Fri, Aug 10, 2012 at 11:42 PM, Stefan Hajnoczi wrote: >> On Fri, Aug 10, 2012 at 01:54:26PM +1000, Peter A. G. Crosthwaite wrote: >>> The sizep arg is populated with the size of the loaded device tree. Since >>> this >>> is one of t

Re: [Qemu-devel] [PATCH] device_tree: load_device_tree(): Allow NULL sizep

2012-08-10 Thread Peter Crosthwaite
On Fri, Aug 10, 2012 at 11:42 PM, Stefan Hajnoczi wrote: > On Fri, Aug 10, 2012 at 01:54:26PM +1000, Peter A. G. Crosthwaite wrote: >> The sizep arg is populated with the size of the loaded device tree. Since >> this >> is one of those informational "please populate" type arguments it should be >

Re: [Qemu-devel] [PATCH] device_tree: load_device_tree(): Allow NULL sizep

2012-08-10 Thread Stefan Hajnoczi
On Fri, Aug 10, 2012 at 01:54:26PM +1000, Peter A. G. Crosthwaite wrote: > The sizep arg is populated with the size of the loaded device tree. Since this > is one of those informational "please populate" type arguments it should be > optional. Guarded writes to *sizep against NULL accordingly. > >

[Qemu-devel] [PATCH] device_tree: load_device_tree(): Allow NULL sizep

2012-08-09 Thread Peter A. G. Crosthwaite
The sizep arg is populated with the size of the loaded device tree. Since this is one of those informational "please populate" type arguments it should be optional. Guarded writes to *sizep against NULL accordingly. Signed-off-by: Peter A. G. Crosthwaite Acked-by: Alexander Graf --- device_tree