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

From: Rusty Russell
Date: Wed Oct 26 2011 - 00:52:02 EST


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

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...
--
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/