Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

API improvement to leverage 8 statcoll counters in flexible manner #4

Merged
merged 9 commits into from Oct 10, 2012

Conversation

Projects
None yet
2 participants

fturgis commented Oct 8, 2012

  • API:
    • initiators names now accepted: alldmm, ma_mpu, dss, iva, ...
    • configuration of filtering of transaction and emif is now possible through --t and --p options
    • 8 counters from 0 to 7 can now be used (last 2 can't filter initiator and transaction r/w/r+w, use them for alldmm and ma_mpu)
    • per counter API: --m7 dss --tr7 r+w --q7 emif2 ... including "all counters" API -m dss --tr r+w -q emif2
    • parameters error checking added and detailed configuration description per counter printed at beginning of log
  • Improvements
    • Error checking:
      • tool detects when 1 counter overflowed at 2^32 and suggests user to not trust results
      • tool detects HW bug where next capture is < previous capture. Tool fixes it by setting next capture=previous capture (no info is lost, just accuracy of this capture)
    • Post-processing: each counter can be drawn independently. Aggregates can also be drawn (EMIF1+EMIF2 -> All EMIF, R EMIF1, R EMIF2, W EMIF1, W EMIF2 -> Rd-Wr, All EMIF, ...). User can choose to display only counter graphs, only aggregates or all
    • added missing USB initiators including USB3
    • "-a 1" output log correctly dumps initiator names (and includes correct transaction qualifiers and probes)
  • HW IP reset:
    • HW IP is now reset by default every second (overflow can then happen only if 1 EMIF sustains 4GB/s during 1 second)
    • --overflow_delay option can tune this reset rate
    • -o -t options still available (allowing to reset less IP but with much more complexity), 2 counters can now be monitored with different thresholds
    • removed trace that informs about HW IP reset

Frederic Turgis added some commits Oct 1, 2012

Frederic Turgis [STATCOLL] Missing initiatore, error check for -m option and capabil…
…ity to use initiator name

    - Add missing USB3 and USB_OTG_FS initiatore
    - -m will tell if initiator is not recognized. Initiators are now dumped with their name, not value
    - -m XXX possible with XXX coming from SCI_MSTID_XXX in help
80d83e3
Frederic Turgis rename trans_qaul to trans_qual in cTools lib aa2eacc
Frederic Turgis Add capability to track overflow on 2 thresholds (use twice -o and -t) 0da7bba
Frederic Turgis 6 counters and better description of counters before first data output 2d15e02
Frederic Turgis clarify that option is -m MA_MPU_1_2, not MA_MPU_1_2 alone b3c8351
Frederic Turgis [STATCOLL] per-counter config, error checking, HW bug handling, overf…
…low handling
c9fbe95
Frederic Turgis [STATCOLL] Reset HW IP after a certain number of iterations in -a 1 m…
…ode instead of counter checking

This is much easier than -o -t options. Tool computes number of iterations through
overflow_delay / capture_delay. Default overflow_delay is 1s. Can be changed by --overflow_delay (float in ms)
e24d8d0
Frederic Turgis [STATCOLL] Extend --overflow_delay to -a 2 mode (direct dump on termi…
…nal)

- Add help to the tool

Misc: make timestamp/counter writing in array more generic, in case we want to add
some new control elementsw
b0099e8
Frederic Turgis [STATCOLL] Add --overflow_delay in "trace" module help 13997d3

@omapconf omapconf added a commit that referenced this pull request Oct 10, 2012

@omapconf omapconf Merge pull request #4 from fturgis/tracebw
API improvement to leverage 8 statcoll counters in flexible manner
507d567

@omapconf omapconf merged commit 507d567 into omapconf:master Oct 10, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment