[Qemu-devel] [Bug 988291] Re: virsh -c qemu+ssh://root@source/system migrate --domain client --desturi qemu+ssh://root@destination/system fails

2012-04-25 Thread Serge Hallyn
O1 is an uptodate oneiric system (qemu-kvm 0.14.1) P1 is an uptodate precise system (qemu-kvm 1.0). Same thing happens with upstream git HEAD from qemu /mnt is an nfs mount shared between the two On O1: root@server-8987:/mnt# kvm -version QEMU emulator version 0.14.1 (qemu-kvm-0.14.1), Copyright

[Qemu-devel] [Bug 988291] Re: virsh -c qemu+ssh://root@source/system migrate --domain client --desturi qemu+ssh://root@destination/system fails

2012-04-25 Thread Serge Hallyn
Lowering the priority. While I feel it is very unfortunate, shutting down the vm, migrating, and restarting *is* a workaround, and it sounds like the bug itself will become 'wontfix' ** Changed in: qemu-kvm (Ubuntu) Importance: Medium => Low ** Changed in: libvirt (Ubuntu) Importance: Medi

[Qemu-devel] [Bug 988291] Re: virsh -c qemu+ssh://root@source/system migrate --domain client --desturi qemu+ssh://root@destination/system fails

2012-04-25 Thread Serge Hallyn
Note this is migrating a vm from qemu-kvm-0.14.1 to qemu-kvm-1.0 and to upstream qemu.git. ** Also affects: qemu Importance: Undecided Status: New -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/b

[Qemu-devel] [Bug 988291] Re: virsh -c qemu+ssh://root@source/system migrate --domain client --desturi qemu+ssh://root@destination/system fails

2012-04-25 Thread Serge Hallyn
Going by http://lists.nongnu.org/archive/html/qemu-devel/2012-03/msg05001.html it looks like upstream simply does not support live migration between versions 0.15 (and older) and 1.0. ** Changed in: libvirt (Ubuntu) Importance: High => Medium ** Changed in: qemu-kvm (Ubuntu) Importance: H

[Qemu-devel] How to build realview-pbx-a9 kernel/rootfs for QEMU by using buildroot?

2012-04-25 Thread 陳韋任
Hi all, I am trying to use buildroot to build a kernel/rootfs for realview-pbx-a9, then use QEMU to emulate realview-pbx-a9. However I have trouble with building a working image. buildroot povides some default config for QEMU but not for realview-pbx-a9, so I borrow qemu_arm_versatile_defconfig

Re: [Qemu-devel] chatty qxl vga mode

2012-04-25 Thread Gerd Hoffmann
On 04/25/12 18:14, Jan Kiszka wrote: > Hi, > > just played with qxl and noticed quite a lot of debug output on the > local console: > > do_spice_init: starting 0.10.1 > spice_server_add_interface: SPICE_INTERFACE_QXL > red_worker_main: begin > display_channel_create: create display channel > curs

[Qemu-devel] [Bug 965867] Re: 9p virtual file system on qemu slow

2012-04-25 Thread M. Mohan Kumar
Hi Max, Could you try passing msize=262144 for 9p mount point and post the results? Host: [root@llm116 media]# ls -lhas file 1.1G -rw-r--r-- 1 root root 1.0G Apr 26 11:05 file [root@llm116 media]# dd if=/dev/zero of=file bs=1M count=1024 1024+0 records in 1024+0 records out 1073741824 bytes (1.1

[Qemu-devel] [PATCH 1/2] pseries: Implement automatic PAPR VIO address allocation

2012-04-25 Thread David Gibson
PAPR virtual IO (VIO) devices require a unique, but otherwise arbitrary, "address" used as a token to the hypercalls which manipulate them. Currently the pseries machine code does an ok job of allocating these addresses when the legacy -net nic / -serial and so forth options are used but will fail

[Qemu-devel] [PATCH 2/2] pseries: Use the same interrupt swizzling for host bridges as p2p bridges

2012-04-25 Thread David Gibson
Currently the pseries PCI code uses a somewhat strange scheme of PCI irq allocation - one per slot up to a maximum that's greater than the usual 4. This scheme more or less worked, because we were able to tell the guest the irq mapping in the device tree, however it's a bit odd and may break assump

[Qemu-devel] [PATCH 0/2] Outstanding pseries updates

2012-04-25 Thread David Gibson
Hi Alex, Here are a couple of patches that I sent to Andreas while you were away, but he didn't feel quite confident to review and commit at the time. The VIO address allocation patch, in particular, helps with a bunch of libvirt problems on Power, and it has conflicts with my IOMMU series which

Re: [Qemu-devel] [PATCH v3 1/4] SSI: Built in multiple device support

2012-04-25 Thread Peter Crosthwaite
On Wed, Apr 25, 2012 at 2:41 AM, Peter Maydell wrote: > On 20 April 2012 03:12, Peter A. G. Crosthwaite > wrote: >> Added support for multiple devices attached to a single SSI bus (Previously >> SSI masters with multiple slaves were emulated as multiple point to point SSI >> busses) > >>  static

Re: [Qemu-devel] [SeaBIOS] [RFC PATCH 9/9] enable memory devices if e820 entry is present

2012-04-25 Thread Wen Congyang
At 04/19/2012 10:08 PM, Vasilis Liaskovitis Wrote: > On a reboot, seabios regenerates srat/ssdt objects. If a valid e820 entry is > found spanning the whole address range of a hotplug memory device, the > device > will be enabled. This ensures persistency of hotplugged memory slots across > V

[Qemu-devel] [PATCH] slirp: don't use "smb ports = 0" option

