Re: [Qemu-devel] [PATCH] We should check virtio_load return code

2012-03-01 Thread Amit Shah
On (Tue) 28 Feb 2012 [15:31:07], Orit Wasserman wrote: Otherwise we crash on error. A description on how to reproduce the crash would be nice. Signed-off-by: Orit Wasserman owass...@redhat.com Signed-off-by: Ulrich Obergfell uober...@redhat.com Acked-by: Amit Shah amit.s...@redhat.com

Re: [Qemu-devel] [PATCH v2] We should check the virtio_load return code

2012-03-01 Thread Amit Shah
... 3) migrate from 1 to 2. Thanks. Acked-by: Amit Shah amit.s...@redhat.com Amit

Re: [Qemu-devel] [PATCH] virtio-serial-bus: use correct lengths in control_out() message

2012-03-12 Thread Amit Shah
On (Sun) 11 Mar 2012 [17:52:59], Michael Tokarev wrote: In case of more than one control message, the code will use size of the largest message so far for all subsequent messages, instead of using size of current one. Fix it. Makes sense. How did you detect this? Any reproducible test-case?

Re: [Qemu-devel] [PATCH] virtio-serial-bus: use correct lengths in control_out() message

2012-03-12 Thread Amit Shah
On (Mon) 12 Mar 2012 [13:22:22], Michael Tokarev wrote: On 12.03.2012 12:59, Amit Shah wrote: On (Sun) 11 Mar 2012 [17:52:59], Michael Tokarev wrote: In case of more than one control message, the code will use size of the largest message so far for all subsequent messages, instead

Re: [Qemu-devel] [PATCH] virtio-serial-bus: use correct lengths in control_out() message

2012-03-13 Thread Amit Shah
On (Mon) 12 Mar 2012 [15:44:07], Michael Tokarev wrote: On 12.03.2012 15:06, Amit Shah wrote: On (Mon) 12 Mar 2012 [13:22:22], Michael Tokarev wrote: On 12.03.2012 12:59, Amit Shah wrote: On (Sun) 11 Mar 2012 [17:52:59], Michael Tokarev wrote: In case of more than one control message

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-14 Thread Amit Shah
On (Wed) 14 Mar 2012 [16:29:50], Wen Congyang wrote: At 03/13/2012 06:47 PM, Avi Kivity Wrote: On 03/13/2012 11:18 AM, Daniel P. Berrange wrote: On Mon, Mar 12, 2012 at 12:33:33PM +0200, Avi Kivity wrote: On 03/12/2012 11:04 AM, Wen Congyang wrote: Do you have any other comments about

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-14 Thread Amit Shah
On (Wed) 14 Mar 2012 [17:53:00], Wen Congyang wrote: At 03/14/2012 05:24 PM, Avi Kivity Wrote: On 03/14/2012 10:29 AM, Wen Congyang wrote: At 03/13/2012 06:47 PM, Avi Kivity Wrote: On 03/13/2012 11:18 AM, Daniel P. Berrange wrote: On Mon, Mar 12, 2012 at 12:33:33PM +0200, Avi Kivity

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-14 Thread Amit Shah
On (Wed) 14 Mar 2012 [18:04:40], Wen Congyang wrote: At 03/14/2012 05:51 PM, Amit Shah Wrote: On (Wed) 14 Mar 2012 [16:29:50], Wen Congyang wrote: At 03/13/2012 06:47 PM, Avi Kivity Wrote: On 03/13/2012 11:18 AM, Daniel P. Berrange wrote: On Mon, Mar 12, 2012 at 12:33:33PM +0200, Avi

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-14 Thread Amit Shah
On (Wed) 14 Mar 2012 [18:52:07], Wen Congyang wrote: At 03/14/2012 06:37 PM, Amit Shah Wrote: On (Wed) 14 Mar 2012 [17:53:00], Wen Congyang wrote: At 03/14/2012 05:24 PM, Avi Kivity Wrote: On 03/14/2012 10:29 AM, Wen Congyang wrote: At 03/13/2012 06:47 PM, Avi Kivity Wrote: On 03/13

[Qemu-devel] [PATCH 1/1] virtio-rng: device to send host entropy to guest

2012-05-16 Thread Amit Shah
virtio There are ways to extend the device to be more generic and collect entropy from other sources, but this is simple enough and works for now. Signed-off-by: Amit Shah amit.s...@redhat.com --- Makefile.objs |1 + hw/pci.h|1 + hw/virtio-pci.c | 50 + hw

Re: [Qemu-devel] [PATCH 1/1] virtio-rng: device to send host entropy to guest

2012-05-16 Thread Amit Shah
On (Wed) 16 May 2012 [13:38:06], Paolo Bonzini wrote: Il 16/05/2012 13:30, Amit Shah ha scritto: hw-obj-$(CONFIG_VIRTIO) += virtio-console.o +hw-obj-$(CONFIG_VIRTIO) += virtio-rng.o This needs to be conditional on CONFIG_LINUX too. Right. Thanks, Amit

Re: [Qemu-devel] [PATCH 1/1] virtio-rng: device to send host entropy to guest

2012-05-16 Thread Amit Shah
On (Wed) 16 May 2012 [14:45:34], Daniel P. Berrange wrote: On Wed, May 16, 2012 at 08:24:22AM -0500, Anthony Liguori wrote: On 05/16/2012 06:30 AM, Amit Shah wrote: The Linux kernel already has a virtio-rng driver, this is the device implementation. When Linux needs more entropy

Re: [Qemu-devel] [PATCH 1/1] virtio-rng: device to send host entropy to guest

2012-05-16 Thread Amit Shah
/16/2012 06:30 AM, Amit Shah wrote: The Linux kernel already has a virtio-rng driver, this is the device implementation. When Linux needs more entropy, it puts a buffer in the vq. We then put entropy into that buffer, and push it back to the guest. Feeding randomness from host's /dev/urandom

Re: [Qemu-devel] [PATCH 1/1] virtio-rng: device to send host entropy to guest

2012-05-16 Thread Amit Shah
On (Wed) 16 May 2012 [08:24:22], Anthony Liguori wrote: On 05/16/2012 06:30 AM, Amit Shah wrote: The Linux kernel already has a virtio-rng driver, this is the device implementation. When Linux needs more entropy, it puts a buffer in the vq. We then put entropy into that buffer, and push

Re: [Qemu-devel] [PATCH] Add event notification for guest balloon changes

2012-05-21 Thread Amit Shah
On (Thu) 17 May 2012 [08:49:44], Daniel P. Berrange wrote: On Wed, May 16, 2012 at 01:58:34PM -0500, Anthony Liguori wrote: On 05/16/2012 01:42 PM, Luiz Capitulino wrote: On Wed, 16 May 2012 11:10:47 +0100 Daniel P. Berrangeberra...@redhat.com wrote: From: Daniel P.

Re: [Qemu-devel] [PATCH] Add event notification for guest balloon changes

2012-05-21 Thread Amit Shah
On (Mon) 21 May 2012 [12:29:52], Daniel P. Berrange wrote: On Mon, May 21, 2012 at 04:44:38PM +0530, Amit Shah wrote: On (Thu) 17 May 2012 [08:49:44], Daniel P. Berrange wrote: On Wed, May 16, 2012 at 01:58:34PM -0500, Anthony Liguori wrote: On 05/16/2012 01:42 PM, Luiz Capitulino wrote

Re: [Qemu-devel] [PATCH 1/1] virtio-rng: device to send host entropy to guest

2012-05-21 Thread Amit Shah
On (Wed) 16 May 2012 [08:48:20], Anthony Liguori wrote: On 05/16/2012 08:45 AM, Daniel P. Berrange wrote: On Wed, May 16, 2012 at 08:24:22AM -0500, Anthony Liguori wrote: On 05/16/2012 06:30 AM, Amit Shah wrote: The Linux kernel already has a virtio-rng driver, this is the device

Re: [Qemu-devel] [PATCH 1/1] virtio-rng: device to send host entropy to guest

2012-05-21 Thread Amit Shah
On (Wed) 16 May 2012 [13:24:10], Anthony Liguori wrote: On 05/16/2012 12:26 PM, Amit Shah wrote: On (Wed) 16 May 2012 [14:45:34], Daniel P. Berrange wrote: On Wed, May 16, 2012 at 08:24:22AM -0500, Anthony Liguori wrote: On 05/16/2012 06:30 AM, Amit Shah wrote: The Linux kernel already has

Re: [Qemu-devel] [PATCH 1/1] virtio-rng: device to send host entropy to guest

2012-05-21 Thread Amit Shah
On (Wed) 16 May 2012 [13:23:11], Anthony Liguori wrote: On 05/16/2012 12:21 PM, Amit Shah wrote: On (Wed) 16 May 2012 [08:24:22], Anthony Liguori wrote: On 05/16/2012 06:30 AM, Amit Shah wrote: The Linux kernel already has a virtio-rng driver, this is the device implementation. When Linux

Re: [Qemu-devel] [PATCH v2 2/3] Add event notification for guest balloon changes

2012-05-21 Thread Amit Shah
On (Mon) 21 May 2012 [17:59:52], Daniel P. Berrange wrote: From: Daniel P. Berrange berra...@redhat.com After setting a balloon target value, applications have to continually poll 'query-balloon' to determine whether the guest has reacted to this request. The virtio-balloon backend knows

Re: [Qemu-devel] [PATCH v2 2/3] Add event notification for guest balloon changes

2012-05-22 Thread Amit Shah
On (Mon) 21 May 2012 [20:50:55], Daniel P. Berrange wrote: On Tue, May 22, 2012 at 01:14:59AM +0530, Amit Shah wrote: On (Mon) 21 May 2012 [17:59:52], Daniel P. Berrange wrote: From: Daniel P. Berrange berra...@redhat.com After setting a balloon target value, applications have

Re: [Qemu-devel] [PATCH 1/1] virtio-rng: device to send host entropy to guest

2012-05-22 Thread Amit Shah
On (Mon) 21 May 2012 [15:34:59], Anthony Liguori wrote: On 05/21/2012 02:39 PM, Amit Shah wrote: On (Wed) 16 May 2012 [13:23:11], Anthony Liguori wrote: On 05/16/2012 12:21 PM, Amit Shah wrote: On (Wed) 16 May 2012 [08:24:22], Anthony Liguori wrote: On 05/16/2012 06:30 AM, Amit Shah wrote

[Qemu-devel] [PATCH v2 0/1] virtio-rng: hardware random number generator

2012-05-25 Thread Amit Shah
for notifying listeners about entropy needed and the bytes asked for by the guest. * Add s390 code Please review, and apply if appropriate. Amit Shah (1): virtio-rng: hardware random number generator device Makefile.objs|1 + hw/pci.h |1 + hw/s390-virtio-bus.c

[Qemu-devel] [PATCH v2 1/1] virtio-rng: hardware random number generator device

