[Qemu-devel] Minor MMU fixes for PowerPC 40x emulation

2010-10-01 Thread John Clark
Hello, I found I had to make a few minor changes to the MMU code for the PowerPC 40x emulation to get NetBSD to run on a virtual PowerPC 405 core with qemu-system-ppcemb. The 'tlbre' instruction was not working, and permission checking for a TLB entry was not as strict as it should be. Diffs are i

[Qemu-devel] [Bug 532733] Re: apt/dpkg in qemu-system-arm hangs if a big task is installed

2010-10-01 Thread Ricardo Salveti
What fails is qemu-arm-static, as rootstock uses it internally to install the packages. -- apt/dpkg in qemu-system-arm hangs if a big task is installed https://bugs.launchpad.net/bugs/532733 You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU

[Qemu-devel] [Bug 532733] Re: apt/dpkg in qemu-system-arm hangs if a big task is installed

2010-10-01 Thread Dr. The Fugitive
This should help a little: I can confirm that this is a problem in rootstock, not just inside qemu or qemu/kvm. Reproducible on host system. System output from uname : u...@lucid:~/kvm$ uname -a Linux lucid 2.6.32-25-generic #44-Ubuntu SMP Fri Sep 17 20:05:27 UTC 2010 x86_64 GNU/Linux Rootstock

Re: [Qemu-devel] [PATCH v2] es1370: Fix compiler warnings for debug code

2010-10-01 Thread Markus Armbruster
Stefan Weil writes: > The patch fixes these gcc warnings: > > ./hw/es1370.c: In function ‘es1370_update_voices’: > ./hw/es1370.c:411: error: format ‘%d’ expects type ‘int’, but argument 3 has > type ‘size_t’ > ./hw/es1370.c: In function ‘es1370_writel’: > ./hw/es1370.c:579: error: format ‘%d’ ex

[Qemu-devel] [PATCH 13/23] QMP: Introduce query commands dispatch table

2010-10-01 Thread Luiz Capitulino
The new table is a copy of HMP's table, containing only QObject handlers. In the near future HMP will be making QMP calls and then we will be able to drop QObject handlers from HMP's table. >From now on, QMP and HMP have different query command dispatch tables. Signed-off-by: Luiz Capitulino --

[Qemu-devel] [Bug 652293] Re: Linux for SPARC does not work

2010-10-01 Thread blueswirl
** Also affects: openbios-sparc (Debian) Importance: Undecided Status: New ** Changed in: qemu Status: New => Invalid ** Changed in: qemu Assignee: (unassigned) => blueswirl (blauwirbel) -- Linux for SPARC does not work https://bugs.launchpad.net/bugs/652293 You received t

[Qemu-devel] [Bug 597402] Re: qemu does not call unlink() on temp files in snapshot mode

2010-10-01 Thread blueswirl
Does not seem to happen anymore: strace -etrace=unlink -esignal= -f ./qemu -drive file=/dev/null,snapshot=on unlink("/tmp/vl.jUbLWz")= 0 ** Changed in: qemu Status: New => Fix Committed -- qemu does not call unlink() on temp files in snapshot mode https://bugs.launchpad.

[Qemu-devel] [PATCH 18/23] Monitor: Rename monitor_handler_ported()

2010-10-01 Thread Luiz Capitulino
That name makes no sense anymore, as dispatch tables have been split, a better name is handler_is_qobject(), which really communicates the handler's type. Signed-off-by: Luiz Capitulino --- monitor.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/monitor.c b/monito

[Qemu-devel] [PATCH 19/23] Monitor: Rename monitor_handler_is_async()

2010-10-01 Thread Luiz Capitulino
Let's follow the convention introduced by the previous commit and call it handler_is_async(). Signed-off-by: Luiz Capitulino --- monitor.c | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/monitor.c b/monitor.c index b9dab78..7959504 100644 --- a/monitor.c +++ b/m

[Qemu-devel] [PATCH 15/23] QMP: Small cleanup in handle_qmp_command()

2010-10-01 Thread Luiz Capitulino
QMP has its own dispatch tables, we can now drop the following checks: o 'info' command: this command doesn't exist in QMP's dispatch table, the right thing will happen when it's issued by a client (ie. command not found error) o monitor_handler_ported(): all QMP handlers ar

[Qemu-devel] [PATCH 21/23] QMP: Introduce qmp_call_cmd()

2010-10-01 Thread Luiz Capitulino
Calls a QObject handler and emits the QMP response, also drops monitor_call_handler() which is now unused. Signed-off-by: Luiz Capitulino --- monitor.c | 39 +++ 1 files changed, 15 insertions(+), 24 deletions(-) diff --git a/monitor.c b/monitor.c index 03f

[Qemu-devel] [PATCH 10/23] Monitor: Introduce the qmp-commands.hx file

