Ido Schimmel <ido...@idosch.org> writes: > On Tue, Jul 23, 2019 at 06:08:21PM +0200, Toke Høiland-Jørgensen wrote: >> Also, presumably the queue will have to change from a struct >> sk_buff_head to something that can hold XDP frames and whatever devlink >> puts there as well, right? > > Good point! > > For HW drops we get an SKB and relevant metadata from devlink about why > the packet was dropped etc. I plan to store a pointer to this metadata > in the SKB control block. > > Let me see how the implementation goes. Even if use sk_buff_head for > now, I will make sure that converting to a more generalized data > structure is straightforward.
Yup, that's fine, just wanted to make sure you were aware of the eventual need to change it :) -Toke
