Re: [Qemu-devel] [PATCH v3 00/10] curl: fix curl read

2013-05-21 Thread Richard W.M. Jones
On Tue, May 21, 2013 at 09:54:15AM +0800, Fam Zheng wrote: On Mon, 05/20 09:49, Richard W.M. Jones wrote: On Mon, May 20, 2013 at 09:41:06AM +0100, Richard W.M. Jones wrote: On Mon, May 20, 2013 at 03:03:34PM +0800, Fam Zheng wrote: CURL library API has changed, the current curl driver

Re: [Qemu-devel] [PATCH v4 00/10] curl: fix curl read

2013-05-22 Thread Richard W.M. Jones
On Wed, May 22, 2013 at 11:16:40AM +0800, Fam Zheng wrote: Changes from v3: 01, 06, 07: Add QLIST_INIT in qemu_open to initialize each list. 07: Move clean up for s-acbs from later patch to here. Use qemu_aio_relase instead of g_free on acb. Fix use-after-free bug. [Rich] This

Re: [Qemu-devel] [PATCH v4 00/10] curl: fix curl read

2013-05-22 Thread Richard W.M. Jones
://people.redhat.com/~rjones/virt-df/ From 0d42755f009c142b85fb07a06a41cee6c7ea68ba Mon Sep 17 00:00:00 2001 From: Richard W.M. Jones rjo...@redhat.com Date: Wed, 22 May 2013 12:11:45 +0100 Subject: [PATCH] block/curl.c: Refuse to open the handle for writes. --- block/curl.c | 4 1 file changed, 4

[Qemu-devel] [PATCH] block/curl.c: Refuse to open the handle for writes.

