Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Operating System/Drivers
  • Assigned To No-one
  • Operating System All players
  • 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 jdgordon - 2008-10-27
Last edited by pamaury - 2010-02-18

FS#9506 - buffer_alloc the dircache stack

seen as there is no point having the dircache thread if its disabled, and because it cant be enabled without rebooting, there is no need having the stack statically allocated…

untested on target…

   dircache_alloc.diff (2.1 KiB)
Closed by  pamaury
2010-02-18 23:16
Reason for closing:  Rejected
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

Recents commits r24657 and r24708 already saved quite some space and reduced the size of the dircache thread stack.

jdgordon commented on 2008-10-28 13:51

new version removes one set of buffer_alloc()’s which is required even if dircache is disabled.

   dircache_alloc.diff (2.6 KiB)
jdgordon commented on 2008-10-28 14:12

OK, So I’m pretty sure I’m only looking at this because i should be studying for exams so I dont really know what to do about this patch… if its about freeing up more RAM when its disabled then there is another 30KB buffer (dir_recursion, line 157) which could be buffer_alloc()-ed which is a hell of a lot more than the ~800 bytes + 1 thread saved by the patch in the previous comment… but if the plan is to (hopefully) make buffer_alloc() be able to tell everything that called it to dump the ram and realloc it later, then I dont know if changing that buffer is helpful in the long run… will freeing up 31K do anything more than give a nice ram delta?

funman commented on 2008-11-03 05:42

+#define DIRCACHE_STACK_SIZE (DEFAULT_STACK_SIZE + 0×900)/sizeof(long)

you should remove /sizeof(long) if buffer_alloc() takes an argument in bytes

jdgordon commented on 2008-11-03 07:48

ah yes, thanks, will do that if we decide to commit this

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing