[PATCH 4/4] Staging: lustre: modified block comments

[PATCH 4/4] Staging: lustre: modified block comments

[Tanvi Surana]

Fix checpatch.pl errors and warnings in lustre
WARNING: Block comments use * on subsequent lines

Signed-Off-By: Tanvi Surana <tanvisurana20@gmail.com>
---
 drivers/staging/lustre/lustre/llite/xattr.c | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/lustre/lustre/llite/xattr.c b/drivers/staging/lustre/lustre/llite/xattr.c
index 8eb43f1..9483e0e 100644
--- a/drivers/staging/lustre/lustre/llite/xattr.c
+++ b/drivers/staging/lustre/lustre/llite/xattr.c
@@ -239,7 +239,8 @@ int ll_setxattr(struct dentry *dentry, const char *name,
 
 		/* Attributes that are saved via getxattr will always have
 		 * the stripe_offset as 0.  Instead, the MDS should be
-		 * allowed to pick the starting OST index.   b=17846 */
+		 * allowed to pick the starting OST index.   b=17846
+		 */
 		if (lump != NULL && lump->lmm_stripe_offset == 0)
 			lump->lmm_stripe_offset = -1;
 
@@ -482,7 +483,8 @@ ssize_t ll_getxattr(struct dentry *dentry, const char *name,
 
 		if (size == 0 && S_ISDIR(inode->i_mode)) {
 			/* XXX directory EA is fix for now, optimize to save
-			 * RPC transfer */
+			 * RPC transfer
+			 */
 			rc = sizeof(struct lov_user_md);
 			goto out;
 		}
@@ -496,8 +498,9 @@ ssize_t ll_getxattr(struct dentry *dentry, const char *name,
 				rc = -ENODATA;
 			}
 		} else {
-			/* LSM is present already after lookup/getattr call.
-			 * we need to grab layout lock once it is implemented */
+		   /* LSM is present already after lookup/getattr call.
+			* we need to grab layout lock once it is implemented
+			*/
 			rc = obd_packmd(ll_i2dtexp(inode), &lmm, lsm);
 			lmmsize = rc;
 		}
@@ -510,7 +513,8 @@ ssize_t ll_getxattr(struct dentry *dentry, const char *name,
 			/* used to call ll_get_max_mdsize() forward to get
 			 * the maximum buffer size, while some apps (such as
 			 * rsync 3.0.x) care much about the exact xattr value
-			 * size */
+			 * size
+			 */
 			rc = lmmsize;
 			goto out;
 		}
@@ -526,7 +530,8 @@ ssize_t ll_getxattr(struct dentry *dentry, const char *name,
 		memcpy(lump, lmm, lmmsize);
 		/* do not return layout gen for getxattr otherwise it would
 		 * confuse tar --xattr by recognizing layout gen as stripe
-		 * offset when the file is restored. See LU-2809. */
+		 * offset when the file is restored. See LU-2809.
+		 */
 		lump->lmm_layout_gen = 0;
 
 		rc = lmmsize;
-- 
1.9.1

Re: [Outreachy kernel] [PATCH 4/4] Staging: lustre: modified block comments

[Greg KH]

On Sat, Mar 12, 2016 at 01:11:00PM +0530, Tanvi Surana wrote:
> Fix checpatch.pl errors and warnings in lustre
> WARNING: Block comments use * on subsequent lines
> 
> Signed-Off-By: Tanvi Surana <tanvisurana20@gmail.com>
> ---
>  drivers/staging/lustre/lustre/llite/xattr.c | 17 +++++++++++------
>  1 file changed, 11 insertions(+), 6 deletions(-)

Patch does not apply at all :(

Re: [Outreachy kernel] [PATCH 4/4] Staging: lustre: modified block comments

[Tanvi Surana]

Alright.

On Tue, Mar 22, 2016 at 12:54 AM, Greg KH <gregkh@linuxfoundation.org> wrote:
> On Sat, Mar 12, 2016 at 01:11:00PM +0530, Tanvi Surana wrote:
>> Fix checpatch.pl errors and warnings in lustre
>> WARNING: Block comments use * on subsequent lines
>>
>> Signed-Off-By: Tanvi Surana <tanvisurana20@gmail.com>
>> ---
>>  drivers/staging/lustre/lustre/llite/xattr.c | 17 +++++++++++------
>>  1 file changed, 11 insertions(+), 6 deletions(-)
>
> Patch does not apply at all :(