#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)

start.S.diff (3.3 KB) - added by Ingolf Steinbach on Dec 3, 2006 at 1:31:12 PM.
start.S.diff

Download all attachments as: .zip

Change History (7)

comment:1 Changed on Sep 8, 2006 at 7:38:11 PM by Joel Sherrill

Status: assignedwaiting

Changed on Dec 3, 2006 at 1:31:12 PM by Ingolf Steinbach

Attachment: start.S.diff added

start.S.diff

comment:2 Changed on May 22, 2008 at 3:00:18 PM by Ingolf Steinbach

attachments.ispatch: 01
attachments.mimetype: application/octet-streamtext/plain

comment:3 Changed on May 22, 2008 at 3:20:28 PM by Ingolf Steinbach

Milestone: 4.9
rep_platform: All

comment:4 Changed on Apr 22, 2010 at 5:33:48 AM by Chris Johns

Cc: Chris Johns added; rtems-bugs@… removed

comment:5 Changed on Dec 17, 2013 at 11:26:05 PM by Bryan Dunsmore

Resolution: invalid
Status: assignedclosed, dunsmoreb@gmail.com

comment:6 Changed on Dec 18, 2013 at 12:28:46 PM by Gedare Bloom

Cc: Gedare Bloom added
Resolution: invalidwontfix

Yes Ingolf thank you for the astute comment. Joel, you could commit this back to the previous release branches if you want to.

Note: See TracTickets for help on using tickets.