netdev - Re: [PATCH 00/39] merge request for WiMAX kernel stack and i2400m driver v2

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <1228397835.28839.23.camel@violet.holtmann.net>
Date:	Thu, 04 Dec 2008 14:37:15 +0100
From:	Marcel Holtmann <marcel@...tmann.org>
To:	Johannes Berg <johannes@...solutions.net>
Cc:	Inaky Perez-Gonzalez <inaky@...ux.intel.com>,
	netdev@...r.kernel.org, wimax@...uxwimax.org,
	Thomas Graf <tgraf@...g.ch>, Patrick McHardy <kaber@...sh.net>
Subject: Re: [PATCH 00/39] merge request for WiMAX kernel stack and i2400m
 driver v2

Hi Johannes,

> > First two ones I plan to tackle ASAP. The dynamic_printk() stuff might take
> > longer and I was wondering if you would be ok with changing that post-29 in
> > case I were about to miss the merge window (got a few very busy weeks
> > ahead of me).
> 
> Fine with me, I guess, I don't even know if the dynprintk is applicable
> (though I'd rather see you make it work if it isn't than add yet another
> huge debug macro file :) )

I switched over the Bluetooth stack to use dynamic_printk(). It is as
easy as just using pr_debug() or dev_dbg(). The rest already works
perfectly fine.

However it is either full debug or nothing. Not a fine grained stuff we
have in the iwlwifi drivers. However not sure if that is actually
needed. It confuses me more and having to unload the driver and reload
it makes me not using it a lot. The dynamic_printk() is really nice
since you can turn it on and off. And since it is on a per module basis
it was enough for Bluetooth. Not sure if that is applicable to
everything, but then it could be extended with debug levels, I guess.

Regards

Marcel

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html