RE: [PATCH] net/hns3: support disable IOVA as PA mode

2023-02-20 Thread Morten Brørup
> From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Monday, 20 February 2023 16.41 > > 20/02/2023 16:35, Bruce Richardson: > > On Mon, Feb 20, 2023 at 04:30:20PM +0100, Thomas Monjalon wrote: > > > 20/02/2023 16:07, Morten Brørup: > > > > With the new viewing angle, the current define RTE

Re: [PATCH] net/hns3: support disable IOVA as PA mode

2023-02-20 Thread Thomas Monjalon
20/02/2023 16:35, Bruce Richardson: > On Mon, Feb 20, 2023 at 04:30:20PM +0100, Thomas Monjalon wrote: > > 20/02/2023 16:07, Morten Brørup: > > > With the new viewing angle, the current define RTE_IOVA_AS_PA makes more > > > sense to me now than before. So we should probably stick with it, rather

Re: [PATCH] net/hns3: support disable IOVA as PA mode

2023-02-20 Thread Bruce Richardson
On Mon, Feb 20, 2023 at 04:30:20PM +0100, Thomas Monjalon wrote: > 20/02/2023 16:07, Morten Brørup: > > With the new viewing angle, the current define RTE_IOVA_AS_PA makes more > > sense to me now than before. So we should probably stick with it, rather > > than introduce something that might con

Re: [PATCH] net/hns3: support disable IOVA as PA mode

2023-02-20 Thread Thomas Monjalon
20/02/2023 16:07, Morten Brørup: > With the new viewing angle, the current define RTE_IOVA_AS_PA makes more > sense to me now than before. So we should probably stick with it, rather than > introduce something that might confuse developers who already have the same > viewing angle. > > But it s

RE: [PATCH] net/hns3: support disable IOVA as PA mode

2023-02-20 Thread Morten Brørup
t; > > > > > > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > > > > > > > > Sent: Monday, 20 February 2023 08.45 > > > > > > > > > > > > > > > > > > 16/02/2023 09:36, Ruifeng Wang: > > > > &g

Re: [PATCH] net/hns3: support disable IOVA as PA mode

2023-02-20 Thread Bruce Richardson
3 11.17 > > > > > > > > > > > > 20/02/2023 10:43, Morten Brørup: > > > > > > > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > > > > > > > Sent: Monday, 20 February 2023 08.45

RE: [PATCH] net/hns3: support disable IOVA as PA mode

2023-02-20 Thread Morten Brørup
> > > > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > > > > > > Sent: Monday, 20 February 2023 08.45 > > > > > > > > > > > > > > 16/02/2023 09:36, Ruifeng Wang: > > > > > > > > From: Chengwen Feng

Re: [PATCH] net/hns3: support disable IOVA as PA mode

2023-02-20 Thread Bruce Richardson
ebruary 2023 08.45 > > > > > > > > > > > > 16/02/2023 09:36, Ruifeng Wang: > > > > > > > From: Chengwen Feng > > > > > > > > Subject: [PATCH] net/hns3: support disable IOVA as PA mode > > > > >

RE: [PATCH] net/hns3: support disable IOVA as PA mode

2023-02-20 Thread Morten Brørup
7 > > > > > > 20/02/2023 10:43, Morten Brørup: > > > > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > > > > Sent: Monday, 20 February 2023 08.45 > > > > > > > > > > 16/02/2023 09:36, Ruifeng Wan

Re: [PATCH] net/hns3: support disable IOVA as PA mode

2023-02-20 Thread Bruce Richardson
t; > > > Sent: Monday, 20 February 2023 08.45 > > > > > > > > 16/02/2023 09:36, Ruifeng Wang: > > > > > From: Chengwen Feng > > > > > > Subject: [PATCH] net/hns3: support disable IOVA as PA mode > > > > > > > &

RE: [PATCH] net/hns3: support disable IOVA as PA mode

2023-02-20 Thread Morten Brørup
2023 09:36, Ruifeng Wang: > > > > From: Chengwen Feng > > > > > Subject: [PATCH] net/hns3: support disable IOVA as PA mode > > > > > > Could we change the title to "support IOVA as VA" ? > > > > The underlying problem is the meson

Re: [PATCH] net/hns3: support disable IOVA as PA mode

2023-02-20 Thread Thomas Monjalon
20/02/2023 10:43, Morten Brørup: > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > Sent: Monday, 20 February 2023 08.45 > > > > 16/02/2023 09:36, Ruifeng Wang: > > > From: Chengwen Feng > > > > Subject: [PATCH] net/hns3: support disable IOVA

RE: [PATCH] net/hns3: support disable IOVA as PA mode

2023-02-20 Thread Morten Brørup
> From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Monday, 20 February 2023 08.45 > > 16/02/2023 09:36, Ruifeng Wang: > > From: Chengwen Feng > > > Subject: [PATCH] net/hns3: support disable IOVA as PA mode > > Could we change the title to "

Re: [PATCH] net/hns3: support disable IOVA as PA mode

2023-02-20 Thread fengchengwen
On 2023/2/20 15:44, Thomas Monjalon wrote: > 16/02/2023 09:36, Ruifeng Wang: >> From: Chengwen Feng >>> Subject: [PATCH] net/hns3: support disable IOVA as PA mode > > Could we change the title to "support IOVA as VA" ? v2 fix it, please review, than

Re: [PATCH] net/hns3: support disable IOVA as PA mode

2023-02-19 Thread Thomas Monjalon
16/02/2023 09:36, Ruifeng Wang: > From: Chengwen Feng > > Subject: [PATCH] net/hns3: support disable IOVA as PA mode Could we change the title to "support IOVA as VA" ? > > > > Claim PMD supports pmd_supports_disable_iova_as_pa. > > > > Signed-off-

RE: [PATCH] net/hns3: support disable IOVA as PA mode

2023-02-16 Thread Ruifeng Wang
> -Original Message- > From: Chengwen Feng > Sent: Tuesday, February 14, 2023 3:12 PM > To: tho...@monjalon.net; ferruh.yi...@amd.com; Ruifeng Wang > ; > Dongdong Liu ; Yisen Zhuang > Cc: dev@dpdk.org > Subject: [PATCH] net/hns3: support disable IOVA as PA mode

Re: [PATCH] net/hns3: support disable IOVA as PA mode

2023-02-14 Thread Dongdong Liu
Hi Chengwen On 2023/2/14 15:11, Chengwen Feng wrote: Claim PMD supports pmd_supports_disable_iova_as_pa. Signed-off-by: Chengwen Feng This patch looks good to me. Many thanks for your work. Acked-by: Dongdong Liu Thanks, Dongdong

[PATCH] net/hns3: support disable IOVA as PA mode

2023-02-13 Thread Chengwen Feng
Claim PMD supports pmd_supports_disable_iova_as_pa. Signed-off-by: Chengwen Feng --- drivers/net/hns3/hns3_rxtx_vec_neon.h | 2 +- drivers/net/hns3/hns3_rxtx_vec_sve.c | 13 + drivers/net/hns3/meson.build | 7 +-- 3 files changed, 11 insertions(+), 11 deletions(-) di