DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] eal: remove exec-env directory
@ 2019-04-01 23:07 Thomas Monjalon
  2019-04-01 23:07 ` Thomas Monjalon
                   ` (4 more replies)
  0 siblings, 5 replies; 34+ messages in thread
From: Thomas Monjalon @ 2019-04-01 23:07 UTC (permalink / raw)
  To: dev; +Cc: bruce.richardson, david.marchand, anand.rawat

Only one header file (rte_kni_common.h) was in the sub-directory
	include/exec-env/
This file was installed in a sub-directory of the same name.
Source and install directories are moved as below:

   lib/librte_eal/linux/eal/include/exec-env/
-> lib/librte_eal/linux/eal/include/

   build/include/exec-env/
-> build/include/

The consequence is to have a file hierarchy a bit more flat.

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
---
 devtools/check-includes.sh                                     | 1 -
 kernel/linux/kni/kni_dev.h                                     | 2 +-
 kernel/linux/kni/kni_fifo.h                                    | 2 +-
 kernel/linux/kni/kni_misc.c                                    | 2 +-
 kernel/linux/kni/kni_net.c                                     | 2 +-
 lib/librte_eal/freebsd/eal/Makefile                            | 3 +--
 lib/librte_eal/linux/eal/Makefile                              | 3 +--
 .../linux/eal/include/{exec-env => }/rte_kni_common.h          | 0
 lib/librte_kni/rte_kni.c                                       | 2 +-
 lib/librte_kni/rte_kni.h                                       | 2 +-
 10 files changed, 8 insertions(+), 11 deletions(-)
 rename lib/librte_eal/linux/eal/include/{exec-env => }/rte_kni_common.h (100%)

diff --git a/devtools/check-includes.sh b/devtools/check-includes.sh
index ba9d00ba3..3629ab4c7 100755
--- a/devtools/check-includes.sh
+++ b/devtools/check-includes.sh
@@ -80,7 +80,6 @@ include_dir=${1:-build/include}
 	'rte_byteorder_32.h' \
 	'rte_byteorder_64.h' \
 	'generic/*' \
-	'exec-env/*' \
 	'rte_vhost.h' \
 	'rte_eth_vhost.h' \
 	'rte_eal_interrupts.h' \
diff --git a/kernel/linux/kni/kni_dev.h b/kernel/linux/kni/kni_dev.h
index 688f574a4..df46aa70e 100644
--- a/kernel/linux/kni/kni_dev.h
+++ b/kernel/linux/kni/kni_dev.h
@@ -24,7 +24,7 @@
 #include <linux/spinlock.h>
 #include <linux/list.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #define KNI_KTHREAD_RESCHEDULE_INTERVAL 5 /* us */
 
 #define MBUF_BURST_SZ 32
diff --git a/kernel/linux/kni/kni_fifo.h b/kernel/linux/kni/kni_fifo.h
index 3f4781c2e..5c91b5537 100644
--- a/kernel/linux/kni/kni_fifo.h
+++ b/kernel/linux/kni/kni_fifo.h
@@ -6,7 +6,7 @@
 #ifndef _KNI_FIFO_H_
 #define _KNI_FIFO_H_
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 /* Skip some memory barriers on Linux < 3.14 */
 #ifndef smp_load_acquire
diff --git a/kernel/linux/kni/kni_misc.c b/kernel/linux/kni/kni_misc.c
index 04c78eb87..31845e10f 100644
--- a/kernel/linux/kni/kni_misc.c
+++ b/kernel/linux/kni/kni_misc.c
@@ -16,7 +16,7 @@
 #include <net/net_namespace.h>
 #include <net/netns/generic.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 #include "compat.h"
 #include "kni_dev.h"
diff --git a/kernel/linux/kni/kni_net.c b/kernel/linux/kni/kni_net.c
index 7371b6d58..be9e6b0b9 100644
--- a/kernel/linux/kni/kni_net.c
+++ b/kernel/linux/kni/kni_net.c
@@ -17,7 +17,7 @@
 #include <linux/kthread.h>
 #include <linux/delay.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #include <kni_fifo.h>
 
 #include "compat.h"
diff --git a/lib/librte_eal/freebsd/eal/Makefile b/lib/librte_eal/freebsd/eal/Makefile
index 5d5abf1bb..55d476e5b 100644
--- a/lib/librte_eal/freebsd/eal/Makefile
+++ b/lib/librte_eal/freebsd/eal/Makefile
@@ -88,7 +88,6 @@ endif
 
 INC :=  # no bsd specific headers
 
-SYMLINK-$(CONFIG_RTE_EXEC_ENV_FREEBSD)-include/exec-env := \
-	$(addprefix include/exec-env/,$(INC))
+SYMLINK-$(CONFIG_RTE_EXEC_ENV_FREEBSD)-include := $(addprefix include/,$(INC))
 
 include $(RTE_SDK)/mk/rte.lib.mk
diff --git a/lib/librte_eal/linux/eal/Makefile b/lib/librte_eal/linux/eal/Makefile
index 932c93309..2aa6e8c6a 100644
--- a/lib/librte_eal/linux/eal/Makefile
+++ b/lib/librte_eal/linux/eal/Makefile
@@ -95,7 +95,6 @@ endif
 
 INC := rte_kni_common.h
 
-SYMLINK-$(CONFIG_RTE_EXEC_ENV_LINUX)-include/exec-env := \
-	$(addprefix include/exec-env/,$(INC))
+SYMLINK-$(CONFIG_RTE_EXEC_ENV_LINUX)-include := $(addprefix include/,$(INC))
 
 include $(RTE_SDK)/mk/rte.lib.mk
diff --git a/lib/librte_eal/linux/eal/include/exec-env/rte_kni_common.h b/lib/librte_eal/linux/eal/include/rte_kni_common.h
similarity index 100%
rename from lib/librte_eal/linux/eal/include/exec-env/rte_kni_common.h
rename to lib/librte_eal/linux/eal/include/rte_kni_common.h
diff --git a/lib/librte_kni/rte_kni.c b/lib/librte_kni/rte_kni.c
index 492e207a3..192f2fed0 100644
--- a/lib/librte_kni/rte_kni.c
+++ b/lib/librte_kni/rte_kni.c
@@ -21,7 +21,7 @@
 #include <rte_tailq.h>
 #include <rte_rwlock.h>
 #include <rte_eal_memconfig.h>
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #include "rte_kni_fifo.h"
 
 #define MAX_MBUF_BURST_NUM            32
diff --git a/lib/librte_kni/rte_kni.h b/lib/librte_kni/rte_kni.h
index 02ca43b4b..9a9a7d754 100644
--- a/lib/librte_kni/rte_kni.h
+++ b/lib/librte_kni/rte_kni.h
@@ -22,7 +22,7 @@
 #include <rte_mempool.h>
 #include <rte_ether.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 #ifdef __cplusplus
 extern "C" {
-- 
2.21.0

^ permalink raw reply	[flat|nested] 34+ messages in thread

* [dpdk-dev] [PATCH] eal: remove exec-env directory
  2019-04-01 23:07 [dpdk-dev] [PATCH] eal: remove exec-env directory Thomas Monjalon
@ 2019-04-01 23:07 ` Thomas Monjalon
  2019-04-02  9:39 ` Bruce Richardson
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 34+ messages in thread
From: Thomas Monjalon @ 2019-04-01 23:07 UTC (permalink / raw)
  To: dev; +Cc: bruce.richardson, david.marchand, anand.rawat

Only one header file (rte_kni_common.h) was in the sub-directory
	include/exec-env/
This file was installed in a sub-directory of the same name.
Source and install directories are moved as below:

   lib/librte_eal/linux/eal/include/exec-env/
-> lib/librte_eal/linux/eal/include/

   build/include/exec-env/
-> build/include/

The consequence is to have a file hierarchy a bit more flat.

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
---
 devtools/check-includes.sh                                     | 1 -
 kernel/linux/kni/kni_dev.h                                     | 2 +-
 kernel/linux/kni/kni_fifo.h                                    | 2 +-
 kernel/linux/kni/kni_misc.c                                    | 2 +-
 kernel/linux/kni/kni_net.c                                     | 2 +-
 lib/librte_eal/freebsd/eal/Makefile                            | 3 +--
 lib/librte_eal/linux/eal/Makefile                              | 3 +--
 .../linux/eal/include/{exec-env => }/rte_kni_common.h          | 0
 lib/librte_kni/rte_kni.c                                       | 2 +-
 lib/librte_kni/rte_kni.h                                       | 2 +-
 10 files changed, 8 insertions(+), 11 deletions(-)
 rename lib/librte_eal/linux/eal/include/{exec-env => }/rte_kni_common.h (100%)

diff --git a/devtools/check-includes.sh b/devtools/check-includes.sh
index ba9d00ba3..3629ab4c7 100755
--- a/devtools/check-includes.sh
+++ b/devtools/check-includes.sh
@@ -80,7 +80,6 @@ include_dir=${1:-build/include}
 	'rte_byteorder_32.h' \
 	'rte_byteorder_64.h' \
 	'generic/*' \
-	'exec-env/*' \
 	'rte_vhost.h' \
 	'rte_eth_vhost.h' \
 	'rte_eal_interrupts.h' \
diff --git a/kernel/linux/kni/kni_dev.h b/kernel/linux/kni/kni_dev.h
index 688f574a4..df46aa70e 100644
--- a/kernel/linux/kni/kni_dev.h
+++ b/kernel/linux/kni/kni_dev.h
@@ -24,7 +24,7 @@
 #include <linux/spinlock.h>
 #include <linux/list.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #define KNI_KTHREAD_RESCHEDULE_INTERVAL 5 /* us */
 
 #define MBUF_BURST_SZ 32
diff --git a/kernel/linux/kni/kni_fifo.h b/kernel/linux/kni/kni_fifo.h
index 3f4781c2e..5c91b5537 100644
--- a/kernel/linux/kni/kni_fifo.h
+++ b/kernel/linux/kni/kni_fifo.h
@@ -6,7 +6,7 @@
 #ifndef _KNI_FIFO_H_
 #define _KNI_FIFO_H_
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 /* Skip some memory barriers on Linux < 3.14 */
 #ifndef smp_load_acquire
diff --git a/kernel/linux/kni/kni_misc.c b/kernel/linux/kni/kni_misc.c
index 04c78eb87..31845e10f 100644
--- a/kernel/linux/kni/kni_misc.c
+++ b/kernel/linux/kni/kni_misc.c
@@ -16,7 +16,7 @@
 #include <net/net_namespace.h>
 #include <net/netns/generic.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 #include "compat.h"
 #include "kni_dev.h"
diff --git a/kernel/linux/kni/kni_net.c b/kernel/linux/kni/kni_net.c
index 7371b6d58..be9e6b0b9 100644
--- a/kernel/linux/kni/kni_net.c
+++ b/kernel/linux/kni/kni_net.c
@@ -17,7 +17,7 @@
 #include <linux/kthread.h>
 #include <linux/delay.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #include <kni_fifo.h>
 
 #include "compat.h"
diff --git a/lib/librte_eal/freebsd/eal/Makefile b/lib/librte_eal/freebsd/eal/Makefile
index 5d5abf1bb..55d476e5b 100644
--- a/lib/librte_eal/freebsd/eal/Makefile
+++ b/lib/librte_eal/freebsd/eal/Makefile
@@ -88,7 +88,6 @@ endif
 
 INC :=  # no bsd specific headers
 
-SYMLINK-$(CONFIG_RTE_EXEC_ENV_FREEBSD)-include/exec-env := \
-	$(addprefix include/exec-env/,$(INC))
+SYMLINK-$(CONFIG_RTE_EXEC_ENV_FREEBSD)-include := $(addprefix include/,$(INC))
 
 include $(RTE_SDK)/mk/rte.lib.mk
diff --git a/lib/librte_eal/linux/eal/Makefile b/lib/librte_eal/linux/eal/Makefile
index 932c93309..2aa6e8c6a 100644
--- a/lib/librte_eal/linux/eal/Makefile
+++ b/lib/librte_eal/linux/eal/Makefile
@@ -95,7 +95,6 @@ endif
 
 INC := rte_kni_common.h
 
-SYMLINK-$(CONFIG_RTE_EXEC_ENV_LINUX)-include/exec-env := \
-	$(addprefix include/exec-env/,$(INC))
+SYMLINK-$(CONFIG_RTE_EXEC_ENV_LINUX)-include := $(addprefix include/,$(INC))
 
 include $(RTE_SDK)/mk/rte.lib.mk
diff --git a/lib/librte_eal/linux/eal/include/exec-env/rte_kni_common.h b/lib/librte_eal/linux/eal/include/rte_kni_common.h
similarity index 100%
rename from lib/librte_eal/linux/eal/include/exec-env/rte_kni_common.h
rename to lib/librte_eal/linux/eal/include/rte_kni_common.h
diff --git a/lib/librte_kni/rte_kni.c b/lib/librte_kni/rte_kni.c
index 492e207a3..192f2fed0 100644
--- a/lib/librte_kni/rte_kni.c
+++ b/lib/librte_kni/rte_kni.c
@@ -21,7 +21,7 @@
 #include <rte_tailq.h>
 #include <rte_rwlock.h>
 #include <rte_eal_memconfig.h>
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #include "rte_kni_fifo.h"
 
 #define MAX_MBUF_BURST_NUM            32
diff --git a/lib/librte_kni/rte_kni.h b/lib/librte_kni/rte_kni.h
index 02ca43b4b..9a9a7d754 100644
--- a/lib/librte_kni/rte_kni.h
+++ b/lib/librte_kni/rte_kni.h
@@ -22,7 +22,7 @@
 #include <rte_mempool.h>
 #include <rte_ether.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 #ifdef __cplusplus
 extern "C" {
-- 
2.21.0


^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH] eal: remove exec-env directory
  2019-04-01 23:07 [dpdk-dev] [PATCH] eal: remove exec-env directory Thomas Monjalon
  2019-04-01 23:07 ` Thomas Monjalon
@ 2019-04-02  9:39 ` Bruce Richardson
  2019-04-02  9:39   ` Bruce Richardson
  2019-04-02 14:43   ` Ferruh Yigit
  2019-04-02  9:57 ` David Marchand
                   ` (2 subsequent siblings)
  4 siblings, 2 replies; 34+ messages in thread
From: Bruce Richardson @ 2019-04-02  9:39 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: dev, david.marchand, anand.rawat

On Tue, Apr 02, 2019 at 01:07:12AM +0200, Thomas Monjalon wrote:
> Only one header file (rte_kni_common.h) was in the sub-directory
> 	include/exec-env/
> This file was installed in a sub-directory of the same name.
> Source and install directories are moved as below:
> 
>    lib/librte_eal/linux/eal/include/exec-env/
> -> lib/librte_eal/linux/eal/include/
> 
>    build/include/exec-env/
> -> build/include/
> 
> The consequence is to have a file hierarchy a bit more flat.
> 
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> ---
>  devtools/check-includes.sh                                     | 1 -
>  kernel/linux/kni/kni_dev.h                                     | 2 +-
>  kernel/linux/kni/kni_fifo.h                                    | 2 +-
>  kernel/linux/kni/kni_misc.c                                    | 2 +-
>  kernel/linux/kni/kni_net.c                                     | 2 +-
>  lib/librte_eal/freebsd/eal/Makefile                            | 3 +--
>  lib/librte_eal/linux/eal/Makefile                              | 3 +--
>  .../linux/eal/include/{exec-env => }/rte_kni_common.h          | 0
>  lib/librte_kni/rte_kni.c                                       | 2 +-
>  lib/librte_kni/rte_kni.h                                       | 2 +-
>  10 files changed, 8 insertions(+), 11 deletions(-)
>  rename lib/librte_eal/linux/eal/include/{exec-env => }/rte_kni_common.h (100%)
> 
+1 for flattening our EAL hierarchy.

Acked-by: Bruce Richardson <bruce.richardson@intel.com>

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH] eal: remove exec-env directory
  2019-04-02  9:39 ` Bruce Richardson
@ 2019-04-02  9:39   ` Bruce Richardson
  2019-04-02 14:43   ` Ferruh Yigit
  1 sibling, 0 replies; 34+ messages in thread
From: Bruce Richardson @ 2019-04-02  9:39 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: dev, david.marchand, anand.rawat

On Tue, Apr 02, 2019 at 01:07:12AM +0200, Thomas Monjalon wrote:
> Only one header file (rte_kni_common.h) was in the sub-directory
> 	include/exec-env/
> This file was installed in a sub-directory of the same name.
> Source and install directories are moved as below:
> 
>    lib/librte_eal/linux/eal/include/exec-env/
> -> lib/librte_eal/linux/eal/include/
> 
>    build/include/exec-env/
> -> build/include/
> 
> The consequence is to have a file hierarchy a bit more flat.
> 
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> ---
>  devtools/check-includes.sh                                     | 1 -
>  kernel/linux/kni/kni_dev.h                                     | 2 +-
>  kernel/linux/kni/kni_fifo.h                                    | 2 +-
>  kernel/linux/kni/kni_misc.c                                    | 2 +-
>  kernel/linux/kni/kni_net.c                                     | 2 +-
>  lib/librte_eal/freebsd/eal/Makefile                            | 3 +--
>  lib/librte_eal/linux/eal/Makefile                              | 3 +--
>  .../linux/eal/include/{exec-env => }/rte_kni_common.h          | 0
>  lib/librte_kni/rte_kni.c                                       | 2 +-
>  lib/librte_kni/rte_kni.h                                       | 2 +-
>  10 files changed, 8 insertions(+), 11 deletions(-)
>  rename lib/librte_eal/linux/eal/include/{exec-env => }/rte_kni_common.h (100%)
> 
+1 for flattening our EAL hierarchy.

Acked-by: Bruce Richardson <bruce.richardson@intel.com>


^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH] eal: remove exec-env directory
  2019-04-01 23:07 [dpdk-dev] [PATCH] eal: remove exec-env directory Thomas Monjalon
  2019-04-01 23:07 ` Thomas Monjalon
  2019-04-02  9:39 ` Bruce Richardson
@ 2019-04-02  9:57 ` David Marchand
  2019-04-02  9:57   ` David Marchand
  2019-04-02 10:03   ` Thomas Monjalon
  2019-04-02 15:13 ` [dpdk-dev] [PATCH v2] " Thomas Monjalon
  2019-04-02 19:30 ` [dpdk-dev] [PATCH v3] " Thomas Monjalon
  4 siblings, 2 replies; 34+ messages in thread
From: David Marchand @ 2019-04-02  9:57 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: dev, Bruce Richardson, Anand Rawat

On Tue, Apr 2, 2019 at 1:07 AM Thomas Monjalon <thomas@monjalon.net> wrote:

> Only one header file (rte_kni_common.h) was in the sub-directory
>         include/exec-env/
> This file was installed in a sub-directory of the same name.
> Source and install directories are moved as below:
>
>    lib/librte_eal/linux/eal/include/exec-env/
> -> lib/librte_eal/linux/eal/include/
>
>    build/include/exec-env/
> -> build/include/
>
> The consequence is to have a file hierarchy a bit more flat.
>
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> ---
>  devtools/check-includes.sh                                     | 1 -
>  kernel/linux/kni/kni_dev.h                                     | 2 +-
>  kernel/linux/kni/kni_fifo.h                                    | 2 +-
>  kernel/linux/kni/kni_misc.c                                    | 2 +-
>  kernel/linux/kni/kni_net.c                                     | 2 +-
>  lib/librte_eal/freebsd/eal/Makefile                            | 3 +--
>  lib/librte_eal/linux/eal/Makefile                              | 3 +--
>  .../linux/eal/include/{exec-env => }/rte_kni_common.h          | 0
>  lib/librte_kni/rte_kni.c                                       | 2 +-
>  lib/librte_kni/rte_kni.h                                       | 2 +-
>  10 files changed, 8 insertions(+), 11 deletions(-)
>  rename lib/librte_eal/linux/eal/include/{exec-env => }/rte_kni_common.h
> (100%)
>

Think you missed some bits in meson.
lib/librte_eal/linux/eal/meson.build:install_subdir('include/exec-env',
install_dir: get_option('includedir'))


> diff --git a/devtools/check-includes.sh b/devtools/check-includes.sh
> index ba9d00ba3..3629ab4c7 100755
> --- a/devtools/check-includes.sh
> +++ b/devtools/check-includes.sh
> @@ -80,7 +80,6 @@ include_dir=${1:-build/include}
>         'rte_byteorder_32.h' \
>         'rte_byteorder_64.h' \
>         'generic/*' \
> -       'exec-env/*' \
>         'rte_vhost.h' \
>         'rte_eth_vhost.h' \
>         'rte_eal_interrupts.h' \
> diff --git a/kernel/linux/kni/kni_dev.h b/kernel/linux/kni/kni_dev.h
> index 688f574a4..df46aa70e 100644
> --- a/kernel/linux/kni/kni_dev.h
> +++ b/kernel/linux/kni/kni_dev.h
> @@ -24,7 +24,7 @@
>  #include <linux/spinlock.h>
>  #include <linux/list.h>
>
> -#include <exec-env/rte_kni_common.h>
> +#include <rte_kni_common.h>
>  #define KNI_KTHREAD_RESCHEDULE_INTERVAL 5 /* us */
>
>  #define MBUF_BURST_SZ 32
> diff --git a/kernel/linux/kni/kni_fifo.h b/kernel/linux/kni/kni_fifo.h
> index 3f4781c2e..5c91b5537 100644
> --- a/kernel/linux/kni/kni_fifo.h
> +++ b/kernel/linux/kni/kni_fifo.h
> @@ -6,7 +6,7 @@
>  #ifndef _KNI_FIFO_H_
>  #define _KNI_FIFO_H_
>
> -#include <exec-env/rte_kni_common.h>
> +#include <rte_kni_common.h>
>
>  /* Skip some memory barriers on Linux < 3.14 */
>  #ifndef smp_load_acquire
> diff --git a/kernel/linux/kni/kni_misc.c b/kernel/linux/kni/kni_misc.c
> index 04c78eb87..31845e10f 100644
> --- a/kernel/linux/kni/kni_misc.c
> +++ b/kernel/linux/kni/kni_misc.c
> @@ -16,7 +16,7 @@
>  #include <net/net_namespace.h>
>  #include <net/netns/generic.h>
>
> -#include <exec-env/rte_kni_common.h>
> +#include <rte_kni_common.h>
>
>  #include "compat.h"
>  #include "kni_dev.h"
> diff --git a/kernel/linux/kni/kni_net.c b/kernel/linux/kni/kni_net.c
> index 7371b6d58..be9e6b0b9 100644
> --- a/kernel/linux/kni/kni_net.c
> +++ b/kernel/linux/kni/kni_net.c
> @@ -17,7 +17,7 @@
>  #include <linux/kthread.h>
>  #include <linux/delay.h>
>
> -#include <exec-env/rte_kni_common.h>
> +#include <rte_kni_common.h>
>  #include <kni_fifo.h>
>
>  #include "compat.h"
> diff --git a/lib/librte_eal/freebsd/eal/Makefile
> b/lib/librte_eal/freebsd/eal/Makefile
> index 5d5abf1bb..55d476e5b 100644
> --- a/lib/librte_eal/freebsd/eal/Makefile
> +++ b/lib/librte_eal/freebsd/eal/Makefile
> @@ -88,7 +88,6 @@ endif
>
>  INC :=  # no bsd specific headers
>
> -SYMLINK-$(CONFIG_RTE_EXEC_ENV_FREEBSD)-include/exec-env := \
> -       $(addprefix include/exec-env/,$(INC))
> +SYMLINK-$(CONFIG_RTE_EXEC_ENV_FREEBSD)-include := $(addprefix
> include/,$(INC))
>
>  include $(RTE_SDK)/mk/rte.lib.mk
> diff --git a/lib/librte_eal/linux/eal/Makefile
> b/lib/librte_eal/linux/eal/Makefile
> index 932c93309..2aa6e8c6a 100644
> --- a/lib/librte_eal/linux/eal/Makefile
> +++ b/lib/librte_eal/linux/eal/Makefile
> @@ -95,7 +95,6 @@ endif
>
>  INC := rte_kni_common.h
>
> -SYMLINK-$(CONFIG_RTE_EXEC_ENV_LINUX)-include/exec-env := \
> -       $(addprefix include/exec-env/,$(INC))
> +SYMLINK-$(CONFIG_RTE_EXEC_ENV_LINUX)-include := $(addprefix
> include/,$(INC))
>
>  include $(RTE_SDK)/mk/rte.lib.mk
> diff --git a/lib/librte_eal/linux/eal/include/exec-env/rte_kni_common.h
> b/lib/librte_eal/linux/eal/include/rte_kni_common.h
> similarity index 100%
> rename from lib/librte_eal/linux/eal/include/exec-env/rte_kni_common.h
> rename to lib/librte_eal/linux/eal/include/rte_kni_common.h
> diff --git a/lib/librte_kni/rte_kni.c b/lib/librte_kni/rte_kni.c
> index 492e207a3..192f2fed0 100644
> --- a/lib/librte_kni/rte_kni.c
> +++ b/lib/librte_kni/rte_kni.c
> @@ -21,7 +21,7 @@
>  #include <rte_tailq.h>
>  #include <rte_rwlock.h>
>  #include <rte_eal_memconfig.h>
> -#include <exec-env/rte_kni_common.h>
> +#include <rte_kni_common.h>
>  #include "rte_kni_fifo.h"
>
>  #define MAX_MBUF_BURST_NUM            32
> diff --git a/lib/librte_kni/rte_kni.h b/lib/librte_kni/rte_kni.h
> index 02ca43b4b..9a9a7d754 100644
> --- a/lib/librte_kni/rte_kni.h
> +++ b/lib/librte_kni/rte_kni.h
> @@ -22,7 +22,7 @@
>  #include <rte_mempool.h>
>  #include <rte_ether.h>
>
> -#include <exec-env/rte_kni_common.h>
> +#include <rte_kni_common.h>
>
>  #ifdef __cplusplus
>  extern "C" {
> --
> 2.21.0
>
>
The rest lgtm.

-- 
David Marchand

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH] eal: remove exec-env directory
  2019-04-02  9:57 ` David Marchand
@ 2019-04-02  9:57   ` David Marchand
  2019-04-02 10:03   ` Thomas Monjalon
  1 sibling, 0 replies; 34+ messages in thread
From: David Marchand @ 2019-04-02  9:57 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: dev, Bruce Richardson, Anand Rawat

On Tue, Apr 2, 2019 at 1:07 AM Thomas Monjalon <thomas@monjalon.net> wrote:

> Only one header file (rte_kni_common.h) was in the sub-directory
>         include/exec-env/
> This file was installed in a sub-directory of the same name.
> Source and install directories are moved as below:
>
>    lib/librte_eal/linux/eal/include/exec-env/
> -> lib/librte_eal/linux/eal/include/
>
>    build/include/exec-env/
> -> build/include/
>
> The consequence is to have a file hierarchy a bit more flat.
>
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> ---
>  devtools/check-includes.sh                                     | 1 -
>  kernel/linux/kni/kni_dev.h                                     | 2 +-
>  kernel/linux/kni/kni_fifo.h                                    | 2 +-
>  kernel/linux/kni/kni_misc.c                                    | 2 +-
>  kernel/linux/kni/kni_net.c                                     | 2 +-
>  lib/librte_eal/freebsd/eal/Makefile                            | 3 +--
>  lib/librte_eal/linux/eal/Makefile                              | 3 +--
>  .../linux/eal/include/{exec-env => }/rte_kni_common.h          | 0
>  lib/librte_kni/rte_kni.c                                       | 2 +-
>  lib/librte_kni/rte_kni.h                                       | 2 +-
>  10 files changed, 8 insertions(+), 11 deletions(-)
>  rename lib/librte_eal/linux/eal/include/{exec-env => }/rte_kni_common.h
> (100%)
>

Think you missed some bits in meson.
lib/librte_eal/linux/eal/meson.build:install_subdir('include/exec-env',
install_dir: get_option('includedir'))


> diff --git a/devtools/check-includes.sh b/devtools/check-includes.sh
> index ba9d00ba3..3629ab4c7 100755
> --- a/devtools/check-includes.sh
> +++ b/devtools/check-includes.sh
> @@ -80,7 +80,6 @@ include_dir=${1:-build/include}
>         'rte_byteorder_32.h' \
>         'rte_byteorder_64.h' \
>         'generic/*' \
> -       'exec-env/*' \
>         'rte_vhost.h' \
>         'rte_eth_vhost.h' \
>         'rte_eal_interrupts.h' \
> diff --git a/kernel/linux/kni/kni_dev.h b/kernel/linux/kni/kni_dev.h
> index 688f574a4..df46aa70e 100644
> --- a/kernel/linux/kni/kni_dev.h
> +++ b/kernel/linux/kni/kni_dev.h
> @@ -24,7 +24,7 @@
>  #include <linux/spinlock.h>
>  #include <linux/list.h>
>
> -#include <exec-env/rte_kni_common.h>
> +#include <rte_kni_common.h>
>  #define KNI_KTHREAD_RESCHEDULE_INTERVAL 5 /* us */
>
>  #define MBUF_BURST_SZ 32
> diff --git a/kernel/linux/kni/kni_fifo.h b/kernel/linux/kni/kni_fifo.h
> index 3f4781c2e..5c91b5537 100644
> --- a/kernel/linux/kni/kni_fifo.h
> +++ b/kernel/linux/kni/kni_fifo.h
> @@ -6,7 +6,7 @@
>  #ifndef _KNI_FIFO_H_
>  #define _KNI_FIFO_H_
>
> -#include <exec-env/rte_kni_common.h>
> +#include <rte_kni_common.h>
>
>  /* Skip some memory barriers on Linux < 3.14 */
>  #ifndef smp_load_acquire
> diff --git a/kernel/linux/kni/kni_misc.c b/kernel/linux/kni/kni_misc.c
> index 04c78eb87..31845e10f 100644
> --- a/kernel/linux/kni/kni_misc.c
> +++ b/kernel/linux/kni/kni_misc.c
> @@ -16,7 +16,7 @@
>  #include <net/net_namespace.h>
>  #include <net/netns/generic.h>
>
> -#include <exec-env/rte_kni_common.h>
> +#include <rte_kni_common.h>
>
>  #include "compat.h"
>  #include "kni_dev.h"
> diff --git a/kernel/linux/kni/kni_net.c b/kernel/linux/kni/kni_net.c
> index 7371b6d58..be9e6b0b9 100644
> --- a/kernel/linux/kni/kni_net.c
> +++ b/kernel/linux/kni/kni_net.c
> @@ -17,7 +17,7 @@
>  #include <linux/kthread.h>
>  #include <linux/delay.h>
>
> -#include <exec-env/rte_kni_common.h>
> +#include <rte_kni_common.h>
>  #include <kni_fifo.h>
>
>  #include "compat.h"
> diff --git a/lib/librte_eal/freebsd/eal/Makefile
> b/lib/librte_eal/freebsd/eal/Makefile
> index 5d5abf1bb..55d476e5b 100644
> --- a/lib/librte_eal/freebsd/eal/Makefile
> +++ b/lib/librte_eal/freebsd/eal/Makefile
> @@ -88,7 +88,6 @@ endif
>
>  INC :=  # no bsd specific headers
>
> -SYMLINK-$(CONFIG_RTE_EXEC_ENV_FREEBSD)-include/exec-env := \
> -       $(addprefix include/exec-env/,$(INC))
> +SYMLINK-$(CONFIG_RTE_EXEC_ENV_FREEBSD)-include := $(addprefix
> include/,$(INC))
>
>  include $(RTE_SDK)/mk/rte.lib.mk
> diff --git a/lib/librte_eal/linux/eal/Makefile
> b/lib/librte_eal/linux/eal/Makefile
> index 932c93309..2aa6e8c6a 100644
> --- a/lib/librte_eal/linux/eal/Makefile
> +++ b/lib/librte_eal/linux/eal/Makefile
> @@ -95,7 +95,6 @@ endif
>
>  INC := rte_kni_common.h
>
> -SYMLINK-$(CONFIG_RTE_EXEC_ENV_LINUX)-include/exec-env := \
> -       $(addprefix include/exec-env/,$(INC))
> +SYMLINK-$(CONFIG_RTE_EXEC_ENV_LINUX)-include := $(addprefix
> include/,$(INC))
>
>  include $(RTE_SDK)/mk/rte.lib.mk
> diff --git a/lib/librte_eal/linux/eal/include/exec-env/rte_kni_common.h
> b/lib/librte_eal/linux/eal/include/rte_kni_common.h
> similarity index 100%
> rename from lib/librte_eal/linux/eal/include/exec-env/rte_kni_common.h
> rename to lib/librte_eal/linux/eal/include/rte_kni_common.h
> diff --git a/lib/librte_kni/rte_kni.c b/lib/librte_kni/rte_kni.c
> index 492e207a3..192f2fed0 100644
> --- a/lib/librte_kni/rte_kni.c
> +++ b/lib/librte_kni/rte_kni.c
> @@ -21,7 +21,7 @@
>  #include <rte_tailq.h>
>  #include <rte_rwlock.h>
>  #include <rte_eal_memconfig.h>
> -#include <exec-env/rte_kni_common.h>
> +#include <rte_kni_common.h>
>  #include "rte_kni_fifo.h"
>
>  #define MAX_MBUF_BURST_NUM            32
> diff --git a/lib/librte_kni/rte_kni.h b/lib/librte_kni/rte_kni.h
> index 02ca43b4b..9a9a7d754 100644
> --- a/lib/librte_kni/rte_kni.h
> +++ b/lib/librte_kni/rte_kni.h
> @@ -22,7 +22,7 @@
>  #include <rte_mempool.h>
>  #include <rte_ether.h>
>
> -#include <exec-env/rte_kni_common.h>
> +#include <rte_kni_common.h>
>
>  #ifdef __cplusplus
>  extern "C" {
> --
> 2.21.0
>
>
The rest lgtm.

-- 
David Marchand

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH] eal: remove exec-env directory
  2019-04-02  9:57 ` David Marchand
  2019-04-02  9:57   ` David Marchand
@ 2019-04-02 10:03   ` Thomas Monjalon
  2019-04-02 10:03     ` Thomas Monjalon
  1 sibling, 1 reply; 34+ messages in thread
From: Thomas Monjalon @ 2019-04-02 10:03 UTC (permalink / raw)
  To: David Marchand; +Cc: dev, Bruce Richardson, Anand Rawat

02/04/2019 11:57, David Marchand:
> On Tue, Apr 2, 2019 at 1:07 AM Thomas Monjalon <thomas@monjalon.net> wrote:
> 
> > Only one header file (rte_kni_common.h) was in the sub-directory
> >         include/exec-env/
> > This file was installed in a sub-directory of the same name.
> > Source and install directories are moved as below:
> >
> >    lib/librte_eal/linux/eal/include/exec-env/
> > -> lib/librte_eal/linux/eal/include/
> >
> >    build/include/exec-env/
> > -> build/include/
> >
> > The consequence is to have a file hierarchy a bit more flat.
> >
> > Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> > ---
> >  devtools/check-includes.sh                                     | 1 -
> >  kernel/linux/kni/kni_dev.h                                     | 2 +-
> >  kernel/linux/kni/kni_fifo.h                                    | 2 +-
> >  kernel/linux/kni/kni_misc.c                                    | 2 +-
> >  kernel/linux/kni/kni_net.c                                     | 2 +-
> >  lib/librte_eal/freebsd/eal/Makefile                            | 3 +--
> >  lib/librte_eal/linux/eal/Makefile                              | 3 +--
> >  .../linux/eal/include/{exec-env => }/rte_kni_common.h          | 0
> >  lib/librte_kni/rte_kni.c                                       | 2 +-
> >  lib/librte_kni/rte_kni.h                                       | 2 +-
> >  10 files changed, 8 insertions(+), 11 deletions(-)
> >  rename lib/librte_eal/linux/eal/include/{exec-env => }/rte_kni_common.h
> > (100%)
> >
> 
> Think you missed some bits in meson.
> lib/librte_eal/linux/eal/meson.build:install_subdir('include/exec-env',
> install_dir: get_option('includedir'))

You're right. I wonder how I have missed it!
In my poor understanding of meson, I think I just have to remove this line.

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH] eal: remove exec-env directory
  2019-04-02 10:03   ` Thomas Monjalon
@ 2019-04-02 10:03     ` Thomas Monjalon
  0 siblings, 0 replies; 34+ messages in thread
From: Thomas Monjalon @ 2019-04-02 10:03 UTC (permalink / raw)
  To: David Marchand; +Cc: dev, Bruce Richardson, Anand Rawat

02/04/2019 11:57, David Marchand:
> On Tue, Apr 2, 2019 at 1:07 AM Thomas Monjalon <thomas@monjalon.net> wrote:
> 
> > Only one header file (rte_kni_common.h) was in the sub-directory
> >         include/exec-env/
> > This file was installed in a sub-directory of the same name.
> > Source and install directories are moved as below:
> >
> >    lib/librte_eal/linux/eal/include/exec-env/
> > -> lib/librte_eal/linux/eal/include/
> >
> >    build/include/exec-env/
> > -> build/include/
> >
> > The consequence is to have a file hierarchy a bit more flat.
> >
> > Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> > ---
> >  devtools/check-includes.sh                                     | 1 -
> >  kernel/linux/kni/kni_dev.h                                     | 2 +-
> >  kernel/linux/kni/kni_fifo.h                                    | 2 +-
> >  kernel/linux/kni/kni_misc.c                                    | 2 +-
> >  kernel/linux/kni/kni_net.c                                     | 2 +-
> >  lib/librte_eal/freebsd/eal/Makefile                            | 3 +--
> >  lib/librte_eal/linux/eal/Makefile                              | 3 +--
> >  .../linux/eal/include/{exec-env => }/rte_kni_common.h          | 0
> >  lib/librte_kni/rte_kni.c                                       | 2 +-
> >  lib/librte_kni/rte_kni.h                                       | 2 +-
> >  10 files changed, 8 insertions(+), 11 deletions(-)
> >  rename lib/librte_eal/linux/eal/include/{exec-env => }/rte_kni_common.h
> > (100%)
> >
> 
> Think you missed some bits in meson.
> lib/librte_eal/linux/eal/meson.build:install_subdir('include/exec-env',
> install_dir: get_option('includedir'))

You're right. I wonder how I have missed it!
In my poor understanding of meson, I think I just have to remove this line.




^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH] eal: remove exec-env directory
  2019-04-02  9:39 ` Bruce Richardson
  2019-04-02  9:39   ` Bruce Richardson
@ 2019-04-02 14:43   ` Ferruh Yigit
  2019-04-02 14:43     ` Ferruh Yigit
  1 sibling, 1 reply; 34+ messages in thread
From: Ferruh Yigit @ 2019-04-02 14:43 UTC (permalink / raw)
  To: Bruce Richardson, Thomas Monjalon; +Cc: dev, david.marchand, anand.rawat

On 4/2/2019 10:39 AM, Bruce Richardson wrote:
> On Tue, Apr 02, 2019 at 01:07:12AM +0200, Thomas Monjalon wrote:
>> Only one header file (rte_kni_common.h) was in the sub-directory
>> 	include/exec-env/
>> This file was installed in a sub-directory of the same name.
>> Source and install directories are moved as below:
>>
>>    lib/librte_eal/linux/eal/include/exec-env/
>> -> lib/librte_eal/linux/eal/include/
>>
>>    build/include/exec-env/
>> -> build/include/
>>
>> The consequence is to have a file hierarchy a bit more flat.
>>
>> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
>> ---
>>  devtools/check-includes.sh                                     | 1 -
>>  kernel/linux/kni/kni_dev.h                                     | 2 +-
>>  kernel/linux/kni/kni_fifo.h                                    | 2 +-
>>  kernel/linux/kni/kni_misc.c                                    | 2 +-
>>  kernel/linux/kni/kni_net.c                                     | 2 +-
>>  lib/librte_eal/freebsd/eal/Makefile                            | 3 +--
>>  lib/librte_eal/linux/eal/Makefile                              | 3 +--
>>  .../linux/eal/include/{exec-env => }/rte_kni_common.h          | 0
>>  lib/librte_kni/rte_kni.c                                       | 2 +-
>>  lib/librte_kni/rte_kni.h                                       | 2 +-
>>  10 files changed, 8 insertions(+), 11 deletions(-)
>>  rename lib/librte_eal/linux/eal/include/{exec-env => }/rte_kni_common.h (100%)
>>
> +1 for flattening our EAL hierarchy.

+1

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH] eal: remove exec-env directory
  2019-04-02 14:43   ` Ferruh Yigit
@ 2019-04-02 14:43     ` Ferruh Yigit
  0 siblings, 0 replies; 34+ messages in thread
From: Ferruh Yigit @ 2019-04-02 14:43 UTC (permalink / raw)
  To: Bruce Richardson, Thomas Monjalon; +Cc: dev, david.marchand, anand.rawat

On 4/2/2019 10:39 AM, Bruce Richardson wrote:
> On Tue, Apr 02, 2019 at 01:07:12AM +0200, Thomas Monjalon wrote:
>> Only one header file (rte_kni_common.h) was in the sub-directory
>> 	include/exec-env/
>> This file was installed in a sub-directory of the same name.
>> Source and install directories are moved as below:
>>
>>    lib/librte_eal/linux/eal/include/exec-env/
>> -> lib/librte_eal/linux/eal/include/
>>
>>    build/include/exec-env/
>> -> build/include/
>>
>> The consequence is to have a file hierarchy a bit more flat.
>>
>> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
>> ---
>>  devtools/check-includes.sh                                     | 1 -
>>  kernel/linux/kni/kni_dev.h                                     | 2 +-
>>  kernel/linux/kni/kni_fifo.h                                    | 2 +-
>>  kernel/linux/kni/kni_misc.c                                    | 2 +-
>>  kernel/linux/kni/kni_net.c                                     | 2 +-
>>  lib/librte_eal/freebsd/eal/Makefile                            | 3 +--
>>  lib/librte_eal/linux/eal/Makefile                              | 3 +--
>>  .../linux/eal/include/{exec-env => }/rte_kni_common.h          | 0
>>  lib/librte_kni/rte_kni.c                                       | 2 +-
>>  lib/librte_kni/rte_kni.h                                       | 2 +-
>>  10 files changed, 8 insertions(+), 11 deletions(-)
>>  rename lib/librte_eal/linux/eal/include/{exec-env => }/rte_kni_common.h (100%)
>>
> +1 for flattening our EAL hierarchy.

+1

^ permalink raw reply	[flat|nested] 34+ messages in thread

* [dpdk-dev] [PATCH v2] eal: remove exec-env directory
  2019-04-01 23:07 [dpdk-dev] [PATCH] eal: remove exec-env directory Thomas Monjalon
                   ` (2 preceding siblings ...)
  2019-04-02  9:57 ` David Marchand
@ 2019-04-02 15:13 ` Thomas Monjalon
  2019-04-02 15:13   ` Thomas Monjalon
  2019-04-02 19:30 ` [dpdk-dev] [PATCH v3] " Thomas Monjalon
  4 siblings, 1 reply; 34+ messages in thread
From: Thomas Monjalon @ 2019-04-02 15:13 UTC (permalink / raw)
  To: dev; +Cc: bruce.richardson, david.marchand, anand.rawat

Only one header file (rte_kni_common.h) was in the sub-directory
	include/exec-env/
This file was installed in a sub-directory of the same name
in the makefile-based build.
Source and install directories are moved as below:

   lib/librte_eal/linux/eal/include/exec-env/
-> lib/librte_eal/linux/eal/include/

   build/include/exec-env/
-> build/include/

The consequence is to have a file hierarchy a bit more flat.

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
---
v2 change: remove subdir install with meson on Linux
---
 devtools/check-includes.sh                                     | 1 -
 kernel/linux/kni/kni_dev.h                                     | 2 +-
 kernel/linux/kni/kni_fifo.h                                    | 2 +-
 kernel/linux/kni/kni_misc.c                                    | 2 +-
 kernel/linux/kni/kni_net.c                                     | 2 +-
 lib/librte_eal/freebsd/eal/Makefile                            | 3 +--
 lib/librte_eal/linux/eal/Makefile                              | 3 +--
 .../linux/eal/include/{exec-env => }/rte_kni_common.h          | 0
 lib/librte_eal/linux/eal/meson.build                           | 1 -
 lib/librte_kni/rte_kni.c                                       | 2 +-
 lib/librte_kni/rte_kni.h                                       | 2 +-
 11 files changed, 8 insertions(+), 12 deletions(-)
 rename lib/librte_eal/linux/eal/include/{exec-env => }/rte_kni_common.h (100%)

diff --git a/devtools/check-includes.sh b/devtools/check-includes.sh
index ba9d00ba3..3629ab4c7 100755
--- a/devtools/check-includes.sh
+++ b/devtools/check-includes.sh
@@ -80,7 +80,6 @@ include_dir=${1:-build/include}
 	'rte_byteorder_32.h' \
 	'rte_byteorder_64.h' \
 	'generic/*' \
-	'exec-env/*' \
 	'rte_vhost.h' \
 	'rte_eth_vhost.h' \
 	'rte_eal_interrupts.h' \
diff --git a/kernel/linux/kni/kni_dev.h b/kernel/linux/kni/kni_dev.h
index 688f574a4..df46aa70e 100644
--- a/kernel/linux/kni/kni_dev.h
+++ b/kernel/linux/kni/kni_dev.h
@@ -24,7 +24,7 @@
 #include <linux/spinlock.h>
 #include <linux/list.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #define KNI_KTHREAD_RESCHEDULE_INTERVAL 5 /* us */
 
 #define MBUF_BURST_SZ 32
diff --git a/kernel/linux/kni/kni_fifo.h b/kernel/linux/kni/kni_fifo.h
index 3f4781c2e..5c91b5537 100644
--- a/kernel/linux/kni/kni_fifo.h
+++ b/kernel/linux/kni/kni_fifo.h
@@ -6,7 +6,7 @@
 #ifndef _KNI_FIFO_H_
 #define _KNI_FIFO_H_
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 /* Skip some memory barriers on Linux < 3.14 */
 #ifndef smp_load_acquire
diff --git a/kernel/linux/kni/kni_misc.c b/kernel/linux/kni/kni_misc.c
index 04c78eb87..31845e10f 100644
--- a/kernel/linux/kni/kni_misc.c
+++ b/kernel/linux/kni/kni_misc.c
@@ -16,7 +16,7 @@
 #include <net/net_namespace.h>
 #include <net/netns/generic.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 #include "compat.h"
 #include "kni_dev.h"
diff --git a/kernel/linux/kni/kni_net.c b/kernel/linux/kni/kni_net.c
index 7371b6d58..be9e6b0b9 100644
--- a/kernel/linux/kni/kni_net.c
+++ b/kernel/linux/kni/kni_net.c
@@ -17,7 +17,7 @@
 #include <linux/kthread.h>
 #include <linux/delay.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #include <kni_fifo.h>
 
 #include "compat.h"
diff --git a/lib/librte_eal/freebsd/eal/Makefile b/lib/librte_eal/freebsd/eal/Makefile
index 5d5abf1bb..55d476e5b 100644
--- a/lib/librte_eal/freebsd/eal/Makefile
+++ b/lib/librte_eal/freebsd/eal/Makefile
@@ -88,7 +88,6 @@ endif
 
 INC :=  # no bsd specific headers
 
-SYMLINK-$(CONFIG_RTE_EXEC_ENV_FREEBSD)-include/exec-env := \
-	$(addprefix include/exec-env/,$(INC))
+SYMLINK-$(CONFIG_RTE_EXEC_ENV_FREEBSD)-include := $(addprefix include/,$(INC))
 
 include $(RTE_SDK)/mk/rte.lib.mk
diff --git a/lib/librte_eal/linux/eal/Makefile b/lib/librte_eal/linux/eal/Makefile
index 932c93309..2aa6e8c6a 100644
--- a/lib/librte_eal/linux/eal/Makefile
+++ b/lib/librte_eal/linux/eal/Makefile
@@ -95,7 +95,6 @@ endif
 
 INC := rte_kni_common.h
 
-SYMLINK-$(CONFIG_RTE_EXEC_ENV_LINUX)-include/exec-env := \
-	$(addprefix include/exec-env/,$(INC))
+SYMLINK-$(CONFIG_RTE_EXEC_ENV_LINUX)-include := $(addprefix include/,$(INC))
 
 include $(RTE_SDK)/mk/rte.lib.mk
diff --git a/lib/librte_eal/linux/eal/include/exec-env/rte_kni_common.h b/lib/librte_eal/linux/eal/include/rte_kni_common.h
similarity index 100%
rename from lib/librte_eal/linux/eal/include/exec-env/rte_kni_common.h
rename to lib/librte_eal/linux/eal/include/rte_kni_common.h
diff --git a/lib/librte_eal/linux/eal/meson.build b/lib/librte_eal/linux/eal/meson.build
index 7e68b2c0d..396bf8235 100644
--- a/lib/librte_eal/linux/eal/meson.build
+++ b/lib/librte_eal/linux/eal/meson.build
@@ -2,7 +2,6 @@
 # Copyright(c) 2017 Intel Corporation
 
 eal_inc += include_directories('include')
-install_subdir('include/exec-env', install_dir: get_option('includedir'))
 
 env_objs = []
 env_headers = []
diff --git a/lib/librte_kni/rte_kni.c b/lib/librte_kni/rte_kni.c
index 492e207a3..192f2fed0 100644
--- a/lib/librte_kni/rte_kni.c
+++ b/lib/librte_kni/rte_kni.c
@@ -21,7 +21,7 @@
 #include <rte_tailq.h>
 #include <rte_rwlock.h>
 #include <rte_eal_memconfig.h>
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #include "rte_kni_fifo.h"
 
 #define MAX_MBUF_BURST_NUM            32
