lkml.org 
[lkml]   [2011]   [Oct]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] module,bug: Add TAINT_OOT_MODULE flag for modules not built in-tree
From
Date
On Tue, 2011-10-25 at 11:38 -0400, Nick Bowler wrote:
> On 2011-10-25 14:26 +1030, Rusty Russell wrote:
> > From: Ben Hutchings <ben@decadent.org.uk>
> > Subject: module,bug: Add TAINT_OOT_MODULE flag for modules not built in-tree
> > Date: Mon, 24 Oct 2011 15:12:28 +0200
> >
> > Use of the GPL or a compatible licence doesn't necessarily make the code
> > any good. We already consider staging modules to be suspect, and this
> > should also be true for out-of-tree modules which may receive very
> > little review.
> >
> > Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
> > Reviewed-by: Dave Jones <davej@redhat.com>
> > Acked-by: Greg Kroah-Hartman <gregkh@suse.de>
> > Signed-off-by: Rusty Russell <rusty@rustcorp.com.au> (patched oops-tracing.txt)
> > ---
> > Documentation/oops-tracing.txt | 2 ++
> > include/linux/kernel.h | 1 +
> > kernel/module.c | 5 +++++
> > kernel/panic.c | 2 ++
> > scripts/mod/modpost.c | 7 +++++++
> > 5 files changed, 17 insertions(+)
>
> This patch prevents the use of lockdep for debugging out of tree
> modules, which is rather mean.

It was already disabled for staging modules, which seems equally
unhelpful. Maybe it should print taint flags at the top of its warnings
instead.

Ben.

--
Ben Hutchings
DNRC Motto: I can please only one person per day.
Today is not your day. Tomorrow isn't looking good either.
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2011-10-25 18:09    [W:3.428 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site