[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [Qemu-devel] [PATCH v2 4/6] prep: move int-ack register f
From: |
Andreas Färber |
Subject: |
Re: [Qemu-ppc] [Qemu-devel] [PATCH v2 4/6] prep: move int-ack register from PReP to Raven PCI emulation |
Date: |
Sun, 29 Apr 2012 22:12:47 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:11.0) Gecko/20120328 Thunderbird/11.0.1 |
Am 29.04.2012 10:32, schrieb Avi Kivity:
> On 04/28/2012 09:46 PM, Andreas Färber wrote:
>> Am 14.04.2012 22:48, schrieb Hervé Poussineau:
>>> Register is one byte-wide (as per specification), so there is no need to
>>> specify endianness.
>>
>> The region was 4 bytes before, now it's 1. What happens when a 4-byte
>> read is attempted at that address? Do we need to specify the valid
>> widths for the MemoryRegion? Or is such a read constructed from this
>> region and (assuming) the return value of an unassigned read?
>
> This area of what happens during access that falls across region
> boundaries is very underspecified in qemu; nor is it clear what happens
> in real hardware (in all its variations).
So, what's your conclusion here? Should we add .valid.max_access_size =
1? Or shall I apply as is?
Andreas
[Qemu-ppc] [PATCH v2 5/6] pcspk: initialize PC speaker if compiled in, Hervé Poussineau, 2012/04/14
[Qemu-ppc] [PATCH v2 6/6] prep: initialize pc speaker, Hervé Poussineau, 2012/04/14
Re: [Qemu-ppc] [Qemu-devel] [PATCH v2 0/6] prep: some fixes and Super I/O emulation, Artyom Tarasenko, 2012/04/15
Re: [Qemu-ppc] [PATCH v2 0/6] prep: some fixes and Super I/O emulation, Hervé Poussineau, 2012/04/28