Rockbox

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

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#8423 - Fixes two database problem

Attached to Project: Rockbox
Opened by Lee Kang Hyuk (alwaysbluepop) - Monday, 07 January 2008, 15:36 GMT+2
Last edited by Miika Pekkarinen (miipekk) - Monday, 07 January 2008, 16:10 GMT+2
Task Type Patches
Category Database
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

I've noticed that tag cache system does not work properly on a certain situation, since r15955. (X5 player and sim, 'load to ram','auto update', and 'dircache' enabled)

1. deleted file's tag entry continues to show up even when that tags are no longer in use in database:
deeply looked into logf, and diff between 15954 and 15955, I finally concluded that necessary codes are removed by accident.
FLAG_DELETED is not saved to masterfd, so the value of in_use[tag] is always minimum 1, not 0.
This situation may causes FS8414(duplicated entries).

2. Whenever I modified a metadata(and do not rename it), Rockbox results in crash(FS8384):
due to newly added 'Resurrection' code, we have to refference old tag_seek, i guess.

I'm not sure if it's right solution, but worked as expected by now, including 'Preserve Runtime Statistics'.
   tagcache_fix.patch (1.3 KiB)
 apps/tagcache.c |   17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

This task depends upon

Closed by  Miika Pekkarinen (miipekk)
Monday, 07 January 2008, 16:10 GMT+2
Reason for closing:  Accepted
Comment by Miika Pekkarinen (miipekk) - Monday, 07 January 2008, 16:03 GMT+2
Thanks, this looks great. Going to commit now.

Loading...