Bug 399006 - ti.targets.*.C6* linker command files are missing .ppinfo and .ppdata sections
Summary: ti.targets.*.C6* linker command files are missing .ppinfo and .ppdata sections
Status: CLOSED FIXED
Alias: None
Product: RTSC (ARCHIVED)
Classification: Technology
Component: Targets (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Sasha Slijepcevic CLA
QA Contact:
URL:
Whiteboard: targets: 3.24.07,3.25.00
Keywords: readme
Depends on:
Blocks:
 
Reported: 2013-01-24 13:44 EST by Dave Russo CLA
Modified: 2013-04-04 17:07 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Dave Russo CLA 2013-01-24 13:44:24 EST 
The C6x generated linker command files are missing the .ppinfo and .ppdata section placement in the generated linker command files.

These sections should be added to the generated linker command files and placed in "data" memory.

Without these sections the linker generates warnings abut lack of placement and the sections may be placed randomly.
Comment 1 Dave Russo CLA 2013-01-24 13:51:00 EST 
Good description of the .ppdata section on this link

http://e2e.ti.com/support/development_tools/compiler/f/343/t/218545.aspx
Comment 2 Sasha Slijepcevic CLA 2013-02-05 14:50:43 EST 
I'll remove "profile" and "coverage" profiles from C6x targets because these are the targets impacted by this bug, but I am guessing we want to remove these profiles from all targets.
Comment 3 Dave Russo CLA 2013-02-05 15:35:20 EST 
The only targets that fail for these profiles are the C6x.  So, we should not remove them from any other target.

Also, the C6x targets do not fail, we are simply getting new warnings from these targets because we are simply not including section placement for the .pp* sections.  An alternative is to simply include these sections in the TI sect map.
Comment 4 Sasha Slijepcevic CLA 2013-02-06 15:06:57 EST 
I removed profiles from xdctargets-g15 because it seemed we were in agreement about that solution. I'll leave the bug open, if we want to talk about it little bit more.
Comment 5 Sasha Slijepcevic CLA 2013-02-06 15:09:24 EST 
I removed the profiles "profile" and "coverage" only from c6x targets. Others are left as they were.
Comment 6 Sasha Slijepcevic CLA 2013-02-21 17:42:04 EST 
- also fixed in xdctargets-f51 for XDCtools 3.24.
We may return to this later to fix it differently, probably by adding sections to the target spec.
Comment 7 Dave Russo CLA 2013-03-04 16:54:35 EST 
In latest 3.25 product (xdctools_3_25_00_42_eng):

    [sanblnx02 745] find . -name "*.xdc" -exec egrep -l '"profile"|coverage' {} /dev/null \;
    ./arm/elf/IArm.xdc
    ./arp32/elf/IARP32.xdc
    ./C28.xdc
    ./C28_float.xdc
    ./C28_large.xdc
    ./msp430/MSP430.xdc
    ./msp430/MSP430X.xdc
    ./msp430/MSP430X_small.xdc
    ./msp430/elf/MSP430.xdc
    ./msp430/elf/MSP430X.xdc
    ./msp430/elf/MSP430X_small.xdc

In latest 2.24 product (xdctools_3_24_07_67_eng):

[sanblnx02 748] find . -name "*.xdc" -exec egrep -l '"profile"|coverage' {} /dev/null \;
    ./arm/elf/IArm.xdc
    ./arp32/elf/ARP32.xdc
    ./arp32/elf/ARP32_far.xdc
    ./C28.xdc
    ./C28_float.xdc
    ./C28_large.xdc
    ./C54.xdc
    ./C54_far.xdc
    ./C55.xdc
    ./C55P_word.xdc
    ./C55_huge.xdc
    ./C55_large.xdc
    ./msp430/MSP430.xdc
    ./msp430/MSP430X.xdc
    ./msp430/MSP430X_small.xdc
    ./msp430/elf/MSP430.xdc
    ./msp430/elf/MSP430X.xdc
    ./msp430/elf/MSP430X_small.xdc