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

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

2022-02-09 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. Rx and Tx are disabled in the secondary process due to memory mapping of the A