[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-ppc] [PATCH] spapr-pci: remove io ports workaround

From: Alexey Kardashevskiy
Subject: Re: [Qemu-ppc] [PATCH] spapr-pci: remove io ports workaround
Date: Tue, 16 Jul 2013 19:10:25 +1000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130514 Thunderbird/17.0.6

On 07/16/2013 07:01 PM, Alexander Graf wrote:
> Am 16.07.2013 um 10:45 schrieb Benjamin Herrenschmidt
> <address@hidden>:
>> On Tue, 2013-07-16 at 10:32 +0200, Alexander Graf wrote:
>>>>> Did I miss anything here?
>>>> No, I don't think so.  The patch looks good.
>>> ... and will break all Mac targets again, no? Not to speak of
>>> non-ppc devices.
>> Do you know why it breaks ? Can you suggest what's wrong and how to
>> fix it maybe ?
> Sure. Get rid of cpu_inX and cpu_outX completely and verify that the
> conversion is doing the "right thing" for each conversion. Some of them
> are really tricky - like MIPS which accesses the RTC in native endian
> (big) in current code. So this patch would definitely break that one
> too.

> I'm actually surprised we have to mark the ioport region little endian
> for sPAPR.

It is not for spapr, it is for devices which expect some endianness and do
not do convertions.

> We already did have it converted to a direct memory api alias
> a while ago and IIRC the only problem back then were the old_portio
> callbacks. I wonder what changed in between.

? We have had the IO memory region hack I am trying to remove for a long time.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]