Re: [Qemu-devel] [PULL 08/11] iscsi: add .bdrv_get_block_status

2013-10-02 Thread Peter Lieven
Am 19.09.2013 15:46, schrieb Paolo Bonzini: > Il 19/09/2013 09:24, Peter Lieven ha scritto: >> do we still need this patch? if yes I can sent one shortly. > No, Stefan prepared it. I'll send a pull request shortly. The patch disables iscsi_get_block_status completely. The C preprocessor can't chec

[Qemu-devel] [PATCH RFC 09/11] s390/qemu: cpu model QMP query-cpu-definitions

2013-10-02 Thread Michael Mueller
This patch implements the QMP command "query-cpu-definitions" in the S390 context. The command returns a descending sorted list of cpu model names derived from the currently (in the current host context) supported set of cpu classes. That means a consumer can successfully request each reported cpu

[Qemu-devel] [PATCH RFC 10/11] s390/qemu: cpu model QMP query-cpu-model

2013-10-02 Thread Michael Mueller
This patch implements a new QMP request named "cpu-model". It returns the cpu model of cpu 0. eg: request: '{"execute" : "query-cpu-model" } answer: '{"return" : "2817-ga1" } Alias names are resolved to their respactive machine type and GA names already during cpu instantiation. Thus, also a cpu

[Qemu-devel] [PATCH] block/iscsi: reenable iscsi_co_get_block_status

2013-10-02 Thread Peter Lieven
Commit f35c934a accidently disabled iscsi_co_get_block_status for all libiscsi versions. Its not possible to check for enumeration constants in the C preprocessor. This patch changes the check to the preprocessor constant LIBISCSI_FEATURE_IOVECTOR which was introduced shortly after get_lba_status s

Re: [Qemu-devel] [PATCH] block/iscsi: reenable iscsi_co_get_block_status

2013-10-02 Thread Paolo Bonzini
Il 02/10/2013 13:52, Peter Lieven ha scritto: > Commit f35c934a accidently disabled iscsi_co_get_block_status for all > libiscsi versions. Its not possible to check for enumeration constants > in the C preprocessor. This patch changes the check to the preprocessor > constant LIBISCSI_FEATURE_IOVECT

Re: [Qemu-devel] [PATCH RFC 10/11] s390/qemu: cpu model QMP query-cpu-model

2013-10-02 Thread Eric Blake
On 10/02/2013 05:33 AM, Michael Mueller wrote: > This patch implements a new QMP request named "cpu-model". It returns > the cpu model of cpu 0. eg: > > request: '{"execute" : "query-cpu-model" } > answer: '{"return" : "2817-ga1" } > > Alias names are resolved to their respactive machine type an

Re: [Qemu-devel] [PATCH V3 2/7] qcow2: free allocated cluster on fail in qcow2_write_snapshots()

2013-10-02 Thread Stefan Hajnoczi
On Mon, Sep 09, 2013 at 10:57:57AM +0800, Wenchao Xia wrote: > Signed-off-by: Wenchao Xia > --- > block/qcow2-snapshot.c |7 +-- > 1 files changed, 5 insertions(+), 2 deletions(-) > > diff --git a/block/qcow2-snapshot.c b/block/qcow2-snapshot.c > index 40393b2..9e2d695 100644 > --- a/blo

Re: [Qemu-devel] [PATCH V3 6/7] qcow2: print message for error path in snapshot creation

2013-10-02 Thread Stefan Hajnoczi
On Mon, Sep 30, 2013 at 04:08:53PM -0600, Eric Blake wrote: > On 09/08/2013 08:58 PM, Wenchao Xia wrote: > > The message will be print out with a macro enabled, which can > > s/print/printed/ > > > be used to check which error path is taken. > > > > Signed-off-by: Wenchao Xia > > --- > > block

Re: [Qemu-devel] [PATCH v6 0/5] Do not set SO_REUSEADDR on Windows

2013-10-02 Thread Eric Blake
On 10/02/2013 04:23 AM, Sebastian Ottlik wrote: > This patchset disables most uses of SO_REUSEADDR on Windows and replaces it > with > calls to the new function socket_set_fast_reuse. On Windows systems the > default > behaviour is equivalent to SO_REUSEADDR on other operating systems. > SO_REUS

Re: [Qemu-devel] [PATCH] block/qcow2: Use bdrv_truncate for size amend

2013-10-02 Thread Stefan Hajnoczi
On Mon, Sep 09, 2013 at 12:17:41PM +0200, Max Reitz wrote: > When amending the size option for a qcow2 image, use bdrv_truncate > instead of qcow2_truncate directly, since the latter will not adjust the > total_sectors count in the BDS structure (whereas the former will). > > Signed-off-by: Max Re

[Qemu-devel] [PATCH V4] disable blkverify external snapshot creation

2013-10-02 Thread Benoît Canet
Hello, Here is the new version of the snapshot forbidding patch. v4: do also the check in bs->file->drv (make it work with quorum) [Benoît] v3: functions return an enum [Jeff] rename forbiding function [Kevin] v2: Use NULL fields to avoid having to fill the new field in every BlockDriv

[Qemu-devel] [PATCH V4] block: Add BlockDriver.bdrv_check_ext_snapshot.

2013-10-02 Thread Benoît Canet
This field is used by blkverify to disable external snapshots creation. I will also be used by block filters like quorum to disable external snapshots creation. Signed-off-by: Benoit Canet --- block.c | 17 + block/blkverify.c | 2 ++ blockdev.c

[Qemu-devel] [PATCH V9 00/11] Quorum block driver

2013-10-02 Thread Benoît Canet
It must be applied on top of "block: Add BlockDriver.bdrv_check_ext_snapshot." This patchset create a block driver implementing a quorum using total qemu disk images. Writes are mirrored on the $total files. For the reading part the $total files are read at the same time and a vote is done to dete

[Qemu-devel] [PATCH V9 01/11] quorum: Create quorum.c, add QuorumSingleAIOCB and QuorumAIOCB.

