Re: [PATCH net-next 10/11] qede: Add basic XDP support

2016-11-28 Thread Mintz, Yuval
> > +       if (act == XDP_PASS) > > +               return true; > > + > > +       /* Count number of packets not to be passed to stack */ > > +       rxq->xdp_no_pass++; > > + > > +       switch (act) { > > +       default: > > +               bpf_warn_invalid_xdp_action(act); > XDP_TX is a vali

Re: [PATCH net-next 10/11] qede: Add basic XDP support

2016-11-28 Thread Mintz, Yuval
> Any particular reason not to allow the XDP prog being set while the > device is closed?  You seem to preserve the program across > close()/open() cycles so the edev->xdp_prog is alive and valid while > device is closed IIUC.  > I think other drivers are allowing setting XDP while closed and it

Re: [PATCH net-next 10/11] qede: Add basic XDP support

2016-11-28 Thread Jakub Kicinski
On Sun, 27 Nov 2016 16:51:12 +0200, Yuval Mintz wrote: > Add support for the ndo_xdp callback. This patch would support XDP_PASS, > XDP_DROP and XDP_ABORTED commands. > > This also adds a per Rx queue statistic which counts number of packets > which didn't reach the stack [due to XDP]. > > Signed

[PATCH net-next 10/11] qede: Add basic XDP support

2016-11-27 Thread Yuval Mintz
Add support for the ndo_xdp callback. This patch would support XDP_PASS, XDP_DROP and XDP_ABORTED commands. This also adds a per Rx queue statistic which counts number of packets which didn't reach the stack [due to XDP]. Signed-off-by: Yuval Mintz --- drivers/net/ethernet/qlogic/qede/qede.h