linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [arm:for-next 13/25] include/linux/error-injection.h:7:10: fatal error: asm/error-injection.h: No such file or directory
@ 2019-08-29  0:49 kbuild test robot
  2019-08-29  6:43 ` Leo Yan
  0 siblings, 1 reply; 4+ messages in thread
From: kbuild test robot @ 2019-08-29  0:49 UTC (permalink / raw)
  To: Leo Yan; +Cc: Russell King, kbuild-all, linux-arm-kernel

[-- Attachment #1: Type: text/plain, Size: 1716 bytes --]

tree:   git://git.armlinux.org.uk/~rmk/linux-arm.git for-next
head:   d0d54dc04e37be14a9e51d9a2e431f302948e99d
commit: 566c290c6498b2fdc04a54556c4e8747f0298c7b [13/25] ARM: 8899/1: arm/arm64: Add support for function error injection
config: arm-allmodconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 7.4.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout 566c290c6498b2fdc04a54556c4e8747f0298c7b
        # save the attached .config to linux build tree
        GCC_VERSION=7.4.0 make.cross ARCH=arm 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   In file included from include/linux/module.h:22:0,
                    from drivers/pps/pps.c:11:
>> include/linux/error-injection.h:7:10: fatal error: asm/error-injection.h: No such file or directory
    #include <asm/error-injection.h>
             ^~~~~~~~~~~~~~~~~~~~~~~
   compilation terminated.

vim +7 include/linux/error-injection.h

540adea3809f61 Masami Hiramatsu 2018-01-13  6  
540adea3809f61 Masami Hiramatsu 2018-01-13 @7  #include <asm/error-injection.h>
540adea3809f61 Masami Hiramatsu 2018-01-13  8  

:::::: The code at line 7 was first introduced by commit
:::::: 540adea3809f61115d2a1ea4ed6e627613452ba1 error-injection: Separate error-injection from kprobe

:::::: TO: Masami Hiramatsu <mhiramat@kernel.org>
:::::: CC: Alexei Starovoitov <ast@kernel.org>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 71416 bytes --]

[-- Attachment #3: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [arm:for-next 13/25] include/linux/error-injection.h:7:10: fatal error: asm/error-injection.h: No such file or directory
  2019-08-29  0:49 [arm:for-next 13/25] include/linux/error-injection.h:7:10: fatal error: asm/error-injection.h: No such file or directory kbuild test robot
@ 2019-08-29  6:43 ` Leo Yan
  2019-08-29  8:32   ` Will Deacon
  0 siblings, 1 reply; 4+ messages in thread
From: Leo Yan @ 2019-08-29  6:43 UTC (permalink / raw)
  To: kbuild test robot; +Cc: Russell King, Will Deacon, kbuild-all, linux-arm-kernel

Hi Russell,

On Thu, Aug 29, 2019 at 08:49:16AM +0800, kbuild test robot wrote:
> tree:   git://git.armlinux.org.uk/~rmk/linux-arm.git for-next
> head:   d0d54dc04e37be14a9e51d9a2e431f302948e99d
> commit: 566c290c6498b2fdc04a54556c4e8747f0298c7b [13/25] ARM: 8899/1: arm/arm64: Add support for function error injection
> config: arm-allmodconfig (attached as .config)
> compiler: arm-linux-gnueabi-gcc (GCC) 7.4.0
> reproduce:
>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         git checkout 566c290c6498b2fdc04a54556c4e8747f0298c7b
>         # save the attached .config to linux build tree
>         GCC_VERSION=7.4.0 make.cross ARCH=arm 
> 
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <lkp@intel.com>
> 
> All errors (new ones prefixed by >>):
> 
>    In file included from include/linux/module.h:22:0,
>                     from drivers/pps/pps.c:11:
> >> include/linux/error-injection.h:7:10: fatal error: asm/error-injection.h: No such file or directory
>     #include <asm/error-injection.h>
>              ^~~~~~~~~~~~~~~~~~~~~~~
>    compilation terminated.

This building error is caused by there have a dependent patch:
error-injection: Consolidate override function definition
https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git/commit/?h=for-next/error-injection&id=45880f7b7b19e043ce0aaa4cb7d05369425c82fa

This patch has been picked up by Will in one of arm64's next branch:
https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git/log/?h=for-next/error-injection

I don't know what's the best practice for the dependency between
two branches, if need me to follow up anything, please let me know.

Thanks,
Leo.

> vim +7 include/linux/error-injection.h
> 
> 540adea3809f61 Masami Hiramatsu 2018-01-13  6  
> 540adea3809f61 Masami Hiramatsu 2018-01-13 @7  #include <asm/error-injection.h>
> 540adea3809f61 Masami Hiramatsu 2018-01-13  8  
> 
> :::::: The code at line 7 was first introduced by commit
> :::::: 540adea3809f61115d2a1ea4ed6e627613452ba1 error-injection: Separate error-injection from kprobe
> 
> :::::: TO: Masami Hiramatsu <mhiramat@kernel.org>
> :::::: CC: Alexei Starovoitov <ast@kernel.org>
> 
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation



_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [arm:for-next 13/25] include/linux/error-injection.h:7:10: fatal error: asm/error-injection.h: No such file or directory
  2019-08-29  6:43 ` Leo Yan
