[Qemu-devel] ioport: qemu-system-sparc64 currently broken

[Qemu-devel] ioport: qemu-system-sparc64 currently broken

[Mark Cave-Ayland]

Hi Jan/Paolo,

I've just updated my local QEMU repository to git master for OpenBIOS 
testing and it seems that the ioport changes break SPARC64. git bisect 
points to this commit:


commit b40acf99bef69fa8ab0f9092ff162fde945eec12
Author: Jan Kiszka <jan.kiszka@siemens.com>
Date:   Mon Jun 24 10:45:09 2013 +0200

     ioport: Switch dispatching to memory core layer

     The current ioport dispatcher is a complex beast, mostly due to the
     need to deal with old portio interface users. But we can overcome it
     without converting all portio users by embedding the required base
     address of a MemoryRegionPortio access into that data structure. That
     removes the need to have the additional MemoryRegionIORange structure
     in the loop on every access.

     To handle old portio memory ops, we simply install dispatching handlers
     for portio memory regions when registering them with the memory core.
     This removes the need for the old_portio field.

     We can drop the additional aliasing of ioport regions and also the
     special address space listener. cpu_in and cpu_out now simply call
     address_space_read/write. And we can concentrate portio handling in a
     single source file.

     Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
     Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>


I suspect that there may be multiple breakages here (as HEAD blows up 
differently with a trap failure), but this is definitely the start of 
the chain.

Reproducing the bug is easy - simply run qemu-system-sparc64 without any 
parameters.


Many thanks,

Mark.

Re: [Qemu-devel] ioport: qemu-system-sparc64 currently broken

[Paolo Bonzini]

Il 27/07/2013 00:21, Mark Cave-Ayland ha scritto:
> I suspect that there may be multiple breakages here (as HEAD blows up
> differently with a trap failure), but this is definitely the start of
> the chain.

Yes, the fixes so far are the following:

commit 2b8fbcd87007a170a9778ec6645c06f543977067
commit 5519ad0c245860526e8d269e1cd20fe239447736
commit e11ef3d184c88a316f48ba2472dcb2937d9fa24b
commit 547e9201c52629eef71b0eb8cb48dacc50bcf759
commit e7342aa39b3193473ded24cbf2233930665796e9

but something else remains.

I'll try to debug it during the hard freeze period.

Paolo

Re: [Qemu-devel] ioport: qemu-system-sparc64 currently broken

[Mark Cave-Ayland]

On 27/07/13 09:55, Paolo Bonzini wrote:

> Il 27/07/2013 00:21, Mark Cave-Ayland ha scritto:
>> I suspect that there may be multiple breakages here (as HEAD blows up
>> differently with a trap failure), but this is definitely the start of
>> the chain.
>
> Yes, the fixes so far are the following:
>
> commit 2b8fbcd87007a170a9778ec6645c06f543977067
> commit 5519ad0c245860526e8d269e1cd20fe239447736
> commit e11ef3d184c88a316f48ba2472dcb2937d9fa24b
> commit 547e9201c52629eef71b0eb8cb48dacc50bcf759
> commit e7342aa39b3193473ded24cbf2233930665796e9
>
> but something else remains.
>
> I'll try to debug it during the hard freeze period.

Thanks Paolo - let me know if there's any further information you need.


ATB,

Mark.