| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <1337183801.18519.7.camel@jlt3.sipsolutions.net> Date: Wed, 16 May 2012 17:56:41 +0200 From: Johannes Berg <johannes@...solutions.net> To: Joe Perches <joe@...ches.com> Cc: David Miller <davem@...emloft.net>, linville@...driver.com, linux-wireless@...r.kernel.org, netdev@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH 0/3] net: mac80211: Neaten debugging On Wed, 2012-05-16 at 08:43 -0700, Joe Perches wrote: > On Wed, 2012-05-16 at 17:30 +0200, Johannes Berg wrote: > > On Wed, 2012-05-16 at 08:22 -0700, Joe Perches wrote: > > > On Wed, 2012-05-16 at 09:59 +0200, Johannes Berg wrote: > > > > I wonder if it makes sense to leave these under > > > > #ifdef though? Why #ifdef something if it's going to be invisible most > > > > of the time anyway? > > > > > > I don't understand your point. > > > #ifdef removal is a good thing. > > > > Yeah, but you left a lot of them under ifdef, and I'm wondering why you > > didn't remove them, or if you should, or ... > > Those mostly use other different > #ifdef CONFIG_SOME_OTHER_CONTROL elements. > > There are I think 3 that I left because they do > not use printk/pr_level but use wiphy_<level> or > netdev_<level>. Hmm, ok. I guess I need to just look at them and decide what should be what. Thanks for the preparation work though :) johannes -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists