]> git.michaelhowe.org Git - packages/o/openafs.git/commitdiff
viced: Improve CallPreamble error messages
authorAndrew Deason <adeason@sinenomine.net>
Sat, 23 Feb 2013 04:46:12 +0000 (22:46 -0600)
committerStephan Wiesand <stephan.wiesand@desy.de>
Thu, 27 Feb 2014 14:41:37 +0000 (06:41 -0800)
These messages are not very useful right now. At least try to say what
host we sent an error to, so we know which host may be experiencing
some troubles as a result.

Reviewed-on: http://gerrit.openafs.org/9381
Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
(cherry picked from commit 6c41b1f740e16b5b9adfe9026630595be6f0699e)

Change-Id: I4e9cf5e0d038c572895b4a31bfdff481ea0b3286
Reviewed-on: http://gerrit.openafs.org/10756
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Reviewed-by: D Brashear <shadow@your-file-system.com>
Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
src/viced/afsfileprocs.c

index 36cbfbdc3a6c47a35f2f2a1001ffd9b0212853f6..46b2a10a4312a5397cf346519e06d3dc4c33d7a4 100644 (file)
@@ -382,7 +382,10 @@ CallPreamble(struct rx_call *acall, int activecall,
   retry:
     tclient = h_FindClient_r(*tconn);
     if (!tclient) {
-       ViceLog(0, ("CallPreamble: Couldn't get client.\n"));
+       ViceLog(0, ("CallPreamble: Couldn't get client struct for host "
+                   "%s:%d, sending busy signal\n",
+                   afs_inet_ntoa_r(rx_HostOf(rx_PeerOf(*tconn)), hoststr),
+                   (int)ntohs(rx_PortOf(rx_PeerOf(*tconn)))));
        H_UNLOCK;
        return VBUSY;
     }
@@ -391,7 +394,10 @@ CallPreamble(struct rx_call *acall, int activecall,
        if (!retry_flag) {
            h_ReleaseClient_r(tclient);
            h_Release_r(thost);
-           ViceLog(0, ("CallPreamble: Couldn't get CPS. Fail\n"));
+           ViceLog(0, ("CallPreamble: Couldn't get CPS for client from host "
+                       "%s:%d, failing request\n",
+                       afs_inet_ntoa_r(thost->host, hoststr),
+                       (int)ntohs(thost->port)));
            H_UNLOCK;
            return -1001;
        }
@@ -421,7 +427,10 @@ CallPreamble(struct rx_call *acall, int activecall,
            h_ReleaseClient_r(tclient);
            h_Release_r(thost);
            H_UNLOCK;
-           ViceLog(0, ("CallPreamble: couldn't reconnect to ptserver\n"));
+           ViceLog(0, ("CallPreamble: couldn't reconnect to ptserver while "
+                       "handling request for %s:%d\n",
+                       afs_inet_ntoa_r(thost->host, hoststr),
+                       (int)ntohs(thost->port)));
            return -1001;
        }