Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1269806

Re: [PATCH] wireless: change cfg80211 regulatory domain info as debug messages

From Dave Young <dyoung@redhat.com>
Newsgroups linux.kernel
Subject Re: [PATCH] wireless: change cfg80211 regulatory domain info as debug messages
Date 2015-11-16 03:00 +0100
Message-ID <qvhfk-Ta-7@gated-at.bofh.it> (permalink)
References <qv04N-6Qp-5@gated-at.bofh.it> <qv9rr-4nZ-9@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 11/15/15 at 09:38am, Joe Perches wrote:
> On Sun, 2015-11-15 at 15:31 +0800, Dave Young wrote:
> > cfg80211 module prints a lot of messages like below. Actually printing
> > once is acceptable but sometimes it will print again and again, it looks
> > very annoying. It is better to change these detail messages to debugging
> > only.
> 
> So maybe add some wrapper that does a pr_info then
> a pr_debug for the second and subsequent uses like:
> 
> #define pr_info_once_then_debug(fmt, ...)			\
> ({								\
> 	static bool __print_once __read_mostly;			\
> 								\
> 	if (!__print_once) {					\
> 		__print_once = true;				\
> 		pr_info(fmt, ##__VA_ARGS__);			\
> 	} else {						\
> 		pr_debug(fmt, ##__VA_ARGS__);			\
> 	}							\
> })
> 

Hmm, it looks too much for this issue, I'm thinking about to add a cmdline
param to disable mute it.

Thanks
Dave
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH] wireless: change cfg80211 regulatory domain info as debug  messages Dave Young <dyoung@redhat.com> - 2015-11-15 08:40 +0100
  Re: [PATCH] wireless: change cfg80211 regulatory domain info as  debug messages Joe Perches <joe@perches.com> - 2015-11-15 18:40 +0100
    Re: [PATCH] wireless: change cfg80211 regulatory domain info as  debug messages Dave Young <dyoung@redhat.com> - 2015-11-16 03:00 +0100
    Re: [PATCH] wireless: change cfg80211 regulatory domain info as  debug messages Johannes Berg <johannes@sipsolutions.net> - 2015-11-16 11:40 +0100
  Re: [PATCH] wireless: change cfg80211 regulatory domain info as  debug messages Stefan Lippers-Hollmann <s.l-h@gmx.de> - 2015-11-15 19:30 +0100
    Re: [PATCH] wireless: change cfg80211 regulatory domain info as  debug messages Dave Young <dyoung@redhat.com> - 2015-11-16 03:00 +0100
    Re: [PATCH] wireless: change cfg80211 regulatory domain info as  debug messages Johannes Berg <johannes@sipsolutions.net> - 2015-11-20 13:00 +0100
      Re: [PATCH] wireless: change cfg80211 regulatory domain info as  debug messages Dave Young <dyoung@redhat.com> - 2015-11-23 02:40 +0100

csiph-web