<div dir="ltr">That&#39;s a task (of many) I&#39;ve been putting on the back burner until the code is cleaner.  It&#39;s also a HUGE change, since there are debug macros everywhere, and they all check a #define&#39;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 &lt;<a href="mailto:julia.lawall@lip6.fr">julia.lawall@lip6.fr</a>&gt; wrote:<br>&gt;<br>&gt;<br>&gt;<br>&gt; On Thu, 21 May 2015, Joe Perches wrote:<br>&gt;<br>&gt; &gt; On Thu, 2015-05-21 at 15:50 -0400, Mike Shuey wrote:<br>&gt; &gt; &gt; Fix many <a href="http://checkpatch.pl">checkpatch.pl</a> warnings.<br>&gt; &gt; []<br>&gt; &gt; &gt; diff --git a/drivers/staging/lustre/lnet/lnet/acceptor.c b/drivers/staging/lustre/lnet/lnet/acceptor.c<br>&gt; &gt; []<br>&gt; &gt; &gt; @@ -99,38 +99,42 @@ lnet_connect_console_error(int rc, lnet_nid_t peer_nid,<br>&gt; &gt; &gt;     switch (rc) {<br>&gt; &gt; &gt;     /* &quot;normal&quot; errors */<br>&gt; &gt; &gt;     case -ECONNREFUSED:<br>&gt; &gt; &gt; -           CNETERR(&quot;Connection to %s at host %pI4h on port %d was refused: check that Lustre is running on that node.\n&quot;,<br>&gt; &gt; &gt; -                   libcfs_nid2str(peer_nid),<br>&gt; &gt; &gt; -                   &amp;peer_ip, peer_port);<br>&gt; &gt; &gt; +           CNETERR(<br>&gt; &gt; &gt; +                   &quot;Connection to %s at host %pI4h on port %d was refused: check that Lustre is running on that node.\n&quot;,<br>&gt; &gt; &gt; +                   libcfs_nid2str(peer_nid), &amp;peer_ip, peer_port);<br>&gt; &gt;<br>&gt; &gt; These are not improvements and checkpatch messages aren&#39;t dicta.<br>&gt; &gt;<br>&gt; &gt; Please don&#39;t convert code unless the conversion makes it better<br>&gt; &gt; for a human reader.<br>&gt; &gt;<br>&gt; &gt; These don&#39;t.<br>&gt;<br>&gt; I haven&#39;t looked into it, but perhaps there is a standard kernel printing<br>&gt; function that these could be converted to directly?<br>&gt;<br>&gt; julia<br></div>