Re: [Qemu-devel] [PATCH] exec: save exception_index field

2014-07-30 Thread Andreas Färber
Hi, Am 31.07.2014 07:41, schrieb Pavel Dovgaluk: > This patch adds subsection with exception_index field to the VMState for > correct saving the CPU state. > Without this patch simulator could miss the pending exception in the saved > virtual machine state. > > Signed-off-by: Pavel Dovgalyuk > -

Re: [Qemu-devel] [RFC PATCH v2 16/49] target: save cpu state fields

2014-07-30 Thread Andreas Färber
Am 17.07.2014 13:03, schrieb Pavel Dovgalyuk: > This patch adds interrupt fields to VMState for correct saving the CPU state. > > Signed-off-by: Pavel Dovgalyuk > --- > target-arm/machine.c |5 - > target-i386/machine.c |5 - > 2 files changed, 8 insertions(+), 2 deletions(-) >

[Qemu-devel] [v2][PATCH 4/5] xen:hw:pci-host:piix: introduce xen_igd_passthrough_i440fx_init

2014-07-30 Thread Tiejun Chen
This is almost same as an original i440fx_init but just work with that xen igd host bridge to passthrough. Signed-off-by: Tiejun Chen --- hw/pci-host/piix.c | 32 include/hw/i386/pc.h | 11 +++ 2 files changed, 43 insertions(+) v2: * Based on patch #2

[Qemu-devel] [v2][PATCH 5/5] xen:hw:i386:pc_piix: introduce new machine for IGD passthrough

2014-07-30 Thread Tiejun Chen
Now we can introduce a new machine, xenigd, specific to IGD passthrough. This can avoid involving other common codes. Signed-off-by: Tiejun Chen --- hw/i386/pc_piix.c | 105 ++ 1 file changed, 105 insertions(+) v2: * Unify prefix with XEN_IGD

[Qemu-devel] [v2][PATCH 1/5] hw:i386:pc_piix: split pc_init1()

2014-07-30 Thread Tiejun Chen
We'd like to split pc_init1 and then we can share something with other stuff. Signed-off-by: Tiejun Chen --- hw/i386/pc_piix.c | 117 +++--- 1 file changed, 93 insertions(+), 24 deletions(-) v2: * Fix some coding style diff --git a/hw/i386/pc_pi

[Qemu-devel] [v2][PATCH 3/5] xen:hw:pci-host:piix: create host bridge to passthrough

2014-07-30 Thread Tiejun Chen
Implement a pci host bridge specific to passthrough. Actually this just inherits the standard one. This is based on http://patchwork.ozlabs.org/patch/363810/. Signed-off-by: Tiejun Chen --- hw/pci-host/piix.c | 42 ++ 1 file changed, 42 insertions(+) v2:

[Qemu-devel] [v2][PATCH 0/5] xen: introduce new machine for IGD passthrough

2014-07-30 Thread Tiejun Chen
v2: * Fix some coding style * New patch to separate i440fx_init * Just add prefix with XEN_IGD_PASSTHROUGH/xen_igd_passthrough * Based on patch #2 to regenerate * Unify prefix with XEN_IGD_PASSTHROUGH/xen_igd_passthrough like patch #3 * Test: boot with a preinstalled ubuntu 14.04 ./i386-softmmu/

[Qemu-devel] [v2][PATCH 2/5] hw:pci-host:piix: split i440fx_init

2014-07-30 Thread Tiejun Chen
We'd like to split i440fx_init and then we can share something with other stuff. Signed-off-by: Tiejun Chen --- hw/pci-host/piix.c | 91 -- 1 file changed, 75 insertions(+), 16 deletions(-) v2: * New patch to separate i440fx_init diff --git

Re: [Qemu-devel] [RFC PATCH v2 00/49] Series short description

2014-07-30 Thread Pavel Dovgaluk
> From: Paolo Bonzini [mailto:pbonz...@redhat.com] > Il 30/07/2014 09:44, Pavel Dovgaluk ha scritto: > >> From: Paolo Bonzini [mailto:paolo.bonz...@gmail.com] On Behalf Of Paolo > >> Bonzini > - patch 16 should also use subsections, and perhaps apply to all other > CPUs too? > >>> > >>>

[Qemu-devel] [PATCH] exec: save exception_index field

2014-07-30 Thread Pavel Dovgaluk
This patch adds subsection with exception_index field to the VMState for correct saving the CPU state. Without this patch simulator could miss the pending exception in the saved virtual machine state. Signed-off-by: Pavel Dovgalyuk --- exec.c | 35 +++ 1 files c

[Qemu-devel] [PATCH qom v1 2/2] memory: remove object_property_add_child_array

2014-07-30 Thread Peter Crosthwaite
Obsoleted by automatic object_property_add arrayification. Signed-off-by: Peter Crosthwaite --- memory.c | 30 +- 1 file changed, 5 insertions(+), 25 deletions(-) diff --git a/memory.c b/memory.c index 64d7176..5272bf9 100644 --- a/memory.c +++ b/memory.c @@ -877,30

[Qemu-devel] [PATCH qom v1 1/2] qom: object_property_add: Add automatic arrayification

2014-07-30 Thread Peter Crosthwaite
If "[*]" is given as the last part of a QOM property name, treat that as an array property. The added property is given the first available name, replacing the * with a decimal number counting from 0. First add with name "foo[*]" will be "foo[0]". Second "foo[1]" and so on. Signed-off-by: Peter C

Re: [Qemu-devel] [PATCH v2 7/9] PPC: e500: Support dynamically spawned sysbus devices

