From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757658Ab0JUVto (ORCPT ); Thu, 21 Oct 2010 17:49:44 -0400 Received: from ozlabs.org ([203.10.76.45]:60567 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752810Ab0JUVtn (ORCPT ); Thu, 21 Oct 2010 17:49:43 -0400 From: Rusty Russell To: Steven Rostedt Subject: Re: [PATCH v2][GIT PULL] tracing: Prevent unloadable modules from using trace_bprintk() Date: Fri, 22 Oct 2010 08:05:14 +1030 User-Agent: KMail/1.13.5 (Linux/2.6.35-22-generic; KDE/4.5.1; i686; ; ) Cc: LKML , Ingo Molnar , Frederic Weisbecker , Andrew Morton , Thomas Gleixner , Linus Torvalds References: <1287668742.16971.585.camel@gandalf.stny.rr.com> In-Reply-To: <1287668742.16971.585.camel@gandalf.stny.rr.com> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-15" Content-Transfer-Encoding: 7bit Message-Id: <201010220805.14454.rusty@rustcorp.com.au> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 22 Oct 2010 12:15:42 am Steven Rostedt wrote: > > Ingo, > > This is based off of my core-2 branch. I'm moved this patch after that > so if anyone has any objections, I can change this patch without holding > off the previous one. I think disabling use in modules is lazy, and allowing it for !MODVERSIONS is just weird. Can't you detect this on module unload and fix it up? Or delay freeing the module until the trace ring is emptied? Cheers, Rusty.