DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] devtools: reduce list of Arm builds tested with meson
@ 2019-11-20 23:42 Thomas Monjalon
  2019-11-20 23:47 ` Thomas Monjalon
  0 siblings, 1 reply; 2+ messages in thread
From: Thomas Monjalon @ 2019-11-20 23:42 UTC (permalink / raw)
  To: honnappa.nagarahalli, jerinj, hemant.agrawal; +Cc: dev, David Marchand

The list of Arm configs is growing:
	config/arm/arm64_armada_linux_gcc
	config/arm/arm64_armv8_linux_gcc
	config/arm/arm64_bluefield_linux_gcc
	config/arm/arm64_dpaa_linux_gcc
	config/arm/arm64_emag_linux_gcc
	config/arm/arm64_n1sdp_linux_gcc
	config/arm/arm64_octeontx2_linux_gcc
	config/arm/arm64_thunderx2_linux_gcc
	config/arm/arm64_thunderx_linux_gcc

In order to keep testing time reasonable,
and also because n1sdp is merged without a related fix in tests,
the list of configs is reduced in the script test-meson-builds.sh.

The list of tested Arm builds becomes:
	build-arm64-host-clang (armv8a)
	build-arm64-bluefield
	build-arm64-dpaa
	build-arm64-octeontx2

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
Acked-by: David Marchand <david.marchand@redhat.com>
---
 devtools/test-meson-builds.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
index e3a5a5d51d..07d42985e6 100755
--- a/devtools/test-meson-builds.sh
+++ b/devtools/test-meson-builds.sh
@@ -117,7 +117,7 @@ export CC="clang"
 build build-arm64-host-clang $c $use_shared \
 	--cross-file $srcdir/config/arm/arm64_armv8_linux_gcc
 # all gcc/arm configurations
-for f in $srcdir/config/arm/arm*gcc ; do
+for f in $srcdir/config/arm/arm64_[bdo]*gcc ; do
 	export CC="$CCACHE gcc"
 	build build-$(basename $f | tr '_' '-' | cut -d'-' -f-2) $c \
 		$use_shared --cross-file $f
-- 
2.23.0


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

* Re: [dpdk-dev] [PATCH] devtools: reduce list of Arm builds tested with meson
  2019-11-20 23:42 [dpdk-dev] [PATCH] devtools: reduce list of Arm builds tested with meson Thomas Monjalon
@ 2019-11-20 23:47 ` Thomas Monjalon
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Monjalon @ 2019-11-20 23:47 UTC (permalink / raw)
  To: bruce.richardson
  Cc: dev, honnappa.nagarahalli, jerinj, hemant.agrawal, David Marchand

21/11/2019 00:42, Thomas Monjalon:
> The list of Arm configs is growing:
> 	config/arm/arm64_armada_linux_gcc
> 	config/arm/arm64_armv8_linux_gcc
> 	config/arm/arm64_bluefield_linux_gcc
> 	config/arm/arm64_dpaa_linux_gcc
> 	config/arm/arm64_emag_linux_gcc
> 	config/arm/arm64_n1sdp_linux_gcc
> 	config/arm/arm64_octeontx2_linux_gcc
> 	config/arm/arm64_thunderx2_linux_gcc
> 	config/arm/arm64_thunderx_linux_gcc
> 
> In order to keep testing time reasonable,
> and also because n1sdp is merged without a related fix in tests,
> the list of configs is reduced in the script test-meson-builds.sh.
> 
> The list of tested Arm builds becomes:
> 	build-arm64-host-clang (armv8a)
> 	build-arm64-bluefield
> 	build-arm64-dpaa
> 	build-arm64-octeontx2
> 
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> Acked-by: David Marchand <david.marchand@redhat.com>

It is keeping one target per vendor (Arm, Marvell, Mellanox, NXP).
I hope it is enough.
When rebuilding all, we had 10 Arm targets (including the host-clang),
it is really too much in my opinion. It cuts from 10 to 4.
And first benefit is to skip n1sdp which we should fix properly in -rc4.

Applied quickly before closing -rc3.




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

end of thread, other threads:[~2019-11-20 23:47 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-20 23:42 [dpdk-dev] [PATCH] devtools: reduce list of Arm builds tested with meson Thomas Monjalon
2019-11-20 23:47 ` 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).