2012-05-25 Thread Amit Shah
asked by the guest: {timestamp: {seconds: 1337966878, microseconds: 517009}, \ event: ENTROPY_NEEDED, data: {bytes: 64}} Signed-off-by: Amit Shah amit.s...@redhat.com --- Makefile.objs|1 + hw/pci.h |1 + hw/s390-virtio-bus.c | 35 + hw/s390-virtio-bus.h

Re: [Qemu-devel] [PATCH v2 1/1] virtio-rng: hardware random number generator device

2012-05-25 Thread Amit Shah
On (Fri) 25 May 2012 [15:00:53], Anthony Liguori wrote: On 05/25/2012 02:32 PM, Amit Shah wrote: The Linux kernel already has a virtio-rng driver, this is the device implementation. When the guest asks for entropy from the virtio hwrng, it puts a buffer in the vq. We then put entropy

Re: [Qemu-devel] [PATCH v2 1/1] virtio-rng: hardware random number generator device

2012-05-28 Thread Amit Shah
On (Mon) 28 May 2012 [09:33:57], Daniel P. Berrange wrote: On Sat, May 26, 2012 at 01:02:49AM +0530, Amit Shah wrote: The Linux kernel already has a virtio-rng driver, this is the device implementation. When the guest asks for entropy from the virtio hwrng, it puts a buffer in the vq

[Qemu-devel] Call for Proposals: Linux Plumbers Conference (Aug 2012)

2012-04-19 Thread Amit Shah
Hello, The Call for Proposals for the Linux Plumbers Conf 2012 is out. We're looking for speakers to talk at the Virtualization microconference as well as the main conference. The deadline for proposal submissions is 1st May. This year's edition of LPC is co-located with LinuxCon NA and will

Re: [Qemu-devel] [PATCH 1/2] virtio: set guest_features before calling set_features callback, not after

2012-04-23 Thread Amit Shah
On (Mon) 23 Apr 2012 [14:26:23], Alon Levy wrote: We could drop the features parameter but that's a little more work and it's not really needed. Signed-off-by: Alon Levy al...@redhat.com Michael, can you ack this? I'm running these through the test suite and will send them both once

[Qemu-devel] [PATCH 1/1] virtio-serial-bus: fix guest_connected init before driver init

2012-04-24 Thread Amit Shah
with VIRTIO_CONFIG_S_DRIVER_OK by even older non multiport drivers. [Amit: Add comment, tweak summary] Signed-off-by: Alon Levy al...@redhat.com Acked-by: Michael S. Tsirkin m...@redhat.com Signed-off-by: Amit Shah amit.s...@redhat.com --- hw/virtio-serial-bus.c | 29

[Qemu-devel] [PULL] virtio-serial: fix probing for features before driver init

2012-04-24 Thread Amit Shah
Hi, Please pull for a virtio-serial fix. The following changes since commit 3c30dd5a68e9fee6af67cfd0d14ed7520820f36a: target-arm: Move reset handling to arm_cpu_reset (2012-04-21 18:13:22 +) are available in the git repository at:

Re: [Qemu-devel] [PATCH 1/1] virtio-serial-bus: fix guest_connected init before driver init

2012-04-24 Thread Amit Shah
On (Tue) 24 Apr 2012 [13:17:16], Michael S. Tsirkin wrote: On Tue, Apr 24, 2012 at 02:55:26PM +0530, Amit Shah wrote: From: Alon Levy al...@redhat.com guest_connected should be false before guest driver initialization, and true after, both for multiport aware and non multiport aware

Re: [Qemu-devel] [PATCH 1/1] virtio-serial-bus: fix guest_connected init before driver init

2012-04-24 Thread Amit Shah
On (Tue) 24 Apr 2012 [13:49:53], Michael S. Tsirkin wrote: On Tue, Apr 24, 2012 at 03:56:44PM +0530, Amit Shah wrote: On (Tue) 24 Apr 2012 [13:17:16], Michael S. Tsirkin wrote: On Tue, Apr 24, 2012 at 02:55:26PM +0530, Amit Shah wrote: From: Alon Levy al...@redhat.com

Re: [Qemu-devel] [PATCH 1/1] virtio-serial-bus: fix guest_connected init before driver init

2012-04-24 Thread Amit Shah
On (Tue) 24 Apr 2012 [14:21:36], Michael S. Tsirkin wrote: On Tue, Apr 24, 2012 at 04:44:10PM +0530, Amit Shah wrote: On (Tue) 24 Apr 2012 [13:49:53], Michael S. Tsirkin wrote: On Tue, Apr 24, 2012 at 03:56:44PM +0530, Amit Shah wrote: On (Tue) 24 Apr 2012 [13:17:16], Michael S. Tsirkin

[Qemu-devel] [RFC PATCH 1/1] virtio-serial-bus: Unset guest_connected at reset and driver reset

2012-04-24 Thread Amit Shah
When a guest driver resets the virtio status to not ready, or when qemu is reset, reset the guest_connected bit and let ports know if they have the guest_close() callback registered. Signed-off-by: Amit Shah amit.s...@redhat.com --- hw/virtio-serial-bus.c | 28 1

[Qemu-devel] [PATCH 2/2] virtio-serial-bus: Unset guest_connected at reset and driver reset

2012-04-26 Thread Amit Shah
When a guest driver resets the virtio status to not ready, or when qemu is reset, reset all ports' guest_connected bit and let port users know of this event if they have the guest_close() callback registered. Reviewed-by: Alon Levy al...@redhat.com Signed-off-by: Amit Shah amit.s...@redhat.com

[Qemu-devel] [PULL] virtio-serial: guest_connected transition fixes

