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

[Rong Chen]

On 3/11/20 9:00 AM, Masahiro Yamada wrote:
> 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!

Hi,

Thanks for your advice, we'll add it into our todo list.

Best Regards,
Rong Chen