Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932129Ab1DFPei (ORCPT ); Wed, 6 Apr 2011 11:34:38 -0400 Received: from mail.perches.com ([173.55.12.10]:4550 "EHLO mail.perches.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932117Ab1DFPeg (ORCPT ); Wed, 6 Apr 2011 11:34:36 -0400 Subject: [Fwd: Fwd: Re: Why is dynamic debug disabled for staging drivers ?] From: Joe Perches To: Jason Baron Cc: LKML , Roland Vossen , devel , gregkh@suse.de Content-Type: text/plain; charset="UTF-8" Date: Wed, 06 Apr 2011 08:34:35 -0700 Message-ID: <1302104075.1683.46.camel@Joe-Laptop> Mime-Version: 1.0 X-Mailer: Evolution 2.32.2 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3236 Lines: 96 Adding Jason and LKML. to cc's... -------- Forwarded Message -------- From: Roland Vossen To: rusty@rustcorp.com.au Cc: devel@linuxdriverproject.org , gregkh@suse.de Subject: Fwd: Re: Why is dynamic debug disabled for staging drivers ? Date: Wed, 6 Apr 2011 17:09:04 +0200 Hello Rusty, I would like to make a change to module.c and noticed with 'git blame' that you were the last person who touched these specific lines in module.c: http://lxr.linux.no/#linux+v2.6.38/kernel/module.c#L2792 ===== Corresponding commit for your reference: commit 811d66a0e1e99902d365497eec7884113a2665bd Author: Rusty Russell Date: Thu Aug 5 12:59:12 2010 -0600 module: group post-relocation functions into post_relocation() This simply hoists more code out of load_module; we also put the identification of the extable and dynamic debug table in with the others in find_module_sections(). We move the taint check to the actual add/remove of the dynamic debug info: this is certain (find_module_sections is too early). Signed-off-by: Rusty Russell Cc: Yehuda Sadeh ============== Can you tell me if the following proposed code change in module.c is ok with you: /* This has to be done once we're sure module name is unique. */ if (!mod->taints || mod->taints == (1U << TAINT_CRAP)) dynamic_debug_setup(info.debug, info.num_debug); Thanks, Roland. -------- Original Message -------- Subject: Re: Why is dynamic debug disabled for staging drivers ? Date: Wed, 6 Apr 2011 07:13:29 -0700 From: Greg KH To: Roland Vossen CC: devel@linuxdriverproject.org On Wed, Apr 06, 2011 at 03:54:37PM +0200, Roland Vossen wrote: > Hi, > > I want to replace the proprietary logging mechanism in brcm80211 > with a Linux mechanism. 'Dynamic debug' seemed to be a good fit. > But, to my disappointment, I discovered that dynamic debugging is > not supported for drivers from the staging dir: > > - staging drivers are marked tainted (ref: > http://lxr.linux.no/#linux+v2.6.38/kernel/module.c#L2417) > > - subsequently tainted drivers don't get dynamic debug goodies (ref: > http://lxr.linux.no/#linux+v2.6.38/kernel/module.c#L2792) > > I wonder if this is intentional behavior. Does anybody know ? Ah, no it isn't. I didn't realize that tainted modules didn't get dynamic debug stuff, that's not nice. I'd be glad to take a fix that allows drivers tainted with the TAINT_CRAP flag to be able to use dynamic debug. In looking at that check, it probably can be fixed to just make sure that the module name is unique (i.e. only a specific taint flag check) which is the real goal of that check in module.c. thanks, greg k-h _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/devel -- 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/