2012-04-26 Thread Amit Shah
guest_connected init before driver init Amit Shah (1): virtio-serial-bus: Unset guest_connected at reset and driver reset hw/virtio-serial-bus.c | 57 +-- 1 files changed, 49 insertions(+), 8 deletions(-) -- 1.7.7.6

[Qemu-devel] [PATCH 1/2] virtio-serial-bus: fix guest_connected init before driver init

2012-04-26 Thread Amit Shah
with VIRTIO_CONFIG_S_DRIVER_OK by even older non multiport drivers. [Amit: Add comment, tweak summary, only set guest_connected and not reset it as a side-effect.] Signed-off-by: Alon Levy al...@redhat.com Acked-by: Michael S. Tsirkin m...@redhat.com Signed-off-by: Amit Shah amit.s

[Qemu-devel] [PATCH] virtio-serial-bus: Unset guest_connected at reset and driver reset

2012-04-27 Thread Amit Shah
When a guest driver resets the virtio status to not ready, or when qemu is reset, reset all ports' guest_connected bit and let port users know of this event if they have the guest_close() callback registered. Reviewed-by: Alon Levy al...@redhat.com Signed-off-by: Amit Shah amit.s...@redhat.com

Re: [Qemu-devel] [RFC PATCH 0/6] virtio-trace: Support virtio-trace

2012-08-20 Thread Amit Shah
On (Tue) 21 Aug 2012 [11:47:16], Rusty Russell wrote: On Thu, 9 Aug 2012 15:46:20 +0530, Amit Shah amit.s...@redhat.com wrote: Hi, On (Tue) 24 Jul 2012 [11:36:57], Yoshihiro YUNOMAE wrote: Hi All, The following patch set provides a low-overhead system for collecting kernel

Re: [Qemu-devel] [PATCH V2 0/6] virtio-trace: Support virtio-trace

2012-08-20 Thread Amit Shah
On (Thu) 09 Aug 2012 [21:30:29], Yoshihiro YUNOMAE wrote: Hi All, The following patch set provides a low-overhead system for collecting kernel tracing data of guests by a host in a virtualization environment. ACK this series. I ran it through the virtio-serial test suite, and there's no

Re: [Qemu-devel] [PATCH 1/1] kvmclock: fix guest stop notification

2012-10-08 Thread Amit Shah
On (Sun) 30 Sep 2012 [20:05:16], Marcelo Tosatti wrote: On Thu, Sep 20, 2012 at 09:46:41AM -0300, Marcelo Tosatti wrote: On Thu, Sep 20, 2012 at 01:55:20PM +0530, Amit Shah wrote: Commit f349c12c0434e29c79ecde89029320c4002f7253 added the guest stop notification, but it did it in a way

Re: [Qemu-devel] [PATCH 1/1] kvmclock: fix guest stop notification

2012-10-08 Thread Amit Shah
On (Sun) 30 Sep 2012 [21:50:07], Amos Kong wrote: - Original Message - On Thu, Sep 20, 2012 at 09:46:41AM -0300, Marcelo Tosatti wrote: On Thu, Sep 20, 2012 at 01:55:20PM +0530, Amit Shah wrote: Commit f349c12c0434e29c79ecde89029320c4002f7253 added the guest stop

Re: [Qemu-devel] [PATCH 1/6] qdev: rework device properties.

2012-10-22 Thread Amit Shah
On (Wed) 17 Oct 2012 [17:14:14], Eduardo Habkost wrote: On Wed, Jul 15, 2009 at 01:43:31PM +0200, Gerd Hoffmann wrote: [...] diff --git a/hw/qdev-properties.c b/hw/qdev-properties.c new file mode 100644 index 000..8b0d0ff --- /dev/null +++ b/hw/qdev-properties.c @@ -0,0 +1,246 @@

Re: [Qemu-devel] [Bug 1066055] Re: Network performance regression with vde_switch

2012-10-22 Thread Amit Shah
out, and the result is: 0d8d7690850eb0cf2b2b60933cf47669a6b6f18f is the first bad commit commit 0d8d7690850eb0cf2b2b60933cf47669a6b6f18f Author: Amit Shah amit.s...@redhat.com Date: Tue Sep 25 00:05:15 2012 +0530 virtio: Introduce virtqueue_get_avail_bytes

Re: [Qemu-devel] [PATCH 1/1] rng backend: open backend in blocking mode

2013-04-03 Thread Amit Shah
On (Tue) 02 Apr 2013 [07:52:19], Anthony Liguori wrote: Amit Shah amit.s...@redhat.com writes: On (Mon) 01 Apr 2013 [09:02:46], Anthony Liguori wrote: Amit Shah amit.s...@redhat.com writes: Opening backends in non-blocking mode isn't necessary, we don't do anything while waiting

Re: [Qemu-devel] [PATCH v4 2/7] virtio-serial-pci: switch to the new API.

2013-04-03 Thread Amit Shah
On (Fri) 29 Mar 2013 [10:02:39], fred.kon...@greensocs.com wrote: From: KONRAD Frederic fred.kon...@greensocs.com Here the virtio-serial-pci is modified for the new API. The device virtio-serial-pci extends virtio-pci. It creates and connects a virtio-serial during the init. The properties

Re: [Qemu-devel] [PATCH v4 0/7] virtio-serial refactoring.

2013-04-03 Thread Amit Shah
On (Fri) 29 Mar 2013 [10:02:37], fred.kon...@greensocs.com wrote: From: KONRAD Frederic fred.kon...@greensocs.com This is the next part of virtio-refactoring. Basically it creates virtio-serial device which extends virtio-device. Then a virtio-serial can be connected on a virtio-bus.

