Re: [Qemu-devel] [PATCH 3/3] scsi-disk: Implement 'REPORT TARGET PORT GROUPS'

2015-11-24 Thread Stefan Hajnoczi
On Mon, Nov 16, 2015 at 03:36:58PM +0100, Hannes Reinecke wrote: > +/* Enable TGPS bit */ > +if (s->wwn) > +outbuf[4] = 1; QEMU coding style: Please always use curly braces, even if the if statement body is just one line. > +static void qdev_enumerate_port_group(PortGroupEnumerate

[Qemu-devel] [PATCH v2 14/14] iotests: Add "qemu-img map" test for VMDK extents

2015-11-24 Thread Fam Zheng
Signed-off-by: Fam Zheng --- tests/qemu-iotests/059 | 10 ++ tests/qemu-iotests/059.out | 38 ++ 2 files changed, 48 insertions(+) diff --git a/tests/qemu-iotests/059 b/tests/qemu-iotests/059 index 0ded0c3..261d8b0 100755 --- a/tests/qemu-iotests/0

Re: [Qemu-devel] [RFC v1] virtio-crypto specification

2015-11-24 Thread Gonglei (Arei)
Hello Varun, > -Original Message- > From: Varun Sethi [mailto:varun.se...@freescale.com] > Sent: Wednesday, November 25, 2015 3:08 AM > Subject: RE: [RFC v1] virtio-crypto specification > > Hi Gonglei, > We have been working on the virtio-ipsec look-aside accelerator device > specificat

[Qemu-devel] [PATCH v2 08/14] sheepdog: Assign bs to file in sd_co_get_block_status

2015-11-24 Thread Fam Zheng
Signed-off-by: Fam Zheng --- block/sheepdog.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/block/sheepdog.c b/block/sheepdog.c index 0f6789e..d5e7ff8 100644 --- a/block/sheepdog.c +++ b/block/sheepdog.c @@ -2740,6 +2740,9 @@ sd_co_get_block_status(BlockDriverState *bs, int64_t sector_n

[Qemu-devel] [PATCH v2 13/14] qemu-img: Use QAPI visitor to generate JSON

2015-11-24 Thread Fam Zheng
A visible improvement is that "filename" is now included in the output if it's valid. Signed-off-by: Fam Zheng --- qemu-img.c | 39 --- tests/qemu-iotests/122.out | 96 ++ 2 files changed, 79 insertions(+), 56 deletions(

[Qemu-devel] [PATCH v2 07/14] qed: Assign bs->file->bs to file in bdrv_qed_co_get_block_status

2015-11-24 Thread Fam Zheng
Signed-off-by: Fam Zheng --- block/qed.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/block/qed.c b/block/qed.c index a6bbd8b..03af9c1 100644 --- a/block/qed.c +++ b/block/qed.c @@ -692,6 +692,7 @@ typedef struct { uint64_t pos; int64_t status; int *pnum; +BlockDriver

[Qemu-devel] [PATCH v2 10/14] vpc: Assign bs->file->bs to file in vpc_co_get_block_status

2015-11-24 Thread Fam Zheng
Signed-off-by: Fam Zheng --- block/vpc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/block/vpc.c b/block/vpc.c index 912f5d0..412ff41 100644 --- a/block/vpc.c +++ b/block/vpc.c @@ -588,6 +588,7 @@ static int64_t coroutine_fn vpc_co_get_block_status(BlockDriverState *bs, if (be32

Re: [Qemu-devel] QEMU versus Facebook's Infer static analysis tool

2015-11-24 Thread Stefan Hajnoczi
On Sat, Nov 14, 2015 at 09:53:44PM +, Peter Maydell wrote: > So I tried out Facebook's Infer static analysis tool (http://fbinfer.com/) > on QEMU this evening, just to see whether it would be able to handle our > codebase and if it would report anything interesting. > > The good news is it was

[Qemu-devel] [PATCH v2 11/14] vmdk: Return extent's file in bdrv_get_block_status

2015-11-24 Thread Fam Zheng
Signed-off-by: Fam Zheng --- block/vmdk.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/block/vmdk.c b/block/vmdk.c index f5a56fd..b60a5af 100644 --- a/block/vmdk.c +++ b/block/vmdk.c @@ -1265,6 +1265,7 @@ static int64_t coroutine_fn vmdk_co_get_block_status(Blo

[Qemu-devel] [PATCH v2 09/14] vdi: Assign bs->file->bs to file in vdi_co_get_block_status

2015-11-24 Thread Fam Zheng
Signed-off-by: Fam Zheng --- block/vdi.c | 1 + 1 file changed, 1 insertion(+) diff --git a/block/vdi.c b/block/vdi.c index 2199fd3..6b1a57b 100644 --- a/block/vdi.c +++ b/block/vdi.c @@ -550,6 +550,7 @@ static int64_t coroutine_fn vdi_co_get_block_status(BlockDriverState *bs, offset = s->

[Qemu-devel] [PATCH v2 05/14] iscsi: Assign bs to file in iscsi_co_get_block_status

2015-11-24 Thread Fam Zheng
Reviewed-by: Eric Blake Signed-off-by: Fam Zheng --- block/iscsi.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/block/iscsi.c b/block/iscsi.c index 2d1e230..8c7f1b3 100644 --- a/block/iscsi.c +++ b/block/iscsi.c @@ -625,6 +625,9 @@ out: if (iTask.task != NULL) { scsi_free

[Qemu-devel] [PATCH v2 04/14] raw: Assign bs to file in raw_co_get_block_status

2015-11-24 Thread Fam Zheng
Reviewed-by: Eric Blake Signed-off-by: Fam Zheng --- block/raw-posix.c | 1 + block/raw_bsd.c | 1 + 2 files changed, 2 insertions(+) diff --git a/block/raw-posix.c b/block/raw-posix.c index 2cd7d68..9988aa4 100644 --- a/block/raw-posix.c +++ b/block/raw-posix.c @@ -1873,6 +1873,7 @@ static i

[Qemu-devel] [PATCH v2 12/14] qemu-img: Make MapEntry a QAPI struct

2015-11-24 Thread Fam Zheng
The "flags" bit mask is expanded to two booleans, "data" and "zero"; "bs" is replaced with "filename" string. Signed-off-by: Fam Zheng --- qapi/block-core.json | 28 qemu-img.c | 48 ++-- 2 files changed, 50 inser

[Qemu-devel] [PATCH v2 06/14] parallels: Assign bs->file->bs to file in parallels_co_get_block_status

2015-11-24 Thread Fam Zheng
Signed-off-by: Fam Zheng --- block/parallels.c | 1 + 1 file changed, 1 insertion(+) diff --git a/block/parallels.c b/block/parallels.c index d1146f1..6552f32 100644 --- a/block/parallels.c +++ b/block/parallels.c @@ -273,6 +273,7 @@ static int64_t coroutine_fn parallels_co_get_block_status(Blo

[Qemu-devel] [PATCH v2 01/14] block: Add "file" output parameter to block status query functions

2015-11-24 Thread Fam Zheng
The added parameter can be used to return the BDS pointer which the valid offset is referring to. It's value should be ignored unless BDRV_BLOCK_OFFSET_VALID in ret is set. Until block drivers fill in the right value, let's clear it explicitly right before calling .bdrv_get_block_status. Signed-o

[Qemu-devel] [PATCH v2 02/14] qcow: Assign bs->file->bs to file in qcow_co_get_block_status

2015-11-24 Thread Fam Zheng
Reviewed-by: Eric Blake Signed-off-by: Fam Zheng --- block/qcow.c | 1 + 1 file changed, 1 insertion(+) diff --git a/block/qcow.c b/block/qcow.c index 558f443..b59383f 100644 --- a/block/qcow.c +++ b/block/qcow.c @@ -509,6 +509,7 @@ static int64_t coroutine_fn qcow_co_get_block_status(BlockDri

[Qemu-devel] [PATCH v2 00/14] qemu-img map: Allow driver to return file of the allocated block

2015-11-24 Thread Fam Zheng
v2: Add Eric's rev-by in patches 2, 4, 5. 01: Refering -> referring in commit message. [Eric] Recurse to "file" for sensible "zero" flag. [Paolo] 12: New. Make MapEntry a QAPI struct. [Paolo, Markus] I stumbled upon this when looking at external bitmap formats. Current "qemu-img m

[Qemu-devel] [PATCH v2 03/14] qcow2: Assign bs->file->bs to file in qcow2_co_get_block_status

2015-11-24 Thread Fam Zheng
Signed-off-by: Fam Zheng --- block/qcow2.c | 1 + 1 file changed, 1 insertion(+) diff --git a/block/qcow2.c b/block/qcow2.c index 836888c..7634c42 100644 --- a/block/qcow2.c +++ b/block/qcow2.c @@ -1302,6 +1302,7 @@ static int64_t coroutine_fn qcow2_co_get_block_status(BlockDriverState *bs,

Re: [Qemu-devel] [PATCH 2/2] trace: show MemoryRegion name, not address

2015-11-24 Thread Stefan Hajnoczi
On Tue, Nov 17, 2015 at 04:37:48PM -0800, Hollis Blanchard wrote: > On 11/13/2015 02:23 AM, Stefan Hajnoczi wrote: > >On Wed, Nov 11, 2015 at 05:09:58PM -0800, Hollis Blanchard wrote: > >>Recording the MemoryRegion pointers isn't helpful, especially since no trace > >>data allows us to correlate th

[Qemu-devel] [PATCH for-2.5] vnc: fix segfault

2015-11-24 Thread Gerd Hoffmann
Commit "c7628bf vnc: only alloc server surface with clients connected" missed one rarely used codepath (cirrus with guest drivers using 2d accel) where we have to check for the server surface being present, to avoid qemu crashing with a NULL pointer dereference. Add the check. Reported-by: Anthon

[Qemu-devel] [PATCH] target-mips/cpu.h: Fix spell error

2015-11-24 Thread Dongxue Zhang
CP0IntCtl_IPPC1, the last letter should be 'i', not 'one'. Signed-off-by: Dongxue Zhang --- target-mips/cpu.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target-mips/cpu.h b/target-mips/cpu.h index 89c01f7..17817c3 100644 --- a/target-mips/cpu.h +++ b/target-mips/cpu.h @@

Re: [Qemu-devel] [PATCH] vfio: Align iova also to IOMMU page size

2015-11-24 Thread Pavel Fedin
Hello! > There are a number of different interesting page sizes here: > * the host kernel page size > * the target CPU architecture's worst-case smallest page size > * the page size the guest kernel is actually using at the moment >(consider a 4K-page guest kernel on a 64K-page host kernel

Re: [Qemu-devel] [PATCH v3 0/4] Bitmap clean-up patches for 2.6

2015-11-24 Thread Vladimir Sementsov-Ogievskiy
Hmm, stop. Very bad thing (sorry, that I didn't realize it before): This breaks my dirty bitmap migration series with its meta bitmaps. Meta bitmap is an additional HBitmap in BdrvDirtyBitmap, which tracks dirtiness of this BdrvDirtyBitmap. And it (meta bitmap) have its own granularity of cour

Re: [Qemu-devel] [PATCH COLO-Frame v11 08/39] migration: Rename the'file' member of MigrationState

2015-11-24 Thread zhanghailiang
On 2015/11/25 2:26, Dr. David Alan Gilbert wrote: * zhanghailiang (zhang.zhanghaili...@huawei.com) wrote: Rename the 'file' member of MigrationState to 'to_dst_file'. Signed-off-by: zhanghailiang Cc: Dr. David Alan Gilbert --- v11: - Only rename 'file' member of MigrationState Reviewed-by:

Re: [Qemu-devel] [PATCH COLO-Frame v11 07/39] migration: Integrate COLO checkpoint process into loadvm

2015-11-24 Thread zhanghailiang
On 2015/11/25 2:14, Dr. David Alan Gilbert wrote: * zhanghailiang (zhang.zhanghaili...@huawei.com) wrote: Switch from normal migration loadvm process into COLO checkpoint process if COLO mode is enabled. We add three new members to struct MigrationIncomingState, 'have_colo_incoming_thread' and

Re: [Qemu-devel] [PATCH v1 1/1] xlnx-ep108: Add support for high DDR memory regions

2015-11-24 Thread Peter Crosthwaite
On Mon, Nov 23, 2015 at 9:00 PM, Alistair Francis wrote: > The Xilinx EP108 supports three memory regions: > - A 2GB region starting at 0 > - A 32GB region starting at 32GB > - A 256GB region starting at 768GB > > This patch adds support for the middle memory region, which is > automatically cr

Re: [Qemu-devel] [Qemu-ppc] [PATCH v3 2/2] target-ppc: Implement rtas_get_sysparm(PROCESSOR_MODULE_INFO)

2015-11-24 Thread Sukadev Bhattiprolu
David, Alexey, Any comments on this patch? Suka Sukadev Bhattiprolu [suka...@linux.vnet.ibm.com] wrote: | Implement RTAS_SYSPARM_PROCESSOR_MODULE_INFO parameter to rtas_get_sysparm() | call in qemu. This call returns the processor module (socket), chip and core | information as specified in sect

Re: [Qemu-devel] [PATCH 1/4] vmxnet3: The vmxnet3 device is a PCIE endpoint

2015-11-24 Thread Shmulik Ladkani
Thanks Jason, On Wed, 25 Nov 2015 10:38:51 +0800, jasow...@redhat.com wrote: > > @@ -2568,6 +2572,7 @@ static void vmxnet3_class_init(ObjectClass *class, > > void *data) > > c->class_id = PCI_CLASS_NETWORK_ETHERNET; > > c->subsystem_vendor_id = PCI_VENDOR_ID_VMWARE; > > c->subsyste

Re: [Qemu-devel] [PATCH v2 0/2] net/tap-win32 bugfixes

2015-11-24 Thread Stefan Weil
Am 25.11.2015 um 04:01 schrieb Jason Wang: > > > On 11/25/2015 07:36 AM, Andrew Baumann wrote: >> Ping? >> >> Jason, I know you planned to leave this for a few days... I just wanted to >> make sure it isn't forgotten :) >> >> Thanks, >> Andrew > > Thanks for working on this. I would expect Stef

Re: [Qemu-devel] RFC: raspberry pi / pi2 / Windows-on-ARM support

2015-11-24 Thread Peter Crosthwaite
On Tue, Nov 24, 2015 at 4:00 PM, Andrew Baumann wrote: > Hi Peter (et al), > > I am working on refactoring the Pi support code as you suggested. I have > split the Pi SOCs into separate objects (bcm2835 and bcm2836) which both > instantiate a third common bcm2835_peripherals device that in turn

Re: [Qemu-devel] [RFC PATCH V2 3/3] Ixgbevf: Add migration support for ixgbevf driver

2015-11-24 Thread Lan Tianyu
On 2015年11月25日 05:20, Michael S. Tsirkin wrote: > I have to say, I was much more interested in the idea > of tracking dirty memory. I have some thoughts about > that one - did you give up on it then? No, our finial target is to keep VF active before doing migration and tracking dirty memory is ess

Re: [Qemu-devel] [PATCH v3 3/3] i.MX: Add an i.MX25 specific CCM class/instance.

2015-11-24 Thread Peter Crosthwaite
On Thu, Nov 19, 2015 at 12:40 PM, Jean-Christophe Dubois wrote: > Signed-off-by: Jean-Christophe Dubois > --- > > Changes since v1: > * rework loging to match other i.MX drivers > > Changes since v2: > * We moved to an inheritance QOM scheme > > hw/arm/fsl-imx25.c | 2 +- > hw/misc/

[Qemu-devel] [PATCH] block/raw-posix.c: Make physical devices usable in QEMU under Mac OS X host

2015-11-24 Thread Programmingkid
Mac OS X can be picky when it comes to allowing the user to use physical devices in QEMU. Most mounted volumes appear to be off limits to QEMU. If an issue is detected, a message is displayed showing the user how to unmount a volume. Signed-off-by: John Arbuckle --- Changed cdromOK variable to c

Re: [Qemu-devel] [RFC PATCH V2 3/3] Ixgbevf: Add migration support for ixgbevf driver

2015-11-24 Thread Alexander Duyck
On Tue, Nov 24, 2015 at 1:20 PM, Michael S. Tsirkin wrote: > On Tue, Nov 24, 2015 at 09:38:18PM +0800, Lan Tianyu wrote: >> This patch is to add migration support for ixgbevf driver. Using >> faked PCI migration capability table communicates with Qemu to >> share migration status and mailbox irq v

Re: [Qemu-devel] [RFC PATCH V2 0/3] IXGBE/VFIO: Add live migration support for SRIOV NIC

2015-11-24 Thread Alexander Duyck
On Tue, Nov 24, 2015 at 7:18 PM, Lan Tianyu wrote: > On 2015年11月24日 22:20, Alexander Duyck wrote: >> I'm still not a fan of this approach. I really feel like this is >> something that should be resolved by extending the existing PCI hot-plug >> rather than trying to instrument this per driver. T

Re: [Qemu-devel] [PATCH REPOST 0/2] Add basic "detach" support for dump-guest-memory

2015-11-24 Thread Peter Xu
On 11/24/2015 08:05 PM, Paolo Bonzini wrote: > > > On 24/11/2015 04:10, Fam Zheng wrote: >> What about all the hot-plug commands that changes the memory layout? > > If the guest is stopped, they shouldn't. device_add does not enable new > BARs for example, the guest does that after it receive

Re: [Qemu-devel] [PATCH REPOST 0/2] Add basic "detach" support for dump-guest-memory

2015-11-24 Thread Fam Zheng
On Wed, 11/25 12:48, Peter Xu wrote: > > > On 11/25/2015 10:46 AM, Fam Zheng wrote: > > On Tue, 11/24 06:49, Eric Blake wrote: > >> On 11/24/2015 04:37 AM, Fam Zheng wrote: > >> > I think the patch should be dropped, and periodic progress reports > should be emitted from within the dump

Re: [Qemu-devel] [PATCH REPOST 0/2] Add basic "detach" support for dump-guest-memory

2015-11-24 Thread Peter Xu
On 11/25/2015 10:46 AM, Fam Zheng wrote: > On Tue, 11/24 06:49, Eric Blake wrote: >> On 11/24/2015 04:37 AM, Fam Zheng wrote: >> I think the patch should be dropped, and periodic progress reports should be emitted from within the dump loops that do the heavy lifting. For the E

[Qemu-devel] [PATCH v3 2/4] block: Hide HBitmap in block dirty bitmap interface

2015-11-24 Thread Fam Zheng
HBitmap is an implementation detail of block dirty bitmap that should be hidden from users. Introduce a BdrvDirtyBitmapIter to encapsulate the underlying HBitmapIter. A small difference in the interface is, before, an HBitmapIter is initialized in place, now the new BdrvDirtyBitmapIter must be dyn

[Qemu-devel] [PATCH v3 1/4] backup: Use Bitmap to replace "s->bitmap"

2015-11-24 Thread Fam Zheng
"s->bitmap" tracks done sectors, we only check bit states without using any iterator which HBitmap is good for. Switch to "Bitmap" which is simpler and more memory efficient. Meanwhile, rename it to done_bitmap, to reflect the intention. Signed-off-by: Fam Zheng Reviewed-by: John Snow --- bloc

[Qemu-devel] [PATCH v3 4/4] hbitmap: Drop "granularity"

2015-11-24 Thread Fam Zheng
Now all callers except the test code use zero, drop the parameter to make the interface cleaner and more intuitive. Test code of hbitmap granularity is removed together. Signed-off-by: Fam Zheng --- block.c| 4 +- include/qemu/hbitmap.h | 20 + tests/test-hbitmap.c | 20

[Qemu-devel] [PATCH v3 3/4] block: Manage granularity in BdrvDirtyBitmap

2015-11-24 Thread Fam Zheng
BdrvDirtyBitmap has to know about granularity because it's in the API, but currently the logic is in HBitmap. Because HBitmap has a delicate implementation that has a multi-layer array of arrays of different granularities, handling the "bit granularity" there only makes it harder to understand. No

[Qemu-devel] [PATCH v3 0/4] Bitmap clean-up patches for 2.6

2015-11-24 Thread Fam Zheng
v3: Split patch 2. [Vladimir] v2: bitmap_set -> set_bit. [Congyang, Paolo, John] Add John Snow's rev-by with that change. This makes a cleaner base for more dirty bitmap work. "granularity" appearing with different representations have always been mind twisting, remove it from HBitmap to make

Re: [Qemu-devel] [PATCH v6] raw-posix.c: Make physical devices usable in QEMU under Mac OS X host

2015-11-24 Thread Programmingkid
On Nov 24, 2015, at 9:38 AM, Kevin Wolf wrote: > > >> +/* If using a physical device */ >> +if (strstart(filename, "/dev/", NULL)) { >> +char bsdPath[MAXPATHLEN]; >> + >> +/* If the physical device is a cdrom */ >> +if (strcmp(filename, "/dev/cdrom") == 0) { > >

Re: [Qemu-devel] [RFC PATCH V2 0/3] IXGBE/VFIO: Add live migration support for SRIOV NIC

2015-11-24 Thread Lan Tianyu
On 2015年11月24日 22:20, Alexander Duyck wrote: > I'm still not a fan of this approach. I really feel like this is > something that should be resolved by extending the existing PCI hot-plug > rather than trying to instrument this per driver. Then you will get the > goodness for multiple drivers and

Re: [Qemu-devel] [PATCH for-2.5] eepro100: Prevent two endless loops

2015-11-24 Thread Jason Wang
On 11/20/2015 07:27 PM, P J P wrote: > +-- On Fri, 20 Nov 2015, Stefan Weil wrote --+ > | include/hw/pci/pci.h:static inline uint##_bits##_t > | ld##_l##_pci_dma(PCIDevice *dev, \ > >I see. > > | Is there an ideal count? If it is too low, it might break some use cases. > | If it is

Re: [Qemu-devel] [PATCH v2 0/2] net/tap-win32 bugfixes

2015-11-24 Thread Jason Wang
On 11/25/2015 07:36 AM, Andrew Baumann wrote: > Ping? > > Jason, I know you planned to leave this for a few days... I just wanted to > make sure it isn't forgotten :) > > Thanks, > Andrew Thanks for working on this. I would expect Stefan to review v2 (since he reviewed v1). Stefan, could you p

Re: [Qemu-devel] [PATCH for 2.6 2/3] block: Hide HBitmap in block dirty bitmap interface

2015-11-24 Thread Fam Zheng
On Tue, 11/24 12:12, Vladimir Sementsov-Ogievskiy wrote: > On 24.11.2015 05:28, Fam Zheng wrote: > >On Mon, 11/23 16:34, John Snow wrote: > >>Hmm, what's the idea, here? > >> > >>This patch does a lot more than just hide hbitmap details from callers > >>of block_dirty_bitmap functions. > >> > >>So

Re: [Qemu-devel] [PATCH REPOST 0/2] Add basic "detach" support for dump-guest-memory

2015-11-24 Thread Fam Zheng
On Tue, 11/24 06:49, Eric Blake wrote: > On 11/24/2015 04:37 AM, Fam Zheng wrote: > > >> I think the patch should be dropped, and periodic progress reports > >> should be emitted from within the dump loops that do the heavy lifting. > >> > >> For the ELF format dumps, that loop appears to reside i

Re: [Qemu-devel] [PATCH 3/4] vmxnet3: Change the offset of the MSIX PBA table

2015-11-24 Thread Jason Wang
On 11/19/2015 06:52 PM, Shmulik Ladkani wrote: > Place the PBA table at 0x1000, as placed by VMware virtual hardware. > > Signed-off-by: Shmulik Ladkani > --- > hw/net/vmxnet3.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/net/vmxnet3.c b/hw/net/vmxnet3.c > index

Re: [Qemu-devel] [PATCH 2/4] vmxnet3: Change offsets of PCI capabilities

2015-11-24 Thread Jason Wang
On 11/19/2015 06:52 PM, Shmulik Ladkani wrote: > Place device reported PCI capabilities at the same offsets as placed by > the VMware virtual hardware: > Express Endpoint at [48], MSI at [84], MSI-X at [9c]. > > Signed-off-by: Shmulik Ladkani > --- > hw/net/vmxnet3.c | 9 ++--- > 1 file c

Re: [Qemu-devel] [PATCH v5 4/4] devicetree: update documentation for fw_cfg ARM bindings

2015-11-24 Thread Rob Herring
On Mon, Nov 23, 2015 at 10:57:44AM -0500, Gabriel L. Somlo wrote: > From: Gabriel Somlo > > Remove fw_cfg hardware interface details from > Documentation/devicetree/bindings/arm/fw-cfg.txt, > and replace them with a pointer to the authoritative > documentation in the QEMU source tree. > > Signed

Re: [Qemu-devel] [PATCH 1/4] vmxnet3: The vmxnet3 device is a PCIE endpoint

2015-11-24 Thread Jason Wang
On 11/19/2015 06:52 PM, Shmulik Ladkani wrote: > Report the 'express endpoint' capability if on a PCIE bus. > > Signed-off-by: Shmulik Ladkani > --- > hw/net/vmxnet3.c | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/hw/net/vmxnet3.c b/hw/net/vmxnet3.c > index 5e3a233..ed286cc 1006

Re: [Qemu-devel] [Qemu-trivial] [PATCH] PCI Trivial: remove superfluous code

2015-11-24 Thread Cao jin
On 11/25/2015 02:30 AM, Marcel Apfelbaum wrote: On 11/23/2015 03:19 PM, Cao jin wrote: remove superfluous code in do_pci_register_device(). See its caller: pci_qdev_realize() Signed-off-by: Cao jin --- hw/pci/pci.c | 1 - 1 file changed, 1 deletion(-) diff --git a/hw/pci/pci.c b/hw/pci/p

Re: [Qemu-devel] make check failed: rcutorture error

2015-11-24 Thread Yijun Zhu
On 2015/11/24 19:54, Paolo Bonzini wrote: > > > On 24/11/2015 12:45, Yijun Zhu wrote: >> Hi all, >> >>I got an error when doing "make check" on qemu code both stable-2.4 and >> master branch, after updating binutils to 2.24-33. >> >>Qemu compiled successfully with following configure co

Re: [Qemu-devel] [PATCH for-2.5 1/1] vhost-user: do not send SET_VRING_ENABLE at start

2015-11-24 Thread Yuanhan Liu
On Tue, Nov 24, 2015 at 11:23:34PM +0200, Michael S. Tsirkin wrote: > On Tue, Nov 24, 2015 at 10:05:27PM +0100, Thibaut Collet wrote: > > On Tue, Nov 24, 2015 at 9:52 PM, Michael S. Tsirkin wrote: > > > On Tue, Nov 24, 2015 at 05:10:36PM +0100, Thibaut Collet wrote: > > >> This patch reverts parti

[Qemu-devel] [PATCH v2 6/6] target-arm: dump-guest-memory: add fpregset notes

2015-11-24 Thread Andrew Jones
Also refactors note init code to avoid code duplication. Signed-off-by: Andrew Jones --- target-arm/arch_dump.c | 161 ++--- 1 file changed, 139 insertions(+), 22 deletions(-) diff --git a/target-arm/arch_dump.c b/target-arm/arch_dump.c index 5debe549

[Qemu-devel] [PATCH v2 5/6] target-arm: support QMP dump-guest-memory

2015-11-24 Thread Andrew Jones
Add the support needed for creating prstatus elf notes. This allows us to use QMP dump-guest-memory. Signed-off-by: Andrew Jones --- target-arm/Makefile.objs | 3 +- target-arm/arch_dump.c | 230 +++ target-arm/cpu-qom.h | 5 ++ target-arm/cp

[Qemu-devel] [PATCH v2 3/6] dump: allow target to set the page size

2015-11-24 Thread Andrew Jones
This is necessary for targets that don't have TARGET_PAGE_SIZE == real-target-page-size. The target should set the page size to the correct one, if known, or, if not known, to the maximum page size it supports. (No functional change.) Signed-off-by: Andrew Jones --- dump.c |

[Qemu-devel] [PATCH v2 2/6] dump: qemunotes aren't commonly needed

2015-11-24 Thread Andrew Jones
Only one of three architectures implementing qmp-dump-guest-memory write qemu notes. And, another architecture (arm/aarch64) is coming, which won't use them either. Make the common implementation truly common. (No functional change.) Signed-off-by: Andrew Jones Reviewed-by: Peter Maydell --- q

[Qemu-devel] [PATCH v2 4/6] dump: allow target to set the physical base

2015-11-24 Thread Andrew Jones
crash assumes the physical base in the kdump subheader of makedumpfile formatted dumps is correct. Zero is not correct for all architectures, so allow it to be changed. (No functional change.) Signed-off-by: Andrew Jones --- dump.c | 4 ++-- include/sysemu/dump-arch.h | 1 +

[Qemu-devel] [PATCH v2 0/6] target-arm: enable qmp-dump-guest-memory

2015-11-24 Thread Andrew Jones
This series brings qmp-dump-guest-memory to arm and aarch64 targets. I've detailed my testing and the results in the following table. v2: changes thanks to Peter's review questions - Threw in the FP registers. Can view them with gdb on elf formatted dumps. - Added comments stating where the r

[Qemu-devel] [PATCH v2 1/6] qapi-schema: dump-guest-memory: Improve text

2015-11-24 Thread Andrew Jones
dump-guest-memory is supported by more than just x86, however the paging option is not. (No functional change.) Signed-off-by: Andrew Jones Reviewed-by: Eric Blake --- qapi-schema.json | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/qapi-schema.json b/qapi-schema.json i

Re: [Qemu-devel] RFC: raspberry pi / pi2 / Windows-on-ARM support

2015-11-24 Thread Andrew Baumann
Hi Peter (et al), I am working on refactoring the Pi support code as you suggested. I have split the Pi SOCs into separate objects (bcm2835 and bcm2836) which both instantiate a third common bcm2835_peripherals device that in turn contains all the common devices. I have also switched the code t

Re: [Qemu-devel] [PATCH v4 0/4] usb-mtp events support

2015-11-24 Thread Bandan Das
Gerd Hoffmann writes: > On Mo, 2015-11-23 at 16:37 -0500, Bandan Das wrote: >> This series adds support for mtp events that are piggybacked on >> top of the Linux provided inotify mechanism. It performs well with some >> light unit testing in a linux guest. The mtp share is still read only, >> bu

Re: [Qemu-devel] [PATCH v2 0/2] net/tap-win32 bugfixes

2015-11-24 Thread Andrew Baumann
Ping? Jason, I know you planned to leave this for a few days... I just wanted to make sure it isn't forgotten :) Thanks, Andrew > -Original Message- > From: Andrew Baumann > Sent: Wednesday, 18 November 2015 11:45 > To: qemu-devel@nongnu.org > Cc: Jason Wang ; Stefan Weil ; > Andrew Bau

Re: [Qemu-devel] KVM/Qemu migration under fast network stress

2015-11-24 Thread Andrey Korolyov
On Wed, Nov 25, 2015 at 1:44 AM, Or Gerlitz wrote: > Hi, > > When doing live migration with iperf running on the migrated VM > over PV/virtio interface -- it doesn't end when the number of threads > 1 :( > nor when we run an IO (fio) benchmark which gets high-throughput. None > of these tests run

Re: [Qemu-devel] [PATCH v3 2/3] i.MX: Split the CCM class into an abstact base class and a concrete class.

2015-11-24 Thread Peter Crosthwaite
On Tue, Nov 24, 2015 at 3:17 PM, Jean-Christophe DUBOIS wrote: > Le 24/11/2015 23:04, Peter Crosthwaite a écrit : >> >> On Thu, Nov 19, 2015 at 12:40 PM, Jean-Christophe Dubois >> wrote: >>> >>> The IMX_CCM class is now the base abstract class that is used by EPIT and >>> GPT >>> timer implementa

[Qemu-devel] [trivial for-2.6] util/id: fully allocate names table

2015-11-24 Thread John Snow
Trivial: this array should be allocated to have ID_MAX entries always. Otherwise if someone were to forget to expand this table, the assertion in the id generator won't actually trigger; it will read junk data. Signed-off-by: John Snow --- util/id.c | 2 +- 1 file changed, 1 insertion(+), 1 dele

Re: [Qemu-devel] [PATCH v3 2/3] i.MX: Split the CCM class into an abstact base class and a concrete class.

2015-11-24 Thread Jean-Christophe DUBOIS
Le 24/11/2015 23:04, Peter Crosthwaite a écrit : On Thu, Nov 19, 2015 at 12:40 PM, Jean-Christophe Dubois wrote: The IMX_CCM class is now the base abstract class that is used by EPIT and GPT timer implementation. IMX31_CCM class is the concrete class implementing CCM for i.MX31 SOC. For now t

[Qemu-devel] KVM/Qemu migration under fast network stress

2015-11-24 Thread Or Gerlitz
Hi, When doing live migration with iperf running on the migrated VM over PV/virtio interface -- it doesn't end when the number of threads > 1 :( nor when we run an IO (fio) benchmark which gets high-throughput. None of these tests run over the network the migration is carried out. We get this "er

[Qemu-devel] [Bug 1519037] Re: qemu-i386 32-bit segfault

2015-11-24 Thread PeteVine
It seems this crash only happens in xterm (and not normal console). Having compared the respective environment vars the culprit turned out to be: TERM=xterm-color You're welcome. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. http

Re: [Qemu-devel] [PATCH v2 1/2] qga: flush explicitly when needed

2015-11-24 Thread Eric Blake
On 11/24/2015 03:19 PM, Michael Roth wrote: >> Hmm. This always attempts fseek() on the first write() to a file, even >> if the file is not also open for read. While guest-file-open is most >> likely used on regular files (and therefore seekable), I'm worried that >> we might have a client that

Re: [Qemu-devel] [PATCH v2 1/2] qga: flush explicitly when needed

2015-11-24 Thread Michael Roth
Quoting Eric Blake (2015-11-24 14:56:15) > On 11/24/2015 11:04 AM, marcandre.lur...@redhat.com wrote: > > From: Marc-André Lureau > > > > According to the specification: > > http://pubs.opengroup.org/onlinepubs/9699919799/functions/fopen.html > > > > "the application shall ensure that output is

Re: [Qemu-devel] [PATCH v3 2/3] i.MX: Split the CCM class into an abstact base class and a concrete class.

2015-11-24 Thread Peter Crosthwaite
On Thu, Nov 19, 2015 at 12:40 PM, Jean-Christophe Dubois wrote: > The IMX_CCM class is now the base abstract class that is used by EPIT and GPT > timer implementation. > > IMX31_CCM class is the concrete class implementing CCM for i.MX31 SOC. > > For now the i.MX25 continues to use the i.MX31 CCM

Re: [Qemu-devel] [PATCH 5/5] target-arm: support QMP dump-guest-memory

2015-11-24 Thread Peter Maydell
On 24 November 2015 at 21:45, Andrew Jones wrote: > On Tue, Nov 24, 2015 at 09:08:29PM +, Peter Maydell wrote: >> Also, if we're currently executing a 32-bit guest when we take >> the core dump, you probably need to call aarch64_sync_32_to_64() >> somewhere. > > I think we're covered here. We

Re: [Qemu-devel] [PATCH 5/5] target-arm: support QMP dump-guest-memory

2015-11-24 Thread Andrew Jones
On Tue, Nov 24, 2015 at 09:08:29PM +, Peter Maydell wrote: > On 24 November 2015 at 20:52, Andrew Jones wrote: > > > > I've pulled a v2 together that I'll be testing and posting soon. Here's > > what I decided to do > > > > 1) Throw the fp registers in. Why not? > > 2) No linux-headers update,

Re: [Qemu-devel] [PATCH for-2.5 1/1] vhost-user: do not send SET_VRING_ENABLE at start

2015-11-24 Thread Michael S. Tsirkin
On Tue, Nov 24, 2015 at 10:05:27PM +0100, Thibaut Collet wrote: > On Tue, Nov 24, 2015 at 9:52 PM, Michael S. Tsirkin wrote: > > On Tue, Nov 24, 2015 at 05:10:36PM +0100, Thibaut Collet wrote: > >> This patch reverts partially commit 3a12f32229a. > >> > >> In case of live migration several queues

Re: [Qemu-devel] [RFC PATCH V2 3/3] Ixgbevf: Add migration support for ixgbevf driver

2015-11-24 Thread Michael S. Tsirkin
On Tue, Nov 24, 2015 at 09:38:18PM +0800, Lan Tianyu wrote: > This patch is to add migration support for ixgbevf driver. Using > faked PCI migration capability table communicates with Qemu to > share migration status and mailbox irq vector index. > > Qemu will notify VF via sending MSIX msg to tri

Re: [Qemu-devel] [PATCH v2 2/4] qemu-iotests: s390x: fix test 051

2015-11-24 Thread Sascha Silbe
Dear Max, Max Reitz writes: > OK, so it is expected for s390x; however, this is strictly speaking not > the output file for s390x but for any platform but PC. That's why I'd > rather not have it in this “generic” reference output. > > This patch already assumes that the iotests only support s390

Re: [Qemu-devel] [PATCH 5/5] target-arm: support QMP dump-guest-memory

2015-11-24 Thread Peter Maydell
On 24 November 2015 at 20:52, Andrew Jones wrote: > > I've pulled a v2 together that I'll be testing and posting soon. Here's > what I decided to do > > 1) Throw the fp registers in. Why not? > 2) No linux-headers update, as we'd also need >include/uapi/linux/elfcore.h and arch/arm/include/asm

Re: [Qemu-devel] [PATCH for-2.5 1/1] vhost-user: do not send SET_VRING_ENABLE at start

2015-11-24 Thread Thibaut Collet
On Tue, Nov 24, 2015 at 9:52 PM, Michael S. Tsirkin wrote: > On Tue, Nov 24, 2015 at 05:10:36PM +0100, Thibaut Collet wrote: >> This patch reverts partially commit 3a12f32229a. >> >> In case of live migration several queues can be enabled and not only the >> first >> one. So inform backend that o

Re: [Qemu-devel] [RFC PATCH V2 09/10] Qemu/VFIO: Add SRIOV VF migration support

2015-11-24 Thread Michael S. Tsirkin
On Tue, Nov 24, 2015 at 09:35:26PM +0800, Lan Tianyu wrote: > This patch is to add SRIOV VF migration support. > Create new device type "vfio-sriov" and add faked PCI migration capability > to the type device. > > The purpose of the new capability > 1) sync migration status with VF driver in the V

Re: [Qemu-devel] [PATCH v2 1/2] qga: flush explicitly when needed

2015-11-24 Thread Eric Blake
On 11/24/2015 11:04 AM, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > According to the specification: > http://pubs.opengroup.org/onlinepubs/9699919799/functions/fopen.html > > "the application shall ensure that output is not directly followed by > input without an intervening

Re: [Qemu-devel] [PATCH 5/5] target-arm: support QMP dump-guest-memory

2015-11-24 Thread Andrew Jones
On Sat, Nov 21, 2015 at 10:05:37AM -0500, Andrew Jones wrote: > On Fri, Nov 20, 2015 at 04:41:21PM -0500, Andrew Jones wrote: > > On Fri, Nov 20, 2015 at 06:19:14PM +, Peter Maydell wrote: > > > On 19 November 2015 at 14:53, Andrew Jones wrote: > > > > + > > > > +if (is_a64(env)) { > > >

Re: [Qemu-devel] [PATCH for-2.5 1/1] vhost-user: do not send SET_VRING_ENABLE at start

2015-11-24 Thread Michael S. Tsirkin
On Tue, Nov 24, 2015 at 05:10:36PM +0100, Thibaut Collet wrote: > This patch reverts partially commit 3a12f32229a. > > In case of live migration several queues can be enabled and not only the first > one. So inform backend that only the first queue is enabled is wrong. > > Since commit 7263a0ad78

Re: [Qemu-devel] [PATCH for-2.5 v2 0/2] qga: flush explicitly when needed

2015-11-24 Thread Eric Blake
On 11/24/2015 11:04 AM, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > Without this change, a write() followed by a read() may lose the > previously written content, as shown in the following test. > > v1->v2: > - replace guchar with unsigned char > - fix implicitly/explictly >

Re: [Qemu-devel] [PATCH v2 2/2] tests: add file-write-read test

2015-11-24 Thread Eric Blake
On 11/24/2015 11:04 AM, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > This test exhibits a POSIX behaviour regarding switching between write > and read. It's undefined result if the application doesn't ensure a > flush between the two operations (with glibc, the flush can be im

Re: [Qemu-devel] [PATCH for-2.5] qga: Better mapping of SEEK_* in guest-file-seek

2015-11-24 Thread Marc-André Lureau
Hi On Tue, Nov 24, 2015 at 8:09 PM, Eric Blake wrote: > On 11/24/2015 11:57 AM, Eric Blake wrote: >> Exposing OS-specific SEEK_ constants in our qapi was a mistake >> (if the host has SEEK_CUR as 1, but the guest has it as 2, then >> the semantics are unclear what should happen); if we had a time

Re: [Qemu-devel] [PATCH for-2.5 4/5] ivshmem: Mark questionable socket type test FIXME

2015-11-24 Thread Marc-André Lureau
- Original Message - > Marc-André Lureau writes: > > > Hi > > > > - Original Message - > >> Marc-André Lureau writes: > >> > >> > - Original Message - > >> >> Signed-off-by: Markus Armbruster > >> >> --- > >> >> hw/misc/ivshmem.c | 1 + > >> >> 1 file changed, 1 inse

Re: [Qemu-devel] [PATCH v4 03/17] ipmi: Add a local BMC simulation

2015-11-24 Thread Corey Minyard
On 11/24/2015 07:31 AM, Cédric Le Goater wrote: > A few comments below, Thanks a bunch for the review. As you probably have guessed, this was not really intended as a fully functional BMC, though it has most of the trappings of what you would need to implement one. I assume you are working on th

Re: [Qemu-devel] [PATCH v2 0/1] ide-test: tidy up after pio race fix

2015-11-24 Thread John Snow
On 11/24/2015 02:36 PM, John Snow wrote: > Post-rc1 cleanup of the ATAPI tests that were problematic during the > rc0 testing window. This set does two things: > > (1) Fix the timeouts to be more deterministic, and > (2) Finish tidying up the PIO loop. > > v2: > - Clarify the commit message on

[Qemu-devel] [PATCH v2 1/1] ide-test: fix timeouts

2015-11-24 Thread John Snow
Use explicit timeouts instead of trying to approximate it by counting the cumulative duration of nsleep calls. In practice, the timeout if inb() dwarfed the nsleep delays, and as a result the real timeout value became a lot larger than 5 seconds. So: change the semantics from "Not sooner than 5 s

[Qemu-devel] [PATCH v2 0/1] ide-test: tidy up after pio race fix

2015-11-24 Thread John Snow
Post-rc1 cleanup of the ATAPI tests that were problematic during the rc0 testing window. This set does two things: (1) Fix the timeouts to be more deterministic, and (2) Finish tidying up the PIO loop. v2: - Clarify the commit message on patch 1, and remove the second needless stack variable.

Re: [Qemu-devel] [RFC PATCH 0/9] vhost-nvme: new qemu nvme backend using nvme target

2015-11-24 Thread Ming Lin
On Tue, 2015-11-24 at 11:51 +0100, Paolo Bonzini wrote: > > On 24/11/2015 08:27, Ming Lin wrote: > > handle_notify (qemu/hw/block/dataplane/virtio-blk.c:126) > > aio_dispatch (qemu/aio-posix.c:329) > > aio_poll (qemu/aio-posix.c:474) > > iothread_run (qemu/iothread.c:45) > > start_thread (pthread_

Re: [Qemu-devel] [RFC v1] virtio-crypto specification

2015-11-24 Thread Varun Sethi
Hi Gonglei, We have been working on the virtio-ipsec look-aside accelerator device specification at the OPNFV DPACC forum. The virtio-ipsec device is aimed at providing the protocol offload capabilities (offered by security hardware accelerator) to the VM. The device supports a control queue an

[Qemu-devel] [Bug 1479717] Re: Auto resize VM doesn't work with windows 10 guest

2015-11-24 Thread Jean-Pierre van Riel
Should someone else stumble upon this, the way to resolve issues for now is to not use gnome boxes, but rather remote-viewer and perhaps there's an issue with BIOS/EFI graphics setup with Windows 10 guests? After a lucky coincidence, flickering seems to have be resolved for me while tweaking somet

Re: [Qemu-devel] [PATCH for 2.6 2/3] block: Hide HBitmap in block dirty bitmap interface

2015-11-24 Thread John Snow
On 11/23/2015 09:28 PM, Fam Zheng wrote: > On Mon, 11/23 16:34, John Snow wrote: >> Hmm, what's the idea, here? >> >> This patch does a lot more than just hide hbitmap details from callers >> of block_dirty_bitmap functions. >> >> So we're changing the backing hbitmap to always be one where g=0 a

Re: [Qemu-devel] [PATCH for-2.5] qga: Better mapping of SEEK_* in guest-file-seek

2015-11-24 Thread Eric Blake
On 11/24/2015 11:57 AM, Eric Blake wrote: > Exposing OS-specific SEEK_ constants in our qapi was a mistake > (if the host has SEEK_CUR as 1, but the guest has it as 2, then > the semantics are unclear what should happen); if we had a time > machine, we would instead expose only a symbolic enum. It

Re: [Qemu-devel] [PATCH 1/2] qga: flush implicitely when needed

2015-11-24 Thread Eric Blake
On 11/24/2015 10:52 AM, Marc-André Lureau wrote: > Hi > > - Original Message - >> On 11/24/2015 09:34 AM, marcandre.lur...@redhat.com wrote: >>> From: Marc-André Lureau >> >> In the subject: s/implicitely/implicitly/ if you are fixing the typo, or >> s/implicitely/explicitly/ if you are t

  1   2   3   4   5   >