Re: [PATCH 0/9 v1 RFC] Generic zcopy_* functions

2020-12-21 Thread Jonathan Lemon
On Mon, Dec 21, 2020 at 05:52:08PM -0500, Willem de Bruijn wrote: > > > > - marking the skb data as inaccessible so skb_condense() > > > > and skb_zeroocopy_clone() leave it alone. > > > > > > Yep. Skipping content access on the Rx path will be interesting. I > > > wonder if that should be a

Re: [PATCH 0/9 v1 RFC] Generic zcopy_* functions

2020-12-21 Thread Willem de Bruijn
> > > - marking the skb data as inaccessible so skb_condense() > > > and skb_zeroocopy_clone() leave it alone. > > > > Yep. Skipping content access on the Rx path will be interesting. I > > wonder if that should be a separate opaque skb feature, independent > > from whether the data is owned

Re: [PATCH 0/9 v1 RFC] Generic zcopy_* functions

2020-12-21 Thread Jonathan Lemon
On Sat, Dec 19, 2020 at 02:00:55PM -0500, Willem de Bruijn wrote: > On Fri, Dec 18, 2020 at 4:27 PM Jonathan Lemon > wrote: > > > > On Fri, Dec 18, 2020 at 03:49:44PM -0500, Willem de Bruijn wrote: > > > On Fri, Dec 18, 2020 at 3:23 PM Jonathan Lemon > > > wrote: > > > > > > > > From: Jonathan

Re: [PATCH 0/9 v1 RFC] Generic zcopy_* functions

2020-12-19 Thread Willem de Bruijn
On Fri, Dec 18, 2020 at 4:27 PM Jonathan Lemon wrote: > > On Fri, Dec 18, 2020 at 03:49:44PM -0500, Willem de Bruijn wrote: > > On Fri, Dec 18, 2020 at 3:23 PM Jonathan Lemon > > wrote: > > > > > > From: Jonathan Lemon > > > > > > This is set of cleanup patches for zerocopy which are intended >

Re: [PATCH 0/9 v1 RFC] Generic zcopy_* functions

2020-12-18 Thread Jonathan Lemon
On Fri, Dec 18, 2020 at 03:49:44PM -0500, Willem de Bruijn wrote: > On Fri, Dec 18, 2020 at 3:23 PM Jonathan Lemon > wrote: > > > > From: Jonathan Lemon > > > > This is set of cleanup patches for zerocopy which are intended > > to allow a introduction of a different zerocopy implementation. > >

Re: [PATCH 0/9 v1 RFC] Generic zcopy_* functions

2020-12-18 Thread Willem de Bruijn
On Fri, Dec 18, 2020 at 3:23 PM Jonathan Lemon wrote: > > From: Jonathan Lemon > > This is set of cleanup patches for zerocopy which are intended > to allow a introduction of a different zerocopy implementation. Can you describe in more detail what exactly is lacking in the current zerocopy inte

[PATCH 0/9 v1 RFC] Generic zcopy_* functions

2020-12-18 Thread Jonathan Lemon
From: Jonathan Lemon This is set of cleanup patches for zerocopy which are intended to allow a introduction of a different zerocopy implementation. The top level api will use the skb_zcopy_*() functions, while the current TCP specific zerocopy would use the sock_zerocopy_*() calls. There should