Re: [PATCH 1/11] oprofile: add check_user_page_readable()

[Andrew Morton <akpm@osdl.org>]

Greg Banks <gnb@melbourne.sgi.com> wrote:
>
> On Tue, 2004-11-09 at 22:04, Andrew Morton wrote:
> > Greg Banks <gnb@melbourne.sgi.com> wrote:
> > >
> > > Add check_user_page_readable() for kernel modules which need
> > >  to follow user space addresses but can't use get_user().
> > 
> > Strange.  What is the usage pattern for this?
> 
> The i386 callgraph code attempts to follow user stacks, from
> an interrupt (perfmon, NMI, or timer)

Yikes.

> where get_user() is
> explicitly disallowed by Documentation/DocBook/kernel-locking.tmpl.
> AFAICS from the ia64 and i386 page fault handlers get_user should
> "just work" and return -EFAULT if the page isn't resident or
> readable, but the doc says...
> 
> Currently this is only an issue for i386.  The ia64 code doesn't
> even try to look at user stacks (shudder).
> 
> >   And why is that usage
> > pattern not racy in the presence of paging activity?
> 
> The i386 backtracer takes the &current->mm->page_table_lock,

But that cannot be taken from interrupt context.  A trylock would be OK I
guess.

> and
> just drops out of the trace early if a page isn't resident.  It
> doesn't expect or try to page in.  After all this is only statistical
> sampling not write() data.
> 
> > 
> > Did you consider use_mm(), in conjunction with get_user()?
> 
> No, but glancing at use_mm() the comment says
> 
>  *      (Note: this routine is intended to be called only
>  *      from a kernel thread context)

It could probably be made to work from a normal process, but not from
interrupt context.

I guess I should apply the patches and take a closer look.