Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#11253 - Add IRAM and ARM asm to test_mem plugin

Attached to Project: Rockbox
Opened by Andree Buschmann (Buschel) - Saturday, 08 May 2010, 20:45 GMT+2
Last edited by Andree Buschmann (Buschel) - Sunday, 09 May 2010, 21:23 GMT+2
Task Type Patches
Category Plugins
Status Closed
Assigned To Andree Buschmann (Buschel)
Player Type All players
Severity Low
Priority Normal
Reported Version Release 3.4
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

This patch adds two things to the test_mem plugin:
1) Refactor the tests in a way to allow the test of IRAM and DRAM.
2) Add ARM asm (ldmia/stmia).
   mem_01.patch (5.5 KiB) 
 apps/plugins/test_mem.c |  147 +++++++++++++++++++++++++++++++++++-------------
 1 file changed, 110 insertions(+), 37 deletions(-)

This task depends upon

Closed by  Andree Buschmann (Buschel)
Sunday, 09 May 2010, 21:23 GMT+2
Reason for closing:  Accepted
Additional comments about closing:  Submitted with r25918.
Comment by Andree Buschmann (Buschel) - Saturday, 08 May 2010, 21:13 GMT+2
Use PLUGIN_USE_IRAM.
   mem_02.patch (5.6 KiB) 
 apps/plugins/test_mem.c |  147 +++++++++++++++++++++++++++++++++++-------------
 1 file changed, 110 insertions(+), 37 deletions(-)

Comment by Nils Wallménius (nls) - Sunday, 09 May 2010, 09:35 GMT+2
It seems you have swapped the asm for the read and write tests so read is storing with stm and write is loading with ldm...
Comment by Andree Buschmann (Buschel) - Sunday, 09 May 2010, 11:13 GMT+2
Good point :o)
   mem_03.patch (5.6 KiB) 
 apps/plugins/test_mem.c |  147 +++++++++++++++++++++++++++++++++++-------------
 1 file changed, 110 insertions(+), 37 deletions(-)

Comment by Andree Buschmann (Buschel) - Sunday, 09 May 2010, 20:00 GMT+2
- display current clock frequency in Hz (as the clock needs some time to switch after a change of boost/unboosted is signalled)
- display results in "x.y MB/s"
- loop count depends on boost state (trade off between test loop duration and precision)
   mem_04.patch (6.1 KiB) 
 apps/plugins/test_mem.c |  158 ++++++++++++++++++++++++++++++++++++------------
 1 file changed, 119 insertions(+), 39 deletions(-)

Comment by Andree Buschmann (Buschel) - Sunday, 09 May 2010, 21:06 GMT+2
Roll back clock-dependent loop count for better readability.
   mem_05.patch (6.4 KiB) 
 apps/plugins/test_mem.c |  161 ++++++++++++++++++++++++++++++++++++------------
 1 file changed, 121 insertions(+), 40 deletions(-)

Loading...