On Mon, 2013-01-07 at 08:47 +0100, Johannes Berg wrote: > On Sun, 2013-01-06 at 20:49 -0800, Joe Perches wrote: > > On Sun, 2013-01-06 at 19:19 -0800, Joe Perches wrote: > > > Maybe these days this should be another vsprintf %p extension > > > like %pM when the DECLARE_MAC_BUF/print_mac uses were converted. > > > > > > (or maybe extend %ph for ssids with %*phs, length, array) > > > > Maybe like this: > > > + * - 'D' For a 80211 SSID, it prints the SSID escaping any non-printable > > + characters with a leading \ and octal or [0nrt\] > > Honestly, I'm not sure it's worth it. Neither am I really, the only value I see in it is the reduction in stack consumption by 100 bytes or so per use. > print_ssid() is used in two or > three legacy drivers only, not in any modern driver, and is unlikely to > be used in the more modern drivers due to tracing etc. Swell. It was just another way to correct those overrun errors Chen Gang found. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
