On Thu, Mar 7, 2019 at 5:31 AM Joel Fernandes joel@joelfernandes.org wrote:
On Sat, Mar 2, 2019 at 2:00 PM kbuild test robot lkp@intel.com wrote:
Hi Joel,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on linus/master] [also build test ERROR on v5.0-rc8] [cannot apply to next-20190301] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Joel-Fernandes-Google/Provide-in-ke... config: sh-allmodconfig (attached as .config) compiler: sh4-linux-gnu-gcc (Debian 8.2.0-11) 8.2.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=8.2.0 make.cross ARCH=sh
All errors (new ones prefixed by >>):
find: 'arch/sh/kernel/module.lds': No such file or directory find: 'arch/sh/kernel/module.lds': No such file or directory
On Sun, Mar 03, 2019 at 08:11:59AM -0800, Joel Fernandes wrote:
This report is for an older version of the patch so ignore it. The issue is already resolved.
Apologies, this issue is real. However it does not cause any failure. It is only causing find command to be a bit noisy.
The below diff fixes it and I'll update the patch for v5:
Let's take a look a little bit closer.
$ find arch -name module.lds arch/ia64/module.lds arch/powerpc/kernel/module.lds arch/riscv/kernel/module.lds arch/arm/kernel/module.lds arch/m68k/kernel/module.lds arch/arm64/kernel/module.lds
(1) module.lds may not exist for some architectures (2) module.lds may be located in a different directory (ia64)
Your fix-up missed (2).
diff --git a/scripts/gen_ikh_data.sh b/scripts/gen_ikh_data.sh index 1fa5628fcc30..9a7ea856acbc 100755 --- a/scripts/gen_ikh_data.sh +++ b/scripts/gen_ikh_data.sh @@ -1,5 +1,6 @@ #!/bin/bash # SPDX-License-Identifier: GPL-2.0 +set -e
spath="$(dirname "$(readlink -f "$0")")" kroot="$spath/.." @@ -11,12 +12,14 @@ file_list=${@:2}
src_file_list="" for f in $file_list; do
if [ ! -f "$kroot/$f" ] && [ ! -d "$kroot/$f" ]; then continue; fi src_file_list="$src_file_list $(echo $f | grep -v OBJDIR)"done
obj_file_list="" for f in $file_list; do f=$(echo $f | grep OBJDIR | sed -e 's/OBJDIR///g')
if [ ! -f $f ] && [ ! -d $f ]; then continue; fi obj_file_list="$obj_file_list $f";done
-- 2.21.0.352.gf09ad66450-goog
-- Best Regards Masahiro Yamada