[U-Boot] 2017.05-RC3 hanging on DM3730

Tom,
I re-synced my U-boot trunk today and did a clean build and found that SPL isn't able to load U-Boot.
U-Boot SPL 2017.05-rc3 (May 04 2017 - 15:48:21)
I haven't had a chance yet to bisect the issue. Do you know when you plan to do the 2017.05 release? I'd like to try and locate the issue before the final release.
adam

On Thu, May 04, 2017 at 04:06:33PM -0500, Adam Ford wrote:
Tom,
I re-synced my U-boot trunk today and did a clean build and found that SPL isn't able to load U-Boot.
U-Boot SPL 2017.05-rc3 (May 04 2017 - 15:48:21)
I haven't had a chance yet to bisect the issue. Do you know when you plan to do the 2017.05 release? I'd like to try and locate the issue before the final release.
I'd like to do the release on Monday. Can you please start bisecting? Thanks!

On Thu, May 4, 2017 at 4:27 PM, Tom Rini trini@konsulko.com wrote:
On Thu, May 04, 2017 at 04:06:33PM -0500, Adam Ford wrote:
Tom,
I re-synced my U-boot trunk today and did a clean build and found that SPL isn't able to load U-Boot.
U-Boot SPL 2017.05-rc3 (May 04 2017 - 15:48:21)
I haven't had a chance yet to bisect the issue. Do you know when you plan to do the 2017.05 release? I'd like to try and locate the issue before the final release.
I'd like to do the release on Monday. Can you please start bisecting?
I bisected it since I won't have time tomorrow or this weekend.
Commit 7584f2e0fb0f ("arm: omap3: Compile clock.c with -marm option to unbreak OMAP3530") seems to break the OMAP3630 (DM3730) in that nothing boots at all:
(dead)
Commit 19a75b8cf8d1("arm: omap3: Bring back ARM errata workaround 725233") at least shows some activity, but not enough to do anything:
U-Boot SPL 2017.03-00007-g19a75b8 (May 04 2017 - 17:53:10)
(dead)
I am wondering if other OMAP36/37 devices have had any issues?
adam
Thanks!
-- Tom

On Thu, May 04, 2017 at 05:58:47PM -0500, Adam Ford wrote:
On Thu, May 4, 2017 at 4:27 PM, Tom Rini trini@konsulko.com wrote:
On Thu, May 04, 2017 at 04:06:33PM -0500, Adam Ford wrote:
Tom,
I re-synced my U-boot trunk today and did a clean build and found that SPL isn't able to load U-Boot.
U-Boot SPL 2017.05-rc3 (May 04 2017 - 15:48:21)
I haven't had a chance yet to bisect the issue. Do you know when you plan to do the 2017.05 release? I'd like to try and locate the issue before the final release.
I'd like to do the release on Monday. Can you please start bisecting?
I bisected it since I won't have time tomorrow or this weekend.
Commit 7584f2e0fb0f ("arm: omap3: Compile clock.c with -marm option to unbreak OMAP3530") seems to break the OMAP3630 (DM3730) in that nothing boots at all:
(dead)
Commit 19a75b8cf8d1("arm: omap3: Bring back ARM errata workaround 725233") at least shows some activity, but not enough to do anything:
U-Boot SPL 2017.03-00007-g19a75b8 (May 04 2017 - 17:53:10)
(dead)
I am wondering if other OMAP36/37 devices have had any issues?
My bbXM is working. Can you try and see which of the errata is breaking your particular setup? Thanks!

