(heck, let's cc lkml - avoid having to go through all this again) On Fri, 4 Jul 2008 14:42:53 -0600 Matthew Wilcox <matthew@xxxxxx> wrote: > On Fri, Jul 04, 2008 at 01:27:16PM -0700, Andrew Morton wrote: > > On Fri, 4 Jul 2008 13:02:05 -0700 (PDT) Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote: > > > > so I think we could easily just say that we extend %p in various ways: > > > > > > > > - %pS - print pointer as a symbol > > > > > > > > and leave tons of room for future extensions for different kinds of > > > > pointers. > > > > If this takes off we might want a register-your-printk-handler > > interface. Maybe. > > > > We also jump through hoops to print things like sector_t and > > resource_size_t. They always need to be cast to `unsiged long long', > > which generates additional stack space and text in some setups. > > The thing is that GCC checks types. So it's fine to add "print this > pointer specially", but you can't in general add new printf arguments > without also hacking GCC. Unless you use -Wno-format, and require > sparse to check special kernel types. It would be excellent if gcc had an extension system so that you could add new printf control chars and maybe even tell gcc how to check them. But of course, if that were to happen, we couldn't use it for 4-5 years. What I had initially proposed was to abuse %S, which takes a wchar_t*. gcc accepts `unsigned long *' for %S. Then, we put the kernel-specific control char after the S, so we can print an inode (rofl) with struct inode *inode; printk("here is an inode: %Si\n", (unsigned long *)inode); Downsides are: - there's a cast, so you could accidentally do printk("here is an inode: %Si\n", (unsigned long *)dentry); - there's a cast, and they're ugly - gcc cannot of course check that the arg matches the control string Unfortunately (and this seems weird), gcc printf checking will not accept a void* for %S: it _has_ to be wchar_t*, and the checker won't permit void* substitution for that. Anyway, Linus took all that and said "let's abuse %p instead". It _will_ accept any pointer so we can instead do: printk("here is an inode: %pi\n", inode); which is nicer. I think the main customers of this are print_symbol(): printk("I am about to call %ps\n", fn); (*fn)(); and NIPQUAD and its ipv6 version. We don't know how much interest there would be in churning NIPQUAD from the net guys. Interestingly, there's also %C (wint_t) which is a 32-bit quantity. So we could just go and say "%C prints an ipv4 address" and be done with it. But there's no way of doing that for ipv6 addresses so things would become asymmetrical there. Another customer is net mac addresses. There are surely others. One which should have been in printf 30 years ago was %b: binary. > > And then there's the perennial "need to cast u64 to unsigned long long > > to print it". If we were to do > > > > printk("%SL", (void *)some_u64); > > > > then that's still bloody ugly, but it'll save a little text-n-stack. > > u64 is easy to fix, and I don't know why we haven't. Just make it > unsigned long long on all architectures. Yeah. Why don't we do that? -- To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
![]() |