All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] linux-user/strace.list: add riscv_hwprobe entry
@ 2025-07-28 17:06 Daniel Henrique Barboza
  2025-07-28 17:15 ` Richard Henderson
  2025-07-29  3:17 ` Alistair Francis
  0 siblings, 2 replies; 3+ messages in thread
From: Daniel Henrique Barboza @ 2025-07-28 17:06 UTC (permalink / raw)
  To: qemu-devel
  Cc: qemu-riscv, alistair.francis, liwei1518, zhiwei_liu, palmer,
	Daniel Henrique Barboza

We're missing a strace entry for riscv_hwprobe, and using -strace will
report it as "Unknown syscall 258".

After this patch we'll have:

$ ./build/qemu-riscv64 -strace test_mutex_riscv
110182 riscv_hwprobe(0x7f207efdc700,1,0,0,0,0) = 0
110182 brk(NULL) = 0x0000000000082000
(...)

Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
---
 linux-user/strace.list | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/linux-user/strace.list b/linux-user/strace.list
index fdf94ef32a..ab818352a9 100644
--- a/linux-user/strace.list
+++ b/linux-user/strace.list
@@ -1716,3 +1716,6 @@
 { TARGET_NR_clock_gettime64, "clock_gettime64" , NULL, print_clock_gettime64,
                            print_syscall_ret_clock_gettime64 },
 #endif
+#ifdef TARGET_NR_riscv_hwprobe
+{ TARGET_NR_riscv_hwprobe, "riscv_hwprobe" , "%s(%p,%d,%d,%d,%d,%d)", NULL, NULL },
+#endif
-- 
2.50.1



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

* Re: [PATCH] linux-user/strace.list: add riscv_hwprobe entry
  2025-07-28 17:06 [PATCH] linux-user/strace.list: add riscv_hwprobe entry Daniel Henrique Barboza
@ 2025-07-28 17:15 ` Richard Henderson
  2025-07-29  3:17 ` Alistair Francis
  1 sibling, 0 replies; 3+ messages in thread
From: Richard Henderson @ 2025-07-28 17:15 UTC (permalink / raw)
  To: qemu-devel

On 7/28/25 07:06, Daniel Henrique Barboza wrote:
> We're missing a strace entry for riscv_hwprobe, and using -strace will
> report it as "Unknown syscall 258".
> 
> After this patch we'll have:
> 
> $ ./build/qemu-riscv64 -strace test_mutex_riscv
> 110182 riscv_hwprobe(0x7f207efdc700,1,0,0,0,0) = 0
> 110182 brk(NULL) = 0x0000000000082000
> (...)
> 
> Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
> ---
>   linux-user/strace.list | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/linux-user/strace.list b/linux-user/strace.list
> index fdf94ef32a..ab818352a9 100644
> --- a/linux-user/strace.list
> +++ b/linux-user/strace.list
> @@ -1716,3 +1716,6 @@
>   { TARGET_NR_clock_gettime64, "clock_gettime64" , NULL, print_clock_gettime64,
>                              print_syscall_ret_clock_gettime64 },
>   #endif
> +#ifdef TARGET_NR_riscv_hwprobe
> +{ TARGET_NR_riscv_hwprobe, "riscv_hwprobe" , "%s(%p,%d,%d,%d,%d,%d)", NULL, NULL },
> +#endif

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~


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

* Re: [PATCH] linux-user/strace.list: add riscv_hwprobe entry
  2025-07-28 17:06 [PATCH] linux-user/strace.list: add riscv_hwprobe entry Daniel Henrique Barboza
  2025-07-28 17:15 ` Richard Henderson
@ 2025-07-29  3:17 ` Alistair Francis
  1 sibling, 0 replies; 3+ messages in thread
From: Alistair Francis @ 2025-07-29  3:17 UTC (permalink / raw)
  To: Daniel Henrique Barboza
  Cc: qemu-devel, qemu-riscv, alistair.francis, liwei1518, zhiwei_liu,
	palmer

On Tue, Jul 29, 2025 at 3:07 AM Daniel Henrique Barboza
<dbarboza@ventanamicro.com> wrote:
>
> We're missing a strace entry for riscv_hwprobe, and using -strace will
> report it as "Unknown syscall 258".
>
> After this patch we'll have:
>
> $ ./build/qemu-riscv64 -strace test_mutex_riscv
> 110182 riscv_hwprobe(0x7f207efdc700,1,0,0,0,0) = 0
> 110182 brk(NULL) = 0x0000000000082000
> (...)
>
> Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>

Thanks!

Applied to riscv-to-apply.next

Alistair

> ---
>  linux-user/strace.list | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/linux-user/strace.list b/linux-user/strace.list
> index fdf94ef32a..ab818352a9 100644
> --- a/linux-user/strace.list
> +++ b/linux-user/strace.list
> @@ -1716,3 +1716,6 @@
>  { TARGET_NR_clock_gettime64, "clock_gettime64" , NULL, print_clock_gettime64,
>                             print_syscall_ret_clock_gettime64 },
>  #endif
> +#ifdef TARGET_NR_riscv_hwprobe
> +{ TARGET_NR_riscv_hwprobe, "riscv_hwprobe" , "%s(%p,%d,%d,%d,%d,%d)", NULL, NULL },
> +#endif
> --
> 2.50.1
>
>


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

end of thread, other threads:[~2025-07-29  3:18 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-28 17:06 [PATCH] linux-user/strace.list: add riscv_hwprobe entry Daniel Henrique Barboza
2025-07-28 17:15 ` Richard Henderson
2025-07-29  3:17 ` Alistair Francis

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.