afs: Add more tracepoints

Add four more tracepoints:

 (1) afs_make_fs_call1 - Split from afs_make_fs_call but takes a filename
     to log also.

 (2) afs_make_fs_call2 - Like the above but takes two filenames to log.

 (3) afs_lookup - Log the result of doing a successful lookup, including a
     negative result (fid 0:0).

 (4) afs_get_tree - Log the set up of a volume for mounting.

It also extends the name buffer on the afs_edit_dir tracepoint to 24 chars
and puts quotes around the filename in the text representation.

Signed-off-by: David Howells <dhowells@redhat.com>

4 files changed, 20 insertions, 13 deletions

diff --git a/fs/afs/dir.c b/fs/afs/dir.c
index 6c8523501639..48706eefc63b 100644
--- a/fs/afs/dir.c
+++ b/fs/afs/dir.c
@@ -908,8 +908,14 @@ static struct dentry *afs_lookup(struct inode *dir, struct dentry *dentry,
                         (void *)(unsigned long)dvnode->status.data_version;
         }
         d = d_splice_alias(inode, dentry);
-        if (!IS_ERR_OR_NULL(d))
+        if (!IS_ERR_OR_NULL(d)) {
                 d->d_fsdata = dentry->d_fsdata;
+                trace_afs_lookup(dvnode, &d->d_name,
+                                 inode ? AFS_FS_I(inode) : NULL);
+        } else {
+                trace_afs_lookup(dvnode, &dentry->d_name,
+                                 inode ? AFS_FS_I(inode) : NULL);
+        }
         return d;
 }
 
diff --git a/fs/afs/fsclient.c b/fs/afs/fsclient.c
index 9d405f96cc91..be4520eb4965 100644
--- a/fs/afs/fsclient.c
+++ b/fs/afs/fsclient.c
@@ -830,7 +830,7 @@ int afs_fs_create(struct afs_fs_cursor *fc,
         *bp++ = 0; /* segment size */
 
         afs_use_fs_server(call, fc->cbi);
-        trace_afs_make_fs_call(call, &vnode->fid);
+        trace_afs_make_fs_call1(call, &vnode->fid, name);
         afs_make_call(&fc->ac, call, GFP_NOFS);
         return afs_wait_for_call_to_complete(call, &fc->ac);
 }
@@ -926,7 +926,7 @@ int afs_fs_remove(struct afs_fs_cursor *fc, struct afs_vnode *vnode,
         }
 
         afs_use_fs_server(call, fc->cbi);
-        trace_afs_make_fs_call(call, &dvnode->fid);
+        trace_afs_make_fs_call1(call, &dvnode->fid, name);
         afs_make_call(&fc->ac, call, GFP_NOFS);
         return afs_wait_for_call_to_complete(call, &fc->ac);
 }
@@ -1019,7 +1019,7 @@ int afs_fs_link(struct afs_fs_cursor *fc, struct afs_vnode *vnode,
         *bp++ = htonl(vnode->fid.unique);
 
         afs_use_fs_server(call, fc->cbi);
-        trace_afs_make_fs_call(call, &vnode->fid);
+        trace_afs_make_fs_call1(call, &vnode->fid, name);
         afs_make_call(&fc->ac, call, GFP_NOFS);
         return afs_wait_for_call_to_complete(call, &fc->ac);
 }
@@ -1134,7 +1134,7 @@ int afs_fs_symlink(struct afs_fs_cursor *fc,
         *bp++ = 0; /* segment size */
 
         afs_use_fs_server(call, fc->cbi);
-        trace_afs_make_fs_call(call, &vnode->fid);
+        trace_afs_make_fs_call1(call, &vnode->fid, name);
         afs_make_call(&fc->ac, call, GFP_NOFS);
         return afs_wait_for_call_to_complete(call, &fc->ac);
 }
@@ -1253,7 +1253,7 @@ int afs_fs_rename(struct afs_fs_cursor *fc,
         }
 
         afs_use_fs_server(call, fc->cbi);
-        trace_afs_make_fs_call(call, &orig_dvnode->fid);
+        trace_afs_make_fs_call2(call, &orig_dvnode->fid, orig_name, new_name);
         afs_make_call(&fc->ac, call, GFP_NOFS);
         return afs_wait_for_call_to_complete(call, &fc->ac);
 }