2013-10-02 Thread Benoît Canet
Signed-off-by: Benoit Canet --- block/Makefile.objs | 1 + block/quorum.c | 54 + 2 files changed, 55 insertions(+) create mode 100644 block/quorum.c diff --git a/block/Makefile.objs b/block/Makefile.objs index 3bb85b5..05a65c2 100644 --

[Qemu-devel] [PATCH V9 10/11] quorum: Add quorum_co_flush().

2013-10-02 Thread Benoît Canet
Makes a vote to select error if any. Signed-off-by: Benoit Canet --- block/quorum.c | 34 ++ 1 file changed, 34 insertions(+) diff --git a/block/quorum.c b/block/quorum.c index 084d030..0a28ab1 100644 --- a/block/quorum.c +++ b/block/quorum.c @@ -641,12 +641,46 @

[Qemu-devel] [PATCH V9 04/11] blkverify: Extract qemu_iovec_clone() and qemu_iovec_compare() from blkverify.

2013-10-02 Thread Benoît Canet
Signed-off-by: Benoit Canet --- block/blkverify.c | 108 +- include/qemu-common.h | 2 + util/iov.c| 103 +++ 3 files changed, 107 insertions(+), 106 deletions(-) diff --git a/block/blkv

[Qemu-devel] [PATCH V9 02/11] quorum: Create BDRVQuorumState and BlkDriver and do init.

2013-10-02 Thread Benoît Canet
Signed-off-by: Benoit Canet --- block/quorum.c | 25 + 1 file changed, 25 insertions(+) diff --git a/block/quorum.c b/block/quorum.c index 76a1fbb..9557e61 100644 --- a/block/quorum.c +++ b/block/quorum.c @@ -15,6 +15,16 @@ #include "block/block_int.h" +/* the follow

[Qemu-devel] [PATCH V9 07/11] quorum: Add quorum_getlength().

2013-10-02 Thread Benoît Canet
Check that every bs file return the same length. If not return -EIO to disable the quorum and avoid length discrepancy. Signed-off-by: Benoit Canet --- block/quorum.c | 26 ++ 1 file changed, 26 insertions(+) diff --git a/block/quorum.c b/block/quorum.c index e235ac1..b3

[Qemu-devel] [PATCH V9 05/11] quorum: Add quorum_aio_readv.

2013-10-02 Thread Benoît Canet
Signed-off-by: Benoit Canet --- block/quorum.c | 40 +++- 1 file changed, 39 insertions(+), 1 deletion(-) diff --git a/block/quorum.c b/block/quorum.c index b49e3c6..f0fc0e9 100644 --- a/block/quorum.c +++ b/block/quorum.c @@ -101,14 +101,23 @@ static int quor

[Qemu-devel] [PATCH RFC v2 2/9] hw/pci: add pci wrappers for allocating and asserting irqs

2013-10-02 Thread Marcel Apfelbaum
Interrupt pin is selected and saved into PCI_INTERRUPT_PIN register during device initialization. Devices should not call directly qemu_set_irq and specify the INTx pin on each call. Added pci_* wrappers to replace qemu_set_irq, qemu_irq_raise, qemu_irq_lower and qemu_irq_pulse, setting the irq ba

[Qemu-devel] [PATCH V9 08/11] quorum: Add quorum_invalidate_cache().

2013-10-02 Thread Benoît Canet
Signed-off-by: Benoit Canet --- block/quorum.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/block/quorum.c b/block/quorum.c index b3649a4..223ad22 100644 --- a/block/quorum.c +++ b/block/quorum.c @@ -565,6 +565,16 @@ static int64_t quorum_getlength(BlockDriverState *bs) r

[Qemu-devel] [PATCH V9 06/11] quorum: Add quorum mechanism.

2013-10-02 Thread Benoît Canet
Use gnutls's SHA-256 to compare versions. Signed-off-by: Benoit Canet --- block/Makefile.objs | 2 +- block/quorum.c| 321 +- configure | 36 ++ include/monitor/monitor.h | 2 + monitor.c | 2

[Qemu-devel] [PATCH V9 09/11] quorum: Add quorum_co_get_block_status.

2013-10-02 Thread Benoît Canet
Signed-off-by: Benoit Canet --- block/quorum.c | 67 ++ 1 file changed, 67 insertions(+) diff --git a/block/quorum.c b/block/quorum.c index 223ad22..084d030 100644 --- a/block/quorum.c +++ b/block/quorum.c @@ -171,6 +171,22 @@ static int qu

[Qemu-devel] [PATCH RFC v2 4/9] hw/vmxnet3: set interrupts using pci irq wrappers

2013-10-02 Thread Marcel Apfelbaum
pci_set_irq uses PCI_INTERRUPT_PIN config register to compute device INTx pin to set. An assert is used to ensure that intx received from the quest OS corresponds to PCI_INTERRUPT_PIN. Signed-off-by: Marcel Apfelbaum --- hw/net/vmxnet3.c | 13 +++-- 1 file changed, 11 insertions(+), 2 d

[Qemu-devel] [PATCH RFC v2 1/9] hw/core: Add interface to allocate and free a single IRQ

2013-10-02 Thread Marcel Apfelbaum
qemu_allocate_irq returns a single qemu_irq. The interface allows to specify an interrupt number. qemu_free_irq frees it. Signed-off-by: Marcel Apfelbaum --- hw/core/irq.c| 16 include/hw/irq.h | 7 +++ 2 files changed, 23 insertions(+) diff --git a/hw/core/irq.c b/hw

[Qemu-devel] [PATCH V9 11/11] quorum: Add quorum_open() and quorum_close().

2013-10-02 Thread Benoît Canet
Example of command line: -drive if=virtio,file.driver=quorum,\ file.children.0.file.filename=1.qcow2,\ file.children.1.file.filename=2.qcow2,\ file.children.2.file.filename=3.qcow2,\ file.vote_threshold=3 file.blkverify=on with file.vote_threshold=2 and two file can be passed to emulated blkverify

[Qemu-devel] [PATCH RFC v2 3/9] hw/pci-bridge: set PCI_INTERRUPT_PIN register before shpc init

2013-10-02 Thread Marcel Apfelbaum
The PCI_INTERRUPT_PIN will be used by shpc init, so was moved before the call to shpc_init. Signed-off-by: Marcel Apfelbaum --- hw/pci-bridge/pci_bridge_dev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/pci-bridge/pci_bridge_dev.c b/hw/pci-bridge/pci_bridge_dev.c index

[Qemu-devel] [PATCH RFC v2 6/9] hw/xhci: set interrupts using pci irq wrappers

2013-10-02 Thread Marcel Apfelbaum
pci_set_irq uses PCI_INTERRUPT_PIN config register to compute device INTx pin to assert/deassert. Removed irq field from xhci state. Signed-off-by: Marcel Apfelbaum --- hw/usb/hcd-xhci.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/hw/usb/hcd-xhci.c b/hw/usb/hcd-xh

Re: [Qemu-devel] [PATCH v2 uq/master 0/2] KVM: issues with XSAVE support

2013-10-02 Thread Paolo Bonzini
Il 13/09/2013 15:55, Paolo Bonzini ha scritto: > This series fixes two migration bugs concerning KVM's XSAVE ioctls. > The second right now is only a theoretical problem, since the only > XSAVE-specific state is AVX and all machines with XSAVE also have AVX. > In the future, this will ensure that m

[Qemu-devel] [PATCH RFC v2 7/9] hw: set interrupts using pci irq wrappers

2013-10-02 Thread Marcel Apfelbaum
pci_set_irq and the other pci irq wrappers use PCI_INTERRUPT_PIN config register to compute device INTx pin to assert/deassert. An irq is allocated using pci_allocate_irq wrapper only if is needed by non pci devices. Signed-off-by: Marcel Apfelbaum --- hw/audio/ac97.c| 4 ++-- hw/audio/

[Qemu-devel] [PATCH RFC v2 8/9] hw/pcie: AER and hot-plug events must use device's interrupt

2013-10-02 Thread Marcel Apfelbaum
The fields hpev_intx and aer_intx were removed because both AER and hot-plug events must use device's interrupt. Assert/deassert interrupts using pci_set_irq wrapper instead. Signed-off-by: Marcel Apfelbaum --- hw/pci/pcie.c | 4 ++-- hw/pci/pcie_aer.c | 4 ++-- include/hw/pci/pcie

Re: [Qemu-devel] [PATCH RFC v2 2/9] hw/pci: add pci wrappers for allocating and asserting irqs

2013-10-02 Thread Michael S. Tsirkin
On Wed, Oct 02, 2013 at 03:41:27PM +0300, Marcel Apfelbaum wrote: > Interrupt pin is selected and saved into PCI_INTERRUPT_PIN > register during device initialization. Devices should not call > directly qemu_set_irq and specify the INTx pin on each call. > > Added pci_* wrappers to replace qemu_se

Re: [Qemu-devel] [PATCH RFC v2 0/9] hw/pci: set irq without selecting INTx pin

2013-10-02 Thread Michael S. Tsirkin
On Wed, Oct 02, 2013 at 03:41:25PM +0300, Marcel Apfelbaum wrote: > Note: Added RFC because not all affected devices were > checked yet. What do you have in mind exactly? > Interrupt pin is selected and saved into PCI_INTERRUPT_PIN > register during device initialization. Devices should no

[Qemu-devel] [PATCH RFC v2 5/9] hw/vfio: set interrupts using pci irq wrappers

2013-10-02 Thread Marcel Apfelbaum
pci_set_irq and the other pci irq wrappers use PCI_INTERRUPT_PIN config register to compute device INTx pin to assert/deassert. Save INTx pin into the config register before calling pci_set_irq Signed-off-by: Marcel Apfelbaum --- hw/misc/vfio.c | 11 ++- 1 file changed, 6 insertions(+),

[Qemu-devel] [PATCH RFC v2 9/9] hw/pci: removed irq field from PCIDevice

2013-10-02 Thread Marcel Apfelbaum
Instead of exposing the the irq field, pci wrappers to qemu_set_irq or qemu_irq_* can be used. Signed-off-by: Marcel Apfelbaum --- hw/pci/pci.c | 2 -- include/hw/pci/pci.h | 3 --- 2 files changed, 5 deletions(-) diff --git a/hw/pci/pci.c b/hw/pci/pci.c index fbfd8f7..f8d0b81 100644 --

[Qemu-devel] [PATCH RFC v2 0/9] hw/pci: set irq without selecting INTx pin

2013-10-02 Thread Marcel Apfelbaum
Note: Added RFC because not all affected devices were checked yet. Interrupt pin is selected and saved into PCI_INTERRUPT_PIN register during device initialization. Devices should not call directly qemu_set_irq and specify the INTx pin. Added pci_* wrappers to replace qemu_set_irq, qemu_ir

Re: [Qemu-devel] [PATCH RFC v2 2/9] hw/pci: add pci wrappers for allocating and asserting irqs

2013-10-02 Thread Marcel Apfelbaum
On Wed, 2013-10-02 at 15:54 +0300, Michael S. Tsirkin wrote: > On Wed, Oct 02, 2013 at 03:41:27PM +0300, Marcel Apfelbaum wrote: > > Interrupt pin is selected and saved into PCI_INTERRUPT_PIN > > register during device initialization. Devices should not call > > directly qemu_set_irq and specify th

Re: [Qemu-devel] [PATCH] block/qcow2: Use bdrv_truncate for size amend

2013-10-02 Thread Max Reitz
On 2013-10-02 14:32, Stefan Hajnoczi wrote: On Mon, Sep 09, 2013 at 12:17:41PM +0200, Max Reitz wrote: When amending the size option for a qcow2 image, use bdrv_truncate instead of qcow2_truncate directly, since the latter will not adjust the total_sectors count in the BDS structure (whereas the

[Qemu-devel] [PATCH V9 03/11] quorum: Add quorum_aio_writev and its dependencies.

2013-10-02 Thread Benoît Canet
Signed-off-by: Benoit Canet --- block/quorum.c | 123 + 1 file changed, 123 insertions(+) diff --git a/block/quorum.c b/block/quorum.c index 9557e61..b49e3c6 100644 --- a/block/quorum.c +++ b/block/quorum.c @@ -64,11 +64,134 @@ struct Quoru

Re: [Qemu-devel] [PATCH RFC v2 0/9] hw/pci: set irq without selecting INTx pin

2013-10-02 Thread Marcel Apfelbaum
On Wed, 2013-10-02 at 15:58 +0300, Michael S. Tsirkin wrote: > On Wed, Oct 02, 2013 at 03:41:25PM +0300, Marcel Apfelbaum wrote: > > Note: Added RFC because not all affected devices were > > checked yet. > > What do you have in mind exactly? Sanity test for *all* modified devices. Any other

Re: [Qemu-devel] [PATCH RFC v2 1/9] hw/core: Add interface to allocate and free a single IRQ

2013-10-02 Thread Michael S. Tsirkin
On Wed, Oct 02, 2013 at 03:41:26PM +0300, Marcel Apfelbaum wrote: > qemu_allocate_irq returns a single qemu_irq. > The interface allows to specify an interrupt number. > > qemu_free_irq frees it. > > Signed-off-by: Marcel Apfelbaum > --- > hw/core/irq.c| 16 > include/hw/ir

Re: [Qemu-devel] [PATCH v2 0/9] target-s390 tcg improvements

2013-10-02 Thread Alexander Graf
On 01.10.2013, at 19:17, Richard Henderson wrote: > Changes v1-v2: > * Squashed patches 1, 2, 4. The feedback from patch 1 called for a >re-ordering of the patch set to better deal with STFL, but I could >not find an ordering that worked well. I think this is probably >just as clea

Re: [Qemu-devel] [Bug 1233225] Re: mips/mipsel linux user float division problem

2013-10-02 Thread Peter Maydell
On 2 October 2013 14:22, Stefan Weil wrote: > The original bug report said that it runs in QEMU system emulation > (which I did not test because of lack of time). As system emulation > uses the same cpu, it should be fine. ...that's what prompted me to ask, actually -- system emulation will pick

Re: [Qemu-devel] [PATCH 1/7] usb: remove old usb-host code

2013-10-02 Thread Jan Kiszka
On 2013-09-19 11:34, Gerd Hoffmann wrote: > The usb-host code has been rewritten for qemu 1.5 to use libusb, > the old code has been left in as temporary fallback. Now we are > two releases further out, targeting the 1.7 release. No major > issues with the new code poped up until now. Time to re

Re: [Qemu-devel] [PATCH v7 00/27] qemu: generate acpi tables for the guest

2013-10-02 Thread Igor Mammedov
On Wed, 2 Oct 2013 00:26:11 +0300 "Michael S. Tsirkin" wrote: > This code can also be found here: > git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git acpi > > While this patch still uses info not available in QOM, I think it's reasonable > to merge it and then refactor as QOM properties cover m

Re: [Qemu-devel] [PATCH] block: vhdx - add migration blocker

2013-10-02 Thread Stefan Hajnoczi
On Tue, Oct 01, 2013 at 11:59:20AM -0400, Jeff Cody wrote: > This blocks migration for VHDX image files, until the > functionality can be supported. > > Signed-off-by: Jeff Cody > --- > block/vhdx.c | 10 ++ > 1 file changed, 10 insertions(+) Strictly speaking live migration is safe sin

Re: [Qemu-devel] [PATCH V3 1/7] qcow2: restore nb_snapshots when fail in snapshot creation

2013-10-02 Thread Stefan Hajnoczi
On Mon, Sep 09, 2013 at 10:57:56AM +0800, Wenchao Xia wrote: > If it is not restored after qcow2_write_snapshots() fail, a core > dump will happen in bdrv_close() since access of invalid pointer. > > Signed-off-by: Wenchao Xia > --- > block/qcow2-snapshot.c |4 +++- > 1 files changed, 3 inse

Re: [Qemu-devel] [PATCH V3 0/7] qcow2: rollback the modification on fail in snapshot creation

2013-10-02 Thread Stefan Hajnoczi
On Mon, Sep 09, 2013 at 10:57:55AM +0800, Wenchao Xia wrote: > V2: > 1: all fail case will goto fail section. > 2: add the goto code. > v3: > Address Stefan's comments: > 2: don't goto fail after allocation failure. > 3: use sn->l1size correctly in qcow2_free_cluster(). > 4-7: add test

Re: [Qemu-devel] [PATCH v7 00/27] qemu: generate acpi tables for the guest

2013-10-02 Thread Michael S. Tsirkin
On Wed, Oct 02, 2013 at 03:05:52PM +0200, Igor Mammedov wrote: > On Wed, 2 Oct 2013 00:26:11 +0300 > "Michael S. Tsirkin" wrote: > > > This code can also be found here: > > git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git acpi > > > > While this patch still uses info not available in QOM, I th

Re: [Qemu-devel] [PATCH] qcow2: Switch L1 table in a single sequence

2013-10-02 Thread Stefan Hajnoczi
On Mon, Sep 30, 2013 at 05:57:21PM +0200, Max Reitz wrote: > Switching the L1 table in memory should be an atomic operation, as far > as possible. Calling qcow2_free_clusters on the old L1 table on disk is > not a good idea when the old L1 table is no longer valid and the address > to the new one h

Re: [Qemu-devel] [PATCH] block: use correct filename for error report

2013-10-02 Thread Dunrong Huang
On Wed, Oct 2, 2013 at 5:48 PM, Stefan Hajnoczi wrote: > On Tue, Sep 24, 2013 at 06:14:01PM +0800, Dunrong Huang wrote: > > The content filename point to will be erased by qemu_opts_absorb_qdict() > > in raw_open_common() in drv->bdrv_file_open() > > > > So it's better to use bs->filename. > > >

[Qemu-devel] [Bug 1234179] [NEW] QEMU segfaults during Windows 7 unattended install

2013-10-02 Thread Lucas Meneghel Rodrigues
Public bug reported: During today's automated qemu.git testing, a segmentation fault while installing Windows 7 SP1 happened. qemu.git top commit: 10/02 01:30:24 INFO | git:0150| git commit ID is a684f3cf9b9b9c3cb82be87aafc463de8974610c (tag v1.4.0-4237-ga684f3c) commit a684f3cf9b9b9c3cb

Re: [Qemu-devel] [PATCH v7 00/27] qemu: generate acpi tables for the guest

2013-10-02 Thread Igor Mammedov
On Wed, 2 Oct 2013 16:30:36 +0300 "Michael S. Tsirkin" wrote: > On Wed, Oct 02, 2013 at 03:05:52PM +0200, Igor Mammedov wrote: > > On Wed, 2 Oct 2013 00:26:11 +0300 > > "Michael S. Tsirkin" wrote: > > > > > This code can also be found here: > > > git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.g

Re: [Qemu-devel] [PATCH -V4 2/4] target-ppc: Fix page table lookup with kvm enabled

2013-10-02 Thread Alexander Graf
On 01.10.2013, at 03:27, Aneesh Kumar K.V wrote: > Alexander Graf writes: > >> On 09/05/2013 10:16 AM, Aneesh Kumar K.V wrote: >>> From: "Aneesh Kumar K.V" >>> >>> With kvm enabled, we store the hash page table information in the >>> hypervisor. >>> Use ioctl to read the htab contents. Withou

Re: [Qemu-devel] [PATCH v7 00/27] qemu: generate acpi tables for the guest

2013-10-02 Thread Michael S. Tsirkin
On Wed, Oct 02, 2013 at 03:52:17PM +0200, Igor Mammedov wrote: > On Wed, 2 Oct 2013 16:30:36 +0300 > "Michael S. Tsirkin" wrote: > > > On Wed, Oct 02, 2013 at 03:05:52PM +0200, Igor Mammedov wrote: > > > On Wed, 2 Oct 2013 00:26:11 +0300 > > > "Michael S. Tsirkin" wrote: > > > > > > > This code

Re: [Qemu-devel] [PATCH -V4 RESEND 0/6] target-ppc: Add support for dumping guest memory using qemu gdb server

2013-10-02 Thread Alexander Graf
On 01.10.2013, at 18:19, Aneesh Kumar K.V wrote: > Hi, > > This patch series implement support for dumping guest memory using qemu gdb > server. The last patch also enable qemu monitor command dump-guest-memory Thanks, applied all but 2/6 to ppc-next. I think the core dump bits should be more

[Qemu-devel] [PATCHv4] block/get_block_status: avoid redundant callouts on raw devices

2013-10-02 Thread Peter Lieven
if a raw device like an iscsi target or host device is used the current implementation makes a second call out to get the block status of bs->file. however, the raw driver already has called bdrv_get_block_status on bs->file. v4: use a flag to detect the raw driver instead of the strncmp hack.

Re: [Qemu-devel] [PATCH RFC 10/11] s390/qemu: cpu model QMP query-cpu-model

2013-10-02 Thread Michael Mueller
On Wed, 02 Oct 2013 06:06:32 -0600 Eric Blake wrote: > On 10/02/2013 05:33 AM, Michael Mueller wrote: > > This patch implements a new QMP request named "cpu-model". It returns > > the cpu model of cpu 0. eg: > > > > request: '{"execute" : "query-cpu-model" } > > answer: '{"return" : "2817-ga1"

[Qemu-devel] [PATCH] qemu-iotests: Correct 026 output

2013-10-02 Thread Max Reitz
Because l2_allocate now frees the unused L2 cluster on error, the according test cases in 026 don't result in one leaked cluster anymore. Signed-off-by: Max Reitz --- This patch depends on and is a follow-up to "qcow2: Free allocated L2 cluster on error" which was part of the "qcow2: Small error

Re: [Qemu-devel] [PATCHv4] block/get_block_status: avoid redundant callouts on raw devices

2013-10-02 Thread Stefan Hajnoczi
On Wed, Oct 02, 2013 at 04:20:36PM +0200, Peter Lieven wrote: > if a raw device like an iscsi target or host device is used > the current implementation makes a second call out to get > the block status of bs->file. however, the raw driver already > has called bdrv_get_block_status on bs->file. >

Re: [Qemu-devel] [PATCHv4] block/get_block_status: avoid redundant callouts on raw devices

2013-10-02 Thread Eric Blake
On 10/02/2013 08:20 AM, Peter Lieven wrote: > if a raw device like an iscsi target or host device is used > the current implementation makes a second call out to get > the block status of bs->file. however, the raw driver already > has called bdrv_get_block_status on bs->file. > > v4: use a flag t

Re: [Qemu-devel] [PATCHv4] block/get_block_status: avoid redundant callouts on raw devices

2013-10-02 Thread Paolo Bonzini
Il 02/10/2013 17:06, Stefan Hajnoczi ha scritto: > Sorry I didn't review this earlier but this flag looks hacky and I'm not > confident about merging the patch yet. > > The patch makes me wonder if the raw_bsd driver should avoid calling > bs->file itself: > > return BDRV_BLOCK_DATA | BDRV_BLOCK_

Re: [Qemu-devel] [PATCH RFC v2 2/9] hw/pci: add pci wrappers for allocating and asserting irqs

2013-10-02 Thread Paolo Bonzini
Il 02/10/2013 14:41, Marcel Apfelbaum ha scritto: > +static inline void pci_irq_pulse(PCIDevice *pci_dev) > +{ > +pci_irq_lower(pci_dev); > +pci_irq_raise(pci_dev); > +} > + Why is this in the opposite order, compared to qemu_irq_pulse? Paolo

Re: [Qemu-devel] [PATCH v2 uq/master 2/2] x86: cpuid: reconstruct leaf 0Dh data

2013-10-02 Thread Gleb Natapov
On Fri, Sep 13, 2013 at 03:55:58PM +0200, Paolo Bonzini wrote: > The data in leaf 0Dh depends on information from other feature bits. > Instead of passing it blindly from the host, compute it based on > whether these feature bits are enabled. > > Signed-off-by: Paolo Bonzini > --- > target-i386/

Re: [Qemu-devel] [PATCH v2 0/6] Improve getauxval support

2013-10-02 Thread Richard Henderson
On 10/01/2013 06:42 PM, Peter Maydell wrote: > On 2 October 2013 06:25, Richard Henderson wrote: >> Ping. > > Does this need updating now that commit 03cfd8faa (which manually > walks through the auxval in get_execfd()) has been committed? Well, such a patch could be added to the series, but it

Re: [Qemu-devel] [PATCHv3 02/20] block: add flags to bdrv_*_write_zeroes

2013-10-02 Thread Eric Blake
On 09/24/2013 07:34 AM, Peter Lieven wrote: > Signed-off-by: Peter Lieven > --- > block-migration.c |2 +- > block.c | 20 +++- > block/backup.c|3 ++- > block/qcow2-cluster.c |2 +- > block/qcow2.c |2 +- > b

Re: [Qemu-devel] [PATCH V4] block: Add BlockDriver.bdrv_check_ext_snapshot.

2013-10-02 Thread Max Reitz
On 2013-10-02 14:33, Benoît Canet wrote: This field is used by blkverify to disable external snapshots creation. I will also be used by block filters like quorum to disable external snapshots creation. Oh, I nearly missed it: s/I will also be/It will also be/ and probably s/external snapshots cr

Re: [Qemu-devel] [PATCHv4] block/get_block_status: avoid redundant callouts on raw devices

2013-10-02 Thread Peter Lieven
Am 02.10.2013 17:13, schrieb Paolo Bonzini: > Il 02/10/2013 17:06, Stefan Hajnoczi ha scritto: >> Sorry I didn't review this earlier but this flag looks hacky and I'm not >> confident about merging the patch yet. >> >> The patch makes me wonder if the raw_bsd driver should avoid calling >> bs->file

Re: [Qemu-devel] [PATCH V4] block: Add BlockDriver.bdrv_check_ext_snapshot.

2013-10-02 Thread Max Reitz
On 2013-10-02 14:33, Benoît Canet wrote: This field is used by blkverify to disable external snapshots creation. I will also be used by block filters like quorum to disable external snapshots creation. Signed-off-by: Benoit Canet --- block.c | 17 + block/bl

Re: [Qemu-devel] [PATCH v2 uq/master 2/2] x86: cpuid: reconstruct leaf 0Dh data

2013-10-02 Thread Paolo Bonzini
Il 02/10/2013 17:21, Gleb Natapov ha scritto: >> -if (kvm_enabled()) { >> -KVMState *s = cs->kvm_state; >> +kvm_mask = >> +kvm_arch_get_supported_cpuid(s, 0xd, 0, R_EAX) | >> +((uint64_t)kvm_arch_get_supported_cpuid(s, 0xd, 0, R_EDX) << >> 32); >

Re: [Qemu-devel] [PATCH v2 uq/master 2/2] x86: cpuid: reconstruct leaf 0Dh data

2013-10-02 Thread Gleb Natapov
On Wed, Oct 02, 2013 at 05:37:31PM +0200, Paolo Bonzini wrote: > Il 02/10/2013 17:21, Gleb Natapov ha scritto: > >> -if (kvm_enabled()) { > >> -KVMState *s = cs->kvm_state; > >> +kvm_mask = > >> +kvm_arch_get_supported_cpuid(s, 0xd, 0, R_EAX) | > >> +

Re: [Qemu-devel] [PATCHv3 04/20] block: introduce bdrv_has_discard_zeroes and bdrv_has_discard_write_zeroes

2013-10-02 Thread Eric Blake
On 09/24/2013 07:34 AM, Peter Lieven wrote: > Signed-off-by: Peter Lieven > --- > block.c | 29 + > include/block/block.h |2 ++ > include/block/block_int.h | 13 + > 3 files changed, 44 insertions(+) > Reviewed-by: Eric Blak

[Qemu-devel] [PATCH] block/iscsi: introduce bdrv_co_{readv, writev, flush_to_disk}

2013-10-02 Thread Peter Lieven
this converts read, write and flush functions from aio to coroutines. Signed-off-by: Peter Lieven --- block/iscsi.c | 405 +++-- 1 file changed, 107 insertions(+), 298 deletions(-) diff --git a/block/iscsi.c b/block/iscsi.c index 5c7baee..f37

Re: [Qemu-devel] [PATCH V4] block: Add BlockDriver.bdrv_check_ext_snapshot.

2013-10-02 Thread Eric Blake
On 10/02/2013 09:33 AM, Max Reitz wrote: > On 2013-10-02 14:33, Benoît Canet wrote: >> This field is used by blkverify to disable external snapshots creation. >> I will also be used by block filters like quorum to disable external >> snapshots >> creation. > Oh, I nearly missed it: s/I will also be

Re: [Qemu-devel] [PATCHv3 05/20] block/raw: add bdrv_has_discard_zeroes and bdrv_has_discard_write_zeroes

2013-10-02 Thread Eric Blake
On 09/24/2013 07:34 AM, Peter Lieven wrote: > Signed-off-by: Peter Lieven > --- > block/raw_bsd.c | 56 > +-- > 1 file changed, 34 insertions(+), 22 deletions(-) > Reviewed-by: Eric Blake -- Eric Blake eblake redhat com+1-919-301-3

Re: [Qemu-devel] [PATCH] block/iscsi: introduce bdrv_co_{readv, writev, flush_to_disk}

2013-10-02 Thread Paolo Bonzini
Il 02/10/2013 17:41, Peter Lieven ha scritto: > this converts read, write and flush functions from aio to coroutines. I'm not sure it's already the time for this... Cancellation sucks in QEMU, and this is going to make things even worse. Paolo > Signed-off-by: Peter Lieven > --- > block/iscsi

Re: [Qemu-devel] [PATCHv3 06/20] block: add BlockLimits structure to BlockDriverState

2013-10-02 Thread Eric Blake
On 09/24/2013 07:35 AM, Peter Lieven wrote: > this patch adds BlockLimits which introduces discard and write_zeroes > limits and alignment information to the BlockDriverState. > > Signed-off-by: Peter Lieven > --- > include/block/block_int.h | 17 + > 1 file changed, 17 inserti

Re: [Qemu-devel] [PATCH] block/iscsi: introduce bdrv_co_{readv, writev, flush_to_disk}

2013-10-02 Thread Peter Lieven
Am 02.10.2013 17:46, schrieb Paolo Bonzini: > Il 02/10/2013 17:41, Peter Lieven ha scritto: >> this converts read, write and flush functions from aio to coroutines. > I'm not sure it's already the time for this... Cancellation sucks in > QEMU, and this is going to make things even worse. Ok, I was

[Qemu-devel] [PATCH v3 uq/master 2/2] x86: cpuid: reconstruct leaf 0Dh data

2013-10-02 Thread Paolo Bonzini
The data in leaf 0Dh depends on information from other feature bits. Instead of passing it blindly from the host, compute it based on whether these feature bits are enabled. Signed-off-by: Paolo Bonzini --- target-i386/cpu.c | 65 --- 1 file ch

Re: [Qemu-devel] [PATCH] block/iscsi: introduce bdrv_co_{readv, writev, flush_to_disk}

2013-10-02 Thread Paolo Bonzini
Il 02/10/2013 17:54, Peter Lieven ha scritto: >> > I'm not sure it's already the time for this... Cancellation sucks in >> > QEMU, and this is going to make things even worse. > Ok, I was not aware. I talked with Kevin about this some months ago > and promised (list CCed) to prepare this for 1.7.0

Re: [Qemu-devel] [PATCH] block/iscsi: introduce bdrv_co_{readv, writev, flush_to_disk}

2013-10-02 Thread Peter Lieven
Am 02.10.2013 17:55, schrieb Paolo Bonzini: > Il 02/10/2013 17:54, Peter Lieven ha scritto: I'm not sure it's already the time for this... Cancellation sucks in QEMU, and this is going to make things even worse. >> Ok, I was not aware. I talked with Kevin about this some months ago >> an

Re: [Qemu-devel] [PATCH RFC v2 5/9] hw/vfio: set interrupts using pci irq wrappers

2013-10-02 Thread Alex Williamson
On Wed, 2013-10-02 at 15:41 +0300, Marcel Apfelbaum wrote: > pci_set_irq and the other pci irq wrappers use > PCI_INTERRUPT_PIN config register to compute device > INTx pin to assert/deassert. > > Save INTx pin into the config register before calling > pci_set_irq > > Signed-off-by: Marcel Apfelb

Re: [Qemu-devel] [PATCHv4] block/get_block_status: avoid redundant callouts on raw devices

2013-10-02 Thread Peter Lieven
Am 02.10.2013 17:13, schrieb Paolo Bonzini: > Il 02/10/2013 17:06, Stefan Hajnoczi ha scritto: >> Sorry I didn't review this earlier but this flag looks hacky and I'm not >> confident about merging the patch yet. >> >> The patch makes me wonder if the raw_bsd driver should avoid calling >> bs->file

Re: [Qemu-devel] [PATCH RFC v2 0/9] hw/pci: set irq without selecting INTx pin

2013-10-02 Thread Alex Williamson
On Wed, 2013-10-02 at 16:05 +0300, Marcel Apfelbaum wrote: > On Wed, 2013-10-02 at 15:58 +0300, Michael S. Tsirkin wrote: > > On Wed, Oct 02, 2013 at 03:41:25PM +0300, Marcel Apfelbaum wrote: > > > Note: Added RFC because not all affected devices were > > > checked yet. > > > > What do you

Re: [Qemu-devel] [PATCH v3 uq/master 2/2] x86: cpuid: reconstruct leaf 0Dh data

2013-10-02 Thread Gleb Natapov
On Wed, Oct 02, 2013 at 05:54:57PM +0200, Paolo Bonzini wrote: > The data in leaf 0Dh depends on information from other feature bits. > Instead of passing it blindly from the host, compute it based on > whether these feature bits are enabled. > Applied both. Thanks. > Signed-off-by: Paolo Bonzini

Re: [Qemu-devel] [Qemu-trivial] [PATCH] qemu-char: Fix potential out of bounds access to local arrays

2013-10-02 Thread Michael Tokarev
01.10.2013 01:04, Stefan Weil wrote: Latest gcc-4.8 supports a new option -fsanitize=address which activates an AddressSanitizer. This AddressSanitizer stops the QEMU system emulation very early because two character arrays of size 8 are potentially written with 9 bytes. Commit 6ea314d91439741e9

Re: [Qemu-devel] [Qemu-trivial] [PATCH] vl: Clean up unnecessary boot_order complications

2013-10-02 Thread Michael Tokarev
01.10.2013 15:47, arm...@redhat.com пишет: From: Markus Armbruster Messed up in commit 8281abd. Thanks, applied to the trivial patches queue. /mjt

Re: [Qemu-devel] [Qemu-trivial] [PATCH] hw/9pfs: Fix errno value for xattr functions

2013-10-02 Thread Michael Tokarev
01.10.2013 15:28, Daniel P. Berrange пишет: From: "Daniel P. Berrange" If there is no operation driver for the xattr type the functions return '-1' and set errno to '-EOPNOTSUPP'. When the calling code sets 'ret = -errno' this turns into a large positive number. In Linux 3.11, the kernel has s

Re: [Qemu-devel] [PATCHv3 07/20] block: honour BlockLimits in bdrv_co_do_write_zeroes

2013-10-02 Thread Eric Blake
On 09/24/2013 07:35 AM, Peter Lieven wrote: > Signed-off-by: Peter Lieven > --- > block.c | 65 > +++ > 1 file changed, 49 insertions(+), 16 deletions(-) > > diff --git a/block.c b/block.c > index ac35cb5..580570a 100644 > --- a/bloc

Re: [Qemu-devel] [PATCHv3 08/20] block: honour BlockLimits in bdrv_co_discard

2013-10-02 Thread Eric Blake
On 09/24/2013 07:35 AM, Peter Lieven wrote: > Signed-off-by: Peter Lieven > --- > block.c | 37 - > 1 file changed, 36 insertions(+), 1 deletion(-) > Reviewed-by: Eric Blake -- Eric Blake eblake redhat com+1-919-301-3266 Libvirt virtualization libr

Re: [Qemu-devel] [PATCHv3 10/20] iscsi: set limits in BlockDriverState

2013-10-02 Thread Eric Blake
On 09/24/2013 07:35 AM, Peter Lieven wrote: > Signed-off-by: Peter Lieven > --- > block/iscsi.c | 14 ++ > 1 file changed, 14 insertions(+) Reviewed-by: Eric Blake -- Eric Blake eblake redhat com+1-919-301-3266 Libvirt virtualization library http://libvirt.org signature

Re: [Qemu-devel] [PATCHv3 11/20] iscsi: add bdrv_has_discard_zeroes and bdrv_has_discard_write_zeroes

2013-10-02 Thread Eric Blake
On 09/24/2013 07:35 AM, Peter Lieven wrote: > Signed-off-by: Peter Lieven > --- > block/iscsi.c | 16 +++- > 1 file changed, 15 insertions(+), 1 deletion(-) > Reviewed-by: Eric Blake -- Eric Blake eblake redhat com+1-919-301-3266 Libvirt virtualization library http://libv

Re: [Qemu-devel] [PATCHv3 13/20] block: introduce bdrv_zeroize

2013-10-02 Thread Eric Blake
On 09/24/2013 07:35 AM, Peter Lieven wrote: > this patch adds a call to completely zero out a block device. > the operation is sped up by checking the block status and > only writing zeroes to the device if they currently do not > return zeroes. optionally the zero writing can be sped up > by setti

Re: [Qemu-devel] [PATCHv3 18/20] qemu-img: add support for fully allocated images

2013-10-02 Thread Eric Blake
On 09/24/2013 07:35 AM, Peter Lieven wrote: > Signed-off-by: Peter Lieven > --- > qemu-img.c |8 +--- > 1 file changed, 5 insertions(+), 3 deletions(-) Missing a counterpart change to qemu-img.texi. > > diff --git a/qemu-img.c b/qemu-img.c > index 926f0a0..c6eff15 100644 > --- a/qemu-i

Re: [Qemu-devel] [PATCH] block/iscsi: reenable iscsi_co_get_block_status

2013-10-02 Thread Stefan Weil
Am 02.10.2013 13:52, schrieb Peter Lieven: > Commit f35c934a accidently disabled iscsi_co_get_block_status for all > libiscsi versions. Its not possible to check for enumeration constants > in the C preprocessor. This patch changes the check to the preprocessor > constant LIBISCSI_FEATURE_IOVECTOR

[Qemu-devel] [PATCH] Update MAINTAINERS

2013-10-02 Thread Anthony Liguori
All of Paul's emails are bouncing and he hasn't been active for some time. Signed-off-by: Anthony Liguori --- MAINTAINERS | 21 + 1 file changed, 5 insertions(+), 16 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 5c3c70c..ab8166a 100644 --- a/MAINTAINERS +++ b/MAI

Re: [Qemu-devel] [PATCHv3 20/20] block/raw: copy BlockLimits on raw_open

2013-10-02 Thread Eric Blake
On 09/24/2013 07:35 AM, Peter Lieven wrote: > Signed-off-by: Peter Lieven > --- > block/raw_bsd.c |1 + > 1 file changed, 1 insertion(+) > > diff --git a/block/raw_bsd.c b/block/raw_bsd.c > index 8dc7bba..7af26ad 100644 > --- a/block/raw_bsd.c > +++ b/block/raw_bsd.c > @@ -159,6 +159,7 @@ st

Re: [Qemu-devel] ioh3420: Add a map_irq function

2013-10-02 Thread Alex Williamson
On Fri, 2013-09-27 at 15:10 -0700, Alexander Duyck wrote: > On 02/28/2013 10:49 AM, Alex Williamson wrote: > > Every bridge needs to know how to map IRQs from it's secondary bus to > > the primary bus. We seem to be direct mapped on ioh3420. This avoids > > segfaults when trying to put assigned d

[Qemu-devel] [Bug 1233225] Re: mips/mipsel linux user float division problem

2013-10-02 Thread Johannes Schauer
For system emulation I used the default which is malta. cheers, josch -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1233225 Title: mips/mipsel linux user float division problem Status in QEMU:

Re: [Qemu-devel] ioh3420: Add a map_irq function

2013-10-02 Thread Alexander Duyck
On 10/02/2013 10:12 AM, Alex Williamson wrote: > On Fri, 2013-09-27 at 15:10 -0700, Alexander Duyck wrote: >> On 02/28/2013 10:49 AM, Alex Williamson wrote: >>> Every bridge needs to know how to map IRQs from it's secondary bus to >>> the primary bus. We seem to be direct mapped on ioh3420. This

  1   2   >