Re: [Qemu-devel] [PATCH 0/2] Fix display resize bugs introduced after recent console revamp

2013-03-20 Thread Gerd Hoffmann
On 03/19/13 20:44, Igor Mitsyanko wrote: Bad things were happening in display update functions every time a screen has to be resized. Good catch, added to the console patches. thanks, Gerd

Re: [Qemu-devel] [RFC PATCH v4 00/30] ACPI memory hotplug

2013-03-20 Thread li guang
在 2013-01-09三的 01:08 +0100,Andreas Färber写道: Am 18.12.2012 13:41, schrieb Vasilis Liaskovitis: Because dimm layout needs to be configured on machine-boot, all dimm devices need to be specified on startup command line (either with populated=on or with populated=off). The dimm information

Re: [Qemu-devel] [RFC PATCH v4 07/30] Add SIZE type to qdev properties

2013-03-20 Thread li guang
在 2012-12-18二的 13:41 +0100,Vasilis Liaskovitis写道: This patch adds a 'SIZE' type property to qdev. It will make dimm description more convenient by allowing sizes to be specified with K,M,G,T prefixes instead of number of bytes e.g.: -device dimm,id=mem0,size=2G,bus=membus.0 Credits go

Re: [Qemu-devel] [RESEND PATCH v2] pciinit: Enable default VGA device

2013-03-20 Thread Gerd Hoffmann
Hi, Turns out it's this: commit 76e58028d28e78431f9de3cee0b3c88d807fa39d Author: Kevin O'Connor ke...@koconnor.net Date: Wed Mar 6 21:50:09 2013 -0500 acpi: Eliminate BDAT parameter passing to DSDT code. The BDAT construct is the only ACPI mechanism that relies on

[Qemu-devel] [PATCH 3/3] virtio-ccw, s390-virtio: Use generic virtio-blk macro.

2013-03-20 Thread Cornelia Huck
Now that virtio-ccw and s390-virtio define all common properties for virtio-blk, we can switch to using the generic DEFINE_VIRTIO_BLK_PROPERTIES macro. CC: Alexander Graf ag...@suse.de Signed-off-by: Cornelia Huck cornelia.h...@de.ibm.com --- hw/s390x/s390-virtio-bus.c |8 +---

[Qemu-devel] [PATCH 1/3] virtio-ccw: Add missing blk chs properties.

2013-03-20 Thread Cornelia Huck
Signed-off-by: Cornelia Huck cornelia.h...@de.ibm.com --- hw/s390x/virtio-ccw.c |1 + 1 file changed, 1 insertion(+) diff --git a/hw/s390x/virtio-ccw.c b/hw/s390x/virtio-ccw.c index 9688835..fc95116 100644 --- a/hw/s390x/virtio-ccw.c +++ b/hw/s390x/virtio-ccw.c @@ -757,6 +757,7 @@ static

[Qemu-devel] [PATCH 0/3] virtio-ccw queue as of 2013/03/20

2013-03-20 Thread Cornelia Huck
Hi, here are again my patches adding missing virtio-blk properties for the s390 transports, this time rebased on top of the virtio-blk refactoring. The third patch is new, bringing s390-virtio and virtio-ccw in line with virtio-pci. Alex, again I'd like an ack from you (for the new patch) so I

[Qemu-devel] [PATCH 2/3] s390-virtio, virtio-ccw: Add config_wce for virtio-blk.

2013-03-20 Thread Cornelia Huck
There's no reason why we wouldn't want to make the cache mode configurable. Acked-by: Alexander Graf ag...@suse.de Signed-off-by: Cornelia Huck cornelia.h...@de.ibm.com --- hw/s390x/s390-virtio-bus.c |1 + hw/s390x/virtio-ccw.c |1 + 2 files changed, 2 insertions(+) diff --git

Re: [Qemu-devel] [PATCH] virtio-blk: Set default serial id

2013-03-20 Thread Stefan Hajnoczi
On Wed, Mar 20, 2013 at 01:56:08PM +0800, Asias He wrote: If user does not specify a serial id, e.g. -device virtio-blk-pci,serial=serial_id or -drive serial=serial_id no serial id will be assigned. Add a default serial id in this case to help identifying the disk in guest.

Re: [Qemu-devel] [PATCH 0/3] virtio-ccw queue as of 2013/03/20

2013-03-20 Thread KONRAD Frédéric
On 20/03/2013 08:43, Cornelia Huck wrote: Hi, here are again my patches adding missing virtio-blk properties for the s390 transports, this time rebased on top of the virtio-blk refactoring. The third patch is new, bringing s390-virtio and virtio-ccw in line with virtio-pci. Alex, again I'd

Re: [Qemu-devel] [PATCH 0/3] virtio-ccw queue as of 2013/03/20

2013-03-20 Thread Cornelia Huck
On Wed, 20 Mar 2013 09:03:13 +0100 KONRAD Frédéric fred.kon...@greensocs.com wrote: On 20/03/2013 08:43, Cornelia Huck wrote: Hi, here are again my patches adding missing virtio-blk properties for the s390 transports, this time rebased on top of the virtio-blk refactoring. The

Re: [Qemu-devel] kvm suspend performance

2013-03-20 Thread Stefan Hajnoczi
On Tue, Mar 19, 2013 at 05:24:59PM +0100, Thomas Knauth wrote: lately I've been playing around with qemu's/kvm's suspend (to disk) and resume. My initial expectation was that both operations are I/O bound. So it surprised me to see that suspend to disk seems to be CPU-bound. Suspending a VM

Re: [Qemu-devel] OOM don't like qemu-kvm

2013-03-20 Thread Stefan Hajnoczi
On Wed, Mar 20, 2013 at 01:16:54AM +0800, Peter Cheung wrote: Hi AllMy compuer is i7 with 32 GB ram without swap, i try to launch 28 vm with 1GB ram each, it runs smoothly.If I launch 64 VMs, after the VMs eat up all the memory, the linux hang. If the linux is lack of memory, the OOM will

Re: [Qemu-devel] Requesting a wiki account.

2013-03-20 Thread Stefan Hajnoczi
On Tue, Mar 19, 2013 at 07:03:24PM -0400, spectre wrote: I've been doing background reading on the wiki and have found multiple dead links / errors I'd like to fix. Could an admin email me off list regarding a wiki account under a different email address / name ? Hi, I will create an account

Re: [Qemu-devel] [PATCH v3] NVMe: Initial commit

2013-03-20 Thread Stefan Hajnoczi
On Tue, Mar 19, 2013 at 01:41:53PM -0600, Keith Busch wrote: +static int nvme_init(PCIDevice *pci_dev) +{ +NvmeCtrl *n = NVME(pci_dev); +NvmeIdCtrl *id = n-id_ctrl; + +int i; +int64_t bs_size; +char serial[sizeof(id-sn)]; +uint8_t *pci_conf; + +if

Re: [Qemu-devel] [PATCH] virtio-blk: Set default serial id

2013-03-20 Thread Asias He
On Wed, Mar 20, 2013 at 08:52:57AM +0100, Stefan Hajnoczi wrote: On Wed, Mar 20, 2013 at 01:56:08PM +0800, Asias He wrote: If user does not specify a serial id, e.g. -device virtio-blk-pci,serial=serial_id or -drive serial=serial_id no serial id will be assigned. Add a

Re: [Qemu-devel] [PATCH] qemu-sockets: Fix assertion failure

2013-03-20 Thread Kevin Wolf
Am 19.03.2013 um 21:34 hat Luiz Capitulino geschrieben: On Wed, 06 Mar 2013 15:46:45 +0100 Laszlo Ersek ler...@redhat.com wrote: On 03/06/13 12:11, Kevin Wolf wrote: Am 06.03.2013 um 12:04 hat Paolo Bonzini geschrieben: Il 06/03/2013 11:48, Kevin Wolf ha scritto: inet_connect_opts()

Re: [Qemu-devel] [PATCH] HLFS driver for QEMU

2013-03-20 Thread harryxiyou
On Mon, Mar 18, 2013 at 11:16 PM, Stefan Hajnoczi stefa...@gmail.com wrote: [...] I looked at the Google Code project before, it looks like a repo that a few people are hacking on. The site is developer-focussed and there is no evidence of users. This is why I asked about the background of

Re: [Qemu-devel] [PATCH 09/11] block: Make find_image_format safe with NULL filename

2013-03-20 Thread Kevin Wolf
Am 20.03.2013 um 03:14 hat Eric Blake geschrieben: On 03/18/2013 11:23 AM, Kevin Wolf wrote: In order to achieve this, the .bdrv_probe callbacks of all drivers must cope with this. The DMG driver is the only one that bases its decision on the filename and it needs to be changed.

Re: [Qemu-devel] [Xen-devel] [PATCHv3] QEMU(upstream): Disable xen's use of O_DIRECT by default as it results in crashes.

2013-03-20 Thread Alex Bligh
Stefano, --On 19 March 2013 15:13:29 + Stefano Stabellini stefano.stabell...@eu.citrix.com wrote: Therefore I think that the current change is not safe, but it is pretty easy to make it safe. You just need to move the call to blk_open from blk_init to blk_connect. Actually you can move

Re: [Qemu-devel] [PATCH 08/11] block: Rename variable to avoid shadowing

2013-03-20 Thread Kevin Wolf
Am 20.03.2013 um 03:07 hat Eric Blake geschrieben: On 03/18/2013 11:23 AM, Kevin Wolf wrote: bdrv_open() uses two different variables called options. Rename one of them to avoid confusion and to allow the outer one to be accessed everywhere. Signed-off-by: Kevin Wolf kw...@redhat.com

[Qemu-devel] [PATCH] virtio-blk-x: fix configuration synchronization.

2013-03-20 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com The virtio-blk-x configuration is not in sync with virtio-blk configuration. So this patch remove the virtio-blk-x configuration field, and use virtio-blk one for setting the properties. This also remove a useless configuration copy in

Re: [Qemu-devel] [PATCH v14 2/4] add a new qevent: QEVENT_GUEST_PANICKED

2013-03-20 Thread Markus Armbruster
Hu Tao hu...@cn.fujitsu.com writes: This event will be emited when the guest is panicked. Signed-off-by: Wen Congyang we...@cn.fujitsu.com Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- QMP/qmp-events.txt| 14 ++ include/monitor/monitor.h | 1 + monitor.c

Re: [Qemu-devel] [PATCH v14 1/4] add a new runstate: RUN_STATE_GUEST_PANICKED

2013-03-20 Thread Markus Armbruster
Hu Tao hu...@cn.fujitsu.com writes: The guest will be in this state when it is panicked. Signed-off-by: Wen Congyang we...@cn.fujitsu.com Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- include/sysemu/sysemu.h | 1 + qapi-schema.json| 5 - qmp.c | 3 +--

[Qemu-devel] [PATCH] Fix I/O throttling pathologic oscillating behavior

2013-03-20 Thread Benoît Canet
Limiting a virtio device backed by a QED file at 150 iops with -drive file=test.qed,if=virtio,cache=none,iops=150 and running in the guest the load.c program lead to an iops oscillating behavior which amplify itself as the time goes on. At first an extract of iostats -x -d 2 would look like

[Qemu-devel] [PATCH] block: fix bdrv_exceed_iops_limits wait computation

2013-03-20 Thread Benoît Canet
This patch fix an I/O throttling behavior triggered by limiting at 150 iops and running a load of 50 threads doing random pwrites on a raw virtio device. After a few moments the iops count start to oscillate steadily between 0 and a value upper than the limit. As the load keep running the period

Re: [Qemu-devel] [PATCH] virtio-blk-x: fix configuration synchronization.

2013-03-20 Thread Cornelia Huck
On Wed, 20 Mar 2013 10:00:01 +0100 fred.kon...@greensocs.com wrote: From: KONRAD Frederic fred.kon...@greensocs.com The virtio-blk-x configuration is not in sync with virtio-blk configuration. So this patch remove the virtio-blk-x configuration field, and use virtio-blk one for setting the

Re: [Qemu-devel] [PATCH v14 3/4] introduce pvevent device to deal with panicked event

2013-03-20 Thread Markus Armbruster
Hu Tao hu...@cn.fujitsu.com writes: pvevent device is used to send guest panic event from guest to qemu. When guest panic happens, pvevent device driver will write a event number to IO port 0x505(which is the IO port occupied by pvevent device, by default). On receiving the event, pvevent

Re: [Qemu-devel] [PATCH v14 3/4] introduce pvevent device to deal with panicked event

2013-03-20 Thread Markus Armbruster
Paolo Bonzini pbonz...@redhat.com writes: Il 14/03/2013 16:59, Gleb Natapov ha scritto: [...] Fine with me. That was just a suggestion. I thought we had singleton qdev flag. We have no_user, but it's broken and not exactly a match for what you want here. For what it's worth, no_user is set

Re: [Qemu-devel] [PATCH v14 3/4] introduce pvevent device to deal with panicked event

2013-03-20 Thread Markus Armbruster
Alexander Graf ag...@suse.de writes: On 14.03.2013, at 10:43, Paolo Bonzini wrote: Il 14/03/2013 10:19, Gleb Natapov ha scritto: On Thu, Mar 14, 2013 at 10:14:12AM +0100, Paolo Bonzini wrote: Il 14/03/2013 09:15, Hu Tao ha scritto: pvevent device is used to send guest panic event from guest

Re: [Qemu-devel] [Xen-devel] [PATCHv3] QEMU(upstream): Disable xen's use of O_DIRECT by default as it results in crashes.

2013-03-20 Thread Paolo Bonzini
Il 20/03/2013 09:33, Alex Bligh ha scritto: Stefano, --On 19 March 2013 15:13:29 + Stefano Stabellini stefano.stabell...@eu.citrix.com wrote: Therefore I think that the current change is not safe, but it is pretty easy to make it safe. You just need to move the call to blk_open from

Re: [Qemu-devel] [PATCH V2 0/3] ARM: Misc ARM big-endian bug fixes

2013-03-20 Thread Fabien Chouteau
On 03/19/2013 12:30 PM, Peter Maydell wrote: On 19 March 2013 11:21, Fabien Chouteau chout...@adacore.com wrote: v2: - I drop the the CPSR.E/SCTLR.E/SCTLR.IE patch. The issue is still too complex for me to submit a good patch. - Fix the GDB byte order for 64bits registers Fabien

Re: [Qemu-devel] [PATCH v14 0/4] pvevent device to deal with guest panic event

2013-03-20 Thread Markus Armbruster
Gleb Natapov g...@redhat.com writes: On Thu, Mar 14, 2013 at 04:15:49PM +0800, Hu Tao wrote: This series introduces a new simulated device, pvevent, to notify qemu when guest panic event happens. Call it something less generic. pvpanic for instance. No objection, just observing that the

[Qemu-devel] [PATCH 03/23] pixman: add qemu_pixman_color()

2013-03-20 Thread Gerd Hoffmann
Helper function to map qemu colors (32bit integer + matching PixelFormat) into pixman_color_t. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- include/ui/qemu-pixman.h |2 ++ ui/qemu-pixman.c | 11 +++ 2 files changed, 13 insertions(+) diff --git

[Qemu-devel] [PATCH 04/23] pixman: render vgafont glyphs into pixman images

2013-03-20 Thread Gerd Hoffmann
Add helper functions to create pixman mask images for glyphs and to render these glyphs into a pixman image. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- include/ui/qemu-pixman.h |7 +++ ui/qemu-pixman.c | 43 +++ 2 files changed,

[Qemu-devel] [PATCH 01/23] exynos4210_fimd.c: fix display resize bug introduced after console revamp

2013-03-20 Thread Gerd Hoffmann
From: Igor Mitsyanko i.mitsya...@gmail.com In exynos4210 display update function, we were acquiring DisplaySurface pointer before calling screen resize function, not paying attention that resize procedure can replace current DisplaySurface with newly allocated one. Right thing to do is to

[Qemu-devel] [PATCH v2 00/23] console: overhaul continued

2013-03-20 Thread Gerd Hoffmann
Hi, Next round of console cleanup patches for review. What is in there? (1) qemu text consoles are rendered using pixman now. (2) Each QemuConsole has its own DisplaySurface now, so we can switch consoles without re-rendering the QemuConsole and update non-active consoles.

[Qemu-devel] [PATCH 08/23] console: add trace events

2013-03-20 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann kra...@redhat.com --- trace-events |3 +++ ui/console.c |4 2 files changed, 7 insertions(+) diff --git a/trace-events b/trace-events index 406fe5f..c241985 100644 --- a/trace-events +++ b/trace-events @@ -958,6 +958,9 @@ dma_bdrv_cb(void *dbs, int ret)

[Qemu-devel] [PATCH 11/23] console: give each QemuConsole its own DisplaySurface

2013-03-20 Thread Gerd Hoffmann
Go away from the global DisplaySurface, give one to each QemuConsole instead. With this patch applied it is possible to call graphics_hw_* functions with qemu consoles which are not the current foreground console. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- include/ui/console.h |1 -

[Qemu-devel] [PATCH 09/23] console: displaystate init revamp

2013-03-20 Thread Gerd Hoffmann
We have only one DisplayState, so there is no need for the next linking, rip it. Also consolidate all displaystate initialization into init_displaystate(). This function is called by vl.c after creating the devices (and thus all QemuConsoles) and before initializing DisplayChangeListensers (aka

[Qemu-devel] [PATCH 05/23] console: use pixman for fill+blit

2013-03-20 Thread Gerd Hoffmann
Zap homegrown pixel shuffeling code, use pixman calls instead. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- ui/console.c | 65 +- 1 file changed, 10 insertions(+), 55 deletions(-) diff --git a/ui/console.c b/ui/console.c index

[Qemu-devel] [PATCH 06/23] console: use pixman for font rendering

2013-03-20 Thread Gerd Hoffmann
Zap homegrown font rendering code, use pixman calls instead. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- ui/console.c | 110 ++ 1 file changed, 11 insertions(+), 99 deletions(-) diff --git a/ui/console.c b/ui/console.c index

[Qemu-devel] [PATCH 21/23] gtk: custom cursor support

2013-03-20 Thread Gerd Hoffmann
Makes gtk ui play nicely with qxl (and vmware_svga) as you can actually see your pointer now ;) Signed-off-by: Gerd Hoffmann kra...@redhat.com --- ui/gtk.c | 25 + 1 file changed, 25 insertions(+) diff --git a/ui/gtk.c b/ui/gtk.c index 7599ff4..512e974 100644 ---

[Qemu-devel] [PATCH 22/23] gtk: show a window for each graphical QemuConsole

2013-03-20 Thread Gerd Hoffmann
Multihead support: For each graphical console we'll create a gtk window, so with multiple graphics cards installed you get a gtk window for each. vte tabs are attached to the console #0 window. --- ui/gtk.c | 40 +++- 1 file changed, 27 insertions(+), 13

[Qemu-devel] [PATCH 02/23] hw/vmware_vga.c: fix screen resize bug introduced after console revamp

2013-03-20 Thread Gerd Hoffmann
From: Igor Mitsyanko i.mitsya...@gmail.com In vmsvga display update function, a pointer to DisplaySurface must be acquired after a call to vmsvga_check_size since this function might replace current DisplaySurface with a new one. Signed-off-by: Igor Mitsyanko i.mitsya...@gmail.com Signed-off-by:

[Qemu-devel] [PATCH 13/23] console: zap g_width + g_height

2013-03-20 Thread Gerd Hoffmann
We have a surface per QemuConsole now, so there is no need to keep track of the QemuConsole size any more as we can query the surface size directly at any time. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- ui/console.c | 32 +--- 1 file changed, 9

[Qemu-devel] [PATCH 12/23] console: simplify screendump

2013-03-20 Thread Gerd Hoffmann
Screendumps are alot simpler as we can update non-active QemuConsoles now. So we only need to update the QemuConsole we want write out, then dump the DisplaySurface content into a ppm file. Done. No console switching needed. No special support code in the gfx card emulation needed. Zap it

[Qemu-devel] [PATCH 16/23] console: add GraphicHwOps

2013-03-20 Thread Gerd Hoffmann
Pass a single GraphicHwOps struct pointer to graphic_console_init, instead of a bunch of function pointers. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/arm/musicpal.c|8 ++-- hw/blizzard.c|9 ++--- hw/cirrus_vga.c |8 ++-- hw/exynos4210_fimd.c |

[Qemu-devel] [PATCH 15/23] console: make DisplayState private to console.c

2013-03-20 Thread Gerd Hoffmann
With gui_* being moved to console.c nobody outside console.c needs access to DisplayState fields any more. Make the struct private. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- include/ui/console.h |8 ui/console.c |8 2 files changed, 8 insertions(+), 8

[Qemu-devel] [PATCH 18/23] xen: re-enable refresh interval reporting for xenfb

2013-03-20 Thread Gerd Hoffmann
xenfb informs the guest about the gui refresh interval so it can avoid pointless work. That logic was temporarely disabled for the DisplayState reorganization. Restore it now, with a proper interface for it. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/xenfb.c | 47

[Qemu-devel] [PATCH 0/5] for spice post load char device hook

2013-03-20 Thread Alon Levy
This reworks my former patch (http://patchwork.ozlabs.org/patch/227678/ - sorry Hans, can't find the version you posted) per Gerd's suggestion. Specifically it adds a new qemu_chr_fe_post_load api that is called by the front end and implemented by the backend. virtio-console implements it, adding

[Qemu-devel] [PATCH 10/23] console: rename vga_hw_*, add QemuConsole param

2013-03-20 Thread Gerd Hoffmann
Add QemuConsole parameter to vga_hw_*, so the interface allows to update non-active consoles (the actual code can't handle this yet, see next patch). Passing NULL is allowed and updates the active console, like the functions do today. While touching all vga_hw_* calls anyway rename that to the

[Qemu-devel] [PATCH 07/23] console: switch color_table_rgb to pixman_color_t

2013-03-20 Thread Gerd Hoffmann
Now that all text console rendering uses pixman we can easily switch the color tables to use pixman_color_t directly. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- ui/console.c | 24 1 file changed, 8 insertions(+), 16 deletions(-) diff --git a/ui/console.c

[Qemu-devel] [PATCH 23/23] qxl: register QemuConsole for secondary cards

2013-03-20 Thread Gerd Hoffmann
Hook secondary qxl cards properly into the qemu console subsystem. Watch multihead in action: qemu -display gtk -vga qxl -device qxl -usbdevice tablet Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/qxl.c |6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git

[Qemu-devel] [PATCH 17/23] console: gui timer fixes

2013-03-20 Thread Gerd Hoffmann
Make gui update rate adaption code in gui_update() actually work. Sprinkle in a tracepoint so you can see the code at work. Remove the update rate adaption code in vnc and make vnc simply use the generic bits instead. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- include/ui/console.h |

[Qemu-devel] [PATCH v2 0/4] for spice post load char device hook

2013-03-20 Thread Alon Levy
This reworks my former patch (http://patchwork.ozlabs.org/patch/227678/ - sorry Hans, can't find the version you posted) per Gerd's suggestion. Specifically it adds a new qemu_chr_fe_post_load api that is called by the front end and implemented by the backend. virtio-console implements it, adding

[Qemu-devel] [PATCH 19/23] console: add qemu_console_is_*

2013-03-20 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann kra...@redhat.com --- include/ui/console.h |6 +++-- ui/console.c | 59 -- ui/curses.c |7 +++--- ui/sdl.c | 24 ++-- ui/vnc.c |6 ++--- 5 files

Re: [Qemu-devel] [PATCH 0/5] for spice post load char device hook

2013-03-20 Thread Alon Levy
Please ignore this email, sent before I remembered to squash one of the patches. This reworks my former patch (http://patchwork.ozlabs.org/patch/227678/ - sorry Hans, can't find the version you posted) per Gerd's suggestion. Specifically it adds a new qemu_chr_fe_post_load api that is called

[Qemu-devel] [PATCH 2/4] virtio-serial: add a post_load callback implemented by port

2013-03-20 Thread Alon Levy
Signed-off-by: Alon Levy al...@redhat.com --- hw/virtio-serial-bus.c | 5 + hw/virtio-serial.h | 2 ++ 2 files changed, 7 insertions(+) diff --git a/hw/virtio-serial-bus.c b/hw/virtio-serial-bus.c index ab7168e..b2a50cb 100644 --- a/hw/virtio-serial-bus.c +++ b/hw/virtio-serial-bus.c @@

[Qemu-devel] [PATCH 14/23] console: move gui_update+gui_setup_refresh from vl.c into console.c

2013-03-20 Thread Gerd Hoffmann
Pure code motion, no functional changes. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- include/ui/console.h |2 -- ui/console.c | 50 ++ vl.c | 49 - 3 files

[Qemu-devel] [PATCH 1/4] char: add a post_load callback

2013-03-20 Thread Alon Levy
Signed-off-by: Alon Levy al...@redhat.com --- include/char/char.h | 12 qemu-char.c | 7 +++ 2 files changed, 19 insertions(+) diff --git a/include/char/char.h b/include/char/char.h index 0326b2a..0fdcaf9 100644 --- a/include/char/char.h +++ b/include/char/char.h @@

[Qemu-devel] [PATCH 20/23] console: allow pinning displaychangelisteners to consoles

2013-03-20 Thread Gerd Hoffmann
DisplayChangeListener gets a new QemuConsole field, which can be set to non-NULL before registering. This will pin the QemuConsole, so that particular DisplayChangeListener will not follow console switches. spice+gtk (which don't support text console input anyway) are switched over to be pinned

[Qemu-devel] [PATCH 3/4] virtio-console: implement post_load to call to qemu_chr_fe_post_load

2013-03-20 Thread Alon Levy
Signed-off-by: Alon Levy al...@redhat.com --- hw/virtio-console.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/hw/virtio-console.c b/hw/virtio-console.c index e2d1c58..a6ff2f7 100644 --- a/hw/virtio-console.c +++ b/hw/virtio-console.c @@ -88,6 +88,16 @@ static void

Re: [Qemu-devel] [PATCH V3 WIP 3/3] disable vhost_verify_ring_mappings check

2013-03-20 Thread Michael S. Tsirkin
On Tue, Mar 19, 2013 at 06:57:08PM -0700, Nicholas A. Bellinger wrote: On Tue, 2013-03-19 at 09:40 +0100, Stefan Hajnoczi wrote: On Tue, Mar 19, 2013 at 08:34:45AM +0800, Asias He wrote: --- hw/vhost.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/vhost.c b/hw/vhost.c

[Qemu-devel] [PATCH 4/4] spice-qemu-char: register interface on post load

2013-03-20 Thread Alon Levy
The target has not seen the guest_connected event via spice_chr_guest_open or spice_chr_write, and so spice server wrongly assumes there is no agent active, while the client continues to send motion events only by the agent channel, which the server ignores. The net effect is that the mouse is

[Qemu-devel] QOM-ify QemuConsoles ...

2013-03-20 Thread Gerd Hoffmann
Hi, So, the big qemu console data structure overhauls is done, patches just posted to the list. I think the next logical step ahead is to QOM-ify the QemuConsoles, so we can link the QemuConsole to the thing actually backing it. For a graphical console that would be the emlated graphic

Re: [Qemu-devel] [PATCH] HLFS driver for QEMU

2013-03-20 Thread Stefan Hajnoczi
On Wed, Mar 20, 2013 at 04:43:17PM +0800, harryxiyou wrote: On Mon, Mar 18, 2013 at 11:16 PM, Stefan Hajnoczi stefa...@gmail.com wrote: [...] I looked at the Google Code project before, it looks like a repo that a few people are hacking on. The site is developer-focussed and there is no

[Qemu-devel] [PATCH] append the terminating '\0' to bootorder string

2013-03-20 Thread Amos Kong
Problem was introduced in commit c8a6ae8b. The last terminating '\0' was lost, use the right length 5 (HALT\0). Reported-by: Gerd Hoffmann kra...@redhat.com Signed-off-by: Amos Kong ak...@redhat.com --- vl.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/vl.c

Re: [Qemu-devel] [Xen-devel] [PATCHv3] QEMU(upstream): Disable xen's use of O_DIRECT by default as it results in crashes.

2013-03-20 Thread Stefano Stabellini
On Tue, 19 Mar 2013, Alex Bligh wrote: Stefano, George, This patch only impacts the PV backend in QEMU, not the IDE interface. PV frontends and backends always disconnect and reconnect during save/restore. So we can be *certain* that bdrv_close at the sender side is called before the

Re: [Qemu-devel] [PATCH 00/35] hw/ reorganization, part 2

2013-03-20 Thread Peter Maydell
On 20 March 2013 00:00, Paolo Bonzini pbonz...@redhat.com wrote: ARM is already quite good in that respect. However, until all architectures are converted cpu_*_init needs to remain because of user-mode targets (where the CPUs are created by common code, there is no board to encapsulate the

Re: [Qemu-devel] [Xen-devel] [PATCHv3] QEMU(upstream): Disable xen's use of O_DIRECT by default as it results in crashes.

2013-03-20 Thread George Dunlap
On 20/03/13 10:24, Stefano Stabellini wrote: That is true. My guess is that nobody really migrates HVM guests without PV drivers installed (it's not even possible on XenServer but xl let you do that if you want to). When the PV drivers initialize at boot time, the IDE disk is closed. Therefore

Re: [Qemu-devel] [PATCH][RFC 0/14] implement power chip

2013-03-20 Thread Peter Maydell
On 20 March 2013 00:56, li guang lig.f...@cn.fujitsu.com wrote: 在 2013-03-19二的 10:15 +,Peter Maydell写道: The point is that how exactly power controllers connect to devices, and which devices respond to reset/suspend/etc is a property of the individual machine being modelled. An x86 PC will

Re: [Qemu-devel] [PATCH v3] Use proper term in TCG README

2013-03-20 Thread Peter Maydell
On 20 March 2013 03:42, 陳韋任 (Wei-Ren Chen) che...@iis.sinica.edu.tw wrote: In TCG, target means the host architecture for which TCG generates the code. Using guest rather than target to make the document more consistent. Signed-off-by: Chen Wei-Ren che...@iis.sinica.edu.tw Reviewed-by:

Re: [Qemu-devel] [PATCH V2 0/3] ARM: Misc ARM big-endian bug fixes

2013-03-20 Thread Peter Maydell
On 20 March 2013 09:28, Fabien Chouteau chout...@adacore.com wrote: On 03/19/2013 12:30 PM, Peter Maydell wrote: On 19 March 2013 11:21, Fabien Chouteau chout...@adacore.com wrote: Fabien Chouteau (3): QAPI: Add ARMEB target-type Add default config for armeb-softmmu Hi Fabien. I'm

Re: [Qemu-devel] [PATCH v14 1/4] add a new runstate: RUN_STATE_GUEST_PANICKED

2013-03-20 Thread Paolo Bonzini
Il 20/03/2013 09:58, Markus Armbruster ha scritto: Let's examine the other transitions to RUN_STATE_PAUSED, and whether they can now come from RUN_STATE_GUEST_PANICKED: * process_incoming_migration_co() No, because we're in RUN_STATE_INMIGRATE here, aren't we? Juan? Yes. * qmp_stop()

Re: [Qemu-devel] [PATCH] block: fix bdrv_exceed_iops_limits wait computation

2013-03-20 Thread Zhi Yong Wu
Reviewed-by: Zhi Yong Wu wu...@linux.vnet.ibm.com On Wed, 2013-03-20 at 10:12 +0100, Benoît Canet wrote: This patch fix an I/O throttling behavior triggered by limiting at 150 iops and running a load of 50 threads doing random pwrites on a raw virtio device. After a few moments the iops

Re: [Qemu-devel] [PATCH 00/35] hw/ reorganization, part 2

2013-03-20 Thread Paolo Bonzini
Il 20/03/2013 11:30, Peter Maydell ha scritto: Basically I don't like the way your categorization scheme seems to be kind of device, except for stuff in hw/ARCH which has a completely different rationale. I'm OK with hw/ARCH having board models, because that's really kind of device, it's just

Re: [Qemu-devel] [PATCH v14 1/4] add a new runstate: RUN_STATE_GUEST_PANICKED

2013-03-20 Thread Markus Armbruster
Markus Armbruster arm...@redhat.com writes: Hu Tao hu...@cn.fujitsu.com writes: The guest will be in this state when it is panicked. Signed-off-by: Wen Congyang we...@cn.fujitsu.com Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- include/sysemu/sysemu.h | 1 + qapi-schema.json|

Re: [Qemu-devel] [Xen-devel] [PATCHv3] QEMU(upstream): Disable xen's use of O_DIRECT by default as it results in crashes.

2013-03-20 Thread Paolo Bonzini
Il 20/03/2013 11:37, George Dunlap ha scritto: That is true. My guess is that nobody really migrates HVM guests without PV drivers installed (it's not even possible on XenServer but xl let you do that if you want to). When the PV drivers initialize at boot time, the IDE disk is closed.

Re: [Qemu-devel] [PATCH] HLFS driver for QEMU

2013-03-20 Thread harryxiyou
On Wed, Mar 20, 2013 at 6:04 PM, Stefan Hajnoczi stefa...@redhat.com wrote: On Wed, Mar 20, 2013 at 04:43:17PM +0800, harryxiyou wrote: On Mon, Mar 18, 2013 at 11:16 PM, Stefan Hajnoczi stefa...@gmail.com wrote: [...] I looked at the Google Code project before, it looks like a repo that a

Re: [Qemu-devel] [Xen-devel] [PATCHv3] QEMU(upstream): Disable xen's use of O_DIRECT by default as it results in crashes.

2013-03-20 Thread Alex Bligh
--On 20 March 2013 12:08:19 +0100 Paolo Bonzini pbonz...@redhat.com wrote: To make it safe, just use cache=none. There are downsides (for example tmpfs does not support it), but perhaps you can use a global option or environment variable to toggle the behavior. If you don't use cache=none,

Re: [Qemu-devel] [PATCH] QMP: TPM QMP and man page documentation updates

2013-03-20 Thread Markus Armbruster
Corey Bryant cor...@linux.vnet.ibm.com writes: On 03/19/2013 03:26 AM, Markus Armbruster wrote: [Note cc: Anthony for QAPI schema expertise] Stefan Berger stef...@linux.vnet.ibm.com writes: On 03/18/2013 12:16 PM, Markus Armbruster wrote: Corey Bryant cor...@linux.vnet.ibm.com writes:

Re: [Qemu-devel] [PATCH] virtio-blk-x: fix configuration synchronization.

2013-03-20 Thread Stefan Hajnoczi
On Wed, Mar 20, 2013 at 10:00:01AM +0100, fred.kon...@greensocs.com wrote: From: KONRAD Frederic fred.kon...@greensocs.com The virtio-blk-x configuration is not in sync with virtio-blk configuration. So this patch remove the virtio-blk-x configuration field, and use virtio-blk one for

Re: [Qemu-devel] [PATCH] qemu-sockets: Fix assertion failure

2013-03-20 Thread Luiz Capitulino
On Wed, 20 Mar 2013 09:39:34 +0100 Kevin Wolf kw...@redhat.com wrote: Am 19.03.2013 um 21:34 hat Luiz Capitulino geschrieben: On Wed, 06 Mar 2013 15:46:45 +0100 Laszlo Ersek ler...@redhat.com wrote: On 03/06/13 12:11, Kevin Wolf wrote: Am 06.03.2013 um 12:04 hat Paolo Bonzini

Re: [Qemu-devel] [PATCH 21/23] gtk: custom cursor support

2013-03-20 Thread Anthony Liguori
Gerd Hoffmann kra...@redhat.com writes: Makes gtk ui play nicely with qxl (and vmware_svga) as you can actually see your pointer now ;) Signed-off-by: Gerd Hoffmann kra...@redhat.com --- ui/gtk.c | 25 + 1 file changed, 25 insertions(+) diff --git a/ui/gtk.c

Re: [Qemu-devel] [PATCH 22/23] gtk: show a window for each graphical QemuConsole

2013-03-20 Thread Anthony Liguori
Gerd Hoffmann kra...@redhat.com writes: Multihead support: For each graphical console we'll create a gtk window, so with multiple graphics cards installed you get a gtk window for each. vte tabs are attached to the console #0 window. This is neat but I'm not sure if the user experience is

Re: [Qemu-devel] [RFC PATCH RDMA support v4: 03/10] more verbose documentation of the RDMA transport

2013-03-20 Thread Michael S. Tsirkin
On Tue, Mar 19, 2013 at 06:52:59PM +0100, Paolo Bonzini wrote: Il 19/03/2013 18:40, Michael R. Hines ha scritto: registration scheme would not work with cgroups because we would be attempting to pin zero pages (for no reason) that cgroups has already kicked out, which would defeat the

[Qemu-devel] [PATCH 1/2] libcacard: remove sql: prefix

2013-03-20 Thread Marc-André Lureau
For some reason, with sql:/ prefix, the PKCS11 modules are not loaded. This patch goes on top of Alon smartcard series. --- libcacard/vcard_emul_nss.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libcacard/vcard_emul_nss.c b/libcacard/vcard_emul_nss.c index

[Qemu-devel] [PATCH 2/2] libcacard: remove default libcoolkey loading

2013-03-20 Thread Marc-André Lureau
Use only the modules defined in the NSS database. --- libcacard/vcard_emul_nss.c | 17 + 1 file changed, 1 insertion(+), 16 deletions(-) diff --git a/libcacard/vcard_emul_nss.c b/libcacard/vcard_emul_nss.c index a0dd32d..7ed94af 100644 --- a/libcacard/vcard_emul_nss.c +++

Re: [Qemu-devel] [PATCH 1/4] char: add a post_load callback

2013-03-20 Thread Anthony Liguori
Alon Levy al...@redhat.com writes: Signed-off-by: Alon Levy al...@redhat.com --- include/char/char.h | 12 qemu-char.c | 7 +++ 2 files changed, 19 insertions(+) diff --git a/include/char/char.h b/include/char/char.h index 0326b2a..0fdcaf9 100644 ---

Re: [Qemu-devel] [PATCH v2 00/15] propagate Errors to do_device_add()

2013-03-20 Thread Markus Armbruster
No longer applies cleanly, but git-am -3 takes care of everything except for a trivial #include conflict. I didn't test the result, though :)

Re: [Qemu-devel] [PATCH v2 02/15] do_device_add(): look up device opts list with qemu_find_opts_err()

2013-03-20 Thread Markus Armbruster
Laszlo Ersek ler...@redhat.com writes: Conversion status (call chains covered or substituted by error propagation marked with square brackets): do_device_add - [qemu_find_opts - error_report] do_device_add - qdev_device_add - qerror_report do_device_add - qdev_device_add - qbus_find -

Re: [Qemu-devel] [PATCH] block: fix bdrv_exceed_iops_limits wait computation

2013-03-20 Thread Stefan Hajnoczi
On Wed, Mar 20, 2013 at 10:12:14AM +0100, Benoît Canet wrote: This patch fix an I/O throttling behavior triggered by limiting at 150 iops and running a load of 50 threads doing random pwrites on a raw virtio device. After a few moments the iops count start to oscillate steadily between 0 and

Re: [Qemu-devel] [PATCH] qemu-sockets: Fix assertion failure

2013-03-20 Thread Kevin Wolf
Am 20.03.2013 um 13:57 hat Luiz Capitulino geschrieben: On Wed, 20 Mar 2013 09:39:34 +0100 Kevin Wolf kw...@redhat.com wrote: Am 19.03.2013 um 21:34 hat Luiz Capitulino geschrieben: inet_connect_addr() has two users: inet_connect_opts() and wait_for_connect(), with this patch both

Re: [Qemu-devel] QOM-ify QemuConsoles ...

2013-03-20 Thread Andreas Färber
Hi Gerd, Am 20.03.2013 10:55, schrieb Gerd Hoffmann: I think the next logical step ahead is to QOM-ify the QemuConsoles, so we can link the QemuConsole to the thing actually backing it. For a graphical console that would be the emlated graphic device. For a text console it would be the

Re: [Qemu-devel] [PATCH] QMP: TPM QMP and man page documentation updates

2013-03-20 Thread Corey Bryant
On 03/20/2013 08:32 AM, Markus Armbruster wrote: Corey Bryant cor...@linux.vnet.ibm.com writes: On 03/19/2013 03:26 AM, Markus Armbruster wrote: [Note cc: Anthony for QAPI schema expertise] Stefan Berger stef...@linux.vnet.ibm.com writes: On 03/18/2013 12:16 PM, Markus Armbruster wrote:

Re: [Qemu-devel] [PATCH 04/35] tpm: reorganize headers and split hardware part

2013-03-20 Thread Corey Bryant
On 03/18/2013 01:34 PM, Paolo Bonzini wrote: The TPM subsystem does not have a good front-end/back-end separation. I think it has very good front-end/back-end separation, but perhaps you mean the code could be moved around and better organized. However, we can at least try to split the

Re: [Qemu-devel] [PATCH] qemu-sockets: Fix assertion failure

2013-03-20 Thread Luiz Capitulino
On Wed, 20 Mar 2013 14:37:59 +0100 Kevin Wolf kw...@redhat.com wrote: Am 20.03.2013 um 13:57 hat Luiz Capitulino geschrieben: On Wed, 20 Mar 2013 09:39:34 +0100 Kevin Wolf kw...@redhat.com wrote: Am 19.03.2013 um 21:34 hat Luiz Capitulino geschrieben: inet_connect_addr() has two

Re: [Qemu-devel] [PATCH v2 05/15] qdev_prop_parse(): change return type to void

2013-03-20 Thread Markus Armbruster
Luiz Capitulino lcapitul...@redhat.com writes: On Thu, 7 Feb 2013 15:12:22 -0200 Eduardo Habkost ehabk...@redhat.com wrote: On Tue, Feb 05, 2013 at 09:39:18PM +0100, Laszlo Ersek wrote: Problems are now reported via Error only. Signed-off-by: Laszlo Ersek ler...@redhat.com ---

[Qemu-devel] [PATCH v4 00/10] virtio-scsi refactoring.

2013-03-20 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com This is the next part of virtio-refactoring. Basically it creates virtio-scsi device which extends virtio-device. Then a virtio-scsi can be connected on a virtio-bus. virtio-scsi-pci, virtio-scsi-s390x, virtio-scsi-ccw are created too, they extend

  1   2   3   >