[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 2/3] linux-user: Rename user_force_sig tracepoint to match fu

From: Laurent Vivier
Subject: Re: [PATCH 2/3] linux-user: Rename user_force_sig tracepoint to match function name
Date: Tue, 18 Jan 2022 12:41:17 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.4.0

Le 14/01/2022 à 16:37, Peter Maydell a écrit :
In commit c599d4d6d6e9bfdb64 in 2016 we renamed the old force_sig()
function to dump_core_and_abort(), but we forgot to rename the
associated tracepoint.  Rename the tracepoint to to match the
function it's called from.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
  linux-user/signal.c     | 2 +-
  linux-user/trace-events | 2 +-
  2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/linux-user/signal.c b/linux-user/signal.c
index f813b4f18e4..bfbbeab9ad2 100644
--- a/linux-user/signal.c
+++ b/linux-user/signal.c
@@ -734,7 +734,7 @@ static void QEMU_NORETURN dump_core_and_abort(int 
      struct sigaction act;
host_sig = target_to_host_signal(target_sig);
-    trace_user_force_sig(env, target_sig, host_sig);
+    trace_user_dump_core_and_abort(env, target_sig, host_sig);
      gdb_signalled(env, target_sig);
/* dump core if supported by target binary format */
diff --git a/linux-user/trace-events b/linux-user/trace-events
index e7d2f54e940..f33717f248a 100644
--- a/linux-user/trace-events
+++ b/linux-user/trace-events
@@ -9,7 +9,7 @@ user_setup_frame(void *env, uint64_t frame_addr) "env=%p 
  user_setup_rt_frame(void *env, uint64_t frame_addr) "env=%p 
  user_do_rt_sigreturn(void *env, uint64_t frame_addr) "env=%p 
  user_do_sigreturn(void *env, uint64_t frame_addr) "env=%p 
-user_force_sig(void *env, int target_sig, int host_sig) "env=%p signal %d (host 
+user_dump_core_and_abort(void *env, int target_sig, int host_sig) "env=%p signal %d 
(host %d)"
  user_handle_signal(void *env, int target_sig) "env=%p signal %d"
  user_host_signal(void *env, int host_sig, int target_sig) "env=%p signal %d 
(target %d)"
  user_queue_signal(void *env, int target_sig) "env=%p signal %d"

Applied to my trivial-patches branch.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]