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: <CAJ-Vmom4n7Sk5P=HMb+Z-FNyfyk87Xbf5Q1kPA4i2eXwQ72kXQ@mail.gmail.com> Date: Mon, 19 Mar 2012 20:39:33 -0700 From: Adrian Chadd <adrian@...ebsd.org> To: Joe Perches <joe@...ches.com> Cc: Jiri Slaby <jirislaby@...il.com>, Nick Kossifidis <mickflemm@...il.com>, "Luis R. Rodriguez" <mcgrof@....qualcomm.com>, Bob Copeland <me@...copeland.com>, "John W. Linville" <linville@...driver.com>, linux-wireless@...r.kernel.org, ath5k-devel@...ts.ath5k.org, netdev@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH wireless-next 2/3] ath5k: Introduce _ath5k_printk to reduce code/text On 18 March 2012 22:18, Joe Perches <joe@...ches.com> wrote: >> Otherwise compiling in debugging will cause a _lot_ of spurious >> register reads to occur that are then tossed. This was one of the big >> reasons for instability and slow performance when AH_DEBUG was >> enabled. > > That doesn't make any sense in this case. > > It's either a call to printk or _ath5_printk > but it's still a call to a function. The FreeBSD HAL used to be like this. I changed it so it didn't evaluate the arguments before it figured out whether or not to do the (k)printf(). I'm just pointing it out as you're (currently) knee deep in the debugging code and it may be useful for you to also think about implementing. adrian -- 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