>-----Original Message----- >From: Intel-wired-lan <intel-wired-lan-bounces@xxxxxxxxxx> On Behalf Of Kurt >Kanzenbach >Sent: Friday, July 12, 2024 2:25 PM >To: Nguyen, Anthony L <anthony.l.nguyen@xxxxxxxxx>; Kitszel, Przemyslaw ><przemyslaw.kitszel@xxxxxxxxx> >Cc: Jesper Dangaard Brouer <hawk@xxxxxxxxxx>; Daniel Borkmann ><daniel@xxxxxxxxxxxxx>; Sriram Yagnaraman ><sriram.yagnaraman@xxxxxxxxxxxx>; Richard Cochran ><richardcochran@xxxxxxxxx>; Kurt Kanzenbach <kurt@xxxxxxxxxxxxx>; John >Fastabend <john.fastabend@xxxxxxxxx>; Alexei Starovoitov <ast@xxxxxxxxxx>; >Benjamin Steinke <benjamin.steinke@xxxxxxxxxxxxxx>; Eric Dumazet ><edumazet@xxxxxxxxxx>; Sriram Yagnaraman ><sriram.yagnaraman@xxxxxxxx>; intel-wired-lan@xxxxxxxxxxxxxxxx; >netdev@xxxxxxxxxxxxxxx; Jakub Kicinski <kuba@xxxxxxxxxx>; >bpf@xxxxxxxxxxxxxxx; Paolo Abeni <pabeni@xxxxxxxxxx>; David S. Miller ><davem@xxxxxxxxxxxxx>; Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx> >Subject: [Intel-wired-lan] [PATCH iwl-next v5 1/4] igb: prepare for AF_XDP >zero-copy support > >From: Sriram Yagnaraman <sriram.yagnaraman@xxxxxxxx> > >Always call igb_xdp_ring_update_tail under __netif_tx_lock, add a comment to >indicate that. This is needed to share the same TX ring between XDP, XSK and >slow paths. > >Remove static qualifiers on the following functions to be able to call from XSK >specific file that is added in the later patches >- igb_xdp_tx_queue_mapping >- igb_xdp_ring_update_tail >- igb_clean_tx_ring >- igb_clean_rx_ring >- igb_run_xdp >- igb_process_skb_fields > >Introduce igb_xdp_is_enabled() to check if an XDP program is assigned to the >device. > >Signed-off-by: Sriram Yagnaraman <sriram.yagnaraman@xxxxxxxx> >Signed-off-by: Kurt Kanzenbach <kurt@xxxxxxxxxxxxx> >--- > drivers/net/ethernet/intel/igb/igb.h | 15 +++++++++++++++ > drivers/net/ethernet/intel/igb/igb_main.c | 29 ++++++++++++++--------------- > 2 files changed, 29 insertions(+), 15 deletions(-) > Tested-by: Chandan Kumar Rout <chandanx.rout@xxxxxxxxx> (A Contingent Worker at Intel)