Opened on 08/28/06 at 14:16:31
Last modified on 12/18/13 at 12:28:46
#1168 closed defect (wontfix)
sparc BSP: Uninitialized variable in startup.S
Reported by: | Ingolf Steinbach | Owned by: | Joel Sherrill |
---|---|---|---|
Priority: | normal | Milestone: | 4.9 |
Component: | bsps | Version: | 4.6 |
Severity: | major | Keywords: | |
Cc: | chrisj@…, gedare@…, dunsmoreb@… | Blocked By: | |
Blocking: |
Description
The startup code of the leon2 RTEMS board support package (in src/rtems-4.6.5/c/src/lib/libbsp/sparc/shared/start.S) copies the data segment from ROM to RAM if it is not already there (line 286ff).
Before this copy operation, a value located in the not yet initialized RAM area is accessed in order to read the size of the RTEMS workspace area (line 220ff).
Please note that this problem may be related to PR 821. I'll try to provide some additional comments to PR 821 later.
Release:
4.6.5
Environment:
RTEMS 4.6.5 on leon2, text and data segment located in ROM, not using mkprom.
How-To-Repeat:
Create a ROM version of an executable (by re-directing the text and data sections to the rom memory area in the linkcmd file) and try to make use of the automatic copying of the data section.
Notice that the application won't boot because the workspace size is unreasonable.
Attachments (1)
Change History (7)
comment:1 Changed on 09/08/06 at 19:38:11 by Joel Sherrill
Status: | assigned → waiting |
---|
Changed on 12/03/06 at 13:31:12 by Ingolf Steinbach
Attachment: | start.S.diff added |
---|
comment:2 Changed on 05/22/08 at 15:00:18 by Ingolf Steinbach
attachments.ispatch: | 0 → 1 |
---|---|
attachments.mimetype: | application/octet-stream → text/plain |
comment:3 Changed on 05/22/08 at 15:20:28 by Ingolf Steinbach
Milestone: | → 4.9 |
---|---|
rep_platform: | → All |
comment:4 Changed on 04/22/10 at 05:33:48 by Chris Johns
Cc: | Chris Johns added; rtems-bugs@… removed |
---|
comment:5 Changed on 12/17/13 at 23:26:05 by Bryan Dunsmore
Resolution: | → invalid |
---|---|
Status: | assigned → closed, dunsmoreb@gmail.com |
comment:6 Changed on 12/18/13 at 12:28:46 by Gedare Bloom
Cc: | Gedare Bloom added |
---|---|
Resolution: | invalid → wontfix |
Yes Ingolf thank you for the astute comment. Joel, you could commit this back to the previous release branches if you want to.
start.S.diff