Re: [PATCH v2 1/2] build: clarify configuration without IOVA field in mbuf

2023-03-13 Thread Thomas Monjalon
09/03/2023 14:10, Bruce Richardson: > On Thu, Mar 09, 2023 at 01:12:51PM +0100, Thomas Monjalon wrote: > > 09/03/2023 12:23, fengchengwen: > > > On 2023/3/9 15:29, Thomas Monjalon wrote: > > > > 09/03/2023 02:43, fengchengwen: > > > >> On 2023/3/7 0:13, Thomas Monjalon wrote: > > > >>> --- a/doc/gu

Re: [PATCH v2 1/2] build: clarify configuration without IOVA field in mbuf

2023-03-09 Thread Bruce Richardson
On Thu, Mar 09, 2023 at 01:12:51PM +0100, Thomas Monjalon wrote: > 09/03/2023 12:23, fengchengwen: > > On 2023/3/9 15:29, Thomas Monjalon wrote: > > > 09/03/2023 02:43, fengchengwen: > > >> On 2023/3/7 0:13, Thomas Monjalon wrote: > > >>> --- a/doc/guides/rel_notes/release_22_11.rst > > >>> +++ b/d

Re: [PATCH v2 1/2] build: clarify configuration without IOVA field in mbuf

2023-03-09 Thread Thomas Monjalon
09/03/2023 12:23, fengchengwen: > On 2023/3/9 15:29, Thomas Monjalon wrote: > > 09/03/2023 02:43, fengchengwen: > >> On 2023/3/7 0:13, Thomas Monjalon wrote: > >>> --- a/doc/guides/rel_notes/release_22_11.rst > >>> +++ b/doc/guides/rel_notes/release_22_11.rst > >>> @@ -504,7 +504,7 @@ ABI Changes >

Re: [PATCH v2 1/2] build: clarify configuration without IOVA field in mbuf

2023-03-09 Thread fengchengwen
On 2023/3/9 15:29, Thomas Monjalon wrote: > 09/03/2023 02:43, fengchengwen: >> On 2023/3/7 0:13, Thomas Monjalon wrote: >>> --- a/doc/guides/rel_notes/release_22_11.rst >>> +++ b/doc/guides/rel_notes/release_22_11.rst >>> @@ -504,7 +504,7 @@ ABI Changes >>>``rte-worker-`` so that DPDK can ac

Re: [PATCH v2 1/2] build: clarify configuration without IOVA field in mbuf

2023-03-08 Thread Thomas Monjalon
09/03/2023 02:43, fengchengwen: > On 2023/3/7 0:13, Thomas Monjalon wrote: > > --- a/doc/guides/rel_notes/release_22_11.rst > > +++ b/doc/guides/rel_notes/release_22_11.rst > > @@ -504,7 +504,7 @@ ABI Changes > >``rte-worker-`` so that DPDK can accommodate lcores higher > > than 99. > > > >

Re: [PATCH v2 1/2] build: clarify configuration without IOVA field in mbuf

2023-03-08 Thread fengchengwen
On 2023/3/7 0:13, Thomas Monjalon wrote: > The impact of the option "enable_iova_as_pa" is explained for users. > > Also the code flag "RTE_IOVA_AS_PA" is renamed as "RTE_IOVA_IN_MBUF" > in order to be more accurate (IOVA mode is decided at runtime), > and more readable in the code. > > Similarly

Re: [PATCH v2 1/2] build: clarify configuration without IOVA field in mbuf

2023-03-06 Thread Thomas Monjalon
06/03/2023 17:39, Bruce Richardson: > On Mon, Mar 06, 2023 at 05:13:27PM +0100, Thomas Monjalon wrote: > > -if dpdk_conf.get('RTE_IOVA_AS_PA') == 0 and not > > pmd_supports_disable_iova_as_pa and not always_enable.contains(drv_path) > > +if not get_option('enable_iova_as_pa

Re: [PATCH v2 1/2] build: clarify configuration without IOVA field in mbuf

2023-03-06 Thread Bruce Richardson
On Mon, Mar 06, 2023 at 05:13:27PM +0100, Thomas Monjalon wrote: > The impact of the option "enable_iova_as_pa" is explained for users. > > Also the code flag "RTE_IOVA_AS_PA" is renamed as "RTE_IOVA_IN_MBUF" > in order to be more accurate (IOVA mode is decided at runtime), > and more readable in

RE: [PATCH v2 1/2] build: clarify configuration without IOVA field in mbuf

2023-03-06 Thread Morten Brørup
> From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Monday, 6 March 2023 17.13 > > The impact of the option "enable_iova_as_pa" is explained for users. > > Also the code flag "RTE_IOVA_AS_PA" is renamed as "RTE_IOVA_IN_MBUF" > in order to be more accurate (IOVA mode is decided at runtime