diff mbox

[v2] pc: e820 qemu_cfg tables need to be packed

Message ID 20101015040706.3828.71113.stgit@s20.home
State New
Headers show

Commit Message

Alex Williamson Oct. 15, 2010, 4:08 a.m. UTC 
We can't let the compiler define the alignment for qemu_cfg data.

Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
---

 v2: Adjust alignment to help non-x86 hosts per Arnd's suggestion

 0.13-stable candidate

 hw/pc.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

Comments

Arnd Bergmann Oct. 15, 2010, 10:20 a.m. UTC | #1 
On Friday 15 October 2010, Alex Williamson wrote:
> We can't let the compiler define the alignment for qemu_cfg data.
> 
> Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
> ---
> 
>  v2: Adjust alignment to help non-x86 hosts per Arnd's suggestion

Ok, looks good now. Thanks!

	Arnd
diff mbox

Patch

diff --git a/hw/pc.c b/hw/pc.c
index 69b13bf..0264e3d 100644
--- a/hw/pc.c
+++ b/hw/pc.c
@@ -75,12 +75,12 @@  struct e820_entry {
     uint64_t address;
     uint64_t length;
     uint32_t type;
-};
+} __attribute((__packed__, __aligned__(4)));
 
 struct e820_table {
     uint32_t count;
     struct e820_entry entry[E820_NR_ENTRIES];
-};
+} __attribute((__packed__, __aligned__(4)));
 
 static struct e820_table e820_table;