• Status Closed
  • Percent Complete
  • Task Type Patches
  • Category Codecs
  • Assigned To No-one
  • Operating System SW-codec
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by saratoga - 2008-10-18
Last edited by Buschel - 2011-06-08

FS#9498 - AAC IRAM improvements

This patch reorganizes variables by removing a seldom used array while putting a more frequently used one in IRAM.

This gives a ~2% speedup on PP5024, but I would be interested in any Coldfire results, where the difference will hopefully be larger.

Closed by  Buschel
2011-06-08 20:00
Reason for closing:  Out of Date
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

The patch (and even the idea behind) does not apply to the reworked libfaad.

Results on H300 (tested with a VBR AAC track, ~190kbps): SVN 188% realtime, +patch 192% realtime. So about the same speedup as on PP502x.

Same test on iPod G2: SVN 155.5% realtime, +patch 157.0% realtime, i.e. about 1% speedup on PP5002.

In that case I will probably save this extra memory and try to use it for AAC-HE and/or dual core optimizations later on.


Available keyboard shortcuts


Task Details

Task Editing