On 9/21/21 11:14 PM, Warner Losh wrote:
Create dummy signal queueing function so we can start to integrate other architectures (at the cost of signals remaining broken) to tame the dependency graph a bit and to bring in signals in a more controlled fashion.Signed-off-by: Warner Losh <i...@bsdimp.com> --- bsd-user/qemu.h | 1 + bsd-user/signal.c | 8 ++++++++ 2 files changed, 9 insertions(+) diff --git a/bsd-user/qemu.h b/bsd-user/qemu.h index 5a2fd87e44..85d1f8fd2a 100644 --- a/bsd-user/qemu.h +++ b/bsd-user/qemu.h @@ -209,6 +209,7 @@ void process_pending_signals(CPUArchState *cpu_env); void signal_init(void); long do_sigreturn(CPUArchState *env); long do_rt_sigreturn(CPUArchState *env); +int queue_signal(CPUArchState *env, int sig, target_siginfo_t *info); abi_long do_sigaltstack(abi_ulong uss_addr, abi_ulong uoss_addr, abi_ulong sp);/* mmap.c */diff --git a/bsd-user/signal.c b/bsd-user/signal.c index ad6d935569..4e7f618944 100644 --- a/bsd-user/signal.c +++ b/bsd-user/signal.c @@ -19,6 +19,14 @@ #include "qemu/osdep.h"#include "qemu.h"+/*
Whacky whitespace.
+ * Queue a signal so that it will be send to the virtual CPU as soon as
+ * possible.
+ */
+int queue_signal(CPUArchState *env, int sig, target_siginfo_t *info)
+{
+ return 1;
+}
Both here and in linux-user, there are no error conditions. We should change the return to void.
Also, consider folding in the signal-common.h cleanup soon. But don't let either hold you up too much with rebasing. Reviewed-by: Richard Henderson <richard.hender...@linaro.org> r~
