Re: [Qemu-devel] [PATCH v2] vmdk: Fix integer overflow in offset calculation

2014-09-22 Thread Fam Zheng
On Fri, 09/19 13:52, Max Reitz wrote: > On 15.09.2014 04:32, Fam Zheng wrote: > >This fixes the bug introduced by commit c6ac36e (vmdk: Optimize cluster > >allocation). > > > >$ ~/build/master/qemu-io /stor/vm/arch.vmdk -c 'write 2G 1k' > >write failed: Invalid argument > > > >Reported-by: Mark Cav

[Qemu-devel] [Bug 1101210] Re: qemu 1.4.2: usb keyboard not fully working

2014-09-22 Thread Bjoern Kerler
Additional : Running the same debian virtual machine under Linux host where "AltGr+<" in order to get "|" on german keyboard seems to work properly, I get the following result running sendkey in console : Keycode 28 released; Keycode 100 pressed; Keycode 86 pressed; Keycode 86 released; Keycode

Re: [Qemu-devel] [PATCH v4] Add HMP command "info memory-devices"

2014-09-22 Thread zhugh
On Fri, 2014-09-19 at 11:34 -0400, Luiz Capitulino wrote: > On Fri, 19 Sep 2014 15:30:19 +0200 > Igor Mammedov wrote: > > > On Thu, 18 Sep 2014 16:09:32 +0800 > > zhugh wrote: > > > > > Hi, > > > > > > Could anyone help to review this patch? > > > If there was no problem, could help to merge i

[Qemu-devel] [PATCH v3] vmdk: Fix integer overflow in offset calculation

2014-09-22 Thread Fam Zheng
This fixes the bug introduced by commit c6ac36e (vmdk: Optimize cluster allocation). $ ~/build/master/qemu-io /stor/vm/arch.vmdk -c 'write 2G 1k' write failed: Invalid argument Reported-by: Mark Cave-Ayland Signed-off-by: Fam Zheng --- v3: A new case 105 instead of embedding in 005. (Max) ---

Re: [Qemu-devel] [PATCH v3] vmdk: Fix integer overflow in offset calculation

2014-09-22 Thread Fam Zheng
On Mon, 09/22 15:13, Fam Zheng wrote: > +# Copyright (C) 1014 Red Hat, Inc. Oops! Resending... Fam

[Qemu-devel] [PATCH v4] vmdk: Fix integer overflow in offset calculation

2014-09-22 Thread Fam Zheng
This fixes the bug introduced by commit c6ac36e (vmdk: Optimize cluster allocation). $ ~/build/master/qemu-io /stor/vm/arch.vmdk -c 'write 2G 1k' write failed: Invalid argument Reported-by: Mark Cave-Ayland Signed-off-by: Fam Zheng --- v4: Fix typo in file header: 1014 -> 2014. v3: A new case

Re: [Qemu-devel] [PATCH v3 04/23] block: Connect BlockBackend and DriveInfo

2014-09-22 Thread Markus Armbruster
Max Reitz writes: > On 16.09.2014 20:12, Markus Armbruster wrote: >> Make the BlockBackend own the DriveInfo. Change blockdev_init() to >> return the BlockBackend instead of the DriveInfo. >> >> Signed-off-by: Markus Armbruster >> --- >> block.c | 2 -- >> block/block-back

Re: [Qemu-devel] [Qemu-trivial] [PATCH v1] vl: Fix possible freed memory accessing

2014-09-22 Thread Michael Tokarev
22.09.2014 10:23, Markus Armbruster wrote: > Michael Tokarev writes: > >> Applied to -trivial, thank you! > > Makes my 'hmp: Remove "info pcmcia"' conflict. Either revert this one > before applying mine, or resolve the conflict and drop the paragraph > about the bug from my commit message. Oka

Re: [Qemu-devel] [PATCH v3 12/23] virtio-blk: Drop redundant VirtIOBlock member conf

2014-09-22 Thread Markus Armbruster
Max Reitz writes: > On 16.09.2014 20:12, Markus Armbruster wrote: >> Commit 12c5674 turned it into a pointer to member blk.conf. >> >> Signed-off-by: Markus Armbruster >> --- >> hw/block/virtio-blk.c | 28 ++-- >> include/hw/virtio/virtio-blk.h | 1 - >> 2 f

Re: [Qemu-devel] [RFC patch 5/6] s390: implement pci instruction

2014-09-22 Thread Frank Blaschka
On Fri, Sep 19, 2014 at 05:12:15PM +0200, Thomas Huth wrote: > > Hi Frank, > > On Fri, 19 Sep 2014 13:54:34 +0200 > frank.blasc...@de.ibm.com wrote: > > > From: Frank Blaschka > > > > This patch implements the s390 pci instructions in qemu. This allows > > to attach qemu pci devices including

Re: [Qemu-devel] [PATCH v3 14/23] hw: Convert from BlockDriverState to BlockBackend, mostly

2014-09-22 Thread Markus Armbruster
Max Reitz writes: > On 16.09.2014 20:12, Markus Armbruster wrote: >> Just four uses of BlockDriverState are left: >> >> * The Xen paravirtual block device backend (xen_disk.c) opens images >>itself when set up via xenbus, bypassing blockdev.c. I figure it >>should go through qmp_blockdev

Re: [Qemu-devel] [PATCH 1/6] qemu-char: Make the filename size for a chardev a #define

2014-09-22 Thread Paolo Bonzini
Il 22/09/2014 01:04, miny...@acm.org ha scritto: > From: Corey Minyard > > Signed-off-by: Corey Minyard > --- > qemu-char.c | 16 +--- > 1 file changed, 9 insertions(+), 7 deletions(-) > > diff --git a/qemu-char.c b/qemu-char.c > index 2a3cb9f..39fb9e4 100644 > --- a/qemu-char.c >

Re: [Qemu-devel] [PATCH 3/6] qemu-char: Move some items into TCPCharDriver

2014-09-22 Thread Paolo Bonzini
Il 22/09/2014 01:04, miny...@acm.org ha scritto: > From: Corey Minyard > > This keeps them from having to be passed around and makes them > available for later functions, like printing and reconnecting. > > Signed-off-by: Corey Minyard > --- > qemu-char.c | 65 > ++

Re: [Qemu-devel] [RFC v2 2/3] Add units-per-idebus property

2014-09-22 Thread Markus Armbruster
Marcel Apfelbaum writes: > On Fri, 2014-09-19 at 11:39 +0200, Markus Armbruster wrote: >> John Snow writes: >> >> > Signed-off-by: John Snow [...] >> > @@ -1583,6 +1584,7 @@ static void machine_class_init(ObjectClass *oc, void >> > *data) >> > mc->hot_add_cpu = qm->hot_add_cpu; >> >

Re: [Qemu-devel] [PATCH 1/1] qemu-img: Cannot create fixed vhdx image

2014-09-22 Thread Kevin Wolf
Am 20.09.2014 um 15:53 hat Michael Tokarev geschrieben: > 18.09.2014 19:17, Adelina Tuvenie wrote: > > > > When trying to create a fixed vhd image qemu-img will return the > > following error: > > > > qemu-img: test.vhdx: Could not create image: Cannot allocate memory > > > > This happens becaus

Re: [Qemu-devel] [PATCH v4] Add HMP command "info memory-devices"

2014-09-22 Thread Markus Armbruster
zhugh writes: > On Fri, 2014-09-19 at 11:34 -0400, Luiz Capitulino wrote: >> On Fri, 19 Sep 2014 15:30:19 +0200 >> Igor Mammedov wrote: >> >> > On Thu, 18 Sep 2014 16:09:32 +0800 >> > zhugh wrote: >> > >> > > Hi, >> > > >> > > Could anyone help to review this patch? >> > > If there was no pr

[Qemu-devel] [Bug 1101210] Re: qemu 1.4.2: usb keyboard not fully working

