- Status Closed
- Percent Complete
- 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
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…
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
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.
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
new version removes one set of buffer_alloc()’s which is required even if dircache is disabled.
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?
+#define DIRCACHE_STACK_SIZE (DEFAULT_STACK_SIZE + 0×900)/sizeof(long)
you should remove /sizeof(long) if buffer_alloc() takes an argument in bytes
ah yes, thanks, will do that if we decide to commit this