Re: [PATCH] module,bug: Add TAINT_OOT_MODULE flag for modules not built in-tree

Rusty Russell
2011-10-26T04:52:00Z

On Tue, 25 Oct 2011 16:17:24 -0400, Dave Jones <davej@redhat.com> wrote:
> commit 7816c45bf13255157c00fb8aca86cb64d825e878
> Author: Roland Vossen <rvossen@broadcom.com>
> Date: Thu Apr 7 11:20:58 2011 +0200
>
> modules: Enabled dynamic debugging for staging modules
...
>
> Signed-off-by: Roland Vossen <rvossen@broadcom.com>
> Acked-by: Jason Baron <jbaron@redhat.com>
> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Greg, you know better. This is why we have maintainers: I can't track
patches I don't see. Grrr...

> If we want to support out of tree modules with this, should we just nuke the
> whole check, or do we still want to prevent certain types of tainted kernels
> from using this stuff ?

It goes back to the first implementation of kernel markers. IIRC, it
was to prevent dynamic debug stuff from circumventing licensing, but
testing for *any* taint seems overbroad. Mathieu?

Thanks,
Rusty.
PS. Can't see how this related to lockdep either...