Re: [Qemu-devel] [PATCH 2/2] tests: Make acpid test compile

2017-08-30 Thread Juan Quintela
"Daniel P. Berrange" wrote: > On Tue, Aug 29, 2017 at 03:17:25PM -0500, Eric Blake wrote: >> On 08/28/2017 09:41 AM, Cédric Le Goater wrote: >> > On 08/23/2017 01:53 PM, Dr. David Alan Gilbert wrote: >> >> * Juan Quintela (quint...@redhat.com) wrote: >> >>> Compiler gets confused with the size of

Re: [Qemu-devel] [PATCH 2/2] tests: Make acpid test compile

2017-08-30 Thread Daniel P. Berrange
On Wed, Aug 30, 2017 at 12:51:23PM +0200, Juan Quintela wrote: > Cedric Le Goater wrote: > > On 08/23/2017 01:53 PM, Dr. David Alan Gilbert wrote: > >> * Juan Quintela (quint...@redhat.com) wrote: > >>> Compiler gets confused with the size of the struct, so move form > >>> g_new0() to g_malloc0().

Re: [Qemu-devel] [PATCH 2/2] tests: Make acpid test compile

2017-08-30 Thread Juan Quintela
Cedric Le Goater wrote: > On 08/23/2017 01:53 PM, Dr. David Alan Gilbert wrote: >> * Juan Quintela (quint...@redhat.com) wrote: >>> Compiler gets confused with the size of the struct, so move form >>> g_new0() to g_malloc0(). >>> >>> I *think* that the problem is in gcc (or glib for that matter),

Re: [Qemu-devel] [PATCH 2/2] tests: Make acpid test compile

2017-08-30 Thread Daniel P. Berrange
On Tue, Aug 29, 2017 at 03:17:25PM -0500, Eric Blake wrote: > On 08/28/2017 09:41 AM, Cédric Le Goater wrote: > > On 08/23/2017 01:53 PM, Dr. David Alan Gilbert wrote: > >> * Juan Quintela (quint...@redhat.com) wrote: > >>> Compiler gets confused with the size of the struct, so move form > >>> g_ne

Re: [Qemu-devel] [PATCH 2/2] tests: Make acpid test compile

2017-08-29 Thread Eric Blake
On 08/28/2017 09:41 AM, Cédric Le Goater wrote: > On 08/23/2017 01:53 PM, Dr. David Alan Gilbert wrote: >> * Juan Quintela (quint...@redhat.com) wrote: >>> Compiler gets confused with the size of the struct, so move form >>> g_new0() to g_malloc0(). >>> >>> I *think* that the problem is in gcc (or

Re: [Qemu-devel] [PATCH 2/2] tests: Make acpid test compile

2017-08-28 Thread Cédric Le Goater
On 08/23/2017 01:53 PM, Dr. David Alan Gilbert wrote: > * Juan Quintela (quint...@redhat.com) wrote: >> Compiler gets confused with the size of the struct, so move form >> g_new0() to g_malloc0(). >> >> I *think* that the problem is in gcc (or glib for that matter), but >> the documentation of the

Re: [Qemu-devel] [PATCH 2/2] tests: Make acpid test compile

2017-08-23 Thread Dr. David Alan Gilbert
* Juan Quintela (quint...@redhat.com) wrote: > Compiler gets confused with the size of the struct, so move form > g_new0() to g_malloc0(). > > I *think* that the problem is in gcc (or glib for that matter), but > the documentation of the g_new0 states that 1sts first argument is an > struct type,

[Qemu-devel] [PATCH 2/2] tests: Make acpid test compile

2017-08-23 Thread Juan Quintela
Compiler gets confused with the size of the struct, so move form g_new0() to g_malloc0(). I *think* that the problem is in gcc (or glib for that matter), but the documentation of the g_new0 states that 1sts first argument is an struct type, and uint32_t is not an struct type. Signed-off-by: Juan