[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 20/34] linux-user: Remove redundant default acti
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 20/34] linux-user: Remove redundant default action check in queue_signal() |
Date: |
Fri, 11 Sep 2015 15:41:11 +0100 |
On 6 September 2015 at 00:57, Timothy E Baldwin
<address@hidden> wrote:
> Both queue_signal() and process_pending_signals() did check for default
> actions of signals, this is redundant and also causes fatal and stopping
> signals to incorrectly cause guest system calls to be interrupted.
>
> The code in queue_signal() is removed.
>
> Signed-off-by: Timothy Edward Baldwin <address@hidden>
Is it really OK to always queue signals? This doesn't seem obviously
true to me, so some explanation of why it is would be helpful.
Also it looks like you've lost the handling of "we got a SIGSEGV
but the guest has blocked SIGSEGV".
thanks
-- PMM
- Re: [Qemu-devel] [PATCH 10/34] linux-user: Support for restarting system calls for Microblaze targets, (continued)
- [Qemu-devel] [PATCH 07/34] linux-user: Support for restarting system calls for PPC targets, Timothy E Baldwin, 2015/09/05
- [Qemu-devel] [PATCH 14/34] linux-user: Use safe_syscall for read and write system calls, Timothy E Baldwin, 2015/09/05
- [Qemu-devel] [PATCH 16/34] linux-user: Use safe_syscall for open and openat system calls, Timothy E Baldwin, 2015/09/05
- [Qemu-devel] [PATCH 08/34] linux-user: Support for restarting system calls for SPARC targets, Timothy E Baldwin, 2015/09/05
- [Qemu-devel] [PATCH 20/34] linux-user: Remove redundant default action check in queue_signal(), Timothy E Baldwin, 2015/09/05
- Re: [Qemu-devel] [PATCH 20/34] linux-user: Remove redundant default action check in queue_signal(),
Peter Maydell <=
- [Qemu-devel] [PATCH 26/34] linux-user: Restart kill() if signal pending, Timothy E Baldwin, 2015/09/05
- [Qemu-devel] [PATCH 25/34] linux-user: Restart exit() if signal pending, Timothy E Baldwin, 2015/09/05
- [Qemu-devel] [PATCH 11/34] linux-user: Support for restarting system calls for SH4 targets, Timothy E Baldwin, 2015/09/05
- [Qemu-devel] [PATCH 27/34] linux-user: pause() should not pause if signal pending, Timothy E Baldwin, 2015/09/05
- [Qemu-devel] [PATCH 05/34] linux-user: Support for restarting system calls for ARM targets, Timothy E Baldwin, 2015/09/05
- [Qemu-devel] [PATCH 13/34] linux-user: Fix signal before blocking system calls race and SA_RESTART, Timothy E Baldwin, 2015/09/05