Skip to content

Commit

Permalink
sigfd: use pthread_sigmask
Browse files Browse the repository at this point in the history
Qemu uses signalfd to figure out, if a signal occured without the need
to actually receive the signal. Instead, it can read from the fd to receive
its news.

Now, we obviously don't always have signalfd around. Especially not on
non-Linux systems. So what we do there is that we create a new thread,
block that thread on all signals and simply call sigwait to wait for a
signal we're interested in to occur.

This all sounds great, but what we're really doing is:

    sigset_t all;

    sigfillset(&all);
    sigprocmask(SIG_BLOCK, &all, NULL);

which - on Darwin - blocks all signals on the current _process_, not only
on the current thread. To block signals on the thread, we can use
pthread_sigmask().

This patch does that, assuming that my above analysis is correct, and thus
renders Qemu useable on Darwin again.

Reported-by: Andreas Färber <andreas.faerber@web.de>
Acked-by: Paolo Bonizni <pbonzini@redhat.com>
CC: Jan Kiszka <jan.kiszka@siemens.com>
CC: Anthony Liguori <anthony@codemonkey.ws>
Signed-off-by: Alexander Graf <agraf@suse.de>
Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
  • Loading branch information
agraf authored and edgarigl committed Jun 10, 2011
1 parent 4482939 commit 09716e4
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion compatfd.c
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ static void *sigwait_compat(void *opaque)
sigset_t all;

sigfillset(&all);
sigprocmask(SIG_BLOCK, &all, NULL);
pthread_sigmask(SIG_BLOCK, &all, NULL);

while (1) {
int sig;
Expand Down

0 comments on commit 09716e4

Please sign in to comment.