[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 14/14] bsd-user/signal: Create a dummy signal queueing functi
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH 14/14] bsd-user/signal: Create a dummy signal queueing function |
Date: |
Sun, 26 Sep 2021 20:34:48 +0200 |
On Sun, Sep 26, 2021 at 8:14 PM Warner Losh <imp@bsdimp.com> wrote:
> On Fri, Sep 24, 2021 at 6:00 AM Richard Henderson
> <richard.henderson@linaro.org> wrote:
>> 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 <imp@bsdimp.com>
>> > ---
>> > bsd-user/qemu.h | 1 +
>> > bsd-user/signal.c | 8 ++++++++
>> > 2 files changed, 9 insertions(+)
>> > +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.
>
>
> In this stubbed out version, there's no errors. But bsd-user can return
> -EAGAIN when
> alloc_sigqueue fails (which it can if there are no free qemu_sigqueue
> structures in
> the preallocated list0. However, having said that, nothing checks the return
> value
> so it's as if it is void... So I'll change it to void here...
kinda related:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg836833.html
- [PATCH 06/14] bsd-user: move TARGET_MC_GET_CLEAR_RET to target_os_signal.h, (continued)
Re: [PATCH 14/14] bsd-user/signal: Create a dummy signal queueing function, Philippe Mathieu-Daudé, 2021/09/25