lkml.org 
[lkml]   [2022]   [Mar]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRE: [PATCH v3] kallsyms: enhance %pS/s/b printing when KALLSYSMS is disabled
From
Hi Luis,

>> ---
>> commit id 'kallsyms: print module name in %ps/S case when KALLSYMS is disabled'
>> needs to be removed from mm(linux-next) tree, current change is
>> with ignorance of this commit. I was not sure how to send patch, with 2 patches
>> consisting reversal commit also, or current approach is correct.
>>
>> v1->v2: hash base address of module, change *fmt to fmt[0] and removed
>> copy paste.
>> v2->v3: fixed review comments from Andy Shevchenko <andriy.shevchenko@linux.intel.com>
>>
>> include/linux/kallsyms.h | 2 +
>> include/linux/module.h | 20 ++++++++++
>> kernel/kallsyms.c | 27 +++++++------
>> kernel/module.c | 4 +-
>> lib/vsprintf.c | 85 ++++++++++++++++++++++++++++++++++------
>
> Hey Maninder, thanks for your patch!
>
> Since this touches kernel/module.c and include/linux/module.h I'd prefer
> this go through modules-next [0], and as you will see that's a different
> world right now. I also have a set of at least 2 other patch sets to
> merge there before yours.
>
> Also, what is on modules-next is not intended to go to Linus for the
> next merge window as the changes there got merged only late, and I want
> at least 2 months of testing on linux-newt before any pull requiest is
> sent to Linus.
>
> Can you rebase to modules-next? I can evaluate the patches then for
> integration there once the other stuff gets merged into that tree too.
>
> [0] https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux.git/log/?h=modules-next
>
> Luis

prepared and verified patch(KALLSYMS enabled and disabled both) on module-next rebase and sent in new mail.
[PATCH 1/1 module-next] kallsyms: enhance %pS/s/b printing when KALLSYSMS is disabled

https://lkml.org/lkml/2022/3/16/7

Thanks,
Maninder Singh[unhandled content-type:application/octet-stream]
\
 
 \ /
  Last update: 2022-03-16 05:44    [W:0.406 / U:0.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site