On Mon, Dec 21, 2020 at 7:09 PM Jonathan Lemon <jonathan.le...@gmail.com> wrote: > > From: Jonathan Lemon <b...@fb.com> > > All 'struct ubuf_info' users should have a callback defined. > Remove the dead code path to consume_skb(), which makes > unwarranted assumptions about how the structure was allocated. > > Signed-off-by: Jonathan Lemon <jonathan.le...@gmail.com>
Please link to the commit I shared that made the consume_skb path obsolete. Before that this would have been unsafe. should have a callback defined -> have a callback defined as of commit 0a4a060bb204 ("sock: fix zerocopy_success regression with msg_zerocopy"). With that explanation why this is correct Acked-by: Willem de Bruijn <will...@google.com>