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

2022-02-17 Thread David Marchand
On Thu, Feb 17, 2022 at 1:48 PM Ferruh Yigit wrote: > > On 2/17/2022 12:44 PM, David Marchand wrote: > > On Wed, Feb 9, 2022 at 10:48 AM Ciara Loftus wrote: > >> @@ -1836,6 +2008,17 @@ rte_pmd_af_xdp_probe(struct rte_vdev_device *dev) > >> return -1; > >> } > >> > >> +

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

2022-02-17 Thread Ferruh Yigit
On 2/17/2022 12:44 PM, David Marchand wrote: On Wed, Feb 9, 2022 at 10:48 AM Ciara Loftus wrote: @@ -1836,6 +2008,17 @@ rte_pmd_af_xdp_probe(struct rte_vdev_device *dev) return -1; } + /* Register IPC callback which shares xsk fds from primary to secondary */ +

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

2022-02-17 Thread David Marchand
On Wed, Feb 9, 2022 at 10:48 AM Ciara Loftus wrote: > @@ -1836,6 +2008,17 @@ rte_pmd_af_xdp_probe(struct rte_vdev_device *dev) > return -1; > } > > + /* Register IPC callback which shares xsk fds from primary to > secondary */ > + if (!afxdp_dev_count) { > +

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

2022-02-16 Thread Loftus, Ciara
> Subject: RE: [PATCH v4] net/af_xdp: re-enable secondary process support > > > > > On 2/11/2022 1:01 PM, Loftus, Ciara wrote: > > >> > > >> On 2/11/2022 9:26 AM, Loftus, Ciara wrote: > > >>>>> > > >>>>> On 2/10/2

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

2022-02-11 Thread Loftus, Ciara
> > On 2/11/2022 1:01 PM, Loftus, Ciara wrote: > >> > >> On 2/11/2022 9:26 AM, Loftus, Ciara wrote: > >>>>> > >>>>> On 2/10/2022 5:47 PM, Loftus, Ciara wrote: > >>>>>>> Subject: Re: [PATCH v4] net/af_xdp: re-enab

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

2022-02-11 Thread Ferruh Yigit
On 2/9/2022 3:29 PM, Stephen Hemminger wrote: On Wed, 9 Feb 2022 09:48:08 + Ciara Loftus wrote: 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

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

2022-02-11 Thread Ferruh Yigit
On 2/11/2022 1:01 PM, Loftus, Ciara wrote: On 2/11/2022 9:26 AM, Loftus, Ciara wrote: On 2/10/2022 5:47 PM, Loftus, Ciara wrote: Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process support On 2/10/2022 3:40 PM, Loftus, Ciara wrote: Subject: Re: [PATCH v4] net/af_xdp: re

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

2022-02-11 Thread Loftus, Ciara
> > On 2/11/2022 9:26 AM, Loftus, Ciara wrote: > >>> > >>> On 2/10/2022 5:47 PM, Loftus, Ciara wrote: > >>>>> Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process > >> support > >>>>> > >>>>>

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

2022-02-11 Thread Ferruh Yigit
On 2/11/2022 9:26 AM, Loftus, Ciara wrote: On 2/10/2022 5:47 PM, Loftus, Ciara wrote: Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process support On 2/10/2022 3:40 PM, Loftus, Ciara wrote: Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process support On 2/9/2022 9

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

2022-02-11 Thread Ferruh Yigit
On 2/11/2022 7:28 AM, Loftus, Ciara wrote: On 2/10/2022 5:47 PM, Loftus, Ciara wrote: Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process support On 2/10/2022 3:40 PM, Loftus, Ciara wrote: Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process support On 2/9/2022 9:48

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

2022-02-11 Thread Loftus, Ciara
> > > > On 2/10/2022 5:47 PM, Loftus, Ciara wrote: > > >> Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process > support > > >> > > >> On 2/10/2022 3:40 PM, Loftus, Ciara wrote: > > >>>> Subject: Re: [PATCH v4] net/af_

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

2022-02-10 Thread Loftus, Ciara
> > On 2/10/2022 5:47 PM, Loftus, Ciara wrote: > >> Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process support > >> > >> On 2/10/2022 3:40 PM, Loftus, Ciara wrote: > >>>> Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary pro

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

2022-02-10 Thread Ferruh Yigit
On 2/10/2022 5:47 PM, Loftus, Ciara wrote: Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process support On 2/10/2022 3:40 PM, Loftus, Ciara wrote: Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process support On 2/9/2022 9:48 AM, Ciara Loftus wrote: Secondary process

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

2022-02-10 Thread Loftus, Ciara
> Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process support > > On 2/10/2022 3:40 PM, Loftus, Ciara wrote: > >> Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process support > >> > >> On 2/9/2022 9:48 AM, Ciara Loftus wrote: >

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

2022-02-10 Thread Ferruh Yigit
On 2/10/2022 3:40 PM, Loftus, Ciara wrote: Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process support On 2/9/2022 9:48 AM, Ciara Loftus wrote: 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

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

2022-02-10 Thread Loftus, Ciara
> Subject: Re: [PATCH v4] net/af_xdp: re-enable secondary process support > > On 2/9/2022 9:48 AM, Ciara Loftus wrote: > > 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 &

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

2022-02-10 Thread Ferruh Yigit
On 2/9/2022 9:48 AM, Ciara Loftus wrote: 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. Rx and Tx are disabled in the secon

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

2022-02-10 Thread Ferruh Yigit
On 2/9/2022 5:55 PM, Ferruh Yigit wrote: On 2/9/2022 9:48 AM, Ciara Loftus wrote: 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

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

2022-02-09 Thread Ferruh Yigit
On 2/9/2022 9:48 AM, Ciara Loftus wrote: 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. Rx and Tx are disabled in the secon

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

2022-02-09 Thread Stephen Hemminger
On Wed, 9 Feb 2022 09:48:08 + Ciara Loftus wrote: > 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. > > Rx and Tx