[Qemu-devel] [PATCH 08/16] pseries: Remove PCI device from PCI host bridge code

2012-03-15 Thread Alexander Graf
From: Alexey Kardashevskiy a...@ozlabs.ru The sPAPR PCI code defines a PCI device spapr-pci-host-bridge-pci which is never used. This came over from the earlier bridge driver we used as a template. Some other bridges appear on their own PCI bus as a device, but that is not true of pSeries

[Qemu-devel] [PATCH 7/9] rtc: split out macros into a header file and use in test case

2012-03-15 Thread Anthony Liguori
Signed-off-by: Anthony Liguori aligu...@us.ibm.com --- hw/mc146818rtc.c | 33 -- hw/mc146818rtc.h |3 +- hw/mc146818rtc_regs.h | 62 + 3 files changed, 63 insertions(+), 35 deletions(-) create mode 100644

[Qemu-devel] [PATCH 5/9] qtest: IRQ interception infrastructure (v2)

2012-03-15 Thread Anthony Liguori
From: Paolo Bonzini pbonz...@redhat.com Signed-off-by: Paolo Bonzini pbonz...@redhat.com Signed-off-by: Anthony Liguori aligu...@us.ibm.com --- v1 - v2 - rebased to latest (aliguori) --- hw/irq.c | 17 +++ hw/irq.h |5 +++ hw/pc_piix.c |7 +++- qtest.c | 92

[Qemu-devel] [PATCH] qapi: escaping the dots in c_var

2012-03-15 Thread Federico Simoncelli
This allows qapi commands and types with dots. Signed-off-by: Federico Simoncelli fsimo...@redhat.com --- scripts/qapi.py |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/scripts/qapi.py b/scripts/qapi.py index 6e05469..4090c55 100644 --- a/scripts/qapi.py +++

[Qemu-devel] [PATCH 9/9] qtest: add clock management

2012-03-15 Thread Anthony Liguori
From: Paolo Bonzini pbonz...@redhat.com This patch combines qtest and -icount together to turn the vm_clock into a source that can be fully managed by the client. To this end new commands clock_step and clock_set are added. Hooking them with libqtest is left as an exercise to the reader.

[Qemu-devel] [PATCH 14/16] PPC: KVM: Synchronize regs on CPU dump

2012-03-15 Thread Alexander Graf
When we dump the CPU registers, there's a certain chance they haven't been synchronized with KVM yet, so we have to manually trigger that. This aligns the code with x86 and fixes a bug where the register state was bogus on invalid/unknown kvm exit reasons. Reported-by: Benjamin Herrenschmidt

[Qemu-devel] [PATCH 1/9] w32: Support tests (make check)

2012-03-15 Thread Anthony Liguori
From: Stefan Weil s...@weilnetz.de Adding $(EXESUF) is needed to make those tests work on w32 hosts, too. v2: Rebased, added new tests, tests sorted alphabetically. Signed-off-by: Stefan Weil s...@weilnetz.de Signed-off-by: Anthony Liguori aligu...@us.ibm.com --- tests/Makefile | 38

[Qemu-devel] [PATCH 3/9] qtest: add C version of test infrastructure

2012-03-15 Thread Anthony Liguori
This also includes a qtest wrapper script to make it easier to launch qtest tests directly. Signed-off-by: Anthony Liguori aligu...@us.ibm.com --- scripts/qtest|5 + tests/Makefile |1 + tests/libqtest.c | 334 ++

[Qemu-devel] illumos/SmartOS support

2012-03-15 Thread Lee Essen
Hi, I've been working on getting the mainline qemu tree working on SmartOS (illumos), the current qemu support seems to be a branch of qemu-kvm and we've been having some problems getting spice working, so moving to the main qemu tree seemed sensible. Anyway, couple of questions… 1. Are you

[Qemu-devel] Illumos/SmartOS support

2012-03-15 Thread Lee Essen
Resend -- not sure what happened to my first mail... Hi, I've been working on getting the mainline qemu tree working on SmartOS (illumos), the current qemu support seems to be a branch of qemu-kvm and we've been having some problems getting spice working, so moving to the main qemu tree

[Qemu-devel] [RFC 00/12] QOM'ify remaining CPUs

2012-03-15 Thread Andreas Färber
Hello, Based on qom-cpu v4 and object_class_get_list() v2, this series converts the remaining CPUs to QOM. Their only intersection point is Makefile.target. TODO: Review LGPLv2.1 text differences (bug in German GNU webpage). TODO: Check that cpu_model_str is set correctly everywhere.

Re: [Qemu-devel] Debug single program in QEMU

2012-03-15 Thread 陳韋任
On Thu, Mar 15, 2012 at 01:26:44PM +0700, Mulyadi Santosa wrote: On Thu, Mar 15, 2012 at 09:12, Yue Chen ycyc...@gmail.com wrote: Hi!~ Now I'd like to see what the micro-ops (intermediate operations generated by TCG) of a single Linux or Windows program, any approach to do this under

Re: [Qemu-devel] [PATCH v3 0/4] qapi: Convert migrate

2012-03-15 Thread Luiz Capitulino
On Mon, 12 Mar 2012 16:58:56 -0300 Luiz Capitulino lcapitul...@redhat.com wrote: v3 o Implement progess counter (for the new HMP command) o Fix English mistakes in changelog o Improve 'detach' parameter description No review comments, assuming this is ok. Applied to the qmp branch.

Re: [Qemu-devel] [RFC][PATCH 11/14 v9] introduce a new monitor command 'dump' to dump guest's memory

