This is the bug/patch tracker for Rockbox. Click here for more information.
Quick links: Bugs · Patches · Rockbox frontpage
FS#11454 - Tuning of compiler options for codecs
Attached to Project:
Rockbox
Opened by Nils Wallménius (nls) - Saturday, 03 July 2010, 21:19 GMT+2
Last edited by Nils Wallménius (nls) - Sunday, 18 July 2010, 21:06 GMT+2
Opened by Nils Wallménius (nls) - Saturday, 03 July 2010, 21:19 GMT+2
Last edited by Nils Wallménius (nls) - Sunday, 18 July 2010, 21:06 GMT+2
|
DetailsI'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. 
codecs_tuned_v2.diff
(5.1 KiB)
apps/codecs/libtta/libtta.make | 4 ++-- apps/codecs/libfaad/libfaad.make | 13 +++++++++++++ apps/codecs/libcook/libcook.make | 15 ++++++++++++++- apps/codecs/lib/libcodec.make | 10 +++++++++- apps/codecs/libwma/libwma.make | 13 +++++++++++++ apps/codecs/liba52/liba52.make | 13 +++++++++++++ apps/codecs/libwavpack/libwavpack.make | 4 ++-- apps/codecs/libalac/libalac.make | 7 ++++++- 8 files changed, 72 insertions(+), 7 deletions(-)
codec_flag_test.diff
(9.2 KiB)
apps/codecs/libtta/libtta.make | 4 ++-- apps/codecs/libffmpegFLAC/shndec.c | 2 +- apps/codecs/libffmpegFLAC/libffmpegFLAC.make | 7 +------ apps/codecs/libmad/libmad.make | 7 +------ apps/codecs/libmusepack/libmusepack.make | 7 +------ apps/codecs/libpcm/libpcm.make | 3 +-- apps/codecs/codecs.make | 2 +- apps/codecs/libwavpack/libwavpack.make | 4 ++-- apps/codecs/SOURCES | 2 +- apps/codecs/libalac/libalac.make | 3 +-- apps/codecs/libtremor/libtremor.make | 8 +------- apps/codecs/librm/librm.make | 3 +-- apps/codecs/libm4a/libm4a.make | 3 +-- apps/codecs/demac/libdemac.make | 3 +-- apps/codecs/libspeex/libspeex.make | 9 +-------- apps/codecs/libasap/libasap.make | 3 +-- apps/codecs/libspc/libspc.make | 3 +-- 17 files changed, 19 insertions(+), 54 deletions(-) |
This task depends upon
Closed by Nils Wallménius (nls)
Sunday, 18 July 2010, 21:06 GMT+2
Reason for closing: Accepted
Additional comments about closing: synced and committed except for the aac and tta parts that were already committed
Sunday, 18 July 2010, 21:06 GMT+2
Reason for closing: Accepted
Additional comments about closing: synced and committed except for the aac and tta parts that were already committed
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).
Edit: Also libmad will be slower when using optimization levels > -O1. So, libmad should not be changed (at least for arm) either.
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.