* [dpdk-dev] [PATCH] devtools: rename build dependency of mlx drivers
@ 2017-10-11 13:28 Thomas Monjalon
2017-10-11 14:39 ` Nélio Laranjeiro
0 siblings, 1 reply; 3+ messages in thread
From: Thomas Monjalon @ 2017-10-11 13:28 UTC (permalink / raw)
To: dev
The Mellanox drivers were requiring MOFED at compilation time.
It is now possible to use the upstream rdma-core package.
So the dependency option is renamed in the build tool.
Fixes: 43e9d9794cde ("net/mlx5: support upstream rdma-core")
Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
---
devtools/test-build.sh | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/devtools/test-build.sh b/devtools/test-build.sh
index c6dfaf0a8..34fba9b96 100755
--- a/devtools/test-build.sh
+++ b/devtools/test-build.sh
@@ -39,7 +39,7 @@ default_path=$PATH
# - DPDK_DEP_ARCHIVE
# - DPDK_DEP_CFLAGS
# - DPDK_DEP_LDFLAGS
-# - DPDK_DEP_MOFED (y/[n])
+# - DPDK_DEP_MLX (y/[n])
# - DPDK_DEP_NUMA ([y]/n)
# - DPDK_DEP_PCAP (y/[n])
# - DPDK_DEP_SSL (y/[n])
@@ -121,7 +121,7 @@ reset_env ()
unset DPDK_DEP_ARCHIVE
unset DPDK_DEP_CFLAGS
unset DPDK_DEP_LDFLAGS
- unset DPDK_DEP_MOFED
+ unset DPDK_DEP_MLX
unset DPDK_DEP_NUMA
unset DPDK_DEP_PCAP
unset DPDK_DEP_SSL
@@ -167,7 +167,7 @@ config () # <directory> <target> <options>
sed -ri 's,(BYPASS=)n,\1y,' $1/.config
test "$DPDK_DEP_ARCHIVE" != y || \
sed -ri 's,(RESOURCE_TAR=)n,\1y,' $1/.config
- test "$DPDK_DEP_MOFED" != y || \
+ test "$DPDK_DEP_MLX" != y || \
sed -ri 's,(MLX._PMD=)n,\1y,' $1/.config
test "$DPDK_DEP_SZE" != y || \
sed -ri 's,(PMD_SZEDATA2=)n,\1y,' $1/.config
--
2.14.1
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [dpdk-dev] [PATCH] devtools: rename build dependency of mlx drivers
2017-10-11 13:28 [dpdk-dev] [PATCH] devtools: rename build dependency of mlx drivers Thomas Monjalon
@ 2017-10-11 14:39 ` Nélio Laranjeiro
2017-11-07 22:48 ` Thomas Monjalon
0 siblings, 1 reply; 3+ messages in thread
From: Nélio Laranjeiro @ 2017-10-11 14:39 UTC (permalink / raw)
To: Thomas Monjalon; +Cc: dev
On Wed, Oct 11, 2017 at 03:28:58PM +0200, Thomas Monjalon wrote:
> The Mellanox drivers were requiring MOFED at compilation time.
> It is now possible to use the upstream rdma-core package.
> So the dependency option is renamed in the build tool.
>
> Fixes: 43e9d9794cde ("net/mlx5: support upstream rdma-core")
>
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> ---
> devtools/test-build.sh | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/devtools/test-build.sh b/devtools/test-build.sh
> index c6dfaf0a8..34fba9b96 100755
> --- a/devtools/test-build.sh
> +++ b/devtools/test-build.sh
> @@ -39,7 +39,7 @@ default_path=$PATH
> # - DPDK_DEP_ARCHIVE
> # - DPDK_DEP_CFLAGS
> # - DPDK_DEP_LDFLAGS
> -# - DPDK_DEP_MOFED (y/[n])
> +# - DPDK_DEP_MLX (y/[n])
> # - DPDK_DEP_NUMA ([y]/n)
> # - DPDK_DEP_PCAP (y/[n])
> # - DPDK_DEP_SSL (y/[n])
> @@ -121,7 +121,7 @@ reset_env ()
> unset DPDK_DEP_ARCHIVE
> unset DPDK_DEP_CFLAGS
> unset DPDK_DEP_LDFLAGS
> - unset DPDK_DEP_MOFED
> + unset DPDK_DEP_MLX
> unset DPDK_DEP_NUMA
> unset DPDK_DEP_PCAP
> unset DPDK_DEP_SSL
> @@ -167,7 +167,7 @@ config () # <directory> <target> <options>
> sed -ri 's,(BYPASS=)n,\1y,' $1/.config
> test "$DPDK_DEP_ARCHIVE" != y || \
> sed -ri 's,(RESOURCE_TAR=)n,\1y,' $1/.config
> - test "$DPDK_DEP_MOFED" != y || \
> + test "$DPDK_DEP_MLX" != y || \
> sed -ri 's,(MLX._PMD=)n,\1y,' $1/.config
> test "$DPDK_DEP_SZE" != y || \
> sed -ri 's,(PMD_SZEDATA2=)n,\1y,' $1/.config
> --
> 2.14.1
Acked-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
--
Nélio Laranjeiro
6WIND
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [dpdk-dev] [PATCH] devtools: rename build dependency of mlx drivers
2017-10-11 14:39 ` Nélio Laranjeiro
@ 2017-11-07 22:48 ` Thomas Monjalon
0 siblings, 0 replies; 3+ messages in thread
From: Thomas Monjalon @ 2017-11-07 22:48 UTC (permalink / raw)
To: dev; +Cc: Nélio Laranjeiro
11/10/2017 16:39, Nélio Laranjeiro:
> On Wed, Oct 11, 2017 at 03:28:58PM +0200, Thomas Monjalon wrote:
> > The Mellanox drivers were requiring MOFED at compilation time.
> > It is now possible to use the upstream rdma-core package.
> > So the dependency option is renamed in the build tool.
> >
> > Fixes: 43e9d9794cde ("net/mlx5: support upstream rdma-core")
> >
> > Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
>
> Acked-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
Applied
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-11-07 22:48 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-11 13:28 [dpdk-dev] [PATCH] devtools: rename build dependency of mlx drivers Thomas Monjalon
2017-10-11 14:39 ` Nélio Laranjeiro
2017-11-07 22:48 ` Thomas Monjalon
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).