Re: OOPS in perf_mmap_close()

From: Peter Zijlstra
Date: Wed May 29 2013 - 03:44:58 EST


On Tue, May 28, 2013 at 02:22:11PM -0400, Vince Weaver wrote:
> On Tue, 28 May 2013, Vince Weaver wrote:
>
> > It looks like this is already in tip, but I can confirm that this
> > patch seems to fix things on my machine and holds up against longer
> > fuzzing runs.
>
> OK, I take it back. Even with the new patch applied, my fuzzer can still
> make the kernel leak user->locked_vm
>
> I assume that the locked_vm value should go back to 0 once a process that
> has a bunch of mmap'd perf_events opened exits?

Yep.

> I admit this is sort of an obscure corner case, but it does mean that a
> user can leak user->locked_vm to the point that "perf record" no longer
> works.

Hurm.. I don't suppose you have an easy reproducer handy eh? I'll go
stare at it. At least the current state is better than before, but
clearly we're not quite there yet.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/