RE: [PATCH v3] net/af_xdp: re-enable secondary process support

2022-02-08 Thread Loftus, Ciara
> > On 2/8/2022 6:42 PM, Stephen Hemminger wrote: > > On Tue, 8 Feb 2022 18:00:27 + > > Ferruh Yigit wrote: > > > >> On 2/8/2022 5:45 PM, Stephen Hemminger wrote: > >>> On Tue, 8 Feb 2022 13:48:00 + > >>> Ciara Loftus wrote: > >>> > +- **Secondary Processes** > + > + Rx

Re: [PATCH v3] net/af_xdp: re-enable secondary process support

2022-02-08 Thread Ferruh Yigit
On 2/8/2022 6:42 PM, Stephen Hemminger wrote: On Tue, 8 Feb 2022 18:00:27 + Ferruh Yigit wrote: On 2/8/2022 5:45 PM, Stephen Hemminger wrote: On Tue, 8 Feb 2022 13:48:00 + Ciara Loftus wrote: +- **Secondary Processes** + + Rx and Tx are not supported for secondary processes du

Re: [PATCH v3] net/af_xdp: re-enable secondary process support

2022-02-08 Thread Stephen Hemminger
On Tue, 8 Feb 2022 18:00:27 + Ferruh Yigit wrote: > On 2/8/2022 5:45 PM, Stephen Hemminger wrote: > > On Tue, 8 Feb 2022 13:48:00 + > > Ciara Loftus wrote: > > > >> +- **Secondary Processes** > >> + > >> + Rx and Tx are not supported for secondary processes due to the > >> single-p

Re: [PATCH v3] net/af_xdp: re-enable secondary process support

2022-02-08 Thread Ferruh Yigit
On 2/8/2022 5:45 PM, Stephen Hemminger wrote: On Tue, 8 Feb 2022 13:48:00 + Ciara Loftus wrote: +- **Secondary Processes** + + Rx and Tx are not supported for secondary processes due to the single-producer + single-consumer nature of the AF_XDP rings. However other operations includin

Re: [PATCH v3] net/af_xdp: re-enable secondary process support

2022-02-08 Thread Stephen Hemminger
On Tue, 8 Feb 2022 13:48:00 + Ciara Loftus wrote: > +- **Secondary Processes** > + > + Rx and Tx are not supported for secondary processes due to the > single-producer > + single-consumer nature of the AF_XDP rings. However other operations > including > + statistics retrieval are permi

[PATCH v3] net/af_xdp: re-enable secondary process support

2022-02-08 Thread Ciara Loftus
Secondary process support had been disabled for the AF_XDP PMD because there was no logic in place to share the AF_XDP socket file descriptors between the processes. This commit introduces this logic using the IPC APIs. Since AF_XDP rings are single-producer single-consumer, rx/tx in the secondary