2010-10-01 Thread Luiz Capitulino
This file contains a copy of the following information from the qemu-monitor.hx file: o QObject handlers entries o QMP documentation (all SQMP/EQMP sections) Right now it's only used to generate the QMP docs in QMP/, but next commits will turn this into QMP's command dispatch table. It's

[Qemu-devel] [PATCH 23/23] QMP/README: Update QMP homepage address

2010-10-01 Thread Luiz Capitulino
Signed-off-by: Luiz Capitulino --- QMP/README |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/QMP/README b/QMP/README index 7e2b51a..80503f2 100644 --- a/QMP/README +++ b/QMP/README @@ -88,4 +88,4 @@ doing any code change. This is so because: Homepage -http:

[Qemu-devel] [PATCH 16/23] Monitor: Drop QMP info from the qemu-monitor.hx file

2010-10-01 Thread Luiz Capitulino
QMP has its own dispatch table and documentation file (qmp-commands.hx), we can now drop the following QMP specific info from qemu-monitor.hx: o SQMP/EQMP sections o The qmp_capabilities command o The query-commands command However, note that QObject handlers entries are not being rem

[Qemu-devel] [PATCH 11/23] QMP: Introduce qmp_find_cmd()

2010-10-01 Thread Luiz Capitulino
Next commit needs this new function: it will introduce the the QMP's command dispatch table and qmp_find_cmd() will be used to search on it. Signed-off-by: Luiz Capitulino --- monitor.c |7 ++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/monitor.c b/monitor.c index f08

[Qemu-devel] [PATCH 12/23] QMP: Introduce command dispatch table

2010-10-01 Thread Luiz Capitulino
Also update QMP functions to use it. The table is generated from the qmp-commands.hx file. >From now on, QMP and HMP have different command dispatch tables. Signed-off-by: Luiz Capitulino --- Makefile.target |7 +-- monitor.c | 11 +-- 2 files changed, 14 insertions(+),

[Qemu-devel] [PATCH 17/23] Monitor: Drop monitor_cmd_user_only()

2010-10-01 Thread Luiz Capitulino
This function was only needed when QMP and HMP were sharing dispatch tables, this is no longer true so just drop it. Signed-off-by: Luiz Capitulino --- monitor.c |5 - monitor.h |1 - 2 files changed, 0 insertions(+), 6 deletions(-) diff --git a/monitor.c b/monitor.c index b43277e..

[Qemu-devel] [PATCH 14/23] QMP: Simplify do_info_commands()

2010-10-01 Thread Luiz Capitulino
We now iterate over QMP's dispatch tables, no need to check for QMP-only handlers anymore. Signed-off-by: Luiz Capitulino --- monitor.c | 13 - 1 files changed, 4 insertions(+), 9 deletions(-) diff --git a/monitor.c b/monitor.c index bf5da50..612ee56 100644 --- a/monitor.c +++ b/m

[Qemu-devel] [PATCH 08/23] Monitor: Drop is_async_return()

2010-10-01 Thread Luiz Capitulino
If I understood it correcty, the is_async_return() logic was only used to prevent QMP from issuing duplicated success responses for asynchronous handlers. However, QMP doesn't use do_info() anymore so this is dead logic and (hopefully) can be safely dropped. Signed-off-by: Luiz Capitulino --- m

[Qemu-devel] [PATCH 20/23] Monitor: Directly call QObject handlers

2010-10-01 Thread Luiz Capitulino
This avoids handle_user_command() calling monitor_call_handler(), which is currently shared with QMP. Signed-off-by: Luiz Capitulino --- monitor.c | 10 +- 1 files changed, 9 insertions(+), 1 deletions(-) diff --git a/monitor.c b/monitor.c index 7959504..03f3c18 100644 --- a/monitor.c

[Qemu-devel] [PATCH 05/23] QMP: handle_qmp_command(): Move 'cmd' sanity check

2010-10-01 Thread Luiz Capitulino
Next commit will change how query commands are handled in a way that the 'cmd' sanity check is also going to be needed for query commands handling. Let's move it out of the else body then. Signed-off-by: Luiz Capitulino --- monitor.c | 10 +- 1 files changed, 5 insertions(+), 5 deleti

[Qemu-devel] [PATCH 03/23] disable guest-provided stats on "info balloon" command

2010-10-01 Thread Luiz Capitulino
From: Eduardo Habkost The addition of memory stats reporting to the virtio balloon causes the 'info balloon' command to become asynchronous. This is a regression because in some cases it can hang the user monitor. This is an alternative to Adam Litke's patch. Adam's patch disabled the correspon

[Qemu-devel] [PATCH 04/23] Monitor: Introduce search_dispatch_table()

