On Mon, Jun 17, 2019 at 7:44 PM Edward Cree <ec...@solarflare.com> wrote: > > On 14/06/2019 15:33, John Hurley wrote: > > Instead of > > returning TC_ACT_REINSERT, change the type to the new TC_ACT_CONSUMED > > which tells the caller that the packet has been stolen by another process > > and that no consume call is required. > Possibly a dumb question, but why does this need a new CONSUMED rather > than, say, taking an additional ref and returning TC_ACT_STOLEN? > > Apart from that, the series lgtm. >
Thanks for comments, Ed. The CONSUMED was to replace the REINSERT function but yes, this is probably not required now. I can respin > -Ed
