linux-kernel - Re: [PATCH v7 00/12] unwind_deferred: Implement sframe handling

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <7eea50a5-e1b0-4319-9a25-cb8b327a836d@linux.ibm.com>
Date: Wed, 2 Jul 2025 14:57:22 +0200
From: Jens Remus <jremus@...ux.ibm.com>
To: Steven Rostedt <rostedt@...dmis.org>, linux-kernel@...r.kernel.org,
        linux-trace-kernel@...r.kernel.org, bpf@...r.kernel.org,
        x86@...nel.org, Indu Bhagat <indu.bhagat@...cle.com>
Cc: Masami Hiramatsu <mhiramat@...nel.org>,
        Mathieu Desnoyers <mathieu.desnoyers@...icios.com>,
        Josh Poimboeuf <jpoimboe@...nel.org>,
        Peter Zijlstra <peterz@...radead.org>, Ingo Molnar <mingo@...nel.org>,
        Jiri Olsa <jolsa@...nel.org>, Namhyung Kim <namhyung@...nel.org>,
        Thomas Gleixner <tglx@...utronix.de>,
        Andrii Nakryiko <andrii@...nel.org>,
        "Jose E. Marchesi" <jemarch@....org>,
        Beau Belgrave <beaub@...ux.microsoft.com>,
        Linus Torvalds <torvalds@...ux-foundation.org>,
        Andrew Morton <akpm@...ux-foundation.org>,
        Jens Axboe <axboe@...nel.dk>, Florian Weimer <fweimer@...hat.com>,
        Heiko Carstens <hca@...ux.ibm.com>, Vasily Gorbik <gor@...ux.ibm.com>
Subject: Re: [PATCH v7 00/12] unwind_deferred: Implement sframe handling

Hello Steve!

On 01.07.2025 20:49, Steven Rostedt wrote:
> This code is based on top of:
> 
>  https://lore.kernel.org/linux-trace-kernel/20250701005321.942306427@goodmis.org/
>  git://git.kernel.org/pub/scm/linux/kernel/git/trace/linux-trace.git unwind/core
> 
> This is the implementation of parsing the SFrame section in an ELF file.

...

> The code for this patch series can be found here:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/trace/linux-trace.git unwind/sframe

Wouldn't it make sense to include your related perf (tools) series [1]
in that branch to ease testing?  Provided you also include the minor
fix [2] to make perf tools work. :-)

Additionally it would make sense to include the patches from Josh that
add SFrame information to the vDSO on x86 [3].

[1]: [PATCH v12 00/11] perf: Support the deferred unwinding infrastructure,
https://lore.kernel.org/linux-trace-kernel/20250701180410.755491417@goodmis.org/

[2]: https://lore.kernel.org/linux-trace-kernel/51903e66-56bc-42a4-b80c-9c3223e2a48a@linux.ibm.com/

[3]: [PATCH v6 0/6] x86/vdso: VDSO updates and fixes for sframes,
https://lore.kernel.org/all/20250425023750.669174660@goodmis.org/

> Changes since v6: https://lore.kernel.org/linux-trace-kernel/20250617225009.233007152@goodmis.org/

> - Moved the addition of the prctl(), that allows libraries to add the elf
>   sections to the kernel, to the last patch and labeled it as "DO NOT APPLY".
>   This should instead be a proper system call and work to make it robust and
>   flexible still needs to be done. The prctl() patch is added for debugging
>   purposes only.

Does PR_SET_VMA [4] create a precedent case for the SFrame prctls?

[4]: https://man7.org/linux/man-pages/man2/pr_set_vma.2const.html

Thanks and regards,
Jens
-- 
Jens Remus
Linux on Z Development (D3303)
+49-7031-16-1128 Office
jremus@...ibm.com

IBM

IBM Deutschland Research & Development GmbH; Vorsitzender des Aufsichtsrats: Wolfgang Wendt; Geschäftsführung: David Faller; Sitz der Gesellschaft: Böblingen; Registergericht: Amtsgericht Stuttgart, HRB 243294
IBM Data Privacy Statement: https://www.ibm.com/privacy/