linux-kernel - Re: [PATCH] Documentation/sysctl: coredump: add %F for pidfd number

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <aFerWSlrlRYMkB2h@eldamar.lan>
Date: Sun, 22 Jun 2025 09:06:01 +0200
From: Salvatore Bonaccorso <carnil@...ian.org>
To: Jonathan Corbet <corbet@....net>
Cc: Christian Brauner <brauner@...nel.org>,
	Luca Boccassi <luca.boccassi@...il.com>,
	Oleg Nesterov <oleg@...hat.com>, linux-doc@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Documentation/sysctl: coredump: add %F for pidfd number

Hi Jonathan,

On Sat, Jun 21, 2025 at 02:18:38PM -0600, Jonathan Corbet wrote:
> Salvatore Bonaccorso <carnil@...ian.org> writes:
> 
> > In commit b5325b2a270f ("coredump: hand a pidfd to the usermode coredump
> > helper") a new core_pattern specifier, %F, was added to provide a pidfs
> > to the usermode helper process referring to the crashed process.
> >
> > Update the documentation to include the new core_pattern specifier.
> >
> > Link: https://github.com/systemd/systemd/pull/37125
> > Link: https://lwn.net/Articles/1024160/
> > Signed-off-by: Salvatore Bonaccorso <carnil@...ian.org>
> > ---
> >  Documentation/admin-guide/sysctl/kernel.rst | 1 +
> >  1 file changed, 1 insertion(+)
> 
> Applied, thanks.

Thank you!

> > FWIW, I was not sure if I should add a 'Fixes: b5325b2a270f
> > ("coredump: hand a pidfd to the usermode coredump helper") to make
> > sure that it get backported as well to the series where a backport of
> > b5325b2a270f landed.
> 
> A Fixes tag is not sufficient to ensure that a patch is backported to
> previous kernels - you must CC stable@...r.kernel.org to be sure.  In
> this case, it's a documentation tweak for a 6.16 change, so I don't
> think there is a lot of urgency here.

Ack (apart the change did land in variou stable series, i.e.
b5325b2a270f ("coredump: hand a pidfd to the usermode coredump
helper") did land in 6.16-rc1 *and* the stable versions v6.15.1,
v6.14.10, v6.12.32, v6.6.93, v6.1.141, v5.15.185, v5.10.238 and
v5.4.294.

You are right it is not urgent, would be more to have documentation
consistent as well in those versions.

Regards,
Salvatore