lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  PHC 
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Tue, 14 Nov 2017 07:20:47 -0500
From:   Willem de Bruijn <>
To:     John Fastabend <>
Cc:     Björn Töpel <>,
        Alexei Starovoitov <>,
        "Karlsson, Magnus" <>,
        "Duyck, Alexander H" <>,
        Alexander Duyck <>,
        Jesper Dangaard Brouer <>,,,
        Daniel Borkmann <>,
        Netdev <>,
        Tushar Dave <>,
        Eric Dumazet <>,
        Björn Töpel <>,
        "Brandeburg, Jesse" <>,
        "Singhai, Anjali" <>,
        "Rosen, Rami" <>,
        "Shaw, Jeffrey B" <>,
        "Yigit, Ferruh" <>,
        "Zhang, Qi Z" <>,
        David Miller <>
Subject: Re: [RFC PATCH 00/14] Introducing AF_PACKET V4 support

>>>> * Limit the scope of the first patchset to Rx only, and introduce Tx
>>>>   in a separate patchset.
>>> all sounds good to me except above bit.
>>> I don't remember people suggesting to split it this way.
>>> What's the value of it without tx?
>> We definitely need Tx for our use-cases! I'll rephrase, so the
>> idea was making the initial patch set without Tx *driver*
>> specific code, e.g. use ndo_xdp_xmit/flush at a later point.
>> So AF_ZEROCOPY, the socket parts, would have Tx support.
>> @John Did I recall that correctly?
> Yep, that is what I said. However, on second thought, without the
> driver tx half I guess tx will be significantly slower.

The idea was that existing packet rings already send without
copying, so the benefit from device driver changes is not obvious.

I would leave them out for now and evaluate before possibly
sending a separate patchset.

Powered by blists - more mailing lists