diff --git a/fs/afs/super.c b/fs/afs/super.c
index 6438849a75c4..ce85ae61f12d 100644
--- a/fs/afs/super.c
+++ b/fs/afs/super.c
@@ -550,6 +550,7 @@ static int afs_get_tree(struct fs_context *fc)
         }
 
         fc->root = dget(sb->s_root);
+        trace_afs_get_tree(as->cell, as->volume);
         _leave(" = 0 [%p]", sb);
         return 0;
 
diff --git a/fs/afs/yfsclient.c b/fs/afs/yfsclient.c
index 73546c2d89ee..5ea0350dc9dd 100644
--- a/fs/afs/yfsclient.c
+++ b/fs/afs/yfsclient.c
@@ -809,7 +809,7 @@ int yfs_fs_create_file(struct afs_fs_cursor *fc,
         yfs_check_req(call, bp);
 
         afs_use_fs_server(call, fc->cbi);
-        trace_afs_make_fs_call(call, &vnode->fid);
+        trace_afs_make_fs_call1(call, &vnode->fid, name);
         afs_make_call(&fc->ac, call, GFP_NOFS);
         return afs_wait_for_call_to_complete(call, &fc->ac);
 }
@@ -873,7 +873,7 @@ int yfs_fs_make_dir(struct afs_fs_cursor *fc,
         yfs_check_req(call, bp);
 
         afs_use_fs_server(call, fc->cbi);
-        trace_afs_make_fs_call(call, &vnode->fid);
+        trace_afs_make_fs_call1(call, &vnode->fid, name);
         afs_make_call(&fc->ac, call, GFP_NOFS);
         return afs_wait_for_call_to_complete(call, &fc->ac);
 }
@@ -964,7 +964,7 @@ int yfs_fs_remove_file2(struct afs_fs_cursor *fc, struct afs_vnode *vnode,
         yfs_check_req(call, bp);
 
         afs_use_fs_server(call, fc->cbi);
-        trace_afs_make_fs_call(call, &dvnode->fid);
+        trace_afs_make_fs_call1(call, &dvnode->fid, name);
         afs_make_call(&fc->ac, call, GFP_NOFS);
         return afs_wait_for_call_to_complete(call, &fc->ac);
 }
@@ -1052,7 +1052,7 @@ int yfs_fs_remove(struct afs_fs_cursor *fc, struct afs_vnode *vnode,
         yfs_check_req(call, bp);
 
         afs_use_fs_server(call, fc->cbi);
-        trace_afs_make_fs_call(call, &dvnode->fid);
+        trace_afs_make_fs_call1(call, &dvnode->fid, name);
         afs_make_call(&fc->ac, call, GFP_NOFS);
         return afs_wait_for_call_to_complete(call, &fc->ac);
 }
@@ -1138,7 +1138,7 @@ int yfs_fs_link(struct afs_fs_cursor *fc, struct afs_vnode *vnode,
         yfs_check_req(call, bp);
 
         afs_use_fs_server(call, fc->cbi);
-        trace_afs_make_fs_call(call, &vnode->fid);
+        trace_afs_make_fs_call1(call, &vnode->fid, name);
         afs_make_call(&fc->ac, call, GFP_NOFS);
         return afs_wait_for_call_to_complete(call, &fc->ac);
 }
@@ -1235,7 +1235,7 @@ int yfs_fs_symlink(struct afs_fs_cursor *fc,
         yfs_check_req(call, bp);
 
         afs_use_fs_server(call, fc->cbi);
-        trace_afs_make_fs_call(call, &dvnode->fid);
+        trace_afs_make_fs_call1(call, &dvnode->fid, name);
         afs_make_call(&fc->ac, call, GFP_NOFS);
         return afs_wait_for_call_to_complete(call, &fc->ac);
 }
@@ -1334,7 +1334,7 @@ int yfs_fs_rename(struct afs_fs_cursor *fc,
         yfs_check_req(call, bp);
 
         afs_use_fs_server(call, fc->cbi);
-        trace_afs_make_fs_call(call, &orig_dvnode->fid);
+        trace_afs_make_fs_call2(call, &orig_dvnode->fid, orig_name, new_name);
         afs_make_call(&fc->ac, call, GFP_NOFS);
         return afs_wait_for_call_to_complete(call, &fc->ac);
 }