- Status Closed
- Percent Complete
- Task Type Patches
- Category Codecs
- Assigned To No-one
- Operating System SW-codec
- Severity Low
- Priority Very Low
- Reported Version Release 3.6
- Due in Version Next release
-
Due Date
Undecided
- Votes
- Private
FS#11454 - Tuning of compiler options for codecs
I’ve run test_codec runs of all the files in test_files (except the ape files) on both my h300 and c200 and come up with this tuning.
speedups are as follows
h300:
cook 8-14%
faad 2-6%
tta 1%
wma 7-14%
tremor ~0.5%
c200:
alac 1%
faad 2-3%
tta 6%
wma 2%
wv 2%
It would be nice if others could run the same type of benchmark on other cpu’s, i’m not sure all arms behave the same so maybe we need more fine grained control.
To do so, just apply the codec_flag_test.diff patch below and edit the CODECFLAGS line in apps/codecs/codecs.make to compile all the codecs with the specified compiler flags.
Note that the flag_test patch does not affect the codeclib since testing that separately for optimal compilation flags might be necessary, it was on h300 at least.
2010-07-18 19:06
Reason for closing: Accepted
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
synced and committed except for the aac and tta parts that were already committed
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
The following error occurs when iPod video's Rockbox builds on r27272 + codec_flag_test.diff.
ld: /home/yoshihisa/rockbox/build/apps/codecs/mpc.elf section `.ibss'
will not fit in region `PLUGIN_IRAM'
ld: region `PLUGIN_IRAM' overflowed by 5808 bytes
collect2: ld returned 1 exit status
On r27272 + codecs_tuned_v2.diff, iPod video's Rockbox builds success.
tta codec : decoding speed ~+3 % (174.73 % → 177.73%). (other codecs does not check).
musepack (mpc) is fully optimized with all the "-Ox". We should not change this.
Edit: Also libmad will be slower when using optimization levels > -O1. So, libmad should not be changed (at least for arm) either.
Buschel, that's what i found too so the patch doesn't change them… The test patch is just wat i used for testing if others wanted to try on targets with cpu's i don't have.
The test patch currently sets O3 and that can easily be changed in codecs.mke if you want to test.