Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Fails to build on ARM, needs to use default setjmp not ucontext
https://bugzilla.redhat.com/show_bug.cgi?id=750805
Summary: Fails to build on ARM, needs to use default setjmp not ucontext Product: Fedora Version: 15 Platform: Unspecified OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: perl-Coro AssignedTo: ppisar@redhat.com ReportedBy: henrik@henriknordstrom.net QAContact: extras-qa@fedoraproject.org CC: fedora-perl-devel-list@redhat.com, mmaslano@redhat.com, kwizart@gmail.com, bochecha@fedoraproject.org, ppisar@redhat.com Classification: Fedora Story Points: --- Type: ---
Description of problem:
ARM do not implement the needed ucontext functions, and need to use the default setjmp method. This is normally the default, except that fedora patches it to hardwire ucontext as default method..
Version-Release number of selected component (if applicable):
perl-Coro-5.372-3.fc15
How reproducible:
always
Steps to Reproduce: 1. Try to rebuild perl-Coro on arm 2. 3.
Actual results:
failed build, crashing in testsuite
Expected results:
successful build
Additional info:
Trivial spec file patch attached.
Please apply patch and sumbit a F15 koji build from which arm can pull the srpm. update request is not stricty needed if the only change relative to current F15 build is this patch.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
--- Comment #1 from Nicolas Chauvet (kwizart) kwizart@gmail.com 2011-11-02 09:20:55 EDT --- Hi Henrik,
Isn't the attachment missing ?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
--- Comment #2 from Henrik Nordström henrik@henriknordstrom.net 2011-11-03 05:00:28 EDT --- Created attachment 531517 --> https://bugzilla.redhat.com/attachment.cgi?id=531517 Disable ucontext forced default on arm, using normal setjmp default.
Indeed. Could have sworn I added it when filing the bug.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
--- Comment #3 from Petr Pisar ppisar@redhat.com 2011-11-03 11:53:34 EDT --- Nicolas, thank you for taking it. I found I cannot rebuild it in F17 because SELinux denies executable non-text memory. I haven't found what to do yet. (A pax marker?)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
--- Comment #4 from Fedora Update System updates@fedoraproject.org 2011-11-03 12:32:22 EDT --- perl-Coro-5.372-6.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/perl-Coro-5.372-6.fc16
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
--- Comment #5 from Fedora Update System updates@fedoraproject.org 2011-11-03 12:34:02 EDT --- perl-Coro-5.372-4.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/perl-Coro-5.372-4.fc15
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
--- Comment #6 from Henrik Nordström henrik@henriknordstrom.net 2011-11-03 21:39:51 EDT --- Created attachment 531681 --> https://bugzilla.redhat.com/attachment.cgi?id=531681 Disable FORTIFY_SOURCE on ARM, incompatible with how setjmp is used.
Thanks. Unfortunately it turns out additional changes is needed on arm. The way Coro uses setjmp is incompatible with FORTIFY_SOURCE.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #7 from Fedora Update System updates@fedoraproject.org 2011-11-04 16:26:25 EDT --- Package perl-Coro-5.372-4.fc15: * should fix your issue, * was pushed to the Fedora 15 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing perl-Coro-5.372-4.fc15' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2011-15373 then log in and leave karma (feedback).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
Nicolas Chauvet (kwizart) kwizart@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |ASSIGNED
--- Comment #8 from Nicolas Chauvet (kwizart) kwizart@gmail.com 2011-11-04 18:12:57 EDT --- (In reply to comment #6)
Created attachment 531681 [details] Disable FORTIFY_SOURCE on ARM, incompatible with how setjmp is used.
Thanks. Unfortunately it turns out additional changes is needed on arm. The way Coro uses setjmp is incompatible with FORTIFY_SOURCE.
Is this a build or runtime issue ? Can we have additional info about wgy it fails with FORTIFY_SOURCE ?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
--- Comment #9 from Henrik Nordström henrik@henriknordstrom.net 2011-11-05 04:52:26 EDT --- It's a runtime issue detected by the test suite. I by accident did not have FORTIFY_SOURCE enabled when testing the first change.
Coro (and pth) both abuses setjmp in the same manner for establishing new stacks per context, but with FORTIFY_SOURCE enabled glibc verifies that setjmp only sets the stack pointer to an earlier stack frame in the current stack and aborts if the stack change looks odd.
*** longjmp causes uninitialized stack frame ***: /usr/bin/perl terminated
it's a valid error message in normal specified use of setjmp, but do not apply to the special (outside specifications) use by Coro for establishing another stack context entirely, outside the current stack.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Fixed In Version| |perl-Coro-5.372-6.fc16 Resolution| |ERRATA Last Closed| |2011-12-12 16:58:42
--- Comment #10 from Fedora Update System updates@fedoraproject.org 2011-12-12 16:58:42 EST --- perl-Coro-5.372-6.fc16 has been pushed to the Fedora 16 stable repository. If problems still persist, please make note of it in this bug report.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
Brendan Conoboy blc@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|CLOSED |ASSIGNED CC| |blc@redhat.com Resolution|ERRATA | Keywords| |Reopened
--- Comment #11 from Brendan Conoboy blc@redhat.com 2012-02-21 15:41:16 EST --- This appears to be broken again in F17 and Rawhide.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
Mark Langsdorf mark.langsdorf@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mark.langsdorf@gmail.com
--- Comment #12 from Mark Langsdorf mark.langsdorf@gmail.com 2012-02-21 15:43:53 EST --- (In reply to comment #11)
Created attachment 531681 [details] Disable FORTIFY_SOURCE on ARM, incompatible with how setjmp is used.
This patch didn't make it into the F17 and Rawhide version and compile fails with the "*** longjmp causes uninitialized stack frame ***: /usr/bin/perl terminated" message.
Re-applying the patch allows the package to be built.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
Brendan Conoboy blc@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Platform|Unspecified |arm Version|15 |17 OS/Version|Unspecified |Linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
--- Comment #13 from Fedora Update System updates@fedoraproject.org 2012-02-22 04:35:44 EST --- perl-Coro-6.07-3.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/FEDORA-2012-2192/perl-Coro-6.07-3.fc...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
Brendan Conoboy blc@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NOTABUG Last Closed|2011-12-12 16:58:42 |2012-02-22 12:43:53
--- Comment #14 from Brendan Conoboy blc@redhat.com 2012-02-22 12:43:53 EST --- Thanks, perl-Coro-6.07-3.fc17 builds successfully on ARM.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
Brendan Conoboy blc@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|NOTABUG |RAWHIDE
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|perl-Coro-5.372-6.fc16 |perl-Coro-6.07-3.fc17 Resolution|RAWHIDE |ERRATA
--- Comment #15 from Fedora Update System updates@fedoraproject.org 2012-02-28 05:48:20 EST --- perl-Coro-6.07-3.fc17 has been pushed to the Fedora 17 stable repository. If problems still persist, please make note of it in this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=750805
Petr Pisar ppisar@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugzilla.redhat.com | |/show_bug.cgi?id=1242138
perl-devel@lists.fedoraproject.org