Changeset 79ae36dd in mainline for uspace/srv/vfs/vfs_node.c

Timestamp:

2011-06-08T19:01:55Z (14 years ago)

Author:

Martin Decky <martin@…>

Branches:

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

Children:

0eff68e

Parents:

764d71e

Message:

new async framework with integrated exchange tracking

• strict isolation between low-level IPC and high-level async framework with integrated exchange tracking
  • each IPC connection is represented by an async_sess_t structure
  • each IPC exchange is represented by an async_exch_t structure
  • exchange management is either based on atomic messages (EXCHANGE_ATOMIC), locking (EXCHANGE_SERIALIZE) or connection cloning (EXCHANGE_CLONE)
• async_obsolete: temporary compatibility layer to keep old async clients working (several pieces of code are currently broken, but only non-essential functionality)
• IPC_M_PHONE_HANGUP is now method no. 0 (for elegant boolean evaluation)
• IPC_M_DEBUG_ALL has been renamed to IPC_M_DEBUG
• IPC_M_PING has been removed (VFS protocol now has VFS_IN_PING)
• console routines in libc have been rewritten for better abstraction
• additional use for libc-private header files (FILE structure opaque to the client)
• various cstyle changes (typos, indentation, missing externs in header files, improved comments, etc.)

File:

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

uspace/srv/vfs/vfs_node.c

@@ -104 +104 @@
         bool free_vfs_node = false;
         bool free_fs_node = false;
-
-        fibril_mutex_lock(&nodes_mutex);
+        
+        fibril_mutex_lock(&nodes_mutex);
+        
         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,
@@ -116 +119 @@
                         [KEY_INDEX] = node->index
                 };
+                
                 hash_table_remove(&nodes, key, 3);
                 free_vfs_node = true;
+                
                 if (!node->lnkcnt)
                         free_fs_node = true;
         }
-        fibril_mutex_unlock(&nodes_mutex);
-
+        
+        fibril_mutex_unlock(&nodes_mutex);
+        
         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);
-                sysarg_t rc;
-                rc = async_req_2_0(phone, VFS_OUT_DESTROY,
-                    (sysarg_t)node->devmap_handle, (sysarg_t)node->index);
+                
+                async_exch_t *exch = vfs_exchange_grab(node->fs_handle);
+                sysarg_t rc = async_req_2_0(exch, VFS_OUT_DESTROY,
+                    (sysarg_t) node->devmap_handle, (sysarg_t)node->index);
+                
                 assert(rc == EOK);
-                vfs_release_phone(node->fs_handle, phone);
+                vfs_exchange_release(exch);
         }
+        
         if (free_vfs_node)
                 free(node);