@ 2019-08-29  8:32   ` Will Deacon
  2019-08-29  9:57     ` Leo Yan
  0 siblings, 1 reply; 4+ messages in thread
From: Will Deacon @ 2019-08-29  8:32 UTC (permalink / raw)
  To: Leo Yan, rmk+kernel; +Cc: kbuild test robot, linux-arm-kernel, kbuild-all

[ Move RMK to To: ]

On Thu, Aug 29, 2019 at 02:43:10PM +0800, Leo Yan wrote:
> On Thu, Aug 29, 2019 at 08:49:16AM +0800, kbuild test robot wrote:
> > tree:   git://git.armlinux.org.uk/~rmk/linux-arm.git for-next
> > head:   d0d54dc04e37be14a9e51d9a2e431f302948e99d
> > commit: 566c290c6498b2fdc04a54556c4e8747f0298c7b [13/25] ARM: 8899/1: arm/arm64: Add support for function error injection
> > config: arm-allmodconfig (attached as .config)
> > compiler: arm-linux-gnueabi-gcc (GCC) 7.4.0
> > reproduce:
> >         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> >         chmod +x ~/bin/make.cross
> >         git checkout 566c290c6498b2fdc04a54556c4e8747f0298c7b
> >         # save the attached .config to linux build tree
> >         GCC_VERSION=7.4.0 make.cross ARCH=arm 
> > 
> > If you fix the issue, kindly add following tag
> > Reported-by: kbuild test robot <lkp@intel.com>
> > 
> > All errors (new ones prefixed by >>):
> > 
> >    In file included from include/linux/module.h:22:0,
> >                     from drivers/pps/pps.c:11:
> > >> include/linux/error-injection.h:7:10: fatal error: asm/error-injection.h: No such file or directory
> >     #include <asm/error-injection.h>
> >              ^~~~~~~~~~~~~~~~~~~~~~~
> >    compilation terminated.
> 
> This building error is caused by there have a dependent patch:
> error-injection: Consolidate override function definition
> https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git/commit/?h=for-next/error-injection&id=45880f7b7b19e043ce0aaa4cb7d05369425c82fa

Ah, I guess you put the ARM patch into the patch system without reference to
the core parts?

> This patch has been picked up by Will in one of arm64's next branch:
> https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git/log/?h=for-next/error-injection
> 
> I don't know what's the best practice for the dependency between
> two branches, if need me to follow up anything, please let me know.

The for-next/error-injection branch here:

https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git/log/?h=for-next/error-injection