diff --git a/lib/librte_kni/rte_kni.h b/lib/librte_kni/rte_kni.h
index 02ca43b4b..9a9a7d754 100644
--- a/lib/librte_kni/rte_kni.h
+++ b/lib/librte_kni/rte_kni.h
@@ -22,7 +22,7 @@
 #include <rte_mempool.h>
 #include <rte_ether.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 #ifdef __cplusplus
 extern "C" {
-- 
2.21.0

^ permalink raw reply	[flat|nested] 34+ messages in thread

* [dpdk-dev] [PATCH v2] eal: remove exec-env directory
  2019-04-02 15:13 ` [dpdk-dev] [PATCH v2] " Thomas Monjalon
@ 2019-04-02 15:13   ` Thomas Monjalon
  0 siblings, 0 replies; 34+ messages in thread
From: Thomas Monjalon @ 2019-04-02 15:13 UTC (permalink / raw)
  To: dev; +Cc: bruce.richardson, david.marchand, anand.rawat

Only one header file (rte_kni_common.h) was in the sub-directory
	include/exec-env/
This file was installed in a sub-directory of the same name
in the makefile-based build.
Source and install directories are moved as below:

   lib/librte_eal/linux/eal/include/exec-env/
-> lib/librte_eal/linux/eal/include/

   build/include/exec-env/
-> build/include/

The consequence is to have a file hierarchy a bit more flat.

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
---
v2 change: remove subdir install with meson on Linux
---
 devtools/check-includes.sh                                     | 1 -
 kernel/linux/kni/kni_dev.h                                     | 2 +-
 kernel/linux/kni/kni_fifo.h                                    | 2 +-
 kernel/linux/kni/kni_misc.c                                    | 2 +-
 kernel/linux/kni/kni_net.c                                     | 2 +-
 lib/librte_eal/freebsd/eal/Makefile                            | 3 +--
 lib/librte_eal/linux/eal/Makefile                              | 3 +--
 .../linux/eal/include/{exec-env => }/rte_kni_common.h          | 0
 lib/librte_eal/linux/eal/meson.build                           | 1 -
 lib/librte_kni/rte_kni.c                                       | 2 +-
 lib/librte_kni/rte_kni.h                                       | 2 +-
 11 files changed, 8 insertions(+), 12 deletions(-)
 rename lib/librte_eal/linux/eal/include/{exec-env => }/rte_kni_common.h (100%)

diff --git a/devtools/check-includes.sh b/devtools/check-includes.sh
index ba9d00ba3..3629ab4c7 100755
--- a/devtools/check-includes.sh
+++ b/devtools/check-includes.sh
@@ -80,7 +80,6 @@ include_dir=${1:-build/include}
 	'rte_byteorder_32.h' \
 	'rte_byteorder_64.h' \
 	'generic/*' \
-	'exec-env/*' \
 	'rte_vhost.h' \
 	'rte_eth_vhost.h' \
 	'rte_eal_interrupts.h' \
diff --git a/kernel/linux/kni/kni_dev.h b/kernel/linux/kni/kni_dev.h
index 688f574a4..df46aa70e 100644
--- a/kernel/linux/kni/kni_dev.h
+++ b/kernel/linux/kni/kni_dev.h
@@ -24,7 +24,7 @@
 #include <linux/spinlock.h>
 #include <linux/list.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #define KNI_KTHREAD_RESCHEDULE_INTERVAL 5 /* us */
 
 #define MBUF_BURST_SZ 32
diff --git a/kernel/linux/kni/kni_fifo.h b/kernel/linux/kni/kni_fifo.h
index 3f4781c2e..5c91b5537 100644
--- a/kernel/linux/kni/kni_fifo.h
+++ b/kernel/linux/kni/kni_fifo.h
@@ -6,7 +6,7 @@
 #ifndef _KNI_FIFO_H_
 #define _KNI_FIFO_H_
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 /* Skip some memory barriers on Linux < 3.14 */
 #ifndef smp_load_acquire
diff --git a/kernel/linux/kni/kni_misc.c b/kernel/linux/kni/kni_misc.c
index 04c78eb87..31845e10f 100644
--- a/kernel/linux/kni/kni_misc.c
+++ b/kernel/linux/kni/kni_misc.c
@@ -16,7 +16,7 @@
 #include <net/net_namespace.h>
 #include <net/netns/generic.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 #include "compat.h"
 #include "kni_dev.h"
diff --git a/kernel/linux/kni/kni_net.c b/kernel/linux/kni/kni_net.c
index 7371b6d58..be9e6b0b9 100644
--- a/kernel/linux/kni/kni_net.c
+++ b/kernel/linux/kni/kni_net.c
@@ -17,7 +17,7 @@
 #include <linux/kthread.h>
 #include <linux/delay.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #include <kni_fifo.h>
 
 #include "compat.h"
diff --git a/lib/librte_eal/freebsd/eal/Makefile b/lib/librte_eal/freebsd/eal/Makefile
index 5d5abf1bb..55d476e5b 100644
--- a/lib/librte_eal/freebsd/eal/Makefile
+++ b/lib/librte_eal/freebsd/eal/Makefile
@@ -88,7 +88,6 @@ endif
 
 INC :=  # no bsd specific headers
 
-SYMLINK-$(CONFIG_RTE_EXEC_ENV_FREEBSD)-include/exec-env := \
-	$(addprefix include/exec-env/,$(INC))
+SYMLINK-$(CONFIG_RTE_EXEC_ENV_FREEBSD)-include := $(addprefix include/,$(INC))
 
 include $(RTE_SDK)/mk/rte.lib.mk
diff --git a/lib/librte_eal/linux/eal/Makefile b/lib/librte_eal/linux/eal/Makefile
index 932c93309..2aa6e8c6a 100644
--- a/lib/librte_eal/linux/eal/Makefile
+++ b/lib/librte_eal/linux/eal/Makefile
@@ -95,7 +95,6 @@ endif
 
 INC := rte_kni_common.h
 
-SYMLINK-$(CONFIG_RTE_EXEC_ENV_LINUX)-include/exec-env := \
-	$(addprefix include/exec-env/,$(INC))
+SYMLINK-$(CONFIG_RTE_EXEC_ENV_LINUX)-include := $(addprefix include/,$(INC))
 
 include $(RTE_SDK)/mk/rte.lib.mk
diff --git a/lib/librte_eal/linux/eal/include/exec-env/rte_kni_common.h b/lib/librte_eal/linux/eal/include/rte_kni_common.h
similarity index 100%
rename from lib/librte_eal/linux/eal/include/exec-env/rte_kni_common.h
rename to lib/librte_eal/linux/eal/include/rte_kni_common.h
diff --git a/lib/librte_eal/linux/eal/meson.build b/lib/librte_eal/linux/eal/meson.build
index 7e68b2c0d..396bf8235 100644
--- a/lib/librte_eal/linux/eal/meson.build
+++ b/lib/librte_eal/linux/eal/meson.build
@@ -2,7 +2,6 @@
 # Copyright(c) 2017 Intel Corporation
 
 eal_inc += include_directories('include')
-install_subdir('include/exec-env', install_dir: get_option('includedir'))
 
 env_objs = []
 env_headers = []
diff --git a/lib/librte_kni/rte_kni.c b/lib/librte_kni/rte_kni.c
index 492e207a3..192f2fed0 100644
--- a/lib/librte_kni/rte_kni.c
+++ b/lib/librte_kni/rte_kni.c
@@ -21,7 +21,7 @@
 #include <rte_tailq.h>
 #include <rte_rwlock.h>
 #include <rte_eal_memconfig.h>
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #include "rte_kni_fifo.h"
 
 #define MAX_MBUF_BURST_NUM            32
diff --git a/lib/librte_kni/rte_kni.h b/lib/librte_kni/rte_kni.h
index 02ca43b4b..9a9a7d754 100644
--- a/lib/librte_kni/rte_kni.h
+++ b/lib/librte_kni/rte_kni.h
@@ -22,7 +22,7 @@
 #include <rte_mempool.h>
 #include <rte_ether.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 #ifdef __cplusplus
 extern "C" {
-- 
2.21.0


^ permalink raw reply	[flat|nested] 34+ messages in thread

* [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-01 23:07 [dpdk-dev] [PATCH] eal: remove exec-env directory Thomas Monjalon
                   ` (3 preceding siblings ...)
  2019-04-02 15:13 ` [dpdk-dev] [PATCH v2] " Thomas Monjalon
@ 2019-04-02 19:30 ` Thomas Monjalon
  2019-04-02 19:30   ` Thomas Monjalon
                     ` (2 more replies)
  4 siblings, 3 replies; 34+ messages in thread
From: Thomas Monjalon @ 2019-04-02 19:30 UTC (permalink / raw)
  To: dev; +Cc: bruce.richardson, david.marchand, anand.rawat

Only one header file (rte_kni_common.h) was in the sub-directory
	include/exec-env/
This file was installed in a sub-directory of the same name
in the makefile-based build.
Source and install directories are moved as below:

   lib/librte_eal/linux/eal/include/exec-env/
-> lib/librte_eal/linux/eal/include/

   build/include/exec-env/
-> build/include/

The consequence is to have a file hierarchy a bit more flat.

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
---
v2: remove subdir install with meson on Linux
v3: add rte_kni_common.h in meson install
---
 devtools/check-includes.sh                                     | 1 -
 kernel/linux/kni/kni_dev.h                                     | 2 +-
 kernel/linux/kni/kni_fifo.h                                    | 2 +-
 kernel/linux/kni/kni_misc.c                                    | 2 +-
 kernel/linux/kni/kni_net.c                                     | 2 +-
 lib/librte_eal/freebsd/eal/Makefile                            | 3 +--
 lib/librte_eal/linux/eal/Makefile                              | 3 +--
 .../linux/eal/include/{exec-env => }/rte_kni_common.h          | 0
 lib/librte_eal/linux/eal/meson.build                           | 3 +--
 lib/librte_kni/rte_kni.c                                       | 2 +-
 lib/librte_kni/rte_kni.h                                       | 2 +-
 11 files changed, 9 insertions(+), 13 deletions(-)
 rename lib/librte_eal/linux/eal/include/{exec-env => }/rte_kni_common.h (100%)

diff --git a/devtools/check-includes.sh b/devtools/check-includes.sh
index ba9d00ba3..3629ab4c7 100755
--- a/devtools/check-includes.sh
+++ b/devtools/check-includes.sh
@@ -80,7 +80,6 @@ include_dir=${1:-build/include}
 	'rte_byteorder_32.h' \
 	'rte_byteorder_64.h' \
 	'generic/*' \
-	'exec-env/*' \
 	'rte_vhost.h' \
 	'rte_eth_vhost.h' \
 	'rte_eal_interrupts.h' \
diff --git a/kernel/linux/kni/kni_dev.h b/kernel/linux/kni/kni_dev.h
index 688f574a4..df46aa70e 100644
--- a/kernel/linux/kni/kni_dev.h
+++ b/kernel/linux/kni/kni_dev.h
@@ -24,7 +24,7 @@
 #include <linux/spinlock.h>
 #include <linux/list.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #define KNI_KTHREAD_RESCHEDULE_INTERVAL 5 /* us */
 
 #define MBUF_BURST_SZ 32
diff --git a/kernel/linux/kni/kni_fifo.h b/kernel/linux/kni/kni_fifo.h
index 3f4781c2e..5c91b5537 100644
--- a/kernel/linux/kni/kni_fifo.h
+++ b/kernel/linux/kni/kni_fifo.h
@@ -6,7 +6,7 @@
 #ifndef _KNI_FIFO_H_
 #define _KNI_FIFO_H_
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 /* Skip some memory barriers on Linux < 3.14 */
 #ifndef smp_load_acquire
diff --git a/kernel/linux/kni/kni_misc.c b/kernel/linux/kni/kni_misc.c
index 04c78eb87..31845e10f 100644
--- a/kernel/linux/kni/kni_misc.c
+++ b/kernel/linux/kni/kni_misc.c
@@ -16,7 +16,7 @@
 #include <net/net_namespace.h>
 #include <net/netns/generic.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 #include "compat.h"
 #include "kni_dev.h"
diff --git a/kernel/linux/kni/kni_net.c b/kernel/linux/kni/kni_net.c
index 7371b6d58..be9e6b0b9 100644
--- a/kernel/linux/kni/kni_net.c
+++ b/kernel/linux/kni/kni_net.c
@@ -17,7 +17,7 @@
 #include <linux/kthread.h>
 #include <linux/delay.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #include <kni_fifo.h>
 
 #include "compat.h"
diff --git a/lib/librte_eal/freebsd/eal/Makefile b/lib/librte_eal/freebsd/eal/Makefile
index 5d5abf1bb..55d476e5b 100644
--- a/lib/librte_eal/freebsd/eal/Makefile
+++ b/lib/librte_eal/freebsd/eal/Makefile
@@ -88,7 +88,6 @@ endif
 
 INC :=  # no bsd specific headers
 
-SYMLINK-$(CONFIG_RTE_EXEC_ENV_FREEBSD)-include/exec-env := \
-	$(addprefix include/exec-env/,$(INC))
+SYMLINK-$(CONFIG_RTE_EXEC_ENV_FREEBSD)-include := $(addprefix include/,$(INC))
 
 include $(RTE_SDK)/mk/rte.lib.mk
diff --git a/lib/librte_eal/linux/eal/Makefile b/lib/librte_eal/linux/eal/Makefile
index 932c93309..2aa6e8c6a 100644
--- a/lib/librte_eal/linux/eal/Makefile
+++ b/lib/librte_eal/linux/eal/Makefile
@@ -95,7 +95,6 @@ endif
 
 INC := rte_kni_common.h
 
-SYMLINK-$(CONFIG_RTE_EXEC_ENV_LINUX)-include/exec-env := \
-	$(addprefix include/exec-env/,$(INC))
+SYMLINK-$(CONFIG_RTE_EXEC_ENV_LINUX)-include := $(addprefix include/,$(INC))
 
 include $(RTE_SDK)/mk/rte.lib.mk
diff --git a/lib/librte_eal/linux/eal/include/exec-env/rte_kni_common.h b/lib/librte_eal/linux/eal/include/rte_kni_common.h
similarity index 100%
rename from lib/librte_eal/linux/eal/include/exec-env/rte_kni_common.h
rename to lib/librte_eal/linux/eal/include/rte_kni_common.h
diff --git a/lib/librte_eal/linux/eal/meson.build b/lib/librte_eal/linux/eal/meson.build
index 7e68b2c0d..d223da638 100644
--- a/lib/librte_eal/linux/eal/meson.build
+++ b/lib/librte_eal/linux/eal/meson.build
@@ -2,10 +2,9 @@
 # Copyright(c) 2017 Intel Corporation
 
 eal_inc += include_directories('include')
-install_subdir('include/exec-env', install_dir: get_option('includedir'))
 
 env_objs = []
-env_headers = []
+env_headers = files('include/rte_kni_common.h')
 env_sources = files('eal_alarm.c',
 		'eal_cpuflags.c',
 		'eal_debug.c',
diff --git a/lib/librte_kni/rte_kni.c b/lib/librte_kni/rte_kni.c
index 492e207a3..192f2fed0 100644
--- a/lib/librte_kni/rte_kni.c
+++ b/lib/librte_kni/rte_kni.c
@@ -21,7 +21,7 @@
 #include <rte_tailq.h>
 #include <rte_rwlock.h>
 #include <rte_eal_memconfig.h>
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #include "rte_kni_fifo.h"
 
 #define MAX_MBUF_BURST_NUM            32
diff --git a/lib/librte_kni/rte_kni.h b/lib/librte_kni/rte_kni.h
index 02ca43b4b..9a9a7d754 100644
--- a/lib/librte_kni/rte_kni.h
+++ b/lib/librte_kni/rte_kni.h
@@ -22,7 +22,7 @@
 #include <rte_mempool.h>
 #include <rte_ether.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 #ifdef __cplusplus
 extern "C" {
-- 
2.21.0

^ permalink raw reply	[flat|nested] 34+ messages in thread

* [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 19:30 ` [dpdk-dev] [PATCH v3] " Thomas Monjalon
@ 2019-04-02 19:30   ` Thomas Monjalon
  2019-04-02 19:46   ` David Marchand
  2019-04-02 19:50   ` Ferruh Yigit
  2 siblings, 0 replies; 34+ messages in thread
From: Thomas Monjalon @ 2019-04-02 19:30 UTC (permalink / raw)
  To: dev; +Cc: bruce.richardson, david.marchand, anand.rawat

Only one header file (rte_kni_common.h) was in the sub-directory
	include/exec-env/
This file was installed in a sub-directory of the same name
in the makefile-based build.
Source and install directories are moved as below:

   lib/librte_eal/linux/eal/include/exec-env/
-> lib/librte_eal/linux/eal/include/

   build/include/exec-env/
-> build/include/

The consequence is to have a file hierarchy a bit more flat.

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
---
v2: remove subdir install with meson on Linux
v3: add rte_kni_common.h in meson install
---
 devtools/check-includes.sh                                     | 1 -
 kernel/linux/kni/kni_dev.h                                     | 2 +-
 kernel/linux/kni/kni_fifo.h                                    | 2 +-
 kernel/linux/kni/kni_misc.c                                    | 2 +-
 kernel/linux/kni/kni_net.c                                     | 2 +-
 lib/librte_eal/freebsd/eal/Makefile                            | 3 +--
 lib/librte_eal/linux/eal/Makefile                              | 3 +--
 .../linux/eal/include/{exec-env => }/rte_kni_common.h          | 0
 lib/librte_eal/linux/eal/meson.build                           | 3 +--
 lib/librte_kni/rte_kni.c                                       | 2 +-
 lib/librte_kni/rte_kni.h                                       | 2 +-
 11 files changed, 9 insertions(+), 13 deletions(-)
 rename lib/librte_eal/linux/eal/include/{exec-env => }/rte_kni_common.h (100%)

diff --git a/devtools/check-includes.sh b/devtools/check-includes.sh
index ba9d00ba3..3629ab4c7 100755
--- a/devtools/check-includes.sh
+++ b/devtools/check-includes.sh
@@ -80,7 +80,6 @@ include_dir=${1:-build/include}
 	'rte_byteorder_32.h' \
 	'rte_byteorder_64.h' \
 	'generic/*' \
-	'exec-env/*' \
 	'rte_vhost.h' \
 	'rte_eth_vhost.h' \
 	'rte_eal_interrupts.h' \
diff --git a/kernel/linux/kni/kni_dev.h b/kernel/linux/kni/kni_dev.h
index 688f574a4..df46aa70e 100644
--- a/kernel/linux/kni/kni_dev.h
+++ b/kernel/linux/kni/kni_dev.h
@@ -24,7 +24,7 @@
 #include <linux/spinlock.h>
 #include <linux/list.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #define KNI_KTHREAD_RESCHEDULE_INTERVAL 5 /* us */
 
 #define MBUF_BURST_SZ 32
diff --git a/kernel/linux/kni/kni_fifo.h b/kernel/linux/kni/kni_fifo.h
index 3f4781c2e..5c91b5537 100644
--- a/kernel/linux/kni/kni_fifo.h
+++ b/kernel/linux/kni/kni_fifo.h
@@ -6,7 +6,7 @@
 #ifndef _KNI_FIFO_H_
 #define _KNI_FIFO_H_
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 /* Skip some memory barriers on Linux < 3.14 */
 #ifndef smp_load_acquire
diff --git a/kernel/linux/kni/kni_misc.c b/kernel/linux/kni/kni_misc.c
index 04c78eb87..31845e10f 100644
--- a/kernel/linux/kni/kni_misc.c
+++ b/kernel/linux/kni/kni_misc.c
@@ -16,7 +16,7 @@
 #include <net/net_namespace.h>
 #include <net/netns/generic.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 #include "compat.h"
 #include "kni_dev.h"
diff --git a/kernel/linux/kni/kni_net.c b/kernel/linux/kni/kni_net.c
index 7371b6d58..be9e6b0b9 100644
--- a/kernel/linux/kni/kni_net.c
+++ b/kernel/linux/kni/kni_net.c
@@ -17,7 +17,7 @@
 #include <linux/kthread.h>
 #include <linux/delay.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #include <kni_fifo.h>
 
 #include "compat.h"
diff --git a/lib/librte_eal/freebsd/eal/Makefile b/lib/librte_eal/freebsd/eal/Makefile
index 5d5abf1bb..55d476e5b 100644
--- a/lib/librte_eal/freebsd/eal/Makefile
+++ b/lib/librte_eal/freebsd/eal/Makefile
@@ -88,7 +88,6 @@ endif
 
 INC :=  # no bsd specific headers
 
-SYMLINK-$(CONFIG_RTE_EXEC_ENV_FREEBSD)-include/exec-env := \
-	$(addprefix include/exec-env/,$(INC))
+SYMLINK-$(CONFIG_RTE_EXEC_ENV_FREEBSD)-include := $(addprefix include/,$(INC))
 
 include $(RTE_SDK)/mk/rte.lib.mk
diff --git a/lib/librte_eal/linux/eal/Makefile b/lib/librte_eal/linux/eal/Makefile
index 932c93309..2aa6e8c6a 100644
--- a/lib/librte_eal/linux/eal/Makefile
+++ b/lib/librte_eal/linux/eal/Makefile
@@ -95,7 +95,6 @@ endif
 
 INC := rte_kni_common.h
 
-SYMLINK-$(CONFIG_RTE_EXEC_ENV_LINUX)-include/exec-env := \
-	$(addprefix include/exec-env/,$(INC))
+SYMLINK-$(CONFIG_RTE_EXEC_ENV_LINUX)-include := $(addprefix include/,$(INC))
 
 include $(RTE_SDK)/mk/rte.lib.mk
diff --git a/lib/librte_eal/linux/eal/include/exec-env/rte_kni_common.h b/lib/librte_eal/linux/eal/include/rte_kni_common.h
similarity index 100%
rename from lib/librte_eal/linux/eal/include/exec-env/rte_kni_common.h
rename to lib/librte_eal/linux/eal/include/rte_kni_common.h
diff --git a/lib/librte_eal/linux/eal/meson.build b/lib/librte_eal/linux/eal/meson.build
index 7e68b2c0d..d223da638 100644
--- a/lib/librte_eal/linux/eal/meson.build
+++ b/lib/librte_eal/linux/eal/meson.build
@@ -2,10 +2,9 @@
 # Copyright(c) 2017 Intel Corporation
 
 eal_inc += include_directories('include')
-install_subdir('include/exec-env', install_dir: get_option('includedir'))
 
 env_objs = []
-env_headers = []
+env_headers = files('include/rte_kni_common.h')
 env_sources = files('eal_alarm.c',
 		'eal_cpuflags.c',
 		'eal_debug.c',
diff --git a/lib/librte_kni/rte_kni.c b/lib/librte_kni/rte_kni.c
index 492e207a3..192f2fed0 100644
--- a/lib/librte_kni/rte_kni.c
+++ b/lib/librte_kni/rte_kni.c
@@ -21,7 +21,7 @@
 #include <rte_tailq.h>
 #include <rte_rwlock.h>
 #include <rte_eal_memconfig.h>
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 #include "rte_kni_fifo.h"
 
 #define MAX_MBUF_BURST_NUM            32
diff --git a/lib/librte_kni/rte_kni.h b/lib/librte_kni/rte_kni.h
index 02ca43b4b..9a9a7d754 100644
--- a/lib/librte_kni/rte_kni.h
+++ b/lib/librte_kni/rte_kni.h
@@ -22,7 +22,7 @@
 #include <rte_mempool.h>
 #include <rte_ether.h>
 
-#include <exec-env/rte_kni_common.h>
+#include <rte_kni_common.h>
 
 #ifdef __cplusplus
 extern "C" {
-- 
2.21.0


^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 19:30 ` [dpdk-dev] [PATCH v3] " Thomas Monjalon
  2019-04-02 19:30   ` Thomas Monjalon
@ 2019-04-02 19:46   ` David Marchand
  2019-04-02 19:46     ` David Marchand
                       ` (2 more replies)
  2019-04-02 19:50   ` Ferruh Yigit
  2 siblings, 3 replies; 34+ messages in thread
From: David Marchand @ 2019-04-02 19:46 UTC (permalink / raw)
  To: Thomas Monjalon, Bruce Richardson; +Cc: dev, Anand Rawat

On Tue, Apr 2, 2019 at 9:30 PM Thomas Monjalon <thomas@monjalon.net> wrote:

> Only one header file (rte_kni_common.h) was in the sub-directory
>         include/exec-env/
> This file was installed in a sub-directory of the same name
> in the makefile-based build.
> Source and install directories are moved as below:
>
>    lib/librte_eal/linux/eal/include/exec-env/
> -> lib/librte_eal/linux/eal/include/
>
>    build/include/exec-env/
> -> build/include/
>
> The consequence is to have a file hierarchy a bit more flat.
>
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
>

Looks good to me.
Reviewed-by: David Marchand <david.marchand@redhat.com>
Tested-by: David Marchand <david.marchand@redhat.com>


Bruce, while testing with meson, I noticed this:
kernel/linux/kni/meson.build:16: WARNING: Passed invalid keyword argument
"console".
WARNING: This will become a hard error in the future.
WARNING: Unknown keyword arguments in target rte_kni: console


-- 
David Marchand

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 19:46   ` David Marchand
@ 2019-04-02 19:46     ` David Marchand
  2019-04-02 19:50     ` Thomas Monjalon
  2019-04-02 19:52     ` Ferruh Yigit
  2 siblings, 0 replies; 34+ messages in thread
From: David Marchand @ 2019-04-02 19:46 UTC (permalink / raw)
  To: Thomas Monjalon, Bruce Richardson; +Cc: dev, Anand Rawat

On Tue, Apr 2, 2019 at 9:30 PM Thomas Monjalon <thomas@monjalon.net> wrote:

> Only one header file (rte_kni_common.h) was in the sub-directory
>         include/exec-env/
> This file was installed in a sub-directory of the same name
> in the makefile-based build.
> Source and install directories are moved as below:
>
>    lib/librte_eal/linux/eal/include/exec-env/
> -> lib/librte_eal/linux/eal/include/
>
>    build/include/exec-env/
> -> build/include/
>
> The consequence is to have a file hierarchy a bit more flat.
>
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
>

Looks good to me.
Reviewed-by: David Marchand <david.marchand@redhat.com>
Tested-by: David Marchand <david.marchand@redhat.com>


Bruce, while testing with meson, I noticed this:
kernel/linux/kni/meson.build:16: WARNING: Passed invalid keyword argument
"console".
WARNING: This will become a hard error in the future.
WARNING: Unknown keyword arguments in target rte_kni: console


-- 
David Marchand

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 19:30 ` [dpdk-dev] [PATCH v3] " Thomas Monjalon
  2019-04-02 19:30   ` Thomas Monjalon
  2019-04-02 19:46   ` David Marchand
@ 2019-04-02 19:50   ` Ferruh Yigit
  2019-04-02 19:50     ` Ferruh Yigit
  2 siblings, 1 reply; 34+ messages in thread
From: Ferruh Yigit @ 2019-04-02 19:50 UTC (permalink / raw)
  To: Thomas Monjalon, dev; +Cc: bruce.richardson, david.marchand, anand.rawat

On 4/2/2019 8:30 PM, Thomas Monjalon wrote:
> Only one header file (rte_kni_common.h) was in the sub-directory
> 	include/exec-env/
> This file was installed in a sub-directory of the same name
> in the makefile-based build.
> Source and install directories are moved as below:
> 
>    lib/librte_eal/linux/eal/include/exec-env/
> -> lib/librte_eal/linux/eal/include/
> 
>    build/include/exec-env/
> -> build/include/
> 
> The consequence is to have a file hierarchy a bit more flat.
> 
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>

Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 19:50   ` Ferruh Yigit
@ 2019-04-02 19:50     ` Ferruh Yigit
  0 siblings, 0 replies; 34+ messages in thread
From: Ferruh Yigit @ 2019-04-02 19:50 UTC (permalink / raw)
  To: Thomas Monjalon, dev; +Cc: bruce.richardson, david.marchand, anand.rawat

On 4/2/2019 8:30 PM, Thomas Monjalon wrote:
> Only one header file (rte_kni_common.h) was in the sub-directory
> 	include/exec-env/
> This file was installed in a sub-directory of the same name
> in the makefile-based build.
> Source and install directories are moved as below:
> 
>    lib/librte_eal/linux/eal/include/exec-env/
> -> lib/librte_eal/linux/eal/include/
> 
>    build/include/exec-env/
> -> build/include/
> 
> The consequence is to have a file hierarchy a bit more flat.
> 
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>

Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 19:46   ` David Marchand
  2019-04-02 19:46     ` David Marchand
@ 2019-04-02 19:50     ` Thomas Monjalon
  2019-04-02 19:50       ` Thomas Monjalon
  2019-04-03  8:41       ` David Marchand
  2019-04-02 19:52     ` Ferruh Yigit
  2 siblings, 2 replies; 34+ messages in thread
From: Thomas Monjalon @ 2019-04-02 19:50 UTC (permalink / raw)
  To: dev; +Cc: David Marchand, Bruce Richardson, Anand Rawat

02/04/2019 21:46, David Marchand:
> On Tue, Apr 2, 2019 at 9:30 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> 
> > Only one header file (rte_kni_common.h) was in the sub-directory
> >         include/exec-env/
> > This file was installed in a sub-directory of the same name
> > in the makefile-based build.
> > Source and install directories are moved as below:
> >
> >    lib/librte_eal/linux/eal/include/exec-env/
> > -> lib/librte_eal/linux/eal/include/
> >
> >    build/include/exec-env/
> > -> build/include/
> >
> > The consequence is to have a file hierarchy a bit more flat.
> >
> > Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> >
> 
> Looks good to me.
> Reviewed-by: David Marchand <david.marchand@redhat.com>
> Tested-by: David Marchand <david.marchand@redhat.com>

Applied

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 19:50     ` Thomas Monjalon
@ 2019-04-02 19:50       ` Thomas Monjalon
  2019-04-03  8:41       ` David Marchand
  1 sibling, 0 replies; 34+ messages in thread
From: Thomas Monjalon @ 2019-04-02 19:50 UTC (permalink / raw)
  To: dev; +Cc: David Marchand, Bruce Richardson, Anand Rawat

02/04/2019 21:46, David Marchand:
> On Tue, Apr 2, 2019 at 9:30 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> 
> > Only one header file (rte_kni_common.h) was in the sub-directory
> >         include/exec-env/
> > This file was installed in a sub-directory of the same name
> > in the makefile-based build.
> > Source and install directories are moved as below:
> >
> >    lib/librte_eal/linux/eal/include/exec-env/
> > -> lib/librte_eal/linux/eal/include/
> >
> >    build/include/exec-env/
> > -> build/include/
> >
> > The consequence is to have a file hierarchy a bit more flat.
> >
> > Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> >
> 
> Looks good to me.
> Reviewed-by: David Marchand <david.marchand@redhat.com>
> Tested-by: David Marchand <david.marchand@redhat.com>

Applied



^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 19:46   ` David Marchand
  2019-04-02 19:46     ` David Marchand
  2019-04-02 19:50     ` Thomas Monjalon
@ 2019-04-02 19:52     ` Ferruh Yigit
  2019-04-02 19:52       ` Ferruh Yigit
                         ` (2 more replies)
  2 siblings, 3 replies; 34+ messages in thread
From: Ferruh Yigit @ 2019-04-02 19:52 UTC (permalink / raw)
  To: David Marchand, Thomas Monjalon, Bruce Richardson; +Cc: dev, Anand Rawat

On 4/2/2019 8:46 PM, David Marchand wrote:
> On Tue, Apr 2, 2019 at 9:30 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> 
>> Only one header file (rte_kni_common.h) was in the sub-directory
>>         include/exec-env/
>> This file was installed in a sub-directory of the same name
>> in the makefile-based build.
>> Source and install directories are moved as below:
>>
>>    lib/librte_eal/linux/eal/include/exec-env/
>> -> lib/librte_eal/linux/eal/include/
>>
>>    build/include/exec-env/
>> -> build/include/
>>
>> The consequence is to have a file hierarchy a bit more flat.
>>
>> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
>>
> 
> Looks good to me.
> Reviewed-by: David Marchand <david.marchand@redhat.com>
> Tested-by: David Marchand <david.marchand@redhat.com>
> 
> 
> Bruce, while testing with meson, I noticed this:
> kernel/linux/kni/meson.build:16: WARNING: Passed invalid keyword argument
> "console".
> WARNING: This will become a hard error in the future.
> WARNING: Unknown keyword arguments in target rte_kni: console

I am not seeing this...

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 19:52     ` Ferruh Yigit
@ 2019-04-02 19:52       ` Ferruh Yigit
  2019-04-02 19:55       ` David Marchand
  2019-04-03 13:34       ` Bruce Richardson
  2 siblings, 0 replies; 34+ messages in thread
From: Ferruh Yigit @ 2019-04-02 19:52 UTC (permalink / raw)
  To: David Marchand, Thomas Monjalon, Bruce Richardson; +Cc: dev, Anand Rawat

On 4/2/2019 8:46 PM, David Marchand wrote:
> On Tue, Apr 2, 2019 at 9:30 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> 
>> Only one header file (rte_kni_common.h) was in the sub-directory
>>         include/exec-env/
>> This file was installed in a sub-directory of the same name
>> in the makefile-based build.
>> Source and install directories are moved as below:
>>
>>    lib/librte_eal/linux/eal/include/exec-env/
>> -> lib/librte_eal/linux/eal/include/
>>
>>    build/include/exec-env/
>> -> build/include/
>>
>> The consequence is to have a file hierarchy a bit more flat.
>>
>> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
>>
> 
> Looks good to me.
> Reviewed-by: David Marchand <david.marchand@redhat.com>
> Tested-by: David Marchand <david.marchand@redhat.com>
> 
> 
> Bruce, while testing with meson, I noticed this:
> kernel/linux/kni/meson.build:16: WARNING: Passed invalid keyword argument
> "console".
> WARNING: This will become a hard error in the future.
> WARNING: Unknown keyword arguments in target rte_kni: console

I am not seeing this...

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 19:52     ` Ferruh Yigit
  2019-04-02 19:52       ` Ferruh Yigit
@ 2019-04-02 19:55       ` David Marchand
  2019-04-02 19:55         ` David Marchand
  2019-04-02 19:57         ` Ferruh Yigit
  2019-04-03 13:34       ` Bruce Richardson
  2 siblings, 2 replies; 34+ messages in thread
From: David Marchand @ 2019-04-02 19:55 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: Thomas Monjalon, Bruce Richardson, dev, Anand Rawat

On Tue, Apr 2, 2019 at 9:52 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:

> On 4/2/2019 8:46 PM, David Marchand wrote:
> > On Tue, Apr 2, 2019 at 9:30 PM Thomas Monjalon <thomas@monjalon.net>
> wrote:
> >
> >> Only one header file (rte_kni_common.h) was in the sub-directory
> >>         include/exec-env/
> >> This file was installed in a sub-directory of the same name
> >> in the makefile-based build.
> >> Source and install directories are moved as below:
> >>
> >>    lib/librte_eal/linux/eal/include/exec-env/
> >> -> lib/librte_eal/linux/eal/include/
> >>
> >>    build/include/exec-env/
> >> -> build/include/
> >>
> >> The consequence is to have a file hierarchy a bit more flat.
> >>
> >> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> >>
> >
> > Looks good to me.
> > Reviewed-by: David Marchand <david.marchand@redhat.com>
> > Tested-by: David Marchand <david.marchand@redhat.com>
> >
> >
> > Bruce, while testing with meson, I noticed this:
> > kernel/linux/kni/meson.build:16: WARNING: Passed invalid keyword argument
> > "console".
> > WARNING: This will become a hard error in the future.
> > WARNING: Unknown keyword arguments in target rte_kni: console
>
> I am not seeing this...
>

Ok, might be my setup:
[dmarchan@dmarchan dpdk]$ meson --version
0.47.2

What should I check ?


-- 
David Marchand

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 19:55       ` David Marchand
@ 2019-04-02 19:55         ` David Marchand
  2019-04-02 19:57         ` Ferruh Yigit
  1 sibling, 0 replies; 34+ messages in thread
From: David Marchand @ 2019-04-02 19:55 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: Thomas Monjalon, Bruce Richardson, dev, Anand Rawat

On Tue, Apr 2, 2019 at 9:52 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:

> On 4/2/2019 8:46 PM, David Marchand wrote:
> > On Tue, Apr 2, 2019 at 9:30 PM Thomas Monjalon <thomas@monjalon.net>
> wrote:
> >
> >> Only one header file (rte_kni_common.h) was in the sub-directory
> >>         include/exec-env/
> >> This file was installed in a sub-directory of the same name
> >> in the makefile-based build.
> >> Source and install directories are moved as below:
> >>
> >>    lib/librte_eal/linux/eal/include/exec-env/
> >> -> lib/librte_eal/linux/eal/include/
> >>
> >>    build/include/exec-env/
> >> -> build/include/
> >>
> >> The consequence is to have a file hierarchy a bit more flat.
> >>
> >> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> >>
> >
> > Looks good to me.
> > Reviewed-by: David Marchand <david.marchand@redhat.com>
> > Tested-by: David Marchand <david.marchand@redhat.com>
> >
> >
> > Bruce, while testing with meson, I noticed this:
> > kernel/linux/kni/meson.build:16: WARNING: Passed invalid keyword argument
> > "console".
> > WARNING: This will become a hard error in the future.
> > WARNING: Unknown keyword arguments in target rte_kni: console
>
> I am not seeing this...
>

Ok, might be my setup:
[dmarchan@dmarchan dpdk]$ meson --version
0.47.2

What should I check ?


-- 
David Marchand

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 19:55       ` David Marchand
  2019-04-02 19:55         ` David Marchand
@ 2019-04-02 19:57         ` Ferruh Yigit
  2019-04-02 19:57           ` Ferruh Yigit
  2019-04-02 20:04           ` David Marchand
  1 sibling, 2 replies; 34+ messages in thread
From: Ferruh Yigit @ 2019-04-02 19:57 UTC (permalink / raw)
  To: David Marchand; +Cc: Thomas Monjalon, Bruce Richardson, dev, Anand Rawat

On 4/2/2019 8:55 PM, David Marchand wrote:
> On Tue, Apr 2, 2019 at 9:52 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
> 
>> On 4/2/2019 8:46 PM, David Marchand wrote:
>>> On Tue, Apr 2, 2019 at 9:30 PM Thomas Monjalon <thomas@monjalon.net>
>> wrote:
>>>
>>>> Only one header file (rte_kni_common.h) was in the sub-directory
>>>>         include/exec-env/
>>>> This file was installed in a sub-directory of the same name
>>>> in the makefile-based build.
>>>> Source and install directories are moved as below:
>>>>
>>>>    lib/librte_eal/linux/eal/include/exec-env/
>>>> -> lib/librte_eal/linux/eal/include/
>>>>
>>>>    build/include/exec-env/
>>>> -> build/include/
>>>>
>>>> The consequence is to have a file hierarchy a bit more flat.
>>>>
>>>> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
>>>>
>>>
>>> Looks good to me.
>>> Reviewed-by: David Marchand <david.marchand@redhat.com>
>>> Tested-by: David Marchand <david.marchand@redhat.com>
>>>
>>>
>>> Bruce, while testing with meson, I noticed this:
>>> kernel/linux/kni/meson.build:16: WARNING: Passed invalid keyword argument
>>> "console".
>>> WARNING: This will become a hard error in the future.
>>> WARNING: Unknown keyword arguments in target rte_kni: console
>>
>> I am not seeing this...
>>
> 
> Ok, might be my setup:
> [dmarchan@dmarchan dpdk]$ meson --version
> 0.47.2
> 
> What should I check ?

I have '0.50.0', but no idea what to check, wait Bruce :) ?

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 19:57         ` Ferruh Yigit
@ 2019-04-02 19:57           ` Ferruh Yigit
  2019-04-02 20:04           ` David Marchand
  1 sibling, 0 replies; 34+ messages in thread
From: Ferruh Yigit @ 2019-04-02 19:57 UTC (permalink / raw)
  To: David Marchand; +Cc: Thomas Monjalon, Bruce Richardson, dev, Anand Rawat

On 4/2/2019 8:55 PM, David Marchand wrote:
> On Tue, Apr 2, 2019 at 9:52 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
> 
>> On 4/2/2019 8:46 PM, David Marchand wrote:
>>> On Tue, Apr 2, 2019 at 9:30 PM Thomas Monjalon <thomas@monjalon.net>
>> wrote:
>>>
>>>> Only one header file (rte_kni_common.h) was in the sub-directory
>>>>         include/exec-env/
>>>> This file was installed in a sub-directory of the same name
>>>> in the makefile-based build.
>>>> Source and install directories are moved as below:
>>>>
>>>>    lib/librte_eal/linux/eal/include/exec-env/
>>>> -> lib/librte_eal/linux/eal/include/
>>>>
>>>>    build/include/exec-env/
>>>> -> build/include/
>>>>
>>>> The consequence is to have a file hierarchy a bit more flat.
>>>>
>>>> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
>>>>
>>>
>>> Looks good to me.
>>> Reviewed-by: David Marchand <david.marchand@redhat.com>
>>> Tested-by: David Marchand <david.marchand@redhat.com>
>>>
>>>
>>> Bruce, while testing with meson, I noticed this:
>>> kernel/linux/kni/meson.build:16: WARNING: Passed invalid keyword argument
>>> "console".
>>> WARNING: This will become a hard error in the future.
>>> WARNING: Unknown keyword arguments in target rte_kni: console
>>
>> I am not seeing this...
>>
> 
> Ok, might be my setup:
> [dmarchan@dmarchan dpdk]$ meson --version
> 0.47.2
> 
> What should I check ?

I have '0.50.0', but no idea what to check, wait Bruce :) ?

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 19:57         ` Ferruh Yigit
  2019-04-02 19:57           ` Ferruh Yigit
@ 2019-04-02 20:04           ` David Marchand
  2019-04-02 20:04             ` David Marchand
  2019-04-02 20:09             ` Thomas Monjalon
  1 sibling, 2 replies; 34+ messages in thread
From: David Marchand @ 2019-04-02 20:04 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: Thomas Monjalon, Bruce Richardson, dev, Anand Rawat

On Tue, Apr 2, 2019 at 9:58 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:

> On 4/2/2019 8:55 PM, David Marchand wrote:
> > On Tue, Apr 2, 2019 at 9:52 PM Ferruh Yigit <ferruh.yigit@intel.com>
> wrote:
> >> On 4/2/2019 8:46 PM, David Marchand wrote:
> >>> Bruce, while testing with meson, I noticed this:
> >>> kernel/linux/kni/meson.build:16: WARNING: Passed invalid keyword
> argument
> >>> "console".
> >>> WARNING: This will become a hard error in the future.
> >>> WARNING: Unknown keyword arguments in target rte_kni: console
> >>
> >> I am not seeing this...
> >>
> >
> > Ok, might be my setup:
> > [dmarchan@dmarchan dpdk]$ meson --version
> > 0.47.2
> >
> > What should I check ?
>
> I have '0.50.0', but no idea what to check, wait Bruce :) ?
>

Looking at the doc..
3 ``pip`` tool, e.g. ``pip3 install meson``. Version 0.42 of meson is
recommended - if the version packaged is too old, the latest version is
generally available from "pip".

Later:
NOTE: for --static builds, DPDK needs to be built with Meson >= 0.46 in
order to
fully generate the list of private dependencies. If DPDK is built with an
older
version of Meson, it might be necessary to manually specify dependencies of
DPDK
PMDs/libraries, for example -lmlx5 -lmnl for librte-pmd-mlx5, or the static
link
step might fail.

And I found this:
meson.build:    meson_version: '>= 0.47.1'

So I suppose I have a working environment and I will just ignore the
warnings :-)

-- 
David Marchand

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 20:04           ` David Marchand
@ 2019-04-02 20:04             ` David Marchand
  2019-04-02 20:09             ` Thomas Monjalon
  1 sibling, 0 replies; 34+ messages in thread
From: David Marchand @ 2019-04-02 20:04 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: Thomas Monjalon, Bruce Richardson, dev, Anand Rawat

On Tue, Apr 2, 2019 at 9:58 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:

> On 4/2/2019 8:55 PM, David Marchand wrote:
> > On Tue, Apr 2, 2019 at 9:52 PM Ferruh Yigit <ferruh.yigit@intel.com>
> wrote:
> >> On 4/2/2019 8:46 PM, David Marchand wrote:
> >>> Bruce, while testing with meson, I noticed this:
> >>> kernel/linux/kni/meson.build:16: WARNING: Passed invalid keyword
> argument
> >>> "console".
> >>> WARNING: This will become a hard error in the future.
> >>> WARNING: Unknown keyword arguments in target rte_kni: console
> >>
> >> I am not seeing this...
> >>
> >
> > Ok, might be my setup:
> > [dmarchan@dmarchan dpdk]$ meson --version
> > 0.47.2
> >
> > What should I check ?
>
> I have '0.50.0', but no idea what to check, wait Bruce :) ?
>

Looking at the doc..
3 ``pip`` tool, e.g. ``pip3 install meson``. Version 0.42 of meson is
recommended - if the version packaged is too old, the latest version is
generally available from "pip".

Later:
NOTE: for --static builds, DPDK needs to be built with Meson >= 0.46 in
order to
fully generate the list of private dependencies. If DPDK is built with an
older
version of Meson, it might be necessary to manually specify dependencies of
DPDK
PMDs/libraries, for example -lmlx5 -lmnl for librte-pmd-mlx5, or the static
link
step might fail.

And I found this:
meson.build:    meson_version: '>= 0.47.1'

So I suppose I have a working environment and I will just ignore the
warnings :-)

-- 
David Marchand

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 20:04           ` David Marchand
  2019-04-02 20:04             ` David Marchand
@ 2019-04-02 20:09             ` Thomas Monjalon
  2019-04-02 20:09               ` Thomas Monjalon
  1 sibling, 1 reply; 34+ messages in thread
From: Thomas Monjalon @ 2019-04-02 20:09 UTC (permalink / raw)
  To: David Marchand; +Cc: Ferruh Yigit, Bruce Richardson, dev, Anand Rawat, bluca

02/04/2019 22:04, David Marchand:
> On Tue, Apr 2, 2019 at 9:58 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
> > On 4/2/2019 8:55 PM, David Marchand wrote:
> > > On Tue, Apr 2, 2019 at 9:52 PM Ferruh Yigit <ferruh.yigit@intel.com>
> > wrote:
> > >> On 4/2/2019 8:46 PM, David Marchand wrote:
> > >>> Bruce, while testing with meson, I noticed this:
> > >>> kernel/linux/kni/meson.build:16: WARNING: Passed invalid keyword
> > argument
> > >>> "console".
> > >>> WARNING: This will become a hard error in the future.
> > >>> WARNING: Unknown keyword arguments in target rte_kni: console
> > >>
> > >> I am not seeing this...
> > >>
> > >
> > > Ok, might be my setup:
> > > [dmarchan@dmarchan dpdk]$ meson --version
> > > 0.47.2
> > >
> > > What should I check ?
> >
> > I have '0.50.0', but no idea what to check, wait Bruce :) ?
> 
> Looking at the doc..
> 3 ``pip`` tool, e.g. ``pip3 install meson``. Version 0.42 of meson is
> recommended - if the version packaged is too old, the latest version is
> generally available from "pip".
> 
> Later:
> NOTE: for --static builds, DPDK needs to be built with Meson >= 0.46 in
> order to
> fully generate the list of private dependencies. If DPDK is built with an
> older
> version of Meson, it might be necessary to manually specify dependencies of
> DPDK
> PMDs/libraries, for example -lmlx5 -lmnl for librte-pmd-mlx5, or the static
> link
> step might fail.
> 
> And I found this:
> meson.build:    meson_version: '>= 0.47.1'