2012-03-15 Thread Luiz Capitulino
On Wed, 14 Mar 2012 14:18:47 -0300 Luiz Capitulino lcapitul...@redhat.com wrote: +ret = write(fd, buf, size); +if (ret != size) { +return -1; +} I think you should use send_all() instead of plain write(). Missed the fact that we actually have qemu_write_full().

[Qemu-devel] [PATCH v4 2/3] Support for MRCC and MCRR instructions

2012-03-15 Thread Alexey Starikovskiy
Signed-off-by: Alexey Starikovskiy aysta...@gmail.com --- target-arm/helper.c| 28 target-arm/helper.h|2 ++ target-arm/translate.c | 47 +-- 3 files changed, 59 insertions(+), 18 deletions(-) diff --git

[Qemu-devel] [PATCH v4 3/3] Minimal ARM LPAE support.

2012-03-15 Thread Alexey Starikovskiy
Sufficient to boot Linux kernel on vexpress-a15 Missing: * Extends the DBGDRAR and DBGDSAR to 64 bits, to hold PAs of up to 40 bits. * Defines two Memory Attribute Indirection Registers, MAIRn, to replace PRRR and NMRR when using the Long-descriptor translation table format. * Provides two

[Qemu-devel] [PATCH v4 1/3] Add support for 64bit ARM system registers

2012-03-15 Thread Alexey Starikovskiy
Signed-off-by: Alexey Starikovskiy aysta...@gmail.com --- target-arm/cpu.h | 10 -- target-arm/helper.c | 14 +++--- target-arm/machine.c | 16 ++-- 3 files changed, 17 insertions(+), 23 deletions(-) diff --git a/target-arm/cpu.h b/target-arm/cpu.h index

Re: [Qemu-devel] [PATCH 3/9] qtest: add C version of test infrastructure

