From: Julian Seward <jseward@acm.org>
Date: Sat, 9 Jul 2005 10:43:45 +0000 (+0000)
Subject: Add comments from PaulM.
X-Git-Tag: svn/VALGRIND_3_0_0~178
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=8bbf43ab9c124c07719b61d3419e95c7640fd126;p=thirdparty%2Fvalgrind.git

Add comments from PaulM.


git-svn-id: svn://svn.valgrind.org/valgrind/trunk@4140
---

diff --git a/coregrind/m_signals.c b/coregrind/m_signals.c
index 2c1a04e8a0..edcdbdb44a 100644
--- a/coregrind/m_signals.c
+++ b/coregrind/m_signals.c
@@ -449,6 +449,11 @@ static void handle_SCSS_change ( Bool force_update )
 #     if !defined(VGP_ppc32_linux)
       ksa.sa_restorer = my_sigreturn;
 #     endif
+      /* Re above ifdef (also the assertion below), PaulM says:
+         The sa_restorer field is not used at all on ppc.  Glibc
+         converts the sigaction you give it into a kernel sigaction,
+         but it doesn't put anything in the sa_restorer field.
+      */
 
       /* block all signals in handler */
       VG_(sigfillset)( &ksa.sa_mask );