Michael S. Tsirkin m...@redhat.com wrote on 05/24/2010 03:17:10 AM:
On Thu, May 20, 2010 at 09:58:06AM -0700, David L Stevens wrote:
[for Michael Tsirkin's vhost development git tree]
This patch fixes a race between guest and host when
adding used buffers wraps the ring. Without it,
netdev-ow...@vger.kernel.org wrote on 05/24/2010 09:13:51 AM:
On Mon, May 24, 2010 at 08:52:40AM -0700, David Stevens wrote:
Michael S. Tsirkin m...@redhat.com wrote on 05/24/2010 03:17:10 AM:
On Thu, May 20, 2010 at 09:58:06AM -0700, David L Stevens wrote:
[for Michael Tsirkin's
Michael S. Tsirkin m...@redhat.com wrote on 05/24/2010 09:42:05 AM:
I see. The logging is still bugg though I think.
Possibly; migration isn't working for me under load even
without mergeable buffers (investigating), so I haven't
yet been able to test wrap w/ logging, but did you see
something
Since datalen carries the difference and will be negative by that amount
from the original loop, what about just adding something like:
}
if (headcount)
heads[headcount-1].len += datalen;
[and really, headcount 0 since datalen 0, so just:
netdev-ow...@vger.kernel.org wrote on 05/10/2010 10:25:57 AM:
On Mon, May 10, 2010 at 10:09:03AM -0700, David Stevens wrote:
Since datalen carries the difference and will be negative by that
amount
from the original loop, what about just adding something like
Michael S. Tsirkin m...@redhat.com wrote on 05/03/2010 03:34:11 AM:
On Wed, Apr 28, 2010 at 01:57:12PM -0700, David L Stevens wrote:
This patch adds mergeable receive buffer support to vhost_net.
Signed-off-by: David L Stevens dlstev...@us.ibm.com
I've been doing some more testing
Michael S. Tsirkin m...@redhat.com wrote on 05/03/2010 08:56:14 AM:
On Mon, May 03, 2010 at 08:39:08AM -0700, David Stevens wrote:
Michael S. Tsirkin m...@redhat.com wrote on 05/03/2010 03:34:11 AM:
On Wed, Apr 28, 2010 at 01:57:12PM -0700, David L Stevens wrote:
This patch adds
Michael S. Tsirkin m...@redhat.com wrote on 04/29/2010 06:45:15 AM:
On Wed, Apr 28, 2010 at 01:57:12PM -0700, David L Stevens wrote:
This patch adds mergeable receive buffer support to vhost_net.
Signed-off-by: David L Stevens dlstev...@us.ibm.com
I have applied this, thanks very much!
Michael S. Tsirkin m...@redhat.com wrote on 04/22/2010 05:02:25 AM:
On Mon, Apr 19, 2010 at 03:12:19PM -0700, David L Stevens wrote:
This patch adds the mergeable RX buffers feature to vhost.
Signed-off-by: David L Stevens dlstev...@us.ibm.com
Looks pretty clean to me.
Could you send
Michael S. Tsirkin m...@redhat.com wrote on 04/22/2010 10:43:49 AM:
On Mon, Apr 19, 2010 at 03:12:19PM -0700, David L Stevens wrote:
This patch adds the mergeable RX buffers feature to vhost.
Signed-off-by: David L Stevens dlstev...@us.ibm.com
BTW, which userspace should I use for
Thanks!
There's some whitespace damage, are you sending with your new
sendmail setup? It seems to have worked for qemu patches ...
Yes, I saw some line wraps in what I received, but I checked
the original draft to be sure and they weren't there. Possibly from
the relay; Sigh.
@@
kvm-ow...@vger.kernel.org wrote on 04/07/2010 11:09:30 AM:
On Wed, Apr 07, 2010 at 10:37:17AM -0700, David Stevens wrote:
Thanks!
There's some whitespace damage, are you sending with your new
sendmail setup? It seems to have worked for qemu patches ...
Yes, I saw some
kvm-ow...@vger.kernel.org wrote on 04/01/2010 03:54:15 AM:
On Wed, Mar 31, 2010 at 03:04:43PM -0700, David Stevens wrote:
+ head.iov_base = (void
*)vhost_get_vq_desc(net-dev,
vq,
+ vq-iov, ARRAY_SIZE(vq-iov), out, in,
NULL,
NULL
Michael S. Tsirkin m...@redhat.com wrote on 03/31/2010 05:02:28 AM:
attached patch seems to be whiespace damaged as well.
Does the origin pass checkpatch.pl for you?
Yes, but I probably broke it in the transfer -- will be more
careful with the next revision.
+
This patch adds support for the Mergeable Receive Buffers feature to
vhost_net.
Changes:
1) generalize descriptor allocation functions to allow multiple
descriptors per packet
2) add socket peek to know datalen at buffer allocation time
3) change
Michael S. Tsirkin m...@redhat.com wrote on 03/07/2010 11:45:33 PM:
+static int skb_head_len(struct sk_buff_head *skq)
+{
+ struct sk_buff *head;
+
+ head = skb_peek(skq);
+ if (head)
+ return head-len;
+ return 0;
+}
+
Michael S. Tsirkin m...@redhat.com wrote on 03/07/2010 07:31:30 AM:
On Tue, Mar 02, 2010 at 05:20:15PM -0700, David Stevens wrote:
This patch generalizes buffer handling functions to
NULL, NULL);
+ head.iov_base = (void *)vhost_get_vq_desc
Michael S. Tsirkin m...@redhat.com wrote on 03/07/2010 08:12:29 AM:
On Tue, Mar 02, 2010 at 05:20:26PM -0700, David Stevens wrote:
This patch adds vnet_hdr processing for mergeable buffer
support to vhost-net.
Signed-off-by: David L Stevens dlstev...@us.ibm.com
diff -ruN net-next
Michael S. Tsirkin m...@redhat.com wrote on 03/07/2010 08:26:33 AM:
On Tue, Mar 02, 2010 at 05:20:34PM -0700, David Stevens wrote:
This patch glues them all together and makes sure we
notify whenever we don't have enough buffers to receive
a max-sized packet, and adds the feature bit
Interesting. Since the feature in question is billed first of all a
performance optimization...
By whom? Although I see some improved performance, I think its real
benefit is improving memory utilization on the guest. Instead of using
75K for an ARP packet, mergeable RX buffers only uses 4K.
These patches add support for mergeable receive buffers to
vhost-net, allowing it to use multiple virtio buffer heads for a single
receive packet.
+-DLS
Signed-off-by: David L Stevens dlstev...@us.ibm.com
--
To unsubscribe from this list: send the line
This patch generalizes buffer handling functions to
support multiple buffer heads.
In-line for viewing, attached for applying.
Signed-off-by: David L Stevens dlstev...@us.ibm.com
diff -ruN net-next-p0/drivers/vhost/net.c net-next-p1/drivers/vhost/net.c
--- net-next-p0/drivers/vhost/net.c
This patch adds vnet_hdr processing for mergeable buffer
support to vhost-net.
Signed-off-by: David L Stevens dlstev...@us.ibm.com
diff -ruN net-next-p1/drivers/vhost/net.c net-next-p2/drivers/vhost/net.c
--- net-next-p1/drivers/vhost/net.c 2010-03-01 11:44:22.0
-0800
+++
This patch glues them all together and makes sure we
notify whenever we don't have enough buffers to receive
a max-sized packet, and adds the feature bit.
Signed-off-by: David L Stevens dlstev...@us.ibm.com
diff -ruN net-next-p2/drivers/vhost/net.c net-next-p3/drivers/vhost/net.c
---
Michael,
There is a 64-bit put_user(), so SET actually works already, but
there just isn't a 64-bit get_user().
It doesn't hurt to make them symmetric, though. The changes look
fine to me, if you want to do both.
I'm looking at adding mergeable rx buffer support and doing the
development
VHOST_GET_FEATURES returns high-order garbage on 32-bit
machines. This patch fixes it to use 64 bits throughout.
+-DLS
[in-line for viewing, attached to avoid whitespace mangling]
Signed-off-by: David L Stevens dlstev...@us.ibm.com
---
I haven't been following this closely, so apologies if the point's been
made, or
if you're talking about unicast addresses here too, but just to be clear:
For multicasting, false positives are ok, false negatives are not
(non-functional),
and if the fixed-size address filter is exceeded,
27 matches
Mail list logo