Re: [dpdk-dev] [PATCH v6 07/17] drivers: replace page size definitions with function

2021-03-01 Thread Andrew Boyer
> On Feb 28, 2021, at 7:53 AM, Thomas Monjalon wrote: > > The page size is often retrieved from the macro PAGE_SIZE. > If PAGE_SIZE is not defined, it is either using hard coded default, > or getting the system value from the UNIX-only function sysconf(). > > Such definitions are replaced with

Re: [dpdk-dev] [PATCH v6 07/17] drivers: replace page size definitions with function

2021-03-01 Thread Bruce Richardson
On Mon, Mar 01, 2021 at 10:53:46AM +0100, Thomas Monjalon wrote: > 01/03/2021 10:41, Bruce Richardson: > > On Sun, Feb 28, 2021 at 01:53:42PM +0100, Thomas Monjalon wrote: > > > The page size is often retrieved from the macro PAGE_SIZE. > > > If PAGE_SIZE is not defined, it is either using hard cod

Re: [dpdk-dev] [PATCH v6 07/17] drivers: replace page size definitions with function

2021-03-01 Thread Thomas Monjalon
01/03/2021 10:41, Bruce Richardson: > On Sun, Feb 28, 2021 at 01:53:42PM +0100, Thomas Monjalon wrote: > > The page size is often retrieved from the macro PAGE_SIZE. > > If PAGE_SIZE is not defined, it is either using hard coded default, > > or getting the system value from the UNIX-only function s

Re: [dpdk-dev] [PATCH v6 07/17] drivers: replace page size definitions with function

2021-03-01 Thread Bruce Richardson
On Sun, Feb 28, 2021 at 01:53:42PM +0100, Thomas Monjalon wrote: > The page size is often retrieved from the macro PAGE_SIZE. > If PAGE_SIZE is not defined, it is either using hard coded default, > or getting the system value from the UNIX-only function sysconf(). > > Such definitions are replaced

[dpdk-dev] [PATCH v6 07/17] drivers: replace page size definitions with function

2021-02-28 Thread Thomas Monjalon
The page size is often retrieved from the macro PAGE_SIZE. If PAGE_SIZE is not defined, it is either using hard coded default, or getting the system value from the UNIX-only function sysconf(). Such definitions are replaced with the generic function rte_mem_page_size() defined for each supported O