On Thu, May 4, 2017 at 6:08 PM, Tom Rini trini@konsulko.com wrote:
On Thu, May 04, 2017 at 05:58:47PM -0500, Adam Ford wrote:
On Thu, May 4, 2017 at 4:27 PM, Tom Rini trini@konsulko.com wrote:
On Thu, May 04, 2017 at 04:06:33PM -0500, Adam Ford wrote:
Tom,
I re-synced my U-boot trunk today and did a clean build and found that SPL isn't able to load U-Boot.
U-Boot SPL 2017.05-rc3 (May 04 2017 - 15:48:21)
I haven't had a chance yet to bisect the issue. Do you know when you plan to do the 2017.05 release? I'd like to try and locate the issue before the final release.
I'd like to do the release on Monday. Can you please start bisecting?
I bisected it since I won't have time tomorrow or this weekend.
Commit 7584f2e0fb0f ("arm: omap3: Compile clock.c with -marm option to unbreak OMAP3530") seems to break the OMAP3630 (DM3730) in that nothing boots at all:
(dead)
Commit 19a75b8cf8d1("arm: omap3: Bring back ARM errata workaround 725233") at least shows some activity, but not enough to do anything:
U-Boot SPL 2017.03-00007-g19a75b8 (May 04 2017 - 17:53:10)
(dead)
I am wondering if other OMAP36/37 devices have had any issues?
My bbXM is working. Can you try and see which of the errata is breaking your particular setup? Thanks!
It appears to be a toolchain issue. I use Buildroot which builds everything from scratch. I specify some items like glibc, gcc version, binutils, etc.
I am not sure why the removal of "CFLAGS_clock.o += -marm" causes an issue.
When I built it with Linaro 2016.11 it worked, but that version is tuned for a corex-A9, but it proves the code is ok. Sorry for the noise.
adam
-- Tom

On Thu, 4 May 2017 18:38:39 -0500 Adam Ford aford173@gmail.com wrote:
On Thu, May 4, 2017 at 6:08 PM, Tom Rini trini@konsulko.com wrote:
On Thu, May 04, 2017 at 05:58:47PM -0500, Adam Ford wrote:
On Thu, May 4, 2017 at 4:27 PM, Tom Rini trini@konsulko.com wrote:
On Thu, May 04, 2017 at 04:06:33PM -0500, Adam Ford wrote:
Tom,
I re-synced my U-boot trunk today and did a clean build and found that SPL isn't able to load U-Boot.
U-Boot SPL 2017.05-rc3 (May 04 2017 - 15:48:21)
I haven't had a chance yet to bisect the issue. Do you know when you plan to do the 2017.05 release? I'd like to try and locate the issue before the final release.
I'd like to do the release on Monday. Can you please start bisecting?
I bisected it since I won't have time tomorrow or this weekend.
Commit 7584f2e0fb0f ("arm: omap3: Compile clock.c with -marm option to unbreak OMAP3530") seems to break the OMAP3630 (DM3730) in that nothing boots at all:
(dead)
Commit 19a75b8cf8d1("arm: omap3: Bring back ARM errata workaround 725233") at least shows some activity, but not enough to do anything:
U-Boot SPL 2017.03-00007-g19a75b8 (May 04 2017 - 17:53:10)
(dead)
I am wondering if other OMAP36/37 devices have had any issues?
My bbXM is working. Can you try and see which of the errata is breaking your particular setup? Thanks!
It appears to be a toolchain issue. I use Buildroot which builds everything from scratch. I specify some items like glibc, gcc version, binutils, etc.
I am not sure why the removal of "CFLAGS_clock.o += -marm" causes an issue.
When I built it with Linaro 2016.11 it worked, but that version is tuned for a corex-A9, but it proves the code is ok. Sorry for the noise.
Hello,
It is actually unlikely to be a toolchain issue, but more like some racy code in U-boot, which happens to be timing sensitive. As the commit message [1] says, it was not a proper fix but a quick and dirty workaround. More information is also available in the mailing list archive [2].
Anyway, thanks a lot for your feedback. It's good to know that DM3730 is also affected. Because this clearly rules out any possible impact of some Thumb2 errata (OMAP3530 had an old bug ridden Cortex-A8, but DM3730 has the latest revision of it).
Right now the OMAP3 clock code has some sort of a timing sensitive latent bug and it may potentially blow up on any future toolchain update (regardless of the use of the ARM or Thumb2 mode). So it's best if somebody finds the root cause and fixes it. But I guess, OMAP3 is very old and does not get much love nowadays.
1. http://git.denx.de/?p=u-boot.git;a=commit;h=7584f2e0fb0f 2. https://lists.denx.de/pipermail/u-boot/2017-March/283078.html