2013-05-22 Thread Richard W.M. Jones
From: Richard W.M. Jones rjo...@redhat.com --- block/curl.c | 4 1 file changed, 4 insertions(+) diff --git a/block/curl.c b/block/curl.c index b8935fd..f1e302b 100644 --- a/block/curl.c +++ b/block/curl.c @@ -406,6 +406,10 @@ static int curl_open(BlockDriverState *bs, QDict *options, int

Re: [Qemu-devel] [RFC] block-trace Low Level Command Supporting Disk Introspection

2013-05-22 Thread Richard W.M. Jones
On Wed, May 22, 2013 at 11:51:16AM -0400, Wolfgang Richter wrote: This is actually interesting. Does the QEMU nbd server support multiple readers? Yes. qemu-nbd has a -e/--shared=N option which appears to do exactly what it says in the man page. $ guestfish -N fs exit $ ls -lh test1.img

Re: [Qemu-devel] [PATCH v2] wdt_i6300esb: fix vmstate versioning

2013-05-22 Thread Richard W.M. Jones
On Wed, May 22, 2013 at 11:32:51AM -0500, Michael Roth wrote: When this VMSD was introduced it's version fields were set to sizeof(I6300State), making them essentially random from build to build, version to version. To fix this, we lock in a high version id and low minimum version id to

Re: [Qemu-devel] [RFC] block-trace Low Level Command Supporting Disk Introspection

2013-05-22 Thread Richard W.M. Jones
On Wed, May 22, 2013 at 02:32:37PM -0400, Wolfgang Richter wrote: On Wed, May 22, 2013 at 12:42 PM, Richard W.M. Jones rjo...@redhat.comwrote: Run up to two extra guestfish instances, with the same result. The fourth guestfish instance hangs at the 'run' command until one of the first

Re: [Qemu-devel] [RFC] block-trace Low Level Command Supporting Disk Introspection

2013-05-22 Thread Richard W.M. Jones
On Wed, May 22, 2013 at 03:38:33PM -0400, Wolfgang Richter wrote: On Wed, May 22, 2013 at 3:26 PM, Richard W.M. Jones rjo...@redhat.comwrote: On Wed, May 22, 2013 at 02:32:37PM -0400, Wolfgang Richter wrote: On Wed, May 22, 2013 at 12:42 PM, Richard W.M. Jones rjo...@redhat.com wrote

Re: [Qemu-devel] [PATCH v5 00/11] curl: fix curl read

2013-05-23 Thread Richard W.M. Jones
v5 tested and works for me. Attached is the test script I'm using. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones virt-p2v converts physical machines to virtual machines. Boot with a live CD or over the network (PXE) and turn machines into KVM guests.

[Qemu-devel] ANNOUNCE: libguestfs 1.22 has been released

2013-05-23 Thread Richard W.M. Jones
I'm pleased to announce the next stable release of libguestfs (1.22). libguestfs is a set of tools for accessing and modifying virtual machine disk images. http://libguestfs.org/ This release represents 5 months of development and has many significant new features including: - access remote

Re: [Qemu-devel] [Libguestfs] ANNOUNCE: libguestfs 1.22 has been released

2013-05-24 Thread Richard W.M. Jones
On Fri, May 24, 2013 at 08:42:37AM +0530, Kashyap Chamarthy wrote: On 05/23/2013 05:53 PM, Richard W.M. Jones wrote: I'm pleased to announce the next stable release of libguestfs (1.22). Great work! == $ ./autogen.sh make -j 7 \ time make -k check LIBGUESTFS_DEBUG=1

Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read

2013-05-24 Thread Richard W.M. Jones
On Fri, May 24, 2013 at 01:36:55PM +0800, Fam Zheng wrote: Changes from v5: 05: Rename bs to s for BDRVCURLState. 06: Use int64_t for offsets. Fix printf format string. Move introducing of use_count to 07. 07: Drop explicit cast. Use int64_t for offsets.

Re: [Qemu-devel] [PATCH] Remove unnecessary break statements

2013-05-24 Thread Richard W.M. Jones
On Fri, May 24, 2013 at 12:19:25PM +0200, Stefan Weil wrote: Fix these warnings from cppcheck: hw/display/cirrus_vga.c:2603: hw/sd/sd.c:348: hw/timer/exynos4210_mct.c:1033: target-arm/translate.c:9886: target-s390x/mem_helper.c:518: target-unicore32/translate.c:1936: style: Consecutive

Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read

2013-05-27 Thread Richard W.M. Jones
On Mon, May 27, 2013 at 10:25:21AM +0800, Fam Zheng wrote: On Fri, 05/24 09:07, Richard W.M. Jones wrote: On Fri, May 24, 2013 at 01:36:55PM +0800, Fam Zheng wrote: Changes from v5: 05: Rename bs to s for BDRVCURLState. 06: Use int64_t for offsets. Fix printf format string

Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read

2013-05-28 Thread Richard W.M. Jones
On Tue, May 28, 2013 at 03:30:49PM +0800, Fam Zheng wrote: Sure, I'm using the attached test script. I used your script to test, but I don't see errors as you posted, attached the output. The only difference is that I put libguestfs in different directory with you and I'm using a linux

Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read

2013-05-28 Thread Richard W.M. Jones
On Tue, May 28, 2013 at 08:47:59AM +0100, Richard W.M. Jones wrote: I'm not sure if a Windows guest is somehow necessary to show the errors. I'll retest with a Linux guest and get back to you about that. Reproducible with Linux guest (remotely over slow Wifi). Also I'm testing against

Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read

2013-05-28 Thread Richard W.M. Jones
After discussion off-list, I've gone back and retested versions 4, 5, and 6 of this patch. I'm using the test script previously attached. I'm using libguestfs (ada94eb9) curl (ba9a) qemu (6a4e177114) all the latest from git. I'm using a 6 GB Windows XP guest. The web server is remote,

Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read

2013-05-28 Thread Richard W.M. Jones
On Tue, May 28, 2013 at 11:35:20AM +0100, Richard W.M. Jones wrote: I'm continuing to investigate. Some more data points: v6 patch, with my laptop plugged directly into the gigabit ethernet switch which is connected to the web server: - Worked perfectly 5 times in a row. v6 patch, with my

Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read

2013-05-28 Thread Richard W.M. Jones
W.M. Jones rjo...@redhat.com Date: Tue, 28 May 2013 12:30:07 +0100 Subject: [PATCH] curl: Increase block timeout. Signed-off-by: Richard W.M. Jones rjo...@redhat.com --- block/curl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/block/curl.c b/block/curl.c index 3e330b6

Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read

2013-05-29 Thread Richard W.M. Jones
On Wed, May 29, 2013 at 09:07:25AM +0800, Fam Zheng wrote: On Tue, 05/28 12:32, Richard W.M. Jones wrote: This fixes the obvious bug. Thanks for figuring out this. Mainline had this 5s timeout so I kept it, but you don't experience this bug, right? Since master doesn't setup a timer

Re: [Qemu-devel] [PATCH uq/master] fix double free the memslot in kvm_set_phys_mem

2013-06-02 Thread Richard W.M. Jones
W.M. Jones rjo...@redhat.com Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones libguestfs lets you edit virtual machines. Supports shell scripting, bindings from many languages. http://libguestfs.org

Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read

2013-06-06 Thread Richard W.M. Jones
On Thu, Jun 06, 2013 at 02:25:46PM +0800, Fam Zheng wrote: v7: 13: Added: curl: change timeout to 30 seconds I tested this against: (1) HTTP to Apache server over slow but local wifi. (2) HTTP to a remote ISO (on another continent). Test (1) is fine. Test (2) gives plenty of I/O

Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read

2013-06-07 Thread Richard W.M. Jones
On Fri, Jun 07, 2013 at 09:54:42AM +0800, Fam Zheng wrote: The weird thing is it doesn't work for me, I'm sure something is wrong with current upstream, although not totally broken. $./qemu-io http://localhost/vm/arch.raw -c 'read -v 0 512' (stuck here forever, no output) This doesn't

Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read

2013-06-07 Thread Richard W.M. Jones
On Fri, Jun 07, 2013 at 03:47:23PM +0800, Fam Zheng wrote: On Fri, 06/07 08:27, Richard W.M. Jones wrote: On Fri, Jun 07, 2013 at 09:54:42AM +0800, Fam Zheng wrote: The weird thing is it doesn't work for me, I'm sure something is wrong with current upstream, although not totally broken

[Qemu-devel] [PATCH] block/curl.c: Refuse to open the handle for writes.

2013-06-10 Thread Richard W.M. Jones
This simple patch avoids a segfault in qemu if the user tries to open a curl disk for writing. This was previously part of Fam Zheng's curl patch series, but it stands alone and we shouldn't forget about it. Rich.

[Qemu-devel] [PATCH] block/curl.c: Refuse to open the handle for writes.

2013-06-10 Thread Richard W.M. Jones
From: Richard W.M. Jones rjo...@redhat.com Signed-off-by: Richard W.M. Jones rjo...@redhat.com Signed-off-by: Fam Zheng f...@redhat.com --- block/curl.c | 4 1 file changed, 4 insertions(+) diff --git a/block/curl.c b/block/curl.c index b8935fd..f1e302b 100644 --- a/block/curl.c +++ b

[Qemu-devel] [PATCH v2] block/curl.c: Refuse to open the handle for writes.

2013-06-10 Thread Richard W.M. Jones
From: Richard W.M. Jones rjo...@redhat.com Signed-off-by: Richard W.M. Jones rjo...@redhat.com Signed-off-by: Fam Zheng f...@redhat.com --- block/curl.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/block/curl.c b/block/curl.c index b8935fd..b634ccf 100644 --- a/block/curl.c +++ b

[Qemu-devel] [PATCH v2] block/curl.c: Refuse to open the handle for writes.

2013-06-10 Thread Richard W.M. Jones
v2: - Use qerror_report to report an error. - Return -EROFS instead of -ENOTSUP.

[Qemu-devel] [PATCH] curl: Don't set curl options on the handle just before it's going to be deleted.

2013-06-10 Thread Richard W.M. Jones
From: Richard W.M. Jones rjo...@redhat.com (Found by Kamil Dudka) Signed-off-by: Richard W.M. Jones rjo...@redhat.com --- block/curl.c | 1 - 1 file changed, 1 deletion(-) diff --git a/block/curl.c b/block/curl.c index b634ccf..bf31efe 100644 --- a/block/curl.c +++ b/block/curl.c @@ -453,7

Re: [Qemu-devel] [Qemu-trivial] [PATCH] curl: Don't set curl options on the handle just before it's going to be deleted.

2013-06-10 Thread Richard W.M. Jones
On Mon, Jun 10, 2013 at 07:34:52PM +0400, Michael Tokarev wrote: 10.06.2013 17:19, Richard W.M. Jones wrote: From: Richard W.M. Jones rjo...@redhat.com (Found by Kamil Dudka) Signed-off-by: Richard W.M. Jones rjo...@redhat.com --- block/curl.c | 1 - 1 file changed, 1 deletion

[Qemu-devel] [PATCH v2] curl: Don't set curl options on the handle just before it's going to be deleted.

2013-06-10 Thread Richard W.M. Jones
From: Richard W.M. Jones rjo...@redhat.com (Found by Kamil Dudka) Signed-off-by: Richard W.M. Jones rjo...@redhat.com Cc: Michael Tokarev m...@tls.msk.ru --- block/curl.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/block/curl.c b/block/curl.c index b634ccf..8d0d45d 100644 --- a/block

Re: [Qemu-devel] [PATCH v8 0/3] block: Add support for Secure Shell (ssh) block device.

2013-10-17 Thread Richard W.M. Jones
On Tue, Apr 09, 2013 at 10:56:30AM +0100, Richard W.M. Jones wrote: Changes since v7: - fsync (ie. bdrv_co_flush_to_disk) is now supported, *if* you have the following patches to libssh2 and OpenSSH: https://bugzilla.mindrot.org/show_bug.cgi?id=1798 (OpenSSH: accepted, but not upstream

Re: [Qemu-devel] [PATCH] MAINTAINERS: add block driver sub-maintainers

2013-10-21 Thread Richard W.M. Jones
/vhdx.c + +VDI +M: Stefan Weil s...@weilnetz.de +S: Maintained +F: block/vdi.c + +iSCSI +M: Ronnie Sahlberg ronniesahlb...@gmail.com +M: Paolo Bonzini pbonz...@redhat.com +S: Supported +F: block/iscsi.c + +SSH +M: Richard W.M. Jones rjo...@redhat.com +S: Supported +F: block/ssh.c

Re: [Qemu-devel] [PATCH] MAINTAINERS: add block driver sub-maintainers

2013-10-21 Thread Richard W.M. Jones
On Mon, Oct 21, 2013 at 09:19:42PM +0100, Anthony Liguori wrote: On Mon, Oct 21, 2013 at 2:26 PM, Stefan Hajnoczi stefa...@redhat.com wrote: There are a number of contributors who maintain block drivers (image formats and protocols). They should be listed in the MAINTAINERS file so that

[Qemu-devel] ARM virtio-scsi - disks never show up

2013-09-05 Thread Richard W.M. Jones
I don't know if there is a more appropriate place to ask about ARM / virtio questions. If there is please let me know. I'm trying to get qemu to boot an ARM appliance. Everything works up to the point where the kernel loads virtio drivers. The disks never show up. The full qemu command line

Re: [Qemu-devel] ARM virtio-scsi - disks never show up

2013-09-05 Thread Richard W.M. Jones
On Thu, Sep 05, 2013 at 10:50:02PM +0100, Peter Maydell wrote: On 5 September 2013 22:41, Richard W.M. Jones rjo...@redhat.com wrote: I don't know if there is a more appropriate place to ask about ARM / virtio questions. If there is please let me know. I'm trying to get qemu to boot

[Qemu-devel] ARM virtio-serial not showing up (was: Re: ARM virtio-scsi - disks never show up)

2013-09-06 Thread Richard W.M. Jones
Have you tried virtio-serial on ARM at all? The port never shows up. Do all virtio-* devices need to be ported specially to virtio-mmio or should they (in theory at least) just work? Is there a command one can use in the guest to list virtio-mmio devices, similar to lspci? Are virtio-mmio

Re: [Qemu-devel] ARM virtio-serial not showing up (was: Re: ARM virtio-scsi - disks never show up)

2013-09-07 Thread Richard W.M. Jones
On Sat, Sep 07, 2013 at 09:49:24AM +0100, Peter Maydell wrote: On 6 September 2013 19:01, Richard W.M. Jones rjo...@redhat.com wrote: Have you tried virtio-serial on ARM at all? The port never shows up. I don't think I've tested the serial-device part of it, but I've used it to provide

Re: [Qemu-devel] ARM virtio-serial not showing up (was: Re: ARM virtio-scsi - disks never show up)

2013-09-08 Thread Richard W.M. Jones
On Sat, Sep 07, 2013 at 09:49:24AM +0100, Peter Maydell wrote: On 6 September 2013 19:01, Richard W.M. Jones rjo...@redhat.com wrote: Have you tried virtio-serial on ARM at all? The port never shows up. I don't think I've tested the serial-device part of it, but I've used it to provide

Re: [Qemu-devel] [Qemu-trivial] [PATCH] q35: Fix typo in constant DEFUALT - DEFAULT.

2013-09-14 Thread Richard W.M. Jones
On Sat, Sep 14, 2013 at 12:47:47PM +0400, Michael Tokarev wrote: 02.09.2013 17:43, Richard W.M. Jones wrote: From: Richard W.M. Jones rjo...@redhat.com Signed-off-by: Richard W.M. Jones rjo...@redhat.com Thanks, applied to the trivial-patches queue. Thanks :-) I case you were wondering

