Re: [Qemu-devel] Re: [PATCH] pc: e820 qemu_cfg tables need to be packed

2010-10-14 Thread Alex Williamson
On Thu, 2010-10-14 at 23:19 +0200, Arnd Bergmann wrote: > On Thursday 14 October 2010 22:59:04 Alex Williamson wrote: > > The structs in question only contain 4 & 8 byte elements, so there > > shouldn't be any change on x86-32 using one-byte aligned packing. > > I'm talking about the alignment of

Re: [Qemu-devel] Re: [PATCH] pc: e820 qemu_cfg tables need to be packed

2010-10-14 Thread Arnd Bergmann
On Thursday 14 October 2010 22:59:04 Alex Williamson wrote: > The structs in question only contain 4 & 8 byte elements, so there > shouldn't be any change on x86-32 using one-byte aligned packing. I'm talking about the alignment of the structure, not the members within the structure. The data stru

Re: [Qemu-devel] Re: [PATCH] pc: e820 qemu_cfg tables need to be packed

2010-10-14 Thread Alex Williamson
On Thu, 2010-10-14 at 22:20 +0200, Arnd Bergmann wrote: > On Thursday 14 October 2010 21:58:08 Alex Williamson wrote: > > If it works anywhere (I assume it works on 32bit), then it's only > > because it happened to get the alignment right. This just makes 64bit > > hosts get it right too. I don't

Re: [Qemu-devel] Re: [PATCH] pc: e820 qemu_cfg tables need to be packed

2010-10-14 Thread Arnd Bergmann
On Thursday 14 October 2010 21:58:08 Alex Williamson wrote: > If it works anywhere (I assume it works on 32bit), then it's only > because it happened to get the alignment right. This just makes 64bit > hosts get it right too. I don't see any compatibility issues, > non-packed + 64bit = broken. T

Re: [Qemu-devel] Re: [PATCH] pc: e820 qemu_cfg tables need to be packed

2010-10-14 Thread Anthony Liguori
On 10/14/2010 02:58 PM, Alex Williamson wrote: On Thu, 2010-10-14 at 14:48 -0500, Anthony Liguori wrote: On 10/14/2010 02:44 PM, Jes Sorensen wrote: On 10/14/10 20:33, Alex Williamson wrote: We can't let the compiler define the alignment for qemu_cfg data. Signed-off-by: Al

Re: [Qemu-devel] Re: [PATCH] pc: e820 qemu_cfg tables need to be packed

2010-10-14 Thread Alex Williamson
On Thu, 2010-10-14 at 14:48 -0500, Anthony Liguori wrote: > On 10/14/2010 02:44 PM, Jes Sorensen wrote: > > On 10/14/10 20:33, Alex Williamson wrote: > > > >> We can't let the compiler define the alignment for qemu_cfg data. > >> > >> Signed-off-by: Alex Williamson > >> --- > >> > >> 0.13 stabl

Re: [Qemu-devel] Re: [PATCH] pc: e820 qemu_cfg tables need to be packed

2010-10-14 Thread Anthony Liguori
On 10/14/2010 02:44 PM, Jes Sorensen wrote: On 10/14/10 20:33, Alex Williamson wrote: We can't let the compiler define the alignment for qemu_cfg data. Signed-off-by: Alex Williamson --- 0.13 stable candidate? ACK I would say so. fw_cfg interfaces are somewhat difficult to rat

[Qemu-devel] Re: [PATCH] pc: e820 qemu_cfg tables need to be packed

2010-10-14 Thread Jes Sorensen
On 10/14/10 20:33, Alex Williamson wrote: > We can't let the compiler define the alignment for qemu_cfg data. > > Signed-off-by: Alex Williamson > --- > > 0.13 stable candidate? ACK I would say so. Jes