This patch makes the configuration based on makefile and the
configuration based on meson to be the same.
Fixes: c6e536e38437 ("build: add more implementers IDs and PNs for ARM")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Ruifeng Wang
Reviewed-by: Honnappa N
To flush a ring not in use, dequeue one by one is wasting cpu cycles.
The patch is to just resetting the head and tail indices to save cpu
cycle.
Gavin Hu (2):
ring: add rte_ring_reset api to flush the ring
hash: flush the rings instead of dequeuing one by one
lib/librte_hash/Makefile
Currently, the flush is done by dequeuing the ring in a while loop. It is much
simpler to flush the queue by resetting the head and tail indices.
Signed-off-by: Gavin Hu
Reviewed-by: Ruifeng Wang
Reviewed-by: Honnappa Nagarahalli
---
lib/librte_ring/rte_ring.h | 20
add extendable bucket feature")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Honnappa Nagarahalli
---
lib/librte_hash/Makefile | 2 +-
lib/librte_hash/rte_cuckoo_hash.c | 11 ---
2 files changed, 5 insertions(+), 8 deletions(-)
diff --git a/lib/librte
add extendable bucket feature")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Honnappa Nagarahalli
---
lib/librte_hash/Makefile | 2 +-
lib/librte_hash/rte_cuckoo_hash.c | 11 ---
2 files changed, 5 insertions(+), 8 deletions(-)
diff --git a/lib/librte
V2: fix the coding style issue(commit message line too long)
V1: To flush a ring not in use, dequeue one by one is wasting cpu cycles.
The patch is to just resetting the head and tail indices to save cpu
cycle.
Gavin Hu (2):
ring: add reset api to flush the ring when not in use
hash: flush
Currently, the flush is done by dequeuing the ring in a while loop. It is
much simpler to flush the queue by resetting the head and tail indices.
Signed-off-by: Gavin Hu
Reviewed-by: Ruifeng Wang
Reviewed-by: Honnappa Nagarahalli
---
lib/librte_ring/rte_ring.h | 20
Currently, the flush is done by dequeuing the ring in a while loop. It is
much simpler to flush the queue by resetting the head and tail indices.
Signed-off-by: Gavin Hu
Reviewed-by: Ruifeng Wang
Reviewed-by: Honnappa Nagarahalli
---
lib/librte_ring/rte_ring.h | 20
V3: Allow experimental API for meson build
V2: Fix the coding style issue(commit message line too long)
V1: To flush a ring not in use, dequeue one by one is wasting cpu cycles.
The patch is to just resetting the head and tail indices to save cpu
cycle.
Gavin Hu (2):
ring: add reset api to
add extendable bucket feature")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Honnappa Nagarahalli
---
lib/librte_hash/Makefile | 2 +-
lib/librte_hash/meson.build | 3 +++
lib/librte_hash/rte_cuckoo_hash.c | 11 ---
3 files changed, 8 insertions(+),
. Move the arm specific implementation to arm folder to remove the
hardcoded implementation.
5. Use atomic primitives, which translate to one-way barriers, instead of
two-way sync primitives, to optimize for performance.
Gavin Hu (5):
test/spinlock: remove 1us delay to create contention
is too much, compromising the goal of benchmarking the lock and
unlock performance.
Signed-off-by: Gavin Hu
Reviewed-by: Ruifeng Wang
Reviewed-by: Joyce Kong
Reviewed-by: Phil Yang
Reviewed-by: Honnappa Nagarahalli
Reviewed-by: Ola Liljedahl
---
test/test/test_spinlock.c | 2 --
1 file
Instead of getting timestamps per iteration, amortize its overhead
can help getting more precise benchmarking results.
Signed-off-by: Gavin Hu
Reviewed-by: Joyce Kong
---
test/test/test_spinlock.c | 28 +++-
1 file changed, 15 insertions(+), 13 deletions(-)
diff --git
To precisely benchmark the spinlock performance, uses the precise
version of getting timestamps, which enforces the timestamps are
obtained at the expected places.
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
---
test/test/test_spinlock.c | 4 ++--
1 file changed, 2 insertions(+), 2
remove the hard code #ifdef RTE_FORCE_INTRINSICS, move the implementation
to the arm specific file, x86 and POWER have their own implementations.
Signed-off-by: Gavin Hu
Reviewed-by: Ruifeng Wang
Reviewed-by: Phil Yang
Reviewed-by: Honnappa Nagarahalli
---
.../common/include/arch/arm
h ***
Core [123] Cost Time = 163615 us
Core [124] Cost Time = 166471 us
Core [125] Cost Time = 189044 us
Core [126] Cost Time = 195745 us
Core [127] Cost Time = 78423 us
Total Cost Time = 27339656 us
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Honnappa Nagarahalli
Reviewed-by:
.
Suggested-by: Gavin Hu
Signed-off-by: Joyce Kong
Reviewed-by: Ruifeng Wang
Reviewed-by: Honnappa Nagarahalli
Reviewed-by: Dharmik Thakkar
Reviewed-by: Ola Liljedahl
Reviewed-by: Gavin Hu
---
test/test/test_ring_perf.c | 82 --
1 file changed, 80
Change to the new wording according to Arm corp:
https://mails.dpdk.org/archives/dev/2018-October/115174.html
otherwise, "Wrong headline lowercase" may be falsely reported.
Fixes: f146ada21d1e ("scripts: add more git log checks")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Change to the new wording according to Arm corp:
https://mails.dpdk.org/archives/dev/2018-October/115174.html
otherwise, "Wrong headline lowercase" may be falsely reported.
Fixes: f146ada21d1e ("scripts: add more git log checks")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
coded implementation.
5. Use atomic primitives, which translate to one-way barriers, instead of
two-way sync primitives, to optimize for performance.
Gavin Hu (5):
test/spinlock: remove 1us delay for correct benchmarking
test/spinlock: get timestamp more precisely
test/spinlock: amortize the co
is too much, compromising the goal of benchmarking the lock and
unlock performance.
Signed-off-by: Gavin Hu
Reviewed-by: Ruifeng Wang
Reviewed-by: Joyce Kong
Reviewed-by: Phil Yang
Reviewed-by: Honnappa Nagarahalli
Reviewed-by: Ola Liljedahl
---
test/test/test_spinlock.c | 2 --
1 file
To precisely benchmark the spinlock performance, uses the precise
version of getting timestamps, which enforces the timestamps are
obtained at the expected places.
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
---
test/test/test_spinlock.c | 4 ++--
1 file changed, 2 insertions(+), 2
__atomic_exchange_n instead of __atomic_exchange_(2/4/8).
For more information, please refer to:
http://mails.dpdk.org/archives/dev/2018-April/096776.html
Fixes: 7bdccb93078e ("eal: fix ARM build with clang")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
---
lib/librte_eal/common/include/generic
Instead of getting timestamps per iteration, amortize its overhead
can help getting more precise benchmarking results.
Signed-off-by: Gavin Hu
Reviewed-by: Joyce Kong
---
test/test/test_spinlock.c | 28 +++-
1 file changed, 15 insertions(+), 13 deletions(-)
diff --git
h ***
Core [123] Cost Time = 163615 us
Core [124] Cost Time = 166471 us
Core [125] Cost Time = 189044 us
Core [126] Cost Time = 195745 us
Core [127] Cost Time = 78423 us
Total Cost Time = 27339656 us
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Honnappa Nagarahalli
Reviewed-by:
Change to the new wording according to Arm corp:
https://mails.dpdk.org/archives/dev/2018-October/115174.html
otherwise, "Wrong headline lowercase" may be falsely reported.
Fixes: f146ada21d1e ("scripts: add more git log checks")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
__atomic_exchange_n instead of __atomic_exchange_(2/4/8).
For more information, please refer to:
http://mails.dpdk.org/archives/dev/2018-April/096776.html
Fixes: 7bdccb93078e ("eal: fix ARM build with clang")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
---
lib/librte_eal/common/include/generic
e the overhead of getting the timestamp by 1 loops.
4. Move the arm specific implementation to arm folder to remove the
hardcoded implementation.
5. Use atomic primitives, which translate to one-way barriers, instead of
two-way sync primitives, to optimize for performance.
Gavin Hu (5):
To precisely benchmark the spinlock performance, uses the precise
version of getting timestamps, which enforces the timestamps are
obtained at the expected places.
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
---
test/test/test_spinlock.c | 4 ++--
1 file changed, 2 insertions(+), 2
is too much, compromising the goal of benchmarking the lock and
unlock performance.
Signed-off-by: Gavin Hu
Reviewed-by: Ruifeng Wang
Reviewed-by: Joyce Kong
Reviewed-by: Phil Yang
Reviewed-by: Honnappa Nagarahalli
Reviewed-by: Ola Liljedahl
---
test/test/test_spinlock.c | 2 --
1 file
h ***
Core [123] Cost Time = 163615 us
Core [124] Cost Time = 166471 us
Core [125] Cost Time = 189044 us
Core [126] Cost Time = 195745 us
Core [127] Cost Time = 78423 us
Total Cost Time = 27339656 us
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Honnappa Nagarahalli
Reviewed-by:
Instead of getting timestamps per iteration, amortize its overhead
can help getting more precise benchmarking results.
Signed-off-by: Gavin Hu
Reviewed-by: Joyce Kong
---
test/test/test_spinlock.c | 28 +++-
1 file changed, 15 insertions(+), 13 deletions(-)
diff --git
From: Joyce Kong
The old implementation is unfair, some threads may take locks aggressively
while leaving the other threads starving for long time. As shown in the
following test, within same period of time, there are threads taking locks
much more times than the others.
The new implementation g
.
Suggested-by: Gavin Hu
Signed-off-by: Joyce Kong
Reviewed-by: Ruifeng Wang
Reviewed-by: Honnappa Nagarahalli
Reviewed-by: Dharmik Thakkar
Reviewed-by: Ola Liljedahl
Reviewed-by: Gavin Hu
---
test/test/test_ring_perf.c | 82 --
1 file changed, 79
Currently, the flush is done by dequeuing the ring in a while loop. It is
much simpler to flush the queue by resetting the head and tail indices.
Signed-off-by: Gavin Hu
Reviewed-by: Ruifeng Wang
Reviewed-by: Honnappa Nagarahalli
---
lib/librte_ring/rte_ring.h | 20
add extendable bucket feature")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Honnappa Nagarahalli
---
lib/librte_hash/Makefile | 2 +-
lib/librte_hash/meson.build | 3 +++
lib/librte_hash/rte_cuckoo_hash.c | 11 ---
3 files changed, 8 insertions(+),
indices to save cpu
cycle.
Gavin Hu (2):
ring: add reset api to flush the ring when not in use
hash: flush the rings instead of dequeuing one by one
Joyce Kong (1):
test/ring: ring perf test case enhancement
lib/librte_hash/Makefile | 2 +-
lib/librte_hash/meson.build
From: Gavin Hu
Currently, the flush is done by dequeuing the ring in a while loop. It is
much simpler to flush the queue by resetting the head and tail indices.
Signed-off-by: gavin hu
Reviewed-by: ruifeng wang
Reviewed-by: honnappa nagarahalli
---
lib/librte_ring/rte_ring.h | 20
From: Gavin Hu
Within rte_hash_reset, calling a while loop to dequeue one by
one from the ring, while not using them at all, is wasting cycles,
The patch just flush the ring by resetting the indices can save cpu
cycles.
Fixes: b26473ff8f4a ("hash: add reset function")
Fixes: 75706568a
experimental API for meson build
V2: Fix the coding style issue(commit message line too long)
V1: To flush a ring not in use, dequeue one by one is wasting cpu cycles.
The patch is to just resetting the head and tail indices to save cpu
cycle.
Gavin Hu (2):
ring: add reset API to flush the ring
From: Joyce Kong
Run ring perf test on all available cores to really verify MPMC operations.
The old way of running on a pair of cores is not enough for MPMC rings.
Suggested-by: gavin hu
Signed-off-by: joyce kong
Reviewed-by: ruifeng wang
Reviewed-by: honnappa nagarahalli
Reviewed-by
save cpu
cycle.
Gavin Hu (2):
ring: add reset API to flush the ring when not in use
hash: flush the rings instead of dequeuing one by one
Joyce Kong (1):
test/ring: ring perf test case enhancement
lib/librte_hash/Makefile | 2 +-
lib/librte_hash/meson.build | 3
From: Gavin Hu
Within rte_hash_reset, calling a while loop to dequeue one by
one from the ring, while not using them at all, is wasting cycles,
The patch just flush the ring by resetting the indices can save cpu
cycles.
Fixes: b26473ff8f4a ("hash: add reset function")
Fixes: 75706568a
From: Joyce Kong
Run ring perf test on all available cores to really verify MPMC operations.
The old way of running on a pair of cores is not enough for MPMC rings.
Suggested-by: Gavin Hu
Signed-off-by: Joyce Kong
Reviewed-by: Ruifeng Wang
Reviewed-by: Honnappa Nagarahalli
Reviewed-by
From: Gavin Hu
Currently, the flush is done by dequeuing the ring in a while loop. It is
much simpler to flush the queue by resetting the head and tail indices.
Signed-off-by: Gavin Hu
Reviewed-by: Ruifeng Wang
Reviewed-by: Honnappa Nagarahalli
---
lib/librte_ring/rte_ring.h | 20
: 8): 5.64
MP/MC bulk enq/dequeue (size: 8): 9.58
SP/SC bulk enq/dequeue (size: 32): 1.98
MP/MC bulk enq/dequeue (size: 32): 2.30
Fixes: 39368ebfc6 ("ring: introduce C11 memory model barrier option")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Honnappa Nagarahalli
R
/dequeue (size: 8): 6.67
MP/MC bulk enq/dequeue (size: 8): 13.12
SP/SC bulk enq/dequeue (size: 32): 2.04
MP/MC bulk enq/dequeue (size: 32): 3.26
Fixes: 39368ebfc6 ("ring: introduce C11 memory model barrier option")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Honnappa N
.
I have a patch to remove the specific options from the project arguments and
add it individually to C_FLAGS. It basically work, but it changed a lot of
meson.build files.
Any comments are welcome.
Best Regards,
Gavin
> -Original Message-
> From: Gavin Hu
> Sent: Tuesday, June
Pre-V5 version of this patch set(two patches) was aimed to fix the
Makefile based cross compiling issue and add a guiding doc.
This V6 patch set included the fixes for meson cross compiling.
Gavin Hu (5):
devtools: fix the Exec format error
build: fix the meson build warning
devtools: fix
From: gavin hu
This is the guide for cross compiling ARM64 DPDK from X86 hosts.
Signed-off-by: Gavin Hu
Reviewed-by: Steve Capper
Reviewed-by: Honnappa Nagarahalli
Acked-by: Marko Kovacevic
---
.../linux_gsg/cross_build_dpdk_for_arm64.rst | 137 +
doc/guides
check compiler
command line: aarch64-linux-gnu-gcc
dpdk/build-arm64-armv8/meson-private/sanitycheckc.c -o
dpdk/build-arm64-armv8/meson-private/sanitycheckc.exe
Fixes: a55277a788 ("devtools: add test script for meson builds")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by:
From: gavin hu
The "-Wimplicit-fallthrough=2" option was introduced into gcc 7.0, it was
enabled when the cross compiler gcc is greater than 7.0, but for the host
side buildtools/pmdinfogen, if the native gcc is older than 7.0, or the
host cc compiler is clang, it should not be enabled
ned-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
---
config/arm/arm64_thunderx_linuxapp_gcc | 1 +
1 file changed, 1 insertion(+)
diff --git a/config/arm/arm64_thunderx_linuxapp_gcc
b/config/arm/arm64_thunderx_linuxapp_gcc
index 7ff34af74..967d9d46d 100644
--- a/
.@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
---
devtools/test-meson-builds.sh | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
index 4afac76dd..74fe5abc8 100755
--- a/dev
unknown-warning-option]
Fixes: a55277a788 ("devtools: add test script for meson builds")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
Reviewed-by: Steve Capper
---
config/meson.build| 3 +--
drivers/meson.build | 3 +++
example
rg
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
---
devtools/test-meson-builds.sh | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
index 6bce3df7f..4afac76dd 100755
--- a/devtools/
script, test ninja or ninja-build and use it to work
out
the command to work out.
c) use host gcc and clang for the general v8a only to avoid overkill, and
use gcc
for others.
Gavin Hu (5):
devtools: fix the Exec format error
build: fix the meson build warning
devtools: fix the
anity check compiler
command line: aarch64-linux-gnu-gcc
dpdk/build-arm64-armv8/meson-private/sanitycheckc.c -o
dpdk/build-arm64-armv8/meson-private/sanitycheckc.exe
Fixes: a55277a788 ("devtools: add test script for meson builds")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Ph
From: gavin hu
This is the guide for cross compiling ARM64 DPDK from X86 hosts.
Signed-off-by: Gavin Hu
Reviewed-by: Steve Capper
Reviewed-by: Honnappa Nagarahalli
Acked-by: Marko Kovacevic
---
.../linux_gsg/cross_build_dpdk_for_arm64.rst | 137 +
doc/guides
From: gavin hu
The "-Wimplicit-fallthrough=2" option was introduced into gcc 7.0, it was
enabled when the cross compiler gcc is greater than 7.0, but for the host
side buildtools/pmdinfogen, if the native gcc is older than 7.0, or the
host cc compiler is clang, it should not be enabled
rg
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
---
devtools/test-meson-builds.sh | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
index 6bce3df7f..15a282978 100755
--- a/devtools/
ned-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
Acked-by: Bruce Richardson
---
config/arm/arm64_thunderx_linuxapp_gcc | 1 +
1 file changed, 1 insertion(+)
diff --git a/config/arm/arm64_thunderx_linuxapp_gcc
b/config/arm/arm64_thunderx_linuxapp_gcc
index 7ff34af74..967d9d
.@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
---
devtools/test-meson-builds.sh | 16
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
index 15a282978..879eda18e 10
unknown-warning-option]
Fixes: a55277a788 ("devtools: add test script for meson builds")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
Reviewed-by: Steve Capper
Acked-by: Bruce Richardson
---
config/meson.build| 3 +--
drivers/meson.bu
script, test ninja or ninja-build and use it to work
out
the command to work out.
c) use host gcc and clang for the general v8a only to avoid overkill, and
use gcc
for others.
4. v8 drop one of the new patch as it a similar fix was already upstreamed
Gavin Hu (4):
build: fix the
From: gavin hu
This is the guide for cross compiling ARM64 DPDK from X86 hosts.
Signed-off-by: Gavin Hu
Reviewed-by: Steve Capper
Reviewed-by: Honnappa Nagarahalli
Acked-by: Marko Kovacevic
---
.../linux_gsg/cross_build_dpdk_for_arm64.rst | 137 +
doc/guides
.@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
---
devtools/test-meson-builds.sh | 12 +++-
1 file changed, 11 insertions(+), 1 deletion(-)
diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
index 8447c704b..f75ebbdb1 100755
---
rg
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
---
devtools/test-meson-builds.sh | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
index f2ad05f32..8447c704b 100755
--- a/devtools/
From: gavin hu
The "-Wimplicit-fallthrough=2" option was introduced into gcc 7.0, it was
enabled when the cross compiler gcc is greater than 7.0, but for the host
side buildtools/pmdinfogen, if the native gcc is older than 7.0, or the
host cc compiler is clang, it should not be enabled
unknown-warning-option]
Fixes: a55277a788 ("devtools: add test script for meson builds")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
Reviewed-by: Steve Capper
Acked-by: Bruce Richardson
---
config/meson.build| 3 +--
drivers/meson.bu
ned-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
Acked-by: Bruce Richardson
---
config/arm/arm64_thunderx_linuxapp_gcc | 1 +
1 file changed, 1 insertion(+)
diff --git a/config/arm/arm64_thunderx_linuxapp_gcc
b/config/arm/arm64_thunderx_linuxapp_gcc
index 7ff34af74..967d9d
Hi Bruce,
This patch was dropped, thanks for your review.
Best Regards,
Gavin
> -Original Message-
> From: Bruce Richardson
> Sent: Thursday, June 14, 2018 6:34 PM
> To: Gavin Hu
> Cc: dev@dpdk.org; sta...@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v6 3/7] devtools: f
> -Original Message-
> From: Bruce Richardson
> Sent: Thursday, June 14, 2018 6:41 PM
> To: Gavin Hu
> Cc: dev@dpdk.org; sta...@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v6 5/7] devtools: fix the missing ninja
> command error on CentOS
>
> On Thu, Jun 1
> -Original Message-
> From: Bruce Richardson
> Sent: Thursday, June 14, 2018 6:45 PM
> To: Gavin Hu
> Cc: dev@dpdk.org; sta...@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v6 7/7] devtools: expand meson cross
> compiling coverage
>
> On Thu, Jun 14, 2018 at
ned-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
Acked-by: Bruce Richardson
---
config/arm/arm64_thunderx_linuxapp_gcc | 1 +
1 file changed, 1 insertion(+)
diff --git a/config/arm/arm64_thunderx_linuxapp_gcc
b/config/arm/arm64_thunderx_linuxapp_gcc
index 7ff34af74..967d9d
From: gavin hu
The "-Wimplicit-fallthrough=2" option was introduced into gcc 7.0, it was
enabled when the cross compiler gcc is greater than 7.0, but for the host
side buildtools/pmdinfogen, if the native gcc is older than 7.0, or the
host cc compiler is clang, it should not be enabled
From: gavin hu
This is the guide for cross compiling ARM64 DPDK from X86 hosts.
Signed-off-by: Gavin Hu
Reviewed-by: Steve Capper
Reviewed-by: Honnappa Nagarahalli
Acked-by: Marko Kovacevic
---
.../linux_gsg/cross_build_dpdk_for_arm64.rst | 137 +
doc/guides
the comments, as well some commit message
refined.
a) move if clause out of the loop for a better program structure.
b) echo the correct ninja command actually used.
Gavin Hu (4):
build: fix the meson build warning
devtools: fix the missing ninja command error
build: fix the meson
;)
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
---
devtools/test-meson-builds.sh | 13 +++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
index f2ad05f32..9bb5b93bd 10
.@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
---
devtools/test-meson-builds.sh | 13 +++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
index 9bb5b93bd..8c33ece93 10
unknown-warning-option]
Fixes: a55277a788 ("devtools: add test script for meson builds")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
Reviewed-by: Steve Capper
Acked-by: Bruce Richardson
---
config/meson.build| 3 +--
drivers/meson.bu
Hi Bruce,
Thanks for your review, new v9 patch set was submitted.
Any more comments are welcome!
Best Regards,
Gavin
> -Original Message-
> From: Bruce Richardson
> Sent: Friday, June 15, 2018 4:28 PM
> To: Gavin Hu
> Cc: dev@dpdk.org; nd ; sta...@dpdk.org
> Subj
the comments, as well some commit message
refined.
a) move if clause out of the loop for a better program structure.
b) echo the correct ninja command actually used.
6. Fix the whitespace issues.
Gavin Hu (4):
build: fix the meson build warning
devtools: fix the missing ninja command
From: gavin hu
This is the guide for cross compiling ARM64 DPDK from X86 hosts.
Signed-off-by: Gavin Hu
Reviewed-by: Steve Capper
Reviewed-by: Honnappa Nagarahalli
Acked-by: Marko Kovacevic
---
.../linux_gsg/cross_build_dpdk_for_arm64.rst | 137 +
doc/guides
;)
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
---
devtools/test-meson-builds.sh | 13 +++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
index f2ad05f32..9bb5b93bd 10
From: gavin hu
The "-Wimplicit-fallthrough=2" option was introduced into gcc 7.0, it was
enabled when the cross compiler gcc is greater than 7.0, but for the host
side buildtools/pmdinfogen, if the native gcc is older than 7.0, or the
host cc compiler is clang, it should not be enabled
ned-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
Acked-by: Bruce Richardson
---
config/arm/arm64_thunderx_linuxapp_gcc | 1 +
1 file changed, 1 insertion(+)
diff --git a/config/arm/arm64_thunderx_linuxapp_gcc
b/config/arm/arm64_thunderx_linuxapp_gcc
index 7ff34af74..967d9d
unknown-warning-option]
Fixes: a55277a788 ("devtools: add test script for meson builds")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
Reviewed-by: Steve Capper
Acked-by: Bruce Richardson
---
config/meson.build| 3 +--
drivers/meson.bu
.@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
---
devtools/test-meson-builds.sh | 13 +++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
index 9bb5b93bd..f1553b7bd 10
Thanks Bruce, I submitted v10 patch set fixing the whitespace issues.
> -Original Message-
> From: Bruce Richardson
> Sent: Thursday, June 14, 2018 6:45 PM
> To: Gavin Hu
> Cc: dev@dpdk.org; sta...@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v6 7/7] devtools:
shorter and more readable.
Gavin Hu (4):
build: fix the meson build warning
devtools: fix the missing ninja command error
build: fix the meson cross compile error
devtools: expand meson cross compiling test coverage
gavin hu (2):
mk: fix makefile based cross build errors
doc: add a guide
From: gavin hu
The "-Wimplicit-fallthrough=2" option was introduced into gcc 7.0, it was
enabled when the cross compiler gcc is greater than 7.0, but for the host
side buildtools/pmdinfogen, if the native gcc is older than 7.0, or the
host cc compiler is clang, it should not be enabled
From: gavin hu
This is the guide for cross compiling ARM64 DPDK from X86 hosts.
Signed-off-by: Gavin Hu
Reviewed-by: Steve Capper
Reviewed-by: Honnappa Nagarahalli
Acked-by: Marko Kovacevic
---
.../linux_gsg/cross_build_dpdk_for_arm64.rst | 137 +
doc/guides
: add test script for meson builds")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
---
devtools/test-meson-builds.sh | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson
;)
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
---
devtools/test-meson-builds.sh | 13 +++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
index f2ad05f32..9bb5b93bd 10
ned-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
Acked-by: Bruce Richardson
---
config/arm/arm64_thunderx_linuxapp_gcc | 1 +
1 file changed, 1 insertion(+)
diff --git a/config/arm/arm64_thunderx_linuxapp_gcc
b/config/arm/arm64_thunderx_linuxapp_gcc
index 7ff34af74..967d9d
unknown-warning-option]
Fixes: a55277a788 ("devtools: add test script for meson builds")
Cc: sta...@dpdk.org
Signed-off-by: Gavin Hu
Reviewed-by: Phil Yang
Reviewed-by: Song Zhu
Reviewed-by: Steve Capper
Acked-by: Bruce Richardson
---
config/meson.build| 3 +--
drivers/meson.bu
Hi Bruce,
I submitted v11, please have a look, thanks for your review.
Best Regards,
Gavin
> -Original Message-
> From: Bruce Richardson
> Sent: Friday, June 15, 2018 10:15 PM
> To: Gavin Hu
> Cc: dev@dpdk.org; nd ; sta...@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH
dir for brevity
Gavin Hu (4):
build: fix the meson build warning
devtools: fix the missing ninja command error
build: fix the meson cross compile error
devtools: expand meson cross compiling test coverage
gavin hu (2):
mk: fix makefile based cross build errors
doc: add a guide doc for
From: gavin hu
This is the guide for cross compiling ARM64 DPDK from X86 hosts.
Signed-off-by: Gavin Hu
Reviewed-by: Steve Capper
Reviewed-by: Honnappa Nagarahalli
Acked-by: Marko Kovacevic
---
.../linux_gsg/cross_build_dpdk_for_arm64.rst | 137 +
doc/guides
1 - 100 of 666 matches
Mail list logo
