Re: [Qemu-devel] [PATCH] spapr-pci: change endianness for io ports space

2013-07-15 Thread Alexey Kardashevskiy
On 07/16/2013 12:05 PM, Anthony Liguori wrote: > On Mon, Jul 15, 2013 at 7:39 PM, Alexey Kardashevskiy wrote: >> On 07/16/2013 10:28 AM, Anthony Liguori wrote: >>> Alexey Kardashevskiy writes: >>> >>> The right solution is to drop the additional IO region on sPAPR and >>> remove the ISA devices m

Re: [Qemu-devel] [PATCH] spapr-pci: change endianness for io ports space

2013-07-15 Thread Anthony Liguori
On Mon, Jul 15, 2013 at 7:39 PM, Alexey Kardashevskiy wrote: > On 07/16/2013 10:28 AM, Anthony Liguori wrote: >> Alexey Kardashevskiy writes: >> >> The right solution is to drop the additional IO region on sPAPR and >> remove the ISA devices marking themselves as "little endian". > > > No, this i

Re: [Qemu-devel] [PATCH] spapr-pci: change endianness for io ports space

2013-07-15 Thread Alexey Kardashevskiy
On 07/16/2013 10:28 AM, Anthony Liguori wrote: > Alexey Kardashevskiy writes: > >> On 07/16/2013 01:02 AM, Anthony Liguori wrote: >>> Alexey Kardashevskiy writes: >>> On 07/13/2013 06:03 PM, David Gibson wrote: > On Fri, Jul 12, 2013 at 05:37:19PM +1000, Alexey Kardashevskiy wrote:

Re: [Qemu-devel] [PATCH] spapr-pci: change endianness for io ports space

2013-07-15 Thread Anthony Liguori
Alexey Kardashevskiy writes: > On 07/16/2013 01:02 AM, Anthony Liguori wrote: >> Alexey Kardashevskiy writes: >> >>> On 07/13/2013 06:03 PM, David Gibson wrote: On Fri, Jul 12, 2013 at 05:37:19PM +1000, Alexey Kardashevskiy wrote: > sPAPR PHB emulates IO ports on PCI via a special memo

Re: [Qemu-devel] [PATCH] spapr-pci: change endianness for io ports space

2013-07-15 Thread Alexey Kardashevskiy
On 07/16/2013 01:02 AM, Anthony Liguori wrote: > Alexey Kardashevskiy writes: > >> On 07/13/2013 06:03 PM, David Gibson wrote: >>> On Fri, Jul 12, 2013 at 05:37:19PM +1000, Alexey Kardashevskiy wrote: sPAPR PHB emulates IO ports on PCI via a special memory region which routes all reads/

Re: [Qemu-devel] [PATCH] spapr-pci: change endianness for io ports space

2013-07-15 Thread Anthony Liguori
Alexey Kardashevskiy writes: > On 07/13/2013 06:03 PM, David Gibson wrote: >> On Fri, Jul 12, 2013 at 05:37:19PM +1000, Alexey Kardashevskiy wrote: >>> sPAPR PHB emulates IO ports on PCI via a special memory region which >>> routes all reads/writes further via cpu_in*/cpu_out* which are eventuall

Re: [Qemu-devel] [PATCH] spapr-pci: change endianness for io ports space

2013-07-14 Thread Alexey Kardashevskiy
On 07/13/2013 06:03 PM, David Gibson wrote: > On Fri, Jul 12, 2013 at 05:37:19PM +1000, Alexey Kardashevskiy wrote: >> sPAPR PHB emulates IO ports on PCI via a special memory region which >> routes all reads/writes further via cpu_in*/cpu_out* which are eventually >> processed by MemoryRegionOps im

Re: [Qemu-devel] [PATCH] spapr-pci: change endianness for io ports space

2013-07-13 Thread David Gibson
On Fri, Jul 12, 2013 at 05:37:19PM +1000, Alexey Kardashevskiy wrote: > sPAPR PHB emulates IO ports on PCI via a special memory region which > routes all reads/writes further via cpu_in*/cpu_out* which are eventually > processed by MemoryRegionOps implemented by devices. Hrm. That double dispatch

[Qemu-devel] [PATCH] spapr-pci: change endianness for io ports space

2013-07-12 Thread Alexey Kardashevskiy
sPAPR PHB emulates IO ports on PCI via a special memory region which routes all reads/writes further via cpu_in*/cpu_out* which are eventually processed by MemoryRegionOps implemented by devices. As devices normally take care of endianness themselves by setting correct MemoryRegionOps::endianness