Re: [Qemu-devel] [PATCH V9 0/2] Implement sync modes for drive-backup.

2013-07-29 Thread Richard W.M. Jones
On Mon, Jul 29, 2013 at 10:52:44AM +0200, Kevin Wolf wrote: Am 26.07.2013 um 20:39 hat Ian Main geschrieben: This patch adds sync modes on top of the work that Stefan Hajnoczi has done. These patches apply on kevin/block. Hopefully all is in order as this is my first QEMU patch. Many

Re: [Qemu-devel] [PATCH V9 0/2] Implement sync modes for drive-backup.

2013-07-29 Thread Richard W.M. Jones
On Mon, Jul 29, 2013 at 05:36:28PM +0800, Fam Zheng wrote: On Mon, 07/29 10:02, Richard W.M. Jones wrote: On Mon, Jul 29, 2013 at 10:52:44AM +0200, Kevin Wolf wrote: Am 26.07.2013 um 20:39 hat Ian Main geschrieben: This patch adds sync modes on top of the work that Stefan Hajnoczi has

Re: [Qemu-devel] [PATCH V9 0/2] Implement sync modes for drive-backup.

2013-07-29 Thread Richard W.M. Jones
On Mon, Jul 29, 2013 at 06:16:48PM +0800, Fam Zheng wrote: Ian's patches are in master, on which I already rebased the first. I can rebase the other one for you if you feel like to test it too. No that's fine, thanks. I just thought Ian had more patches which were not upstream, but if they're