2014-07-30 Thread Peter Crosthwaite
On Thu, Jul 31, 2014 at 12:55 AM, Peter Maydell wrote: > On 2 July 2014 19:01, Alexander Graf wrote: >> For e500 our approach to supporting dynamically spawned sysbus devices is to >> create a simple bus from the guest's point of view within which we map those >> devices dynamically. >> >> +/

Re: [Qemu-devel] [PATCH 3/9] sysbus: remove unused function sysbus_del_io

2014-07-30 Thread Peter Crosthwaite
On Wed, Jul 30, 2014 at 8:27 PM, Paolo Bonzini wrote: > Signed-off-by: Paolo Bonzini Reviewed-by: Peter Crosthwaite > --- > hw/core/sysbus.c| 5 - > include/hw/sysbus.h | 1 - > 2 files changed, 6 deletions(-) > > diff --git a/hw/core/sysbus.c b/hw/core/sysbus.c > index f4e760d..414e2

Re: [Qemu-devel] [PATCH 01/15] qemu coroutine: support bypass mode

2014-07-30 Thread Ming Lei
On Thu, Jul 31, 2014 at 7:37 AM, Paolo Bonzini wrote: > Il 30/07/2014 19:15, Ming Lei ha scritto: >> On Wed, Jul 30, 2014 at 9:45 PM, Paolo Bonzini wrote: >>> Il 30/07/2014 13:39, Ming Lei ha scritto: This patch introduces several APIs for supporting bypass qemu coroutine in case of bei

Re: [Qemu-devel] [PATCH 14/15] hw/block/virtio-blk: create num_queues vqs if dataplane is enabled

2014-07-30 Thread Ming Lei
On Wed, Jul 30, 2014 at 11:25 PM, Paolo Bonzini wrote: > Il 30/07/2014 17:12, Michael S. Tsirkin ha scritto: >>> > >>> > Dataplane must not be a change to the guest ABI. If you implement this >>> > feature you have to implement it for both dataplane and non-dataplne. >>> > IMO, no matter if the

Re: [Qemu-devel] [PATCH 07/15] dataplane: use object pool to speed up allocation for virtio blk request

2014-07-30 Thread Ming Lei
On Wed, Jul 30, 2014 at 10:14 PM, Paolo Bonzini wrote: > Il 30/07/2014 13:39, Ming Lei ha scritto: >> g_slice_new(VirtIOBlockReq), its free pair and access the instance >> is a bit slow since sizeof(VirtIOBlockReq) takes more than 40KB, >> so use object pool to speed up its allocation and release.

[Qemu-devel] [PATCH v2] po: Add Chinese translation

2014-07-30 Thread Fam Zheng
Signed-off-by: Fam Zheng --- v2: Fix typo for "_View". --- po/zh_CN.po | 86 + 1 file changed, 86 insertions(+) create mode 100644 po/zh_CN.po diff --git a/po/zh_CN.po b/po/zh_CN.po new file mode 100644 index 000..7ffba30 --- /dev

Re: [Qemu-devel] [PATCH] po: Add Chinese translation

2014-07-30 Thread Fam Zheng
On Thu, 07/31 10:44, Fam Zheng wrote: > +#: ui/gtk.c:1783 > +msgid "_View" > +msgstr "试图(_V)" Typo! Will post v2. Fam

[Qemu-devel] [PATCH target-arm v1 1/1] sd: sdhci: Fix ADMA dma_memory_read access

2014-07-30 Thread Peter Crosthwaite
This dma_memory_read was giving too big a size when begin was non-zero. This could cause segfaults in some circumstances. Fix. Signed-off-by: Peter Crosthwaite --- hw/sd/sdhci.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hw/sd/sdhci.c b/hw/sd/sdhci.c index b5a9eee..f9

[Qemu-devel] [PATCH] po: Add Chinese translation

2014-07-30 Thread Fam Zheng
Signed-off-by: Fam Zheng --- po/zh_CN.po | 86 + 1 file changed, 86 insertions(+) create mode 100644 po/zh_CN.po diff --git a/po/zh_CN.po b/po/zh_CN.po new file mode 100644 index 000..5724ef9 --- /dev/null +++ b/po/zh_CN.po @@ -0,0

Re: [Qemu-devel] [PATCH v3 2/7] bootindex: add del_boot_device_path function

2014-07-30 Thread Gonglei (Arei)
Hi, > > Introduce a del_boot_device_path() cleanup fw_cfg content > > when hot-unplugging devcie refer to bootindex. > > s/devcie/device/ > > sounds odd; maybe: > > Introduce del_boot_device_path() to clean up fw_cfg content when > hot-unplugging a device that refers to a bootindex. > OK. Than

Re: [Qemu-devel] [PATCH v3 2/7] bootindex: add del_boot_device_path function

2014-07-30 Thread Eric Blake
On 07/25/2014 10:45 PM, arei.gong...@huawei.com wrote: > From: Gonglei > > Introduce a del_boot_device_path() cleanup fw_cfg content > when hot-unplugging devcie refer to bootindex. s/devcie/device/ sounds odd; maybe: Introduce del_boot_device_path() to clean up fw_cfg content when hot-unplugg

Re: [Qemu-devel] [PATCH 08/15] virtio: decrease size of VirtQueueElement

2014-07-30 Thread Ming Lei
On Wed, Jul 30, 2014 at 10:40 PM, Michael S. Tsirkin wrote: > On Wed, Jul 30, 2014 at 03:51:22PM +0200, Paolo Bonzini wrote: >> Il 30/07/2014 13:39, Ming Lei ha scritto: >> > diff --git a/include/hw/virtio/virtio.h b/include/hw/virtio/virtio.h >> > index a60104c..943e72f 100644 >> > --- a/include/

Re: [Qemu-devel] [PATCH 08/15] virtio: decrease size of VirtQueueElement

2014-07-30 Thread Ming Lei
On Wed, Jul 30, 2014 at 9:51 PM, Paolo Bonzini wrote: > Il 30/07/2014 13:39, Ming Lei ha scritto: >> diff --git a/include/hw/virtio/virtio.h b/include/hw/virtio/virtio.h >> index a60104c..943e72f 100644 >> --- a/include/hw/virtio/virtio.h >> +++ b/include/hw/virtio/virtio.h >> @@ -84,12 +84,17 @@

Re: [Qemu-devel] [PATCH v3 5/7] qmp: add set-bootindex command

2014-07-30 Thread Gonglei (Arei)
Hi, > -Original Message- > From: Eric Blake [mailto:ebl...@redhat.com] > Sent: Thursday, July 31, 2014 6:37 AM > Subject: Re: [PATCH v3 5/7] qmp: add set-bootindex command > > On 07/25/2014 10:45 PM, arei.gong...@huawei.com wrote: > > From: Gonglei > > > > Adds "set-bootindex id=xx,booti

[Qemu-devel] [PATCH v1 1/1] hexdump: Add null guard on output file.

2014-07-30 Thread Peter Crosthwaite
To avoid callsites with optional output having to NULL guard. Signed-off-by: Peter Crosthwaite --- Noting in-tree is affected by this yet, but I though I'd get this out of the way straight-up rather than elongate other series. util/hexdump.c | 3 +++ 1 file changed, 3 insertions(+) diff --git

Re: [Qemu-devel] [PATCH 09/15] linux-aio: fix submit aio as a batch

2014-07-30 Thread Paolo Bonzini
Il 30/07/2014 19:32, Ming Lei ha scritto: > On Wed, Jul 30, 2014 at 9:59 PM, Paolo Bonzini wrote: >> Il 30/07/2014 13:39, Ming Lei ha scritto: >>> In the enqueue path, we can't complete request, otherwise >>> "Co-routine re-entered recursively" may be caused, so this >>> patch fixes the issue with

Re: [Qemu-devel] [PATCH 01/15] qemu coroutine: support bypass mode

2014-07-30 Thread Paolo Bonzini
Il 30/07/2014 19:15, Ming Lei ha scritto: > On Wed, Jul 30, 2014 at 9:45 PM, Paolo Bonzini wrote: >> Il 30/07/2014 13:39, Ming Lei ha scritto: >>> This patch introduces several APIs for supporting bypass qemu coroutine >>> in case of being not necessary and for performance's sake. >> >> No, this i

Re: [Qemu-devel] [PATCH v3 5/7] qmp: add set-bootindex command

2014-07-30 Thread Eric Blake
On 07/25/2014 10:45 PM, arei.gong...@huawei.com wrote: > From: Gonglei > > Adds "set-bootindex id=xx,bootindex=xx,suffix=xx" QMP command. > > Example QMP command: > -> { "execute": "set-bootindex", "arguments": { "id": "ide0-0-1", > "bootindex": 1, "suffix": "/disk@0"}} > <- { "return": {} } >

[Qemu-devel] [PATCH 4/4] qtest/ide: Uninitialize PC allocator

2014-07-30 Thread John Snow
Use the new call to pc_alloc_uninit as a test for the new pathways. The leak checking / assert pathways are not enabled in this patch, leaving this as an option to future test writers. Signed-off-by: John Snow --- tests/ide-test.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/ide-

[Qemu-devel] [PATCH 2/4] libqos: Change free function called in malloc

2014-07-30 Thread John Snow
From: Marc Marí Signed-off-by: Marc Marí Signed-off-by: John Snow --- tests/libqos/malloc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/libqos/malloc.h b/tests/libqos/malloc.h index 46f6000..5565381 100644 --- a/tests/libqos/malloc.h +++ b/tests/libqos/malloc.h @@

[Qemu-devel] [PATCH v2 0/4] libqos: add a simple first-fit memory allocator

2014-07-30 Thread John Snow
This set collects two patches by Marc Marí already on the mailing list, but goes further by adding a simple memory allocator that allows us to track and debug freed memory, and optionally keep track of any leaks. v2: use QTAILQ as a basis for the linked list implementation instead. Correct an

[Qemu-devel] [PATCH 3/4] libqos: add a simple first-fit memory allocator

2014-07-30 Thread John Snow
Implement a simple first-fit memory allocator that attempts to keep track of leased blocks of memory in order to be able to re-use blocks. Additionally, allow the user to specify when initializing the device that upon cleanup, we would like to assert that there are no blocks in use. This may be us

[Qemu-devel] [PATCH 1/4] libqos: Correct mask to align size to PAGE_SIZE in malloc-pc

2014-07-30 Thread John Snow
From: Marc Marí Signed-off-by: Paolo Bonzini Signed-off-by: Marc Marí Signed-off-by: John Snow --- tests/libqos/malloc-pc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/libqos/malloc-pc.c b/tests/libqos/malloc-pc.c index db1496c..2efd095 100644 --- a/tests/libqos/

Re: [Qemu-devel] latest rc: virtio-blk hangs forever after migration

2014-07-30 Thread Marcin Gibuła
W dniu 2014-07-30 15:38, Paolo Bonzini pisze: Il 30/07/2014 14:02, Marcin Gibuła ha scritto: without it: s/without/with/ of course... called do_kvm_cpu_synchronize_state_always called do_kvm_cpu_synchronize_state_always called do_kvm_cpu_synchronize_state: vcpu not dirty, getting registers c

Re: [Qemu-devel] [PATCH 7/8] block/qcow2: Speed up zero cluster expansion

