Re: [PATCH 3/4] Staging: Lustre Fixing multiline block comments in lnetst.h

From: Greg Kroah-Hartman
Date: Tue Jul 11 2017 - 13:15:08 EST


On Fri, Jul 07, 2017 at 01:47:04AM +0000, Craig Inches wrote:
> This fixes multiple block statements found not to match
> style as per checkpatch
>
> Signed-off-by: Craig Inches <Craig@xxxxxxxxxxxxxxx>
> ---
> drivers/staging/lustre/include/linux/lnet/lnetst.h | 129 +++++++++++++--------
> 1 file changed, 81 insertions(+), 48 deletions(-)
>
> diff --git a/drivers/staging/lustre/include/linux/lnet/lnetst.h b/drivers/staging/lustre/include/linux/lnet/lnetst.h
> index ea736f8d5231..a4f9ff01d458 100644
> --- a/drivers/staging/lustre/include/linux/lnet/lnetst.h
> +++ b/drivers/staging/lustre/include/linux/lnet/lnetst.h
> @@ -54,7 +54,8 @@
> #define LSTIO_GROUP_ADD 0xC10 /* add group */
> #define LSTIO_GROUP_LIST 0xC11 /* list all groups in session */
> #define LSTIO_GROUP_INFO 0xC12 /* query default information of
> - * specified group */
> + * specified group
> + */
> #define LSTIO_GROUP_DEL 0xC13 /* delete group */
> #define LSTIO_NODES_ADD 0xC14 /* add nodes to specified group */
> #define LSTIO_GROUP_UPDATE 0xC15 /* update group */
> @@ -102,27 +103,32 @@ struct lstcon_test_ent {
> int tse_type; /* test type */
> int tse_loop; /* loop count */
> int tse_concur; /* concurrency of test */
> -}; /*** test summary entry, for
> - *** list_batch command */
> +}; /* test summary entry, for
> + * list_batch command
> + */

That's odd, what was the *** stuff for?

I'd like to get a lustre maintainer's ack for all of these before I
apply them...

thanks,

greg k-h