Changeset df908b3 in mainline for uspace/srv/vfs/vfs_lookup.c

Timestamp:

2010-08-27T17:04:29Z (14 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

3249673, ecd2775

Parents:

ddd7118

Message:

Use async relations introduced in the previous changeset to implement
vfs_grab_phone() and vfs_release_phone().

With this change, VFS will not be pointlessly connecting and disconnecting
phones on each VFS request.

Judging from the output of the top utility, this reduces the share of VFS on
kernel time by about 7% (from 25% down to 18%) and DEVFS kernel time by about 5%
(from 28% to 23%). This also makes CONSOLE the biggest consumer of kernel time,
while it used to be third after DEVFS and VFS.

File:

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

uspace/srv/vfs/vfs_lookup.c

@@ -168 +168 @@
         ipcarg_t rc;
         async_wait_for(req, &rc);
-        vfs_release_phone(phone);
+        vfs_release_phone(root->fs_handle, phone);
         
         fibril_mutex_lock(&plb_mutex);
@@ -215 +215 @@
         ipcarg_t rc;
         async_wait_for(req, &rc);
-        vfs_release_phone(phone);
+        vfs_release_phone(result->triplet.fs_handle, phone);
         
         if (rc == EOK) {