Re: [Qemu-devel] [PATCH v2] qemu-char: eliminate busy waiting on can_read returning zero

2013-04-06 Thread Amit Shah
On (Fri) 05 Apr 2013 [17:59:33], Paolo Bonzini wrote: The character backend refactoring introduced an undesirable busy wait. The busy wait happens if can_read returns zero and there is data available on the character device's file descriptor. Then, the I/O watch will fire continuously and,

Re: [Qemu-devel] [PATCH 1/1] rng backend: open backend in blocking mode

2013-04-08 Thread Amit Shah
On (Wed) 03 Apr 2013 [15:08:40], Anthony Liguori wrote: Amit Shah amit.s...@redhat.com writes: On (Tue) 02 Apr 2013 [07:52:19], Anthony Liguori wrote: Amit Shah amit.s...@redhat.com writes: backends/rng-random.c:44:entropy_available: assertion failed: (len != -1) without

Re: [Qemu-devel] [PATCH v6 0/7] virtio-serial refactoring.

2013-04-10 Thread Amit Shah
are you ok with that? Thanks, this looks good. Acked-by: Amit Shah amit.s...@redhat.com Amit

Re: [Qemu-devel] [PATCH] qemu-char: another io_add_watch_poll fix

2013-04-10 Thread Amit Shah
On (Wed) 10 Apr 2013 [15:23:27], Paolo Bonzini wrote: After attaching the source, we have to remove the reference we hold to it, because we do not hold anymore a pointer to the source. If we do not do this, removing the source will not finalize it and will not drop the real I/O watch source.

Re: [Qemu-devel] [PATCH] qemu-char: another io_add_watch_poll fix

2013-04-12 Thread Amit Shah
On (Thu) 11 Apr 2013 [10:58:30], Paolo Bonzini wrote: Il 10/04/2013 19:59, Amit Shah ha scritto: On (Wed) 10 Apr 2013 [15:23:27], Paolo Bonzini wrote: After attaching the source, we have to remove the reference we hold to it, because we do not hold anymore a pointer to the source. If we

Re: [Qemu-devel] [PATCH v3 0/8] virtio-rng refactoring.

2013-04-16 Thread Amit Shah
made basic tests (with linux guests) on: * qemu-system-i386 Looks OK. Acked-by: Amit Shah amit.s...@redhat.com Thanks, Amit

[Qemu-devel] [PATCH 1/1] rng random backend: check for -EAGAIN errors on read

2013-04-16 Thread Amit Shah
. Reported-by: yunpingzheng yunzh...@redhat.com Signed-off-by: Amit Shah amit.s...@redhat.com --- backends/rng-random.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/backends/rng-random.c b/backends/rng-random.c index d5761f2..830360c 100644 --- a/backends/rng-random.c +++ b/backends/rng

Re: [Qemu-devel] [PATCH 1/1] rng random backend: check for -EAGAIN errors on read

2013-04-16 Thread Amit Shah
On (Tue) 16 Apr 2013 [15:51:30], Andreas Färber wrote: Am 16.04.2013 12:28, schrieb Amit Shah: Not handling EAGAIN triggers the assert qemu/backends/rng-random.c:44:entropy_available: assertion failed: (len != -1) Aborted (core dumped) This happens when starting a guest

[Qemu-devel] [PATCH 0/8] virtio-rng: hardware random number generator device

2012-10-26 Thread Amit Shah
- Bugfix: rng_backend_request_entropy() in chr_read() should pass len, not size - Do not request for more entropy in chr_read() once the current request is served; there are other ways this notification will go out. Amit Shah (1): virtio-rng: hardware random number generator device

[Qemu-devel] [PATCH 1/8] object: add object_property_add_bool (v2)

2012-10-26 Thread Amit Shah
From: Anthony Liguori aligu...@us.ibm.com Signed-off-by: Anthony Liguori aligu...@us.ibm.com --- include/qemu/object.h | 16 ++ qom/object.c | 56 + 2 files changed, 72 insertions(+), 0 deletions(-) diff --git

[Qemu-devel] [PATCH 2/8] qdev: add realized property and make adding child bus implied by realize

2012-10-26 Thread Amit Shah
From: Anthony Liguori aligu...@us.ibm.com Signed-off-by: Anthony Liguori aligu...@us.ibm.com --- hw/qdev.c | 36 +++- 1 files changed, 35 insertions(+), 1 deletions(-) diff --git a/hw/qdev.c b/hw/qdev.c index 9b9aba3..5e34c22 100644 --- a/hw/qdev.c +++

[Qemu-devel] [PATCH 3/8] vl: add -object option to create QOM objects from the command line

2012-10-26 Thread Amit Shah
From: Anthony Liguori aligu...@us.ibm.com This will create a new QOM object in the '/objects' path. Note that properties are set in order which allows for simple objects to be initialized entirely with this option and then realized. This option is roughly equivalent to -device but for things

[Qemu-devel] [PATCH 7/8] rng-egd: introduce EGD compliant RNG backend

2012-10-26 Thread Amit Shah
From: Anthony Liguori aligu...@us.ibm.com This backend talks EGD to a CharDriverState. A typical way to invoke this would be: qemu -chardev socket,host=localhost,port=1024,id=chr0 \ -object rng-egd,chardev=chr0,id=egd0 \ -device virtio-rng-pci,rng=egd0 Signed-off-by: Anthony Liguori

[Qemu-devel] [PATCH 4/8] vl: add -late-object to create QOM objects after machine init

2012-10-26 Thread Amit Shah
From: Anthony Liguori aligu...@us.ibm.com In order to create qdev objects via -late-object, we almost always have to specify the parent_bus which is usually created during machine init. Until we properly support two stage init, introduce a -late-object option that allows for creation of objects