2012-04-25 Thread Nikolaus Rath
Hello, The "smb ports = 0" option causes recent samba versions to crash. It was introduced in commit 15ef3e with log message "Samba 3 support". However, a value of 0 has never been officially supported by smb and is also not necessary: if stdin is a socket, smb does not try to listen on any po

[Qemu-devel] [PATCH v2 4/4] softfloat: Replace int16 type with int_fast16_t

2012-04-25 Thread Andreas Färber
From: Andreas Färber Based on the following Coccinelle patch: @@ typedef int16, int_fast16_t; @@ -int16 +int_fast16_t Avoids a workaround for AIX. Add typedef for pre-10 Solaris. Signed-off-by: Andreas Färber Cc: malc Cc: Ben Taylor --- fpu/softfloat-macros.h | 18 fpu/softfloa

[Qemu-devel] [PATCH 02/16] qemu-option: parse_option_number(): use error_set()

2012-04-25 Thread Luiz Capitulino
Note that qemu_opt_parse() callers still expect automatic error reporting with QError, so qemu_opts_parse() calls qerror_report_err() to keep the same semantics. Signed-off-by: Luiz Capitulino --- qemu-option.c | 26 +++--- 1 file changed, 19 insertions(+), 7 deletions(-)

[Qemu-devel] [PATCH v2 3/4] softfloat: Replace uint16 type with uint_fast16_t

2012-04-25 Thread Andreas Färber
From: Andreas Färber Based on the following Coccinelle patch: @@ typedef uint16, uint_fast16_t; @@ -uint16 +uint_fast16_t Fixes the build of the Cocoa frontend on Mac OS X and avoids a workaround for AIX. For pre-10 Solaris include osdep.h. Reported-by: Pavel Borzenkov Reported-by: Rui Carmo

[Qemu-devel] [PATCH v2 1/4] softfloat: Fix mixups of int and int16

2012-04-25 Thread Andreas Färber
From: Andreas Färber normalizeFloat{32,64}Subnormal() expect the exponent as int16, not int. This went unnoticed since int16 and uint16 were both typedef'ed to int. Signed-off-by: Andreas Färber Reviewed-by: Peter Maydell --- fpu/softfloat.c |4 ++-- 1 files changed, 2 insertions(+), 2 de

[Qemu-devel] [PATCH v2 2/4] target-mips: Move definition of uint_fast{8, 16}_t to osdep.h

2012-04-25 Thread Andreas Färber
From: Andreas Färber osdep.h is included via qemu-common.h. Prepares for use of [u]int_fast*_t types in softfloat code. Signed-off-by: Andreas Färber Cc: Ben Taylor Cc: Aurélien Jarno Cc: Peter Maydell --- osdep.h |6 ++ target-mips/cpu.h |7 --- 2 files changed,

[Qemu-devel] [PATCH v2 0/4] softfloat: Fix build on Darwin (uint16)

2012-04-25 Thread Andreas Färber
Hello, Since shortly before 1.0 the build on Mac OS X host is broken due to a type conflict for uint16 between SoftFloat and system headers. The conflict stems from SoftFloat using int for uint16 (i.e., 32 bits) whereas Apple wants 16 bits, and no correct workaround was found in time. Since these

[Qemu-devel] [PATCH 08/16] qemu-option: introduce qemu_opt_set_err()

2012-04-25 Thread Luiz Capitulino
This is like qemu_opt_set(), except that it takes an Error argument. This new function allows for a incremental conversion of code using qemu_opt_set(). Signed-off-by: Luiz Capitulino --- qemu-option.c |6 ++ qemu-option.h |2 ++ 2 files changed, 8 insertions(+) diff --git a/qemu-o

[Qemu-devel] [PATCH 16/16] qapi: convert netdev_del

2012-04-25 Thread Luiz Capitulino
Signed-off-by: Anthony Liguori Signed-off-by: Luiz Capitulino --- hmp-commands.hx |3 +-- hmp.c|9 + hmp.h|1 + net.c| 11 +-- net.h|1 - qapi-schema.json | 14 ++ qmp-commands.hx |5 + 7 fi

[Qemu-devel] [PATCH 04/16] qemu-option: parse_option_size(): use error_set()