There is a miss of doc update in this patch:
	http://git.dpdk.org/dpdk/commit/?id=eded3f928c74

The minimum version is really 0.47.1.

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 20:09             ` Thomas Monjalon
@ 2019-04-02 20:09               ` Thomas Monjalon
  0 siblings, 0 replies; 34+ messages in thread
From: Thomas Monjalon @ 2019-04-02 20:09 UTC (permalink / raw)
  To: David Marchand; +Cc: Ferruh Yigit, Bruce Richardson, dev, Anand Rawat, bluca

02/04/2019 22:04, David Marchand:
> On Tue, Apr 2, 2019 at 9:58 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
> > On 4/2/2019 8:55 PM, David Marchand wrote:
> > > On Tue, Apr 2, 2019 at 9:52 PM Ferruh Yigit <ferruh.yigit@intel.com>
> > wrote:
> > >> On 4/2/2019 8:46 PM, David Marchand wrote:
> > >>> Bruce, while testing with meson, I noticed this:
> > >>> kernel/linux/kni/meson.build:16: WARNING: Passed invalid keyword
> > argument
> > >>> "console".
> > >>> WARNING: This will become a hard error in the future.
> > >>> WARNING: Unknown keyword arguments in target rte_kni: console
> > >>
> > >> I am not seeing this...
> > >>
> > >
> > > Ok, might be my setup:
> > > [dmarchan@dmarchan dpdk]$ meson --version
> > > 0.47.2
> > >
> > > What should I check ?
> >
> > I have '0.50.0', but no idea what to check, wait Bruce :) ?
> 
> Looking at the doc..
> 3 ``pip`` tool, e.g. ``pip3 install meson``. Version 0.42 of meson is
> recommended - if the version packaged is too old, the latest version is
> generally available from "pip".
> 
> Later:
> NOTE: for --static builds, DPDK needs to be built with Meson >= 0.46 in
> order to
> fully generate the list of private dependencies. If DPDK is built with an
> older
> version of Meson, it might be necessary to manually specify dependencies of
> DPDK
> PMDs/libraries, for example -lmlx5 -lmnl for librte-pmd-mlx5, or the static
> link
> step might fail.
> 
> And I found this:
> meson.build:    meson_version: '>= 0.47.1'

There is a miss of doc update in this patch:
	http://git.dpdk.org/dpdk/commit/?id=eded3f928c74

The minimum version is really 0.47.1.




^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 19:50     ` Thomas Monjalon
  2019-04-02 19:50       ` Thomas Monjalon
@ 2019-04-03  8:41       ` David Marchand
  2019-04-03  8:41         ` David Marchand
  1 sibling, 1 reply; 34+ messages in thread
From: David Marchand @ 2019-04-03  8:41 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: dev, Bruce Richardson, Anand Rawat

On Tue, Apr 2, 2019 at 9:51 PM Thomas Monjalon <thomas@monjalon.net> wrote:

> 02/04/2019 21:46, David Marchand:
> > On Tue, Apr 2, 2019 at 9:30 PM Thomas Monjalon <thomas@monjalon.net>
> wrote:
> >
> > > Only one header file (rte_kni_common.h) was in the sub-directory
> > >         include/exec-env/
> > > This file was installed in a sub-directory of the same name
> > > in the makefile-based build.
> > > Source and install directories are moved as below:
> > >
> > >    lib/librte_eal/linux/eal/include/exec-env/
> > > -> lib/librte_eal/linux/eal/include/
> > >
> > >    build/include/exec-env/
> > > -> build/include/
> > >
> > > The consequence is to have a file hierarchy a bit more flat.
> > >
> > > Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> > >
> >
> > Looks good to me.
> > Reviewed-by: David Marchand <david.marchand@redhat.com>
> > Tested-by: David Marchand <david.marchand@redhat.com>
>
> Applied
>


As a followup, I noticed this:

[dmarchan@dmarchan dpdk]$ for file in $(find lib/librte_eal/*/eal -name
*.h); do echo $(basename $file); git grep -l $(basename $file); echo; done
eal_alarm_private.h
lib/librte_eal/freebsd/eal/eal_alarm.c
lib/librte_eal/freebsd/eal/eal_interrupts.c

eal_vfio.h
lib/librte_eal/linux/eal/eal.c
lib/librte_eal/linux/eal/eal_interrupts.c
lib/librte_eal/linux/eal/eal_vfio.c
lib/librte_eal/linux/eal/eal_vfio_mp_sync.c

rte_kni_common.h
kernel/linux/kni/kni_dev.h
kernel/linux/kni/kni_fifo.h
kernel/linux/kni/kni_misc.c
kernel/linux/kni/kni_net.c
lib/librte_eal/linux/eal/Makefile
lib/librte_kni/rte_kni.c
lib/librte_kni/rte_kni.h

At the moment, kni is the only special case where users need to set a
CFLAGS -I...eal/include for.
But we have some drivers directly referencing linux/eal, linux/eal/include
and a funny $(SYSTEM)app/eal:

[dmarchan@dmarchan dpdk]$ git grep CFLAGS.*/eal
drivers/bus/dpaa/Makefile:CFLAGS += -I$(RTE_SDK)/lib/librte_eal/linux/eal
drivers/bus/pci/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/$(SYSTEM)app/eal
drivers/bus/vmbus/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/$(SYSTEM)app/eal
drivers/crypto/caam_jr/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/linux/eal
drivers/crypto/dpaa2_sec/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/linux/eal
drivers/crypto/dpaa_sec/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/linux/eal
drivers/event/dpaa/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/linux/eal/include
drivers/event/dpaa2/Makefile:CFLAGS += -I$(RTE_SDK)/lib/librte_eal/linux/eal
drivers/mempool/dpaa2/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/linux/eal
drivers/net/dpaa/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/linux/eal/include
drivers/net/dpaa2/Makefile:CFLAGS += -I$(RTE_SDK)/lib/librte_eal/linux/eal
drivers/raw/dpaa2_qdma/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/linux/eal

Preparing a patch.

-- 
David Marchand

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-03  8:41       ` David Marchand
@ 2019-04-03  8:41         ` David Marchand
  0 siblings, 0 replies; 34+ messages in thread
From: David Marchand @ 2019-04-03  8:41 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: dev, Bruce Richardson, Anand Rawat

On Tue, Apr 2, 2019 at 9:51 PM Thomas Monjalon <thomas@monjalon.net> wrote:

> 02/04/2019 21:46, David Marchand:
> > On Tue, Apr 2, 2019 at 9:30 PM Thomas Monjalon <thomas@monjalon.net>
> wrote:
> >
> > > Only one header file (rte_kni_common.h) was in the sub-directory
> > >         include/exec-env/
> > > This file was installed in a sub-directory of the same name
> > > in the makefile-based build.
> > > Source and install directories are moved as below:
> > >
> > >    lib/librte_eal/linux/eal/include/exec-env/
> > > -> lib/librte_eal/linux/eal/include/
> > >
> > >    build/include/exec-env/
> > > -> build/include/
> > >
> > > The consequence is to have a file hierarchy a bit more flat.
> > >
> > > Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> > >
> >
> > Looks good to me.
> > Reviewed-by: David Marchand <david.marchand@redhat.com>
> > Tested-by: David Marchand <david.marchand@redhat.com>
>
> Applied
>


As a followup, I noticed this:

[dmarchan@dmarchan dpdk]$ for file in $(find lib/librte_eal/*/eal -name
*.h); do echo $(basename $file); git grep -l $(basename $file); echo; done
eal_alarm_private.h
lib/librte_eal/freebsd/eal/eal_alarm.c
lib/librte_eal/freebsd/eal/eal_interrupts.c

eal_vfio.h
lib/librte_eal/linux/eal/eal.c
lib/librte_eal/linux/eal/eal_interrupts.c
lib/librte_eal/linux/eal/eal_vfio.c
lib/librte_eal/linux/eal/eal_vfio_mp_sync.c

rte_kni_common.h
kernel/linux/kni/kni_dev.h
kernel/linux/kni/kni_fifo.h
kernel/linux/kni/kni_misc.c
kernel/linux/kni/kni_net.c
lib/librte_eal/linux/eal/Makefile
lib/librte_kni/rte_kni.c
lib/librte_kni/rte_kni.h

At the moment, kni is the only special case where users need to set a
CFLAGS -I...eal/include for.
But we have some drivers directly referencing linux/eal, linux/eal/include
and a funny $(SYSTEM)app/eal:

[dmarchan@dmarchan dpdk]$ git grep CFLAGS.*/eal
drivers/bus/dpaa/Makefile:CFLAGS += -I$(RTE_SDK)/lib/librte_eal/linux/eal
drivers/bus/pci/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/$(SYSTEM)app/eal
drivers/bus/vmbus/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/$(SYSTEM)app/eal
drivers/crypto/caam_jr/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/linux/eal
drivers/crypto/dpaa2_sec/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/linux/eal
drivers/crypto/dpaa_sec/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/linux/eal
drivers/event/dpaa/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/linux/eal/include
drivers/event/dpaa2/Makefile:CFLAGS += -I$(RTE_SDK)/lib/librte_eal/linux/eal
drivers/mempool/dpaa2/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/linux/eal
drivers/net/dpaa/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/linux/eal/include
drivers/net/dpaa2/Makefile:CFLAGS += -I$(RTE_SDK)/lib/librte_eal/linux/eal
drivers/raw/dpaa2_qdma/Makefile:CFLAGS +=
-I$(RTE_SDK)/lib/librte_eal/linux/eal

Preparing a patch.

-- 
David Marchand

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-02 19:52     ` Ferruh Yigit
  2019-04-02 19:52       ` Ferruh Yigit
  2019-04-02 19:55       ` David Marchand
@ 2019-04-03 13:34       ` Bruce Richardson
  2019-04-03 13:34         ` Bruce Richardson
  2 siblings, 1 reply; 34+ messages in thread
From: Bruce Richardson @ 2019-04-03 13:34 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: David Marchand, Thomas Monjalon, dev, Anand Rawat

On Tue, Apr 02, 2019 at 08:52:12PM +0100, Ferruh Yigit wrote:
> On 4/2/2019 8:46 PM, David Marchand wrote:
> > On Tue, Apr 2, 2019 at 9:30 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> > 
> >> Only one header file (rte_kni_common.h) was in the sub-directory
> >>         include/exec-env/
> >> This file was installed in a sub-directory of the same name
> >> in the makefile-based build.
> >> Source and install directories are moved as below:
> >>
> >>    lib/librte_eal/linux/eal/include/exec-env/
> >> -> lib/librte_eal/linux/eal/include/
> >>
> >>    build/include/exec-env/
> >> -> build/include/
> >>
> >> The consequence is to have a file hierarchy a bit more flat.
> >>
> >> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> >>
> > 
> > Looks good to me.
> > Reviewed-by: David Marchand <david.marchand@redhat.com>
> > Tested-by: David Marchand <david.marchand@redhat.com>
> > 
> > 
> > Bruce, while testing with meson, I noticed this:
> > kernel/linux/kni/meson.build:16: WARNING: Passed invalid keyword argument
> > "console".
> > WARNING: This will become a hard error in the future.
> > WARNING: Unknown keyword arguments in target rte_kni: console
> 
> I am not seeing this...

It's ok. The keyword is new in newer versions which obviously won't give an
error on it being invalid then. Older versions just ignore it, other than
printing the warning you see.

In short, harmless, ignore! :-)

/Bruce

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [dpdk-dev] [PATCH v3] eal: remove exec-env directory
  2019-04-03 13:34       ` Bruce Richardson
@ 2019-04-03 13:34         ` Bruce Richardson
  0 siblings, 0 replies; 34+ messages in thread
From: Bruce Richardson @ 2019-04-03 13:34 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: David Marchand, Thomas Monjalon, dev, Anand Rawat

On Tue, Apr 02, 2019 at 08:52:12PM +0100, Ferruh Yigit wrote:
> On 4/2/2019 8:46 PM, David Marchand wrote:
> > On Tue, Apr 2, 2019 at 9:30 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> > 
> >> Only one header file (rte_kni_common.h) was in the sub-directory
> >>         include/exec-env/
> >> This file was installed in a sub-directory of the same name
> >> in the makefile-based build.
> >> Source and install directories are moved as below:
> >>
> >>    lib/librte_eal/linux/eal/include/exec-env/
> >> -> lib/librte_eal/linux/eal/include/
> >>
> >>    build/include/exec-env/
> >> -> build/include/
> >>
> >> The consequence is to have a file hierarchy a bit more flat.
> >>
> >> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> >>
> > 
> > Looks good to me.
> > Reviewed-by: David Marchand <david.marchand@redhat.com>
> > Tested-by: David Marchand <david.marchand@redhat.com>
> > 
> > 
> > Bruce, while testing with meson, I noticed this:
> > kernel/linux/kni/meson.build:16: WARNING: Passed invalid keyword argument
> > "console".
> > WARNING: This will become a hard error in the future.
> > WARNING: Unknown keyword arguments in target rte_kni: console
> 
> I am not seeing this...

It's ok. The keyword is new in newer versions which obviously won't give an
error on it being invalid then. Older versions just ignore it, other than
printing the warning you see.

In short, harmless, ignore! :-)

/Bruce

^ permalink raw reply	[flat|nested] 34+ messages in thread

end of thread, other threads:[~2019-04-03 13:34 UTC | newest]

Thread overview: 34+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-01 23:07 [dpdk-dev] [PATCH] eal: remove exec-env directory Thomas Monjalon
2019-04-01 23:07 ` Thomas Monjalon
2019-04-02  9:39 ` Bruce Richardson
2019-04-02  9:39   ` Bruce Richardson
2019-04-02 14:43   ` Ferruh Yigit
2019-04-02 14:43     ` Ferruh Yigit
2019-04-02  9:57 ` David Marchand
2019-04-02  9:57   ` David Marchand
2019-04-02 10:03   ` Thomas Monjalon
2019-04-02 10:03     ` Thomas Monjalon
2019-04-02 15:13 ` [dpdk-dev] [PATCH v2] " Thomas Monjalon
2019-04-02 15:13   ` Thomas Monjalon
2019-04-02 19:30 ` [dpdk-dev] [PATCH v3] " Thomas Monjalon
2019-04-02 19:30   ` Thomas Monjalon
2019-04-02 19:46   ` David Marchand
2019-04-02 19:46     ` David Marchand
2019-04-02 19:50     ` Thomas Monjalon
2019-04-02 19:50       ` Thomas Monjalon
2019-04-03  8:41       ` David Marchand
2019-04-03  8:41         ` David Marchand
2019-04-02 19:52     ` Ferruh Yigit
2019-04-02 19:52       ` Ferruh Yigit
2019-04-02 19:55       ` David Marchand
2019-04-02 19:55         ` David Marchand
2019-04-02 19:57         ` Ferruh Yigit
2019-04-02 19:57           ` Ferruh Yigit
2019-04-02 20:04           ` David Marchand
2019-04-02 20:04             ` David Marchand
2019-04-02 20:09             ` Thomas Monjalon
2019-04-02 20:09               ` Thomas Monjalon
2019-04-03 13:34       ` Bruce Richardson
2019-04-03 13:34         ` Bruce Richardson
2019-04-02 19:50   ` Ferruh Yigit
2019-04-02 19:50     ` Ferruh Yigit

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).