Bug 925305 - elfutils: tests fail on F-21
Summary: elfutils: tests fail on F-21
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: elfutils
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Roland McGrath
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: ARM64, F-ExcludeArch-aarch64
TreeView+ depends on / blocked
 
Reported: 2013-03-23 00:23 UTC by Dennis Gilmore
Modified: 2014-01-14 18:06 UTC (History)
6 users (show)

Fixed In Version: elfutils-0.158-1.fc21
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-01-14 18:06:33 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Dennis Gilmore 2013-03-23 00:23:19 UTC
Support for the ARM 64 bit CPU architecture (aarch64) was introduced in 
autoconf 2.69.  elfutils appears to use an earlier version of 
autoconf, preventing its being built.  This can be fixed in of three ways (In order of preference):

1. Work with upstream to migrate the package to autoconf 2.69.

2. Rerun autoconf or autoreconf in %prep or %build prior to running 
configure.

3. Apply the patch at http://ausil.fedorapeople.org/aarch64/elfutils/elfutils-aarch64.patch
which updates config.guess and config.sub to recognize aarch64.

Comment 1 Mark Wielaard 2013-03-23 20:44:22 UTC
We can certainly run autoreconf to pick up the latest autoconf. That will give you an elfutils that can be used for the architectures it already supports. But elfutils doesn't have an ARM64 backend at the moment. Is anybody planning on writing one? Alternatively could someone attach some ARM64 ELF object that we can use as testcase for writing one?

Comment 2 Petr Machata 2013-03-25 11:15:57 UTC
(In reply to comment #1)
> Is anybody planning on writing one?

Not that I know of.  We should sync with Linaro on that not to duplicate effort.  I'd be interested in working on it.

Comment 3 Mark Wielaard 2013-09-30 14:49:49 UTC
Petr now has a branch with initial support, not yet integrated into mainline:
https://git.fedorahosted.org/cgit/elfutils.git/log/?h=pmachata/aarch64

Comment 4 Petr Machata 2013-11-27 00:15:42 UTC
The aarch64 support (sans unwinding) is now upstream.

Comment 5 Peter Robinson 2013-12-02 22:25:17 UTC
So elfutils builds but some of the tests fail on elfutils-0.157-2.fc21

A sample of the errors are below but a full build log can be found here (any of the fc21 builds) 

http://arm.koji.fedoraproject.org/koji/packageinfo?packageID=1626

============================================================================
Testsuite summary for elfutils 0.157
============================================================================
# TOTAL: 101
# PASS:  98
# SKIP:  0
# XFAIL: 0
# FAIL:  3
# XPASS: 0
# ERROR: 0
============================================================================
See tests/test-suite.log
Please report to https://bugzilla.redhat.com/
============================================================================
make[3]: *** [test-suite.log] Error 1
make[2]: *** [check-TESTS] Error 2
make[1]: *** [check-am] Error 2
make: *** [check-recursive] Error 1
+ cat tests/test-suite.log
==========================================
   elfutils 0.157: tests/test-suite.log
==========================================
# TOTAL: 101
# PASS:  98
# SKIP:  0
# XFAIL: 0
# FAIL:  3
# XPASS: 0
# ERROR: 0
.. contents:: :depth: 2
FAIL: run-strip-reloc.sh
========================
*** failure --reloc-debug-sections not smaller /builddir/build/BUILD/elfutils-0.157/src/strip.o
FAIL: run-elflint-self.sh
=========================
unknown machine type 183
section [ 9] '.rela.dyn': relocation 0: invalid type
section [ 9] '.rela.dyn': relocation 1: invalid type
section [ 9] '.rela.dyn': relocation 2: invalid type
section [10] '.rela.plt': relocation 0: invalid type
section [10] '.rela.plt': relocation 1: invalid type
section [10] '.rela.plt': relocation 2: invalid type
section [10] '.rela.plt': relocation 3: invalid type

Comment 6 Mark Wielaard 2013-12-02 22:32:40 UTC
(In reply to Peter Robinson from comment #5)
> So elfutils builds but some of the tests fail on elfutils-0.157-2.fc21
> 
> A sample of the errors are below but a full build log can be found here (any
> of the fc21 builds) 
> 
> http://arm.koji.fedoraproject.org/koji/packageinfo?packageID=1626

Sorry, but what are you precisely testing here?
Did you backport the upstream git patches for aarch64 support to the elfutils-0.157-2.fc21 package?

Comment 7 Peter Robinson 2013-12-02 22:33:55 UTC
(In reply to Mark Wielaard from comment #6)
> (In reply to Peter Robinson from comment #5)
> > So elfutils builds but some of the tests fail on elfutils-0.157-2.fc21
> > 
> > A sample of the errors are below but a full build log can be found here (any
> > of the fc21 builds) 
> > 
> > http://arm.koji.fedoraproject.org/koji/packageinfo?packageID=1626
> 
> Sorry, but what are you precisely testing here?
> Did you backport the upstream git patches for aarch64 support to the
> elfutils-0.157-2.fc21 package?

Nope, it's the mainline Fedora package with the same NVR, what else would I be testing?

Comment 8 Mark Wielaard 2013-12-02 22:40:14 UTC
(In reply to Peter Robinson from comment #7)
> (In reply to Mark Wielaard from comment #6)
> > (In reply to Peter Robinson from comment #5)
> > > So elfutils builds but some of the tests fail on elfutils-0.157-2.fc21
> > > 
> > > A sample of the errors are below but a full build log can be found here (any
> > > of the fc21 builds) 
> > > 
> > > http://arm.koji.fedoraproject.org/koji/packageinfo?packageID=1626
> > 
> > Sorry, but what are you precisely testing here?
> > Did you backport the upstream git patches for aarch64 support to the
> > elfutils-0.157-2.fc21 package?
> 
> Nope, it's the mainline Fedora package with the same NVR, what else would I
> be testing?

OK. Then just wait for the 0.158 upstream release. Should be out in ~2 weeks.

Note that the aarch64 support depends on the following binutils fix:

    http://cygwin.com/ml/binutils/2013-07/msg00013.html

Does anybody know whether that is already in fc21?

Comment 9 Peter Robinson 2013-12-02 23:17:35 UTC
> > Nope, it's the mainline Fedora package with the same NVR, what else would I
> > be testing?
> 
> OK. Then just wait for the 0.158 upstream release. Should be out in ~2 weeks.

OK, will keep an eye out for it

> Note that the aarch64 support depends on the following binutils fix:
> 
>     http://cygwin.com/ml/binutils/2013-07/msg00013.html
> 
> Does anybody know whether that is already in fc21?

Yes, looking at the tarball it looks like the patch is there

Comment 10 Peter Robinson 2014-01-14 18:06:33 UTC
Seems OK now so will close


Note You need to log in before you can comment on or make changes to this bug.