My employer is committed to producing product with the DM3730 until mid 2020's. Even if nobody else pushes OMAP3, I am willing to experiment and try code. If someone has something they need me to try.
Adam
On Jul 24, 2017 10:10 PM, "Siarhei Siamashka" siarhei.siamashka@gmail.com wrote:
On Thu, 4 May 2017 18:38:39 -0500 Adam Ford aford173@gmail.com wrote:
On Thu, May 4, 2017 at 6:08 PM, Tom Rini trini@konsulko.com wrote:
On Thu, May 04, 2017 at 05:58:47PM -0500, Adam Ford wrote:
On Thu, May 4, 2017 at 4:27 PM, Tom Rini trini@konsulko.com wrote:
On Thu, May 04, 2017 at 04:06:33PM -0500, Adam Ford wrote:
Tom,
I re-synced my U-boot trunk today and did a clean build and found
that
SPL isn't able to load U-Boot.
U-Boot SPL 2017.05-rc3 (May 04 2017 - 15:48:21)
I haven't had a chance yet to bisect the issue. Do you know when
you
plan to do the 2017.05 release? I'd like to try and locate the
issue
before the final release.
I'd like to do the release on Monday. Can you please start
bisecting?
I bisected it since I won't have time tomorrow or this weekend.
Commit 7584f2e0fb0f ("arm: omap3: Compile clock.c with -marm option to unbreak OMAP3530") seems to break the OMAP3630 (DM3730) in that nothing boots at all:
(dead)
Commit 19a75b8cf8d1("arm: omap3: Bring back ARM errata workaround 725233") at least shows some activity, but not enough to do anything:
U-Boot SPL 2017.03-00007-g19a75b8 (May 04 2017 - 17:53:10)
(dead)
I am wondering if other OMAP36/37 devices have had any issues?
My bbXM is working. Can you try and see which of the errata is
breaking
your particular setup? Thanks!
It appears to be a toolchain issue. I use Buildroot which builds everything from scratch. I specify some items like glibc, gcc version, binutils, etc.
I am not sure why the removal of "CFLAGS_clock.o += -marm" causes an
issue.
When I built it with Linaro 2016.11 it worked, but that version is tuned for a corex-A9, but it proves the code is ok. Sorry for the noise.
Hello,
It is actually unlikely to be a toolchain issue, but more like some racy code in U-boot, which happens to be timing sensitive. As the commit message [1] says, it was not a proper fix but a quick and dirty workaround. More information is also available in the mailing list archive [2].
Anyway, thanks a lot for your feedback. It's good to know that DM3730 is also affected. Because this clearly rules out any possible impact of some Thumb2 errata (OMAP3530 had an old bug ridden Cortex-A8, but DM3730 has the latest revision of it).
Right now the OMAP3 clock code has some sort of a timing sensitive latent bug and it may potentially blow up on any future toolchain update (regardless of the use of the ARM or Thumb2 mode). So it's best if somebody finds the root cause and fixes it. But I guess, OMAP3 is very old and does not get much love nowadays.
- http://git.denx.de/?p=u-boot.git;a=commit;h=7584f2e0fb0f
- https://lists.denx.de/pipermail/u-boot/2017-March/283078.html
-- Best regards, Siarhei Siamashka

Hello Adam,
Am 04.05.2017 um 23:06 schrieb Adam Ford:
Tom,
I re-synced my U-boot trunk today and did a clean build and found that SPL isn't able to load U-Boot.
U-Boot SPL 2017.05-rc3 (May 04 2017 - 15:48:21)
I haven't had a chance yet to bisect the issue. Do you know when you plan to do the 2017.05 release? I'd like to try and locate the issue before the final release.
May you want to try tbot [1]? It has a testcase with which you can git bisect automagically [2]
Ok, tbot setup is not done in a second... but may you find some time to look into it...
bye, Heiko [1] https://github.com/hsdenx/tbot http://www.tbot.tools/main.html [2] https://github.com/hsdenx/tbot/blob/master/src/tc/tc_board_git_bisect.py [3] log for an automated git bisect on tqm5200 board with BDI attached http://xeidos.ddns.net/buildbot/builders/tqm5200s/builds/3/steps/shell/logs/...
participants (4)
-
Adam Ford
-
Heiko Schocher
-
Siarhei Siamashka
-
Tom Rini