Re: [RFC][PATCH] x86: ptrace: Add function argument access API

From: Steven Rostedt
Date: Fri Oct 12 2018 - 14:22:48 EST


On Fri, 12 Oct 2018 11:21:28 -0700
Andy Lutomirski <luto@xxxxxxxxxxxxxx> wrote:

> On Fri, Oct 12, 2018 at 9:26 AM Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
> >
> >
> > Anyone have any issues with this patch?
> >
>
> I'm conceptually okay with it. That being said,
> regs_within_kernel_stack(), which you're indirectly using, is
> off-by-a-few. And updating it to use probe_kernel_read() might be
> nice for robustness.
>

100% agree. I'll add a patch on top of it to make the changes, and keep
what Masami had originally for full history.

Thanks for taking the time to take a look at it.

-- Steve