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

2014-04-14 Thread Alexander Graf
On 10.02.14 06:12, Alexey Kardashevskiy wrote: On 02/08/2014 01:06 AM, Alexander Graf wrote: On 07.02.2014, at 14:44, Greg Kurz gk...@linux.vnet.ibm.com wrote: In the past, IO space could not be mapped into the memory address space so we introduced a workaround for that. Nowadays it does not

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

2014-02-09 Thread Alexey Kardashevskiy
On 02/08/2014 01:06 AM, Alexander Graf wrote: On 07.02.2014, at 14:44, Greg Kurz gk...@linux.vnet.ibm.com wrote: In the past, IO space could not be mapped into the memory address space so we introduced a workaround for that. Nowadays it does not look necessary so we can remove the

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

2014-02-07 Thread Greg Kurz
In the past, IO space could not be mapped into the memory address space so we introduced a workaround for that. Nowadays it does not look necessary so we can remove the workaround and make sPAPR PCI configuration simplier. Signed-off-by: Greg Kurz gk...@linux.vnet.ibm.com --- There has been a

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

2014-02-07 Thread Alexander Graf
On 07.02.2014, at 14:44, Greg Kurz gk...@linux.vnet.ibm.com wrote: In the past, IO space could not be mapped into the memory address space so we introduced a workaround for that. Nowadays it does not look necessary so we can remove the workaround and make sPAPR PCI configuration simplier.

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

2014-01-06 Thread Greg Kurz
On Fri, 03 Jan 2014 09:08:21 +1100 Alexey Kardashevskiy a...@ozlabs.ru wrote: Please read the rest of this thread. It does not visibly break things but with this patch QEMU starts calling unassigned_mem_accepts() (normally silent) which is not a good sign. Hmm... this is only because

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

2014-01-06 Thread Alexey Kardashevskiy
On 01/06/2014 10:12 PM, Greg Kurz wrote: On Fri, 03 Jan 2014 09:08:21 +1100 Alexey Kardashevskiy a...@ozlabs.ru wrote: Please read the rest of this thread. It does not visibly break things but with this patch QEMU starts calling unassigned_mem_accepts() (normally silent) which is not a good

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

2014-01-02 Thread Alexander Graf
On 11.12.2013, at 07:47, Alexey Kardashevskiy a...@ozlabs.ru wrote: On 12/10/2013 06:47 PM, Greg Kurz wrote: On Tue, 10 Dec 2013 13:43:05 +1100 Alexey Kardashevskiy a...@ozlabs.ru wrote: On 12/10/2013 03:33 AM, Greg Kurz wrote: In the past, IO space could not be mapped into the memory

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

2014-01-02 Thread Alexey Kardashevskiy
On 01/03/2014 08:04 AM, Alexander Graf wrote: On 11.12.2013, at 07:47, Alexey Kardashevskiy a...@ozlabs.ru wrote: On 12/10/2013 06:47 PM, Greg Kurz wrote: On Tue, 10 Dec 2013 13:43:05 +1100 Alexey Kardashevskiy a...@ozlabs.ru wrote: On 12/10/2013 03:33 AM, Greg Kurz wrote: In the past,

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

2014-01-02 Thread Alexander Graf
On 02.01.2014, at 23:08, Alexey Kardashevskiy a...@ozlabs.ru wrote: On 01/03/2014 08:04 AM, Alexander Graf wrote: On 11.12.2013, at 07:47, Alexey Kardashevskiy a...@ozlabs.ru wrote: On 12/10/2013 06:47 PM, Greg Kurz wrote: On Tue, 10 Dec 2013 13:43:05 +1100 Alexey Kardashevskiy

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

2014-01-02 Thread Alexey Kardashevskiy
On 03.01.2014 9:09, Alexander Graf wrote: On 02.01.2014, at 23:08, Alexey Kardashevskiy a...@ozlabs.ru wrote: On 01/03/2014 08:04 AM, Alexander Graf wrote: On 11.12.2013, at 07:47, Alexey Kardashevskiy a...@ozlabs.ru wrote: On 12/10/2013 06:47 PM, Greg Kurz wrote: On Tue, 10 Dec 2013

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

2013-12-17 Thread Alexey Kardashevskiy
On 12/17/2013 06:52 PM, Greg Kurz wrote: On Wed, 11 Dec 2013 18:07:58 +1100 Alexey Kardashevskiy a...@ozlabs.ru wrote: Hm. Nack. This fails: ./qemu-system-ppc64 \ -trace events=qemu_trace_events \ -L qemu-ppc64-bios/ \ -nographic \ -vga none \ -device \

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

2013-12-16 Thread Greg Kurz
On Wed, 11 Dec 2013 18:07:58 +1100 Alexey Kardashevskiy a...@ozlabs.ru wrote: Hm. Nack. This fails: ./qemu-system-ppc64 \ -trace events=qemu_trace_events \ -L qemu-ppc64-bios/ \ -nographic \ -vga none \ -device \

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

