Re: [Qemu-devel] [PATCH] ioport: remove LITTLE_ENDIAN mark for portio

2013-07-12 Thread Alexander Graf
with. The dispatch layer really has no purpose in splitting I/O requests in the first place... Cc: Hervé Poussineau hpous...@reactos.org Cc: Alex Graf ag...@suse.de Cc: Benjamin Herrenschmidt b...@kernel.crashing.org Signed-off-by: Anthony Liguori aligu...@us.ibm.com Acked-by: Alexander Graf ag

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-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 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

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 devices

Re: [Qemu-devel] [Qemu-ppc] [PATCH 2/2] Add Enhanced Three-Speed Ethernet Controller (eTSEC)

2013-07-16 Thread Alexander Graf
On 07/16/2013 05:28 PM, Fabien Chouteau wrote: On 07/16/2013 04:06 AM, Scott Wood wrote: On 07/10/2013 12:10:02 PM, Fabien Chouteau wrote: This implementation doesn't include ring priority, TCP/IP Off-Load, QoS. Signed-off-by: Fabien Chouteauchout...@adacore.com From the code comments I

Re: [Qemu-devel] [PATCH] ppc kvm: fix to compile

2013-07-16 Thread Alexander Graf
On 16.07.2013, at 12:00, Alexey Kardashevskiy wrote: Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru Why? sPAPRTCE is a typedef struct that's defined in include/hw/ppc/spapr.h which is included from target-ppc/kvm.c. So it should work just fine. Alex --- target-ppc/kvm.c | 4 ++-- 1

Re: [Qemu-devel] [Qemu-ppc] [PATCH] ppc kvm: fix to compile

2013-07-16 Thread Alexander Graf
On 17.07.2013, at 01:12, Alexander Graf wrote: On 16.07.2013, at 12:00, Alexey Kardashevskiy wrote: Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru Why? sPAPRTCE is a typedef struct that's defined in include/hw/ppc/spapr.h which is included from target-ppc/kvm.c. So it should work

Re: [Qemu-devel] [Qemu-ppc] [PATCH 2/2] Add Enhanced Three-Speed Ethernet Controller (eTSEC)

2013-07-17 Thread Alexander Graf
Am 17.07.2013 um 10:24 schrieb Fabien Chouteau chout...@adacore.com: On 07/16/2013 06:54 PM, Scott Wood wrote: On 07/16/2013 11:15:51 AM, Fabien Chouteau wrote: On 07/16/2013 05:37 PM, Alexander Graf wrote: On 07/16/2013 05:28 PM, Fabien Chouteau wrote: On 07/16/2013 04:06 AM, Scott Wood

Re: [Qemu-devel] [Qemu-ppc] [PATCH 2/2] Add Enhanced Three-Speed Ethernet Controller (eTSEC)

2013-07-17 Thread Alexander Graf
On 17.07.2013, at 12:17, Fabien Chouteau wrote: On 07/16/2013 07:50 PM, Scott Wood wrote: On 07/16/2013 10:28:28 AM, Fabien Chouteau wrote: On 07/16/2013 04:06 AM, Scott Wood wrote: On 07/10/2013 12:10:02 PM, Fabien Chouteau wrote: +obj-$(CONFIG_ETSEC) += etsec.o etsec_registers.o

Re: [Qemu-devel] [PATCH] RFC: hcd-ohci: add dma error handling

2013-07-17 Thread Alexander Graf
On 17.07.2013, at 13:15, Benjamin Herrenschmidt wrote: On Wed, 2013-07-17 at 19:46 +1000, Alexey Kardashevskiy wrote: Current hcd-ohci does not handle DMA errors which can actually happen. However it is not clear what approach should be used here - for example, get_dwords returns positive

Re: [Qemu-devel] Possibility of unaligned DMA accesses via the QEMU DMA API?

2013-07-17 Thread Alexander Graf
On 17.07.2013, at 14:52, Mark Cave-Ayland wrote: On 17/07/13 09:16, Kevin Wolf wrote: Hi Kevin, Thanks for the reply - CC to qemu-devel as requested. I've been testing some of Alex Graf's patches for running Darwin under QEMU PPC and have been experiencing some timeout problems on

Re: [Qemu-devel] Possibility of unaligned DMA accesses via the QEMU DMA API?

