Created attachment 902458 [details] valgrind.spec.ppc64le.patch Description of problem: valgrind do not build on ppc64le archi Version-Release number of selected component (if applicable): valgrind-3.9.0-15.svn20140513r13961.fc21 Steps to Reproduce: 1.fedpkg clone -a valgrind 2.fedpkg mockbuild Additional info: The code changes are available in following bz: # https://bugs.kde.org/show_bug.cgi?id=334384: Base compilation enablement # https://bugs.kde.org/show_bug.cgi?id=334834: Run time enablement # https://bugs.kde.org/show_bug.cgi?id=334836: Test suite changes for ppc64le I did a scratch build that succeeded on ppcle koji http://ppc-le.koji.fedoraproject.org/koji/taskinfo?taskID=76179 with related patches I will append in attachments.
Created attachment 902459 [details] valgrind_ppc64le_step1.patch
Created attachment 902460 [details] valgrind_ppc64le_step2.patch
Created attachment 902461 [details] valgrind_ppc64le_step3.patch
Thanks for that. I did an earlier scratch build as discussed here: http://thread.gmane.org/gmane.comp.debugging.valgrind.devel/26830 One issue is that the testsuite doesn't build correctly. If you look at your build.log you'll see that because the test suite doesn't build none of the tests are ran: http://ppc-le.koji.fedoraproject.org/kojifiles/work/tasks/6180/76180/build.log Which makes it a little hard to know how well the port is working. As soon as that is fixed and those patches are accepted upstream I'll rebase the valgrind package in rawhide.
FWIW http://pkgs.fedoraproject.org/cgit/llvm.git/tree/0001-PPC64LE-ELFv2-ABI-updates-for-the-.opd-section.patch is the fix llvm uses for ... /usr/bin/ld: .opd not allowed in ABI version 2 jm_insns-jm-insns.o: error adding symbols: Bad value ... that appears in the build.log
(In reply to Dan Horák from comment #5) > FWIW > http://pkgs.fedoraproject.org/cgit/llvm.git/tree/0001-PPC64LE-ELFv2-ABI- > updates-for-the-.opd-section.patch is the fix llvm uses for > > ... > /usr/bin/ld: .opd not allowed in ABI version 2 > jm_insns-jm-insns.o: error adding symbols: Bad value > ... > that appears in the build.log yes partially, but in fact the step3 patch I retrieved from https://bugs.kde.org/show_bug.cgi?id=334836 is incomplete because some changes for files in none/tests/ppc32 are missing for files in none/tests/ppc64. I will work on that.
Created attachment 902565 [details] valgrind_ppc64le_step3.patch valgrind_ppc64le_step3.patch updated as some missing changes in previous version.
Perhaps I'm blind, but why the duplicated blocks? #elif defined (VGA_ppc64le) ... #elif defined (VGA_ppc64le) ...
(In reply to Jakub Jelinek from comment #8) > Perhaps I'm blind, but why the duplicated blocks? > #elif defined (VGA_ppc64le) > ... > #elif defined (VGA_ppc64le) > ... good point :) I am asking to the owner of the original patch
(In reply to Michel Normand from comment #7) > Created attachment 902565 [details] > valgrind_ppc64le_step3.patch > > valgrind_ppc64le_step3.patch updated as some missing changes in previous > version. the associated scratch build has make check complete but still have some reporting errors http://ppc-le.koji.fedoraproject.org/koji/taskinfo?taskID=76372
(In reply to Michel Normand from comment #10) > (In reply to Michel Normand from comment #7) > > Created attachment 902565 [details] > > valgrind_ppc64le_step3.patch > > > > valgrind_ppc64le_step3.patch updated as some missing changes in previous > > version. > > the associated scratch build has make check complete but still have some > reporting errors > http://ppc-le.koji.fedoraproject.org/koji/taskinfo?taskID=76372 Thanks. That already looks fairly good. I think a couple of these have already been fixed upstream. In particular valgrind svn r13983 should fix the pthread_create_WRK issue. Please do work with upstream to get these patches fixed and integrated. Then we can rebase the valgrind package in fedora rawhide to include the ppc64le support.
Although the patches are still not upstream it seems things are close now to be integrated. Since I will be a away for a while I integrated them into the valgrind.spec already: 3.9.0-20.svn20140718r14176 - Update to upstream svn r14176 Remove valgrind-3.9.0-arm64-user_regs.patch - Add ppc64le support valgrind-3.9.0-ppc64le-initial.patch valgrind-3.9.0-ppc64le-functional.patch valgrind-3.9.0-ppc64le-test.patch valgrind-3.9.0-ppc64le-extra.patch Uploaded to both f21 and rawhide. Scratch build on ppc64/ppc64le looks good (test results included in the build.log): http://ppc.koji.fedoraproject.org/koji/taskinfo?taskID=1957511