On Mon, Dec 08, 2008 at 10:23:28AM -0500, John David Anglin wrote: > > The prettiest way to do it would be to add a %p handler for > > resource_size_t args, cheat on the pointing bit, and clean up the > > in-tree users of %llx/ull-casts. > > I went through the warnings that I saw yesterday implementing Helge's > %p suggestion. It's not beautiful but it works. I'll post tonight. > > I only fixed the occurences that popped up in my build. I'm sure that > more are lurking. > > Feel free to implement %p handler. > > I also fixed some other warnings and will post for comment when I separate > the diffs. > sadly %p can't work, since void * doesn't hold resource_size_t with CONFIG_RESOURCES_64BIT on 32-bit, since resource_size_t would be ULL, which is why the %p handler would work (although you'd either have to pass &(resource_size_t) or do a bit of ick around the va_args to snatch the correct width instead of just void *). but yeah, that's a global kernel issue, not limited to our little niche. i'll apply your patch, and another one to make CONFIG_RESOURCES_64BIT depend on !PARISC, since it really only applies to PAE on x86 and some niche crap embeddeded powerpc. cheers, kyle -- To unsubscribe from this list: send the line "unsubscribe linux-parisc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
