Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Operating System/Drivers
  • Assigned To No-one
  • Operating System PortalPlayer-based
  • 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 Buschel - 2008-05-12
Last edited by Buschel - 2008-05-14

FS#8987 - use 128KB IRAM for PP5022/PP5024

This patch enabled the additional 32KB IRAM for PP5022 and PP5024 processors. This is kind of a first approach and uses 64KB for core and 64KB for plugin/codec.

An additional patch for the musepack decoder takes advantage of this additional IRAM and is sped up by ~6.6%.

EDIT_1: Exchanged the iram-patch (removed the mpc-related changes).
EDIT_2: Exchanged the mpc-patch (removed iram-unrelated changed).

Closed by  Buschel
2008-05-14 20:30
Reason for closing:  Accepted
Additional comments about closing:  

Committed with r17506 and 17507.

I would really prefer to give the entire extra 32KB of IRAM to codecs. They can use it a lot more then core.

saratoga: v2 give 80KB of IRAM to plugin/codec, core's IRAM is untouched.

I tried the patch on X5 (Coldfire MCF5250, also having 128KB or IRAM). It sped up decoding of my test track from 482% realtime to 615% realtime (~27% speedup)!

I've attached the modified patch, which also corrects the checks for PP5022 and PP5024. Those macros don't work like the CPU_* group macros. They are always defined, which would cause v1 to break on any other CPU than PP5022/PP5024. You're supposed to check CONFIG_CPU.

Regarding IRAM distribution - I'd prefer to keep even distribution between core and codecs/plugins, like it's already done in SVN for MCF5250. While IRAM might be quite useful for codecs, it might also be quite useful in the core (e.g. for better DSP code), and there are often also some target specific tasks in the core which profit from extra IRAM.

I don't see any point in 64/64k on PP. We're only using about 2/3 of the available IRAM as it is, and theres no indication that theres some compelling feature coming that would require as much or more IRAM as we currently use for the entire core. 48K is arguably too much already, and should be reduced in order to save battery life decoding . . .

I won't argue too much for Coldfire though, since I don't have any of those targets.

iram_v02 patch submitted with r17506, mpc_v02-patch submitted with r17507.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing