Opened 3 years ago

Last modified 3 months ago

#5641 new bug

The -L flag should not exist

Reported by: augustss Owned by:
Priority: normal Milestone: 7.12.1
Component: Profiling Version: 7.2.1
Keywords: Cc: dterei, mail@…
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: None/Unknown Test Case:
Blocked By: #3024 Blocking:
Related Tickets: Differential Revisions:

Description

Why does the -L flag exist? The .hp file should contain untruncated strings, and any truncation of the names should be done in hp2ps.

Change History (10)

comment:1 Changed 3 years ago by igloo

  • Milestone set to 7.6.1

comment:2 Changed 3 years ago by dterei

  • Cc dterei added

comment:3 Changed 3 years ago by igloo

  • Milestone changed from 7.6.1 to 7.6.2

comment:4 Changed 2 years ago by nomeata

  • Cc mail@… added

Does having a milestone imply that this is a desired feature and just waits for implementation? Might be some low hanging fruit for a bypassing contributor.

comment:5 Changed 2 years ago by simonmar

  • difficulty set to Unknown

It means we think the suggestion is sensible, but with a priority of "normal" it isn't likely to get done by us (we only tend to get to the "high" stuff).

Regarding this ticket specifically, hp2ps needs to be thrown away and rewritten in Haskell. Last I heard hp2pretty wasn't quite there yet.

comment:6 Changed 2 years ago by igloo

  • Blocked By 3024 added

comment:7 Changed 2 years ago by igloo

  • Blocked By

comment:8 Changed 9 months ago by thoughtpolice

  • Milestone changed from 7.6.2 to 7.10.1

Moving to 7.10.1.

comment:9 Changed 4 months ago by thomie

  • Blocked By

For reference, in commit 16871485a44ba8a6e93e40f30af7ea46839e0c4c:

Author: Ravi Nanavati <>
Date:   Fri Sep 29 22:51:15 2006 +0000

    rts_ccs_length
    
    Add the -L RTS flag to control the length of the cost-centre stacks reported in
    a heap profile.

And in rts/RtsFlags.c:

 "  -L<chars>      Maximum length of a cost-centre stack in a heap profile",
 "                 (default: 25)",

Undocumented as far as I can tell in the user's guide.

Last edited 4 months ago by thomie (previous) (diff)

comment:10 Changed 3 months ago by thoughtpolice

  • Milestone changed from 7.10.1 to 7.12.1

Moving to 7.12.1 milestone; if you feel this is an error and should be addressed sooner, please move it back to the 7.10.1 milestone.

Note: See TracTickets for help on using tickets.