Date: Wed, 30 Jan 2008 03:08:54 +0100 (CET)
From: Jan Engelhardt <jengelh@computergmbh.de>
To: Pavel Roskin <proski@gnu.org>
cc: Jon Masters <jonathan@jonmasters.org>, linux-kernel@vger.kernel.org,
       Rusty Russell <rusty@rustcorp.com.au>,
       Giridhar Pemmasani <pgiri@yahoo.com>
Subject: Re: ndiswrapper and GPL-only symbols redux
In-Reply-To: <1201657701.24898.14.camel@dv>
Message-ID: <Pine.LNX.4.64.0801300308260.22578@fbirervta.pbzchgretzou.qr>
References: <1201641765.18773.35.camel@dv>  <1201652420.2271.91.camel@perihelion>
 <1201657701.24898.14.camel@dv>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 878
Lines: 19


On Jan 29 2008 20:48, Pavel Roskin wrote:
>On Tue, 2008-01-29 at 19:20 -0500, Jon Masters wrote:
>
>> Yes it is. But I thought the existing code was intending to taint the
>> kernel (that's what it does), so it would really help to identify why it
>> tainted the kernel, by calling add_taint_module instead of add_taint. I
>> didn't put the existing match in there...don't shoot the messenger :)
>
>So, it's the same thing as in year 2006.  Good intentions, unexpected
>side effects, and a long discussion.

Perhaps module.c needs more comments explaining why the ndis line
is there, and why it's correct and noone should touch it.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/