Thread (9 messages) 9 messages, 3 authors, 2025-09-09

Re: [PATCH] man/man2/sigaction.2: Update si_code list with Linux v6.16

From: Carlos O'Donell <hidden>
Date: 2025-09-03 15:37:38

On 9/2/25 11:08 PM, Thiago Jung Bauermann wrote:
To be honest, my motivation to write this patch was just to fix the fact
that SEGV_CPERR was not mentioned. Then I noticed that other constants
were missing too, and thought it would be easy enough to add them as
well...
Welcome to the rabbit hole! :-)
  
quoted
  (a) Adding the constants to C libraries in a conformant way.

  (b) Cleaning up the UAPI header to be conforming and work with
      the existing C libraries to include it indirectly.

  (c) Cleaning up both headers to allow dual inclusion with
      additional constants showing up as needed.

In summary:

  - This patch contains 2 constants that don't work today in a glibc-based
    userspace.

  - The existing man page documents 1 constants that doesn't work today
    with the standard #include <signal.h>.
As you suggested (thanks!), I sent a patch to the glibc mailing list
adding the si_codes it's missing to its siginfo-conts.h header:

https://inbox.sourceware.org/libc-alpha/20250903024151.3030839-1-thiago.bauermann@linaro.org/

If that one is accepted, hopefully this patch can go in?
Yes, that works for me.

I've reviewed the glibc submission and it needs a change, but I think we
can get v2 or v3 integrated relatively easily.

-- 
Cheers,
Carlos.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help