netdev - Re: [PATCH 2/7] rtlwifi: Standardize RT_PRINT

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, 05 Jan 2012 10:44:17 -0600
From:	Larry Finger <Larry.Finger@...inger.net>
To:	Joe Perches <joe@...ches.com>
CC:	Stanislaw Gruszka <sgruszka@...hat.com>,
	Chaoming Li <chaoming_li@...lsil.com.cn>,
	"John W. Linville" <linville@...driver.com>,
	linux-wireless@...r.kernel.org, netdev@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/7] rtlwifi: Standardize RT_PRINT_DATA macro and uses

On 01/05/2012 10:29 AM, Joe Perches wrote:
> On Thu, 2012-01-05 at 12:43 +0100, Stanislaw Gruszka wrote:
>> Hi Joe
>
> Morning Stanislaw.
>
>> On Wed, Jan 04, 2012 at 07:40:40PM -0800, Joe Perches wrote:
>>>   	RT_PRINT_DATA(rtlpriv, COMP_CMD, DBG_DMESG,
>>
>> Since you are working on it, can you also get rid of this DBG_*
>> parameter, if rtlwifi maintainers are ok with that? It is additional
>> console loglvevel, standard kernel loglevel should be used.
>
> Not quite, all of the RT_PRINT_DATA are emitted at KERN_DEBUG.
> These DBG_*s are just a normal debug verbosity level.
>
> It does seem odd though to have a DBG_EMERG and not always
> emit it.

Those are always logged, no matter what debug level is set.

> What might also be useful for any embedded use is to add
> CONFIG_RTLWIFI_DEBUG to conditionally remove all of the
> debug logging code to reduce the largish object size.
>
> This reduces the object size by about 1/3 (250KB)
> when CONFIG_RTLWIFI_DEBUG is not set.
>
> $ size drivers/net/wireless/rtlwifi/built-in.o*
>     text	   data	    bss	    dec	    hex	filename
>   368722	  55333	  94224	 518279	  7e887	drivers/net/wireless/rtlwifi/built-in.o.new
>   586904	  55333	 127216	 769453	  bbdad	drivers/net/wireless/rtlwifi/built-in.o.old
>
> Signed-off-by: Joe Perches<joe@...ches.com>

This change seems sensible to me.

Larry

>
> ---
>
>   drivers/net/wireless/rtlwifi/Kconfig |    5 +++++
>   drivers/net/wireless/rtlwifi/debug.h |   34 ++++++++++++++++++++++++++++++++++
>   2 files changed, 39 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/net/wireless/rtlwifi/Kconfig b/drivers/net/wireless/rtlwifi/Kconfig
> index d6c42e6..44b9c0a 100644
> --- a/drivers/net/wireless/rtlwifi/Kconfig
> +++ b/drivers/net/wireless/rtlwifi/Kconfig
> @@ -49,6 +49,11 @@ config RTLWIFI
>   	depends on RTL8192CE || RTL8192CU || RTL8192SE || RTL8192DE
>   	default m
>
> +config RTLWIFI_DEBUG
> +	tristate "Additional debugging output"
> +	depends on RTL8192CE || RTL8192CU || RTL8192SE || RTL8192DE
> +	default y
> +
>   config RTL8192C_COMMON
>   	tristate
>   	depends on RTL8192CE || RTL8192CU
> diff --git a/drivers/net/wireless/rtlwifi/debug.h b/drivers/net/wireless/rtlwifi/debug.h
> index b024c23..fd5600c 100644
> --- a/drivers/net/wireless/rtlwifi/debug.h
> +++ b/drivers/net/wireless/rtlwifi/debug.h
> @@ -156,6 +156,8 @@ enum dbgp_flag_e {
>   	DBGP_TYPE_MAX
>   };
>
> +#ifdef CONFIG_RTLWIFI_DEBUG
> +
>   #define RT_ASSERT(_exp, fmt, ...)					\
>   do {									\
>   	if (!(_exp)) {							\
> @@ -195,5 +197,37 @@ do {									\
>   	}								\
>   } while (0)
>
> +#else
> +
> +struct rtl_priv;
> +
> +__printf(2, 3)
> +static inline void RT_ASSERT(int exp, const char *fmt, ...)
> +{
> +}
> +
> +__printf(4, 5)
> +static inline void RT_TRACE(struct rtl_priv *rtlpriv,
> +			    int comp, int level,
> +			    const char *fmt, ...)
> +{
> +}
> +
> +__printf(4, 5)
> +static inline void RTPRINT(struct rtl_priv *rtlpriv,
> +			   int dbgtype, int dbgflag,
> +			   const char *fmt, ...)
> +{
> +}
> +
> +static inline void RT_PRINT_DATA(struct rtl_priv *rtlpriv,
> +				 int comp, int level,
> +				 const char *titlestring,
> +				 const void *hexdata, size_t hexdatalen)
> +{
> +}
> +
> +#endif
> +
>   void rtl_dbgp_flag_init(struct ieee80211_hw *hw);
>   #endif
>
>
>
>

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