Re: [LKP] Re: [PATCH v3 1/2] modpost: rework and consolidate logging interface

[Masahiro Yamada]

On Wed, Mar 11, 2020 at 12:56 AM Philip Li <philip.li@xxxxxxxxx> wrote:
>
> On Tue, Mar 10, 2020 at 12:32:00PM +0100, Jessica Yu wrote:
> > +++ Masahiro Yamada [09/03/20 20:03 +0900]:
> > > On Mon, Mar 9, 2020 at 7:58 PM Jessica Yu <jeyu@xxxxxxxxxx> wrote:
> > > >
> > > > +++ Masahiro Yamada [09/03/20 19:49 +0900]:
> > > > >On Mon, Mar 9, 2020 at 7:39 PM Jessica Yu <jeyu@xxxxxxxxxx> wrote:
> > > > >>
> > > > >> Ah, sorry, I mean the kbuild 0-day bot errors. I am just realizing
> > > > >> the 0-day bot emails are not CC'd to lkml. Here is the error I got
> > > > >> from the bot:
> > > > >>
> > > > >> ---
> > > > >>
> > > > >> I love your patch! Yet something to improve:
> > > > >>
> > > > >> [auto build test ERROR on linus/master]
> > > > >> [also build test ERROR on v5.6-rc4 next-20200306]
> > > > >> [if your patch is applied to the wrong git tree, please drop us a note to help
> > > > >> improve the system. BTW, we also suggest to use '--base' option to specify the
> > > > >> base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
> > > > >>
> > > > >> url:    https://github.com/0day-ci/linux/commits/Jessica-Yu/modpost-rework-and-consolidate-logging-interface/20200307-052346
> > > > >> base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 30fe0d07fd7b27d41d9b31a224052cc4e910947a
> > > > >> config: sh-randconfig-a001-20200306 (attached as .config)
> > > > >> compiler: sh4-linux-gcc (GCC) 7.5.0
> > > > >> reproduce:
> > > > >>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> > > > >>         chmod +x ~/bin/make.cross
> > > > >>         # save the attached .config to linux build tree
> > > > >>         GCC_VERSION=7.5.0 make.cross ARCH=sh
> > > > >>
> > > > >> If you fix the issue, kindly add following tag
> > > > >> Reported-by: kbuild test robot <lkp@xxxxxxxxx>
> > > > >>
> > > > >> All errors (new ones prefixed by >>):
> > > > >>
> > > > >> >> ERROR: modpost: "adc_single" [arch/sh/boards/mach-hp6xx/hp6xx_apm.ko] undefined!
> > > > >
> > > > >
> > > > >Indeed, this one is odd.
> > > > >I have no idea...
> > > >
> > > > I've pushed the patches to a branch to let the kbuild bot run through its
> > > > build tests again, and if I have extra time today I will try to
> > > > reproduce this and let you know the results.
> > > >
> > > > Thanks,
> > > >
> > > > Jessica
> > >
> > >
> > > Ah, Now I see.
> > >
> > >
> > > Because you added "modpost:" prefix.
> > >
> > >
> > >
> > > The previous error message:
> > >
> > > ERROR: "adc_single" [arch/sh/boards/mach-hp6xx/hp6xx_apm.ko] undefined!
> > >
> > >
> > > The new error message:
> > >
> > > ERROR: modpost: "adc_single" [arch/sh/boards/mach-hp6xx/hp6xx_apm.ko] undefined!
> > >
> > >
> > >
> > > So, the bot assumed it was a new error.
> >
> > Nice catch! Hm, I suppose we need to let the LKP folks know about the
> > change in error message. CC'd LKP mailing list.
> Thanks for the info, we will look into this to handle the
> changed error.
>
> >
> > Jessica
> > _______________________________________________
> > LKP mailing list -- lkp@xxxxxxxxxxxx
> > To unsubscribe send an email to lkp-leave@xxxxxxxxxxxx



Could you improve the report by adding more context?

Currently, only new errors/warnings are shown by '>>'.


If fixed ones had been shown by '<<',
we would have easily noticed that
this was just a matter of message format.

<< ERROR: "adc_single" [arch/sh/boards/mach-hp6xx/hp6xx_apm.ko] undefined!
>> ERROR: modpost: "adc_single" [arch/sh/boards/mach-hp6xx/hp6xx_apm.ko] undefined!




-- 
Best Regards
Masahiro Yamada