* build regression from c153693: Simplify module TOC handling @ 2016-02-09 16:28 Peter Robinson 2016-02-09 21:02 ` Dinar Valeev 0 siblings, 1 reply; 3+ messages in thread From: Peter Robinson @ 2016-02-09 16:28 UTC (permalink / raw) To: amodra, linuxppc-dev, mpe, kernel Hi Alan, Your patch for "powerpc: Simplify module TOC handling" is causing the Fedora ppc64le to fail to build with depmod failures. Reverting the commit fixes it for us on rawhide. We're getting the out put below, full logs at [1]. Let me know if you have any other queries. Regards, Peter [1] http://ppc.koji.fedoraproject.org/kojifiles/work/tasks/5115/3125115/build.log + depmod -b . -aeF ./System.map 4.5.0-0.rc2.git0.1.fc24.ppc64le Depmod failure + '[' -s depmod.out ']' + echo 'Depmod failure' + cat depmod.out depmod: WARNING: /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/arch/powerpc/platforms/powernv/opal-prd.ko needs unknown symbol .TOC. depmod: WARNING: /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/arch/powerpc/platforms/pseries/pseries_energy.ko needs unknown symbol .TOC. depmod: WARNING: /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/arch/powerpc/platforms/pseries/hvcserver.ko needs unknown symbol .TOC. depmod: WARNING: /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/arch/powerpc/kvm/kvm.ko needs unknown symbol .TOC. depmod: WARNING: /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/arch/powerpc/kvm/kvm-pr.ko needs unknown symbol .TOC. depmod: WARNING: /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/arch/powerpc/kvm/kvm-hv.ko needs unknown symbol .TOC. depmod: WARNING: /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/kernel/rcu/rcutorture.ko needs unknown symbol .TOC. depmod: WARNING: /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/kernel/trace/ring_buffer_benchmark.ko needs unknown symbol .TOC. depmod: WARNING: /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/kernel/torture.ko needs unknown symbol .TOC. depmod: WARNING: /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/fs/nfs_common/nfs_acl.ko needs unknown symbol .TOC. depmod: WARNING: /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/fs/nfs_common/grace.ko needs unknown symbol .TOC. ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: build regression from c153693: Simplify module TOC handling 2016-02-09 16:28 build regression from c153693: Simplify module TOC handling Peter Robinson @ 2016-02-09 21:02 ` Dinar Valeev 2016-02-09 23:03 ` Michael Ellerman 0 siblings, 1 reply; 3+ messages in thread From: Dinar Valeev @ 2016-02-09 21:02 UTC (permalink / raw) To: Peter Robinson; +Cc: amodra, linuxppc-dev, Michael Ellerman, kernel On Tue, Feb 9, 2016 at 5:28 PM, Peter Robinson <pbrobinson@gmail.com> wrote: > Hi Alan, > > Your patch for "powerpc: Simplify module TOC handling" is causing the > Fedora ppc64le to fail to build with depmod failures. Reverting the > commit fixes it for us on rawhide. Anton's patch [1] fixes it. [1] https://build.opensuse.org/package/view_file/Base:System/kmod/depmod-Ignore_PowerPC64_ABIv2_.TOC.symbol.patch > > We're getting the out put below, full logs at [1]. Let me know if you > have any other queries. > > Regards, > Peter > > [1] http://ppc.koji.fedoraproject.org/kojifiles/work/tasks/5115/3125115/build.log > > + depmod -b . -aeF ./System.map 4.5.0-0.rc2.git0.1.fc24.ppc64le > Depmod failure > + '[' -s depmod.out ']' > + echo 'Depmod failure' > + cat depmod.out > depmod: WARNING: > /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/arch/powerpc/platforms/powernv/opal-prd.ko > needs unknown symbol .TOC. > depmod: WARNING: > /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/arch/powerpc/platforms/pseries/pseries_energy.ko > needs unknown symbol .TOC. > depmod: WARNING: > /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/arch/powerpc/platforms/pseries/hvcserver.ko > needs unknown symbol .TOC. > depmod: WARNING: > /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/arch/powerpc/kvm/kvm.ko > needs unknown symbol .TOC. > depmod: WARNING: > /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/arch/powerpc/kvm/kvm-pr.ko > needs unknown symbol .TOC. > depmod: WARNING: > /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/arch/powerpc/kvm/kvm-hv.ko > needs unknown symbol .TOC. > depmod: WARNING: > /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/kernel/rcu/rcutorture.ko > needs unknown symbol .TOC. > depmod: WARNING: > /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/kernel/trace/ring_buffer_benchmark.ko > needs unknown symbol .TOC. > depmod: WARNING: > /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/kernel/torture.ko > needs unknown symbol .TOC. > depmod: WARNING: > /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/fs/nfs_common/nfs_acl.ko > needs unknown symbol .TOC. > depmod: WARNING: > /builddir/build/BUILDROOT/kernel-4.5.0-0.rc2.git0.1.fc24.ppc64le/./lib/modules/4.5.0-0.rc2.git0.1.fc24.ppc64le/kernel/fs/nfs_common/grace.ko > needs unknown symbol .TOC. > _______________________________________________ > Linuxppc-dev mailing list > Linuxppc-dev@lists.ozlabs.org > https://lists.ozlabs.org/listinfo/linuxppc-dev ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: build regression from c153693: Simplify module TOC handling 2016-02-09 21:02 ` Dinar Valeev @ 2016-02-09 23:03 ` Michael Ellerman 0 siblings, 0 replies; 3+ messages in thread From: Michael Ellerman @ 2016-02-09 23:03 UTC (permalink / raw) To: Dinar Valeev, Peter Robinson; +Cc: amodra, linuxppc-dev, kernel On Tue, 2016-02-09 at 22:02 +0100, Dinar Valeev wrote: > On Tue, Feb 9, 2016 at 5:28 PM, Peter Robinson <pbrobinson@gmail.com> wrote: > > Hi Alan, > > > > Your patch for "powerpc: Simplify module TOC handling" is causing the > > Fedora ppc64le to fail to build with depmod failures. Reverting the > > commit fixes it for us on rawhide. > Anton's patch [1] fixes it. > > [1] https://build.opensuse.org/package/view_file/Base:System/kmod/depmod-Ignore_PowerPC64_ABIv2_.TOC.symbol.patch Yep, you need an updated depmod. Anton sent a patch to linux-modules, reproduced below for the benefit of the list archive: depmod: Ignore PowerPC64 ABIv2 .TOC. symbol The .TOC. symbol on the PowerPC64 ABIv2 identifies the GOT pointer, similar to how other architectures use _GLOBAL_OFFSET_TABLE_. This is not a symbol that needs relocation, and should be ignored by depmod. --- tools/depmod.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/depmod.c b/tools/depmod.c index 6e9bb4d..a2e07c1 100644 --- a/tools/depmod.c +++ b/tools/depmod.c @@ -2153,6 +2153,8 @@ static void depmod_add_fake_syms(struct depmod *depmod) depmod_symbol_add(depmod, "__this_module", true, 0, NULL); /* On S390, this is faked up too */ depmod_symbol_add(depmod, "_GLOBAL_OFFSET_TABLE_", true, 0, NULL); + /* On PowerPC64 ABIv2, .TOC. is more or less _GLOBAL_OFFSET_TABLE_ */ + depmod_symbol_add(depmod, "TOC.", true, 0, NULL); } static int depmod_load_symvers(struct depmod *depmod, const char *filename) -- 2.5.0 ^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-02-09 23:03 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-02-09 16:28 build regression from c153693: Simplify module TOC handling Peter Robinson 2016-02-09 21:02 ` Dinar Valeev 2016-02-09 23:03 ` Michael Ellerman
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).