Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1624751
Summary: | Reboot loop on Orange Pi PC | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Alessio <alciregi> |
Component: | binutils | Assignee: | Nick Clifton <nickc> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | 29 | CC: | airlied, aoliva, bskeggs, dvlasenk, ewk, fweimer, hdegoede, ichavero, itamar, jakub, jarodwilson, jglisse, john.j5live, jonathan, josef, kernel-maint, linville, mchehab, mjg59, nickc, pbrobinson, steved |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | armv7hl | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | binutils-2.31.1-13.fc30 | Doc Type: | Bug Fix |
Doc Text: |
Cause:
A change to linker meant that configurable constants
such as the pagesize were evaluated after the parsing
of linker scripts. This in turn meant that if a linker
script referred to one of these constants, it could use
the wrong value.
Consequence:
Packages that use linker scripts that refer to these
constants could end up using the wrong values, which
could result in failure of the package to execute
properly.
Fix:
Delay the evaluation of expressions in linker scripts
until after the configurable constants have been
assigned their values.
Result:
Packages no longer break.
|
Story Points: | --- |
Clone Of: | Environment: | ||
Last Closed: | 2018-09-11 12:44:01 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 245418 |
Description
Alessio
2018-09-03 08:13:59 UTC
It's not a kernel issue but actually a U-Boot issue caused by a binutils issue now fixed upstream, we just need to rebuild once the fix lands in Fedora. https://lists.denx.de/pipermail/u-boot/2018-September/339967.html https://sourceware.org/bugzilla/show_bug.cgi?id=23571 It's not a kernel issue but actually a U-Boot issue caused by a binutils issue now fixed upstream, we just need to rebuild once the fix lands in Fedora. https://lists.denx.de/pipermail/u-boot/2018-September/339967.html https://sourceware.org/bugzilla/show_bug.cgi?id=23571 I have added the patch to the binutils sources and built new rpms. Please try: binutils-2.31.1-13.fc29 binutils-2.31.1-13.fc30 It should also be noted that this problem can be avoided in the U-Boot linker script if the align expression is put on the left of the colon. Like this: .__secure_start #ifndef CONFIG_ARMV7_SECURE_BASE ALIGN(CONSTANT(COMMONPAGESIZE)) #endif : { KEEP(*(.__secure_start)) } This has the advantage that the solution will work with older versions of the binutils package. > binutils-2.31.1-13.fc30 Tried this one and it works as expected and fixes the problem for me on a u-boot scratch build, will push a proper build shortly. > It should also be noted that this problem can be avoided in the U-Boot linker > script if the align expression is put on the left of the colon. Like this: > > .__secure_start > #ifndef CONFIG_ARMV7_SECURE_BASE > ALIGN(CONSTANT(COMMONPAGESIZE)) > #endif > : { > KEEP(*(.__secure_start)) > } > > This has the advantage that the solution will work with older versions of the > binutils package. Reported that upstream for the maintainer of the affected devices to deal with. Thanks a lot nick, this is much appreciated. arm-trusted-firmware-1.5-4.20180830.fc29 bcm283x-firmware-20180829-3.ec3f856.fc29 uboot-tools-2018.09-0.4.rc3.fc29 has been submitted as an update to Fedora 29. https://bodhi.fedoraproject.org/updates/FEDORA-2018-56bc88dfb2 arm-trusted-firmware-1.5-4.20180830.fc29, bcm283x-firmware-20180829-3.ec3f856.fc29, uboot-tools-2018.09-0.4.rc3.fc29 has been pushed to the Fedora 29 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2018-56bc88dfb2 arm-trusted-firmware-1.5-4.20180830.fc29, bcm283x-firmware-20180829-3.ec3f856.fc29, uboot-tools-2018.09-0.4.rc3.fc29 has been pushed to the Fedora 29 stable repository. If problems still persist, please make note of it in this bug report. I can confirm that now the Orange Pi PC boots. Thanks. |