lkml.org 
[lkml]   [2019]   [Jul]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH AUTOSEL 5.2 119/249] vhost_net: disable zerocopy by default
    Date
    From: Jason Wang <jasowang@redhat.com>

    [ Upstream commit 098eadce3c622c07b328d0a43dda379b38cf7c5e ]

    Vhost_net was known to suffer from HOL[1] issues which is not easy to
    fix. Several downstream disable the feature by default. What's more,
    the datapath was split and datacopy path got the support of batching
    and XDP support recently which makes it faster than zerocopy part for
    small packets transmission.

    It looks to me that disable zerocopy by default is more
    appropriate. It cold be enabled by default again in the future if we
    fix the above issues.

    [1] https://patchwork.kernel.org/patch/3787671/

    Signed-off-by: Jason Wang <jasowang@redhat.com>
    Acked-by: Michael S. Tsirkin <mst@redhat.com>
    Signed-off-by: David S. Miller <davem@davemloft.net>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/vhost/net.c | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    diff --git a/drivers/vhost/net.c b/drivers/vhost/net.c
    index d57ebdd616d9..247e5585af5d 100644
    --- a/drivers/vhost/net.c
    +++ b/drivers/vhost/net.c
    @@ -35,7 +35,7 @@

    #include "vhost.h"

    -static int experimental_zcopytx = 1;
    +static int experimental_zcopytx = 0;
    module_param(experimental_zcopytx, int, 0444);
    MODULE_PARM_DESC(experimental_zcopytx, "Enable Zero Copy TX;"
    " 1 -Enable; 0 - Disable");
    --
    2.20.1
    \
     
     \ /
      Last update: 2019-07-15 17:15    [W:3.651 / U:0.484 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site