| Message ID | 8736xsv8w3.fsf@concordia.ellerman.id.au (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On 06/11/2018 11:03 PM, Michael Ellerman wrote: > kbuild test robot <lkp@intel.com> writes: > >> Hi Laura, >> >> I love your patch! Yet something to improve: >> >> [auto build test ERROR on linus/master] >> [also build test ERROR on v4.17 next-20180608] >> [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/Laura-Abbott/scripts-Preprocess-module-common-lds/20180612-083632 >> config: i386-randconfig-a1-201823 (attached as .config) >> compiler: gcc-4.9 (Debian 4.9.4-2) 4.9.4 >> reproduce: >> # save the attached .config to linux build tree >> make ARCH=i386 >> >> All errors (new ones prefixed by >>): >> >>>> ld: cannot open linker script file scripts/module-common.lds: No such file or directory > > This seems to need the following. > > cheers > > > diff --git a/Makefile b/Makefile > index 73f0bb2c7a98..55a5725b6606 100644 > --- a/Makefile > +++ b/Makefile > @@ -425,7 +425,7 @@ KBUILD_AFLAGS_KERNEL := > KBUILD_CFLAGS_KERNEL := > KBUILD_AFLAGS_MODULE := -DMODULE > KBUILD_CFLAGS_MODULE := -DMODULE > -KBUILD_LDFLAGS_MODULE := -T $(srctree)/scripts/module-common.lds > +KBUILD_LDFLAGS_MODULE := -T $(objtree)/scripts/module-common.lds > LDFLAGS := > GCC_PLUGINS_CFLAGS := > > Thanks for pointing that out. I think I missed that when refactoring. I'll fix that in the next version plus adding your powerpc patch. Thanks, Laura -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Laura Abbott <labbott@redhat.com> writes: > On 06/11/2018 11:03 PM, Michael Ellerman wrote: >> kbuild test robot <lkp@intel.com> writes: ... >>> All errors (new ones prefixed by >>): >>> >>>>> ld: cannot open linker script file scripts/module-common.lds: No such file or directory >> >> This seems to need the following. >> >> diff --git a/Makefile b/Makefile >> index 73f0bb2c7a98..55a5725b6606 100644 >> --- a/Makefile >> +++ b/Makefile >> @@ -425,7 +425,7 @@ KBUILD_AFLAGS_KERNEL := >> KBUILD_CFLAGS_KERNEL := >> KBUILD_AFLAGS_MODULE := -DMODULE >> KBUILD_CFLAGS_MODULE := -DMODULE >> -KBUILD_LDFLAGS_MODULE := -T $(srctree)/scripts/module-common.lds >> +KBUILD_LDFLAGS_MODULE := -T $(objtree)/scripts/module-common.lds >> LDFLAGS := >> GCC_PLUGINS_CFLAGS := > > Thanks for pointing that out. I think I missed that when refactoring. > I'll fix that in the next version plus adding your powerpc patch. Thanks. cheers -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Makefile b/Makefile index 73f0bb2c7a98..55a5725b6606 100644 --- a/Makefile +++ b/Makefile @@ -425,7 +425,7 @@ KBUILD_AFLAGS_KERNEL := KBUILD_CFLAGS_KERNEL := KBUILD_AFLAGS_MODULE := -DMODULE KBUILD_CFLAGS_MODULE := -DMODULE -KBUILD_LDFLAGS_MODULE := -T $(srctree)/scripts/module-common.lds +KBUILD_LDFLAGS_MODULE := -T $(objtree)/scripts/module-common.lds LDFLAGS := GCC_PLUGINS_CFLAGS :=