On 7/2/20 5:05 PM, Wei Yongjun wrote: > Fix sparse build warning: > > mm/slub.c:1267:6: warning: > symbol 'parse_slub_debug_flags' was not declared. Should it be static? Right, thanks. > Reported-by: Hulk Robot <hulkci@xxxxxxxxxx> > Signed-off-by: Wei Yongjun <weiyongjun1@xxxxxxxxxx> Acked-by: Vlastimil Babka <vbabka@xxxxxxx> Could be squashed to mmotm patch: mm-slub-extend-slub_debug-syntax-for-multiple-blocks.patch > --- > mm/slub.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mm/slub.c b/mm/slub.c > index 62d2de56549e..cd260f18e6cb 100644 > --- a/mm/slub.c > +++ b/mm/slub.c > @@ -1264,7 +1264,7 @@ static noinline int free_debug_processing( > * > * returns the start of next block if there's any, or NULL > */ > -char * > +static char * > parse_slub_debug_flags(char *str, slab_flags_t *flags, char **slabs, bool init) > { > bool higher_order_disable = false; >