cifs: clean up checks in cifs_echo_request
authorJeff Layton <jlayton@redhat.com>
Wed, 9 Feb 2011 17:01:42 +0000 (12:01 -0500)
committerSteve French <sfrench@us.ibm.com>
Thu, 10 Feb 2011 03:44:20 +0000 (03:44 +0000)
Follow-on patch to 7e90d705 which is already in Steve's tree...

The check for tcpStatus == CifsGood is not meaningful since it doesn't
indicate whether the NEGOTIATE request has been done. Also, clarify
why we're checking for maxBuf == 0.

Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
fs/cifs/connect.c

index 10011e99b34dc6412c8f0e3d15263547a870da93..161f24ca4f6ee3090ea766641ab6baecd6236440 100644 (file)
@@ -338,10 +338,11 @@ cifs_echo_request(struct work_struct *work)
                                        struct TCP_Server_Info, echo.work);
 
        /*
-        * We cannot send an echo until the NEGOTIATE_PROTOCOL request is done.
-        * Also, no need to ping if we got a response recently
+        * We cannot send an echo until the NEGOTIATE_PROTOCOL request is
+        * done, which is indicated by maxBuf != 0. Also, no need to ping if
+        * we got a response recently
         */
-       if ((server->tcpStatus != CifsGood) || (server->maxBuf == 0) ||
+       if (server->maxBuf == 0 ||
            time_before(jiffies, server->lstrp + SMB_ECHO_INTERVAL - HZ))
                goto requeue_echo;