[v2] signal(7) fixes

[PATCH v2 1/2] signal.7: Clearly describe ucontext kernel dump to userspace

Posted by Dev Jain 1 year, 6 months ago

The set of blocked signals is updated from struct sigaction after the
kernel dumps ucontext. Mention this to avoid misunderstanding.

Signed-off-by: Dev Jain <dev.jain@arm.com>
---
 man/man7/signal.7 | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/man/man7/signal.7 b/man/man7/signal.7
index 4ba5306c8..7d22a7cfe 100644
--- a/man/man7/signal.7
+++ b/man/man7/signal.7
@@ -282,7 +282,13 @@ the thread's alternate signal stack settings.
 .B SA_SIGINFO
 flag, then the above information is accessible via the
 .I ucontext_t
-object that is pointed to by the third argument of the signal handler.)
+object that is pointed to by the third argument of the signal handler.
+This object reflects the state at which the signal is delivered, rather
+than in the handler;
+for example, the mask of blocked signals stored in this object will not
+contain the mask of new signals blocked through
+.BR sigaction(2)).
+
 .IP (1.4)
 Any signals specified in
 .I act\->sa_mask
-- 
2.34.1

Re: [PATCH v2 1/2] signal.7: Clearly describe ucontext kernel dump to userspace

Posted by Alejandro Colomar 1 year, 6 months ago

On Tue, Jun 11, 2024 at 02:38:22PM GMT, Dev Jain wrote:
> The set of blocked signals is updated from struct sigaction after the
> kernel dumps ucontext. Mention this to avoid misunderstanding.
> 
> Signed-off-by: Dev Jain <dev.jain@arm.com>

Hi Dev,

I've applied this patch, with some minor tweaks:
<https://www.alejandro-colomar.es/src/alx/linux/man-pages/man-pages.git/commit/?h=contrib&id=9e1f221b2099ee2c936ce4d6562a61e1e253af47>

Have a lovely day!
Alex

> ---
>  man/man7/signal.7 | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/man/man7/signal.7 b/man/man7/signal.7
> index 4ba5306c8..7d22a7cfe 100644
> --- a/man/man7/signal.7
> +++ b/man/man7/signal.7
> @@ -282,7 +282,13 @@ the thread's alternate signal stack settings.
>  .B SA_SIGINFO
>  flag, then the above information is accessible via the
>  .I ucontext_t
> -object that is pointed to by the third argument of the signal handler.)
> +object that is pointed to by the third argument of the signal handler.
> +This object reflects the state at which the signal is delivered, rather
> +than in the handler;
> +for example, the mask of blocked signals stored in this object will not
> +contain the mask of new signals blocked through
> +.BR sigaction(2)).
> +
>  .IP (1.4)
>  Any signals specified in
>  .I act\->sa_mask
> -- 
> 2.34.1
> 
> 

-- 
<https://www.alejandro-colomar.es/>

Re: [PATCH v2 1/2] signal.7: Clearly describe ucontext kernel dump to userspace

Posted by Alejandro Colomar 1 year, 6 months ago

On Mon, Jun 17, 2024 at 12:54:10PM GMT, Alejandro Colomar wrote:
> On Tue, Jun 11, 2024 at 02:38:22PM GMT, Dev Jain wrote:
> > The set of blocked signals is updated from struct sigaction after the
> > kernel dumps ucontext. Mention this to avoid misunderstanding.
> > 
> > Signed-off-by: Dev Jain <dev.jain@arm.com>
> 
> Hi Dev,
> 
> I've applied this patch, with some minor tweaks:
> <https://www.alejandro-colomar.es/src/alx/linux/man-pages/man-pages.git/commit/?h=contrib&id=9e1f221b2099ee2c936ce4d6562a61e1e253af47>

Whoops; I found another issue, and amended:
<https://www.alejandro-colomar.es/src/alx/linux/man-pages/man-pages.git/commit/?h=contrib&id=b8136d84eb3b9b1db0dd6f54f47c1c104682f44f>


> 
> Have a lovely day!
> Alex
> 
> > ---
> >  man/man7/signal.7 | 8 +++++++-
> >  1 file changed, 7 insertions(+), 1 deletion(-)
> > 
> > diff --git a/man/man7/signal.7 b/man/man7/signal.7
> > index 4ba5306c8..7d22a7cfe 100644
> > --- a/man/man7/signal.7
> > +++ b/man/man7/signal.7
> > @@ -282,7 +282,13 @@ the thread's alternate signal stack settings.
> >  .B SA_SIGINFO
> >  flag, then the above information is accessible via the
> >  .I ucontext_t
> > -object that is pointed to by the third argument of the signal handler.)
> > +object that is pointed to by the third argument of the signal handler.
> > +This object reflects the state at which the signal is delivered, rather
> > +than in the handler;
> > +for example, the mask of blocked signals stored in this object will not
> > +contain the mask of new signals blocked through
> > +.BR sigaction(2)).
> > +
> >  .IP (1.4)
> >  Any signals specified in
> >  .I act\->sa_mask
> > -- 
> > 2.34.1
> > 
> > 
> 
> -- 
> <https://www.alejandro-colomar.es/>



-- 
<https://www.alejandro-colomar.es/>