RE: [PATCH v2] net/af_xdp: fix umem map size for zero copy

2024-05-29 Thread Morten Brørup
> From: Loftus, Ciara [mailto:ciara.lof...@intel.com] > Sent: Wednesday, 29 May 2024 14.58 > > > From: Du, Frank > > Sent: Thursday, May 23, 2024 8:56 AM > > > > > From: Morten Brørup > > > Sent: Thursday, May 23, 2024 3:41 PM > > > > > > > From: Du, Frank [mailto:frank...@intel.com] > > > > Sen

RE: [PATCH v2] net/af_xdp: fix umem map size for zero copy

2024-05-29 Thread Loftus, Ciara
> -Original Message- > From: Du, Frank > Sent: Thursday, May 23, 2024 8:56 AM > To: Morten Brørup ; Ferruh Yigit > ; dev@dpdk.org; Andrew Rybchenko > ; Burakov, Anatoly > > Cc: Loftus, Ciara > Subject: RE: [PATCH v2] net/af_xdp: fix umem map size for zer

RE: [PATCH v2] net/af_xdp: fix umem map size for zero copy

2024-05-23 Thread Du, Frank
> -Original Message- > From: Morten Brørup > Sent: Thursday, May 23, 2024 3:41 PM > To: Du, Frank ; Ferruh Yigit ; > dev@dpdk.org; Andrew Rybchenko ; Burakov, > Anatoly > Cc: Loftus, Ciara > Subject: RE: [PATCH v2] net/af_xdp: fix umem map size for zero cop

RE: [PATCH v2] net/af_xdp: fix umem map size for zero copy

2024-05-23 Thread Morten Brørup
> From: Du, Frank [mailto:frank...@intel.com] > Sent: Thursday, 23 May 2024 08.56 > > > From: Morten Brørup > > Sent: Wednesday, May 22, 2024 3:27 PM > > > > > From: Du, Frank [mailto:frank...@intel.com] > > > Sent: Wednesday, 22 May 2024 03.25 > > > > > > > From: Ferruh Yigit > > > > Sent: Wedn

RE: [PATCH v2] net/af_xdp: fix umem map size for zero copy

2024-05-22 Thread Du, Frank
> -Original Message- > From: Morten Brørup > Sent: Wednesday, May 22, 2024 3:27 PM > To: Du, Frank ; Ferruh Yigit ; > dev@dpdk.org; Andrew Rybchenko ; Burakov, > Anatoly > Cc: Loftus, Ciara > Subject: RE: [PATCH v2] net/af_xdp: fix umem map size for zero cop

Re: [PATCH v2] net/af_xdp: fix umem map size for zero copy

2024-05-22 Thread Ferruh Yigit
On 5/22/2024 12:03 PM, Morten Brørup wrote: >> From: Ferruh Yigit [mailto:ferruh.yi...@amd.com] >> Sent: Wednesday, 22 May 2024 12.01 >> >> On 5/22/2024 2:25 AM, Du, Frank wrote: From: Ferruh Yigit Sent: Wednesday, May 22, 2024 1:58 AM Isn't there a mempool flag that can help u

RE: [PATCH v2] net/af_xdp: fix umem map size for zero copy

2024-05-22 Thread Morten Brørup
> From: Ferruh Yigit [mailto:ferruh.yi...@amd.com] > Sent: Wednesday, 22 May 2024 12.01 > > On 5/22/2024 2:25 AM, Du, Frank wrote: > >> From: Ferruh Yigit > >> Sent: Wednesday, May 22, 2024 1:58 AM > >> > >> Isn't there a mempool flag that can help us figure out mempool is not IOVA > >> contiguou

Re: [PATCH v2] net/af_xdp: fix umem map size for zero copy

2024-05-22 Thread Ferruh Yigit
On 5/22/2024 8:26 AM, Morten Brørup wrote: >> From: Du, Frank [mailto:frank...@intel.com] >> Sent: Wednesday, 22 May 2024 03.25 >> >>> From: Ferruh Yigit >>> Sent: Wednesday, May 22, 2024 1:58 AM >>> >>> On 5/11/2024 6:26 AM, Frank Du wrote: The current calculation assumes that the mbufs are

Re: [PATCH v2] net/af_xdp: fix umem map size for zero copy

2024-05-22 Thread Ferruh Yigit
On 5/22/2024 2:25 AM, Du, Frank wrote: >> -Original Message- >> From: Ferruh Yigit >> Sent: Wednesday, May 22, 2024 1:58 AM >> To: Du, Frank ; dev@dpdk.org; Andrew Rybchenko >> ; Morten Brørup >> >> Cc: Loftus, Ciara ; Burakov, Anatoly >>

RE: [PATCH v2] net/af_xdp: fix umem map size for zero copy

2024-05-22 Thread Morten Brørup
> From: Du, Frank [mailto:frank...@intel.com] > Sent: Wednesday, 22 May 2024 03.25 > > > From: Ferruh Yigit > > Sent: Wednesday, May 22, 2024 1:58 AM > > > > On 5/11/2024 6:26 AM, Frank Du wrote: > > > The current calculation assumes that the mbufs are contiguous. > > > However, this assumption i

RE: [PATCH v2] net/af_xdp: fix umem map size for zero copy

2024-05-21 Thread Du, Frank
> -Original Message- > From: Ferruh Yigit > Sent: Wednesday, May 22, 2024 1:58 AM > To: Du, Frank ; dev@dpdk.org; Andrew Rybchenko > ; Morten Brørup > > Cc: Loftus, Ciara ; Burakov, Anatoly > > Subject: Re: [PATCH v2] net/af_xdp: fix umem map size for zero cop

Re: [PATCH v2] net/af_xdp: fix umem map size for zero copy

2024-05-21 Thread Ferruh Yigit
On 5/11/2024 6:26 AM, Frank Du wrote: > The current calculation assumes that the mbufs are contiguous. However, > this assumption is incorrect when the memory spans across a huge page. > Correct to directly read the size from the mempool memory chunks. > > Signed-off-by: Frank Du > > --- > v2: >

Re: [PATCH v2] net/af_xdp: fix umem map size for zero copy

2024-05-21 Thread Ferruh Yigit
On 5/11/2024 6:26 AM, Frank Du wrote: > The current calculation assumes that the mbufs are contiguous. However, > this assumption is incorrect when the memory spans across a huge page. > Correct to directly read the size from the mempool memory chunks. > > Signed-off-by: Frank Du > Hi Frank, As

RE: [PATCH v2] net/af_xdp: fix umem map size for zero copy

2024-05-19 Thread Du, Frank
> -Original Message- > From: Loftus, Ciara > Sent: Friday, May 17, 2024 9:20 PM > To: Du, Frank > Cc: dev@dpdk.org > Subject: RE: [PATCH v2] net/af_xdp: fix umem map size for zero copy > > > > > The current calculation assumes that the mbufs are contigu

RE: [PATCH v2] net/af_xdp: fix umem map size for zero copy

2024-05-17 Thread Loftus, Ciara
> > The current calculation assumes that the mbufs are contiguous. However, > this assumption is incorrect when the memory spans across a huge page. > Correct to directly read the size from the mempool memory chunks. > > Signed-off-by: Frank Du Hi Frank, Thanks for the patch. Before your patc