Re: [dpdk-dev] [PATCH v3] test: remove meson dependency on /proc file

2020-04-15 Thread Thomas Monjalon
10/04/2020 22:47, Lukasz Wojciechowski: > W dniu 10.04.2020 o 16:41, Thomas Monjalon pisze: > > 10/04/2020 15:25, Bruce Richardson: > >> On Fri, Apr 10, 2020 at 02:27:30PM +0200, Thomas Monjalon wrote: > >>> 10/04/2020 12:42, Bruce Richardson: > On Fri, Apr 10, 2020 at 12:29:50PM +0200, Thomas

Re: [dpdk-dev] [PATCH v3] test: remove meson dependency on /proc file

2020-04-10 Thread Stephen Hemminger
On Fri, 10 Apr 2020 22:47:44 +0200 Lukasz Wojciechowski wrote: > W dniu 10.04.2020 o 16:41, Thomas Monjalon pisze: > > 10/04/2020 15:25, Bruce Richardson: > >> On Fri, Apr 10, 2020 at 02:27:30PM +0200, Thomas Monjalon wrote: > >>> 10/04/2020 12:42, Bruce Richardson: > On Fri, Apr 10, 2

Re: [dpdk-dev] [PATCH v3] test: remove meson dependency on /proc file

2020-04-10 Thread Lukasz Wojciechowski
W dniu 10.04.2020 o 16:41, Thomas Monjalon pisze: > 10/04/2020 15:25, Bruce Richardson: >> On Fri, Apr 10, 2020 at 02:27:30PM +0200, Thomas Monjalon wrote: >>> 10/04/2020 12:42, Bruce Richardson: On Fri, Apr 10, 2020 at 12:29:50PM +0200, Thomas Monjalon wrote: > --- a/app/test/meson.buil

Re: [dpdk-dev] [PATCH v3] test: remove meson dependency on /proc file

2020-04-10 Thread Thomas Monjalon
10/04/2020 15:25, Bruce Richardson: > On Fri, Apr 10, 2020 at 02:27:30PM +0200, Thomas Monjalon wrote: > > 10/04/2020 12:42, Bruce Richardson: > > > On Fri, Apr 10, 2020 at 12:29:50PM +0200, Thomas Monjalon wrote: > > > > --- a/app/test/meson.build > > > > +++ b/app/test/meson.build > > > > has_hu

Re: [dpdk-dev] [PATCH v3] test: remove meson dependency on /proc file

2020-04-10 Thread Aaron Conole
Thomas Monjalon writes: > Meson is detecting the path /proc/sys/vm/nr_hugepages in the call to cat > in app/test/meson.build and then adding it as a build dependency. > This causes build loop if the timestamp of this file keeps changing. > > It is fixed by hiding hugepage check in a shell script.

Re: [dpdk-dev] [PATCH v3] test: remove meson dependency on /proc file

2020-04-10 Thread Bruce Richardson
On Fri, Apr 10, 2020 at 02:27:30PM +0200, Thomas Monjalon wrote: > 10/04/2020 12:42, Bruce Richardson: > > On Fri, Apr 10, 2020 at 12:29:50PM +0200, Thomas Monjalon wrote: > > > --- a/app/test/meson.build > > > +++ b/app/test/meson.build > > > has_hugepage = true > > > if is_linux > > > > Since

Re: [dpdk-dev] [PATCH v3] test: remove meson dependency on /proc file

2020-04-10 Thread Thomas Monjalon
10/04/2020 12:42, Bruce Richardson: > On Fri, Apr 10, 2020 at 12:29:50PM +0200, Thomas Monjalon wrote: > > --- a/app/test/meson.build > > +++ b/app/test/meson.build > > has_hugepage = true > > if is_linux > > Since you check for linux in the script, you can drop these two lines, The issue is fo

Re: [dpdk-dev] [PATCH v3] test: remove meson dependency on /proc file

2020-04-10 Thread Bruce Richardson
On Fri, Apr 10, 2020 at 12:29:50PM +0200, Thomas Monjalon wrote: > Meson is detecting the path /proc/sys/vm/nr_hugepages in the call to cat > in app/test/meson.build and then adding it as a build dependency. > This causes build loop if the timestamp of this file keeps changing. > > It is fixed by

[dpdk-dev] [PATCH v3] test: remove meson dependency on /proc file

2020-04-10 Thread Thomas Monjalon
Meson is detecting the path /proc/sys/vm/nr_hugepages in the call to cat in app/test/meson.build and then adding it as a build dependency. This causes build loop if the timestamp of this file keeps changing. It is fixed by hiding hugepage check in a shell script. Fixes: 77784ef0fba8 ("test: allow