Re: [PATCH] coroutine-sigaltstack: Add SIGUSR2 mutex

2021-01-27 Thread Eric Blake
On 1/25/21 6:03 AM, Max Reitz wrote: > Disposition (action) for any given signal is global for the process. > When two threads run coroutine-sigaltstack's qemu_coroutine_new() > concurrently, they may interfere with each other: One of them may revert > the SIGUSR2 handler to SIG_DFL, between the ot

Re: [PATCH] coroutine-sigaltstack: Add SIGUSR2 mutex

2021-01-26 Thread Laszlo Ersek
On 01/26/21 14:16, Max Reitz wrote: > On 26.01.21 13:44, Vladimir Sementsov-Ogievskiy wrote: >> 25.01.2021 15:03, Max Reitz wrote: >>> Disposition (action) for any given signal is global for the process. >>> When two threads run coroutine-sigaltstack's qemu_coroutine_new() >>> concurrently, they ma

Re: [PATCH] coroutine-sigaltstack: Add SIGUSR2 mutex

2021-01-26 Thread Stefan Hajnoczi
On Mon, Jan 25, 2021 at 01:03:05PM +0100, Max Reitz wrote: > Disposition (action) for any given signal is global for the process. > When two threads run coroutine-sigaltstack's qemu_coroutine_new() > concurrently, they may interfere with each other: One of them may revert > the SIGUSR2 handler to S

Re: [PATCH] coroutine-sigaltstack: Add SIGUSR2 mutex

2021-01-26 Thread Max Reitz
On 26.01.21 13:44, Vladimir Sementsov-Ogievskiy wrote: 25.01.2021 15:03, Max Reitz wrote: Disposition (action) for any given signal is global for the process. When two threads run coroutine-sigaltstack's qemu_coroutine_new() concurrently, they may interfere with each other: One of them may rever

Re: [PATCH] coroutine-sigaltstack: Add SIGUSR2 mutex

2021-01-26 Thread Vladimir Sementsov-Ogievskiy
25.01.2021 15:03, Max Reitz wrote: Disposition (action) for any given signal is global for the process. When two threads run coroutine-sigaltstack's qemu_coroutine_new() concurrently, they may interfere with each other: One of them may revert the SIGUSR2 handler to SIG_DFL, between the other thre

Re: [PATCH] coroutine-sigaltstack: Add SIGUSR2 mutex

2021-01-25 Thread Laszlo Ersek
On 01/25/21 13:03, Max Reitz wrote: > Disposition (action) for any given signal is global for the process. > When two threads run coroutine-sigaltstack's qemu_coroutine_new() > concurrently, they may interfere with each other: One of them may revert > the SIGUSR2 handler to SIG_DFL, between the oth

[PATCH] coroutine-sigaltstack: Add SIGUSR2 mutex

2021-01-25 Thread Max Reitz
Disposition (action) for any given signal is global for the process. When two threads run coroutine-sigaltstack's qemu_coroutine_new() concurrently, they may interfere with each other: One of them may revert the SIGUSR2 handler to SIG_DFL, between the other thread (a) setting up coroutine_trampolin