> On 6/24/20 5:33 PM, Lorenzo Bianconi wrote: [...] > > + if (stats->redirect) > > + xdp_do_flush_map(); > > + > > + rcu_read_unlock_bh(); /* resched point, may call do_softirq() */ > > xdp_clear_return_frame_no_direct(); > > Hm, this looks incorrect. Why do you call the xdp_clear_return_frame_no_direct() /after/ > the possibility where there is a rescheduling point for softirq? Hi Daniel, yes, right. Thx for spotting this, I will fix in v5. Regards, Lorenzo > > > return nframes; > > >
Attachment:
signature.asc
Description: PGP signature
