[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1.1 09/24] omap_gpmc/nseries/tusb6010: convert
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH v1.1 09/24] omap_gpmc/nseries/tusb6010: convert to memory API |
Date: |
Tue, 9 Aug 2011 10:41:24 +0100 |
On 9 August 2011 10:26, Avi Kivity <address@hidden> wrote:
> On 08/09/2011 12:23 PM, Peter Maydell wrote:
>>
>> On 9 August 2011 10:02, Avi Kivity<address@hidden> wrote:
>> > +static const MemoryRegionOps omap_gpmc_ops = {
>> > + .read = omap_gpmc_read,
>> > + .write = omap_gpmc_write,
>> > + .endianness = DEVICE_NATIVE_ENDIAN,
>> > + .valid = {
>> > + .min_access_size = 4,
>> > + .max_access_size = 4,
>> > + },
>> > };
>>
>> Does this give debug trace output for missized accesses?
>
> Only after we implement and configure a debug policy in memory.c
>
>> That's the main point of the badread/badwrite functions...
>
> These are broken now, aren't they?
They successfully print a message, which as I say is the
main point. The OMAP TRM typically says the behaviour isn't
defined for wrong-size accesses, so guest code doing it is
broken anyhow and so the error in the current implementation
is IMHO not so serious.
I think if (as here) it turns out that we need more functionality
in memory.c then we should be implementing it, not silently
dropping things in the conversion process.
-- PMM
- Re: [Qemu-devel] [PATCH 09/24] omap_gpmc/nseries/tusb6010: convert to memory API, (continued)
- Re: [Qemu-devel] [PATCH 09/24] omap_gpmc/nseries/tusb6010: convert to memory API, Peter Maydell, 2011/08/08
- Re: [Qemu-devel] [PATCH 09/24] omap_gpmc/nseries/tusb6010: convert to memory API, Avi Kivity, 2011/08/09
- Re: [Qemu-devel] [PATCH 09/24] omap_gpmc/nseries/tusb6010: convert to memory API, Peter Maydell, 2011/08/09
- Re: [Qemu-devel] [PATCH 09/24] omap_gpmc/nseries/tusb6010: convert to memory API, Avi Kivity, 2011/08/09
- Re: [Qemu-devel] [PATCH 09/24] omap_gpmc/nseries/tusb6010: convert to memory API, Peter Maydell, 2011/08/09
- Re: [Qemu-devel] [PATCH 09/24] omap_gpmc/nseries/tusb6010: convert to memory API, Avi Kivity, 2011/08/09
- Re: [Qemu-devel] [PATCH 09/24] omap_gpmc/nseries/tusb6010: convert to memory API, Avi Kivity, 2011/08/09
- [Qemu-devel] [PATCH v1.1 09/24] omap_gpmc/nseries/tusb6010: convert to memory API, Avi Kivity, 2011/08/09
- Re: [Qemu-devel] [PATCH v1.1 09/24] omap_gpmc/nseries/tusb6010: convert to memory API, Peter Maydell, 2011/08/09
- Re: [Qemu-devel] [PATCH v1.1 09/24] omap_gpmc/nseries/tusb6010: convert to memory API, Avi Kivity, 2011/08/09
- Re: [Qemu-devel] [PATCH v1.1 09/24] omap_gpmc/nseries/tusb6010: convert to memory API,
Peter Maydell <=
- [Qemu-devel] [PATCH v1.2 09/24] omap_gpmc/nseries/tusb6010: convert to memory API, Avi Kivity, 2011/08/09
- Re: [Qemu-devel] [PATCH v1.1 09/24] omap_gpmc/nseries/tusb6010: convert to memory API, Avi Kivity, 2011/08/09
[Qemu-devel] [PATCH 18/24] versatile_pci: convert to memory API, Avi Kivity, 2011/08/08
[Qemu-devel] [PATCH 22/24] pci: add pci_address_space(), Avi Kivity, 2011/08/08
[Qemu-devel] [PATCH 24/24] 440fx: fix PAM, PCI holes, Avi Kivity, 2011/08/08