* [U-Boot] "arm: keep all sections in ELF file" breaks Arndale
@ 2013-12-17 15:24 Andre Przywara
2013-12-17 15:30 ` Tom Rini
0 siblings, 1 reply; 10+ messages in thread
From: Andre Przywara @ 2013-12-17 15:24 UTC (permalink / raw)
To: u-boot
Hi,
the Arndale board does not work anymore with current master HEAD.
If I turn on the board, I see exactly nothing.
I bisected it down to:
47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7 is the first bad commit
commit 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7
Author: Albert ARIBAUD <albert.u.boot@aribaud.net>
Date: Thu Nov 7 14:21:46 2013 +0100
arm: keep all sections in ELF file
That patch looks like Chinese to me, does one of you have an idea what's
wrong here?
Are we missing a section, maybe for the SPL build?
Regards,
Andre.
^ permalink raw reply [flat|nested] 10+ messages in thread* [U-Boot] "arm: keep all sections in ELF file" breaks Arndale 2013-12-17 15:24 [U-Boot] "arm: keep all sections in ELF file" breaks Arndale Andre Przywara @ 2013-12-17 15:30 ` Tom Rini 2013-12-17 15:47 ` Andre Przywara 0 siblings, 1 reply; 10+ messages in thread From: Tom Rini @ 2013-12-17 15:30 UTC (permalink / raw) To: u-boot -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 12/17/2013 10:24 AM, Andre Przywara wrote: > Hi, > > the Arndale board does not work anymore with current master HEAD. > If I turn on the board, I see exactly nothing. > > I bisected it down to: > 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7 is the first bad commit > commit 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7 > Author: Albert ARIBAUD <albert.u.boot@aribaud.net> > Date: Thu Nov 7 14:21:46 2013 +0100 > > arm: keep all sections in ELF file > > That patch looks like Chinese to me, does one of you have an idea what's > wrong here? > Are we missing a section, maybe for the SPL build? So on Arndale, what file do you boot exactly? It sounds like not u-boot.img but some tool stripping / modifying u-boot (or spl/u-boot-spl) and having relied on certain sections being stripped out before being run. - -- Tom -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBAgAGBQJSsG4KAAoJENk4IS6UOR1Wd0gP/jve70MGVIbZ7I6ZNVfBMXlC kUCrbskbSSNhLvp+prf7X4YGAqGkqzHm2kbdPS1BC3nEDbpJxvQodN1DYDat8I9U 7Et3fMLqpWzRSs/8FyN6ISDfo/hn4DSKKLRlHi/sg4RWYhDWIfwNByyVN2Hcpicb grUqz27GoJbTWld8ojkDCRILYS22RgCXQCdpnDvDv+a1Yvx6pb2/etH2VykOnFdN FVz0vHr7ecGbDgLuK7elKR0aJhyXkU8z6w2TAW713d+MkXkuHytbO8d4DtIysRrj L7quOdVH5qWyY8UivMZbSWU/UfExu6lGF8HhzlebWc2x80Ghhyxyc8i0/PpmEL20 n5rrBAAPNBKbbN5YlhZ4XnXIbp8o9FFI6rvXj/Gx6sfbU14BtYkkYnc5IeEz2RcK dMXYycAdp+7ZzYoMGUYOBR+GCTQ38f0ZFJgkOlmYgShPYEHDMNrW40SKm2zR7xPR LpQ/YI1XzkuAdoQ6+Plx/Felf5KTS8ul3y55PqWPr8UFcFCggG84WL/aTOsVxCnL 6CM7h4RY0ainSHUw431JoPdILcYQIbJz84vo8hOoIdlXX3+dffSHH0g6Eg4QRHEO 75UK0JMXCHUhg519dy8OEPYwi8jZOlDyx+lEqbhaTYorXxYJzCZmTOpWFVarix7c fhyXLvN+llrt1v3mjgbb =siI2 -----END PGP SIGNATURE----- ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] "arm: keep all sections in ELF file" breaks Arndale 2013-12-17 15:30 ` Tom Rini @ 2013-12-17 15:47 ` Andre Przywara 2013-12-17 15:57 ` Tom Rini 0 siblings, 1 reply; 10+ messages in thread From: Andre Przywara @ 2013-12-17 15:47 UTC (permalink / raw) To: u-boot (CCing Inderpal, Chander and Minkyu) On 12/17/2013 04:30 PM, Tom Rini wrote: > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 > > On 12/17/2013 10:24 AM, Andre Przywara wrote: >> Hi, >> >> the Arndale board does not work anymore with current master HEAD. >> If I turn on the board, I see exactly nothing. >> >> I bisected it down to: >> 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7 is the first bad commit >> commit 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7 >> Author: Albert ARIBAUD <albert.u.boot@aribaud.net> >> Date: Thu Nov 7 14:21:46 2013 +0100 >> >> arm: keep all sections in ELF file >> >> That patch looks like Chinese to me, does one of you have an idea what's >> wrong here? >> Are we missing a section, maybe for the SPL build? > > So on Arndale, what file do you boot exactly? It sounds like not > u-boot.img but some tool stripping / modifying u-boot (or > spl/u-boot-spl) and having relied on certain sections being stripped out > before being run. Good point. Indeed I boot u-boot-dtb.bin (the SPL is spl/arndale-spl.bin). So the Makefile does: cat u-boot.bin u-boot.dtb >u-boot-dtb.bin I guess the address under which U-boot expects to find the attached dtb changes due to the non-discarding. Does any of the Arndale people have an idea how to fix this? Regards, Andre. > > - -- > Tom > -----BEGIN PGP SIGNATURE----- > Version: GnuPG v1.4.11 (GNU/Linux) > Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ > > iQIcBAEBAgAGBQJSsG4KAAoJENk4IS6UOR1Wd0gP/jve70MGVIbZ7I6ZNVfBMXlC > kUCrbskbSSNhLvp+prf7X4YGAqGkqzHm2kbdPS1BC3nEDbpJxvQodN1DYDat8I9U > 7Et3fMLqpWzRSs/8FyN6ISDfo/hn4DSKKLRlHi/sg4RWYhDWIfwNByyVN2Hcpicb > grUqz27GoJbTWld8ojkDCRILYS22RgCXQCdpnDvDv+a1Yvx6pb2/etH2VykOnFdN > FVz0vHr7ecGbDgLuK7elKR0aJhyXkU8z6w2TAW713d+MkXkuHytbO8d4DtIysRrj > L7quOdVH5qWyY8UivMZbSWU/UfExu6lGF8HhzlebWc2x80Ghhyxyc8i0/PpmEL20 > n5rrBAAPNBKbbN5YlhZ4XnXIbp8o9FFI6rvXj/Gx6sfbU14BtYkkYnc5IeEz2RcK > dMXYycAdp+7ZzYoMGUYOBR+GCTQ38f0ZFJgkOlmYgShPYEHDMNrW40SKm2zR7xPR > LpQ/YI1XzkuAdoQ6+Plx/Felf5KTS8ul3y55PqWPr8UFcFCggG84WL/aTOsVxCnL > 6CM7h4RY0ainSHUw431JoPdILcYQIbJz84vo8hOoIdlXX3+dffSHH0g6Eg4QRHEO > 75UK0JMXCHUhg519dy8OEPYwi8jZOlDyx+lEqbhaTYorXxYJzCZmTOpWFVarix7c > fhyXLvN+llrt1v3mjgbb > =siI2 > -----END PGP SIGNATURE----- > ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] "arm: keep all sections in ELF file" breaks Arndale 2013-12-17 15:47 ` Andre Przywara @ 2013-12-17 15:57 ` Tom Rini 2013-12-18 14:06 ` Andre Przywara 0 siblings, 1 reply; 10+ messages in thread From: Tom Rini @ 2013-12-17 15:57 UTC (permalink / raw) To: u-boot -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 12/17/2013 10:47 AM, Andre Przywara wrote: > (CCing Inderpal, Chander and Minkyu) > > On 12/17/2013 04:30 PM, Tom Rini wrote: > On 12/17/2013 10:24 AM, Andre Przywara wrote: >>>> Hi, >>>> >>>> the Arndale board does not work anymore with current master HEAD. >>>> If I turn on the board, I see exactly nothing. >>>> >>>> I bisected it down to: >>>> 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7 is the first bad commit >>>> commit 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7 >>>> Author: Albert ARIBAUD <albert.u.boot@aribaud.net> >>>> Date: Thu Nov 7 14:21:46 2013 +0100 >>>> >>>> arm: keep all sections in ELF file >>>> >>>> That patch looks like Chinese to me, does one of you have an idea what's >>>> wrong here? >>>> Are we missing a section, maybe for the SPL build? > > So on Arndale, what file do you boot exactly? It sounds like not > u-boot.img but some tool stripping / modifying u-boot (or > spl/u-boot-spl) and having relied on certain sections being stripped out > before being run. > >> Good point. Indeed I boot u-boot-dtb.bin (the SPL is >> spl/arndale-spl.bin). So the Makefile does: >> cat u-boot.bin u-boot.dtb >u-boot-dtb.bin > >> I guess the address under which U-boot expects to find the attached dtb >> changes due to the non-discarding. > >> Does any of the Arndale people have an idea how to fix this? That's a little confusing as we use objcopy -O binary to make u-boot.bin/u-boot-spl.bin and that would, I would think, drop these sections anyhow. But some checking of binary size with the patch in question locally reverted would help shed some light here. - -- Tom -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBAgAGBQJSsHR8AAoJENk4IS6UOR1W+7MQALP3dn79SRWgTTxHQ1SVMUbB FC0mVsupa296m5WfcDRPkijtZecAeTkFGVmizTSqjLepKhV5Tnqy4uQePhwxwdcg v9oC5/hq7NXuJIB+bzUyc05kznrBqqJ4nyS7FGJDZNmXYhbtt6BdUDSrPMVX1eLk TkPPZ1ksD2oyJL8el/f8LlQRrKu68GaJY63qdvgFtRfGaQ+OuRYLJZ6ow3IZDbuu +Cl9easvbEaq/db/t2U72s5VozGyMgx855w+aU9eZW7OmS2pSpSjETx3VBRW8vDv HsHKFwykHOtajp644oGEQHvUA7PWZdPsfFOh38POp+7x5YYYNSdNO3D0yZ0PmzrD E7w6p4bWUlZK/xQ3Bi5BYKI4Ps+0Rf64fGUHc13ML9P0aG4quNHDhHm98UT9EEWD ZoJLRccoGvZjGNonz/Q8Hx18NWBfcFpwSFHhxMGUGfWiyjSDwaiplpXHujcOXlSC zZQs7WT4vPfaWsTdpyaREEZ+4W6nJgf2Zyaakdn/ul+6EUjERrHEvVIe9RrsG7Ut NsOCk0P0txWW+gEHrViorNL6n142Fb/0gwhJnAvUmpvPqOZlQqaDvE998/PzBWmi E7cY+CEktxaTHGFWsSo/QYS+AUZBx1k7Oq+yKnFXn+pvLnnr4Z9qF2cF/RByJ2v3 1vBudkiMaREUsuKm6H6x =aMwD -----END PGP SIGNATURE----- ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] "arm: keep all sections in ELF file" breaks Arndale 2013-12-17 15:57 ` Tom Rini @ 2013-12-18 14:06 ` Andre Przywara 2013-12-21 10:46 ` Albert ARIBAUD 0 siblings, 1 reply; 10+ messages in thread From: Andre Przywara @ 2013-12-18 14:06 UTC (permalink / raw) To: u-boot On 12/17/2013 04:57 PM, Tom Rini wrote: > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 > > On 12/17/2013 10:47 AM, Andre Przywara wrote: >> (CCing Inderpal, Chander and Minkyu) >> >> On 12/17/2013 04:30 PM, Tom Rini wrote: >> On 12/17/2013 10:24 AM, Andre Przywara wrote: >>>>> Hi, >>>>> >>>>> the Arndale board does not work anymore with current master HEAD. >>>>> If I turn on the board, I see exactly nothing. >>>>> >>>>> I bisected it down to: >>>>> 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7 is the first bad commit >>>>> commit 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7 >>>>> Author: Albert ARIBAUD <albert.u.boot@aribaud.net> >>>>> Date: Thu Nov 7 14:21:46 2013 +0100 >>>>> >>>>> arm: keep all sections in ELF file >>>>> >>>>> That patch looks like Chinese to me, does one of you have an idea what's >>>>> wrong here? >>>>> Are we missing a section, maybe for the SPL build? >> >> So on Arndale, what file do you boot exactly? It sounds like not >> u-boot.img but some tool stripping / modifying u-boot (or >> spl/u-boot-spl) and having relied on certain sections being stripped out >> before being run. >> >>> Good point. Indeed I boot u-boot-dtb.bin (the SPL is >>> spl/arndale-spl.bin). So the Makefile does: >>> cat u-boot.bin u-boot.dtb >u-boot-dtb.bin >> >>> I guess the address under which U-boot expects to find the attached dtb >>> changes due to the non-discarding. >> >>> Does any of the Arndale people have an idea how to fix this? > > That's a little confusing as we use objcopy -O binary to make > u-boot.bin/u-boot-spl.bin and that would, I would think, drop these > sections anyhow. But some checking of binary size with the patch in > question locally reverted would help shed some light here. So I did objdump -h on ./u-boot, the diff (sorted by the section name, as the order of the section differed, and the size) is: .ARM.attributes 0000002d -.bss 00048b10 +.bss 00048b08 .bss_end 00000000 .bss_start 00000000 .comment 0000001c @@ -14,8 +14,12 @@ .debug_loc 00045a4e .debug_ranges 00006f70 .debug_str 000123b4 +.dynamic 00000080 +.dynstr 0000001d +.dynsym 00000060 .got.plt 0000000c .hash 0000002c +.interp 00000011 .rel.dyn 00006fd8 .rodata 0000ae47 .text 0002cdf0 - works, + is broken. + is HEAD as of yesterday, - is the same with the patch in question reverted. So the patch _adds_ some sections to the file. u-boot-spl is exactly the same, for u-boot-spl.bin the broken version ends earlier (but until then is identical). arndale-spl.bin (which I flash) is different, though. Arndale uses CONFIG_OF_SEPARATE, and .text stays the same AFAICS. I can do some more debugging tomorrow, am grateful for any hints. Regards, Andre. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] "arm: keep all sections in ELF file" breaks Arndale 2013-12-18 14:06 ` Andre Przywara @ 2013-12-21 10:46 ` Albert ARIBAUD 2013-12-21 20:57 ` Andre Przywara 2014-01-07 14:04 ` [U-Boot] "arm: keep all sections in ELF file" breaks Arndale Andre Przywara 0 siblings, 2 replies; 10+ messages in thread From: Albert ARIBAUD @ 2013-12-21 10:46 UTC (permalink / raw) To: u-boot Hi Andre, On Wed, 18 Dec 2013 15:06:04 +0100, Andre Przywara <andre.przywara@linaro.org> wrote: > On 12/17/2013 04:57 PM, Tom Rini wrote: > > -----BEGIN PGP SIGNED MESSAGE----- > > Hash: SHA1 > > > > On 12/17/2013 10:47 AM, Andre Przywara wrote: > >> (CCing Inderpal, Chander and Minkyu) > >> > >> On 12/17/2013 04:30 PM, Tom Rini wrote: > >> On 12/17/2013 10:24 AM, Andre Przywara wrote: > >>>>> Hi, > >>>>> > >>>>> the Arndale board does not work anymore with current master HEAD. > >>>>> If I turn on the board, I see exactly nothing. > >>>>> > >>>>> I bisected it down to: > >>>>> 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7 is the first bad commit > >>>>> commit 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7 > >>>>> Author: Albert ARIBAUD <albert.u.boot@aribaud.net> > >>>>> Date: Thu Nov 7 14:21:46 2013 +0100 > >>>>> > >>>>> arm: keep all sections in ELF file > >>>>> > >>>>> That patch looks like Chinese to me, does one of you have an idea what's > >>>>> wrong here? > >>>>> Are we missing a section, maybe for the SPL build? > >> > >> So on Arndale, what file do you boot exactly? It sounds like not > >> u-boot.img but some tool stripping / modifying u-boot (or > >> spl/u-boot-spl) and having relied on certain sections being stripped out > >> before being run. > >> > >>> Good point. Indeed I boot u-boot-dtb.bin (the SPL is > >>> spl/arndale-spl.bin). So the Makefile does: > >>> cat u-boot.bin u-boot.dtb >u-boot-dtb.bin > >> > >>> I guess the address under which U-boot expects to find the attached dtb > >>> changes due to the non-discarding. > >> > >>> Does any of the Arndale people have an idea how to fix this? > > > > That's a little confusing as we use objcopy -O binary to make > > u-boot.bin/u-boot-spl.bin and that would, I would think, drop these > > sections anyhow. But some checking of binary size with the patch in > > question locally reverted would help shed some light here. > > So I did objdump -h on ./u-boot, the diff (sorted by the section name, > as the order of the section differed, and the size) is: > > .ARM.attributes 0000002d > -.bss 00048b10 > +.bss 00048b08 > .bss_end 00000000 > .bss_start 00000000 > .comment 0000001c > @@ -14,8 +14,12 @@ > .debug_loc 00045a4e > .debug_ranges 00006f70 > .debug_str 000123b4 > +.dynamic 00000080 > +.dynstr 0000001d > +.dynsym 00000060 > .got.plt 0000000c > .hash 0000002c > +.interp 00000011 > .rel.dyn 00006fd8 > .rodata 0000ae47 > .text 0002cdf0 > > - works, + is broken. > + is HEAD as of yesterday, - is the same with the patch in question > reverted. So the patch _adds_ some sections to the file. > u-boot-spl is exactly the same, for u-boot-spl.bin the broken version > ends earlier (but until then is identical). arndale-spl.bin (which I > flash) is different, though. > Arndale uses CONFIG_OF_SEPARATE, and .text stays the same AFAICS. > > I can do some more debugging tomorrow, am grateful for any hints. Sorry for chiming in late. Did you try this? http://patchwork.ozlabs.org/patch/300928/ Amicalement, -- Albert. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] "arm: keep all sections in ELF file" breaks Arndale 2013-12-21 10:46 ` Albert ARIBAUD @ 2013-12-21 20:57 ` Andre Przywara 2013-12-21 23:25 ` Andre Przywara 2014-01-07 14:04 ` [U-Boot] "arm: keep all sections in ELF file" breaks Arndale Andre Przywara 1 sibling, 1 reply; 10+ messages in thread From: Andre Przywara @ 2013-12-21 20:57 UTC (permalink / raw) To: u-boot On 12/21/2013 11:46 AM, Albert ARIBAUD wrote: > Hi Andre, > > On Wed, 18 Dec 2013 15:06:04 +0100, Andre Przywara > <andre.przywara@linaro.org> wrote: > >> On 12/17/2013 04:57 PM, Tom Rini wrote: >>> -----BEGIN PGP SIGNED MESSAGE----- >>> Hash: SHA1 >>> >>> On 12/17/2013 10:47 AM, Andre Przywara wrote: >>>> (CCing Inderpal, Chander and Minkyu) >>>> >>>> On 12/17/2013 04:30 PM, Tom Rini wrote: >>>> On 12/17/2013 10:24 AM, Andre Przywara wrote: >>>>>>> Hi, >>>>>>> >>>>>>> the Arndale board does not work anymore with current master HEAD. >>>>>>> If I turn on the board, I see exactly nothing. >>>>>>> >>>>>>> I bisected it down to: >>>>>>> 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7 is the first bad commit >>>>>>> commit 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7 >>>>>>> Author: Albert ARIBAUD <albert.u.boot@aribaud.net> >>>>>>> Date: Thu Nov 7 14:21:46 2013 +0100 >>>>>>> >>>>>>> arm: keep all sections in ELF file >>>>>>> >>>>>>> That patch looks like Chinese to me, does one of you have an idea what's >>>>>>> wrong here? >>>>>>> Are we missing a section, maybe for the SPL build? >>>> >>>> So on Arndale, what file do you boot exactly? It sounds like not >>>> u-boot.img but some tool stripping / modifying u-boot (or >>>> spl/u-boot-spl) and having relied on certain sections being stripped out >>>> before being run. >>>> >>>>> Good point. Indeed I boot u-boot-dtb.bin (the SPL is >>>>> spl/arndale-spl.bin). So the Makefile does: >>>>> cat u-boot.bin u-boot.dtb >u-boot-dtb.bin >>>> >>>>> I guess the address under which U-boot expects to find the attached dtb >>>>> changes due to the non-discarding. >>>> >>>>> Does any of the Arndale people have an idea how to fix this? >>> >>> That's a little confusing as we use objcopy -O binary to make >>> u-boot.bin/u-boot-spl.bin and that would, I would think, drop these >>> sections anyhow. But some checking of binary size with the patch in >>> question locally reverted would help shed some light here. >> >> So I did objdump -h on ./u-boot, the diff (sorted by the section name, >> as the order of the section differed, and the size) is: >> >> .ARM.attributes 0000002d >> -.bss 00048b10 >> +.bss 00048b08 >> .bss_end 00000000 >> .bss_start 00000000 >> .comment 0000001c >> @@ -14,8 +14,12 @@ >> .debug_loc 00045a4e >> .debug_ranges 00006f70 >> .debug_str 000123b4 >> +.dynamic 00000080 >> +.dynstr 0000001d >> +.dynsym 00000060 >> .got.plt 0000000c >> .hash 0000002c >> +.interp 00000011 >> .rel.dyn 00006fd8 >> .rodata 0000ae47 >> .text 0002cdf0 >> >> - works, + is broken. >> + is HEAD as of yesterday, - is the same with the patch in question >> reverted. So the patch _adds_ some sections to the file. >> u-boot-spl is exactly the same, for u-boot-spl.bin the broken version >> ends earlier (but until then is identical). arndale-spl.bin (which I >> flash) is different, though. >> Arndale uses CONFIG_OF_SEPARATE, and .text stays the same AFAICS. >> >> I can do some more debugging tomorrow, am grateful for any hints. > > Sorry for chiming in late. Did you try this? > > http://patchwork.ozlabs.org/patch/300928/ I cannot reach the server. Can you give me a hint what this patch is about (or a Subject: line to search for on the ML?) Thanks, Andre. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] "arm: keep all sections in ELF file" breaks Arndale 2013-12-21 20:57 ` Andre Przywara @ 2013-12-21 23:25 ` Andre Przywara 2014-01-29 15:37 ` [U-Boot] "arm: keep all sections in ELF file" breaks Zynq Michal Simek 0 siblings, 1 reply; 10+ messages in thread From: Andre Przywara @ 2013-12-21 23:25 UTC (permalink / raw) To: u-boot On 12/21/2013 09:57 PM, Andre Przywara wrote: > On 12/21/2013 11:46 AM, Albert ARIBAUD wrote: >> Hi Andre, >> >> On Wed, 18 Dec 2013 15:06:04 +0100, Andre Przywara >> <andre.przywara@linaro.org> wrote: >> >>> On 12/17/2013 04:57 PM, Tom Rini wrote: >>>> -----BEGIN PGP SIGNED MESSAGE----- >>>> Hash: SHA1 >>>> >>>> On 12/17/2013 10:47 AM, Andre Przywara wrote: >>>>> (CCing Inderpal, Chander and Minkyu) >>>>> >>>>> On 12/17/2013 04:30 PM, Tom Rini wrote: >>>>> On 12/17/2013 10:24 AM, Andre Przywara wrote: >>>>>>>> Hi, >>>>>>>> >>>>>>>> the Arndale board does not work anymore with current master HEAD. >>>>>>>> If I turn on the board, I see exactly nothing. >>>>>>>> >>>>>>>> I bisected it down to: >>>>>>>> 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7 is the first bad commit >>>>>>>> commit 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7 >>>>>>>> Author: Albert ARIBAUD <albert.u.boot@aribaud.net> >>>>>>>> Date: Thu Nov 7 14:21:46 2013 +0100 >>>>>>>> >>>>>>>> arm: keep all sections in ELF file >>>>>>>> >>>>>>>> That patch looks like Chinese to me, does one of you have an >>>>>>>> idea what's >>>>>>>> wrong here? >>>>>>>> Are we missing a section, maybe for the SPL build? >>>>> >>>>> So on Arndale, what file do you boot exactly? It sounds like not >>>>> u-boot.img but some tool stripping / modifying u-boot (or >>>>> spl/u-boot-spl) and having relied on certain sections being >>>>> stripped out >>>>> before being run. >>>>> >>>>>> Good point. Indeed I boot u-boot-dtb.bin (the SPL is >>>>>> spl/arndale-spl.bin). So the Makefile does: >>>>>> cat u-boot.bin u-boot.dtb >u-boot-dtb.bin >>>>> >>>>>> I guess the address under which U-boot expects to find the >>>>>> attached dtb >>>>>> changes due to the non-discarding. >>>>> >>>>>> Does any of the Arndale people have an idea how to fix this? >>>> >>>> That's a little confusing as we use objcopy -O binary to make >>>> u-boot.bin/u-boot-spl.bin and that would, I would think, drop these >>>> sections anyhow. But some checking of binary size with the patch in >>>> question locally reverted would help shed some light here. >>> >>> So I did objdump -h on ./u-boot, the diff (sorted by the section name, >>> as the order of the section differed, and the size) is: >>> >>> .ARM.attributes 0000002d >>> -.bss 00048b10 >>> +.bss 00048b08 >>> .bss_end 00000000 >>> .bss_start 00000000 >>> .comment 0000001c >>> @@ -14,8 +14,12 @@ >>> .debug_loc 00045a4e >>> .debug_ranges 00006f70 >>> .debug_str 000123b4 >>> +.dynamic 00000080 >>> +.dynstr 0000001d >>> +.dynsym 00000060 >>> .got.plt 0000000c >>> .hash 0000002c >>> +.interp 00000011 >>> .rel.dyn 00006fd8 >>> .rodata 0000ae47 >>> .text 0002cdf0 >>> >>> - works, + is broken. >>> + is HEAD as of yesterday, - is the same with the patch in question >>> reverted. So the patch _adds_ some sections to the file. >>> u-boot-spl is exactly the same, for u-boot-spl.bin the broken version >>> ends earlier (but until then is identical). arndale-spl.bin (which I >>> flash) is different, though. >>> Arndale uses CONFIG_OF_SEPARATE, and .text stays the same AFAICS. >>> >>> I can do some more debugging tomorrow, am grateful for any hints. >> >> Sorry for chiming in late. Did you try this? >> >> http://patchwork.ozlabs.org/patch/300928/ > > I cannot reach the server. Can you give me a hint what this patch is > about (or a Subject: line to search for on the ML?) OK, now the server works again. Looks like this could fix it on the first glance, but I momentarily cannot test this. Will try to do this ASAP. Thanks, Andr?. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] "arm: keep all sections in ELF file" breaks Zynq 2013-12-21 23:25 ` Andre Przywara @ 2014-01-29 15:37 ` Michal Simek 0 siblings, 0 replies; 10+ messages in thread From: Michal Simek @ 2014-01-29 15:37 UTC (permalink / raw) To: u-boot Hi, I have also problem with this patch but from the other side. Tool for booting image generation expects that LOAD section is the first. That's why I have to remove PHDR, INTERP sections. Not sure about DYNAMIC section which is also there but behind LOAD section. I can easily create zynq private linker script but maybe there is better way how to remove these section for us via LDFLAGS_FINAL in config.mk. Unfortunatelly I haven't found how to do it. Can someone know if it is possible to remove some sections via ld command line? Thanks, Michal -- Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91 w: www.monstr.eu p: +42-0-721842854 Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/ Maintainer of Linux kernel - Xilinx Zynq ARM architecture Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 263 bytes Desc: OpenPGP digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140129/634ac939/attachment.pgp> ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] "arm: keep all sections in ELF file" breaks Arndale 2013-12-21 10:46 ` Albert ARIBAUD 2013-12-21 20:57 ` Andre Przywara @ 2014-01-07 14:04 ` Andre Przywara 1 sibling, 0 replies; 10+ messages in thread From: Andre Przywara @ 2014-01-07 14:04 UTC (permalink / raw) To: u-boot On 12/21/2013 11:46 AM, Albert ARIBAUD wrote: > Hi Andre, > > On Wed, 18 Dec 2013 15:06:04 +0100, Andre Przywara > <andre.przywara@linaro.org> wrote: > >>>>>>> the Arndale board does not work anymore with current master HEAD. >>>>>>> If I turn on the board, I see exactly nothing. >>>>>>> >>>>>>> I bisected it down to: >>>>>>> 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7 is the first bad commit >>>>>>> commit 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7 >>>>>>> Author: Albert ARIBAUD <albert.u.boot@aribaud.net> >>>>>>> Date: Thu Nov 7 14:21:46 2013 +0100 >>>>>>> >>>>>>> arm: keep all sections in ELF file >>>>>>> .... Albert, > > Sorry for chiming in late. Did you try this? > > http://patchwork.ozlabs.org/patch/300928/ OK, got around to test this now, it indeed fixes the issue for me. In this original thread of yours you mentioned a v2, has this been sent out yet? Please make sure to push this patch into the release, otherwise we will have a regression with Arndale (and others). Thanks, Andre. ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2014-01-29 15:37 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-12-17 15:24 [U-Boot] "arm: keep all sections in ELF file" breaks Arndale Andre Przywara 2013-12-17 15:30 ` Tom Rini 2013-12-17 15:47 ` Andre Przywara 2013-12-17 15:57 ` Tom Rini 2013-12-18 14:06 ` Andre Przywara 2013-12-21 10:46 ` Albert ARIBAUD 2013-12-21 20:57 ` Andre Przywara 2013-12-21 23:25 ` Andre Przywara 2014-01-29 15:37 ` [U-Boot] "arm: keep all sections in ELF file" breaks Zynq Michal Simek 2014-01-07 14:04 ` [U-Boot] "arm: keep all sections in ELF file" breaks Arndale Andre Przywara
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.