2010-10-01 Thread Luiz Capitulino
It's a generic version of monitor_find_command() which searches the dispatch table passed as an argument. Future commits will introduce new dispatch tables, so we need common code to search them. Signed-off-by: Luiz Capitulino --- monitor.c | 10 -- 1 files changed, 8 insertions(+), 2

[Qemu-devel] [PATCH 07/23] Monitor: Drop QMP bits from do_info()

2010-10-01 Thread Luiz Capitulino
As of last commit, QMP doesn't use do_info() anymore. Simplify it. Signed-off-by: Luiz Capitulino --- monitor.c | 39 --- 1 files changed, 8 insertions(+), 31 deletions(-) diff --git a/monitor.c b/monitor.c index ff65f38..4fc0ad3 100644 --- a/monitor.c +++

[Qemu-devel] [PATCH 09/23] Monitor: Convert do_info() back to HMP

2010-10-01 Thread Luiz Capitulino
This is a HMP specific handler, it makes no sense to have it under QMP. Signed-off-by: Luiz Capitulino --- monitor.c |5 ++--- qemu-monitor.hx |3 +-- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/monitor.c b/monitor.c index 7f2af49..f0854c4 100644 --- a/monitor.c

[Qemu-devel] [PATCH 02/23] Add option to turn on JSON pretty printing in monitor

2010-10-01 Thread Luiz Capitulino
From: Daniel P. Berrange Expaned '-mon' arg to allow a 'pretty=on' flag. This makes the monitor pretty print its replies to easy human debugging / reading Signed-off-by: Daniel P. Berrange Signed-off-by: Luiz Capitulino --- monitor.c |5 - monitor.h |1 + qemu-config.c |

[Qemu-devel] [PATCH 06/23] QMP: Don't use do_info()

2010-10-01 Thread Luiz Capitulino
Since its inception, QMP has been using HMP's do_info() function to run query commands. This was a bad choice, as it made do_info() more complex and contributed to couple QMP and HMP. This commit fixes that by doing the following changes: 1. Introduce qmp_find_query_cmd() and use it to directl

[Qemu-devel] [PATCH 01/23] Add support for JSON pretty printing

2010-10-01 Thread Luiz Capitulino
From: Daniel P. Berrange The monitor does not pretty-print JSON output, so that everything will be on a single line reply. When JSON docs get large this is quite unpleasant to read. For the future command line capabilities query ability, huge JSON docs will be available. This needs the ability to

[Qemu-devel] [PULL 00/23]: Monitor queue

2010-10-01 Thread Luiz Capitulino
Hi Anthony, The following patches have been sent to the list and look good to me, I've also tested them. The changes (since 2c50e26efdb7e405ecacf89e6a57a60179226dd8) are available in the following repository: git://repo.or.cz/qemu/qmp-unstable.git for-anthony Daniel P. Berrange (2): A

[Qemu-devel] [PATCH] fix '/' and '|' on russian keymap

2010-10-01 Thread Eduardo Habkost
Patch from Oleg L. Sadov. Removes broken 'slash' and 'bar' definitions (that were duplicated) from the 'ru' keymap. I can't test it myself, but the the fix was reported to be working, at: https://bugzilla.redhat.com/show_bug.cgi?id=580637 Signed-off-by: Eduardo Habkost --- pc-bios/keymaps/ru |

[Qemu-devel] [PATCH v2] es1370: Fix compiler warnings for debug code

2010-10-01 Thread Stefan Weil
The patch fixes these gcc warnings: ./hw/es1370.c: In function ‘es1370_update_voices’: ./hw/es1370.c:411: error: format ‘%d’ expects type ‘int’, but argument 3 has type ‘size_t’ ./hw/es1370.c: In function ‘es1370_writel’: ./hw/es1370.c:579: error: format ‘%d’ expects type ‘int’, but argument 3 ha

[Qemu-devel] [Bug 592028] Re: sparc32plus user test failed

2010-10-01 Thread blueswirl
Cross-OS emulation is not supported for user emulators. ** Changed in: qemu Status: New => Invalid ** Changed in: qemu Assignee: (unassigned) => blueswirl (blauwirbel) -- sparc32plus user test failed https://bugs.launchpad.net/bugs/592028 You received this bug notification because y

[Qemu-devel] [Bug 568053] Re: requires MSYS coreutils ext sub-package to build on Windows

2010-10-01 Thread blueswirl
pc-bios/optionrom/signrom.sh uses 'dd'. ** Changed in: qemu Status: New => Confirmed -- requires MSYS coreutils ext sub-package to build on Windows https://bugs.launchpad.net/bugs/568053 You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to

[Qemu-devel] [Bug 423910] Re: openbios-sparc has no installation candidate