2014-07-30 Thread Max Reitz
On 30.07.2014 22:31, Eric Blake wrote: On 07/30/2014 10:14 AM, Eric Blake wrote: On 07/25/2014 12:07 PM, Max Reitz wrote: Actually, we do not need to allocate a new data cluster for every zero cluster to be expanded: It is completely sufficient to rely on qcow2's COW part and instead create a s

Re: [Qemu-devel] [PATCH 7/8] block/qcow2: Speed up zero cluster expansion

2014-07-30 Thread Eric Blake
On 07/30/2014 10:14 AM, Eric Blake wrote: > On 07/25/2014 12:07 PM, Max Reitz wrote: >> Actually, we do not need to allocate a new data cluster for every zero >> cluster to be expanded: It is completely sufficient to rely on qcow2's >> COW part and instead create a single zero cluster and reuse it

Re: [Qemu-devel] [PATCH 7/8] block/qcow2: Speed up zero cluster expansion

2014-07-30 Thread Max Reitz
On 30.07.2014 18:14, Eric Blake wrote: On 07/25/2014 12:07 PM, Max Reitz wrote: Actually, we do not need to allocate a new data cluster for every zero cluster to be expanded: It is completely sufficient to rely on qcow2's COW part and instead create a single zero cluster and reuse it as much as

Re: [Qemu-devel] [PATCH 2/8] qemu-img: Add progress output for amend

2014-07-30 Thread Max Reitz
On 30.07.2014 16:55, Eric Blake wrote: On 07/25/2014 12:07 PM, Max Reitz wrote: Now that bdrv_amend_options() supports a status callback, use it to display a progress report. Signed-off-by: Max Reitz --- qemu-img.c | 26 +++--- 1 file changed, 23 insertions(+), 3 deletio

Re: [Qemu-devel] [PATCH alt 3/7] qemu-img: Fix insignifcant memleak

2014-07-30 Thread Max Reitz
On 30.07.2014 16:58, Eric Blake wrote: On 07/26/2014 01:22 PM, Max Reitz wrote: As soon as options is set in img_amend(), it needs to be freed before the function returns. This leak is rather insignifcant, as qemu-img will s/insignifcant/insignificant/ I wonder how I was able to get it wrong

[Qemu-devel] questions about host side of virtio-serial

2014-07-30 Thread Chris Friesen
Hi, I'm working on a native user of virtio-serial (ie, not going via the qemu guest agent). The information at "http://www.linux-kvm.org/page/Virtio-serial_API"; does a good job of describing the guest side of things, but has very little information about the host side of things. In partic

[Qemu-devel] [PATCH] AHCI read/write corruption with int13h