2014-09-22 Thread Bjoern Kerler
xev log using ubuntu host and debian virtual machine for pressing altgr + < key to obtain "|" using german layout : KeyPress event, serial 43, synthetic NO, window 0x161, root 0x43, subw 0x0, time 4278157, (165,-105), root:(446,164), state 0x0, keycode 108 (keysym 0xfe03, ISO_Level3_Sh

Re: [Qemu-devel] [PATCH 5/6] qemu-char: Add reconnecting to client sockets

2014-09-22 Thread Paolo Bonzini
Il 22/09/2014 01:04, miny...@acm.org ha scritto: > From: Corey Minyard > > Adds a "reconnect" option to socket backends that gives a reconnect > timeout. This only applies to client sockets. If the other end > of a socket closes the connection, qemu will attempt to reconnect > after the given n

Re: [Qemu-devel] [PATCH 6/6] qemu-char: Print the remote and local addresses for a socket

2014-09-22 Thread Paolo Bonzini
Il 22/09/2014 01:04, miny...@acm.org ha scritto: > From: Corey Minyard > > It seems that it might be a good idea to know what is at the remote > end of a socket for tracking down issues. So add that to the > socket filename. > > Signed-off-by: Corey Minyard > --- > qemu-char.c | 27 ++

Re: [Qemu-devel] [PATCH v4] Add HMP command "info memory-devices"

2014-09-22 Thread Igor Mammedov
On Mon, 22 Sep 2014 09:59:06 +0200 Markus Armbruster wrote: > zhugh writes: > > > On Fri, 2014-09-19 at 11:34 -0400, Luiz Capitulino wrote: > >> On Fri, 19 Sep 2014 15:30:19 +0200 > >> Igor Mammedov wrote: > >> > >> > On Thu, 18 Sep 2014 16:09:32 +0800 > >> > zhugh wrote: > >> > > >> > > Hi

Re: [Qemu-devel] [PATCH 4/6] qemu-char: set socket filename to disconnected when not connected

2014-09-22 Thread Paolo Bonzini
Il 22/09/2014 01:04, miny...@acm.org ha scritto: > From: Corey Minyard > > This way we can tell if the socket is connected or not. It also splits > the string conversions out into separate functions to make this more > convenient. > > Signed-off-by: Corey Minyard > --- > qemu-char.c | 102 >

Re: [Qemu-devel] [PATCH 2/6] qemu-char: Rework qemu_chr_open_socket() for reconnect

2014-09-22 Thread Paolo Bonzini
Il 22/09/2014 01:04, miny...@acm.org ha scritto: > From: Corey Minyard > > Move all socket configuration to qmp_chardev_open_socket(). > qemu_chr_open_socket_fd() just opens the socket. This is getting ready > for the reconnect code, which will call open_sock_fd() on a reconnect > attempt. > >

Re: [Qemu-devel] [RFC PATCH v2 09/10] virtio-scsi-dataplane: Code to run virtio-scsi on iothread

2014-09-22 Thread Paolo Bonzini
Il 22/09/2014 07:56, Fam Zheng ha scritto: >> > >> > Please add these to VirtIOSCSI rather than VirtIOSCSICommon. Same for >> > the new functions you declare below. > What's the rationale, please? Asking because especially the VirtIOSCSIVring > fields are the dataplane counterparts of VirtQueue f

[Qemu-devel] [PULL 7/9] block: allow creation of fixed vhdx images

2014-09-22 Thread Michael Tokarev
From: Adelina Tuvenie When trying to create a fixed vhd image qemu-img will return the following error: qemu-img: test.vhdx: Could not create image: Cannot allocate memory This happens because of a incorrect check in vhdx.c. Specifficaly, in vhdx_create_bat(), after allocating memory for the B

[Qemu-devel] [PULL 1/9] qemu-char: Permit only a single "stdio" character device

2014-09-22 Thread Michael Tokarev
From: Li Liu When more than one is used, the terminal settings aren't restored correctly on exit. Fixable. However, such usage makes no sense, because the users race for input, so outlaw it instead. If you want to connect multiple things to stdio, use the mux chardev. Signed-off-by: Li Liu R

[Qemu-devel] [PULL 9/9] arch_init: Setting QEMU_ARCH enum straight

2014-09-22 Thread Michael Tokarev
From: Bastian Koppelmann Every QEMU_ARCH is now in (1 << n) notation, instead of a mixture of decimal and hexadecimal. Signed-off-by: Bastian Koppelmann Signed-off-by: Michael Tokarev --- include/sysemu/arch_init.h | 34 +- 1 file changed, 17 insertions(+),

[Qemu-devel] [PULL 0/9] Trivial patches for 2014-09-22

2014-09-22 Thread Michael Tokarev
Here's a next (small) batch of trivial stuff. Accumulated for over 2 weeks, but still quite small. Random tiny things here and there. Please consider pulling/applying. Thanks, /mjt The following changes since commit 07e2863d0271ac6c05206d8ce9e4f4c39b25d3ea: exec.c: fix setting 1-byte-long w

[Qemu-devel] [PULL 5/9] configure: trivial fixes

2014-09-22 Thread Michael Tokarev
From: Gonglei Make them consistent with the others. Cc: qemu-triv...@nongnu.org Signed-off-by: Gonglei Signed-off-by: Michael Tokarev --- configure |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 6c3d6cd..eb9cbcd 100755 --- a/configure

[Qemu-devel] [PULL 3/9] rdma: Fix incorrect description in comments

2014-09-22 Thread Michael Tokarev
From: zhanghailiang Since we have supported memory hotplug, VM's ram include pc.ram and hotplug-memory. Fix the confused description for rdma migration: pc.ram -> VM's ram Signed-off-by: zhanghailiang Signed-off-by: Michael Tokarev --- docs/rdma.txt|6 +++--- migration-rdma.c |4

[Qemu-devel] [PULL 8/9] pc: Add missing 'static' attribute

2014-09-22 Thread Michael Tokarev
From: Stefan Weil This fixes a warning from smatch (static code analysis). Signed-off-by: Stefan Weil Signed-off-by: Michael Tokarev --- hw/i386/pc.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/i386/pc.c b/hw/i386/pc.c index 2c2e9dc..82a7daa 100644 --- a/hw/i386/

[Qemu-devel] [PULL 2/9] Fix typos and misspellings in comments

2014-09-22 Thread Michael Tokarev
From: zhanghailiang formated -> formatted gaurantee -> guarantee shear -> sheer Signed-off-by: zhanghailiang Signed-off-by: Michael Tokarev --- hw/ppc/spapr.c |2 +- libcacard/vcard_emul_nss.c |4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/hw/ppc/s

[Qemu-devel] [PULL 6/9] vl: Print maxmem in hex format for error message

2014-09-22 Thread Michael Tokarev
From: zhanghailiang In error message, maxmem is printed in Dec but ram_size in Hex. It is better to print them in same format. Also use error_report instead of fprintf. Reviewed-By: Igor Mammedov Signed-off-by: zhanghailiang Signed-off-by: Michael Tokarev --- vl.c | 22 +++-

[Qemu-devel] [PULL 4/9] xen-hvm.c: Always return -1 when failure occurs in xen_hvm_init()

2014-09-22 Thread Michael Tokarev
From: Chen Gang When failure occurs, it need to use "return -1" instead of exit(1), so an upper layer has a chance to print failure information, too. For simplicity, in xen_hvm_init(), also use '-1' instead of all '-errno', since all related upper callers always exit(1) on failure. It is not a

Re: [Qemu-devel] [PULL 13/21] apic_common: vapic_paddr synchronization fix

2014-09-22 Thread Pavel Dovgaluk
> -Original Message- > From: Paolo Bonzini [mailto:pbonz...@redhat.com] > Sent: Friday, September 19, 2014 5:42 PM > To: Pavel Dovgaluk; qemu-devel@nongnu.org > Subject: Re: [PULL 13/21] apic_common: vapic_paddr synchronization fix > > Il 19/09/2014 14:50, Pavel Dovgaluk ha scritto: > > v

Re: [Qemu-devel] [PATCH 0/3] Fix confused output for alias properties

2014-09-22 Thread Gonglei (Arei)
Hi, Paolo > -Original Message- > From: Gonglei (Arei) > Sent: Wednesday, September 17, 2014 10:32 AM > To: 'Paolo Bonzini'; Markus Armbruster > Cc: Huangweidong (C); aligu...@amazon.com; Michael S. Tsirkin; > ag...@suse.de; qemu-devel@nongnu.org; stefa...@redhat.com; Huangpeng > (Peter); l

Re: [Qemu-devel] [PATCH 0/3] Fix confused output for alias properties

2014-09-22 Thread Michael S. Tsirkin
On Mon, Sep 15, 2014 at 10:44:36PM +0800, arei.gong...@huawei.com wrote: > From: Gonglei > > At present, people have no way to know they should > have a specific format for alias properties. > > Example: > > before output: > > virtio-blk-pci.physical_block_size=uint16 > virtio-blk-pci.logical

Re: [Qemu-devel] [RFC PATCH v2 09/10] virtio-scsi-dataplane: Code to run virtio-scsi on iothread

2014-09-22 Thread Fam Zheng
On Mon, 09/22 10:09, Paolo Bonzini wrote: > Il 22/09/2014 07:56, Fam Zheng ha scritto: > >> > > >> > Please add these to VirtIOSCSI rather than VirtIOSCSICommon. Same for > >> > the new functions you declare below. > > What's the rationale, please? Asking because especially the VirtIOSCSIVring >

Re: [Qemu-devel] [PATCH] virtio-balloon: Fix ballooning not working correctly when hotplug memory

2014-09-22 Thread Igor Mammedov
On Mon, 22 Sep 2014 11:36:35 +0800 zhanghailiang wrote: > Hi Igor, > > Thanks for your reviewing... > > > On Mon, 15 Sep 2014 20:29:38 +0800 > > zhanghailiang wrote: > > > >> When do memory balloon, it references the ram_size as the real ram size of > >> VM, > >> But here ram_size is not incl

Re: [Qemu-devel] [PATCH 0/3] Fix confused output for alias properties

2014-09-22 Thread Gonglei (Arei)
> From: Michael S. Tsirkin [mailto:m...@redhat.com] > Sent: Monday, September 22, 2014 4:34 PM > Subject: Re: [PATCH 0/3] Fix confused output for alias properties > > On Mon, Sep 15, 2014 at 10:44:36PM +0800, arei.gong...@huawei.com wrote: > > From: Gonglei > > > > At present, people have no way

Re: [Qemu-devel] [PATCH 1/2] pc-dimm: No numa option shouldn't break hotplug memory feature

2014-09-22 Thread Tang Chen
Hi Igor, On 09/19/2014 08:26 PM, Igor Mammedov wrote: On Wed, 17 Sep 2014 16:32:20 +0800 Hu Tao wrote: On Tue, Sep 16, 2014 at 06:39:15PM +0800, zhanghailiang wrote: If we do not configure numa option, memory hotplug should work as well. It should not depend on numa option. Steps to reprodu

Re: [Qemu-devel] [PATCH] translate-all.c: memory walker initial address miscalculation

2014-09-22 Thread Mikhail Ilin
ping http://patchwork.ozlabs.org/patch/386918/ On 08.09.2014 17:28, m.i...@samsung.com wrote: From: Mikhail Ilyin The initial base address is miscalculated in walk_memory_regions(). It has to be shifted TARGET_PAGE_BITS more. Holder variables are extended to target_ulong size otherwise they d

[Qemu-devel] [PATCH 2/2] xenfb: Add comment documentation

2014-09-22 Thread Owen smith
Add documentation for page-ref, page-gref and event-channel. Signed-off-by: Owen smith --- xen/include/public/io/fbif.h | 25 + 1 file changed, 25 insertions(+) diff --git a/xen/include/public/io/fbif.h b/xen/include/public/io/fbif.h index cc25aab..ba3f524 100644 --- a/x

[Qemu-devel] [PATCH 1/2] xenfb: Add comment documentation

2014-09-22 Thread Owen smith
Add documentation for feature-abs-pointer, feature-no-abs-rescale, feature-no-console, page-ref, page-gref and event-channel Signed-off-by: Owen smith --- xen/include/public/io/kbdif.h | 74 +++ 1 file changed, 74 insertions(+) diff --git a/xen/include/pu

[Qemu-devel] [PATCH 0/2] xenfb: Document public headers

2014-09-22 Thread Owen smith
Add comments documenting the xenstore nodes used by the vfb and vkbd devices. Documents changes in qemu patch series posted here http://lists.gnu.org/archive/html/qemu-devel/2014-09/msg03472.html Owen smith (2): xenfb: Add comment documentation xenfb: Add comment documentation xen/include/pu

Re: [Qemu-devel] [PATCH 0/3] Fix confused output for alias properties

2014-09-22 Thread Paolo Bonzini
Il 22/09/2014 10:34, Michael S. Tsirkin ha scritto: > Basically this patch brings back historical HMP behaviour. > As far as I could tell, it wasn't changed intentionally. It was changed intentionally. Or rather, the change was known at the time Stefan made it. > So how about applying this first

Re: [Qemu-devel] [PULL 13/21] apic_common: vapic_paddr synchronization fix

2014-09-22 Thread Paolo Bonzini
Il 22/09/2014 10:21, Pavel Dovgaluk ha scritto: > >> -Original Message- >> From: Paolo Bonzini [mailto:pbonz...@redhat.com] >> Sent: Friday, September 19, 2014 5:42 PM >> To: Pavel Dovgaluk; qemu-devel@nongnu.org >> Subject: Re: [PULL 13/21] apic_common: vapic_paddr synchronization fix >>

[Qemu-devel] [PATCH] docs: add blkdebug block driver documentation

2014-09-22 Thread Stefan Hajnoczi
The blkdebug block driver is undocumented. Documenting it is worthwhile since it offers powerful error injection features that are used by qemu-iotests test cases. This document will make it easier for people to learn about and use blkdebug. Signed-off-by: Stefan Hajnoczi --- docs/blkdebug.txt

Re: [Qemu-devel] [PATCH v4] Add HMP command "info memory-devices"

2014-09-22 Thread Markus Armbruster
Igor Mammedov writes: > On Mon, 22 Sep 2014 09:59:06 +0200 > Markus Armbruster wrote: > >> zhugh writes: >> >> > On Fri, 2014-09-19 at 11:34 -0400, Luiz Capitulino wrote: >> >> On Fri, 19 Sep 2014 15:30:19 +0200 >> >> Igor Mammedov wrote: >> >> >> >> > On Thu, 18 Sep 2014 16:09:32 +0800 >> >

Re: [Qemu-devel] [PATCH 0/3] Fix confused output for alias properties

2014-09-22 Thread Markus Armbruster
"Michael S. Tsirkin" writes: > On Mon, Sep 15, 2014 at 10:44:36PM +0800, arei.gong...@huawei.com wrote: >> From: Gonglei >> >> At present, people have no way to know they should >> have a specific format for alias properties. >> >> Example: >> >> before output: >> >> virtio-blk-pci.physical

Re: [Qemu-devel] [PATCH 0/3] Fix confused output for alias properties

2014-09-22 Thread Gonglei (Arei)
> From: Paolo Bonzini [mailto:paolo.bonz...@gmail.com] On Behalf Of Paolo > Bonzini > Sent: Monday, September 22, 2014 5:15 PM > To: Michael S. Tsirkin; Gonglei (Arei) > Cc: ag...@suse.de; Huangweidong (C); aligu...@amazon.com; Huangpeng > (Peter); qemu-devel@nongnu.org; stefa...@redhat.com; > lcap

Re: [Qemu-devel] [PATCH 4/4] block: avoid creating oversized writes in multiwrite_merge

2014-09-22 Thread Peter Lieven
On 19.09.2014 15:33, Paolo Bonzini wrote: Il 19/09/2014 00:56, Peter Lieven ha scritto: So I think if we treat it just as a hint for multiwrite, we can avoid writing code to split oversized requests. They always worked so far, we can certainly wait until we have a real bug fix. I would not tre

Re: [Qemu-devel] [PATCH 1/2] pc-dimm: No numa option shouldn't break hotplug memory feature

2014-09-22 Thread zhanghailiang
On 2014/9/22 17:03, Tang Chen wrote: Hi Igor, On 09/19/2014 08:26 PM, Igor Mammedov wrote: On Wed, 17 Sep 2014 16:32:20 +0800 Hu Tao wrote: On Tue, Sep 16, 2014 at 06:39:15PM +0800, zhanghailiang wrote: If we do not configure numa option, memory hotplug should work as well. It should not de

Re: [Qemu-devel] [PATCH v4] vmdk: Fix integer overflow in offset calculation

2014-09-22 Thread Stefan Hajnoczi
On Mon, Sep 22, 2014 at 03:15:44PM +0800, Fam Zheng wrote: > This fixes the bug introduced by commit c6ac36e (vmdk: Optimize cluster > allocation). > > $ ~/build/master/qemu-io /stor/vm/arch.vmdk -c 'write 2G 1k' > write failed: Invalid argument > > Reported-by: Mark Cave-Ayland > Signed-off-by:

Re: [Qemu-devel] [PATCH 0/3] Fix confused output for alias properties

2014-09-22 Thread Paolo Bonzini
Il 22/09/2014 11:33, Gonglei (Arei) ha scritto: > Yes, I knew this issue which object A's property name may > is not the same with object B, so I had posted v2 to fix it. This doesn't change the fact that ObjectProperty is a generic struct, and adding alias-specific fields there is wrong. Paolo

Re: [Qemu-devel] [PATCH] docs: add blkdebug block driver documentation

2014-09-22 Thread Paolo Bonzini
Il 22/09/2014 11:20, Stefan Hajnoczi ha scritto: > The blkdebug block driver is undocumented. Documenting it is worthwhile > since it offers powerful error injection features that are used by > qemu-iotests test cases. > > This document will make it easier for people to learn about and use > blkd

Re: [Qemu-devel] [PATCH] blkdebug: show an error for invalid event names

2014-09-22 Thread Stefan Hajnoczi
On Sat, Sep 20, 2014 at 09:32:35AM +, Gonglei (Arei) wrote: > > @@ -315,10 +320,21 @@ static int read_config(BDRVBlkdebugState *s, const > > char *filename, > > > > d.s = s; > > d.action = ACTION_INJECT_ERROR; > > -qemu_opts_foreach(&inject_error_opts, add_rule, &d, 0); > > +

Re: [Qemu-devel] [PATCH] blkdebug: show an error for invalid event names

2014-09-22 Thread Gonglei (Arei)
> From: Stefan Hajnoczi [mailto:stefa...@redhat.com] > Sent: Monday, September 22, 2014 6:28 PM > Subject: Re: [Qemu-devel] [PATCH] blkdebug: show an error for invalid event > names > > On Sat, Sep 20, 2014 at 09:32:35AM +, Gonglei (Arei) wrote: > > > @@ -315,10 +320,21 @@ static int read_conf

Re: [Qemu-devel] [BUG] Guest kernel divide error in kvm_unlock_kick

2014-09-22 Thread Paolo Bonzini
Il 11/09/2014 19:03, Chris Webb ha scritto: > Paolo Bonzini wrote: > >> This is a hypercall that should have kicked VCPU 3 (see rcx). >> >> Can you please apply this patch and gather a trace of the host >> (using "trace-cmd -e kvm qemu-kvm ")? > > Sure, no problem. I've built the trace-cmd tool

Re: [Qemu-devel] [PATCH 1/2] pc-dimm: No numa option shouldn't break hotplug memory feature

2014-09-22 Thread Michael S. Tsirkin
On Fri, Sep 19, 2014 at 02:37:46PM +0200, Igor Mammedov wrote: > On Tue, 16 Sep 2014 18:39:15 +0800 > zhanghailiang wrote: > > > If we do not configure numa option, memory hotplug should work as well. > > It should not depend on numa option. > > > > Steps to reproduce: > > (1) Start VM: qemu-kvm

Re: [Qemu-devel] [PATCH v2] ivshmem: use error_report

2014-09-22 Thread Michael S. Tsirkin
On Fri, Sep 19, 2014 at 04:42:59PM +0800, zhanghailiang wrote: > On 2014/9/19 15:34, zhanghailiang wrote: > >On 2014/9/19 7:17, Andrew Jones wrote: > >>Replace all the fprintf(stderr, ...) calls with error_report. > >>Also make sure exit() consistently uses the error code 1. A few calls > >>used -1

Re: [Qemu-devel] [PATCH v3 1/4] ivshmem: Check ivshmem_read() size argument

2014-09-22 Thread Michael S. Tsirkin
On Mon, Sep 15, 2014 at 06:40:05PM +0200, Andreas Färber wrote: > From: Stefan Hajnoczi > > The third argument to the fd_read() callback implemented by > ivshmem_read() is the number of bytes, not a flags field. Fix this and > check we received enough bytes before accessing the buffer pointer. >

Re: [Qemu-devel] [PATCH v3 2/4] ivshmem: validate incoming_posn value from server

2014-09-22 Thread Michael S. Tsirkin
On Mon, Sep 15, 2014 at 06:40:06PM +0200, Andreas Färber wrote: > From: Stefan Hajnoczi > > Check incoming_posn to avoid out-of-bounds array accesses if the ivshmem > server on the host sends invalid values. > > Cc: Cam Macdonell > Reported-by: Sebastian Krahmer > Signed-off-by: Stefan Hajnocz

Re: [Qemu-devel] [PATCH v3 3/4] ivshmem: Fix potential OOB r/w access

2014-09-22 Thread Michael S. Tsirkin
On Mon, Sep 15, 2014 at 06:40:07PM +0200, Andreas Färber wrote: > From: Sebastian Krahmer > > Fix OOB access via malformed incoming_posn parameters > and check that requested memory is actually alloc'ed. > > Signed-off-by: Sebastian Krahmer > [AF: Rebased, cleanups, avoid fd leak] > Cc: qemu-st

Re: [Qemu-devel] [PATCH v3 4/4] ivshmem: Fix fd leak on error

2014-09-22 Thread Michael S. Tsirkin
On Mon, Sep 15, 2014 at 06:40:08PM +0200, Andreas Färber wrote: > Reported-by: Stefan Hajnoczi > Cc: qemu-sta...@nongnu.org > Signed-off-by: Andreas Färber Reviewed-by: Michael S. Tsirkin > --- > hw/misc/ivshmem.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/hw/misc/ivshmem.c b/h

Re: [Qemu-devel] [PATCH] blkdebug: show an error for invalid event names

2014-09-22 Thread Kevin Wolf
Am 20.09.2014 um 10:55 hat Stefan Hajnoczi geschrieben: > It is easy to typo a blkdebug configuration and waste a lot of time > figuring out why no rules are matching. > > Push the Error** down into add_rule() so we can report an error when the > event name is invalid. > > Signed-off-by: Stefan H

Re: [Qemu-devel] [PATCH 0/3] Fix confused output for alias properties

2014-09-22 Thread Gonglei (Arei)
> From: Paolo Bonzini [mailto:paolo.bonz...@gmail.com] On Behalf Of Paolo > Bonzini > Sent: Monday, September 22, 2014 6:04 PM > To: Gonglei (Arei); Michael S. Tsirkin > Cc: Huangweidong (C); aligu...@amazon.com; qemu-devel@nongnu.org; > ag...@suse.de; stefa...@redhat.com; Huangpeng (Peter); > lcap

[Qemu-devel] [PATCH 1/2] save registration order of machine types

2014-09-22 Thread Laszlo Ersek
Commit 261747f1 ("vl: Use MachineClass instead of global QEMUMachine list") broke the ordering of the machine types in the user-visible output of qemu-system- -M \? This occurred because registration was rebased from a manually maintained linked list to GLib hash tables: qemu_register_ma

[Qemu-devel] [PATCH 2/2] machine_parse(): list supported machine types in their registration order

2014-09-22 Thread Laszlo Ersek
Based on the registration order captured in the previous patch, we sort the ad-hoc list printed for qemu-system- -M \? The GLib documentation is not overly clear if g_slist_sort() allocates a new list or reorders the input list in place, but: - it slightly suggests the latter, - and there i

[Qemu-devel] [PATCH 0/2] list supported machine types in their registration order

2014-09-22 Thread Laszlo Ersek
The patch messages say it all. Thanks, Laszlo Laszlo Ersek (2): save registration order of machine types machine_parse(): list supported machine types in their registration order include/hw/boards.h | 2 ++ include/sysemu/sysemu.h | 2 ++ hw/i386/pc.c| 2 ++ vl.c

Re: [Qemu-devel] [PATCH v3 2/2] Add configure option --enable-pc-1-0-qemu-kvm

2014-09-22 Thread Michael S. Tsirkin
On Sun, Sep 21, 2014 at 03:38:59PM +0100, Alex Bligh wrote: > Add a configure option --enable-pc-1-0-qemu-kvm and the > corresponding --disable-pc-1-0-qemu-kvm, defaulting > to disabled. > > Rename machine type pc-1.0 to pc-1.0-qemu-git. > > Make pc-1.0 machine type an alias of either pc-1.0-qemu

Re: [Qemu-devel] [Patch v2 1/8] stm32f205_timer: Add the stm32f205 Timer

2014-09-22 Thread Peter Crosthwaite
On Fri, Sep 19, 2014 at 2:54 PM, Alistair Francis wrote: > This patch adds the stm32f205 timers: TIM2, TIM3, TIM4 and TIM5 > to QEMU. > > Signed-off-by: Alistair Francis > --- > PUBLIC > V2: > - Reorder the Makefile config > - Fix up the debug printing > - Correct the timer event trigger > Cha

[Qemu-devel] [PULL v2 02/59] ide/atapi: Mark non-data commands as complete

2014-09-22 Thread Stefan Hajnoczi
From: John Snow When the command completion code in IDE and AHCI was unified to put all command completion inside of a callback, "cmd_done," we neglected to ensure that all AHCI/ATAPI command paths would eventually register as finished. for the PCI interface to IDE this is not a problem because c

[Qemu-devel] [PULL v2 07/59] block: Drop bdrv_em_co_aiocb_info.cancel

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng Also drop the now unused ->done pointer. Signed-off-by: Fam Zheng Signed-off-by: Stefan Hajnoczi --- block.c | 21 - 1 file changed, 21 deletions(-) diff --git a/block.c b/block.c index a9a48df..1ce7b99 100644 --- a/block.c +++ b/block.c @@ -4763,22 +4763,

[Qemu-devel] [PULL v2 01/59] block/vhdx.c: Mark parent_vhdx_guid variable as unused

2014-09-22 Thread Stefan Hajnoczi
From: Peter Maydell The parent_vhdx_guid variable is defined but never used, which provokes complaints from newer versions of clang. Since the variable definition is here acting as documentation of the image format, mark it with the 'unused' attribute to keep the compiler happy rather than simply

[Qemu-devel] [PULL v2 00/59] Block patches

2014-09-22 Thread Stefan Hajnoczi
v2: * Replaced g_assert_false(...) with g_assert(!...) [Peter] The following changes since commit 07e2863d0271ac6c05206d8ce9e4f4c39b25d3ea: exec.c: fix setting 1-byte-long watchpoints (2014-09-19 17:42:16 +0100) are available in the git repository at: git://github.com/stefanha/qemu.git tag

[Qemu-devel] [PULL v2 04/59] ide/ahci: Check for -ECANCELED in aio callbacks

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng Before, bdrv_aio_cancel will either complete the request (like normal) and call CB with an actual return code, or skip calling the request (for example when the IO req is not submitted by thread pool yet). We will change bdrv_aio_cancel to do it differently: always call CB before

[Qemu-devel] [PULL v2 03/59] aio-win32: fix uninitialized use of have_select_revents

2014-09-22 Thread Stefan Hajnoczi
From: Paolo Bonzini Always initialize it with the return value of aio_prepare. Reported-by: TeLeMan Signed-off-by: Paolo Bonzini Reviewed-by: Stefan Hajnoczi Reviewed-by: Fam Zheng Signed-off-by: Stefan Hajnoczi --- aio-win32.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) dif

[Qemu-devel] [PULL v2 05/59] block: Add refcnt in BlockDriverAIOCB

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng This will be useful in synchronous cancel emulation with bdrv_aio_cancel_async. Signed-off-by: Fam Zheng Signed-off-by: Stefan Hajnoczi --- block.c | 12 +++- include/block/aio.h | 2 ++ 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/block.

[Qemu-devel] [PULL v2 21/59] sheepdog: Convert sd_aiocb_info.cancel to .cancel_async

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng Also drop the now unused SheepdogAIOCB.finished field. Note that this aio is internal to sheepdog driver and has NULL cb and opaque, and should be unused at all. Signed-off-by: Fam Zheng Signed-off-by: Stefan Hajnoczi --- block/sheepdog.c | 46 +++--

[Qemu-devel] [PULL v2 08/59] block: Drop bdrv_em_aiocb_info.cancel

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng Signed-off-by: Fam Zheng Signed-off-by: Stefan Hajnoczi --- block.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/block.c b/block.c index 1ce7b99..45d3a5b 100644 --- a/block.c +++ b/block.c @@ -4681,18 +4681,8 @@ typedef struct BlockDriverAIOCBSync { int i

[Qemu-devel] [PULL v2 06/59] block: Add bdrv_aio_cancel_async

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng This is the async version of bdrv_aio_cancel, which doesn't block the caller. It guarantees that the cb is called either before returning or some time later. bdrv_aio_cancel can base on bdrv_aio_cancel_async, later we can convert all .io_cancel implementations to .io_cancel_async

[Qemu-devel] [PULL v2 13/59] archipelago: Drop archipelago_aiocb_info.cancel

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng The cancelled flag is no longer useful. Later the request will complete as before, and cb will be called. Signed-off-by: Fam Zheng Signed-off-by: Stefan Hajnoczi --- block/archipelago.c | 17 + 1 file changed, 1 insertion(+), 16 deletions(-) diff --git a/block

[Qemu-devel] [PULL v2 20/59] rbd: Drop rbd_aiocb_info.cancel

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng And also drop the now unused "cancelled" field. Signed-off-by: Fam Zheng Signed-off-by: Stefan Hajnoczi --- block/rbd.c | 23 +-- 1 file changed, 1 insertion(+), 22 deletions(-) diff --git a/block/rbd.c b/block/rbd.c index b7f7d5f..e5341fc 100644 --- a/blo

Re: [Qemu-devel] [libvirt] [PATCH v3 2/2] Add configure option --enable-pc-1-0-qemu-kvm

2014-09-22 Thread Daniel P. Berrange
On Mon, Sep 22, 2014 at 02:36:55PM +0300, Michael S. Tsirkin wrote: > On Sun, Sep 21, 2014 at 03:38:59PM +0100, Alex Bligh wrote: > > Add a configure option --enable-pc-1-0-qemu-kvm and the > > corresponding --disable-pc-1-0-qemu-kvm, defaulting > > to disabled. > > > > Rename machine type pc-1.0

[Qemu-devel] [PULL v2 11/59] dma: Convert dma_aiocb_info.cancel to .cancel_async

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng Just forward the request to bdrv_aio_cancel_async. Signed-off-by: Fam Zheng Signed-off-by: Stefan Hajnoczi --- dma-helpers.c | 20 1 file changed, 4 insertions(+), 16 deletions(-) diff --git a/dma-helpers.c b/dma-helpers.c index f6811fa..207b21e 100644 --

[Qemu-devel] [PULL v2 22/59] win32-aio: Drop win32_aiocb_info.cancel

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng Signed-off-by: Fam Zheng Signed-off-by: Stefan Hajnoczi --- block/win32-aio.c | 14 -- 1 file changed, 14 deletions(-) diff --git a/block/win32-aio.c b/block/win32-aio.c index 5030e32..eed86f7 100644 --- a/block/win32-aio.c +++ b/block/win32-aio.c @@ -106,22 +106,8

[Qemu-devel] [PULL v2 14/59] blkdebug: Drop blkdebug_aiocb_info.cancel

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng Signed-off-by: Fam Zheng Signed-off-by: Stefan Hajnoczi --- block/blkdebug.c | 15 +-- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/block/blkdebug.c b/block/blkdebug.c index 69b330e..08131b3 100644 --- a/block/blkdebug.c +++ b/block/blkdebug.c @@ -5

[Qemu-devel] [PULL v2 09/59] thread-pool: Convert thread_pool_aiocb_info.cancel to cancel_async

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng The .cancel_async shares the same the first half with .cancel: try to steal the request if not submitted yet. In this case set the elem to THREAD_DONE status and ret to -ECANCELED, which means thread_pool_completion_bh will call the cb with -ECANCELED. If the request is already s

[Qemu-devel] [PULL v2 12/59] iscsi: Convert iscsi_aiocb_info.cancel to .cancel_async

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng Also drop the unused field "canceled". Signed-off-by: Fam Zheng Signed-off-by: Stefan Hajnoczi --- block/iscsi.c | 17 ++--- 1 file changed, 2 insertions(+), 15 deletions(-) diff --git a/block/iscsi.c b/block/iscsi.c index 84bcae8..f8328d6 100644 --- a/block/iscsi

[Qemu-devel] [PULL v2 27/59] aio-win32: avoid out-of-bounds access to the events array

2014-09-22 Thread Stefan Hajnoczi
From: Paolo Bonzini If ret is WAIT_TIMEOUT and there was an event returned by select(), we can write to a location after the end of the array. But in that case we can retry the WaitForMultipleObjects call with the same set of events, so just move the event[ret - WAIT_OBJECT_0] assignment inside

[Qemu-devel] [PULL v2 23/59] ide: Convert trim_aiocb_info.cancel to .cancel_async

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng We know that either bh is scheduled or ide_issue_trim_cb will be called again, so we just set i, j and ret to the right values. In both cases, ide_trim_bh_cb will be called. Also forward the cancellation to the iocb->aiocb which we get from bdrv_aio_discard. Signed-off-by: Fam Z

[Qemu-devel] [PULL v2 10/59] linux-aio: Convert laio_aiocb_info.cancel to .cancel_async

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng Just call io_cancel (2), if it fails, it means the request is not canceled, so the event loop will eventually call qemu_laio_process_completion. In qemu_laio_process_completion, change to call the cb unconditionally. It is required by bdrv_aio_cancel_async. Signed-off-by: Fam Zh

[Qemu-devel] [PULL v2 19/59] quorum: Convert quorum_aiocb_info.cancel to .cancel_async

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng Before, we cancel all the child requests with bdrv_aio_cancel, then free the acb.. Now we just kick off asynchronous cancellation of child requests and return, we know quorum_aio_cb will be called later, so in the end quorum_aio_finalize will take care of calling the caller's cb.

[Qemu-devel] [PULL v2 28/59] block: Introduce "null" drivers

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng This is an analogue to Linux null_blk. It can be used for testing or benchmarking block device emulation and general block layer functionalities such as coroutines and throttling, where disk IO is not necessary or wanted. Use null-aio:// for AIO version, and null-co:// for corout

[Qemu-devel] [PULL v2 24/59] block: Drop AIOCBInfo.cancel

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng Now that all the implementations are converted to asynchronous version and we can emulate synchronous cancellation with it. Let's drop the unused member. Signed-off-by: Fam Zheng Signed-off-by: Stefan Hajnoczi --- block.c | 24 ++-- include/bloc

[Qemu-devel] [PULL v2 18/59] quorum: fix quorum_aio_cancel()

2014-09-22 Thread Stefan Hajnoczi
From: Liu Yuan For a fifo read pattern, we only have one running aio (possible other cases that has less number than num_children in the future), so we need to check if .acb is NULL against bdrv_aio_cancel() to avoid segfault. Cc: Eric Blake Cc: Benoit Canet Cc: Kevin Wolf Cc: Stefan Hajnoczi

[Qemu-devel] [PULL v2 15/59] blkverify: Drop blkverify_aiocb_info.cancel

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng Also the finished pointer is not used any more. Signed-off-by: Fam Zheng Signed-off-by: Stefan Hajnoczi --- block/blkverify.c | 19 --- 1 file changed, 19 deletions(-) diff --git a/block/blkverify.c b/block/blkverify.c index 163064c..460393f 100644 --- a/block

[Qemu-devel] [PULL v2 29/59] qapi: Sort BlockdevDriver enum data list

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng Reviewed-by: Benoît Canet Signed-off-by: Fam Zheng Reviewed-by: Eric Blake Message-id: 1410415798-20673-3-git-send-email-f...@redhat.com Signed-off-by: Stefan Hajnoczi --- qapi/block-core.json | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/qapi

[Qemu-devel] [PULL v2 31/59] qapi/block: Add "fatal" to BLOCK_IMAGE_CORRUPTED

2014-09-22 Thread Stefan Hajnoczi
From: Max Reitz Not every BLOCK_IMAGE_CORRUPTED event must be fatal; for example, when reading from an image, they should generally not be. Nonetheless, even an image only read from may of course be corrupted and this can be detected during normal operation. In this case, a non-fatal event should

[Qemu-devel] [PULL v2 16/59] curl: Drop curl_aiocb_info.cancel

2014-09-22 Thread Stefan Hajnoczi
From: Fam Zheng Signed-off-by: Fam Zheng Signed-off-by: Stefan Hajnoczi --- block/curl.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/block/curl.c b/block/curl.c index 938f9d9..6f5d6ae 100644 --- a/block/curl.c +++ b/block/curl.c @@ -613,14 +613,8 @@ out_noclean: return -EINVA

  1   2   3   4   >