2010-10-01 Thread blueswirl
Packaging problem in Ubuntu, not upstream. ** Also affects: qemu-kvm (Ubuntu) Importance: Undecided Status: New ** Changed in: qemu Status: New => Invalid -- openbios-sparc has no installation candidate https://bugs.launchpad.net/bugs/423910 You received this bug notification b

Re: [Qemu-devel] QEMU throughput is down with SMP

2010-10-01 Thread Stefan Hajnoczi
On Fri, Oct 1, 2010 at 4:04 PM, Venkateswararao Jujjuri (JV) wrote: > On 10/1/2010 6:38 AM, Ryan Harper wrote: >> >> * Stefan Hajnoczi  [2010-10-01 03:48]: >>> >>> On Thu, Sep 30, 2010 at 8:19 PM, Venkateswararao Jujjuri (JV) >>>  wrote: On 9/30/2010 2:13 AM, Stefan Hajnoczi wrote:

Re: [Qemu-devel] QEMU throughput is down with SMP

2010-10-01 Thread Venkateswararao Jujjuri (JV)
On 10/1/2010 6:38 AM, Ryan Harper wrote: * Stefan Hajnoczi [2010-10-01 03:48]: On Thu, Sep 30, 2010 at 8:19 PM, Venkateswararao Jujjuri (JV) wrote: On 9/30/2010 2:13 AM, Stefan Hajnoczi wrote: On Thu, Sep 30, 2010 at 1:50 AM, Venkateswararao Jujjuri (JV) wrote: Code: Mainline QEMU (g

[Qemu-devel] [Bug 652293] Re: Linux for SPARC does not work

2010-10-01 Thread Roman Franchuk
> It looks like the version of OpenBIOS in Debian sqeeze is buggy. Thank you very much. I downloaded version from your link, and it works. -- Linux for SPARC does not work https://bugs.launchpad.net/bugs/652293 You received this bug notification because you are a member of qemu- devel-ml, which

[Qemu-devel] Re: [PATCH 10/19] QMP: Introduce query commands dispatch table

2010-10-01 Thread Anthony Liguori
On 10/01/2010 08:49 AM, Luiz Capitulino wrote: On Fri, 01 Oct 2010 08:31:11 -0500 Anthony Liguori wrote: On 09/30/2010 03:56 PM, Luiz Capitulino wrote: The new table is a copy of HMP's table, containing only QObject handlers. In the near future HMP will be making QMP calls and then

[Qemu-devel] Re: [PATCH 10/19] QMP: Introduce query commands dispatch table

2010-10-01 Thread Luiz Capitulino
On Fri, 01 Oct 2010 08:31:11 -0500 Anthony Liguori wrote: > On 09/30/2010 03:56 PM, Luiz Capitulino wrote: > > The new table is a copy of HMP's table, containing only QObject > > handlers. > > > > In the near future HMP will be making QMP calls and then we will > > be able to drop QObject handler

Re: [Qemu-devel] QEMU throughput is down with SMP

2010-10-01 Thread Ryan Harper
* Stefan Hajnoczi [2010-10-01 03:48]: > On Thu, Sep 30, 2010 at 8:19 PM, Venkateswararao Jujjuri (JV) > wrote: > > On 9/30/2010 2:13 AM, Stefan Hajnoczi wrote: > >> > >> On Thu, Sep 30, 2010 at 1:50 AM, Venkateswararao Jujjuri (JV) > >>  wrote: > >>> > >>> Code: Mainline QEMU (git://git.qemu.org

[Qemu-devel] Re: [PATCH 10/19] QMP: Introduce query commands dispatch table

2010-10-01 Thread Anthony Liguori
On 09/30/2010 03:56 PM, Luiz Capitulino wrote: The new table is a copy of HMP's table, containing only QObject handlers. In the near future HMP will be making QMP calls and then we will be able to drop QObject handlers from HMP's table. > From now on, QMP and HMP have different query command di

Re: [Qemu-devel] QEMU throughput is down with SMP

2010-10-01 Thread Stefan Hajnoczi
On Thu, Sep 30, 2010 at 8:19 PM, Venkateswararao Jujjuri (JV) wrote: > On 9/30/2010 2:13 AM, Stefan Hajnoczi wrote: >> >> On Thu, Sep 30, 2010 at 1:50 AM, Venkateswararao Jujjuri (JV) >>  wrote: >>> >>> Code: Mainline QEMU (git://git.qemu.org/qemu.git) >>> Machine: LS21 blade. >>> Disk: Local dis

Re: [Qemu-devel] [PATCH v2 00/19] Monitor: split HMP and QMP dispatch tables

2010-10-01 Thread Markus Armbruster
Luiz Capitulino writes: > This is a respin with the following small changes: > > v1 - v2: > o Renamed qemu-monitor-qmp.hx to qmp-commands.hx > o Added a new patch which renames qemu-monitor.hx to hmp-commands.hx > o Other minor changes ACK