[Qemu-devel] Re: [PATCH 3/8] sparc64: fix 32bit load sign extension

[Paolo Bonzini]

On 06/03/2010 09:59 PM, Igor Kovalenko wrote:
> On Thu, Jun 3, 2010 at 7:42 PM, Paolo Bonzini<address@hidden>  wrote:
> > On 06/03/2010 05:25 PM, Alexander Graf wrote:
> > > Am 03.06.2010 um 15:18 schrieb Paolo Bonzini<address@hidden>:
> > >
> > > > On 06/01/2010 10:12 PM, Igor V. Kovalenko wrote:
> > > > > From: Igor V. Kovalenko<address@hidden>
> > > > >
> > > > > - change return type of ldl_* to uint32_t to prevent unwanted sign
> > > > > extension
> > > > > visible in sparc64 load alternate address space methods
> > > > > - note this change makes ldl_* softmmu implementations match ldl_phys
> > > > > one
> > > >
> > > > This patch breaks -kernel/-initrd.
> > >
> > > Breaks it where and when?
> >
> > x86_64 TCG reboots after the "Probing EDD" step.
>
> My local build appears to work, qemu-system-x86_64 loads my gentoo linux setup.
> I use x86_64 host, gcc 4.4.3, qemu configured with ./configure
> --prefix=/inst --target-list=sparc64-softmmu,x86_64-softmmu

Normal boot works.  Only -kernel/-initrd fails.

Paolo