Changeset f17667a in mainline for uspace/srv/vfs/vfs_node.c

Timestamp:

2008-02-17T13:32:53Z (17 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

2616965d

Parents:

b5553a2

Message:

Add the VFS_FREE operation. This operation frees up whatever resources used by
a file system node for which there is no name (i.e. an unlinked node).

Cleanup VFS operations enums and remove unneeded VFS operations.

File:

• uspace/srv/vfs/vfs_node.c (modified) (4 diffs)

uspace/srv/vfs/vfs_node.c

@@ -44 +44 @@
 #include <libadt/hash_table.h>
 #include <assert.h>
+#include <async.h>
+#include <errno.h>
 
 /** Futex protecting the VFS node hash table. */
@@ -102 +104 @@
 void vfs_node_delref(vfs_node_t *node)
 {
+        bool free_vfs_node = false;
+        bool free_fs_node = false;
+
         futex_down(&nodes_futex);
         if (node->refcnt-- == 1) {
+                /*
+                 * We are dropping the last reference to this node.
+                 * Remove it from the VFS node hash table.
+                 */
                 unsigned long key[] = {
                         [KEY_FS_HANDLE] = node->fs_handle,
@@ -110 +119 @@
                 };
                 hash_table_remove(&nodes, key, 3);
+                free_vfs_node = true;
+                if (!node->lnkcnt)
+                        free_fs_node = true;
         }
         futex_up(&nodes_futex);
+
+        if (free_fs_node) {
+                /* 
+                 * The node is not visible in the file system namespace.
+                 * Free up its resources.
+                 */
+                int phone = vfs_grab_phone(node->fs_handle);
+                ipcarg_t rc;
+                rc = async_req_2_0(phone, VFS_FREE, (ipcarg_t)node->dev_handle,
+                    (ipcarg_t)node->index);
+                assert(rc == EOK);
+                vfs_release_phone(phone);
+        }
+        if (free_vfs_node)
+                free(node);
 }
 
@@ -197 +224 @@
 void nodes_remove_callback(link_t *item)
 {
-        vfs_node_t *node = hash_table_get_instance(item, vfs_node_t, nh_link);
-        free(node);
 }