2012-04-25 Thread Luiz Capitulino
Signed-off-by: Luiz Capitulino --- qemu-option.c | 20 +++- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/qemu-option.c b/qemu-option.c index a8b50af..61354af 100644 --- a/qemu-option.c +++ b/qemu-option.c @@ -203,7 +203,8 @@ static void parse_option_number(cons

[Qemu-devel] [PATCH 03/16] qemu-option: parse_option_bool(): use error_set()

2012-04-25 Thread Luiz Capitulino
Note that set_option_parameter() callers still expect automatic error reporting with QError, so set_option_parameter() calls qerror_report_err() to keep the same semantics. Signed-off-by: Luiz Capitulino --- qemu-option.c | 22 +++--- 1 file changed, 15 insertions(+), 7 deletio

[Qemu-devel] [PATCH 06/16] qemu-option: qemu_opts_validate(): use error_set()

2012-04-25 Thread Luiz Capitulino
net_client_init() propagates the error up by calling qerror_report_err(), because its users expect QError semantics. Signed-off-by: Luiz Capitulino --- net.c |6 +- qemu-option.c | 13 + qemu-option.h |2 +- 3 files changed, 11 insertions(+), 10 deletions(-) di

[Qemu-devel] [PATCH 01/16] qemu-option: qemu_opts_create(): use error_set()

2012-04-25 Thread Luiz Capitulino
This commit converts qemu_opts_create() from qerror_report() to error_set(). Currently, most calls to qemu_opts_create() can't fail, so most callers don't need any changes. The two cases where code checks for qemu_opts_create() erros are: 1. Initialization code in vl.c. All of them print their

[Qemu-devel] [PATCH 05/16] qemu-option: qemu_opt_parse(): use error_set()

2012-04-25 Thread Luiz Capitulino
The functions opt_set() and qemu_opts_validate() both call qemu_opt_parse(), but their callers expect QError semantics. Thus, both functions call qerro_report_err() to keep the expected semantics. Signed-off-by: Luiz Capitulino --- qemu-option.c | 36 +--- 1 fil

[Qemu-devel] [PATCH 13/16] net: purge the monitor object from all init functions

2012-04-25 Thread Luiz Capitulino
The only backend that really uses it is the socket one, which calls monitor_get_fd(). But it can use 'cur_mon' instead. Signed-off-by: Luiz Capitulino --- hw/pci-hotplug.c |2 +- hw/usb/dev-network.c |2 +- net.c| 18 +++--- net.h|2 +

[Qemu-devel] [PATCH] xtensa: remove unnecessary include of dyngen-exec.h

2012-04-25 Thread Peter Portante
--- xtensa-semi.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/xtensa-semi.c b/xtensa-semi.c index b7c8c34..1c8a19e 100644 --- a/xtensa-semi.c +++ b/xtensa-semi.c @@ -30,7 +30,6 @@ #include #include #include "cpu.h" -#include "dyngen-exec.h" #include "helper.h" #

[Qemu-devel] [PATCH 11/16] qemu-config: find_list(): use error_set()

2012-04-25 Thread Luiz Capitulino
Note that qemu_find_opts() and qemu_config_parse() need to call error_report() to maintain their semantics on error. Signed-off-by: Luiz Capitulino --- qemu-config.c | 32 +--- 1 file changed, 25 insertions(+), 7 deletions(-) diff --git a/qemu-config.c b/qemu-confi

[Qemu-devel] [PATCH 15/16] qapi: convert netdev_add

2012-04-25 Thread Luiz Capitulino
This is not a full QAPI conversion, but an intermediate step. In essence, do_netdev_add() is split into three functions: 1. netdev_add(): performs the actual work. This function is fully converted to Error (thus, it's "qapi-friendly") 2. qmp_netdev_add(): the QMP front-end for netdev_add()

[Qemu-devel] [PATCH 10/16] qerror: introduce QERR_INVALID_OPTION_GROUP

2012-04-25 Thread Luiz Capitulino
Signed-off-by: Luiz Capitulino --- qerror.c |4 qerror.h |3 +++ 2 files changed, 7 insertions(+) diff --git a/qerror.c b/qerror.c index 96fbe71..5d92096 100644 --- a/qerror.c +++ b/qerror.c @@ -156,6 +156,10 @@ static const QErrorStringTable qerror_table[] = { .desc =

Re: [Qemu-devel] [PATCH] xtensa, hw: remove unnecessary include of dyngen-exec.h

2012-04-25 Thread Peter Portante
On 04/25/2012 04:56 PM, Max Filippov wrote: On 04/26/2012 12:31 AM, Peter Portante wrote: On 04/25/2012 04:07 PM, Max Filippov wrote: On 04/25/2012 10:47 PM, Peter Portante wrote: Signed-off-by: Peter Portante --- hw/spapr_hcall.c | 1 - xtensa-semi.c | 1 - 2 files changed, 0 insertions(+), 2 d

[Qemu-devel] [PATCH] Fix SPI SD emulation

2012-04-25 Thread Paul Brook
When in SPI mode, we give a bogus response to CMD8 (part of the SD physical spec v2). Fixing this also makes most drivers also issue CMD58. Despite some skeleton code in hw/ssi-sd.c this isn't actually implemented in hw/sd.h. CMD58 is valid in both idle and active states, so the hardcoded status

[Qemu-devel] [PATCH 12/16] qemu-config: introduce qemu_find_opts_err()

2012-04-25 Thread Luiz Capitulino
This is like qemu_find_opts(), except that it takes an Error argument. This new function allows for a incremental conversion of code using qemu_find_opts(). Signed-off-by: Luiz Capitulino --- qemu-config.c |5 + qemu-config.h |3 +++ 2 files changed, 8 insertions(+) diff --git a/qe

[Qemu-devel] [PATCH 07/16] qemu-option: opt_set(): use error_set()

2012-04-25 Thread Luiz Capitulino
The functions qemu_opt_set() and opts_do_parse() both call opt_set(), but their callers expect QError semantics. Thus, both functions call qerro_report_err() to keep the expected semantics. Signed-off-by: Luiz Capitulino --- qemu-option.c | 31 --- 1 file changed, 2

[Qemu-devel] [PATCH 14/16] net: net_client_init(): use error_set()

2012-04-25 Thread Luiz Capitulino
Callers are changed to use qerror_report_err() to keep their QError semantics. Signed-off-by: Luiz Capitulino --- hw/pci-hotplug.c |8 ++-- hw/usb/dev-network.c |7 +-- net.c| 53 +++--- net.h|2

Re: [Qemu-devel] [PATCH v2 14/14] QemuOpts: correctly skip escaped , id=

2012-04-25 Thread Anthony Liguori
On 04/11/2012 04:34 PM, Paolo Bonzini wrote: QemuOpts would parse incorrectly something that included ",,id=" (with an escaped comma) in a parameter value. Fix this by using get_param_value. Signed-off-by: Paolo Bonzini This breaks qemu-test. Specifically: out=`hmp netdev_add user,id=foo`

Re: [Qemu-devel] [PATCH v2 12/14] cmdline: convert -acpitable to QemuOpts

2012-04-25 Thread Anthony Liguori
On 04/11/2012 04:34 PM, Paolo Bonzini wrote: Signed-off-by: Paolo Bonzini --- arch_init.c | 11 ++ arch_init.h |1 - hw/acpi.c | 96 +++-- hw/pc.h |2 +- qemu-config.c | 54

Re: [Qemu-devel] [PATCH] xtensa, hw: remove unnecessary include of dyngen-exec.h

2012-04-25 Thread Max Filippov
On 04/26/2012 12:31 AM, Peter Portante wrote: On 04/25/2012 04:07 PM, Max Filippov wrote: On 04/25/2012 10:47 PM, Peter Portante wrote: Signed-off-by: Peter Portante --- hw/spapr_hcall.c | 1 - xtensa-semi.c | 1 - 2 files changed, 0 insertions(+), 2 deletions(-) diff --git a/hw/spapr_hcall.c b/

Re: [Qemu-devel] [PATCH] xtensa, hw: remove unnecessary include of dyngen-exec.h

2012-04-25 Thread Peter Portante
On 04/25/2012 04:30 PM, Andreas Färber wrote: Am 25.04.2012 20:47, schrieb Peter Portante: Signed-off-by: Peter Portante --- hw/spapr_hcall.c |1 - xtensa-semi.c|1 - 2 files changed, 0 insertions(+), 2 deletions(-) diff --git a/hw/spapr_hcall.c b/hw/spapr_hcall.c index 94bb504.

[Qemu-devel] [PATCH 09/16] qemu-option: qemu_opts_from_qdict(): use error_set()

2012-04-25 Thread Luiz Capitulino
do_device_add() and do_netdev_add() call qerror_report_err() to maintain their QError semantics. Signed-off-by: Luiz Capitulino --- hw/qdev-monitor.c |7 +-- net.c |5 - qemu-option.c | 31 --- qemu-option.h |3 ++- 4 files ch

[Qemu-devel] [PATCH v3 00/16]: qapi: convert netdev_add & netdev_del

2012-04-25 Thread Luiz Capitulino
v3 - fix patch split (a hunk pertaining to patch 09/16 was in patch 15/16) blockdev.c |2 +- hmp-commands.hx |6 +- hmp.c| 30 + hmp.h|2 + hw/pci-hotplug.c |8 ++- hw/qdev-monitor.c|7 +- hw/usb/dev-network.c |

Re: [Qemu-devel] [PATCH v4 0/5] block: add optional 'speed' parameter to block-stream

2012-04-25 Thread Luiz Capitulino
On Wed, 25 Apr 2012 16:50:59 +0100 Stefan Hajnoczi wrote: > Eric Blake raised concerns about the inability to start block jobs with a > speed > limit. Current the user needs to follow up the block-stream command with > block-job-set-speed. There is a window of time while the new block job is >

Re: [Qemu-devel] [PATCH] xtensa, hw: remove unnecessary include of dyngen-exec.h

2012-04-25 Thread Peter Portante
On 04/25/2012 04:07 PM, Max Filippov wrote: On 04/25/2012 10:47 PM, Peter Portante wrote: Signed-off-by: Peter Portante --- hw/spapr_hcall.c |1 - xtensa-semi.c|1 - 2 files changed, 0 insertions(+), 2 deletions(-) diff --git a/hw/spapr_hcall.c b/hw/spapr_hcall.c index 94bb504..

Re: [Qemu-devel] [PATCH] xtensa, hw: remove unnecessary include of dyngen-exec.h

2012-04-25 Thread Andreas Färber
Am 25.04.2012 20:47, schrieb Peter Portante: > Signed-off-by: Peter Portante > --- > hw/spapr_hcall.c |1 - > xtensa-semi.c|1 - > 2 files changed, 0 insertions(+), 2 deletions(-) > > diff --git a/hw/spapr_hcall.c b/hw/spapr_hcall.c > index 94bb504..88c1fab 100644 > --- a/hw/spapr_hc

Re: [Qemu-devel] [PATCH] xtensa, hw: remove unnecessary include of dyngen-exec.h

2012-04-25 Thread Max Filippov
On 04/25/2012 10:47 PM, Peter Portante wrote: Signed-off-by: Peter Portante --- hw/spapr_hcall.c |1 - xtensa-semi.c|1 - 2 files changed, 0 insertions(+), 2 deletions(-) diff --git a/hw/spapr_hcall.c b/hw/spapr_hcall.c index 94bb504..88c1fab 100644 --- a/hw/spapr_hcall.c +++ b/h

Re: [Qemu-devel] [PATCH] configure: Virtfs doesn't require libcap.

2012-04-25 Thread Anthony Liguori
On 04/22/2012 05:16 AM, Kusanagi Kouichi wrote: Only proxy helper does. Signed-off-by: Kusanagi Kouichi This broke the build: http://spunk.home.kraxel.org/bb/builders/ubuntu-default/builds/52 Since the improve seems to be purely cosmetic, I'm going to revert this change. Regards, Anthony L

Re: [Qemu-devel] [PATCH 1/1 v3] Allow machines to configure the QEMU_VERSION that's exposed via hardware

2012-04-25 Thread Anthony Liguori
On 04/13/2012 02:16 PM, Crístian Viana wrote: Based on the following conversation: http://mid.gmane.org/4f69f05b.5010...@codemonkey.ws Which reminds me - qemu sticks the release version in guest visible places like CPU version. This is wrong and causes windows guests to print messages about dr

[Qemu-devel] [PATCH] xtensa, hw: remove unnecessary include of dyngen-exec.h

2012-04-25 Thread Peter Portante
Signed-off-by: Peter Portante --- hw/spapr_hcall.c |1 - xtensa-semi.c|1 - 2 files changed, 0 insertions(+), 2 deletions(-) diff --git a/hw/spapr_hcall.c b/hw/spapr_hcall.c index 94bb504..88c1fab 100644 --- a/hw/spapr_hcall.c +++ b/hw/spapr_hcall.c @@ -1,6 +1,5 @@ #include "sysemu

[Qemu-devel] [Bug 988125] Re: Generated smb.conf needs to declare state directory

2012-04-25 Thread Nikolaus Rath
This option is available since 3.4.0. I just sent a patch to qemu-devel. Am I going to get some confirmation when it is applied? I'm unfamiliar with the development model. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs

Re: [Qemu-devel] [PATCH v3 1/4] SSI: Built in multiple device support

2012-04-25 Thread Paul Brook
> Im happy to spend the 10 mins updating stellaris.c accordingly, but is > someone sitting on a binary package and brief instructions or some > such to regression test it? Do you of this machine have some sort of > kernel image handy? I've attached a tarball with some test binaries. They're built

[Qemu-devel] [PATCH 3/3] hw/exynos4210.c: set chipid_and_omr array size to TARGET_PAGE_SIZE

2012-04-25 Thread Igor Mitsyanko
chipid_mem is initialized with memory_region_init_ram_ptr() function with chipid_and_omr array as buffer argument. During initialization memory region size is aligned to page size, but size of static array chipid_and_omr is less then TARGET_PAGE_SIZE. This could result in errors in some cases, s

[Qemu-devel] [PATCH 12/12] Makefile: Simplify compilation of target-*/cpu.c

2012-04-25 Thread Andreas Färber
All targets except for ppc now have a standalone cpu.c file. Signed-off-by: Andreas Färber --- Makefile.target | 16 +--- 1 files changed, 1 insertions(+), 15 deletions(-) diff --git a/Makefile.target b/Makefile.target index f7b2e71..364aea2 100644 --- a/Makefile.target +++ b/Make

[Qemu-devel] [PATCH 04/12] target-sh4: QOM'ify CPU reset

2012-04-25 Thread Andreas Färber
Move code from cpu_state_reset() to QOM superh_cpu_reset(). Signed-off-by: Andreas Färber Reviewed-by: Peter Maydell --- target-sh4/cpu.c | 21 - target-sh4/translate.c | 22 ++ 2 files changed, 22 insertions(+), 21 deletions(-) diff --git a/ta

[Qemu-devel] [PATCH 05/12] target-sh4: Start QOM'ifying CPU init

2012-04-25 Thread Andreas Färber
Move code from cpu_sh4_init() into a QOM initfn. Signed-off-by: Andreas Färber --- target-sh4/cpu.c | 11 +++ target-sh4/translate.c |2 -- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/target-sh4/cpu.c b/target-sh4/cpu.c index 84d4672..a1a177f 100644 --- a/

[Qemu-devel] [PATCH v3 1/5] block: use Error mechanism instead of -errno for block_job_create()

2012-04-25 Thread Stefan Hajnoczi
The block job API uses -errno return values internally and we convert these to Error in the QMP functions. This is ugly because the Error should be created at the point where we still have all the relevant information. More importantly, it is hard to add new error cases to this case since we quic

Re: [Qemu-devel] [PATCH] fix paaudio.c warnings

2012-04-25 Thread malc
On Wed, 25 Apr 2012, Gerd Hoffmann wrote: Thanks. Applied. -- mailto:av1...@comtv.ru

[Qemu-devel] [PATCH v3 4/5] block: add 'speed' optional parameter to block-stream

2012-04-25 Thread Stefan Hajnoczi
Allow streaming operations to be started with an initial speed limit. This eliminates the window of time between starting streaming and issuing block-job-set-speed. Users should use the new optional 'speed' parameter instead so that speed limits are in effect immediately when the job starts. Sign

[Qemu-devel] [PATCH 08/12] target-m68k: Start QOM'ifying CPU init

2012-04-25 Thread Andreas Färber
Move model-independent code from cpu_m68k_init() into a QOM initfn. Signed-off-by: Andreas Färber Reviewed-by: Laurent Vivier Tested-by: Laurent Vivier --- target-m68k/cpu.c|9 + target-m68k/helper.c |2 +- 2 files changed, 10 insertions(+), 1 deletions(-) diff --git a/tar

[Qemu-devel] [PATCH 11/12] target-mips: Start QOM'ifying CPU init

2012-04-25 Thread Andreas Färber
Move code not dependent on mips_def_t from cpu_mips_init() into a QOM initfn, as a start. Signed-off-by: Andreas Färber Reviewed-by: Richard Henderson --- target-mips/cpu.c |9 + target-mips/translate.c |1 - 2 files changed, 9 insertions(+), 1 deletions(-) diff --git a/t

[Qemu-devel] [PATCH 01/12] MAINTAINERS: Downgrade target-m68k to Odd Fixes

2012-04-25 Thread Andreas Färber
m68k patches are not being handled, so let's downgrade to Odd Fixes. Signed-off-by: Andreas Färber Cc: Paul Brook --- MAINTAINERS |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index cce37e7..8ae9158 100644 --- a/MAINTAINERS +++ b/MAINTAINERS

[Qemu-devel] [PATCH 07/12] target-m68k: QOM'ify CPU reset

2012-04-25 Thread Andreas Färber
Move code from cpu_state_reset() into QOM m68k_cpu_reset(). Signed-off-by: Andreas Färber Reviewed-by: Laurent Vivier Tested-by: Laurent Vivier --- target-m68k/cpu.c| 16 +++- target-m68k/helper.c | 18 ++ 2 files changed, 17 insertions(+), 17 deletions(-)

[Qemu-devel] [PATCH 2/3] exynos4210.c: register chipid_mem and rom_mem with vmstate

2012-04-25 Thread Igor Mitsyanko
Even if we do not register newly created RAM MemoryRegion for migration with vmstate_register_ram_global() function, ram_save_live() still saves this region to snapshot file with empty idstr=="". Consequently this results in error during VM restoration in ram_load(). Register chipid_mem and rom_mem

[Qemu-devel] [PATCH] fix paaudio.c warnings

2012-04-25 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann --- audio/paaudio.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/audio/paaudio.c b/audio/paaudio.c index e6708d0..aa15f16 100644 --- a/audio/paaudio.c +++ b/audio/paaudio.c @@ -505,8 +505,7 @@ fail: pa_stream_unref (stream);

Re: [Qemu-devel] [PATCH] softfloat: float32_to_float16() should do inexact instead of underflow for rounding case

2012-04-25 Thread Peter Maydell
On 25 April 2012 17:11, Alexey Starikovskiy wrote: > See A2.7.7 > Floating-point exceptions > > Inexact. The flag is set to 1 if the result of an operation is not > equivalent to the value that > would be produced if the operation were performed with unbounded > precision and exponent > range. Yo

[Qemu-devel] chatty qxl vga mode

2012-04-25 Thread Jan Kiszka
Hi, just played with qxl and noticed quite a lot of debug output on the local console: do_spice_init: starting 0.10.1 spice_server_add_interface: SPICE_INTERFACE_QXL red_worker_main: begin display_channel_create: create display channel cursor_channel_create: create cursor channel Is this intenti

[Qemu-devel] [PATCH] softfloat: float32_to_float16() should do inexact instead of underflow for rounding case

2012-04-25 Thread Alexey Starikovskiy
See A2.7.7 Floating-point exceptions Inexact. The flag is set to 1 if the result of an operation is not equivalent to the value that would be produced if the operation were performed with unbounded precision and exponent range. Underflow. The flag is set to 1 if the absolute value of the result of

Re: [Qemu-devel] [PATCH 1/2 v8] add-cow file format

2012-04-25 Thread Stefan Hajnoczi
On Wed, Apr 18, 2012 at 9:18 AM, Dong Xu Wang wrote: QEMU normally does not cache data, only metadata. There are a couple of reasons: 1. The host page cache already provides this, we don't need to duplicate this in QEMU. The user can decide whether to cache data in host memory or not using the

[Qemu-devel] [PATCH v3 5/5] qemu-iotests: add block-stream speed value test case

2012-04-25 Thread Stefan Hajnoczi
Add tests to exercise the InvalidParameter 'speed' error code path, as well as the regular success case for setting the speed. The block-stream 'speed' parameter allows the speed limit of the job to be applied immediately when the job starts instead of issuing a separate block-job-set-speed comman

[Qemu-devel] [PATCH v4 5/5] qemu-iotests: add block-stream speed value test case

2012-04-25 Thread Stefan Hajnoczi
Add tests to exercise the InvalidParameter 'speed' error code path, as well as the regular success case for setting the speed. The block-stream 'speed' parameter allows the speed limit of the job to be applied immediately when the job starts instead of issuing a separate block-job-set-speed comman

[Qemu-devel] [PATCH v4 4/5] block: add 'speed' optional parameter to block-stream

2012-04-25 Thread Stefan Hajnoczi
Allow streaming operations to be started with an initial speed limit. This eliminates the window of time between starting streaming and issuing block-job-set-speed. Users should use the new optional 'speed' parameter instead so that speed limits are in effect immediately when the job starts. Sign

[Qemu-devel] [PATCH v4 3/5] block: change block-job-set-speed argument from 'value' to 'speed'

2012-04-25 Thread Stefan Hajnoczi
Signed-off-by: Stefan Hajnoczi Acked-by: Kevin Wolf --- block.c |6 +++--- block/stream.c |8 block_int.h |4 ++-- blockdev.c |4 ++-- hmp-commands.hx |4 ++-- qapi-schema.json |4 ++-- qmp-commands.hx |2 +- 7 files changed, 16 insert

[Qemu-devel] [PATCH v4 2/5] block: use Error mechanism instead of -errno for block_job_set_speed()

2012-04-25 Thread Stefan Hajnoczi
There are at least two different errors that can occur in block_job_set_speed(): the job might not support setting speeds or the value might be invalid. Use the Error mechanism to report the error where it occurs. Signed-off-by: Stefan Hajnoczi Acked-by: Kevin Wolf --- block.c | 17

[Qemu-devel] [PATCH v4 1/5] block: use Error mechanism instead of -errno for block_job_create()

2012-04-25 Thread Stefan Hajnoczi
The block job API uses -errno return values internally and we convert these to Error in the QMP functions. This is ugly because the Error should be created at the point where we still have all the relevant information. More importantly, it is hard to add new error cases to this case since we quic

[Qemu-devel] [PATCH v4 0/5] block: add optional 'speed' parameter to block-stream

2012-04-25 Thread Stefan Hajnoczi
Eric Blake raised concerns about the inability to start block jobs with a speed limit. Current the user needs to follow up the block-stream command with block-job-set-speed. There is a window of time while the new block job is running but block-job-set-speed has not been processed yet. This seri

Re: [Qemu-devel] [PATCH v3 5/5] qemu-iotests: add block-stream speed value test case

2012-04-25 Thread Stefan Hajnoczi
On Wed, Apr 25, 2012 at 3:21 PM, Eric Blake wrote: > On 04/25/2012 07:17 AM, Stefan Hajnoczi wrote: >> Add tests to exercise the InvalidParameter 'speed' error code path, as >> well as the regular success case for setting the speed.  The >> block-stream 'speed' parameter allows the speed limit of

[Qemu-devel] [PATCH 06/12] target-m68k: QOM'ify CPU

2012-04-25 Thread Andreas Färber
Embed CPUM68KState as first member of QOM M68kCPU. Drop cpu_m68k_close() in favor of object_delete(). Let CPUClass::reset() call cpu_state_reset() for now. Signed-off-by: Andreas Färber Reviewed-by: Laurent Vivier Tested-by: Laurent Vivier --- Makefile.target |1 + target-m68k/cpu-q

[Qemu-devel] [PATCH 4/6] MAINTAINERS: Indicate type of SCM

2012-04-25 Thread Andreas Färber
T: lines are supposed to indicate whether it's git. Signed-off-by: Andreas Färber --- MAINTAINERS |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 7781023..305dbf9 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -437,7 +437,7 @@ M: Pao

[Qemu-devel] [PATCH 02/12] MAINTAINERS: Downgrade target-mips and target-sh4 to Odd Fixes

2012-04-25 Thread Andreas Färber
Patches are not being handled, so let's downgrade to Odd Fixes. Signed-off-by: Andreas Färber Cc: Aurélien Jarno --- MAINTAINERS |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 8ae9158..32d8dc7 100644 --- a/MAINTAINERS +++ b/MAINTAINER

[Qemu-devel] [PATCH 1/6] MAINTAINERS: Fix PC file pattern

2012-04-25 Thread Andreas Färber
Only one pattern is allowed per F: line, split it in two. Signed-off-by: Andreas Färber Cc: Anthony Liguori --- MAINTAINERS |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index cce37e7..83d3568 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -

Re: [Qemu-devel] [SeaBIOS] [PATCH 11/12] Migrate 64bit entries to 64bit pci regions

2012-04-25 Thread Gerd Hoffmann
On 04/25/12 14:51, Kevin O'Connor wrote: > On Wed, Apr 25, 2012 at 06:25:07PM +1200, Alexey Korolev wrote: >> On 25/04/12 13:48, Kevin O'Connor wrote: >>> On Tue, Apr 24, 2012 at 06:25:39PM +1200, Alexey Korolev wrote: +pci_region_map_entries(busses, &r64_mem); +pci_region

Re: [Qemu-devel] [PATCH] tracetool: use Python 2.5-compatible exception handling syntax

2012-04-25 Thread Stefan Hajnoczi
On Wed, Apr 25, 2012 at 2:35 PM, Andreas Färber wrote: > Am 25.04.2012 11:42, schrieb Stefan Hajnoczi: >> The newer "except as :" syntax is not >> supported by Python 2.5, we need to use "except , >> :". >> >> Tested all trace backends with Python 2.5.6. >> >> Reported-by: Andreas Färber >> Sign

[Qemu-devel] [PULL] MAINTAINERS tidying and stable trees

2012-04-25 Thread Andreas Färber
Hello Anthony, Please pull the uncontroversial parts of MAINTAINERS updates. I'm leaving out the Maintained upgrades and would ask you to apply the 0.15 patch once we've figured that out. For darwin-user I'll send a separate PULL. Cc: Anthony Liguori The following changes since commit cf36b31d

[Qemu-devel] [PATCH 5/6] MAINTAINERS: Fix SCM tree for virtio-9p

2012-04-25 Thread Andreas Färber
Instead of a Web link, T: is supposed to indicate type of SCM and pullable URL, so switch to the git:// URL. Also harmonize the spacing between sections while at it. Signed-off-by: Andreas Färber Acked-by: Aneesh Kumar K.V --- MAINTAINERS |3 +-- 1 files changed, 1 insertions(+), 2 deletion

[Qemu-devel] [PATCH 6/6] MAINTAINERS: Document all stable trees

2012-04-25 Thread Andreas Färber
We currently host stable trees for 0.10, 0.14, 0.15 and 1.0. Sort in descending order. It is expected that further non-stable sections will be added above these so this order avoids scrolling through an ever-growing list of stable trees. Signed-off-by: Andreas Färber --- MAINTAINERS | 22 +

[Qemu-devel] [PATCH 3/6] MAINTAINERS: Fix TCI file pattern

2012-04-25 Thread Andreas Färber
tcg/tci is a directory, so for recursive semantics add a trailing slash. Signed-off-by: Andreas Färber Acked-by: Stefan Weil --- MAINTAINERS |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index ea32786..7781023 100644 --- a/MAINTAINERS +++ b/

[Qemu-devel] [PATCH 2/6] MAINTAINERS: Fix virtio-9p file pattern

2012-04-25 Thread Andreas Färber
Only one pattern is allowed per F: line, split in two. Signed-off-by: Andreas Färber Acked-by: Aneesh Kumar K.V --- MAINTAINERS |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 83d3568..ea32786 100644 --- a/MAINTAINERS +++ b/MAINTAINERS

[Qemu-devel] [PATCH 10/12] target-mips: QOM'ify CPU

2012-04-25 Thread Andreas Färber
Embed CPUMIPSState as first member of QOM MIPSCPU. Let CPUClass::reset() call cpu_state_reset() for now. Signed-off-by: Andreas Färber Reviewed-by: Richard Henderson --- Makefile.target |3 ++ target-mips/cpu-qom.h | 74 +++ target-mi

[Qemu-devel] [PATCH 0/3] Exynos4210: license and RAM vmstate fixes

2012-04-25 Thread Igor Mitsyanko
PATCH1 drops "All rights reserved" from all exynos-related file's licenses. PATCH2 registers RO MemoryRegions for live migration. This is needed because ram_save_live saves all MemoryRegions no matter if they were registered or not. This is probably a bug in QEMU because It's not written anywhere

[Qemu-devel] [PATCH v3 3/5] block: change block-job-set-speed argument from 'value' to 'speed'

2012-04-25 Thread Stefan Hajnoczi
Signed-off-by: Stefan Hajnoczi Acked-by: Kevin Wolf --- block.c |6 +++--- block/stream.c |8 block_int.h |4 ++-- blockdev.c |4 ++-- hmp-commands.hx |4 ++-- qapi-schema.json |4 ++-- qmp-commands.hx |2 +- 7 files changed, 16 insert

Re: [Qemu-devel] [PATCH v6 5/5] FreeSCALE i.MX31 support: KZM-ARM11-01 evaluation board

2012-04-25 Thread Paul Brook
> On 23 April 2012 23:21, Peter Chubb wrote: > > Peter> Are these two devices really on the same IRQ? > > > > Yes. A single interrupt line comes from the FPGA into the AVIC. > > Inside the FPGA the interrupts for the UARTs, SD card and NAND flash > > are connected to that single interrupt line.

Re: [Qemu-devel] [PATCH] pci: Remove partial overrun checking from pci_host_config_{read, write} common

2012-04-25 Thread David Gibson
On Wed, Apr 25, 2012 at 11:08:12AM +0300, Michael S. Tsirkin wrote: > On Mon, Apr 16, 2012 at 02:16:24PM +1000, David Gibson wrote: > > Currently the pci_host_config_{read,write}_common() functions clamp the > > given access size to prevent it from overruning the size of config space. > > This does

Re: [Qemu-devel] Fwd: buildbot failure in qemu on ubuntu-default

2012-04-25 Thread Anthony Liguori
On 04/25/2012 08:56 AM, Gerd Hoffmann wrote: Original Message Subject: buildbot failure in qemu on ubuntu-default Date: Tue, 24 Apr 2012 21:28:11 +0200 From: build...@spunk.home.kraxel.org To: kra...@gmail.com The Buildbot has detected a failed build on builder ubuntu-default

Re: [Qemu-devel] [PATCH] tracetool: use Python 2.5-compatible exception handling syntax

2012-04-25 Thread Lluís Vilanova
Stefan Hajnoczi writes: > The newer "except as :" syntax is not > supported by Python 2.5, we need to use "except , > :". > Tested all trace backends with Python 2.5.6. > Reported-by: Andreas Färber > Signed-off-by: Stefan Hajnoczi Reviewed-by: Lluís Vilanova > --- > scripts/tracetool.py

[Qemu-devel] [PATCH v3 2/5] block: use Error mechanism instead of -errno for block_job_set_speed()

2012-04-25 Thread Stefan Hajnoczi
There are at least two different errors that can occur in block_job_set_speed(): the job might not support setting speeds or the value might be invalid. Use the Error mechanism to report the error where it occurs. Signed-off-by: Stefan Hajnoczi Acked-by: Kevin Wolf --- block.c | 17

[Qemu-devel] [PATCH v3 0/5] block: add optional 'speed' parameter to block-stream

2012-04-25 Thread Stefan Hajnoczi
Eric Blake raised concerns about the inability to start block jobs with a speed limit. Current the user needs to follow up the block-stream command with block-job-set-speed. There is a window of time while the new block job is running but block-job-set-speed has not been processed yet. This seri

[Qemu-devel] [PATCH 09/12] target-m68k: Add QOM CPU subclasses

2012-04-25 Thread Andreas Färber
Move code from cpu_m68k_set_model() into model-specific initfns and inline the remaining parts into cpu_m68k_init(). Let m68k_cpu_list() print CPU classes alphabetically except for "any". Signed-off-by: Andreas Färber Reviewed-by: Laurent Vivier Tested-by: Laurent Vivier --- target-m68k/cpu.c

[Qemu-devel] [PATCH 1/3] exynos4210: drop "All rights reserved" line from files license

2012-04-25 Thread Igor Mitsyanko
It has been noted that "All rights reserved" statement conflicts with GPL, remove it. Signed-off-by: Igor Mitsyanko --- hw/exynos4210.c |2 +- hw/exynos4210.h |2 +- hw/exynos4210_combiner.c |3 +-- hw/exynos4210_fimd.c |3 +-- hw/exynos4210_gic.c |

[Qemu-devel] [PATCH 03/12] target-sh4: QOM'ify CPU

2012-04-25 Thread Andreas Färber
Embed CPUSH4State as first member of SuperHCPU. Let CPUClass::reset() call cpu_state_reset() for now. Signed-off-by: Andreas Färber Reviewed-by: Peter Maydell --- Makefile.target|1 + target-sh4/cpu-qom.h | 70 target-sh4/cpu.c

[Qemu-devel] [PULL] QOM CPUState for sh4, m68k and mips

2012-04-25 Thread Andreas Färber
Hello Anthony, Blue, Please pull the remainder of the QOM CPU conversions: sh4, m68k and mips. There was no reaction from the listed maintainers to my patches for weeks nor to the MAINTAINERS RFC for one week, so please apply to let us proceed. Cc: Anthony Liguori Cc: Blue Swirl Cc: Aurélien

  1   2   >