DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH] devtools: add RISC-V vector in meson build test
@ 2025-11-10  2:52 Sun Yuechi
  2025-11-10  2:54 ` sunyuechi
  0 siblings, 1 reply; 3+ messages in thread
From: Sun Yuechi @ 2025-11-10  2:52 UTC (permalink / raw)
  To: dev; +Cc: Sun Yuechi, Bruce Richardson

Signed-off-by: Sun Yuechi <sunyuechi@iscas.ac.cn>
---
 devtools/test-meson-builds.sh | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
index 4fff1f7177..4f07f84eb0 100755
--- a/devtools/test-meson-builds.sh
+++ b/devtools/test-meson-builds.sh
@@ -290,6 +290,10 @@ build build-ppc64-power8-gcc $f ABI $use_shared
 f=$srcdir/config/riscv/riscv64_linux_gcc
 build build-riscv64-generic-gcc $f ABI $use_shared
 
+# RISC-V vector (rv64gcv)
+f=$srcdir/config/riscv/riscv64_rv64gcv_linux_gcc
+build build-riscv64_rv64gcv_gcc $f ABI $use_shared
+
 # Test installation of the x86-generic target, to be used for checking
 # the sample apps build using the pkg-config file for cflags and libs
 load_env cc
-- 
2.51.2


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

* Re: [PATCH] devtools: add RISC-V vector in meson build test
  2025-11-10  2:52 [PATCH] devtools: add RISC-V vector in meson build test Sun Yuechi
@ 2025-11-10  2:54 ` sunyuechi
  2025-11-10  9:39   ` Thomas Monjalon
  0 siblings, 1 reply; 3+ messages in thread
From: sunyuechi @ 2025-11-10  2:54 UTC (permalink / raw)
  To: dev; +Cc: Bruce Richardson, thomas

&gt; Maybe we want to replace the generic RISC-V build with the vector one.
&gt; It could be discussed in another patch.

I think both scalar and vector build detection should exist,
because I've seen several times in other projects that after adding vector optimization, 
the scalar version build failed due to incorrect compilation conditions. 
Some cautious projects also test various extension combinations. What do you think?


&gt; -----原始邮件-----
&gt; 发件人: "Sun Yuechi" <sunyuechi@iscas.ac.cn>
&gt; 发送时间: 2025-11-10 10:52:20 (星期一)
&gt; 收件人: dev@dpdk.org
&gt; 抄送: "Sun Yuechi" <sunyuechi@iscas.ac.cn>, "Bruce Richardson" <bruce.richardson@intel.com>
&gt; 主题: [PATCH] devtools: add RISC-V vector in meson build test
&gt; 
&gt; Signed-off-by: Sun Yuechi <sunyuechi@iscas.ac.cn>
&gt; ---
&gt;  devtools/test-meson-builds.sh | 4 ++++
&gt;  1 file changed, 4 insertions(+)
&gt; 
&gt; diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh
&gt; index 4fff1f7177..4f07f84eb0 100755
&gt; --- a/devtools/test-meson-builds.sh
&gt; +++ b/devtools/test-meson-builds.sh
&gt; @@ -290,6 +290,10 @@ build build-ppc64-power8-gcc $f ABI $use_shared
&gt;  f=$srcdir/config/riscv/riscv64_linux_gcc
&gt;  build build-riscv64-generic-gcc $f ABI $use_shared
&gt;  
&gt; +# RISC-V vector (rv64gcv)
&gt; +f=$srcdir/config/riscv/riscv64_rv64gcv_linux_gcc
&gt; +build build-riscv64_rv64gcv_gcc $f ABI $use_shared
&gt; +
&gt;  # Test installation of the x86-generic target, to be used for checking
&gt;  # the sample apps build using the pkg-config file for cflags and libs
&gt;  load_env cc
&gt; -- 
&gt; 2.51.2
</sunyuechi@iscas.ac.cn></bruce.richardson@intel.com></sunyuechi@iscas.ac.cn></sunyuechi@iscas.ac.cn>

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

* Re: [PATCH] devtools: add RISC-V vector in meson build test
  2025-11-10  2:54 ` sunyuechi
@ 2025-11-10  9:39   ` Thomas Monjalon
  0 siblings, 0 replies; 3+ messages in thread
From: Thomas Monjalon @ 2025-11-10  9:39 UTC (permalink / raw)
  To: sunyuechi; +Cc: dev, Bruce Richardson, David Marchand

10/11/2025 03:54, sunyuechi@iscas.ac.cn:
> &gt; Maybe we want to replace the generic RISC-V build with the vector one.
> &gt; It could be discussed in another patch.
> 
> I think both scalar and vector build detection should exist,
> because I've seen several times in other projects that after adding vector optimization, 
> the scalar version build failed due to incorrect compilation conditions. 
> Some cautious projects also test various extension combinations. What do you think?

We can test it in a CI, but test-meson-builds.sh is supposed to be run
very frequently by all maintainers. I think it is too much.
As a comparison, in this script we test only
1 Arm target, 1 LoongArch, 1 Power.



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

end of thread, other threads:[~2025-11-10  9:39 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-11-10  2:52 [PATCH] devtools: add RISC-V vector in meson build test Sun Yuechi
2025-11-10  2:54 ` sunyuechi
2025-11-10  9:39   ` 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).