LU-2321 debug: Quiet uninitialized target connects

When a target is not fully initialized and receives a connection
attempt, it will refuse the connection and print a message to the
console stating this fact. This is "normal operation", so rather
than using LCONSOLE_WARN to convey this information, this patch
modifies it to use CDEBUG.

Signed-off-by: Prakash Surya <surya1@llnl.gov>
Change-Id: Ia5d098fcfe2ad9ae2a1406166087db8181496a7d
Reviewed-on: http://review.whamcloud.com/4640
Tested-by: Hudson
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>

diff --git a/lustre/ldlm/ldlm_lib.c b/lustre/ldlm/ldlm_lib.c
index b4f7a6c..40df0e4 100644 (file)
--- a/lustre/ldlm/ldlm_lib.c
+++ b/lustre/ldlm/ldlm_lib.c
@@ -813,11 +813,11 @@ int target_handle_connect(struct ptlrpc_request *req)
         if (target->obd_no_conn) {
                spin_unlock(&target->obd_dev_lock);
 
         if (target->obd_no_conn) {
                spin_unlock(&target->obd_dev_lock);
 

-                LCONSOLE_WARN("%s: Temporarily refusing client connection "
-                              "from %s\n", target->obd_name,
-                              libcfs_nid2str(req->rq_peer.nid));
-                GOTO(out, rc = -EAGAIN);
-        }
+               CDEBUG(D_INFO, "%s: Temporarily refusing client connection "
+                              "from %s\n", target->obd_name,
+                              libcfs_nid2str(req->rq_peer.nid));
+               GOTO(out, rc = -EAGAIN);
+       }

 
        /* Make sure the target isn't cleaned up while we're here. Yes,
         * there's still a race between the above check and our incref here.
 
        /* Make sure the target isn't cleaned up while we're here. Yes,
         * there's still a race between the above check and our incref here.