lkml.org 
[lkml]   [2011]   [Jun]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCHv2 RFC 0/4] virtio and vhost-net capacity handling
    On Thu, Jun 02, 2011 at 06:42:35PM +0300, Michael S. Tsirkin wrote:
    > OK, here's a new attempt to use the new capacity api. I also added more
    > comments to clarify the logic. Hope this is more readable. Let me know
    > pls.
    >
    > This is on top of the patches applied by Rusty.
    >
    > Warning: untested. Posting now to give people chance to
    > comment on the API.

    OK, this seems to have survived some testing so far,
    after I dropped patch 4 and fixed build for patch 3
    (build fixup patch sent in reply to the original).

    I'll be mostly offline until Sunday, would appreciate
    testing reports.

    git://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git
    virtio-net-xmit-polling-v2
    git://git.kernel.org/pub/scm/linux/kernel/git/mst/qemu-kvm.git
    virtio-net-event-idx-v3

    Thanks!

    > Changes from v1:
    > - fix comment in patch 2 to correct confusion noted by Rusty
    > - rewrite patch 3 along the lines suggested by Rusty
    > note: it's not exactly the same but I hope it's close
    > enough, the main difference is that mine does limited
    > polling even in the unlikely xmit failure case.
    > - added a patch to not return capacity from add_buf
    > it always looked like a weird hack
    >
    > Michael S. Tsirkin (4):
    > virtio_ring: add capacity check API
    > virtio_net: fix tx capacity checks using new API
    > virtio_net: limit xmit polling
    > Revert "virtio: make add_buf return capacity remaining:
    >
    > drivers/net/virtio_net.c | 111 ++++++++++++++++++++++++++----------------
    > drivers/virtio/virtio_ring.c | 10 +++-
    > include/linux/virtio.h | 7 ++-
    > 3 files changed, 84 insertions(+), 44 deletions(-)
    >
    > --
    > 1.7.5.53.gc233e


    \
     
     \ /
      Last update: 2011-06-07 18:11    [W:2.413 / U:0.012 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site