# Re: \[PATCH v8 10/12\] unwind\_user/sframe: Enable debugging in uaccess regions

- From: Steven Rostedt \<rostedt@goodmis.org\>
- Date: 2025-07-08 13:23:58 
- List: linux-trace-kernel
- Page: https://ratatoskr.run/linux-trace-kernel/2025/07/3125494
- Thread: https://ratatoskr.run/linux-trace-kernel/2025/07/3124692/t.md

```text
On Mon, 7 Jul 2025 20:38:35 -0700
Linus Torvalds [ref] wrote:

> On Mon, 7 Jul 2025 at 19:12, Steven Rostedt [ref] wrote:

> This patch is disgusting, in other words. It's wrong. STOP IT.
> 

No problem, I can easily drop it.

I just took Josh's PoC patches and posted them. This particular series
still needs a bit of work. That's one of the reasons I split it out of the
other series. The core series is at the stage for acceptance and looking
for feedback from other maintainers. This series is still a work in
progress. Others have asked me to post them so they can start playing with
it.

We still need to come up with a system call with a robust API to allow the
dynamic linker to tell the kernel where the SFrames are for the libraries
it loads. Hence the "DO NOT APPLY" patch at the end (which I just noticed the
subject text got dropped when I pulled it into git from patchwork and sent
out this version, at least the change long still suggest it shouldn't be
applied).

But I will remove this patch from the queue. I never even used this
debugging. What I did was inject trace_printk() all over the place to make
sure it was working as expected.

Thanks,

-- Steve
```