is stable, so I suppose either:

   * That could be pulled into the ARM tree, or
   * The ARM part could wait until the core stuff has landed in mainline, or
   * I could take the ARM patch via the arm64 tree if Russell is ok with it

Will

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [arm:for-next 13/25] include/linux/error-injection.h:7:10: fatal error: asm/error-injection.h: No such file or directory
  2019-08-29  8:32   ` Will Deacon
@ 2019-08-29  9:57     ` Leo Yan
  0 siblings, 0 replies; 4+ messages in thread
From: Leo Yan @ 2019-08-29  9:57 UTC (permalink / raw)
  To: Will Deacon, rmk+kernel; +Cc: kbuild test robot, linux-arm-kernel, kbuild-all

Hi Will,

On Thu, Aug 29, 2019 at 09:32:00AM +0100, Will Deacon wrote:
> [ Move RMK to To: ]
> 
> On Thu, Aug 29, 2019 at 02:43:10PM +0800, Leo Yan wrote:
> > On Thu, Aug 29, 2019 at 08:49:16AM +0800, kbuild test robot wrote:
> > > tree:   git://git.armlinux.org.uk/~rmk/linux-arm.git for-next
> > > head:   d0d54dc04e37be14a9e51d9a2e431f302948e99d
> > > commit: 566c290c6498b2fdc04a54556c4e8747f0298c7b [13/25] ARM: 8899/1: arm/arm64: Add support for function error injection
> > > config: arm-allmodconfig (attached as .config)
> > > compiler: arm-linux-gnueabi-gcc (GCC) 7.4.0
> > > reproduce:
> > >         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> > >         chmod +x ~/bin/make.cross
> > >         git checkout 566c290c6498b2fdc04a54556c4e8747f0298c7b
> > >         # save the attached .config to linux build tree
> > >         GCC_VERSION=7.4.0 make.cross ARCH=arm 
> > > 
> > > If you fix the issue, kindly add following tag
> > > Reported-by: kbuild test robot <lkp@intel.com>
> > > 
> > > All errors (new ones prefixed by >>):
> > > 
> > >    In file included from include/linux/module.h:22:0,
> > >                     from drivers/pps/pps.c:11:
> > > >> include/linux/error-injection.h:7:10: fatal error: asm/error-injection.h: No such file or directory
> > >     #include <asm/error-injection.h>
> > >              ^~~~~~~~~~~~~~~~~~~~~~~
> > >    compilation terminated.
> > 
> > This building error is caused by there have a dependent patch:
> > error-injection: Consolidate override function definition
> > https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git/commit/?h=for-next/error-injection&id=45880f7b7b19e043ce0aaa4cb7d05369425c82fa
> 
> Ah, I guess you put the ARM patch into the patch system without reference to
> the core parts?

Yes, this is exactly what I did.  Sorry I only put the ARM patch and
didn't give out the whole context.

> > This patch has been picked up by Will in one of arm64's next branch:
> > https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git/log/?h=for-next/error-injection
> > 
> > I don't know what's the best practice for the dependency between
> > two branches, if need me to follow up anything, please let me know.
> 
> The for-next/error-injection branch here:
> 
> https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git/log/?h=for-next/error-injection
> 
> is stable, so I suppose either:
> 
>    * That could be pulled into the ARM tree, or
>    * The ARM part could wait until the core stuff has landed in mainline, or
>    * I could take the ARM patch via the arm64 tree if Russell is ok with it

Thanks for these options.  I'd like to know what's Russell's decision,
then I will look closely to know what I should follow up or not.

Thanks,
Leo Yan

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2019-08-29  9:57 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-08-29  0:49 [arm:for-next 13/25] include/linux/error-injection.h:7:10: fatal error: asm/error-injection.h: No such file or directory kbuild test robot
2019-08-29  6:43 ` Leo Yan
2019-08-29  8:32   ` Will Deacon
2019-08-29  9:57     ` Leo Yan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).