[Qemu-devel] Re: [PATCH] pc: Fix e820 fw_cfg for big endian

2010-11-09 Thread Jes Sorensen
On 11/08/10 04:57, Alex Williamson wrote:
 Signed-off-by: Alex Williamson alex.william...@redhat.com
 ---
 
  Compile tested only.  Only current user is kvm, no cross-arch users.
 
  hw/pc.c |   14 +++---
  1 files changed, 7 insertions(+), 7 deletions(-)

Patch looks fine to me, but are there any systems out there using e820
on big endian hardware?

Jes



Re: [Qemu-devel] Re: [PATCH] pc: Fix e820 fw_cfg for big endian

2010-11-09 Thread Jes Sorensen
On 11/09/10 13:42, Alexander Graf wrote:
 
 On 09.11.2010, at 11:57, Jes Sorensen wrote:
 
 On 11/08/10 04:57, Alex Williamson wrote:
 Signed-off-by: Alex Williamson alex.william...@redhat.com
 ---

 Compile tested only.  Only current user is kvm, no cross-arch users.

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

 Patch looks fine to me, but are there any systems out there using e820
 on big endian hardware?
 
 This fixes things when host endianness is big endian.

Ah right, then it's all fine :)

Cheers,
Jes




Re: [Qemu-devel] Re: [PATCH] pc: Fix e820 fw_cfg for big endian

2010-11-09 Thread Alexander Graf

On 09.11.2010, at 11:57, Jes Sorensen wrote:

 On 11/08/10 04:57, Alex Williamson wrote:
 Signed-off-by: Alex Williamson alex.william...@redhat.com
 ---
 
 Compile tested only.  Only current user is kvm, no cross-arch users.
 
 hw/pc.c |   14 +++---
 1 files changed, 7 insertions(+), 7 deletions(-)
 
 Patch looks fine to me, but are there any systems out there using e820
 on big endian hardware?

This fixes things when host endianness is big endian.


Alex