Re: [lustre-devel] [PATCH 07/10] staging: lustre: cleanup comment style for lnet selftest

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Mar 11, 2016 at 10:24:00PM -0800, Greg Kroah-Hartman wrote:
> On Sat, Mar 12, 2016 at 01:39:01AM +0000, Dilger, Andreas wrote:
> > On 2016/03/11, 18:29, "lustre-devel on behalf of James Simmons"
> > <lustre-devel-bounces@xxxxxxxxxxxxxxxx on behalf of
> > jsimmons@xxxxxxxxxxxxx> wrote:
> > 
> > >Apply a consistent style for comments in the lnet selftest
> > >code.
> > >
> > >Signed-off-by: James Simmons <jsimmons@xxxxxxxxxxxxx>
> > >---
> > > drivers/staging/lustre/lnet/selftest/brw_test.c  |    8 ++--
> > > drivers/staging/lustre/lnet/selftest/conctl.c    |   50
> > >+++++++++++-----------
> > > drivers/staging/lustre/lnet/selftest/conrpc.c    |   23 +++++-----
> > > drivers/staging/lustre/lnet/selftest/console.c   |   11 +++--
> > > drivers/staging/lustre/lnet/selftest/framework.c |   20 ++++----
> > > drivers/staging/lustre/lnet/selftest/ping_test.c |    2 +-
> > > drivers/staging/lustre/lnet/selftest/rpc.c       |   46
> > >++++++++++----------
> > > drivers/staging/lustre/lnet/selftest/rpc.h       |    2 +-
> > > drivers/staging/lustre/lnet/selftest/selftest.h  |    3 +-
> > > drivers/staging/lustre/lnet/selftest/timer.c     |    6 +-
> > > 10 files changed, 87 insertions(+), 84 deletions(-)
> > >
> > >diff --git a/drivers/staging/lustre/lnet/selftest/brw_test.c
> > >b/drivers/staging/lustre/lnet/selftest/brw_test.c
> > >index eebc924..6ac4d02 100644
> > >--- a/drivers/staging/lustre/lnet/selftest/brw_test.c
> > >+++ b/drivers/staging/lustre/lnet/selftest/brw_test.c
> > >@@ -86,7 +86,7 @@ brw_client_init(sfw_test_instance_t *tsi)
> > > 		opc = breq->blk_opc;
> > > 		flags = breq->blk_flags;
> > > 		npg = breq->blk_npg;
> > >-		/*
> > >+		/**
> > > 		 * NB: this is not going to work for variable page size,
> > > 		 * but we have to keep it for compatibility
> > > 		 */
> > 
> > The "/**" comment opener is only for header comment blocks that
> > have markup in them.  I don't think that is kernel style for
> > normal multi-line comments in the code.
> 
> Yes, that is correct.  James, can you fix this up and resend this
> series?

Sorry, I meant the series from this patch onward.  I've applied the
first 6.
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux