<div dir="ltr">That's a task (of many) I've been putting on the back burner until the code is cleaner.  It's also a HUGE change, since there are debug macros everywhere, and they all check a #define'd mask to see if they should fire, and the behavior is likely governed by parts of the lustre user land tools as well.<br><br>Suggestions are welcome.  Do other parts of the linux kernel define complex debugging macros like these, or is this a lustre-ism?  Any suggestions on how to handle this more in line with existing drivers?<br><br>--<br>Mike Shuey<br><br>On Thu, May 21, 2015 at 5:29 PM, Julia Lawall <<a href="mailto:julia.lawall@lip6.fr">julia.lawall@lip6.fr</a>> wrote:<br>><br>><br>><br>> On Thu, 21 May 2015, Joe Perches wrote:<br>><br>> > On Thu, 2015-05-21 at 15:50 -0400, Mike Shuey wrote:<br>> > > Fix many <a href="http://checkpatch.pl">checkpatch.pl</a> warnings.<br>> > []<br>> > > diff --git a/drivers/staging/lustre/lnet/lnet/acceptor.c b/drivers/staging/lustre/lnet/lnet/acceptor.c<br>> > []<br>> > > @@ -99,38 +99,42 @@ lnet_connect_console_error(int rc, lnet_nid_t peer_nid,<br>> > >     switch (rc) {<br>> > >     /* "normal" errors */<br>> > >     case -ECONNREFUSED:<br>> > > -           CNETERR("Connection to %s at host %pI4h on port %d was refused: check that Lustre is running on that node.\n",<br>> > > -                   libcfs_nid2str(peer_nid),<br>> > > -                   &peer_ip, peer_port);<br>> > > +           CNETERR(<br>> > > +                   "Connection to %s at host %pI4h on port %d was refused: check that Lustre is running on that node.\n",<br>> > > +                   libcfs_nid2str(peer_nid), &peer_ip, peer_port);<br>> ><br>> > These are not improvements and checkpatch messages aren't dicta.<br>> ><br>> > Please don't convert code unless the conversion makes it better<br>> > for a human reader.<br>> ><br>> > These don't.<br>><br>> I haven't looked into it, but perhaps there is a standard kernel printing<br>> function that these could be converted to directly?<br>><br>> julia<br></div>