[Qemu-devel] [PATCH] block/iscsi.c: Fix printf format error.

2013-07-31 Thread Richard W.M. Jones
From: Richard W.M. Jones rjo...@redhat.com The error on armv7hl was: block/iscsi.c: In function ‘is_request_lun_aligned’: block/iscsi.c:251:26: error: format ‘%ld’ expects argument of type ‘long int’, but argument 3 has type ‘int64_t’ [-Werror=format=] iscsilun

[Qemu-devel] [PATCH v2] block/iscsi.c: Fix printf format error.

2013-07-31 Thread Richard W.M. Jones
From: Richard W.M. Jones rjo...@redhat.com The error on armv7hl was: block/iscsi.c: In function ‘is_request_lun_aligned’: block/iscsi.c:251:26: error: format ‘%ld’ expects argument of type ‘long int’, but argument 3 has type ‘int64_t’ [-Werror=format=] iscsilun

[Qemu-devel] [PATCH v3] block/iscsi.c: Fix printf format error.

2013-07-31 Thread Richard W.M. Jones
From: Richard W.M. Jones rjo...@redhat.com The error on armv7hl was: block/iscsi.c: In function ‘is_request_lun_aligned’: block/iscsi.c:251:26: error: format ‘%ld’ expects argument of type ‘long int’, but argument 3 has type ‘int64_t’ [-Werror=format=] iscsilun

[Qemu-devel] Using virtio-mmio

2013-07-31 Thread Richard W.M. Jones
It's quite exciting the virtio-mmio made it into qemu. Are extra guest kernel components needed too? (I'm using 3.9.9-302.fc19.armv7hl but could try a later kernel) Anyway, I tried to get it to work, but can't quite work out the qemu command line. So far I have: $

Re: [Qemu-devel] Using virtio-mmio

2013-07-31 Thread Richard W.M. Jones
On Wed, Jul 31, 2013 at 11:01:11PM +0100, Peter Maydell wrote: On 31 July 2013 22:56, Richard W.M. Jones rjo...@redhat.com wrote: It's quite exciting the virtio-mmio made it into qemu. Are extra guest kernel components needed too? (I'm using 3.9.9-302.fc19.armv7hl but could try a later

Re: [Qemu-devel] Using virtio-mmio

2013-08-01 Thread Richard W.M. Jones
On Thu, Aug 01, 2013 at 10:32:14AM +0100, Peter Maydell wrote: On 31 July 2013 23:45, Richard W.M. Jones rjo...@redhat.com wrote: ~/d/qemu/arm-softmmu/qemu-system-arm \ -m 512 -M vexpress-a9 -machine kernel_irqchip=on \ The combination of 'vexpress-a9' and kernel_irqchip=on don't make

Re: [Qemu-devel] Using virtio-mmio

2013-08-01 Thread Richard W.M. Jones
On Thu, Aug 01, 2013 at 11:58:15AM +0100, Peter Maydell wrote: Unfortunately at this point you run into the classic issue of trying to get an ARM kernel running, which is that a huge class of config errors all have the failure mode just sits there with no serial output. This is remarkably

Re: [Qemu-devel] [PATCH v2] curl: refuse to open URL from HTTP server without range support