[Qemu-devel] [PATCH 6/8] rng-urandom: add an RNG backend that uses /dev/urandom

2012-10-26 Thread Amit Shah
From: Anthony Liguori aligu...@us.ibm.com The filename can be overridden but it expects a non-blocking source of entropy. A typical invocation would be: qemu -object rng-urandom,id=rng0 -device virtio-rng-pci,rng=rng0 Signed-off-by: Anthony Liguori aligu...@us.ibm.com ---

[Qemu-devel] [PATCH 8/8] virtio-rng: hardware random number generator device

2012-10-26 Thread Amit Shah
The Linux kernel already has a virtio-rng driver, this is the device implementation. When the guest asks for entropy from the virtio hwrng, it puts a buffer in the vq. We then put entropy into that buffer, and push it back to the guest. Signed-off-by: Amit Shah amit.s...@redhat.com Signed-off

[Qemu-devel] [PATCH 5/8] rng: add RndBackend abstract object class

2012-10-26 Thread Amit Shah
From: Anthony Liguori aligu...@us.ibm.com This is the backend used by devices that need to request entropy. Signed-off-by: Anthony Liguori aligu...@us.ibm.com --- Makefile.objs |2 + backends/Makefile.objs |1 + backends/rng.c | 93

Re: [Qemu-devel] [PATCH 0/6] add paravirtualization hwrng support

2012-10-29 Thread Amit Shah
On (Fri) 26 Oct 2012 [13:24:06], Anthony Liguori wrote: H. Peter Anvin h...@zytor.com writes: On 10/26/2012 08:42 AM, Anthony Liguori wrote: Is /dev/random even appropriate to feed rngd? rngd needs _a lot_ of entropy to even start working. Its randomness test works in groups of

Re: [Qemu-devel] [PATCH 0/6] add paravirtualization hwrng support

2012-10-29 Thread Amit Shah
On (Fri) 26 Oct 2012 [09:43:34], Anthony Liguori wrote: Hi, This series implements the backend and frontend infrastructure for virtio-rng. This is similar to previous series sent out by both Amit and myself although it has been trimmed down considerably. In terms of backends, a file and

Re: [Qemu-devel] [Bug 1066055] Re: Network performance regression with vde_switch

2012-11-01 Thread Amit Shah
On (Tue) 23 Oct 2012 [14:55:03], Stefan Hajnoczi wrote: On Mon, Oct 22, 2012 at 06:50:00AM -0700, Edivaldo de Araujo Pereira wrote: I didn't take enough time to uderstand the code, so unfortunately I fear there is not much I could do to solve the problem, apart from trying your

Re: [Qemu-devel] [Bug 1066055] Re: Network performance regression with vde_switch

2012-11-01 Thread Amit Shah
On (Thu) 01 Nov 2012 [14:04:11], Michael S. Tsirkin wrote: On Thu, Nov 01, 2012 at 02:49:18PM +0530, Amit Shah wrote: On (Tue) 23 Oct 2012 [14:55:03], Stefan Hajnoczi wrote: On Mon, Oct 22, 2012 at 06:50:00AM -0700, Edivaldo de Araujo Pereira wrote: I didn't take enough time

Re: [Qemu-devel] [PATCH] virtio: limit avail bytes lookahead

2012-11-02 Thread Amit Shah
limiting lookahead, and use in virtqueue_avail_bytes. Signed-off-by: Michael S. Tsirkin m...@redhat.com Reported-by: Edivaldo de Araujo Pereira edivaldoapere...@yahoo.com.br Acked-by: Amit Shah amit.s...@redhat.com Amit

[Qemu-devel] [PATCH 3/3] virtio-rng: fix typos, comments

2012-11-09 Thread Amit Shah
Fix typos and update comments to match current implementation. Signed-off-by: Amit Shah amit.s...@redhat.com --- hw/virtio-rng.c|1 - include/qemu/rng.h |6 +++--- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/hw/virtio-rng.c b/hw/virtio-rng.c index bb19cfa..290b2b6

[Qemu-devel] [PATCH 0/3] Fixes for virtio-rng

2012-11-09 Thread Amit Shah
Hey Anthony, Here are the fixes for the virtio-rng code -- ideally these should be folded in the series before you merge, but if the original series is ready to apply, these can go in as follow-on patches. Amit Shah (3): virtio-rng: use virtqueue_get_avail_bytes, fix migration virtio-rng

[Qemu-devel] [PATCH 1/3] virtio-rng: use virtqueue_get_avail_bytes, fix migration

2012-11-09 Thread Amit Shah
Popping an elem from the vq just to find out its length causes problems with save/load later on. Use the new virtqueue_get_avail_bytes() function instead, saves us the complexity in the migration code. Signed-off-by: Amit Shah amit.s...@redhat.com --- hw/virtio-rng.c | 69

[Qemu-devel] [PATCH 2/3] virtio-rng: remove extra request for entropy

2012-11-09 Thread Amit Shah
If we just got a few bytes from the backends, but the guest is ready to accept more, don't poke the backends once more for more bytes; it already knows (from the request in handle_input()). Signed-off-by: Amit Shah amit.s...@redhat.com --- hw/virtio-rng.c | 10 -- 1 files changed, 0

Re: [Qemu-devel] [PATCH] hw/virtio-serial-bus: post_load send_event when vm is running

2012-11-16 Thread Amit Shah
On (Wed) 14 Nov 2012 [15:09:07], Alon Levy wrote: Add a new timer based on vm_clock for 1 ns in the future from post_load to do the event send in case host_connected differs between migration source and target. RHBZ: 867366 Signed-off-by: Alon Levy al...@redhat.com ---

[Qemu-devel] [PATCH 0/4] [For 0.13] virtio-rng: Fixes

2012-11-20 Thread Amit Shah
as well, but that's for later. Please review and apply. Amit Shah (4): virtio-rng: use virtqueue_get_avail_bytes, fix migration virtio-rng: remove extra request for entropy virtio-rng: disable timer on device removal virtio-rng: fix typos, comments hw/virtio-rng.c| 92

[Qemu-devel] [PATCH 2/4] virtio-rng: remove extra request for entropy

2012-11-20 Thread Amit Shah
If we got fewer bytes from the backend than requested, don't poke the backend for more bytes; the guest will ask for more (or if the guest has already asked for more, the backend knows about it via handle_input()). Signed-off-by: Amit Shah amit.s...@redhat.com --- hw/virtio-rng.c | 7 --- 1

[Qemu-devel] [PATCH 3/4] virtio-rng: disable timer on device removal

2012-11-20 Thread Amit Shah
Disable the rate-limit timer on device remove (e.g. hot-unplug). Signed-off-by: Amit Shah amit.s...@redhat.com --- hw/virtio-rng.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/virtio-rng.c b/hw/virtio-rng.c index cf5a8ff..c8a6da7 100644 --- a/hw/virtio-rng.c +++ b/hw/virtio-rng.c

[Qemu-devel] [PATCH 1/4] virtio-rng: use virtqueue_get_avail_bytes, fix migration

2012-11-20 Thread Amit Shah
Popping an elem from the vq just to find out its length causes problems with save/load later on. Use the new virtqueue_get_avail_bytes() function instead, saves us the complexity in the migration code, as well as makes the migration endian-safe. Signed-off-by: Amit Shah amit.s...@redhat.com

[Qemu-devel] [PATCH 4/4] virtio-rng: fix typos, comments

2012-11-20 Thread Amit Shah
Fix typos, whitespace and update comments to match current implementation. Signed-off-by: Amit Shah amit.s...@redhat.com --- hw/virtio-rng.c| 7 +++ include/qemu/rng.h | 6 +++--- 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/hw/virtio-rng.c b/hw/virtio-rng.c index

[Qemu-devel] [PATCH v2 0/4] [For 1.3] virtio-rng: fixes

2012-11-20 Thread Amit Shah
as well, but that's for later. v2: * fix typo in patch 4 * travel back to present, fix cover letter subject line * add Anthony's reviewed-by tags to patches. Please apply. Amit Shah (4): virtio-rng: use virtqueue_get_avail_bytes, fix migration virtio-rng: remove extra request for entropy

[Qemu-devel] [PATCH v2 1/4] virtio-rng: use virtqueue_get_avail_bytes, fix migration

2012-11-20 Thread Amit Shah
Popping an elem from the vq just to find out its length causes problems with save/load later on. Use the new virtqueue_get_avail_bytes() function instead, saves us the complexity in the migration code, as well as makes the migration endian-safe. Signed-off-by: Amit Shah amit.s...@redhat.com

Re: [Qemu-devel] [PATCH 4/4] virtio-rng: fix typos, comments

2012-11-20 Thread Amit Shah
On (Tue) 20 Nov 2012 [19:43:16], Anthony Liguori wrote: * The backend does not need to pass the full amount of data to @receive_entropy - * but will pass at a value greater than 0. + * but will pass a a value greater than 0. Your typo fix has a typo. Woops, series re-sent.

Re: [Qemu-devel] [PATCH 1/4] virtio-rng: use virtqueue_get_avail_bytes, fix migration

2012-11-20 Thread Amit Shah
On (Tue) 20 Nov 2012 [19:44:55], Anthony Liguori wrote: Amit Shah amit.s...@redhat.com writes: Popping an elem from the vq just to find out its length causes problems with save/load later on. Use the new virtqueue_get_avail_bytes() function instead, saves us the complexity

[Qemu-devel] [PATCH v2 2/4] virtio-rng: remove extra request for entropy

2012-11-20 Thread Amit Shah
If we got fewer bytes from the backend than requested, don't poke the backend for more bytes; the guest will ask for more (or if the guest has already asked for more, the backend knows about it via handle_input()). Signed-off-by: Amit Shah amit.s...@redhat.com Reviewed-by: Anthony Liguori aligu

[Qemu-devel] [PATCH v2 3/4] virtio-rng: disable timer on device removal

2012-11-20 Thread Amit Shah
Disable the rate-limit timer on device remove (e.g. hot-unplug). Signed-off-by: Amit Shah amit.s...@redhat.com Reviewed-by: Anthony Liguori aligu...@us.ibm.com --- hw/virtio-rng.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/virtio-rng.c b/hw/virtio-rng.c index cf5a8ff..c8a6da7

[Qemu-devel] [PATCH v2 4/4] virtio-rng: fix typos, comments

2012-11-20 Thread Amit Shah
Fix typos, whitespace and update comments to match current implementation. Signed-off-by: Amit Shah amit.s...@redhat.com --- hw/virtio-rng.c| 7 +++ include/qemu/rng.h | 6 +++--- 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/hw/virtio-rng.c b/hw/virtio-rng.c index

Re: [Qemu-devel] Char flow control patches

2013-02-10 Thread Amit Shah
On (Wed) 30 Jan 2013 [08:38:50], Anthony Liguori wrote: Amit Shah amit.s...@redhat.com writes: Hi Anthony, I did some basic testing of the char flow control patches from your char-flow.2 branch. With the following patch applied, things seem to be working fine. I tested the isa

Re: [Qemu-devel] Char flow control patches

2013-02-11 Thread Amit Shah
On (Mon) 11 Feb 2013 [12:37:34], Amit Shah wrote: On (Wed) 30 Jan 2013 [08:38:50], Anthony Liguori wrote: Amit Shah amit.s...@redhat.com writes: Hi Anthony, I did some basic testing of the char flow control patches from your char-flow.2 branch. With the following patch applied

Re: [Qemu-devel] [PATCH 00/19] chardev flow control

2013-02-18 Thread Amit Shah
. This is based on a rebased version of an old series of mine plus a number of improvements from Amit Shah. This series works fine, resolves the client sockets issue. Acked-by: Amit Shah amit.s...@redhat.com Amit

Re: [Qemu-devel] [PATCH 10/19] char: add gio watch fn for tcp backends

2013-02-19 Thread Amit Shah
On (Mon) 18 Feb 2013 [15:48:07], Anthony Liguori wrote: From: Amit Shah amit.s...@redhat.com Signed-off-by: Amit Shah amit.s...@redhat.com --- qemu-char.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/qemu-char.c b/qemu-char.c index e59e580..160bcea 100644 --- a/qemu

Re: [Qemu-devel] [PATCH 10/19] char: add gio watch fn for tcp backends

2013-02-19 Thread Amit Shah
On (Tue) 19 Feb 2013 [07:55:36], Anthony Liguori wrote: Amit Shah amit.s...@redhat.com writes: On (Mon) 18 Feb 2013 [15:48:07], Anthony Liguori wrote: From: Amit Shah amit.s...@redhat.com Signed-off-by: Amit Shah amit.s...@redhat.com --- qemu-char.c | 7 +++ 1 file changed

Re: [Qemu-devel] [PATCH] spice-qemu-char: register interface on post load

2012-12-23 Thread Amit Shah
On (Sun) 23 Dec 2012 [23:35:29], Alon Levy wrote: The target has not seen the guest_connected event via spice_chr_guest_open or spice_chr_write, and so spice server wrongly assumes there is no agent active, while the client continues to send motion events only by the agent channel, which the

Re: [Qemu-devel] [RFC 3/3] virtio-balloon: add auto-ballooning support

2013-01-11 Thread Amit Shah
Hi Luiz, On (Tue) 18 Dec 2012 [18:16:55], Luiz Capitulino wrote: The auto-ballooning feature automatically performs balloon inflate or deflate based on host and guest memory pressure. This can help to avoid swapping or worse in both, host and guest. Auto-ballooning has a host and a guest

Re: [Qemu-devel] [RFC 3/3] virtio-balloon: add auto-ballooning support

2013-01-25 Thread Amit Shah
On (Mon) 14 Jan 2013 [09:58:30], Luiz Capitulino wrote: On Sat, 12 Jan 2013 02:02:32 +0530 Amit Shah amit.s...@redhat.com wrote: Hi Luiz, On (Tue) 18 Dec 2012 [18:16:55], Luiz Capitulino wrote: The auto-ballooning feature automatically performs balloon inflate or deflate based

[Qemu-devel] Char flow control patches

2013-01-30 Thread Amit Shah
Hi Anthony, I did some basic testing of the char flow control patches from your char-flow.2 branch. With the following patch applied, things seem to be working fine. I tested the isa-serial and virtio-serial devices. I haven't yet tested with virtio-serial flow control, but at least no

Re: [Qemu-devel] Char flow control patches

2013-02-01 Thread Amit Shah
On (Wed) 30 Jan 2013 [08:38:50], Anthony Liguori wrote: Amit Shah amit.s...@redhat.com writes: Hi Anthony, I did some basic testing of the char flow control patches from your char-flow.2 branch. With the following patch applied, things seem to be working fine. I tested the isa

[Qemu-devel] [PATCH 0/4] char: flow control: fixes, virtio-serial flow control

2013-02-04 Thread Amit Shah
. Patch 4 adds flow control to virtio-serial ports. With these applied, I tested flow control with virtio-serial ports with a unix socket backend, and things are working fine. Amit Shah (4): char: poll a frontend only if it's writable char: fix return value of can_read functions on no input char

[Qemu-devel] [PATCH 1/4] char: poll a frontend only if it's writable

2013-02-04 Thread Amit Shah
If the frontend reports it's not writable, don't start polling it for data yet. Signed-off-by: Amit Shah amit.s...@redhat.com --- qemu-char.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qemu-char.c b/qemu-char.c index 2b714cf..5731d02 100644 --- a/qemu-char.c +++ b/qemu

[Qemu-devel] [PATCH 2/4] char: fix return value of can_read functions on no input

2013-02-04 Thread Amit Shah
When the can_read functions report the frontend isn't ready to accept data yet, return 'FALSE' instead of 'TRUE'. Signed-off-by: Amit Shah amit.s...@redhat.com --- qemu-char.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/qemu-char.c b/qemu-char.c index 5731d02

[Qemu-devel] [PATCH 3/4] char: add gio watch fn for tcp backends

2013-02-04 Thread Amit Shah
Signed-off-by: Amit Shah amit.s...@redhat.com --- qemu-char.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/qemu-char.c b/qemu-char.c index 7fa9372..01b0190 100644 --- a/qemu-char.c +++ b/qemu-char.c @@ -2484,6 +2484,12 @@ static ssize_t tcp_chr_recv(CharDriverState *chr, char *buf

<    5   6   7   8   9   10   11   12   13   14   >