[184/622] lustre: llite: Fix style issues for symlink.c

Commit Message

James Simmons Feb. 27, 2020, 9:10 p.m. UTC

From: Arshad Hussain <[e-mail]>

This patch fixes issues reported by checkpatch
for file fs/lustre/llite/symlink.c

WC-bug-id: https://jira.whamcloud.com/browse/LU-6142
Lustre-commit: e486703b5278 ("LU-6142 llite: Fix style issues for symlink.c")
Signed-off-by: Arshad Hussain <[e-mail]>
Reviewed-on: https://review.whamcloud.com/33811
Reviewed-by: Andreas Dilger <[e-mail]>
Reviewed-by: Ben Evans <[e-mail]>
Signed-off-by: James Simmons <[e-mail]>
---
 fs/lustre/llite/symlink.c | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

Patch

diff --git a/fs/lustre/llite/symlink.c b/fs/lustre/llite/symlink.c
index 0690fdb..d2922d1 100644
--- a/fs/lustre/llite/symlink.c
+++ b/fs/lustre/llite/symlink.c
@@ -53,7 +53,8 @@  static int ll_readlink_internal(struct inode *inode,
 		int print_limit = min_t(int, PAGE_SIZE - 128, symlen);
 
 		*symname = lli->lli_symlink_name;
-		/* If the total CDEBUG() size is larger than a page, it
+		/*
+		 * If the total CDEBUG() size is larger than a page, it
 		 * will print a warning to the console, avoid this by
 		 * printing just the last part of the symlink.
 		 */
@@ -97,11 +98,11 @@  static int ll_readlink_internal(struct inode *inode,
 	}
 
 	*symname = req_capsule_server_get(&(*request)->rq_pill, &RMF_MDT_MD);
-	if (!*symname ||
-	    strnlen(*symname, symlen) != symlen - 1) {
+	if (!*symname || strnlen(*symname, symlen) != symlen - 1) {
 		/* not full/NULL terminated */
-		CERROR("inode %lu: symlink not NULL terminated string of length %d\n",
-		       inode->i_ino, symlen - 1);
+		CERROR("%s: inode " DFID ": symlink not NULL terminated string of length %d\n",
+		       ll_get_fsname(inode->i_sb, NULL, 0),
+		       PFID(ll_inode2fid(inode)), symlen - 1);
 		rc = -EPROTO;
 		goto failed;
 	}
@@ -143,7 +144,8 @@  static const char *ll_get_link(struct dentry *dentry,
 		return ERR_PTR(rc);
 	}
 
-	/* symname may contain a pointer to the request message buffer,
+	/*
+	 * symname may contain a pointer to the request message buffer,
 	 * we delay request releasing then.
 	 */
 	set_delayed_call(done, ll_put_link, request);