<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 10, 2013 at 7:38 AM, Mathieu Desnoyers <span dir="ltr"><<a href="mailto:mathieu.desnoyers@efficios.com" target="_blank">mathieu.desnoyers@efficios.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Dave,<br>
<br>
Nice catch ! I've reworked the patch to also fix compat_arch_x86.c and<br>
use SIG_BLOCK at those sites (details in changelog). Here is the version<br>
I plan to merge into userspace rcu. Thanks for spotting this!<br>
<br>
Mathieu<br>
<br>
commit 6ed4b2e64c6cef5d6d1a1c1943f29dbf4edf026a<br>
Author: Mathieu Desnoyers <<a href="mailto:mathieu.desnoyers@efficios.com">mathieu.desnoyers@efficios.com</a>><br>
Date:   Fri May 10 07:30:18 2013 -0400<br>
    Updates by Mathieu Desnoyers:<br>
<br>
    - Use SIG_BLOCK instead of SIG_SETMASK when setting a filled mask. This<br>
      has the same behavior in this case (since we're blocking all signals),<br>
      but is semantically neater: if we ever some signals from that mask,<br></blockquote><div><br></div><div>if we ever what? Looks like a missing word in this changelog. <br></div><div> </div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

      we'd like to to a union with the signal mask already blocked by the<br>
      application.<br>
    - Also fix incorrect signal masking in compat_arch_x86.c.<br>
<br>
    Reported-by: David Pelton <<a href="mailto:dpelton@ciena.com">dpelton@ciena.com</a>><br>
    Signed-off-by: Mathieu Desnoyers <<a href="mailto:mathieu.desnoyers@efficios.com">mathieu.desnoyers@efficios.com</a>></blockquote></div></div></div>