2013-06-17 Thread Richard W.M. Jones
On Mon, Jun 17, 2013 at 06:09:23PM +0800, Fam Zheng wrote: +if (real_size = strlen(accept_len) What's accept_len? This patch gives me a couple of compile errors: block/curl.c: In function ‘curl_header_cb’: block/curl.c:120:9: error: ‘real_size’ undeclared (first use in this function)

Re: [Qemu-devel] [PATCH v2] curl: refuse to open URL from HTTP server without range support

2013-06-17 Thread Richard W.M. Jones
On Mon, Jun 17, 2013 at 07:02:00PM +0800, Fam Zheng wrote: On Mon, 06/17 11:54, Richard W.M. Jones wrote: On Mon, Jun 17, 2013 at 06:09:23PM +0800, Fam Zheng wrote: +if (real_size = strlen(accept_len) What's accept_len? This patch gives me a couple of compile errors: Oops

Re: [Qemu-devel] block: Review of .has_zero_init use

2013-06-25 Thread Richard W.M. Jones
On Tue, Jun 25, 2013 at 01:39:11PM +0200, Kevin Wolf wrote: * ssh - currently has_zero_init = 1 (is this correct?) [...] It might be possible that the correct value depends on the backend on the server side for some protocols - for example, I think for SSH it depends on whether you

[Qemu-devel] [PATCH] block/ssh: Set bdrv_has_zero_init according to the file type.

2013-06-25 Thread Richard W.M. Jones
From: Richard W.M. Jones rjo...@redhat.com If the remote is a regular file, set it to true (ie. reads of uninitialized areas in a newly created file will return zeroes). If we can't prove that, return false (a safe default). Tested by adding a debugging print statement [not part of this commit

Re: [Qemu-devel] Monitoring Screen Activity in QEMU/KVM

2013-06-26 Thread Richard W.M. Jones
On Wed, Jun 26, 2013 at 07:47:47AM +0530, Shehbaz Jaffer wrote: I want to determine the amount of screen activity taking place on VGA monitor/ Screen for different applications (eg. playing vlc video, normal typing.) When I do not start the X server, I can easily determine the screen

Re: [Qemu-devel] [PATCH] block: change default of .has_zero_init to 0

2013-06-27 Thread Richard W.M. Jones
On Thu, Jun 27, 2013 at 03:52:34PM +0200, Peter Lieven wrote: +/* save default */ A minor thing, but s/save/safe/ :-) Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones virt-p2v converts physical machines to virtual machines. Boot with a live CD or

[Qemu-devel] Re: qdev: Some ISA devices don't handle second instantiation gracefully (was: isa-applesmc doesn't handle second instantiation gracefully)

2010-10-13 Thread Richard W.M. Jones
On Tue, Oct 12, 2010 at 03:00:13PM +0200, Markus Armbruster wrote: Markus Armbruster arm...@redhat.com writes: When I try -device isa-applesmc -device isa-applesmc, I get WARNING: Using AppleSMC with invalid key qemu: hardware error: register_ioport_read: invalid opaque

Re: [Qemu-devel] Hitting 29 NIC limit

2010-10-14 Thread Richard W.M. Jones
On Thu, Oct 14, 2010 at 01:10:47PM +0100, Daniel P. Berrange wrote: Or a PCI bridge to wire up more PCI buses, so we raise the max limit for any type of device we emulate. Break the 29/30/31 virtio-blk limit ... please! Rich. -- Richard Jones, Virtualization Group, Red Hat

Re: [Qemu-devel] Hitting 29 NIC limit

2010-10-14 Thread Richard W.M. Jones
On Thu, Oct 14, 2010 at 04:57:36PM -0500, Anthony Liguori wrote: On 10/14/2010 04:42 PM, Richard W.M. Jones wrote: On Thu, Oct 14, 2010 at 01:10:47PM +0100, Daniel P. Berrange wrote: Or a PCI bridge to wire up more PCI buses, so we raise the max limit for any type of device we emulate. Break

Tracing block devices (was: Re: [Qemu-devel] Static tracepoint control via trace-event)

2010-10-19 Thread Richard W.M. Jones
On Tue, Oct 19, 2010 at 03:59:36PM +0200, Jan Kiszka wrote: Once we have -trace events=..., defining the list of active tracepoints before starting qemu will be trivial (e.g. via a config file). Of course, this requires that all tracepoints are built-in... Sorry that I've not been following

Re: Tracing block devices (was: Re: [Qemu-devel] Static tracepoint control via trace-event)

2010-10-21 Thread Richard W.M. Jones
On Thu, Oct 21, 2010 at 07:20:27AM +0200, Christoph Hellwig wrote: On Tue, Oct 19, 2010 at 03:29:51PM +0100, Richard W.M. Jones wrote: On Tue, Oct 19, 2010 at 03:59:36PM +0200, Jan Kiszka wrote: Once we have -trace events=..., defining the list of active tracepoints before starting qemu

Re: [Qemu-devel] [PATCH] wdt_i6300esb: register a reset function

2010-12-12 Thread Richard W.M. Jones
On Sat, Dec 11, 2010 at 06:39:03PM +, Blue Swirl wrote: Thanks, applied. Wait! This patch is incomplete. I already posted a complete patch already some months ago (twice) but it was ignored both times: http://www.mail-archive.com/qemu-devel@nongnu.org/msg42716.html

Re: [Qemu-devel] [PATCH] wdt_i6300esb: register a reset function

2010-12-12 Thread Richard W.M. Jones
On Sun, Dec 12, 2010 at 10:59:59AM +, Blue Swirl wrote: On Sun, Dec 12, 2010 at 10:08 AM, Richard W.M. Jones rjo...@redhat.com wrote: On Sat, Dec 11, 2010 at 06:39:03PM +, Blue Swirl wrote: Thanks, applied. Wait!  This patch is incomplete. I already posted a complete patch

[Qemu-devel] Re: [PATCH v3 2/7] wdt_ib700: Don't use SoftFloat int64 type

2010-12-18 Thread Richard W.M. Jones
On Sat, Dec 18, 2010 at 05:25:26PM +0100, Andreas Färber wrote: softfloat.h's int64 type has least-width semantics, but this doesn't seem intended here, so use plain int64_t. v3: * Split off. Cc: Richard W.M. Jones rjo...@redhat.com Signed-off-by: Andreas Färber andreas.faer...@web.de

Re: [Qemu-devel] Re: [PATCH v3 2/7] wdt_ib700: Don't use SoftFloat int64 type

2010-12-19 Thread Richard W.M. Jones
On Sun, Dec 19, 2010 at 01:51:22PM +0100, Andreas Färber wrote: Am 18.12.2010 um 17:47 schrieb Richard W.M. Jones: On Sat, Dec 18, 2010 at 05:25:26PM +0100, Andreas Färber wrote: softfloat.h's int64 type has least-width semantics, but this doesn't seem intended here, so use plain int64_t

[Qemu-devel] Re: [PATCH] qemu-kvm: Switch to upstream -enable-kvm semantics

2010-12-21 Thread Richard W.M. Jones
On Tue, Dec 21, 2010 at 04:41:03PM +0100, Markus Armbruster wrote: Like this? upstream qemu | default |-enable-kvm +---+--- KVM available | disabled | enabled KVM unavailable | disabled |fail qemu-kvm| default |-enable-kvm| -no-kvm

[Qemu-devel] Re: [PATCH] qemu-kvm: Switch to upstream -enable-kvm semantics

2010-12-21 Thread Richard W.M. Jones
On Tue, Dec 21, 2010 at 04:00:32PM +, Richard W.M. Jones wrote: Markus, any idea when we might get the -accel option appearing in released versions of qemu/KVM? Sorry, I thought this email wasn't going out to a public list. I should be more careful next time. I'll say instead: We really

[Qemu-devel] Re: [fedora-virt] Fedora Virt status

2011-01-14 Thread Richard W.M. Jones
On Thu, Jan 13, 2011 at 08:27:17AM -0800, Boris Derzhavets wrote: Windows 7 KVM install runs unbelievably fast on SL 6 ( alpha 3) vs F14 with the most recent libvirt stuff : Boris can you help me to parse this ... SL6 = Scientific Linux 6? And you're saying that Windows 7 is much faster on

Re: [Qemu-devel] Re: [fedora-virt] Fedora Virt status

2011-01-14 Thread Richard W.M. Jones
On Fri, Jan 14, 2011 at 12:39:39PM -0800, Boris Derzhavets wrote: Newer virt-manager versions also allow setting this in the UI. I just set IDE Disk1 cashe mode none  before install. I thought you said they were using virtio? Rich. -- Richard Jones, Virtualization Group, Red Hat

Re: [Qemu-devel] Re: [fedora-virt] Fedora Virt status

2011-01-14 Thread Richard W.M. Jones
You shouldn't use local files (esp. not qcow2) for performance testing. What does it look like if you use an LV for disk? Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones virt-p2v converts physical machines to virtual machines. Boot with a live CD or

[Qemu-devel] Re: [fedora-virt] Fedora Virt status (XML for SL6)

2011-01-15 Thread Richard W.M. Jones
On Fri, Jan 14, 2011 at 12:17:42PM -0800, Boris Derzhavets wrote:     disk type='block' device='disk'   driver name='qemu' type='raw' cache='none'/ This one (SL6) has cache=none, but the Fedora 14 XML you posted does not. Rich. -- Richard Jones, Virtualization Group, Red Hat

[Qemu-devel] Re: [fedora-virt] Fedora Virt status (XML for SL6)

2011-01-15 Thread Richard W.M. Jones
On Sat, Jan 15, 2011 at 03:15:05AM -0800, Boris Derzhavets wrote: Please, view thread. I've read every part of the thread(s). You don't make it easy to follow what you're talking about. I would suggest: (1) Organize your thoughts and experiments logically. (2) When you have done (1), post a

Re: [Qemu-devel] Re: [RFC][PATCH v6 08/23] virtagent: add va.getfile RPC

2011-01-24 Thread Richard W.M. Jones
On Fri, Jan 21, 2011 at 12:23:40PM -0600, Michael Roth wrote: getfile() is confusingly named however, it's really just a means to peek at a text file like /proc/meminfo. You might as well reuse the libguestfs API here because you get the benefit of all the code that's been written, all the

Re: [Qemu-devel] Re: [RFC][PATCH v6 08/23] virtagent: add va.getfile RPC

2011-01-24 Thread Richard W.M. Jones
On Mon, Jan 24, 2011 at 10:08:09PM +, Richard W.M. Jones wrote: You might as well reuse the libguestfs API here because you get the benefit of all the code that's been written, all the tools on top, and a far more comprehensive API that would take you another 2 years to implement. To put

Re: [Qemu-devel] Re: [RFC][PATCH v6 08/23] virtagent: add va.getfile RPC

2011-01-24 Thread Richard W.M. Jones
On Mon, Jan 24, 2011 at 04:26:09PM -0600, Anthony Liguori wrote: On 01/24/2011 04:20 PM, Richard W.M. Jones wrote: On Mon, Jan 24, 2011 at 10:08:09PM +, Richard W.M. Jones wrote: You might as well reuse the libguestfs API here because you get the benefit of all the code that's been written

Re: [Qemu-devel] Re: [RFC][PATCH v6 08/23] virtagent: add va.getfile RPC

2011-01-25 Thread Richard W.M. Jones
On Mon, Jan 24, 2011 at 05:40:05PM -0600, Anthony Liguori wrote: BTW, how dependent is guestfsd on the guest that libguestfs uses? I wasn't even aware that it could be used outside of that context. The daemon is compiled separately -- separate ./configure, make, etc. You can run it on its own.

Re: [Qemu-devel] Re: [RFC][PATCH v6 08/23] virtagent: add va.getfile RPC

2011-01-25 Thread Richard W.M. Jones
On Tue, Jan 25, 2011 at 09:12:15AM -0600, Anthony Liguori wrote: How much infrastructure does guestfd depend on within the guest? Do you need a full install with all of the bells and whistles or does it degrade cleanly when certain tools aren't available? On Linux these are the libraries,

Re: [Qemu-devel] Re: [RFC][PATCH v6 08/23] virtagent: add va.getfile RPC

2011-01-26 Thread Richard W.M. Jones
I posted my thoughts about how this could work here: https://www.redhat.com/archives/libguestfs/2011-January/msg00066.html Rich. PS. You don't need to be a subscriber to post to that list -- I manually triage any messages sent by non-subscribers. -- Richard Jones, Virtualization Group, Red

Re: [Qemu-devel] [PATCH 0/2] virtagent - fsfreeze support

2011-02-01 Thread Richard W.M. Jones
On Tue, Feb 01, 2011 at 02:25:12PM +0300, Vasiliy G Tolstov wrote: On Tue, 2011-02-01 at 11:58 +0100, jes.soren...@redhat.com wrote: From: Jes Sorensen jes.soren...@redhat.com Hi This is a first attempt to add fsfreeze support to virtagent. The idea is for the guest agent to walk

Re: [Qemu-devel] [PATCH 0/2] virtagent - fsfreeze support

2011-02-01 Thread Richard W.M. Jones
On Tue, Feb 01, 2011 at 11:04:47PM +0300, Vasiliy G Tolstov wrote: On Tue, 2011-02-01 at 16:04 +, Richard W.M. Jones wrote: There are some experimental patches to libguestfs to do live filesystem and partition manipulations now: https://www.redhat.com/archives/libguestfs/2011

Re: [Qemu-devel] Buildbot for qemu.git/master

2011-02-07 Thread Richard W.M. Jones
On Sat, Feb 05, 2011 at 04:36:11PM +, Stefan Hajnoczi wrote: Occassionally a commit that breaks the build gets merged into qemu.git/master. Build testing manually across all host platforms is not feasible for most developers. Remember we cover 32- and 64-bit x86 Linux, Windows, and

Re: [Qemu-devel] [PATCH] move 'unsafe' to end of caching modes in help

2010-08-03 Thread Richard W.M. Jones
On Tue, Jul 27, 2010 at 10:26:10AM +0200, Markus Armbruster wrote: Anthony Liguori anth...@codemonkey.ws writes: On 07/26/2010 02:19 PM, Avi Kivity wrote: [...] Regardless, outside of Windows users qemu will mostly be consumed via distribution branches, with different levels of backport

Re: [Qemu-devel] [PATCHv3] Load bootsplash.jpg if present

2010-08-03 Thread Richard W.M. Jones
On Mon, Aug 02, 2010 at 12:47:53PM -0400, Kevin O'Connor wrote: On Mon, Aug 02, 2010 at 11:33:33AM -0500, Anthony Liguori wrote: On 08/02/2010 11:11 AM, Kevin O'Connor wrote: Load the bootsplash.jpg file into fw_cfg if it is found in the roms directory. Sorry, I should have provided

[Qemu-devel] Anyone seeing huge slowdown launching qemu with Linux 2.6.35?

2010-08-03 Thread Richard W.M. Jones
qemu compiled from today's git. Using the following command line: $qemudir/x86_64-softmmu/qemu-system-x86_64 -L $qemudir/pc-bios \ -drive file=/dev/null,if=virtio \ -enable-kvm \ -nodefaults \ -nographic \ -serial stdio \ -m 500 \ -no-reboot \ -no-hpet \ -net

Re: [Qemu-devel] Anyone seeing huge slowdown launching qemu with Linux 2.6.35?

2010-08-03 Thread Richard W.M. Jones
On Tue, Aug 03, 2010 at 02:33:02PM +0300, Gleb Natapov wrote: On Tue, Aug 03, 2010 at 12:13:06PM +0100, Richard W.M. Jones wrote: qemu compiled from today's git. Using the following command line: $qemudir/x86_64-softmmu/qemu-system-x86_64 -L $qemudir/pc-bios \ -drive file=/dev

Re: [Qemu-devel] Anyone seeing huge slowdown launching qemu with Linux 2.6.35?

2010-08-03 Thread Richard W.M. Jones
On Tue, Aug 03, 2010 at 03:37:14PM +0300, Gleb Natapov wrote: On Tue, Aug 03, 2010 at 01:10:00PM +0100, Richard W.M. Jones wrote: I can't see anything about this in the kernel changelog. Can you point me to the commit or the key phrase to look for? 7972995b0c346de76 Thanks - I see

Re: [Qemu-devel] [PATCHv3] Load bootsplash.jpg if present

2010-08-03 Thread Richard W.M. Jones
On Tue, Aug 03, 2010 at 07:51:02AM -0500, Anthony Liguori wrote: If menu=off, it probably just shouldn't display. I assume libguestfs is passing menu=off... We don't actually, but it is a tremendously good idea so I'll add it, thanks :-) Rich. -- Richard Jones, Virtualization Group, Red Hat

Re: [Qemu-devel] [PATCHv3] Load bootsplash.jpg if present

2010-08-03 Thread Richard W.M. Jones
On Tue, Aug 03, 2010 at 01:54:12PM +0100, Richard W.M. Jones wrote: On Tue, Aug 03, 2010 at 07:51:02AM -0500, Anthony Liguori wrote: If menu=off, it probably just shouldn't display. I assume libguestfs is passing menu=off... We don't actually, but it is a tremendously good idea so I'll