2013-07-18 Thread Alexander Graf
On 18.07.2013, at 09:41, Kevin Wolf wrote: Am 17.07.2013 um 22:12 hat Mark Cave-Ayland geschrieben: On 17/07/13 14:35, Kevin Wolf wrote: Okay, so I've had a quick look at that DMA controller, and it seems that for a complete emulation, there's no way around using a bounce buffer (and

Re: [Qemu-devel] [PATCH] PPC: dbdma: macio: Fix format specifiers (build regression)

2013-07-18 Thread Alexander Graf
, unaligned, io-addr + io-len - unaligned); switch (s-dma_cmd) { Ping? Ping²? Acked-by: Alexander Graf ag...@suse.de I assume this can go through the trivial tree? Or directly get applied by Anthony? Alex

[Qemu-devel] [PATCH] PPC: E500: Generate device tree on reset

2013-07-22 Thread Alexander Graf
the generation into a reset function. That way we just generate it new every time we reset, solving both of the above issues. Signed-off-by: Alexander Graf ag...@suse.de --- hw/ppc/e500.c | 49 - 1 file changed, 40 insertions(+), 9 deletions(-) diff --git a/hw

[Qemu-devel] [PATCH 9/9] PPC: E500: Add PlatBus device tree walker

2013-07-22 Thread Alexander Graf
change in the future. Signed-off-by: Alexander Graf ag...@suse.de --- hw/ppc/e500.c | 54 ++ 1 file changed, 50 insertions(+), 4 deletions(-) diff --git a/hw/ppc/e500.c b/hw/ppc/e500.c index 8548622..e0d0a19 100644 --- a/hw/ppc/e500.c +++ b/hw/ppc

Re: [Qemu-devel] [PATCH] spapr: add yet another maintainer

2013-06-12 Thread Alexander Graf
On 13.06.2013, at 02:08, Alexey Kardashevskiy wrote: On 06/13/2013 12:33 AM, Alexander Graf wrote: On 12.06.2013, at 16:27, Alexey Kardashevskiy wrote: Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru --- MAINTAINERS |1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS

Re: [Qemu-devel] [Qemu-ppc] [PATCH] target-ppc: Change default machine for 64-bit

2013-06-13 Thread Alexander Graf
On 17.05.2013, at 09:42, Andreas Färber wrote: Am 17.05.2013 06:25, schrieb David Gibson: Currently, for qemu-system-ppc64, the default machine type is 'mac99'. Since the mac99 machine is not being actively maintained, and shows quite a few signs of bitrot, Please be more specific than

Re: [Qemu-devel] [PATCH v2 RFC] virtio-pci: fix LE host/BE guest capacity for blk

2013-06-13 Thread Alexander Graf
On 28.05.2013, at 12:18, Michael S. Tsirkin wrote: When a BE guest reads capacity from an LE host virtio-blk device or vice versa, it will get the dwords of the qword field swapped. As virtio-blk is the only one with such a quirk, and as non-pci transports don't do byte-swaps at all, solve

Re: [Qemu-devel] [PATCH 4/4] linux-user: Restore original behavior of the -E and -U command-line options.

2013-06-13 Thread Alexander Graf
On 29.05.2013, at 15:50, Thomas Schwinge wrote: Revert the change in behavior that had been introducecd in commit fc9c54124d134dbd76338a92a91804dab2df8166 for the -E and -U command-line options, but keep the comma-splitting for the QEMU_SET_ENV and QEMU_UNSET_ENV environment variables.

Re: [Qemu-devel] [RFC] sanitize memory on system reset

2013-06-14 Thread Alexander Graf
Am 14.06.2013 um 08:56 schrieb Christian Borntraeger borntrae...@de.ibm.com: On 13/06/13 13:56, Anthony Liguori wrote: Markus Armbruster arm...@redhat.com writes: Peter Lieven p...@kamp.de writes: On 13.06.2013 10:40, Stefan Hajnoczi wrote: On Thu, Jun 13, 2013 at 08:09:09AM +0200,

Re: [Qemu-devel] [Qemu-trivial] [PATCH] spapr: add yet another maintainer

2013-06-14 Thread Alexander Graf
--- a/MAINTAINERS +++ b/MAINTAINERS @@ -430,6 +430,7 @@ F: hw/isa/pc87312.[hc] sPAPR M: David Gibson da...@gibson.dropbear.id.au M: Alexander Graf ag...@suse.de +M: Alexey Kardashevskiy a...@ozlabs.ru Since apparently there was no objections, I'm applying this to -trivial. However

Re: [Qemu-devel] [PATCH 0/6] Clean up bogus default boot order

2013-06-14 Thread Alexander Graf
this stupid crap. Max Filippov and Peter Maydell already cleaned up Xtensa and ARM the same way. PPC patches are: Acked-by: Alexander Graf ag...@suse.de As for the functional change of defaulting boot_order to NULL on most boards, I'm reasonably indifferent. I suppose the purpose of the rename

Re: [Qemu-devel] [PATCH 0/2] ppc: QOM'ify openpic device

2013-06-18 Thread Alexander Graf
On 18.06.2013, at 03:58, Andreas Färber wrote: Hi Alex, Here's another mini-series cleaning up and converting the original OpenPIC device. Thanks, applied both. Scott, could you please send a follow-up patch that converts the kvm mpic device the same way? Thanks! Alex

Re: [Qemu-devel] [PATCH ppc-next fixup v2] intc/openpic_kvm: Fix QOM and build issues

2013-06-18 Thread Alexander Graf
On 16.06.2013, at 21:30, Andreas Färber wrote: Signed-off-by: Andreas Färber afaer...@suse.de --- v1 - v2: * Renamed busdev to parent_obj; * Fixed one remaining occurrence of FROM_SYSBUS(). Thanks, applied to ppc-next. Alex

Re: [Qemu-devel] [PATCH 0/2] ppc: QOM'ify openpic device

2013-06-18 Thread Alexander Graf
On 18.06.2013, at 14:35, Andreas Färber wrote: Am 18.06.2013 14:32, schrieb Alexander Graf: On 18.06.2013, at 03:58, Andreas Färber wrote: Hi Alex, Here's another mini-series cleaning up and converting the original OpenPIC device. Thanks, applied both. Thanks. Scott, could

Re: [Qemu-devel] [PATCH 0/3] ppc: QOM'ify mpc8544_guts device

2013-06-18 Thread Alexander Graf
On 09.06.2013, at 22:47, Andreas Färber wrote: Hi Alex, Here's a mini-series cleaning up the mpc8544_guts device. One user of SysBusDeviceClass::init less to worry about. ;) Thanks, applied to ppc-next. Alex

Re: [Qemu-devel] [PATCH ppc-next] target-ppc: Drop redundant flags assignments from CPU families

2013-06-18 Thread Alexander Graf
; assignments. PPC_NONE itself is still in use in translate.c. Suggested-by: Alexander Graf ag...@suse.de Signed-off-by: Andreas Färber afaer...@suse.de Thanks, applied to ppc-next. Alex

Re: [Qemu-devel] [PATCH 1/2] PPC: Add dump_mmu() for 6xx

2013-06-18 Thread Alexander Graf
On 18.06.2013, at 16:53, Fabien Chouteau wrote: (qemu) info tlb is a very useful tool for debugging, so I implemented the missing 6xx version. Signed-off-by: Fabien Chouteau chout...@adacore.com --- target-ppc/mmu_helper.c | 39 +++ 1 file changed, 39

Re: [Qemu-devel] [PATCH for-1.5] ppc: do not register IABR SPR twice for 603e

2013-06-18 Thread Alexander Graf
On 11.05.2013, at 21:03, Hervé Poussineau wrote: IABR SPR is already registered in gen_spr_603(), called from init_proc_603E(). Signed-off-by: Hervé Poussineau hpous...@reactos.org Thanks, applied to ppc-next. Sorry for the long delay - I was on vacation :) Alex ---

Re: [Qemu-devel] [0/3] Accumulated ppc patches

2013-06-18 Thread Alexander Graf
On 15.06.2013, at 03:51, David Gibson wrote: Hi Alex, I see that you're now back from your holiday. Here, rather belatedly, is my series of ppc related patches collected while acting ppc maintainer. As you can see there aren't many at all. I'm not sure if that's because I wasn't looking

Re: [Qemu-devel] [PATCH 2/2] PPC: Fix GDB read on code area for PPC6xx

2013-06-18 Thread Alexander Graf
On 18.06.2013, at 16:53, Fabien Chouteau wrote: On PPC 6xx, data and code have separated TLBs. Until now QEMU was only looking at data TLBs, which is not good when GDB wants to read code. This patch adds a second call to get_physical_address() with an ACCESS_CODE type of access when the

Re: [Qemu-devel] [0/3] Accumulated ppc patches

2013-06-19 Thread Alexander Graf
On 19.06.2013, at 01:20, Alexey Kardashevskiy wrote: On 06/19/2013 01:49 AM, Alexander Graf wrote: On 15.06.2013, at 03:51, David Gibson wrote: Hi Alex, I see that you're now back from your holiday. Here, rather belatedly, is my series of ppc related patches collected while acting

Re: [Qemu-devel] [PATCH 05/12] spapr-vty: add qtest test case

2013-06-19 Thread Alexander Graf
On 19.06.2013, at 22:40, Anthony Liguori wrote: Pretty basic for the moment but the interface is pretty simple. Signed-off-by: Anthony Liguori aligu...@us.ibm.com --- tests/Makefile | 3 ++ tests/spapr-vty-test.c | 89 ++ 2 files

Re: [Qemu-devel] [PATCH 07/12] spapr-rtas: add CPU argument to RTAS calls

2013-06-19 Thread Alexander Graf
On 19.06.2013, at 22:40, Anthony Liguori wrote: RTAS is a hypervisor provided binary blob that a guest loads and calls into to execute certain functions. It's similar to the vsyscall page in Linux or the short lived VMCI paravirt interface from VMware. The QEMU implementation of the RTAS

Re: [Qemu-devel] [PATCH 08/12] spapr-rtas: use hypercall interface and remove special vty interfaces

2013-06-19 Thread Alexander Graf
On 19.06.2013, at 22:40, Anthony Liguori wrote: Signed-off-by: Anthony Liguori aligu...@us.ibm.com --- hw/char/spapr_vty.c| 36 ++-- hw/ppc/spapr_rtas.c| 18 ++ include/hw/ppc/spapr_vio.h | 2 -- 3 files changed, 28

Re: [Qemu-devel] [PATCH 09/12] spapr-vio: move special case handling for reg=0 to vio

2013-06-19 Thread Alexander Graf
On 19.06.2013, at 22:40, Anthony Liguori wrote: The creatively named reg field is a hypervisor assigned global identifier for a virtual device. Despite the fact that no device is assigned a reg of 0, guests still use it to refer to early console. Instead of handling this in the VTY

Re: [Qemu-devel] [PATCH 10/12] spapr-vty: refactor the code to improve consistency

2013-06-19 Thread Alexander Graf
...@us.ibm.com Reviewed-by: Alexander Graf ag...@suse.de Alex

Re: [Qemu-devel] [PATCH 11/12] spapr-vio: pass type to spapr_vio_find_by_reg()

2013-06-19 Thread Alexander Graf
On 19.06.2013, at 22:40, Anthony Liguori wrote: Today if an invalid hcall is made to an otherwise valid device we abort due to QOM casting. This adds a parameter to find_by_reg() to indicate which type of device we're looking for. This should also adjust the hack to only match reg=0 for

Re: [Qemu-devel] [PATCH] pseries: Fix compiler warning (conversion of pointer to integral value)

2013-06-19 Thread Alexander Graf
-by: Alexander Graf ag...@suse.de I suppose this one goes through the trivial tree? Alex --- hw/ppc/spapr.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c index 218ea23..6e6f6a5 100644 --- a/hw/ppc/spapr.c +++ b/hw/ppc/spapr.c @@ -670,7 +670,7

Re: [Qemu-devel] [PATCH 05/12] spapr-vty: add qtest test case

