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]
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

Powered by blists - more mailing lists