Changeset 430de97 in mainline for uspace/srv/fs/fat/fat_idx.c

Timestamp:

2010-01-31T18:07:24Z (14 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

1735f3e, 2e07f62c

Parents:

64bc4b6

Message:

Add fat_node_fini_by_dev_handle() and finish fat_unmounted().

File:

• uspace/srv/fs/fat/fat_idx.c (modified) (5 diffs)

uspace/srv/fs/fat/fat_idx.c

@@ -149 +149 @@
 {
         dev_handle_t dev_handle = (dev_handle_t)key[UPH_DH_KEY];
-        fat_cluster_t pfc = (fat_cluster_t)key[UPH_PFC_KEY];
-        unsigned pdi = (unsigned)key[UPH_PDI_KEY];
+        fat_cluster_t pfc;
+        unsigned pdi;
         fat_idx_t *fidx = list_get_instance(item, fat_idx_t, uph_link);
 
-        return (dev_handle == fidx->dev_handle) && (pfc == fidx->pfc) &&
-            (pdi == fidx->pdi);
+        switch (keys) {
+        case 1:
+                return (dev_handle == fidx->dev_handle);
+        case 3:
+                pfc = (fat_cluster_t) key[UPH_PFC_KEY];
+                pdi = (unsigned) key[UPH_PDI_KEY];
+                return (dev_handle == fidx->dev_handle) && (pfc == fidx->pfc) &&
+                    (pdi == fidx->pdi);
+        default:
+                assert((keys == 1) || (keys == 3));
+        }
 }
 
@@ -197 +206 @@
 {
         dev_handle_t dev_handle = (dev_handle_t)key[UIH_DH_KEY];
-        fs_index_t index = (fs_index_t)key[UIH_INDEX_KEY];
+        fs_index_t index;
         fat_idx_t *fidx = list_get_instance(item, fat_idx_t, uih_link);
 
-        return (dev_handle == fidx->dev_handle) && (index == fidx->index);
+        switch (keys) {
+        case 1:
+                return (dev_handle == fidx->dev_handle);
+        case 2:
+                index = (fs_index_t) key[UIH_INDEX_KEY];
+                return (dev_handle == fidx->dev_handle) &&
+                    (index == fidx->index);
+        default:
+                assert((keys == 1) || (keys == 2));
+        }
 }
 
 static void idx_remove_callback(link_t *item)
 {
-        /* nothing to do */
+        fat_idx_t *fidx = list_get_instance(item, fat_idx_t, uih_link);
+
+        free(fidx);
 }
 
@@ -486 +506 @@
                 [UIH_INDEX_KEY] = idx->index,
         };
+        dev_handle_t dev_handle = idx->dev_handle;
+        fs_index_t index = idx->index;
 
         assert(idx->pfc == FAT_CLST_RES0);
@@ -498 +520 @@
         fibril_mutex_unlock(&used_lock);
         /* Release the VFS index. */
-        fat_index_free(idx->dev_handle, idx->index);
-        /* Deallocate the structure. */
-        free(idx);
+        fat_index_free(dev_handle, index);
+        /* The index structure itself is freed in idx_remove_callback(). */
 }
 
@@ -543 +564 @@
 void fat_idx_fini_by_dev_handle(dev_handle_t dev_handle)
 {
-        unused_t *u;
-
-        u = unused_find(dev_handle, true);
+        unsigned long ikey[] = {
+                [UIH_DH_KEY] = dev_handle
+        };
+        unsigned long pkey[] = {
+                [UPH_DH_KEY] = dev_handle
+        };
+
+        /*
+         * Remove this instance's index structure from up_hash and ui_hash.
+         * Process up_hash first and ui_hash second because the index structure
+         * is actually removed in idx_remove_callback(). 
+         */
+        fibril_mutex_lock(&used_lock);
+        hash_table_remove(&up_hash, pkey, 1);
+        hash_table_remove(&ui_hash, ikey, 1);
+        fibril_mutex_unlock(&used_lock);
+
+        /*
+         * Free the unused and freed structures for this instance.
+         */
+        unused_t *u = unused_find(dev_handle, true);
         assert(u);
         list_remove(&u->link);