On (10/20/17 15:08), Petr Mladek wrote: > On Thu 2017-10-19 15:42:35, Sergey Senozhatsky wrote: > > Sorry for the delay and thanks for taking a look. > > > > I'll try to re-spin the patch set by the end of this week/early next > > week. > > > > > > On (10/04/17 13:53), Petr Mladek wrote: > > [..] > > > Note that kallsyms_lookup() and module_address_lookup() is used > > > in many other situations. > > > > we dereference only things that can be dereferenced. > > so calling it on already dereferenced address, or address > > that does need to be dereferenced is OK. > > My concern is that it changes the behavior. It will suddenly return > another information for addresses that were not dereference before. OK. I'd be really-really surprised to find out that anyone did kallsyms_lookup()/module_address_lookup() on func descriptors, but I understand your concerns. I'll try to keep everything within vsprintf(). -ss -- 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