Re: [PATCH 0/9] remove some of unneeded kallsyms includes

From: Sergey Senozhatsky
Date: Fri Dec 08 2017 - 19:00:20 EST


On (12/08/17 14:24), Andrew Morton wrote:
> On Fri, 8 Dec 2017 11:56:07 +0900 Sergey Senozhatsky <sergey.senozhatsky.work@xxxxxxxxx> wrote:
>
> > A small patch set that removes some kallsyms includes
> > here and there. Mostly those kallsyms includes are leftovers:
> > printk() gained %pS/%pF modifiers support some time ago, so
> > print_symbol() and friends became sort of unneeded [along with
> > print_fn_descriptor_symbol() deprecation], thus some of the
> > users were converted to pS/pF. This patch set just cleans up
> > that convertion.
> >
> > We still have a number of print_symbol() users [which
> > must be converted to ps/pf, print_symbol() uses a stack buffer
> > KSYM_SYMBOL_LEN to do what printk(ps/pf) can do], but this is
> > out of scope.
> >
> > I compile tested the patch set; but, as always and
> > usual, would be great if 0day build robot double check it.
>
> I grabbed everything and shall drop any patches which later turn up in
> the various subsystem trees.

thank you!

-ss