2013-12-10 Thread Alexey Kardashevskiy
On 12/10/2013 06:47 PM, Greg Kurz wrote: On Tue, 10 Dec 2013 13:43:05 +1100 Alexey Kardashevskiy a...@ozlabs.ru wrote: On 12/10/2013 03:33 AM, Greg Kurz wrote: In the past, IO space could not be mapped into the memory address space so we introduced a workaround for that. Nowadays it does not

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

2013-12-10 Thread Alexey Kardashevskiy
On 12/11/2013 05:47 PM, Alexey Kardashevskiy wrote: On 12/10/2013 06:47 PM, Greg Kurz wrote: On Tue, 10 Dec 2013 13:43:05 +1100 Alexey Kardashevskiy a...@ozlabs.ru wrote: On 12/10/2013 03:33 AM, Greg Kurz wrote: In the past, IO space could not be mapped into the memory address space so we

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

2013-12-09 Thread Greg Kurz
In the past, IO space could not be mapped into the memory address space so we introduced a workaround for that. Nowadays it does not look necessary so we can remove the workaround and make sPAPR PCI configuration simplier. This workaround has also an evil side effect with virtio devices: because

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

2013-12-09 Thread Alexey Kardashevskiy
On 12/10/2013 03:33 AM, Greg Kurz wrote: In the past, IO space could not be mapped into the memory address space so we introduced a workaround for that. Nowadays it does not look necessary so we can remove the workaround and make sPAPR PCI configuration simplier. This workaround has also an

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

2013-12-09 Thread Greg Kurz
On Tue, 10 Dec 2013 13:43:05 +1100 Alexey Kardashevskiy a...@ozlabs.ru wrote: On 12/10/2013 03:33 AM, Greg Kurz wrote: In the past, IO space could not be mapped into the memory address space so we introduced a workaround for that. Nowadays it does not look necessary so we can remove the

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

2013-07-16 Thread Paolo Bonzini
Il 16/07/2013 07:19, Alexey Kardashevskiy ha scritto: In the past, IO space could not be mapped into the memory address space so we introduced a workaround for that. Nowadays it does not look necessary so we can remove the workaround and make sPAPR PCI configuration simplier. This also

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

2013-07-16 Thread Alexander Graf
Am 16.07.2013 um 08:20 schrieb Paolo Bonzini pbonz...@redhat.com: Il 16/07/2013 07:19, Alexey Kardashevskiy ha scritto: In the past, IO space could not be mapped into the memory address space so we introduced a workaround for that. Nowadays it does not look necessary so we can remove the

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

2013-07-16 Thread Alexey Kardashevskiy
On 07/16/2013 06:32 PM, Alexander Graf wrote: Am 16.07.2013 um 08:20 schrieb Paolo Bonzini pbonz...@redhat.com: Il 16/07/2013 07:19, Alexey Kardashevskiy ha scritto: In the past, IO space could not be mapped into the memory address space so we introduced a workaround for that. Nowadays

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

2013-07-16 Thread Alexander Graf
Am 16.07.2013 um 10:37 schrieb Alexey Kardashevskiy a...@ozlabs.ru: On 07/16/2013 06:32 PM, Alexander Graf wrote: Am 16.07.2013 um 08:20 schrieb Paolo Bonzini pbonz...@redhat.com: Il 16/07/2013 07:19, Alexey Kardashevskiy ha scritto: In the past, IO space could not be mapped into the

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

2013-07-16 Thread Benjamin Herrenschmidt
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

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

2013-07-16 Thread Alexander Graf
Am 16.07.2013 um 10:45 schrieb Benjamin Herrenschmidt b...@kernel.crashing.org: 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

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

2013-07-16 Thread Alexey Kardashevskiy
On 07/16/2013 07:01 PM, Alexander Graf wrote: Am 16.07.2013 um 10:45 schrieb Benjamin Herrenschmidt b...@kernel.crashing.org: 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

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

2013-07-15 Thread Alexander Graf
On 15.07.2013, at 05:24, Alexey Kardashevskiy wrote: As it looks like all portio users have migrated to new portio api, the workaround with memory access to io ports routing is no more needed. This also fixes a bug with byte swapping as the io region was marked as little endian while it

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

2013-07-15 Thread Alexey Kardashevskiy
On 07/15/2013 04:06 PM, Alexander Graf wrote: On 15.07.2013, at 05:24, Alexey Kardashevskiy wrote: As it looks like all portio users have migrated to new portio api, the workaround with memory access to io ports routing is no more needed. This also fixes a bug with byte swapping as the

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

2013-07-15 Thread Alexey Kardashevskiy
In the past, IO space could not be mapped into the memory address space so we introduced a workaround for that. Nowadays it does not look necessary so we can remove the workaround and make sPAPR PCI configuration simplier. This also removes all byte swappings as it is not PHB's to take care of

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

2013-07-14 Thread Alexey Kardashevskiy
As it looks like all portio users have migrated to new portio api, the workaround with memory access to io ports routing is no more needed. This also fixes a bug with byte swapping as the io region was marked as little endian while it should not do any swapping at all. Signed-off-by: Alexey