2013-06-19 Thread Alexander Graf
On 19.06.2013, at 23:43, Anthony Liguori wrote: Alexander Graf ag...@suse.de writes: On 19.06.2013, at 22:40, Anthony Liguori wrote: Pretty basic for the moment but the interface is pretty simple. Signed-off-by: Anthony Liguori aligu...@us.ibm.com --- tests/Makefile | 3

Re: [Qemu-devel] [PATCH 08/12] spapr-rtas: use hypercall interface and remove special vty interfaces

2013-06-19 Thread Alexander Graf
On 19.06.2013, at 23:45, Anthony Liguori wrote: Alexander Graf ag...@suse.de writes: On 19.06.2013, at 22:40, Anthony Liguori wrote: Signed-off-by: Anthony Liguori aligu...@us.ibm.com --- hw/char/spapr_vty.c| 36 ++-- hw/ppc/spapr_rtas.c

Re: [Qemu-devel] [PATCH 09/12] spapr-vio: move special case handling for reg=0 to vio

2013-06-19 Thread Alexander Graf
On 19.06.2013, at 23:49, Anthony Liguori wrote: Alexander Graf ag...@suse.de writes: On 19.06.2013, at 22:40, Anthony Liguori wrote: The creatively named reg field is a hypervisor assigned global identifier for a virtual device. Despite the fact that no device is assigned a reg of 0

Re: [Qemu-devel] [PATCH] pseries: Fix compiler warning (conversion of pointer to integral value)

2013-06-20 Thread Alexander Graf
Am 20.06.2013 um 07:10 schrieb Michael Tokarev m...@tls.msk.ru: 20.06.2013 01:40, Alexander Graf wrote: On 19.06.2013, at 23:08, Stefan Weil wrote: This kind of type cast must use uintptr_t or target_ulong to be portable for hosts with sizeof(void *) != sizeof(long). Here the value

Re: [Qemu-devel] [PATCH 1/2] PPC: Add dump_mmu() for 6xx

2013-06-20 Thread Alexander Graf
On 18.06.2013, at 18:04, Fabien Chouteau wrote: On 06/18/2013 05:31 PM, Alexander Graf wrote: +for (type = 0; type 2; type++) You need braces on these. Please run your patch through checkpatch.pl :). I did ;) - ./scripts/checkpatch.pl 0001-PPC-Add-dump_mmu-for-6xx.patch total: 0

[Qemu-devel] [PATCH] Graphics: Switch to 800x600x32 as default mode

2013-06-20 Thread Alexander Graf
to 32 bit color as the default graphic mode. Reported-by: Mark Cave-Ayland mark.cave-ayl...@ilande.co.uk Signed-off-by: Alexander Graf ag...@suse.de --- arch_init.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch_init.c b/arch_init.c index a8b91ee..8ba3d07 100644

Re: [Qemu-devel] [PATCH v2] s390: Implement dump-guest-memory support for target s390x

2013-06-20 Thread Alexander Graf
On 12.06.2013, at 10:09, Jens Freimann wrote: From: Ekaterina Tumanova tuman...@linux.vnet.ibm.com With this patch dump-guest-memory on s390 produces an ELF formatted, crash-readable dump. In order to implement this, the arch-specific part of dump-guest-memory was added:

Re: [Qemu-devel] [PATCH] Graphics: Switch to 800x600x32 as default mode

2013-06-20 Thread Alexander Graf
On 20.06.2013, at 14:19, Peter Maydell wrote: On 20 June 2013 13:09, Alexander Graf ag...@suse.de wrote: We have stayed at 800x600x15 as default graphics mode for the last 9 years. If there ever was a reason to be there, surely nobody remembers it. However, recently non-Linux PPC guests

Re: [Qemu-devel] [PATCH 02/12] qtest: add spapr hypercall support

2013-06-20 Thread Alexander Graf
Am 20.06.2013 um 17:42 schrieb Anthony Liguori aligu...@us.ibm.com: Andreas Färber afaer...@suse.de writes: Am 19.06.2013 22:40, schrieb Anthony Liguori: Signed-off-by: Anthony Liguori aligu...@us.ibm.com --- qtest.c | 29 + tests/libqtest.c | 18

Re: [Qemu-devel] [PATCH 02/12] qtest: add spapr hypercall support

2013-06-20 Thread Alexander Graf
On 20.06.2013, at 20:58, Anthony Liguori wrote: Alexander Graf ag...@suse.de writes: Am 20.06.2013 um 17:42 schrieb Anthony Liguori aligu...@us.ibm.com: Andreas Färber afaer...@suse.de writes: Am 19.06.2013 22:40, schrieb Anthony Liguori: Signed-off-by: Anthony Liguori aligu

Re: [Qemu-devel] [PATCH] booke_ppc: limit booke timer to max when timeout overflow

2013-06-20 Thread Alexander Graf
On 12.06.2013, at 14:30, Bharat Bhushan wrote: Limit watchdog and fit timer to maximum timeout value which qemu timer can support (INT64_MAX). This maximum timeout will be hundreds of years, so limiting to max timeout is pretty safe. Signed-off-by: Bharat Bhushan

Re: [Qemu-devel] [PATCH v4] target-ppc: Introduce unrealizefn for PowerPCCPU

2013-06-20 Thread Alexander Graf
On 09.06.2013, at 22:11, Andreas Färber wrote: Use it to clean up the opcode table, resolving a former TODO from Jocelyn. Also switch from malloc() to g_malloc(). Signed-off-by: Andreas Färber afaer...@suse.de Thanks, applied to ppc-next. Alex

Re: [Qemu-devel] [PATCH] pseries: Support for in-kernel XICS interrupt controller

2013-06-20 Thread Alexander Graf
On 05.06.2013, at 09:39, Alexey Kardashevskiy wrote: From: David Gibson da...@gibson.dropbear.id.au Recent (host) kernels support emulating the PAPR defined XICS interrupt controller system within KVM. This patch allows qemu to initialize and configure the in-kernel XICS, and keep its

Re: [Qemu-devel] [PATCH v2] e600 core for MPC86xx processors

2013-06-20 Thread Alexander Graf
On 26.05.2013, at 19:41, Julio Guerra wrote: MPC86xx processors are based on the e600 core, which is not the case in qemu where it is based on the 7400 processor. This patch creates the e600 core and instantiates the MPC86xx processors based on it. Therefore, adding the high BATs and the

Re: [Qemu-devel] [PATCH 1/3] spapr pci msi: rework

2013-06-21 Thread Alexander Graf
On 21.06.2013, at 11:22, Alexey Kardashevskiy wrote: Previously every PCI host bridge implemented its own MSI memory window in order to catch msi_notify()/msix_notify() calls from various QEMU MSI-capable devives such as virtio-pci or vfio and redirect them to the guest via qemu_pulse_irq().

Re: [Qemu-devel] [PATCH 2/3] KVM: add kvm_arch_irqchip_add_msi_route

2013-06-21 Thread Alexander Graf
On 21.06.2013, at 11:22, Alexey Kardashevskiy wrote: At the moment kvm_irqchip_add_msi_route() adds routing entries for every MSI IRQ we want to use with IRQFD. However on PPC64-pSeries no routing is required as QEMU is already doing MSIMessage to IRQ converson. When the guest needs to

Re: [Qemu-devel] [PATCH 1/3] spapr pci msi: rework

2013-06-21 Thread Alexander Graf
On 21.06.2013, at 13:58, Anthony Liguori wrote: Alexey Kardashevskiy a...@ozlabs.ru writes: On 06/21/2013 08:31 PM, Alexander Graf wrote: On 21.06.2013, at 11:22, Alexey Kardashevskiy wrote: Previously every PCI host bridge implemented its own MSI memory window in order to catch

Re: [Qemu-devel] [PATCH v2] e600 core for MPC86xx processors

2013-06-21 Thread Alexander Graf
On 21.06.2013, at 14:01, Julio Guerra wrote: 2013/6/21 Alexander Graf ag...@suse.de: On 26.05.2013, at 19:41, Julio Guerra wrote: MPC86xx processors are based on the e600 core, which is not the case in qemu where it is based on the 7400 processor. This patch creates the e600 core

Re: [Qemu-devel] [PATCH 2/3] KVM: add kvm_arch_irqchip_add_msi_route

2013-06-21 Thread Alexander Graf
On 21.06.2013, at 14:03, Benjamin Herrenschmidt wrote: On Fri, 2013-06-21 at 12:33 +0200, Alexander Graf wrote: +int kvm_arch_irqchip_add_msi_route(KVMState *s, MSIMessage msg) +{ +if (!kvm_msi_via_irqfd_allowed) +return -1; + +msg.address -= spapr-msi_win_addr

Re: [Qemu-devel] [PATCH 2/3] KVM: add kvm_arch_irqchip_add_msi_route

2013-06-21 Thread Alexander Graf
On 21.06.2013, at 14:10, Benjamin Herrenschmidt wrote: On Fri, 2013-06-21 at 14:05 +0200, Alexander Graf wrote: Oh, I thought that was obvious. BookE also has in-kernel emulation and also does set kvm_msi_via_irqfd_allowed. The code above is spapr specific and would simply rewrite valid

Re: [Qemu-devel] [PATCH V2] PPC: Add dump_mmu() for 6xx

2013-06-21 Thread Alexander Graf
On 21.06.2013, at 15:26, Fabien Chouteau wrote: (qemu) info tlb is a very useful tool for debugging, so I implemented the missing 6xx version. Signed-off-by: Fabien Chouteau chout...@adacore.com Thanks, applied to ppc-next. It seems like a bunch of this could be shared with the hash32 MMU

Re: [Qemu-devel] [PATCH 2/3] KVM: add kvm_arch_irqchip_add_msi_route

2013-06-21 Thread Alexander Graf
On 21.06.2013, at 23:54, Benjamin Herrenschmidt wrote: On Fri, 2013-06-21 at 15:46 +0200, Alexander Graf wrote: Not sure. We could just declare a direct virq==irq mode in which msi.data == virq == irq. No need for any translation then. Maybe. Beware that MSI data is only 16-bit on the wire

Re: [Qemu-devel] [PATCH 2/2] PPC: Fix GDB read on code area for PPC6xx

2013-06-21 Thread Alexander Graf
On 21.06.2013, at 20:10, Jan Kiszka wrote: On 2013-06-18 17:34, Alexander Graf wrote: On 18.06.2013, at 16:53, Fabien Chouteau wrote: On PPC 6xx, data and code have separated TLBs. Until now QEMU was only looking at data TLBs, which is not good when GDB wants to read code. This patch

Re: [Qemu-devel] [Qemu-ppc] [PATCH v2] e600 core for MPC86xx processors

2013-06-21 Thread Alexander Graf
On 21.06.2013, at 01:16, Alexander Graf wrote: On 26.05.2013, at 19:41, Julio Guerra wrote: MPC86xx processors are based on the e600 core, which is not the case in qemu where it is based on the 7400 processor. This patch creates the e600 core and instantiates the MPC86xx processors

Re: [Qemu-devel] KVM call agenda for 2013-06-25

2013-06-21 Thread Alexander Graf
On 20.06.2013, at 14:47, Michael S. Tsirkin wrote: Please, send any topic that you are interested in covering. VFIO with platform devices Alex

[Qemu-devel] [PATCH] PPC: Introduce an alias cache for faster lookups

2013-06-21 Thread Alexander Graf
. Instead, we can just go directly from the alias name to the target class pointer. This patch brings -cpu ? to reasonable times again. Before: real0m4.716s After: real0m0.025s Signed-off-by: Alexander Graf ag...@suse.de --- It would be even nicer if we could have the alias table parsed

[Qemu-devel] [PATCH] PPC: Add clock-frequency export for Mac machines

2013-06-22 Thread Alexander Graf
...@ilande.co.uk Signed-off-by: Alexander Graf ag...@suse.de --- hw/ppc/mac_newworld.c | 2 ++ hw/ppc/mac_oldworld.c | 2 ++ include/hw/ppc/ppc.h | 1 + 3 files changed, 5 insertions(+) diff --git a/hw/ppc/mac_newworld.c b/hw/ppc/mac_newworld.c index d8e4db3..6bf892a 100644 --- a/hw/ppc

[Qemu-devel] [PATCH v2] PPC: Add clock-frequency export for Mac machines

2013-06-22 Thread Alexander Graf
...@ilande.co.uk Signed-off-by: Alexander Graf ag...@suse.de --- v1 - v2: - rename fwcfg var to CLOCKFREQ to better reflect what it means --- hw/ppc/mac_newworld.c | 2 ++ hw/ppc/mac_oldworld.c | 2 ++ include/hw/ppc/ppc.h | 1 + 3 files changed, 5 insertions(+) diff --git a/hw/ppc

Re: [Qemu-devel] [PATCH v2] e600 core for MPC86xx processors

2013-06-24 Thread Alexander Graf
Am 24.06.2013 um 08:13 schrieb Julio Guerra gu...@julio.in: 2013/6/24 Alexey Kardashevskiy a...@ozlabs.ru: On 06/21/2013 10:03 PM, Alexander Graf wrote: On 21.06.2013, at 14:01, Julio Guerra wrote: 2013/6/21 Alexander Graf ag...@suse.de: On 26.05.2013, at 19:41, Julio Guerra wrote

Re: [Qemu-devel] [PATCH 1/2] device_tree: Add qemu_devtree_setprop_sized_cells() utility function

2013-06-24 Thread Alexander Graf
On 24.06.2013, at 12:22, Peter Maydell wrote: We already have a qemu_devtree_setprop_cells() which sets a dtb property to an array of cells whose values are specified by varargs. However for the fairly common case of setting a property to a list of addresses or of address,size pairs the

Re: [Qemu-devel] [PATCH] RFC kvm irqfd: add directly mapped MSI IRQ support

2013-06-24 Thread Alexander Graf
On 24.06.2013, at 14:32, Anthony Liguori wrote: Gleb Natapov g...@redhat.com writes: On Sun, Jun 23, 2013 at 10:06:05AM -0500, Anthony Liguori wrote: On Thu, Jun 20, 2013 at 11:46 PM, Alex Williamson alex.william...@redhat.com wrote: On Fri, 2013-06-21 at 12:49 +1000, Alexey Kardashevskiy

Re: [Qemu-devel] [PATCH v3] s390/ipl: Fix boot order

2013-06-24 Thread Alexander Graf
On 18.06.2013, at 14:31, Christian Borntraeger wrote: The latest ipl code adaptions collided with some of the virtio refactoring rework. This resulted in always booting the first disk. Let's fix booting from a given ID. The new code also checks for command lines without bootindex to avoid

Re: [Qemu-devel] [PATCH 2/2] s390/IPL: Allow boot from other ssid than 0

2013-06-24 Thread Alexander Graf
On 17.06.2013, at 14:29, Christian Borntraeger wrote: From: Dominik Dingel din...@linux.vnet.ibm.com We now take the subchannel set id also into account to find the boot device. If we want to use a subchannel set other than the default set 0, we first need to enable the mss facility.

Re: [Qemu-devel] [PATCH] RFC kvm irqfd: add directly mapped MSI IRQ support

2013-06-24 Thread Alexander Graf
On 24.06.2013, at 16:31, Anthony Liguori wrote: Michael S. Tsirkin m...@redhat.com writes: On Mon, Jun 24, 2013 at 08:34:52AM -0500, Anthony Liguori wrote: Gleb Natapov g...@redhat.com writes: Isn't this more or less what Avi's previous proposal was around changing the APIC interfaces

[Qemu-devel] [PATCH] PPC: g3beige: Move secondary IDE bus to mac-io

2013-06-24 Thread Alexander Graf
On a real G3 Beige the secondary IDE bus lives on the mac-io chip, not on some random PCI device. Move it there to become more compatible. While at it, also clean up the IDE channel connection logic. Signed-off-by: Alexander Graf ag...@suse.de --- hw/ide/macio.c| 2 +- hw/misc/macio

[Qemu-devel] [PATCH 2/2] PPC: Newworld: Add second uninorth control register set

2013-06-24 Thread Alexander Graf
Mac OS X requires a second uninorth register set to be mapped a few bytes above the first one. Let's just expose it to make it happy. Signed-off-by: Alexander Graf ag...@suse.de --- hw/ppc/mac_newworld.c | 4 1 file changed, 4 insertions(+) diff --git a/hw/ppc/mac_newworld.c b/hw/ppc

[Qemu-devel] [PATCH 1/2] PPC: Newworld: Add uninorth token register

2013-06-24 Thread Alexander Graf
Mac OS X expects the uninorth control register set to contain one register that always reads back what it writes in. Expose that. This is just a temporary hack. Eventually, we want to expose the uninorth (/uni-n in device tree) as a separate QOM device. Signed-off-by: Alexander Graf ag

[Qemu-devel] [PATCH 0/2] PPC: Newworld: Some uni-n hacks

2013-06-24 Thread Alexander Graf
kernel, I am able to boot up to the point where the guest does not find its rootfs. Alex Alexander Graf (2): PPC: Newworld: Add uninorth token register PPC: Newworld: Add second uninorth control register set hw/ppc/mac_newworld.c | 17 +++-- 1 file changed, 15 insertions(+), 2

Re: [Qemu-devel] KVM call agenda for 2013-06-25

2013-06-25 Thread Alexander Graf
On 22.06.2013, at 03:01, Alexander Graf wrote: On 20.06.2013, at 14:47, Michael S. Tsirkin wrote: Please, send any topic that you are interested in covering. VFIO with platform devices A few notes ahead of the discussion: -problem we are trying to solve is: 1. how the kernel can

Re: [Qemu-devel] [PATCH 1/2] device_tree: Add qemu_devtree_setprop_sized_cells() utility function

2013-06-26 Thread Alexander Graf
On 26.06.2013, at 10:49, Peter Maydell wrote: On 26 June 2013 00:38, David Gibson da...@gibson.dropbear.id.au wrote: On Mon, Jun 24, 2013 at 12:02:39PM +0100, Peter Maydell wrote: On 24 June 2013 11:56, Alexander Graf ag...@suse.de wrote: This looks pretty complicated for something actually

Re: [Qemu-devel] [PATCH 1/2] device_tree: Add qemu_devtree_setprop_sized_cells() utility function

2013-06-26 Thread Alexander Graf
On 26.06.2013, at 12:50, Peter Maydell wrote: On 26 June 2013 11:31, Alexander Graf ag...@suse.de wrote: I think it makes sense to make this API special-purpose for reg. We currently have a generic put any number of 32bit values into the property function (qemu_devtree_setprop_cells). Yes

[Qemu-devel] [PATCH] mac-io: Add escc-legacy memory alias region

2013-06-26 Thread Alexander Graf
' \ -cdrom 10.4.iso -boot d Signed-off-by: Alexander Graf ag...@suse.de --- hw/misc/macio/macio.c | 47 +++ 1 file changed, 47 insertions(+) diff --git a/hw/misc/macio/macio.c b/hw/misc/macio/macio.c index 9ac004a..d9971e2 100644 --- a/hw/misc/macio/macio.c

Re: [Qemu-devel] [PATCH 1/2] device_tree: Add qemu_devtree_setprop_sized_cells() utility function

2013-06-26 Thread Alexander Graf
On 06/26/2013 02:38 PM, Peter Crosthwaite wrote: Hi, On Wed, Jun 26, 2013 at 8:50 PM, Peter Maydellpeter.mayd...@linaro.org wrote: On 26 June 2013 11:31, Alexander Grafag...@suse.de wrote: I think it makes sense to make this API special-purpose for reg. We currently have a generic put any

[Qemu-devel] [PATCH] PPC: Add non-kvm stub file

2013-06-28 Thread Alexander Graf
There are cases where a kvm provided function is called from generic hw code that doesn't know whether kvm is available or not. Provide a stub file which can provide simple replacement functions for those cases. Signed-off-by: Alexander Graf ag...@suse.de diff --git a/target-ppc/Makefile.objs b

Re: [Qemu-devel] [Qemu-ppc] [PATCH] PPC: Add non-kvm stub file

2013-06-28 Thread Alexander Graf
On 28.06.2013, at 13:54, Alexander Graf wrote: There are cases where a kvm provided function is called from generic hw code that doesn't know whether kvm is available or not. Provide a stub file which can provide simple replacement functions for those cases. Signed-off-by: Alexander Graf

Re: [Qemu-devel] [PATCH 3/6] Added Aarch64 CPU Initialization, Get and Put Registers Support.

2013-06-28 Thread Alexander Graf
On 28.06.2013, at 14:11, Mian M. Hamayun wrote: From: Mian M. Hamayun m.hama...@virtualopensystems.com The init function tries to initialize with Foundation models first and on failure retries initializing on Fast Models. Get and Put Registers deal with the basic state of Aarch64 CPUs

[Qemu-devel] [PATCH] arch_init/ram_load: fix compilation on 32bit hosts

2013-06-28 Thread Alexander Graf
Commit 87d2f825 broke compilation of arch_init.c on 32bit hosts. Fix it. Signed-off-by: Alexander Graf ag...@suse.de --- arch_init.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/arch_init.c b/arch_init.c index 560d018..f46bdb1 100644 --- a/arch_init.c +++ b

Re: [Qemu-devel] [PATCH v2 01/12] ARM: Export cpu_env

2013-06-28 Thread Alexander Graf
On 28.06.2013, at 16:20, Peter Maydell wrote: On 30 April 2013 13:00, Andreas Färber afaer...@suse.de wrote: Am 30.04.2013 13:54, schrieb Alexander Graf: Am 30.04.2013 um 13:42 schrieb Andreas Färber afaer...@suse.de: Am 30.04.2013 08:36, schrieb John Rigby: From: Alexander Graf ag

Re: [Qemu-devel] [PATCH] arch_init/ram_load: fix compilation on 32bit hosts

2013-06-28 Thread Alexander Graf
On 28.06.2013, at 16:33, Andreas Färber wrote: Am 28.06.2013 16:15, schrieb Alexander Graf: Commit 87d2f825 broke compilation of arch_init.c on 32bit hosts. Fix it. Signed-off-by: Alexander Graf ag...@suse.de Cf. http://patchwork.ozlabs.org/patch/255122/ :) Ah, nice. I'll drop my patch

[Qemu-devel] [PATCH] PPC: Mac: Fix guest exported tbfreq values

2013-06-29 Thread Alexander Graf
We can tell the guest the frequency of its time base through fwcfg. However, we tell it a different value from the speed tb actually runs at. Let's fix it and make the tbfreq initialization and the fwcfg exposure use the same values. Signed-off-by: Alexander Graf ag...@suse.de --- hw/ppc

Re: [Qemu-devel] [PATCH 3/6] Added Aarch64 CPU Initialization, Get and Put Registers Support.

2013-06-29 Thread Alexander Graf
On 29.06.2013, at 19:48, Mian M. Hamayun wrote: On 06/28/2013 02:43 PM, Alexander Graf wrote: On 28.06.2013, at 14:11, Mian M. Hamayun wrote: diff --git a/linux-headers/linux/kvm.h b/linux-headers/linux/kvm.h index c614070..4df5292 100644 --- a/linux-headers/linux/kvm.h +++ b/linux

[Qemu-devel] [PATCH 02/15] PPC: g3beige: Move secondary IDE bus to mac-io

2013-06-29 Thread Alexander Graf
On a real G3 Beige the secondary IDE bus lives on the mac-io chip, not on some random PCI device. Move it there to become more compatible. While at it, also clean up the IDE channel connection logic. Signed-off-by: Alexander Graf ag...@suse.de --- v1 - v2: - fix IRQ mapping --- hw/ide

[Qemu-devel] [PATCH 09/15] PPC: dbdma: Introduce kick function

2013-06-29 Thread Alexander Graf
The DBDMA engine really is running all the time, waiting for input. However we don't want to waste cycles constantly polling. So introduce a kick function that data providers can call to notify the DBDMA controller of new input. Signed-off-by: Alexander Graf ag...@suse.de --- hw/misc/macio

[Qemu-devel] [PATCH 01/15] PPC: Mac: Fix guest exported tbfreq values

2013-06-29 Thread Alexander Graf
We can tell the guest the frequency of its time base through fwcfg. However, we tell it a different value from the speed tb actually runs at. Let's fix it and make the tbfreq initialization and the fwcfg exposure use the same values. Signed-off-by: Alexander Graf ag...@suse.de --- hw/ppc

[Qemu-devel] [PATCH 00/15] PPC: Mac OS X guest bringup

2013-06-29 Thread Alexander Graf
macos-v1 Enjoy! Alex Alexander Graf (15): PPC: Mac: Fix guest exported tbfreq values PPC: g3beige: Move secondary IDE bus to mac-io PPC: Macio: Replace tabs with spaces PPC: dbdma: Replace tabs with spaces PPC: Mac: Add debug prints in macio and dbdma code PPC: dbdma: Fix debug print

[Qemu-devel] [PATCH 03/15] PPC: Macio: Replace tabs with spaces

2013-06-29 Thread Alexander Graf
s/^I//g on the file. Signed-off-by: Alexander Graf ag...@suse.de --- hw/ide/macio.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/hw/ide/macio.c b/hw/ide/macio.c index 7a1c573..82409dc 100644 --- a/hw/ide/macio.c +++ b/hw/ide/macio.c @@ -55,7 +55,7 @@ static

[Qemu-devel] [PATCH 05/15] PPC: Mac: Add debug prints in macio and dbdma code

2013-06-29 Thread Alexander Graf
disabled by default of course. Signed-off-by: Alexander Graf ag...@suse.de --- hw/ide/macio.c| 39 ++- hw/misc/macio/mac_dbdma.c | 12 ++-- 2 files changed, 48 insertions(+), 3 deletions(-) diff --git a/hw/ide/macio.c b/hw/ide/macio.c index

[Qemu-devel] [PATCH 06/15] PPC: dbdma: Fix debug print

2013-06-29 Thread Alexander Graf
There was a debug print that didn't compile for me because the format and the arguments weren't in sync. Fix it up. Signed-off-by: Alexander Graf ag...@suse.de --- hw/misc/macio/mac_dbdma.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hw/misc/macio/mac_dbdma.c b/hw/misc

[Qemu-devel] [PATCH 14/15] PPC: dbdma: Support unaligned DMA access

2013-06-29 Thread Alexander Graf
to cache these unaligned sectors until the next DMA request, at which point the data might be successfully flushed from the pipe. Signed-off-by: Alexander Graf ag...@suse.de --- hw/ide/macio.c | 128 ++--- include/hw/ppc/mac_dbdma.h | 3 ++ 2

[Qemu-devel] [PATCH 12/15] PPC: dbdma: Move processing to io

2013-06-29 Thread Alexander Graf
Soon we will introduce intermediate processing pauses which will allow the bottom half to restart a DMA request that couldn't be fulfilled yet. For that to work, move the processing variable into the io struct which is what DMA providers work with. Signed-off-by: Alexander Graf ag...@suse.de

<    1   2   3   4   5   6   7   8   9   10   >