Re: [Qemu-devel] Anyone seeing huge slowdown launching qemu with Linux 2.6.35?

2010-08-03 Thread Richard W.M. Jones
On Tue, Aug 03, 2010 at 04:19:39PM +0300, Avi Kivity wrote: On 08/03/2010 03:48 PM, Richard W.M. Jones wrote: Thanks for the explanation. I'll repost my DMA-like fw-cfg patch once I've rebased it and done some more testing. This huge regression for a common operation (implementing -initrd

Re: [Qemu-devel] Anyone seeing huge slowdown launching qemu with Linux 2.6.35?

2010-08-03 Thread Richard W.M. Jones
On Tue, Aug 03, 2010 at 05:38:25PM +0300, Avi Kivity wrote: The time will only continue to grow as you add features and as the distro bloats naturally. Much better to create it once and only update it if some dependent file changes (basically the current on-the-fly code + save a list of

[Qemu-devel] [PATCH REPOST] fw_cfg: Allow guest to read kernel etc via fast, synchronous DMA-type operation.

2010-08-03 Thread Richard W.M. Jones
-by: Richard W.M. Jones rjo...@redhat.com --- hw/fw_cfg.c | 37 ++- hw/fw_cfg.h | 20 -- pc-bios/optionrom/linuxboot.S | 11 +- pc-bios/optionrom/optionrom.h | 43

Re: [Qemu-devel] Anyone seeing huge slowdown launching qemu with Linux 2.6.35?

2010-08-03 Thread Richard W.M. Jones
On Tue, Aug 03, 2010 at 07:10:18PM +0300, Avi Kivity wrote: -kernel and -initrd is a developer's interface intended to make life easier for users that use qemu to develop kernels. It was not intended as a high performance DMA engine. Neither was the firmware _configuration_ interface. That

Re: [Qemu-devel] Anyone seeing huge slowdown launching qemu with Linux 2.6.35?

2010-08-03 Thread Richard W.M. Jones
On Tue, Aug 03, 2010 at 11:39:43AM -0500, Anthony Liguori wrote: Let's be fair. I think we've all agreed to adjust the fw_cfg interface to implement DMA. The only requirement was that the DMA operation not be triggered from a single port I/O but rather based on a polling operation which

Re: [Qemu-devel] Anyone seeing huge slowdown launching qemu with Linux 2.6.35?

2010-08-03 Thread Richard W.M. Jones
On Tue, Aug 03, 2010 at 07:48:17PM +0300, Avi Kivity wrote: On 08/03/2010 07:44 PM, Avi Kivity wrote: It's not a good path to follow. Tomorrow we'll need to load 300MB initrds and we'll have to rework this yet again. Meanwhile the kernel and virtio support demand loading of any image size

Re: [Qemu-devel] Anyone seeing huge slowdown launching qemu with Linux 2.6.35?

2010-08-03 Thread Richard W.M. Jones
On Tue, Aug 03, 2010 at 07:44:49PM +0300, Avi Kivity wrote: On 08/03/2010 07:28 PM, Richard W.M. Jones wrote: I have posted a small patch which makes this 650x faster without appreciable complication. It doesn't appear to support live migration, or hiding the feature for -M older. AFAICT

Re: [Qemu-devel] Anyone seeing huge slowdown launching qemu with Linux 2.6.35?

2010-08-03 Thread Richard W.M. Jones
On Tue, Aug 03, 2010 at 08:58:10PM +0300, Avi Kivity wrote: Richard, can you test kvm.git master? it already contains one fix and we plan to add more. Yup, I will ... Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming blog:

Re: [Qemu-devel] Anyone seeing huge slowdown launching qemu with Linux 2.6.35?

2010-08-03 Thread Richard W.M. Jones
On Tue, Aug 03, 2010 at 10:22:22PM +0300, Avi Kivity wrote: On 08/03/2010 10:13 PM, Richard W.M. Jones wrote: On Tue, Aug 03, 2010 at 09:43:39PM +0300, Avi Kivity wrote: libguestfs does not depend on an x86 architectural feature. qemu-system-x86_64 emulates a PC, and PCs don't have -kernel

Re: [Qemu-devel] Anyone seeing huge slowdown launching qemu with Linux 2.6.35?

2010-08-03 Thread Richard W.M. Jones
On Tue, Aug 03, 2010 at 10:24:41PM +0300, Avi Kivity wrote: Why do we need to transfer roms? These are devices on the memory bus or pci bus, it just needs to be there at the right address. Boot splash should just be another rom as it would be on a real system. Just like the initrd? Rich.

Re: [Qemu-devel] Anyone seeing huge slowdown launching qemu with Linux 2.6.35?

2010-08-04 Thread Richard W.M. Jones
On Wed, Aug 04, 2010 at 08:54:35AM +0300, Avi Kivity wrote: On 08/04/2010 01:06 AM, Richard W.M. Jones wrote: On Tue, Aug 03, 2010 at 10:24:41PM +0300, Avi Kivity wrote: Why do we need to transfer roms? These are devices on the memory bus or pci bus, it just needs to be there at the right

Re: [Qemu-devel] Anyone seeing huge slowdown launching qemu with Linux 2.6.35?

2010-08-04 Thread Richard W.M. Jones
On Wed, Aug 04, 2010 at 12:52:23PM +0300, Avi Kivity wrote: On 08/04/2010 12:24 PM, Richard W.M. Jones wrote: Just like the initrd? There isn't enough address space for a 100MB initrd in ROM. Because of limits of the original PC, sure, where you had to fit everything in 0xa-0xf

  1   2   3   4   5   6   7   8   9   10   >