Re: [Qemu-devel] [PATCH v2 3/3] WHPX: fix some compiler warnings

2018-05-16 Thread Lucian Petrut
Oh, right, got it. Sorry for the confusion, I’ll take that out of the patch. Thanks, Lucian Petrut From: Paolo Bonzini Sent: Wednesday, May 16, 2018 1:42 PM To: Lucian Petrut; petrutlucia...@gmail.com

Re: [Qemu-devel] [PATCH v2 3/3] WHPX: fix some compiler warnings

2018-05-16 Thread Paolo Bonzini
On 16/05/2018 12:21, Lucian Petrut wrote: > Hi, > > Thanks for reviewing those changes. > > /“can you send a patch with the bugfix only”/ > > By “the bugfix only” you mean the previous commit, the one that > dynamically loads the DLLs? > > FWIW, without this patch, i386 targets won’t compile, s

Re: [Qemu-devel] [PATCH v2 3/3] WHPX: fix some compiler warnings

2018-05-16 Thread Lucian Petrut
Hi, Thanks for reviewing those changes. “can you send a patch with the bugfix only” By “the bugfix only” you mean the previous commit, the one that dynamically loads the DLLs? FWIW, without this patch, i386 targets won’t compile, so I’d consider that an improvement. Probably I should’ve explic

Re: [Qemu-devel] [PATCH v2 3/3] WHPX: fix some compiler warnings

2018-05-16 Thread Paolo Bonzini
On 15/05/2018 19:35, petrutlucia...@gmail.com wrote: > From: Lucian Petrut > > This patch fixes a few compiler warnings, especially in case of > x86 targets, where the number of registers was not properly handled > and could cause an overflow. > > Signed-off-by: Alessandro Pilotti > Signed-off-

[Qemu-devel] [PATCH v2 3/3] WHPX: fix some compiler warnings

2018-05-15 Thread petrutlucian94
From: Lucian Petrut This patch fixes a few compiler warnings, especially in case of x86 targets, where the number of registers was not properly handled and could cause an overflow. Signed-off-by: Alessandro Pilotti Signed-off-by: Justin Terry (VM) Signed-off-by: Lucian Petrut --- target/i386