commit 337e922685d11f80fa9e7b164511e043044b6302
author Miika Pekkarinen <miipekk@ihme.org> Fri Jul 01 13:48:28 2011 +0000
committer Miika Pekkarinen <miipekk@ihme.org> Fri Jul 01 13:48:28 2011 +0000
tree 0a728c0476cbd41b91b189fbf995f2914a4bf360
parent 563a185ff4521633e47d598210615c35acc44830

Allocate indices directly in ramcache header. Add version code to
tagcache state dump header. Debug: hdr-structure corruption might
happen after rebuilding of dircache.


git-svn-id: svn://svn.rockbox.org/rockbox/trunk@30109 a1c6a512-1295-4272-9138-f99709370657

apps/tagcache.c

diff --git a/apps/tagcache.c b/apps/tagcache.c
index f242324..089bba1 100644
--- a/apps/tagcache.c
+++ b/apps/tagcache.c
@@ -206,15 +206,16 @@
 #ifdef HAVE_TC_RAMCACHE
 /* Header is created when loading database to ram. */
 struct ramcache_header {
-    struct index_entry *indices; /* Master index file content */
     char *tags[TAG_COUNT];       /* Tag file content (not including filename tag) */
     int entry_count[TAG_COUNT];  /* Number of entries in the indices. */
+    struct index_entry indices[0]; /* Master index file content */
 };
 
 # ifdef HAVE_EEPROM_SETTINGS
 struct statefile_header {
+    int32_t magic;                /* Statefile version number */
     struct master_header mh;      /* Header from the master index */
-    struct ramcache_header *hdr;
+    struct ramcache_header *hdr;  /* Old load address of hdr for relocation */
     struct tagcache_stat tc_stat;
 };
 # endif
@@ -3793,7 +3794,6 @@
     hdr = buffer_alloc(tc_stat.ramcache_allocated + 128);
     memset(hdr, 0, sizeof(struct ramcache_header));
     memcpy(&current_tcmh, &tcmh, sizeof current_tcmh);
-    hdr->indices = (struct index_entry *)(hdr + 1);
     logf("tagcache: %d bytes allocated.", tc_stat.ramcache_allocated);
 
     return true;
@@ -3818,6 +3818,7 @@
     hdr = buffer_alloc(0);
     rc = read(fd, &shdr, sizeof(struct statefile_header));
     if (rc != sizeof(struct statefile_header)
+        || shdr.magic != TAGCACHE_STATEFILE_MAGIC
         || shdr.mh.tch.magic != TAGCACHE_MAGIC)
     {
         logf("incorrect statefile");
@@ -3843,7 +3844,6 @@
     memcpy(&tc_stat, &shdr.tc_stat, sizeof(struct tagcache_stat));
     
     /* Now fix the pointers */
-    hdr->indices = (struct index_entry *)((long)hdr->indices + offpos);
     for (i = 0; i < TAG_COUNT; i++)
         hdr->tags[i] += offpos;
     
@@ -3869,6 +3869,7 @@
     }
     
     /* Create the header */
+    shdr.magic = TAGCACHE_STATEFILE_MAGIC;
     shdr.hdr = hdr;
     memcpy(&shdr.mh, &current_tcmh, sizeof current_tcmh);
     memcpy(&shdr.tc_stat, &tc_stat, sizeof tc_stat);
@@ -3924,6 +3925,8 @@
     /* Load the master index table. */
     for (i = 0; i < tcmh.tch.entry_count; i++)
     {
+        /* DEBUG: After tagcache commit and dircache rebuild, hdr-sturcture
+         * may become corrupt. */
         rc = ecread_index_entry(fd, idx);
         if (rc != sizeof(struct index_entry))
         {
@@ -3997,8 +4000,6 @@
                 int dc;
 # endif
                 
-                // FIXME: This is wrong!
-                // idx = &hdr->indices[hdr->entry_count[i]];
                 idx = &hdr->indices[fe->idx_id];
                 
                 if (fe->tag_length >= (long)sizeof(buf)-1)

apps/tagcache.h

diff --git a/apps/tagcache.h b/apps/tagcache.h
index 6f23bfc..1dad012 100644
--- a/apps/tagcache.h
+++ b/apps/tagcache.h
@@ -52,6 +52,9 @@
 /* Tag Cache Header version 'TCHxx'. Increment when changing internal structures. */
 #define TAGCACHE_MAGIC  0x5443480e
 
+/* Dump store/restore header version 'TCSxx'. */
+#define TAGCACHE_STATEFILE_MAGIC 0x54435301
+
 /* How much to allocate extra space for ramcache. */
 #define TAGCACHE_RESERVE 32768