2012-03-15 Thread Stefan Hajnoczi
On Thu, Mar 15, 2012 at 1:37 PM, Anthony Liguori aligu...@us.ibm.com wrote: +    sock = socket(PF_UNIX, SOCK_STREAM, 0); +    g_assert_no_errno(sock); + +    addr.sun_family = AF_UNIX; +    snprintf(addr.sun_path, sizeof(addr.sun_path), %s, socket_path); + +    pid = fork(); +    if (pid

Re: [Qemu-devel] [PATCH] qapi: escaping the dots in c_var

2012-03-15 Thread Michael Roth
On Thu, Mar 15, 2012 at 01:53:38PM +, Federico Simoncelli wrote: This allows qapi commands and types with dots. Signed-off-by: Federico Simoncelli fsimo...@redhat.com --- scripts/qapi.py |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/scripts/qapi.py

Re: [Qemu-devel] [PATCH v2 1/5] error: add error_set_file_open_failed

2012-03-15 Thread Luiz Capitulino
On Thu, 15 Mar 2012 00:55:10 +0200 Alon Levy al...@redhat.com wrote: Signed-off-by: Alon Levy al...@redhat.com --- cpus.c |4 ++-- error.c | 44 error.h |4 qerror.c | 36

Re: [Qemu-devel] [PATCH 3/9] qtest: add C version of test infrastructure

2012-03-15 Thread Kevin Wolf
Am 15.03.2012 15:42, schrieb Stefan Hajnoczi: On Thu, Mar 15, 2012 at 1:37 PM, Anthony Liguori aligu...@us.ibm.com wrote: +sock = socket(PF_UNIX, SOCK_STREAM, 0); +g_assert_no_errno(sock); + +addr.sun_family = AF_UNIX; +snprintf(addr.sun_path, sizeof(addr.sun_path), %s,

Re: [Qemu-devel] [PATCH v2 3/5] qapi: convert screendump

2012-03-15 Thread Luiz Capitulino
On Thu, 15 Mar 2012 00:55:12 +0200 Alon Levy al...@redhat.com wrote: Signed-off-by: Alon Levy al...@redhat.com --- hmp-commands.hx |3 +-- hmp.c|8 hmp.h|1 + monitor.c|8 qapi-schema.json | 13 +

Re: [Qemu-devel] [PATCH v2 2/5] vga_hw_screen_dump: add Error** param

2012-03-15 Thread Luiz Capitulino
On Thu, 15 Mar 2012 00:55:11 +0200 Alon Levy al...@redhat.com wrote: Signed-off-by: Alon Levy al...@redhat.com Please, explain why. More comments at the bottom. --- console.c |5 +++-- console.h |6 -- hw/blizzard.c |3 ++- hw/g364fb.c |4 +++-

Re: [Qemu-devel] [PATCH v2 5/5] blockdev: use error_set_file_open_failed

2012-03-15 Thread Luiz Capitulino
On Thu, 15 Mar 2012 00:55:14 +0200 Alon Levy al...@redhat.com wrote: This is a little trickier, since those calls chain in many fun ways and produce sometimes their own return values reusing existing errno values for similar meanings. In that respect error_set_file_open_failed specifically

Re: [Qemu-devel] [PATCH 3/9] qtest: add C version of test infrastructure

2012-03-15 Thread Anthony Liguori
On 03/15/2012 09:42 AM, Stefan Hajnoczi wrote: On Thu, Mar 15, 2012 at 1:37 PM, Anthony Liguorialigu...@us.ibm.com wrote: +sock = socket(PF_UNIX, SOCK_STREAM, 0); +g_assert_no_errno(sock); + +addr.sun_family = AF_UNIX; +snprintf(addr.sun_path, sizeof(addr.sun_path), %s,

Re: [Qemu-devel] [PATCH] qapi: escaping the dots in c_var

2012-03-15 Thread Luiz Capitulino
On Thu, 15 Mar 2012 09:51:10 -0500 Michael Roth mdr...@linux.vnet.ibm.com wrote: On Thu, Mar 15, 2012 at 01:53:38PM +, Federico Simoncelli wrote: This allows qapi commands and types with dots. Signed-off-by: Federico Simoncelli fsimo...@redhat.com --- scripts/qapi.py |2 +-

Re: [Qemu-devel] [PATCH 3/9] qtest: add C version of test infrastructure

2012-03-15 Thread Paolo Bonzini
Il 15/03/2012 15:58, Kevin Wolf ha scritto: I believe Kevin suggested using -qtest unix:%s and creating the listen socket in the test program rather than inside QEMU. The advantage is that we never sleep(3), instead we accept(2) the connection from QEMU and get going right away. This

Re: [Qemu-devel] [PATCH 4/9] make: add check targets based on gtester (v2)

2012-03-15 Thread Paolo Bonzini
Il 15/03/2012 14:37, Anthony Liguori ha scritto: + echo $LINE You probably want echo $LINE here, or you could just replace the whole loop with this: sed -e '/?xml/d' -e '/gtester/d' $@ Paolo

Re: [Qemu-devel] [PATCH v4 3/3] Minimal ARM LPAE support.

2012-03-15 Thread Avi Kivity
On 03/15/2012 04:30 PM, Alexey Starikovskiy wrote: Sufficient to boot Linux kernel on vexpress-a15 Missing: * Extends the DBGDRAR and DBGDSAR to 64 bits, to hold PAs of up to 40 bits. * Defines two Memory Attribute Indirection Registers, MAIRn, to replace PRRR and NMRR when using the

Re: [Qemu-devel] [PATCH v2 1/5] error: add error_set_file_open_failed

2012-03-15 Thread Alon Levy
On Thu, Mar 15, 2012 at 11:52:32AM -0300, Luiz Capitulino wrote: On Thu, 15 Mar 2012 00:55:10 +0200 Alon Levy al...@redhat.com wrote: Signed-off-by: Alon Levy al...@redhat.com --- cpus.c |4 ++-- error.c | 44

Re: [Qemu-devel] [PATCH v2 2/5] vga_hw_screen_dump: add Error** param

2012-03-15 Thread Alon Levy
On Thu, Mar 15, 2012 at 11:54:07AM -0300, Luiz Capitulino wrote: On Thu, 15 Mar 2012 00:55:11 +0200 Alon Levy al...@redhat.com wrote: Signed-off-by: Alon Levy al...@redhat.com Please, explain why. More comments at the bottom. Sure. --- console.c |5 +++-- console.h

Re: [Qemu-devel] [PATCH v2 3/5] qapi: convert screendump

2012-03-15 Thread Alon Levy
On Thu, Mar 15, 2012 at 11:55:31AM -0300, Luiz Capitulino wrote: On Thu, 15 Mar 2012 00:55:12 +0200 Alon Levy al...@redhat.com wrote: Signed-off-by: Alon Levy al...@redhat.com --- hmp-commands.hx |3 +-- hmp.c|8 hmp.h|1 + monitor.c

Re: [Qemu-devel] [PATCH v2 5/5] blockdev: use error_set_file_open_failed

2012-03-15 Thread Alon Levy
On Thu, Mar 15, 2012 at 11:56:29AM -0300, Luiz Capitulino wrote: On Thu, 15 Mar 2012 00:55:14 +0200 Alon Levy al...@redhat.com wrote: This is a little trickier, since those calls chain in many fun ways and produce sometimes their own return values reusing existing errno values for

Re: [Qemu-devel] [PATCH v4 3/3] Minimal ARM LPAE support.

2012-03-15 Thread Peter Maydell
On 15 March 2012 15:20, Avi Kivity a...@redhat.com wrote: Please add code that detects the use of unimplemented features and aborts I'm not a great fan of letting the guest cause QEMU to abort... -- PMM

Re: [Qemu-devel] [PATCH v4 3/3] Minimal ARM LPAE support.

2012-03-15 Thread Avi Kivity
On 03/15/2012 05:24 PM, Peter Maydell wrote: On 15 March 2012 15:20, Avi Kivity a...@redhat.com wrote: Please add code that detects the use of unimplemented features and aborts I'm not a great fan of letting the guest cause QEMU to abort... Why not? It (the guest) is going to malfunction

Re: [Qemu-devel] [PATCH v4 3/3] Minimal ARM LPAE support.

2012-03-15 Thread Peter Maydell
On 15 March 2012 15:31, Avi Kivity a...@redhat.com wrote: On 03/15/2012 05:24 PM, Peter Maydell wrote: On 15 March 2012 15:20, Avi Kivity a...@redhat.com wrote: Please add code that detects the use of unimplemented features and aborts I'm not a great fan of letting the guest cause QEMU to

Re: [Qemu-devel] [PATCH v4 3/3] Minimal ARM LPAE support.

2012-03-15 Thread Avi Kivity
On 03/15/2012 05:37 PM, Peter Maydell wrote: On 15 March 2012 15:31, Avi Kivity a...@redhat.com wrote: On 03/15/2012 05:24 PM, Peter Maydell wrote: On 15 March 2012 15:20, Avi Kivity a...@redhat.com wrote: Please add code that detects the use of unimplemented features and aborts I'm

Re: [Qemu-devel] [PATCH] qapi: escaping the dots in c_var

2012-03-15 Thread Michael Roth
On Thu, Mar 15, 2012 at 12:03:30PM -0300, Luiz Capitulino wrote: On Thu, 15 Mar 2012 09:51:10 -0500 Michael Roth mdr...@linux.vnet.ibm.com wrote: On Thu, Mar 15, 2012 at 01:53:38PM +, Federico Simoncelli wrote: This allows qapi commands and types with dots. Signed-off-by:

Re: [Qemu-devel] Illumos/SmarOS support

2012-03-15 Thread Lee Essen
On 15/03/2012 13:12, Andreas Färber wrote: Something went wrong with your reply... While it contains a quote it does not reference the mail you replied to, that way messages get lost among the constant chatter on qemu-devel. Hmmm ... sorry -- I'm having to cut and paste emails between mail

Re: [Qemu-devel] [PATCH v4 3/3] Minimal ARM LPAE support.

2012-03-15 Thread Peter Maydell
On 15 March 2012 15:42, Avi Kivity a...@redhat.com wrote: On 03/15/2012 05:37 PM, Peter Maydell wrote: It would be cleaner to cause the guest to take an exception IMHO. (This is typically what ought to happen anyway if the guest accesses an unimplemented cp15 register, although our current

[Qemu-devel] [PATCH v3] Man page: Add -global description

2012-03-15 Thread Miroslav Rezanina
There's only TODO information in qemu man page for -global option. This is a basic description of this option with simple example. Signed-off-by: Miroslav Rezanina mreza...@redhat.com Patch: -- diff --git a/qemu-options.hx b/qemu-options.hx index daefce3..db8be37 100644 --- a/qemu-options.hx

Re: [Qemu-devel] Illumos/SmarOS support

2012-03-15 Thread Andreas Färber
Am 15.03.2012 16:48, schrieb Lee Essen: On 15/03/2012 13:12, Andreas Färber wrote: Something went wrong with your reply... While it contains a quote it does not reference the mail you replied to, that way messages get lost among the constant chatter on qemu-devel. Hmmm ... sorry -- I'm

Re: [Qemu-devel] [PATCH v2 1/5] error: add error_set_file_open_failed

2012-03-15 Thread Luiz Capitulino
On Thu, 15 Mar 2012 17:21:01 +0200 Alon Levy al...@redhat.com wrote: On Thu, Mar 15, 2012 at 11:52:32AM -0300, Luiz Capitulino wrote: On Thu, 15 Mar 2012 00:55:10 +0200 Alon Levy al...@redhat.com wrote: Signed-off-by: Alon Levy al...@redhat.com --- cpus.c |4

Re: [Qemu-devel] [PATCH v5 03/43] monitor: Don't access registers through CPUState

2012-03-15 Thread Lluís Vilanova
Andreas Färber writes: Use CPUX86State etc. instead (hand-converted). Is there any reason not to move this into CPU${arch}State and instead provide virtual methods in CPUState? For example (a CPUState argument is implicit on all methods): int reg_get_number(const char *regname); const

[Qemu-devel] [PATCH] xen-mapcache: don't unmap locked entry during mapcache invalidation

2012-03-15 Thread Julien Grall
When an IOREQ_TYPE_INVALIDATE is sent to QEMU, it invalidates all entry of the map cache even if it's locked. QEMU is not able to know that entry was invalidated, so when an IO access is requested a segfault occured. Signed-off-by: Julien Grall julien.gr...@citrix.com --- xen-mapcache.c |3

[Qemu-devel] [PATCH] pci: fix double free of romfile property

2012-03-15 Thread Anthony Liguori
The qdev property release function frees any string properties. This was resulting in a double free during hot unplug. It manifests in network devices because block devices have a NULL romfile property by default. Cc: Michael Tsirkin m...@redhat.com Signed-off-by: Anthony Liguori

[Qemu-devel] qemu gdb issue

2012-03-15 Thread Jacques
Hi folks, I'm busy with a small project and I'm running into problems debugging qemu. I hope this is the right place to get help. I'm running an application in qemu through the userspace qemu-i386 and attaching to the process with gdb. I have pygdb scripts that then interact with gdb. The issue

Re: [Qemu-devel] [patch V4 1/4] i.MX UART support

2012-03-15 Thread Peter Maydell
On 9 March 2012 03:27, peter.ch...@nicta.com.au wrote: +typedef struct { +    SysBusDevice busdev; +    MemoryRegion iomem; +    int32_t readbuff; + +    uint32_t usr1; +    uint32_t usr2; +    uint32_t ucr1; +    uint32_t ucr2; +    uint32_t uts1; + +    /* +     * The registers

Re: [Qemu-devel] [PATCH] xen-mapcache: don't unmap locked entry during mapcache invalidation

2012-03-15 Thread Stefano Stabellini
On Thu, 15 Mar 2012, Julien Grall wrote: When an IOREQ_TYPE_INVALIDATE is sent to QEMU, it invalidates all entry of the map cache even if it's locked. QEMU is not able to know that entry was invalidated, so when an IO access is requested a segfault occured. The problem here is the long term

Re: [Qemu-devel] [patch V4 2/4] FreeSCALE i.MX31 support: Timers

2012-03-15 Thread Peter Maydell
On 9 March 2012 03:27, peter.ch...@nicta.com.au wrote: Implement the timers on the FreeScale i.MX31 SoC. This is not a complete implementation, but gives enough for Linux to boot and run. In particular external triggers, which are not useful under QEMU, are not implemented. CODING_STYLE

Re: [Qemu-devel] [patch V4 2/4] FreeSCALE i.MX31 support: Timers

2012-03-15 Thread Peter Maydell
...that would work better if I cc'd the people I meant to... On 15 March 2012 17:08, Peter Maydell peter.mayd...@linaro.org wrote: On 9 March 2012 03:27,  peter.ch...@nicta.com.au wrote: Implement the timers on the FreeScale i.MX31 SoC. This is not a complete implementation, but gives enough

Re: [Qemu-devel] [PATCH] xen-mapcache: don't unmap locked entry during mapcache invalidation

2012-03-15 Thread Andres Lagar-Cavilla
On Thu, 15 Mar 2012, Julien Grall wrote: When an IOREQ_TYPE_INVALIDATE is sent to QEMU, it invalidates all entry of the map cache even if it's locked. QEMU is not able to know that entry was invalidated, so when an IO access is requested a segfault occured. The problem here is the long

Re: [Qemu-devel] [PATCH] xen-mapcache: don't unmap locked entry during mapcache invalidation

2012-03-15 Thread Olaf Hering
On Thu, Mar 15, Stefano Stabellini wrote: On Thu, 15 Mar 2012, Julien Grall wrote: When an IOREQ_TYPE_INVALIDATE is sent to QEMU, it invalidates all entry of the map cache even if it's locked. QEMU is not able to know that entry was invalidated, so when an IO access is requested a

Re: [Qemu-devel] qemu gdb issue

2012-03-15 Thread Mulyadi Santosa
Hi... On Thu, Mar 15, 2012 at 23:03, Jacques jacq...@rambo-mes.net wrote: I'm running an application in qemu through the userspace qemu-i386 and attaching to the process with gdb. I have pygdb scripts that then interact with gdb. The issue is that at some point I want to change $eip and

Re: [Qemu-devel] [patch V4 3/4] FreeSCALE i.MX31 support: AVIC

2012-03-15 Thread Peter Maydell
On 9 March 2012 03:27, peter.ch...@nicta.com.au wrote: Implement the FreeSCALE i.MX31 advanced vectored interrupt controller, at least to the extent it is used by Linux 3.0.x Vectors are not implemented. Signed-off-by: Philip O'Sullivan phil...@ok-labs.com Signed-off-by: Peter Chubb

Re: [Qemu-devel] [patch V4 4/4] FreeSCALE i.MX31 support: KZM-ARM11-01 evaluation board

2012-03-15 Thread Peter Maydell
On 9 March 2012 03:27, peter.ch...@nicta.com.au wrote: Board support for Kyoto Micro's KZM-ARM11-01, an evaluation board built around the FreeScale i.MX31. Signed-off-by: Philip O'Sullivan phil...@ok-labs.com Signed-off-by: Peter Chubb peter.ch...@nicta.com.au ---  Makefile.target |    1

Re: [Qemu-devel] Adding make check to the QEMU buildbot

2012-03-15 Thread Stefan Weil
Am 15.03.2012 09:06, schrieb Stefan Hajnoczi: QEMU has grown a number of sanity tests that can be run using make check. They are fast and do not require many resources. Is it possible to add make check after the build? We may have to deal with some failures in the beginning - either due to

Re: [Qemu-devel] [PATCH] xen-mapcache: don't unmap locked entry during mapcache invalidation

2012-03-15 Thread Stefano Stabellini
On Thu, 15 Mar 2012, Olaf Hering wrote: On Thu, Mar 15, Stefano Stabellini wrote: On Thu, 15 Mar 2012, Julien Grall wrote: When an IOREQ_TYPE_INVALIDATE is sent to QEMU, it invalidates all entry of the map cache even if it's locked. QEMU is not able to know that entry was

[Qemu-devel] [Bug 948675] Re: QEMU is crashing when called with -vga none

2012-03-15 Thread Serge Hallyn
** Also affects: qemu-kvm (Ubuntu) Importance: Undecided Status: New ** Changed in: qemu-kvm (Ubuntu) Importance: Undecided = High -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/948675

Re: [Qemu-devel] [Xen-devel] [PATCH] xen-mapcache: don't unmap locked entry during mapcache invalidation

2012-03-15 Thread Tim Deegan
At 17:14 + on 15 Mar (1331831693), Stefano Stabellini wrote: On Thu, 15 Mar 2012, Julien Grall wrote: When an IOREQ_TYPE_INVALIDATE is sent to QEMU, it invalidates all entry of the map cache even if it's locked. QEMU is not able to know that entry was invalidated, so when an IO

Re: [Qemu-devel] [Xen-devel] [PATCH] xen-mapcache: don't unmap locked entry during mapcache invalidation

2012-03-15 Thread Andres Lagar-Cavilla
At 17:14 + on 15 Mar (1331831693), Stefano Stabellini wrote: On Thu, 15 Mar 2012, Julien Grall wrote: When an IOREQ_TYPE_INVALIDATE is sent to QEMU, it invalidates all entry of the map cache even if it's locked. QEMU is not able to know that entry was invalidated, so when an IO

Re: [Qemu-devel] [Xen-devel] [PATCH] xen-mapcache: don't unmap locked entry during mapcache invalidation

2012-03-15 Thread Stefano Stabellini
On Thu, 15 Mar 2012, Tim Deegan wrote: At 17:14 + on 15 Mar (1331831693), Stefano Stabellini wrote: On Thu, 15 Mar 2012, Julien Grall wrote: When an IOREQ_TYPE_INVALIDATE is sent to QEMU, it invalidates all entry of the map cache even if it's locked. QEMU is not able to know

Re: [Qemu-devel] [Xen-devel] [PATCH] xen-mapcache: don't unmap locked entry during mapcache invalidation

2012-03-15 Thread Tim Deegan
At 10:32 -0700 on 15 Mar (1331807562), Andres Lagar-Cavilla wrote: At 17:14 + on 15 Mar (1331831693), Stefano Stabellini wrote: On Thu, 15 Mar 2012, Julien Grall wrote: When an IOREQ_TYPE_INVALIDATE is sent to QEMU, it invalidates all entry of the map cache even if it's locked.

[Qemu-devel] [PATCH] gdbserver: Don't send a GDB syscall until the system CPU is stopped

2012-03-15 Thread Peter Maydell
From: Meador Inge mead...@codesourcery.com Fix an issue where the GDB server implementation was sending GDB syscall requests while the system CPU was still running. Syscall requests must be sent while the CPU is stopped otherwise replies from the GDB client might get dropped and the GDB server

[Qemu-devel] [Bug 948675] Re: QEMU is crashing when called with -vga none

2012-03-15 Thread Anthony Liguori
** Changed in: qemu Status: Confirmed = Fix Committed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/948675 Title: QEMU is crashing when called with -vga none Status in QEMU: Fix

Re: [Qemu-devel] [PATCH] gdbserver: Don't send a GDB syscall until the system CPU is stopped

2012-03-15 Thread Andreas Färber
Am 15.03.2012 18:49, schrieb Peter Maydell: From: Meador Inge mead...@codesourcery.com Fix an issue where the GDB server implementation was sending GDB syscall requests while the system CPU was still running. Syscall requests must be sent while the CPU is stopped otherwise replies from the

[Qemu-devel] [PATCH v6 0/6] save/restore on Xen

2012-03-15 Thread Stefano Stabellini
Hi all, this is the sixth version of the Xen save/restore patch series. We have been discussing this issue for quite a while on #qemu and qemu-devel: http://marc.info/?l=qemu-develm=132346828427314w=2 http://marc.info/?l=qemu-develm=132377734605464w=2 Please review the second patch: Introduce

Re: [Qemu-devel] [PATCH v5 03/43] monitor: Don't access registers through CPUState

2012-03-15 Thread Andreas Färber
Am 15.03.2012 17:15, schrieb Lluís Vilanova: Andreas Färber writes: Use CPUX86State etc. instead (hand-converted). Is there any reason not to move this into CPU${arch}State and instead provide virtual methods in CPUState? Yes, a very simple one: This is one of the prerequisite patches to

Re: [Qemu-devel] [PATCH 2/2] Drop obsolete nographic timer

2012-03-15 Thread Marek Vasut
Dear Marek Vasut, Dear Jan Kiszka, On 2012-03-10 07:19, Marek Vasut wrote: Dear Jan Kiszka, We flush coalesced MMIO in the device models now, and VNC - for which this was once introduced - is also fine without it as it has its own refresh timer. Signed-off-by: Jan

[Qemu-devel] [PATCH] qemu-ga: for w32, fix leaked handle ov.hEvent in ga_channel_write()

2012-03-15 Thread Jeff Cody
In the function ga_channel_write(), the handle ov.hEvent is created by the call to CreateEvent(). However, the handle is not closed prior to the function return. This patch closes the handle before the return of the function. Kudos to Paolo Bonzini for spotting this bug. Signed-off-by: Jeff

[Qemu-devel] [PATCH v6 6/6] xen: do not allocate RAM during INMIGRATE runstate

2012-03-15 Thread Stefano Stabellini
From: Anthony PERARD anthony.per...@citrix.com Signed-off-by: Anthony PERARD anthony.per...@citrix.com Signed-off-by: Stefano Stabellini stefano.stabell...@eu.citrix.com --- xen-all.c |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/xen-all.c b/xen-all.c index

[Qemu-devel] [PATCH v6 4/6] xen: record physmap changes to xenstore

2012-03-15 Thread Stefano Stabellini
Write to xenstore any physmap changes so that the hypervisor can be aware of them. Read physmap changes from xenstore on boot. Signed-off-by: Stefano Stabellini stefano.stabell...@eu.citrix.com --- xen-all.c | 78 - 1 files changed,

[Qemu-devel] [PATCH v6 5/6] xen mapcache: check if memory region has moved.

2012-03-15 Thread Stefano Stabellini
From: Anthony PERARD anthony.per...@citrix.com This patch changes the xen_map_cache behavior. Before trying to map a guest addr, mapcache will look into the list of range of address that have been moved (physmap/set_memory). There is currently one memory space like this, the vram, moved from were

[Qemu-devel] [PATCH v6 2/6] Introduce save_devices

2012-03-15 Thread Stefano Stabellini
- add an is_ram flag to SaveStateEntry; - register_savevm_live sets is_ram for live_savevm devices; - introduce a save_devices QAPI command that can be used to save the state of all devices, but not the RAM or the block devices of the VM. Changes in v6: - remove the is_ram parameter from

[Qemu-devel] [PATCH v6 1/6] cirrus_vga: do not reset videoram

2012-03-15 Thread Stefano Stabellini
There is no need to set the videoram to 0xff in cirrus_reset, because it is the BIOS' job. Signed-off-by: Stefano Stabellini stefano.stabell...@eu.citrix.com Reviewed-by: Avi Kivity a...@redhat.com --- hw/cirrus_vga.c |4 1 files changed, 0 insertions(+), 4 deletions(-) diff --git

Re: [Qemu-devel] [PATCH] qemu-ga: for w32, fix leaked handle ov.hEvent in ga_channel_write()

2012-03-15 Thread Michael Roth
On Thu, Mar 15, 2012 at 02:26:18PM -0400, Jeff Cody wrote: In the function ga_channel_write(), the handle ov.hEvent is created by the call to CreateEvent(). However, the handle is not closed prior to the function return. This patch closes the handle before the return of the function.

[Qemu-devel] [PATCH v6 3/6] Set runstate to INMIGRATE earlier

2012-03-15 Thread Stefano Stabellini
Set runstate to RUN_STATE_INMIGRATE as soon as we can on resume. Signed-off-by: Stefano Stabellini stefano.stabell...@eu.citrix.com Acked-by: Luiz Capitulino lcapitul...@redhat.com --- vl.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/vl.c b/vl.c index

Re: [Qemu-devel] [PATCH RFC v4 06/20] target-arm: Move CPU feature flags out of CPUState

2012-03-15 Thread Paul Brook
The internal CPU feature flags were only ever set in cpu_reset_model_id(). Therefore move their initialization into ARMCPUClass. We might want to tweak them in the future though (e.g., -cpu cortex-r4,+fpu), so keep a copy in ARMCPU. This in turn means we need to infer features for both

Re: [Qemu-devel] [PATCH RFC v4 08/20] target-arm: Store cp15 c0_c1 and c0_c2 in ARMCPUClass

2012-03-15 Thread Paul Brook
For now set them in the reset function. +/* TODO Move these into arm_cpu_initfn() once no longer zeroed above.*/ +memcpy(env-cp15.c0_c1, klass-cp15.c0_c1, 8 * sizeof(uint32_t)); +memcpy(env-cp15.c0_c2, klass-cp15.c0_c2, 8 * sizeof(uint32_t)); + Why bother copying them into the

[Qemu-devel] BSOD when using Hyper-V extensions with Win2k8R2 guests

2012-03-15 Thread Michael Roth
Hi Vadim, I've been trying to use -cpu qemu64,hv_relaxed as a workaround for a CLOCK_WATCHDOG_TIMEOUT BSOD on Windows 2008 R2 guests when a vcpu's execution is delayed for an extended period. When I do this however I get the following the BSOD:

Re: [Qemu-devel] [PATCH RFC v4 08/20] target-arm: Store cp15 c0_c1 and c0_c2 in ARMCPUClass

2012-03-15 Thread Peter Maydell
On 15 March 2012 19:08, Paul Brook p...@codesourcery.com wrote: For now set them in the reset function. +    /* TODO Move these into arm_cpu_initfn() once no longer zeroed above.*/ +    memcpy(env-cp15.c0_c1, klass-cp15.c0_c1, 8 * sizeof(uint32_t)); +    memcpy(env-cp15.c0_c2,

Re: [Qemu-devel] [PATCH 3/5] exec: fix code tlb entry misused as iotlb in get_page_addr_code()

2012-03-15 Thread jcmvbkbc
Hi. get_page_addr_code() reads a code tlb entry, but interprets it as an iotlb entry. This works by accident since the low bits of a RAM code tlb entry are clear, and match a RAM iotlb entry. This accident is about to unhappen, so fix the code to use an iotlb entry (using the code entry with

Re: [Qemu-devel] [PATCH RFC v4 08/20] target-arm: Store cp15 c0_c1 and c0_c2 in ARMCPUClass

2012-03-15 Thread Alexey Starikovskiy
On Thu, Mar 15, 2012 at 11:20 PM, Peter Maydell peter.mayd...@linaro.org wrote: On 15 March 2012 19:08, Paul Brook p...@codesourcery.com wrote: For now set them in the reset function. +    /* TODO Move these into arm_cpu_initfn() once no longer zeroed above.*/ +    memcpy(env-cp15.c0_c1,

Re: [Qemu-devel] [RFC 05/12] target-i386: QOM'ify CPU

2012-03-15 Thread Eduardo Habkost
On Wed, Mar 14, 2012 at 06:53:29PM +0100, Andreas Färber wrote: [...] +/** + * X86CPUClass: + * @parent_reset: The parent class' reset handler. + * + * An x86 CPU model or family. + */ +typedef struct X86CPUClass { +/* private */ +CPUClass parent_class; +/* public */ + +

[Qemu-devel] [Bug 899961] Re: qemu/kvm locks up when run 32bit userspace with 64bit kernel

2012-03-15 Thread Michael Tokarev
I tried bisecting it today, again. I think you did mean 145e11e840500e04a4d0a624918bb17596be19e9 as the M point (the large merge), not b195043003d90ea4027ea01cc7a6c974ac915108 (which is the first commit in that merge) ;) Actually I did the same already, just didn't post to the bugreport. But

Re: [Qemu-devel] [PATCH RFC v4 08/20] target-arm: Store cp15 c0_c1 and c0_c2 in ARMCPUClass

2012-03-15 Thread Peter Maydell
On 15 March 2012 19:29, Alexey Starikovskiy aysta...@gmail.com wrote: On Thu, Mar 15, 2012 at 11:20 PM, Peter Maydell peter.mayd...@linaro.org wrote: On 15 March 2012 19:08, Paul Brook p...@codesourcery.com wrote: For now set them in the reset function. +    /* TODO Move these into

Re: [Qemu-devel] BSOD when using Hyper-V extensions with Win2k8R2 guests

2012-03-15 Thread Vadim Rozenfeld
On Thursday, March 15, 2012 09:14:39 PM Michael Roth wrote: Hi Vadim, I've been trying to use -cpu qemu64,hv_relaxed as a workaround for a CLOCK_WATCHDOG_TIMEOUT BSOD on Windows 2008 R2 guests when a vcpu's execution is delayed for an extended period. When I do this however I get the

Re: [Qemu-devel] BSOD when using Hyper-V extensions with Win2k8R2 guests

2012-03-15 Thread Anthony Liguori
On 03/15/2012 02:54 PM, Vadim Rozenfeld wrote: On Thursday, March 15, 2012 09:14:39 PM Michael Roth wrote: Hi Vadim, I've been trying to use -cpu qemu64,hv_relaxed as a workaround for a CLOCK_WATCHDOG_TIMEOUT BSOD on Windows 2008 R2 guests when a vcpu's execution is delayed for an extended

Re: [Qemu-devel] [PATCH] vga: disable default VGA if appropriate -device is used

2012-03-15 Thread Blue Swirl
On Thu, Mar 15, 2012 at 12:56, Paolo Bonzini pbonz...@redhat.com wrote: This is a partial revert of commits a369da5 (vga: improve VGA logic, committed 2012-01-22) and c5bd4f3 (vga: fix -nodefaults -device VGA, 2012-01-24) which broke command-line option parsing in different ways. Since commit

Re: [Qemu-devel] [PATCH 1/2] Isolation groups

2012-03-15 Thread Alex Williamson
On Wed, 2012-03-14 at 20:58 +1100, David Gibson wrote: On Tue, Mar 13, 2012 at 10:49:47AM -0600, Alex Williamson wrote: On Wed, 2012-03-14 at 01:33 +1100, David Gibson wrote: On Mon, Mar 12, 2012 at 04:32:54PM -0600, Alex Williamson wrote: +/* + * Add a device to an isolation group.

Re: [Qemu-devel] [PATCH v6 2/6] Introduce save_devices

2012-03-15 Thread Anthony Liguori
On 03/15/2012 01:19 PM, Stefano Stabellini wrote: - add an is_ram flag to SaveStateEntry; - register_savevm_live sets is_ram for live_savevm devices; - introduce a save_devices QAPI command that can be used to save the state of all devices, but not the RAM or the block devices of the VM.

Re: [Qemu-devel] [PULL] Malta patches

2012-03-15 Thread Stefan Weil
Am 25.02.2012 15:43, schrieb Stefan Weil: Hi Aurelien, could you please pull some Malta patches which I had sent in January? http://patchwork.ozlabs.org/patch/138394/ http://patchwork.ozlabs.org/patch/138392/ http://patchwork.ozlabs.org/patch/138393/ http://patchwork.ozlabs.org/patch/138391/

Re: [Qemu-devel] BSOD when using Hyper-V extensions with Win2k8R2 guests

2012-03-15 Thread Vadim Rozenfeld
On Thursday, March 15, 2012 10:00:11 PM Anthony Liguori wrote: On 03/15/2012 02:54 PM, Vadim Rozenfeld wrote: On Thursday, March 15, 2012 09:14:39 PM Michael Roth wrote: Hi Vadim, I've been trying to use -cpu qemu64,hv_relaxed as a workaround for a CLOCK_WATCHDOG_TIMEOUT BSOD on

Re: [Qemu-devel] [PATCH v6 2/6] Introduce save_devices

2012-03-15 Thread Luiz Capitulino
On Thu, 15 Mar 2012 15:16:15 -0500 Anthony Liguori anth...@codemonkey.ws wrote: On 03/15/2012 01:19 PM, Stefano Stabellini wrote: - add an is_ram flag to SaveStateEntry; - register_savevm_live sets is_ram for live_savevm devices; - introduce a save_devices QAPI command that can be used

[Qemu-devel] [PATCHv4 00/11] cleanup/consolidate iovec functions

2012-03-15 Thread Michael Tokarev
This is cleanup/consolidation of iovec-related low-level routines in qemu. The plan is to make library functions more understandable, consistent and useful, and to drop numerous implementations of the same thing. The patch changes prototypes of several iov and qiov functions to match each other,

[Qemu-devel] [PATCHv4 09/11] export iov_send_recv() and use it in iov_send() and iov_recv()

2012-03-15 Thread Michael Tokarev
Rename do_sendv_recvv() to iov_send_recv(), change its last arg (do_send) from int to bool, export it in iov.h, and made the two callers of it (iov_send() and iov_recv()) to be trivial #defines just adding 5th arg. iov_send_recv() will be used later. Signed-off-by: Michael Tokarev

[Qemu-devel] [PATCHv4 08/11] rename qemu_sendv to iov_send, change proto and move declarations to iov.h

2012-03-15 Thread Michael Tokarev
Rename arguments and use size_t for sizes instead of int, from int qemu_sendv(int sockfd, struct iovec *iov, int len, int iov_offset) to ssize_t iov_send(int sockfd, struct iovec *iov, size_t offset, size_t bytes) The main motivation was to make it clear that length and

Re: [Qemu-devel] [patch V4 2/4] FreeSCALE i.MX31 support: Timers

2012-03-15 Thread Peter Chubb
Peter == Peter Maydell peter.mayd...@linaro.org writes: Peter ...that would work better if I cc'd the people I meant to... Peter On 15 March 2012 17:08, Peter Maydell Peter peter.mayd...@linaro.org wrote: On 9 March 2012 03:27,  peter.ch...@nicta.com.au wrote: Implement the timers on the

[Qemu-devel] [PATCHv4 07/11] change qemu_iovec_to_buf() to match other to, from_buf functions

2012-03-15 Thread Michael Tokarev
It now allows specifying offset within qiov to start from and amount of bytes to copy. Actual implementation is just a call to iov_to_buf(). Signed-off-by: Michael Tokarev m...@tls.msk.ru --- block.c |2 +- block/iscsi.c |2 +- block/qcow.c |2 +- block/qcow2.c |2 +-

Re: [Qemu-devel] [patch V4 1/4] i.MX UART support

2012-03-15 Thread Peter Chubb
Thanks for your reviewing time and expertise Peter. It's much appreciated. May I add your Reviewed-By: line to the imx-serial patch too? The only change is CamelCasing the typedef. Peter C -- Dr Peter Chubb peter.chubb AT nicta.com.au

<    1   2   3   >