2014-07-30 Thread Eniac Zhang
The AHCI controller code in Qemu has a bug that it will use the wrong LBA address when Seabios tries to access LBA>128GB (aka 127.5GB limit http://www.hardwaresecrets.com/printpage/Hard-Disk-Drives-Capacity-Limits/482). When we needs to access the LBA>0xfff, 28bit LBA is not sufficient thu

[Qemu-devel] [PATCH] AHCI read/write corruption with int13h

2014-07-30 Thread Eniac Zhang
The AHCI controller code in Qemu has a bug that it will use the wrong LBA address when Seabios tries to access LBA>128GB (aka 127.5GB limit http://www.hardwaresecrets.com/printpage/Hard-Disk-Drives-Capacity-Limits/482). When we needs to access the LBA>0xfff, 28bit LBA is not sufficient thu

[Qemu-devel] [Bug 1350435] Re: tcg.c:1693: tcg fatal error

2014-07-30 Thread Peter Maydell
That patch is not in mainline because it's an appalling hack. If we care about multi-threaded guests we need to fix them properly, not paper over the issues by constraining multiple threads to one CPU in the hopes the race conditions don't bite us so often. -- You received this bug notification b

[Qemu-devel] [PATCH v7 4/4] sclp-s390: Add memory hotplug SCLPs

2014-07-30 Thread Matthew Rosato
Add memory information to read SCP info and add handlers for Read Storage Element Information, Attach Storage Element, Assign Storage and Unassign Storage. Signed-off-by: Matthew Rosato --- hw/s390x/sclp.c| 259 ++-- target-s390x/cpu.h | 15

[Qemu-devel] [PATCH v7 1/4] sclp-s390: Add device to manage s390 memory hotplug

2014-07-30 Thread Matthew Rosato
Add sclpMemoryHotplugDev to contain associated data structures, etc. Signed-off-by: Matthew Rosato --- hw/s390x/sclp.c | 30 ++ include/hw/s390x/sclp.h | 20 2 files changed, 50 insertions(+) diff --git a/hw/s390x/sclp.c b/hw/s390x/sc

[Qemu-devel] [PATCH v7 3/4] s390-virtio: Apply same memory boundaries as virtio-ccw

2014-07-30 Thread Matthew Rosato
Although s390-virtio won't support memory hotplug, it should enforce the same memory boundaries so that it can use shared codepaths (like read_SCP_info). Signed-off-by: Matthew Rosato --- hw/s390x/s390-virtio.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a

[Qemu-devel] [PATCH v7 2/4] virtio-ccw: Include standby memory when calculating storage increment

2014-07-30 Thread Matthew Rosato
When determining the memory increment size, use the maxmem size if it was specified. Signed-off-by: Matthew Rosato --- hw/s390x/s390-virtio-ccw.c | 46 qemu-options.hx|3 ++- target-s390x/cpu.h |3 +++ 3 files changed, 43

[Qemu-devel] [PATCH v7 0/4] s390: Support for Hotplug of Standby Memory

2014-07-30 Thread Matthew Rosato
This patchset adds support in s390 for a pool of standby memory, which can be set online/offline by the guest (ie, via chmem). The standby pool of memory is allocated as the difference between the initial memory setting and the maxmem setting. As part of this work, additional results are provided

[Qemu-devel] [PATCH v8 4/5] QMP: Add support for Archipelago

2014-07-30 Thread Chrysostomos Nanakos
Introduce new enum BlockdevOptionsArchipelago. @volume: #Name of the Archipelago volume image @mport: #'mport' is the port number on which mapperd is listening. This is optional and if not specified, QEMU will make Archipelago

[Qemu-devel] [Bug 1350435] Re: tcg.c:1693: tcg fatal error

2014-07-30 Thread Serge Hallyn
Hi, have already tested with that patch to verify that it fixes the issue? If I put qemu + that patch into a ppa, will your infrastructure allow you to test that way? I'm a bit concerned about this patch, as it appears to be one which has been in the Suse tree for quite some time, begging the qu

Re: [Qemu-devel] [PATCH 1/4] hw:i386:pc_piix: split pc_init1()

2014-07-30 Thread Michael S. Tsirkin
On Wed, Jul 30, 2014 at 04:19:58PM +0800, Chen, Tiejun wrote: > On 2014/7/29 19:26, Michael S. Tsirkin wrote: > >On Thu, Jul 24, 2014 at 07:30:26PM +0800, Tiejun Chen wrote: > >>We'd like to split pc_init1 and then we can share something > >>with other stuff. > >> > >>Signed-off-by: Tiejun Chen >

Re: [Qemu-devel] [PATCH 4/4] xen:hw:i386:pc_piix: introduce new machine for IGD passthrough

2014-07-30 Thread Michael S. Tsirkin
On Wed, Jul 30, 2014 at 04:31:24PM +0800, Chen, Tiejun wrote: > On 2014/7/29 19:29, Michael S. Tsirkin wrote: > >On Thu, Jul 24, 2014 at 07:30:29PM +0800, Tiejun Chen wrote: > >>Now we can introduce a new machine, xenigd, specific to IGD > >>passthrough. This can avoid involving other common codes.

Re: [Qemu-devel] [PATCH 3/4] xen:hw:pci-host:piix: introduce xen_igd_i440fx_init

2014-07-30 Thread Michael S. Tsirkin
On Wed, Jul 30, 2014 at 04:24:25PM +0800, Chen, Tiejun wrote: > On 2014/7/29 19:19, Michael S. Tsirkin wrote: > >On Thu, Jul 24, 2014 at 07:30:28PM +0800, Tiejun Chen wrote: > >>This is almost same as an original i440fx_init but just > >>work with that xen igd host bridge to passthrough. > >> > >>S

Re: [Qemu-devel] [PATCH 2/4] xen:hw:pci-host:piix: create host bridge to passthrough

2014-07-30 Thread Michael S. Tsirkin
On Wed, Jul 30, 2014 at 04:20:13PM +0800, Chen, Tiejun wrote: > On 2014/7/29 19:17, Michael S. Tsirkin wrote: > >On Thu, Jul 24, 2014 at 07:30:27PM +0800, Tiejun Chen wrote: > >>Implement that pci > > > >s/that/a/ > > Fixed. > > > > >>host bridge to specific > > > >s/to specific/specific/ > > Fi

Re: [Qemu-devel] 9p mapped-* security model infos are architecture-specific

2014-07-30 Thread Aneesh Kumar K.V
"Aneesh Kumar K.V" writes: > Michael Tokarev writes: > >> Apparently the the mapped-* security models results in a raw bytes >> being dumped to host without any architecture normalization (in >> host byte order). This may even lead to security issues in guest >> when the same files are served f

Re: [Qemu-devel] [PATCH 09/15] linux-aio: fix submit aio as a batch

2014-07-30 Thread Ming Lei
On Wed, Jul 30, 2014 at 9:59 PM, Paolo Bonzini wrote: > Il 30/07/2014 13:39, Ming Lei ha scritto: >> In the enqueue path, we can't complete request, otherwise >> "Co-routine re-entered recursively" may be caused, so this >> patch fixes the issue with below ideas: >> >> - for -EAGAIN or parti

Re: [Qemu-devel] [PATCH 10/15] linux-aio: increase max event to 256

2014-07-30 Thread Ming Lei
On Wed, Jul 30, 2014 at 10:00 PM, Paolo Bonzini wrote: > Il 30/07/2014 13:39, Ming Lei ha scritto: >> This patch increases max event to 256 for the comming >> virtio-blk multi virtqueue support. >> >> Signed-off-by: Ming Lei >> --- >> block/linux-aio.c |2 +- >> 1 file changed, 1 insertion(+

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Laszlo Ersek
On 07/30/14 18:35, Stefan Berger wrote: > Laszlo Ersek wrote on 07/30/2014 12:18:02 PM: > >> From: Laszlo Ersek >> To: "Michael S. Tsirkin" , Stefan Berger/Watson/IBM@IBMUS >> Cc: qemu-devel@nongnu.org, Stefan Berger >> Date: 07/30/2014 12:18 PM >> Subject: Re: [PATCH v2] Add ACPI tables for TP

Re: [Qemu-devel] [PATCH 01/15] qemu coroutine: support bypass mode

2014-07-30 Thread Ming Lei
On Wed, Jul 30, 2014 at 9:45 PM, Paolo Bonzini wrote: > Il 30/07/2014 13:39, Ming Lei ha scritto: >> This patch introduces several APIs for supporting bypass qemu coroutine >> in case of being not necessary and for performance's sake. > > No, this is wrong. Dataplane *must* use the same code as n

[Qemu-devel] [Bug 1350435] Re: tcg.c:1693: tcg fatal error

2014-07-30 Thread LocutusOfBorg
** 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/bugs/1350435 Title: tcg.c:1693: tcg fatal error Status in launchpad-buildd: New Status

Re: [Qemu-devel] Dynamic QEMU platform device instantiation in machine files: phone call on Wed July 30

2014-07-30 Thread Eric Auger
Dear all, Please find my notes for today's call. Feel free to correct and add any comments. Best Regards Eric Agenda: discuss dynamic instantiation of QEMU platform devices and especially discuss where we put the code associated to their dt node generation and also qom binding. Attendees: -

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Stefan Berger
Laszlo Ersek wrote on 07/30/2014 12:18:02 PM: > From: Laszlo Ersek > To: "Michael S. Tsirkin" , Stefan Berger/Watson/IBM@IBMUS > Cc: qemu-devel@nongnu.org, Stefan Berger > Date: 07/30/2014 12:18 PM > Subject: Re: [PATCH v2] Add ACPI tables for TPM > > On 07/30/14 18:10, Michael S. Tsirkin wro

Re: [Qemu-devel] [PATCH v7 4/5] QMP: Add support for Archipelago

2014-07-30 Thread Eric Blake
On 07/23/2014 08:07 AM, Chrysostomos Nanakos wrote: > Introduce new enum BlockdevOptionsArchipelago. > > @volume: #Name of the Archipelago volume image > > @mport: #'mport' is the port number on which mapperd is > listening. This is optional and if

[Qemu-devel] [Bug 1098729] Re: qemu-user-static for armhf: segfault in threaded code

2014-07-30 Thread LocutusOfBorg
https://launchpadlibrarian.net/181070813/buildlog_ubuntu-utopic- armhf.hedgewars_0.9.21~alpha~7716~ubuntu14.10.1_FAILEDTOBUILD.txt.gz this started happening after the new deploy of trusty in buildds -- You received this bug notification because you are a member of qemu- devel-ml, which is subscr

Re: [Qemu-devel] [PATCH alt 4/7] block/qcow2: Implement status CB for amend

2014-07-30 Thread Eric Blake
On 07/26/2014 01:22 PM, Max Reitz wrote: > The only really time-consuming operation potentially performed by > qcow2_amend_options() is zero cluster expansion when downgrading qcow2 > images from compat=1.1 to compat=0.10, so report status of that > operation and that operation only through the sta

Re: [Qemu-devel] 9p mapped-* security model infos are architecture-specific

2014-07-30 Thread Aneesh Kumar K.V
Michael Tokarev writes: > Apparently the the mapped-* security models results in a raw bytes > being dumped to host without any architecture normalization (in > host byte order). This may even lead to security issues in guest > when the same files are served from another host for example. > > Th

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Laszlo Ersek
On 07/30/14 18:11, Michael S. Tsirkin wrote: > On Wed, Jul 30, 2014 at 06:07:28PM +0200, Laszlo Ersek wrote: >> On 07/30/14 17:52, Michael S. Tsirkin wrote: >>> How does EFI want to handle TCPA? Does caller allocate it >>> log and fill in the address? >> >> TPM 1.2 seems to be completely absent fr

[Qemu-devel] [Bug 1098729] Re: qemu-user-static for armhf: segfault in threaded code

2014-07-30 Thread LocutusOfBorg
** Changed in: qemu Status: New => Confirmed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1098729 Title: qemu-user-static for armhf: segfault in threaded code Status in QEMU: Confirmed

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Laszlo Ersek
On 07/30/14 18:07, Michael S. Tsirkin wrote: > On Wed, Jul 30, 2014 at 06:02:21PM +0200, Laszlo Ersek wrote: >> On 07/30/14 17:37, Michael S. Tsirkin wrote: >> >>> 1. execute alloc instructions, building a data structure mapping fwcfg >>>file names to memory. >> >> Yes, edk2 currently lacks a g

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Stefan Berger
"Michael S. Tsirkin" wrote on 07/30/2014 12:05:56 PM: > From: "Michael S. Tsirkin" > To: Stefan Berger/Watson/IBM@IBMUS > Cc: Laszlo Ersek , qemu-devel@nongnu.org, Stefan > Berger > Date: 07/30/2014 12:05 PM > Subject: Re: [PATCH v2] Add ACPI tables for TPM > > On Wed, Jul 30, 2014 at 11:59:4

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Laszlo Ersek
On 07/30/14 18:10, Michael S. Tsirkin wrote: > On Wed, Jul 30, 2014 at 12:03:46PM -0400, Stefan Berger wrote: >> Laszlo Ersek wrote on 07/30/2014 11:58:52 AM: >>> In the short term, probably skip TCPA, or advise users in documentation >>> not to enable the TPM device when running OVMF. > > Hmm b

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Michael S. Tsirkin
On Wed, Jul 30, 2014 at 12:03:46PM -0400, Stefan Berger wrote: > Laszlo Ersek wrote on 07/30/2014 11:58:52 AM: > > > From: Laszlo Ersek > > To: Stefan Berger/Watson/IBM@IBMUS > > Cc: "Michael S. Tsirkin" , qemu-devel@nongnu.org, > > Stefan Berger > > Date: 07/30/2014 11:59 AM > > Subject: Re: [

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Michael S. Tsirkin
On Wed, Jul 30, 2014 at 06:02:21PM +0200, Laszlo Ersek wrote: > On 07/30/14 17:37, Michael S. Tsirkin wrote: > > > 1. execute alloc instructions, building a data structure mapping fwcfg > >file names to memory. > > Yes, edk2 currently lacks a good (== sub-linear) dictionary data type. > This

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Laszlo Ersek
On 07/30/14 18:05, Michael S. Tsirkin wrote: > On Wed, Jul 30, 2014 at 11:59:43AM -0400, Stefan Berger wrote: >> "Michael S. Tsirkin" wrote on 07/30/2014 11:50:36 AM: >> >>> From: "Michael S. Tsirkin" >>> To: Stefan Berger/Watson/IBM@IBMUS >>> Cc: Laszlo Ersek , qemu-devel@nongnu.org, Stefan >>>

Re: [Qemu-devel] [PATCH 7/8] block/qcow2: Speed up zero cluster expansion

2014-07-30 Thread Eric Blake
On 07/25/2014 12:07 PM, Max Reitz wrote: > Actually, we do not need to allocate a new data cluster for every zero > cluster to be expanded: It is completely sufficient to rely on qcow2's > COW part and instead create a single zero cluster and reuse it as much > as possible. > > Signed-off-by: Max

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Stefan Berger
Laszlo Ersek wrote on 07/30/2014 12:07:28 PM: > From: Laszlo Ersek > To: "Michael S. Tsirkin" > Cc: Stefan Berger/Watson/IBM@IBMUS, qemu-devel@nongnu.org, Stefan > Berger > Date: 07/30/2014 12:07 PM > Subject: Re: [PATCH v2] Add ACPI tables for TPM > > On 07/30/14 17:52, Michael S. Tsirkin w

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Michael S. Tsirkin
On Wed, Jul 30, 2014 at 06:07:28PM +0200, Laszlo Ersek wrote: > On 07/30/14 17:52, Michael S. Tsirkin wrote: > > On Wed, Jul 30, 2014 at 05:37:26PM +0200, Laszlo Ersek wrote: > >> On 07/30/14 17:10, Stefan Berger wrote: > >>> Laszlo Ersek wrote on 07/30/2014 10:36:38 AM: > >>> > From: Laszlo

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Laszlo Ersek
On 07/30/14 17:52, Michael S. Tsirkin wrote: > On Wed, Jul 30, 2014 at 05:37:26PM +0200, Laszlo Ersek wrote: >> On 07/30/14 17:10, Stefan Berger wrote: >>> Laszlo Ersek wrote on 07/30/2014 10:36:38 AM: >>> From: Laszlo Ersek To: "Michael S. Tsirkin" , Stefan Berger/Watson/IBM@IBMUS

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Michael S. Tsirkin
On Wed, Jul 30, 2014 at 11:59:43AM -0400, Stefan Berger wrote: > "Michael S. Tsirkin" wrote on 07/30/2014 11:50:36 AM: > > > From: "Michael S. Tsirkin" > > To: Stefan Berger/Watson/IBM@IBMUS > > Cc: Laszlo Ersek , qemu-devel@nongnu.org, Stefan > > Berger > > Date: 07/30/2014 11:50 AM > > Subjec

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Stefan Berger
Laszlo Ersek wrote on 07/30/2014 11:58:52 AM: > From: Laszlo Ersek > To: Stefan Berger/Watson/IBM@IBMUS > Cc: "Michael S. Tsirkin" , qemu-devel@nongnu.org, > Stefan Berger > Date: 07/30/2014 11:59 AM > Subject: Re: [PATCH v2] Add ACPI tables for TPM > > On 07/30/14 17:44, Stefan Berger wrote:

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Laszlo Ersek
On 07/30/14 17:37, Michael S. Tsirkin wrote: > 1. execute alloc instructions, building a data structure mapping fwcfg >file names to memory. Yes, edk2 currently lacks a good (== sub-linear) dictionary data type. This week I started porting a red-black tree library that I had originally writte

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Stefan Berger
"Michael S. Tsirkin" wrote on 07/30/2014 11:50:36 AM: > From: "Michael S. Tsirkin" > To: Stefan Berger/Watson/IBM@IBMUS > Cc: Laszlo Ersek , qemu-devel@nongnu.org, Stefan > Berger > Date: 07/30/2014 11:50 AM > Subject: Re: [PATCH v2] Add ACPI tables for TPM > > On Wed, Jul 30, 2014 at 11:29:3

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Laszlo Ersek
On 07/30/14 17:44, Stefan Berger wrote: > Laszlo Ersek wrote on 07/30/2014 11:41:10 AM: > >> From: Laszlo Ersek >> To: Stefan Berger/Watson/IBM@IBMUS, "Michael S. Tsirkin" >> Cc: qemu-devel@nongnu.org, Stefan Berger >> Date: 07/30/2014 11:41 AM >> Subject: Re: [PATCH v2] Add ACPI tables for TP

Re: [Qemu-devel] [PATCH v7 0/5] Support Archipelago as a QEMU block backend

2014-07-30 Thread Stefan Hajnoczi
On Wed, Jul 23, 2014 at 05:07:28PM +0300, Chrysostomos Nanakos wrote: > v7: > - Fix coding style issues. > - Rename __archipelago_submit_request function to archipelago_submit_request. > - Set X_NONBLOCK flag to xseg_receive(). > - Return -EIO to .bdrv_getlength() if archipelago_volume_info() f

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Michael S. Tsirkin
On Wed, Jul 30, 2014 at 05:37:26PM +0200, Laszlo Ersek wrote: > On 07/30/14 17:10, Stefan Berger wrote: > > Laszlo Ersek wrote on 07/30/2014 10:36:38 AM: > > > >> From: Laszlo Ersek > >> To: "Michael S. Tsirkin" , Stefan Berger/Watson/IBM@IBMUS > >> Cc: qemu-devel@nongnu.org, Stefan Berger > >>

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Michael S. Tsirkin
On Wed, Jul 30, 2014 at 11:29:36AM -0400, Stefan Berger wrote: > "Michael S. Tsirkin" wrote on 07/30/2014 11:20:41 AM: > > > From: "Michael S. Tsirkin" > > To: Stefan Berger/Watson/IBM@IBMUS > > Cc: Laszlo Ersek , qemu-devel@nongnu.org, Stefan > > Berger > > Date: 07/30/2014 11:20 AM > > Subjec

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Stefan Berger
Laszlo Ersek wrote on 07/30/2014 11:41:10 AM: > From: Laszlo Ersek > To: Stefan Berger/Watson/IBM@IBMUS, "Michael S. Tsirkin" > Cc: qemu-devel@nongnu.org, Stefan Berger > Date: 07/30/2014 11:41 AM > Subject: Re: [PATCH v2] Add ACPI tables for TPM > > On 07/30/14 17:29, Stefan Berger wrote: >

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Laszlo Ersek
On 07/30/14 17:29, Stefan Berger wrote: > "Michael S. Tsirkin" wrote on 07/30/2014 11:20:41 AM: > >> From: "Michael S. Tsirkin" >> To: Stefan Berger/Watson/IBM@IBMUS >> Cc: Laszlo Ersek , qemu-devel@nongnu.org, Stefan >> Berger >> Date: 07/30/2014 11:20 AM >> Subject: Re: [PATCH v2] Add ACPI ta

Re: [Qemu-devel] [Bug 1349722] [NEW] qemu-io: Exit code is always zero

2014-07-30 Thread Stefan Hajnoczi
On Tue, Jul 29, 2014 at 08:07:44AM -, Maria Kustova wrote: > The qemu-io always returns zero on exit independently on errors occurred > during the command execution. > > Example, > > $ qemu-io -c 'write 128 234' /tmp/run1/test-1/test.img > > offset 128 is not sector aligned > > $ echo $? >

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Michael S. Tsirkin
On Wed, Jul 30, 2014 at 05:15:37PM +0200, Laszlo Ersek wrote: > On 07/30/14 16:46, Michael S. Tsirkin wrote: > > On Wed, Jul 30, 2014 at 04:36:38PM +0200, Laszlo Ersek wrote: > >> On 07/30/14 15:20, Michael S. Tsirkin wrote: > >>> On Tue, Jul 29, 2014 at 06:52:19AM -0400, Stefan Berger wrote: > >>>

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Laszlo Ersek
On 07/30/14 17:10, Stefan Berger wrote: > Laszlo Ersek wrote on 07/30/2014 10:36:38 AM: > >> From: Laszlo Ersek >> To: "Michael S. Tsirkin" , Stefan Berger/Watson/IBM@IBMUS >> Cc: qemu-devel@nongnu.org, Stefan Berger >> Date: 07/30/2014 10:36 AM >> Subject: Re: [PATCH v2] Add ACPI tables for TP

Re: [Qemu-devel] [PATCH 6/8] block/qcow2: Simplify shared L2 handling in amend

2014-07-30 Thread Eric Blake
On 07/25/2014 12:07 PM, Max Reitz wrote: > Currently, we have a bitmap for keeping track of which clusters have > been created during the zero cluster expansion process. This was > necessary because we need to properly increase the refcount for shared > L2 tables. > > However, now we can simply ta

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Stefan Berger
"Michael S. Tsirkin" wrote on 07/30/2014 11:25:25 AM: > From: "Michael S. Tsirkin" > To: Stefan Berger/Watson/IBM@IBMUS > Cc: ler...@redhat.com, qemu-devel@nongnu.org, Stefan Berger > > Date: 07/30/2014 11:25 AM > Subject: Re: [PATCH v2] Add ACPI tables for TPM > > On Wed, Jul 30, 2014 at 11:

Re: [Qemu-devel] [PATCH 5/8] block/qcow2: Implement status CB for amend

2014-07-30 Thread Eric Blake
On 07/25/2014 12:07 PM, Max Reitz wrote: > The only really time-consuming operation potentially performed by > qcow2_amend_options() is zero cluster expansion when downgrading qcow2 > images from compat=1.1 to compat=0.10, so report status of that > operation and that operation only through the sta

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Stefan Berger
"Michael S. Tsirkin" wrote on 07/30/2014 11:20:41 AM: > From: "Michael S. Tsirkin" > To: Stefan Berger/Watson/IBM@IBMUS > Cc: Laszlo Ersek , qemu-devel@nongnu.org, Stefan > Berger > Date: 07/30/2014 11:20 AM > Subject: Re: [PATCH v2] Add ACPI tables for TPM > > On Wed, Jul 30, 2014 at 11:10:2

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Laszlo Ersek
On 07/30/14 17:03, Igor Mammedov wrote: > On Wed, 30 Jul 2014 16:36:38 +0200 > Laszlo Ersek wrote: > >> On 07/30/14 15:20, Michael S. Tsirkin wrote: >>> On Tue, Jul 29, 2014 at 06:52:19AM -0400, Stefan Berger wrote: From: Stefan Berger Add an SSDT ACPI table for the TPM device. >>

Re: [Qemu-devel] [PATCH v2] Add ACPI tables for TPM

2014-07-30 Thread Michael S. Tsirkin
On Wed, Jul 30, 2014 at 11:13:07AM -0400, Stefan Berger wrote: > "Michael S. Tsirkin" wrote on 07/30/2014 11:07:28 AM: > > > > > >If you need bios to allocate this memory, then we will > > > >need a new allocation type for this, add it to linker > > > >in bios and qemu. > > > > > > Why does the

Re: [Qemu-devel] [PATCH 0/2] virtio-rng: Fix memory leaks in virtio_rng_device_realize

2014-07-30 Thread Stefan Hajnoczi
On Tue, Jul 29, 2014 at 07:28:56PM -0400, John Snow wrote: > My previous commit, 713e8a10, did not address the fact that > the error checking pathways do not attempt to call > virtio_cleanup and thus can leak memory in hotplug scenarios. > > This patchset shuffles around the error checking so it d

  1   2   3   >