Retire the AFS_PTR_FMT macro 30/13830/6
authorCheyenne Wills <cwills@sinenomine.net>
Tue, 24 Sep 2019 21:59:47 +0000 (15:59 -0600)
committerBenjamin Kaduk <kaduk@mit.edu>
Fri, 27 Sep 2019 15:01:45 +0000 (11:01 -0400)
Originally '%x' was commonly used as the printf specifier for formatting
pointer values.

Commit 37fc3b01445cd6446f09c476ea2db47fea544b7d introduced the
AFS_PTR_FMT macro to support platform-dependent printf format specifiers
for pointer representation. This macro defined the format specifier as
'%p' for Windows, and '%x' for non-Windows platforms.

Commit 2cf12c43c6a5822212f1d4e42dca7c059a1a9000 changed the printf
pointer format specifier from '%x' to '%p' on non-Windows platforms as
well, so at this point '%p' is the printf pointer format specifier for
all supported platforms.

Since the AFS_PRT_FMT macro is no longer platform-dependent, and all C89
compilers support the '%p' specifier, retire the macro to simplify the
printf format strings.

Change-Id: I0cb13cccbe6a8d0000edd162b623ddcdb74c1cf7
Reviewed-on: https://gerrit.openafs.org/13830
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>

16 files changed:
src/budb/database.c
src/budb/db_hash.c
src/budb/db_text.c
src/config/stds.h
src/dir/salvage.c
src/lwp/lwp.c
src/rx/rx.c
src/rx/rx_packet.c
src/viced/host.c
src/viced/state_analyzer.c
src/vol/clone.c
src/vol/vg_cache.c
src/vol/vnode.c
src/xstat/xstat_cm.c
src/xstat/xstat_fs.c
src/xstat/xstat_fs_test.c

index a57d830..a68b1f2 100644 (file)
@@ -108,8 +108,8 @@ dbread(struct ubik_trans *ut, afs_int32 pos, void *buff, afs_int32 len)
     }
     code = ubik_Read(ut, buff, len);
     if (code) {
-       LogError(code, "dbread: ubik_Read pos %d, buff %"AFS_PTR_FMT
-                ", len %d\n", pos, buff, len);
+       LogError(code, "dbread: ubik_Read pos %d, buff %p, len %d\n",
+                pos, buff, len);
        ERROR(code);
     }
 
@@ -143,8 +143,8 @@ cdbread(struct ubik_trans *ut, int type, afs_int32 pos, void *buff, afs_int32 le
     }
     code = ubik_Read(ut, buff, len);
     if (code) {
-       LogError(code, "cdbread: ubik_Read pos 0x%x, buff %"AFS_PTR_FMT
-                ", len %d\n", pos, buff, len);
+       LogError(code, "cdbread: ubik_Read pos 0x%x, buff %p, len %d\n",
+                pos, buff, len);
        ERROR(code);
     }
 
index 72bb983..e87f8fc 100644 (file)
@@ -883,7 +883,7 @@ ht_HashIn(struct ubik_trans *ut,
                      htonl(ea));
     if (code)
        return BUDB_IO;
-    LogDebug(5, "Hashin: set %"AFS_PTR_FMT" to %d\n",
+    LogDebug(5, "Hashin: set %p to %d\n",
             &block->b.bucket[bo], htonl(ea));
 
     pentries = &ht->entries;
index e88ef51..d66d834 100644 (file)
@@ -69,8 +69,8 @@ GetText(struct rx_call *call, afs_uint32 lockHandle, afs_int32 textType,
     char *textPtr;
     afs_int32 code;
 
-    LogDebug(5, "GetText: type %d, offset %d, nextOffset %"AFS_PTR_FMT
-            ", maxLength %d\n", textType, offset, nextOffset, maxLength);
+    LogDebug(5, "GetText: type %d, offset %d, nextOffset %p, maxLength %d\n",
+               textType, offset, nextOffset, maxLength);
 
     if (callPermitted(call) == 0) {
        code = BUDB_NOTPERMITTED;
index c54697e..18ab2e3 100644 (file)
@@ -229,12 +229,10 @@ typedef struct afsUUID afsUUID;
 #ifdef AFS_NT40_ENV
 # define AFS_INT64_FMT "I64d"
 # define AFS_UINT64_FMT "I64u"
-# define AFS_PTR_FMT   "p"
 # define AFS_SIZET_FMT "Iu"
 #else
 # define AFS_INT64_FMT "lld"
 # define AFS_UINT64_FMT "llu"
-# define AFS_PTR_FMT   "p"
 # ifdef PRINTF_TAKES_Z_LEN
 #  define AFS_SIZET_FMT "zu"
 # else
index a626317..3fb195c 100644 (file)
@@ -291,8 +291,8 @@ DirOK(void *file)
 
            /* A null name is no good */
            if (ep->name[0] == '\000') {
-               printf("Dir entry %"AFS_PTR_FMT
-                      " in chain %d has bogus (null) name.\n", ep, i);
+               printf("Dir entry %p in chain %d has bogus (null) name.\n",
+                       ep, i);
                DRelease(&entrybuf, 0);
                DRelease(&headerbuf, 0);
                return 0;
@@ -300,8 +300,8 @@ DirOK(void *file)
 
            /* The entry flag better be FFIRST */
            if (ep->flag != FFIRST) {
-               printf("Dir entry %"AFS_PTR_FMT
-                      " in chain %d has bogus flag field.\n", ep, i);
+               printf("Dir entry %p in chain %d has bogus flag field.\n",
+                       ep, i);
                DRelease(&entrybuf, 0);
                DRelease(&headerbuf, 0);
                return 0;
@@ -310,8 +310,7 @@ DirOK(void *file)
            /* Check the size of the name */
            j = strlen(ep->name);
            if (j >= MAXENAME) {        /* MAXENAME counts the null */
-               printf("Dir entry %"AFS_PTR_FMT
-                      " in chain %d has too-long name.\n", ep, i);
+               printf("Dir entry %p in chain %d has too-long name.\n", ep, i);
                DRelease(&entrybuf, 0);
                DRelease(&headerbuf, 0);
                return 0;
@@ -327,8 +326,7 @@ DirOK(void *file)
 
            /* Hash the name and make sure it is in the correct name hash */
            if ((j = afs_dir_DirHash(ep->name)) != i) {
-               printf("Dir entry %"AFS_PTR_FMT
-                      " should be in hash bucket %d but IS in %d.\n",
+               printf("Dir entry %p should be in hash bucket %d but IS in %d.\n",
                       ep, j, i);
                DRelease(&entrybuf, 0);
                DRelease(&headerbuf, 0);
@@ -341,8 +339,7 @@ DirOK(void *file)
                    havedot = 1;
                } else {
                    printf
-                       ("Dir entry %"AFS_PTR_FMT
-                        ", index 13 has name '%s' should be '.'\n",
+                       ("Dir entry %p, index 13 has name '%s' should be '.'\n",
                         ep, ep->name);
                    DRelease(&entrybuf, 0);
                    DRelease(&headerbuf, 0);
@@ -356,8 +353,7 @@ DirOK(void *file)
                    havedotdot = 1;
                } else {
                    printf
-                       ("Dir entry %"AFS_PTR_FMT
-                        ", index 14 has name '%s' should be '..'\n",
+                       ("Dir entry %p, index 14 has name '%s' should be '..'\n",
                         ep, ep->name);
                    DRelease(&entrybuf, 0);
                    DRelease(&headerbuf, 0);
index 7599475..199d5ed 100644 (file)
@@ -925,7 +925,7 @@ Dispatcher(void)
 #endif
        printf("stackcheck = %u: stack = %u \n", lwp_cpptr->stackcheck,
               *(int *)lwp_cpptr->stack);
-       printf("topstack = 0x%" AFS_PTR_FMT ": stackptr = 0x%" AFS_PTR_FMT ": stacksize = 0x%x\n",
+       printf("topstack = %p: stackptr = %p: stacksize = 0x%x\n",
               (void *)(uintptr_t)lwp_cpptr->context.topstack,
               (void *)(uintptr_t)lwp_cpptr->stack,
               lwp_cpptr->stacksize);
index a9225bc..8fe51b6 100644 (file)
@@ -1425,7 +1425,7 @@ static void
 rxi_WakeUpTransmitQueue(struct rx_call *call)
 {
     if (call->tqWaiters || (call->flags & RX_CALL_TQ_WAIT)) {
-       dpf(("call %"AFS_PTR_FMT" has %d waiters and flags %d\n",
+       dpf(("call %p has %d waiters and flags %d\n",
             call, call->tqWaiters, call->flags));
 #ifdef RX_ENABLE_LOCKS
        MUTEX_ASSERT(&call->lock);
@@ -1455,7 +1455,7 @@ rx_NewCall(struct rx_connection *conn)
     SPLVAR;
 
     clock_NewTime();
-    dpf(("rx_NewCall(conn %"AFS_PTR_FMT")\n", conn));
+    dpf(("rx_NewCall(conn %p)\n", conn));
 
     NETPRI;
     clock_GetTime(&queueTime);
@@ -1669,7 +1669,7 @@ rx_NewCall(struct rx_connection *conn)
     MUTEX_EXIT(&call->lock);
     USERPRI;
 
-    dpf(("rx_NewCall(call %"AFS_PTR_FMT")\n", call));
+    dpf(("rx_NewCall(call %p)\n", call));
     return call;
 }
 
@@ -2176,7 +2176,7 @@ rx_GetCall(int tno, struct rx_service *cur_service, osi_socket * socketp)
 #endif
 
        rxi_calltrace(RX_CALL_START, call);
-       dpf(("rx_GetCall(port=%d, service=%d) ==> call %"AFS_PTR_FMT"\n",
+       dpf(("rx_GetCall(port=%d, service=%d) ==> call %p\n",
             call->conn->service->servicePort, call->conn->service->serviceId,
             call));
 
@@ -2387,7 +2387,7 @@ rx_EndCall(struct rx_call *call, afs_int32 rc)
     afs_int32 error;
     SPLVAR;
 
-    dpf(("rx_EndCall(call %"AFS_PTR_FMT" rc %d error %d abortCode %d)\n",
+    dpf(("rx_EndCall(call %p rc %d error %d abortCode %d)\n",
           call, rc, call->error, call->abortCode));
 
     NETPRI;
@@ -2657,7 +2657,7 @@ rxi_NewCall(struct rx_connection *conn, int channel)
     struct opr_queue *cursor;
 #endif
 
-    dpf(("rxi_NewCall(conn %"AFS_PTR_FMT", channel %d)\n", conn, channel));
+    dpf(("rxi_NewCall(conn %p, channel %d)\n", conn, channel));
 
     /* Grab an existing call structure, or allocate a new one.
      * Existing call structures are assumed to have been left reset by
@@ -3418,7 +3418,7 @@ rxi_ReceivePacket(struct rx_packet *np, osi_socket socket,
  * this is the first time the packet has been seen */
     packetType = (np->header.type > 0 && np->header.type < RX_N_PACKET_TYPES)
        ? rx_packetTypes[np->header.type - 1] : "*UNKNOWN*";
-    dpf(("R %d %s: %x.%d.%d.%d.%d.%d.%d flags %d, packet %"AFS_PTR_FMT"\n",
+    dpf(("R %d %s: %x.%d.%d.%d.%d.%d.%d flags %d, packet %p\n",
         np->header.serial, packetType, ntohl(host), ntohs(port), np->header.serviceId,
         np->header.epoch, np->header.cid, np->header.callNumber,
         np->header.seq, np->header.flags, np));
@@ -3886,7 +3886,7 @@ rxi_ReceiveDataPacket(struct rx_call *call,
         if (rx_stats_active)
             rx_atomic_inc(&rx_stats.noPacketBuffersOnRead);
        rxi_calltrace(RX_TRACE_DROP, call);
-       dpf(("packet %"AFS_PTR_FMT" dropped on receipt - quota problems\n", np));
+       dpf(("packet %p dropped on receipt - quota problems\n", np));
         /* We used to clear the receive queue here, in an attempt to free
          * packets. However this is unsafe if the queue has received a
          * soft ACK for the final packet */
@@ -3940,7 +3940,7 @@ rxi_ReceiveDataPacket(struct rx_call *call,
                && opr_queue_First(&call->rq, struct rx_packet, entry)->header.seq == seq) {
                 if (rx_stats_active)
                     rx_atomic_inc(&rx_stats.dupPacketsRead);
-               dpf(("packet %"AFS_PTR_FMT" dropped on receipt - duplicate\n", np));
+               dpf(("packet %p dropped on receipt - duplicate\n", np));
                rxi_CancelDelayedAckEvent(call);
                np = rxi_SendAck(call, np, serial, RX_ACK_DUPLICATE, istack);
                ackNeeded = 0;
@@ -5149,7 +5149,7 @@ rxi_ClearReceiveQueue(struct rx_call *call)
 #ifdef RXDEBUG_PACKET
         call->rqc -= count;
         if ( call->rqc != 0 )
-            dpf(("rxi_ClearReceiveQueue call %"AFS_PTR_FMT" rqc %u != 0\n", call, call->rqc));
+         dpf(("rxi_ClearReceiveQueue call %p rqc %u != 0\n", call, call->rqc));
 #endif
        call->flags &= ~(RX_CALL_RECEIVE_DONE | RX_CALL_HAVE_LAST);
     }
@@ -5266,7 +5266,7 @@ rxi_ConnectionError(struct rx_connection *conn,
     if (error) {
        int i;
 
-       dpf(("rxi_ConnectionError conn %"AFS_PTR_FMT" error %d\n", conn, error));
+       dpf(("rxi_ConnectionError conn %p error %d\n", conn, error));
 
        MUTEX_ENTER(&conn->conn_data_lock);
        if (rxevent_Cancel(&conn->challengeEvent))
@@ -5311,7 +5311,7 @@ void
 rxi_CallError(struct rx_call *call, afs_int32 error)
 {
     MUTEX_ASSERT(&call->lock);
-    dpf(("rxi_CallError call %"AFS_PTR_FMT" error %d call->error %d\n", call, error, call->error));
+    dpf(("rxi_CallError call %p error %d call->error %d\n", call, error, call->error));
     if (call->error)
        error = call->error;
 
@@ -5340,7 +5340,7 @@ rxi_ResetCall(struct rx_call *call, int newcall)
     struct rx_packet *packet;
 
     MUTEX_ASSERT(&call->lock);
-    dpf(("rxi_ResetCall(call %"AFS_PTR_FMT", newcall %d)\n", call, newcall));
+    dpf(("rxi_ResetCall(call %p, newcall %d)\n", call, newcall));
 
     /* Notify anyone who is waiting for asynchronous packet arrival */
     if (call->arrivalProc) {
@@ -5401,7 +5401,7 @@ rxi_ResetCall(struct rx_call *call, int newcall)
 
     rxi_ClearTransmitQueue(call, 1);
     if (call->tqWaiters || (flags & RX_CALL_TQ_WAIT)) {
-        dpf(("rcall %"AFS_PTR_FMT" has %d waiters and flags %d\n", call, call->tqWaiters, call->flags));
+       dpf(("rcall %p has %d waiters and flags %d\n", call, call->tqWaiters, call->flags));
     }
     call->flags = 0;
 
@@ -6193,7 +6193,7 @@ rxi_Start(struct rx_call *call, int istack)
                                             nXmitPackets, istack);
                            goto restart;
                        }
-                        dpf(("call %d xmit packet %"AFS_PTR_FMT"\n",
+                      dpf(("call %d xmit packet %p\n",
                               *(call->callNumber), p));
                        call->xmitList[nXmitPackets++] = p;
                    }
@@ -6965,7 +6965,7 @@ rxi_ComputeRoundTripTime(struct rx_packet *p,
        return;                 /* somebody set the clock back, don't count this time. */
 
     clock_Sub(&thisRtt, sentp);
-    dpf(("rxi_ComputeRoundTripTime(call=%d packet=%"AFS_PTR_FMT" rttp=%d.%06d sec)\n",
+    dpf(("rxi_ComputeRoundTripTime(call=%d packet=%p rttp=%d.%06d sec)\n",
           p->header.callNumber, p, thisRtt.sec, thisRtt.usec));
 
     if (clock_IsZero(&thisRtt)) {
@@ -7064,8 +7064,10 @@ rxi_ComputeRoundTripTime(struct rx_packet *p,
     peer->rtt_dev = call->rtt_dev;
     peer->rtt = call->rtt;
 
-    dpf(("rxi_ComputeRoundTripTime(call=%d packet=%"AFS_PTR_FMT" rtt=%d ms, srtt=%d ms, rtt_dev=%d ms, timeout=%d.%06d sec)\n",
-          p->header.callNumber, p, MSEC(&thisRtt), call->rtt >> 3, call->rtt_dev >> 2, (call->rto.sec), (call->rto.usec)));
+    dpf(("rxi_ComputeRoundTripTime(call=%d packet=%p rtt=%d ms, srtt=%d ms, "
+        "rtt_dev=%d ms, timeout=%d.%06d sec)\n",
+        p->header.callNumber, p, MSEC(&thisRtt), call->rtt >> 3,
+        call->rtt_dev >> 2, (call->rto.sec), (call->rto.usec)));
 }
 
 
index d9822af..c52937e 100644 (file)
@@ -871,7 +871,7 @@ rx_CheckPackets(void)
 static void
 rxi_FreePacketNoLock(struct rx_packet *p)
 {
-    dpf(("Free %"AFS_PTR_FMT"\n", p));
+    dpf(("Free %p\n", p));
 
     RX_FPQ_MARK_FREE(p);
     rx_nFreePackets++;
@@ -884,7 +884,7 @@ static void
 rxi_FreePacketTSFPQ(struct rx_packet *p, int flush_global)
 {
     struct rx_ts_info_t * rx_ts_info;
-    dpf(("Free %"AFS_PTR_FMT"\n", p));
+    dpf(("Free %p\n", p));
 
     RX_TS_INFO_GET(rx_ts_info);
     RX_TS_FPQ_CHECKIN(rx_ts_info,p);
@@ -1183,7 +1183,7 @@ rxi_AllocPacketNoLock(int class)
 
     RX_TS_FPQ_CHECKOUT(rx_ts_info,p);
 
-    dpf(("Alloc %"AFS_PTR_FMT", class %d\n", p, class));
+    dpf(("Alloc %p, class %d\n", p, class));
 
 
     /* have to do this here because rx_FlushWrite fiddles with the iovs in
@@ -1241,7 +1241,7 @@ rxi_AllocPacketNoLock(int class)
     opr_queue_Remove(&p->entry);
     RX_FPQ_MARK_USED(p);
 
-    dpf(("Alloc %"AFS_PTR_FMT", class %d\n", p, class));
+    dpf(("Alloc %p, class %d\n", p, class));
 
 
     /* have to do this here because rx_FlushWrite fiddles with the iovs in
@@ -1279,7 +1279,7 @@ rxi_AllocPacketTSFPQ(int class, int pull_global)
 
     RX_TS_FPQ_CHECKOUT(rx_ts_info,p);
 
-    dpf(("Alloc %"AFS_PTR_FMT", class %d\n", p, class));
+    dpf(("Alloc %p, class %d\n", p, class));
 
     /* have to do this here because rx_FlushWrite fiddles with the iovs in
      * order to truncate outbound packets.  In the near future, may need
@@ -2352,7 +2352,7 @@ rxi_SendPacket(struct rx_call *call, struct rx_connection *conn,
 #endif
 #ifdef RXDEBUG
     }
-    dpf(("%c %d %s: %x.%u.%u.%u.%u.%u.%u flags %d, packet %"AFS_PTR_FMT" len %d\n",
+    dpf(("%c %d %s: %x.%u.%u.%u.%u.%u.%u flags %d, packet %p len %d\n",
           deliveryType, p->header.serial, rx_packetTypes[p->header.type - 1], ntohl(peer->host),
           ntohs(peer->port), p->header.serial, p->header.epoch, p->header.cid, p->header.callNumber,
           p->header.seq, p->header.flags, p, p->length));
@@ -2545,7 +2545,7 @@ rxi_SendPacketList(struct rx_call *call, struct rx_connection *conn,
 
     osi_Assert(p != NULL);
 
-    dpf(("%c %d %s: %x.%u.%u.%u.%u.%u.%u flags %d, packet %"AFS_PTR_FMT" len %d\n",
+    dpf(("%c %d %s: %x.%u.%u.%u.%u.%u.%u flags %d, packet %p len %d\n",
           deliveryType, p->header.serial, rx_packetTypes[p->header.type - 1], ntohl(peer->host),
           ntohs(peer->port), p->header.serial, p->header.epoch, p->header.cid, p->header.callNumber,
           p->header.seq, p->header.flags, p, p->length));
index 995900a..fe10ac0 100644 (file)
@@ -827,7 +827,7 @@ h_TossStuff_r(struct host *host)
        char hoststr[16];
        if (wasdeleted) {
            /* someone locked the host while HOSTDELETED was set; that is bad */
-           ViceLog(0, ("Warning:  h_TossStuff_r failed; Host %" AFS_PTR_FMT
+           ViceLog(0, ("Warning:  h_TossStuff_r failed; Host %p"
                        " (%s:%d flags 0x%x) was locked.\n",
                        host, afs_inet_ntoa_r(host->z.host, hoststr), ntohs(host->z.port),
                        (unsigned)host->z.hostFlags));
@@ -844,7 +844,7 @@ h_TossStuff_r(struct host *host)
        char hoststr[16];
        if (wasdeleted) {
            /* someone grabbed a ref while HOSTDELETED was set; that is bad */
-           ViceLog(0, ("Warning:  h_TossStuff_r failed; Host %" AFS_PTR_FMT
+           ViceLog(0, ("Warning:  h_TossStuff_r failed; Host %p"
                        " (%s:%d flags 0x%x) was held.\n",
                        host, afs_inet_ntoa_r(host->z.host, hoststr), ntohs(host->z.port),
                        (unsigned)host->z.hostFlags));
@@ -1130,7 +1130,7 @@ h_AddHostToUuidHashTable_r(struct afsUUID *uuid, struct host *host)
                afsUUID_to_string(&chain->hostPtr->z.interface->uuid, uuid1,
                                  127);
                afsUUID_to_string(uuid, uuid2, 127);
-               ViceLog(125, ("h_AddHostToUuidHashTable_r: host %" AFS_PTR_FMT " (uuid %s) exists as %s:%d (uuid %s)\n",
+               ViceLog(125, ("h_AddHostToUuidHashTable_r: host %p (uuid %s) exists as %s:%d (uuid %s)\n",
                              host, uuid1,
                              afs_inet_ntoa_r(chain->hostPtr->z.host, hoststr),
                              ntohs(chain->hostPtr->z.port), uuid2));
@@ -1177,7 +1177,7 @@ h_DeleteHostFromUuidHashTable_r(struct host *host)
          opr_Assert(uth->hostPtr);
         if (uth->hostPtr == host) {
             ViceLog(125,
-                    ("h_DeleteHostFromUuidHashTable_r: host %" AFS_PTR_FMT " (uuid %s %s:%d)\n",
+                    ("h_DeleteHostFromUuidHashTable_r: host %p (uuid %s %s:%d)\n",
                      host, uuid1, afs_inet_ntoa_r(host->z.host, hoststr),
                      ntohs(host->z.port)));
             *uhp = uth->next;
@@ -1186,7 +1186,7 @@ h_DeleteHostFromUuidHashTable_r(struct host *host)
         }
      }
      ViceLog(125,
-            ("h_DeleteHostFromUuidHashTable_r: host %" AFS_PTR_FMT " (uuid %s %s:%d) not found\n",
+            ("h_DeleteHostFromUuidHashTable_r: host %p (uuid %s %s:%d) not found\n",
              host, uuid1, afs_inet_ntoa_r(host->z.host, hoststr),
              ntohs(host->z.port)));
      return 0;
@@ -1208,7 +1208,7 @@ invalidateInterfaceAddr_r(struct host *host, afs_uint32 addr, afs_uint16 port)
     opr_Assert(host);
     opr_Assert(host->z.interface);
 
-    ViceLog(125, ("invalidateInterfaceAddr : host %" AFS_PTR_FMT " (%s:%d) addr %s:%d\n",
+    ViceLog(125, ("invalidateInterfaceAddr : host %p (%s:%d) addr %s:%d\n",
                  host, afs_inet_ntoa_r(host->z.host, hoststr),
                  ntohs(host->z.port), afs_inet_ntoa_r(addr, hoststr2),
                  ntohs(port)));
@@ -1253,7 +1253,7 @@ removeAddress_r(struct host *host, afs_uint32 addr, afs_uint16 port)
     if (!host->z.interface || host->z.interface->numberOfInterfaces == 1) {
        if (host->z.host == addr && host->z.port == port) {
            ViceLog(25,
-                   ("Removing only address for host %" AFS_PTR_FMT " (%s:%d), deleting host.\n",
+                   ("Removing only address for host %p (%s:%d), deleting host.\n",
                     host, afs_inet_ntoa_r(host->z.host, hoststr), ntohs(host->z.port)));
            host->z.hostFlags |= HOSTDELETED;
             /*
@@ -1265,7 +1265,7 @@ removeAddress_r(struct host *host, afs_uint32 addr, afs_uint16 port)
              */
         } else {
            ViceLog(0,
-                   ("Removing address that does not belong to host %" AFS_PTR_FMT " (%s:%d).\n",
+                   ("Removing address that does not belong to host %p (%s:%d).\n",
                     host, afs_inet_ntoa_r(host->z.host, hoststr), ntohs(host->z.port)));
         }
     } else {
@@ -1275,7 +1275,7 @@ removeAddress_r(struct host *host, afs_uint32 addr, afs_uint16 port)
            for (i=0; i < host->z.interface->numberOfInterfaces; i++) {
                if (host->z.interface->interface[i].valid) {
                    ViceLog(25,
-                            ("Removed address for host %" AFS_PTR_FMT " (%s:%d), new primary interface %s:%d.\n",
+                            ("Removed address for host %p (%s:%d), new primary interface %s:%d.\n",
                               host, afs_inet_ntoa_r(host->z.host, hoststr), ntohs(host->z.port),
                               afs_inet_ntoa_r(host->z.interface->interface[i].addr, hoststr2),
                               ntohs(host->z.interface->interface[i].port)));
@@ -1288,7 +1288,7 @@ removeAddress_r(struct host *host, afs_uint32 addr, afs_uint16 port)
 
            if (i == host->z.interface->numberOfInterfaces) {
                 ViceLog(25,
-                         ("Removed only address for host %" AFS_PTR_FMT " (%s:%d), no valid alternate interfaces, deleting host.\n",
+                        ("Removed only address for host %p (%s:%d), no valid alternate interfaces, deleting host.\n",
                           host, afs_inet_ntoa_r(host->z.host, hoststr), ntohs(host->z.port)));
                host->z.hostFlags |= HOSTDELETED;
                 /* addr/port was removed from the hash table */
@@ -1330,7 +1330,7 @@ createHostAddrHashChain_r(int index, afs_uint32 addr, afs_uint16 port, struct ho
     chain->addr = addr;
     chain->port = port;
     hostAddrHashTable[index] = chain;
-    ViceLog(125, ("h_AddHostToAddrHashTable_r: host %" AFS_PTR_FMT " added as %s:%d\n",
+    ViceLog(125, ("h_AddHostToAddrHashTable_r: host %p added as %s:%d\n",
                  host, afs_inet_ntoa_r(addr, hoststr), ntohs(port)));
 }
 
@@ -1355,8 +1355,8 @@ reconcileHosts_r(afs_uint32 addr, afs_uint16 port, struct host *newHost,
     char hoststr[16];
 
     ViceLog(125,
-           ("reconcileHosts_r: addr %s:%d newHost %" AFS_PTR_FMT " oldHost %"
-            AFS_PTR_FMT "\n", afs_inet_ntoa_r(addr, hoststr), ntohs(port),
+           ("reconcileHosts_r: addr %s:%d newHost %p oldHost %p\n",
+            afs_inet_ntoa_r(addr, hoststr), ntohs(port),
             newHost, oldHost));
 
     opr_Assert(oldHost != newHost);
@@ -1479,7 +1479,7 @@ h_AddHostToAddrHashTable_r(afs_uint32 addr, afs_uint16 port, struct host *host)
        if (chain->addr == addr && chain->port == port) {
            if (chain->hostPtr == host) {
                ViceLog(125,
-                       ("h_AddHostToAddrHashTable_r: host %" AFS_PTR_FMT " (%s:%d) already hashed\n",
+                       ("h_AddHostToAddrHashTable_r: host %p (%s:%d) already hashed\n",
                          host, afs_inet_ntoa_r(chain->addr, hoststr),
                          ntohs(chain->port)));
                return;
@@ -1521,7 +1521,7 @@ addInterfaceAddr_r(struct host *host, afs_uint32 addr, afs_uint16 port)
        if (host->z.interface->interface[i].addr == addr &&
             host->z.interface->interface[i].port == port) {
            ViceLog(125,
-                   ("addInterfaceAddr : found host %" AFS_PTR_FMT " (%s:%d) adding %s:%d%s\n",
+                   ("addInterfaceAddr : found host %p (%s:%d) adding %s:%d%s\n",
                     host, afs_inet_ntoa_r(host->z.host, hoststr),
                     ntohs(host->z.port), afs_inet_ntoa_r(addr, hoststr2),
                     ntohs(port), host->z.interface->interface[i].valid ? "" :
@@ -1535,7 +1535,7 @@ addInterfaceAddr_r(struct host *host, afs_uint32 addr, afs_uint16 port)
         }
     }
 
-    ViceLog(125, ("addInterfaceAddr : host %" AFS_PTR_FMT " (%s:%d) adding %s:%d\n",
+    ViceLog(125, ("addInterfaceAddr : host %p (%s:%d) adding %s:%d\n",
                  host, afs_inet_ntoa_r(host->z.host, hoststr),
                  ntohs(host->z.port), afs_inet_ntoa_r(addr, hoststr2),
                  ntohs(port)));
@@ -1578,7 +1578,7 @@ removeInterfaceAddr_r(struct host *host, afs_uint32 addr, afs_uint16 port)
     opr_Assert(host);
     opr_Assert(host->z.interface);
 
-    ViceLog(125, ("removeInterfaceAddr : host %" AFS_PTR_FMT " (%s:%d) addr %s:%d\n",
+    ViceLog(125, ("removeInterfaceAddr : host %p (%s:%d) addr %s:%d\n",
                  host, afs_inet_ntoa_r(host->z.host, hoststr),
                  ntohs(host->z.port), afs_inet_ntoa_r(addr, hoststr2),
                  ntohs(port)));
@@ -1886,7 +1886,7 @@ h_GetHost_r(struct rx_connection *tcon)
              * waited for the lock. */
            h_Unlock_r(host);
            ViceLog(125,
-                   ("Host %" AFS_PTR_FMT " (%s:%d) starting h_Lookup again\n",
+                   ("Host %p (%s:%d) starting h_Lookup again\n",
                     host, afs_inet_ntoa_r(host->z.host, hoststr),
                     ntohs(host->z.port)));
            h_Release_r(host);
@@ -1968,7 +1968,7 @@ h_GetHost_r(struct rx_connection *tcon)
                 * that we maintain some extra callback state information */
                if (host->z.interface) {
                    ViceLog(0,
-                           ("Host %" AFS_PTR_FMT " (%s:%d) used to support WhoAreYou, deleting.\n",
+                           ("Host %p (%s:%d) used to support WhoAreYou, deleting.\n",
                             host,
                             afs_inet_ntoa_r(host->z.host, hoststr),
                             ntohs(host->z.port)));
@@ -2015,7 +2015,7 @@ h_GetHost_r(struct rx_connection *tcon)
                        removeAddress_r(host, haddr, hport);
                } else {
                    ViceLog(25,
-                           ("Uuid doesn't match host %" AFS_PTR_FMT " (%s:%d).\n",
+                           ("Uuid doesn't match host %p (%s:%d).\n",
                             host, afs_inet_ntoa_r(host->z.host, hoststr), ntohs(host->z.port)));
 
                    removeAddress_r(host, host->z.host, host->z.port);
@@ -2052,7 +2052,7 @@ h_GetHost_r(struct rx_connection *tcon)
                      * callback connection, and destroy the old one.
                      */
                     struct rx_connection *rxconn;
-                    ViceLog(0,("CB: ProbeUuid for host %" AFS_PTR_FMT " (%s:%d) failed %d\n",
+                   ViceLog(0,("CB: ProbeUuid for host %p (%s:%d) failed %d\n",
                               host,
                               afs_inet_ntoa_r(host->z.host, hoststr),
                               ntohs(host->z.port),code2));
@@ -2102,7 +2102,7 @@ h_GetHost_r(struct rx_connection *tcon)
                goto gethost_out;
            } else {
                ViceLog(0,
-                       ("CB: WhoAreYou failed for host %" AFS_PTR_FMT " (%s:%d), error %d\n",
+                       ("CB: WhoAreYou failed for host %p (%s:%d), error %d\n",
                         host, afs_inet_ntoa_r(host->z.host, hoststr),
                         ntohs(host->z.port), code));
                host->z.hostFlags |= VENUSDOWN;
@@ -2120,13 +2120,13 @@ h_GetHost_r(struct rx_connection *tcon)
        if (!(host->z.hostFlags & ALTADDR)) {
            /* another thread is doing the initialisation */
            ViceLog(125,
-                   ("Host %" AFS_PTR_FMT " (%s:%d) waiting for host-init to complete\n",
+                   ("Host %p (%s:%d) waiting for host-init to complete\n",
                     host, afs_inet_ntoa_r(host->z.host, hoststr),
                     ntohs(host->z.port)));
            h_Lock_r(host);
            h_Unlock_r(host);
            ViceLog(125,
-                   ("Host %" AFS_PTR_FMT " (%s:%d) starting h_Lookup again\n",
+                   ("Host %p (%s:%d) starting h_Lookup again\n",
                     host, afs_inet_ntoa_r(host->z.host, hoststr),
                     ntohs(host->z.port)));
            h_Release_r(host);
@@ -2193,7 +2193,7 @@ h_GetHost_r(struct rx_connection *tcon)
                if (!pident)
                    rx_SetSpecific(tcon, rxcon_ident_key, identP);
                ViceLog(25,
-                       ("Host %" AFS_PTR_FMT " (%s:%d) does not support WhoAreYou.\n",
+                       ("Host %p (%s:%d) does not support WhoAreYou.\n",
                         host, afs_inet_ntoa_r(host->z.host, hoststr),
                         ntohs(host->z.port)));
                code = 0;
@@ -2212,7 +2212,7 @@ h_GetHost_r(struct rx_connection *tcon)
                if (!pident)
                    rx_SetSpecific(tcon, rxcon_ident_key, identP);
                ViceLog(25,
-                       ("WhoAreYou success on host %" AFS_PTR_FMT " (%s:%d)\n",
+                       ("WhoAreYou success on host %p (%s:%d)\n",
                         host, afs_inet_ntoa_r(host->z.host, hoststr),
                         ntohs(host->z.port)));
            }
@@ -2273,7 +2273,7 @@ h_GetHost_r(struct rx_connection *tcon)
                             * MultiProbeAlternateAddress_r() will remove the
                             * alternate interfaces that do not have the same
                             * Uuid. */
-                           ViceLog(0,("CB: ProbeUuid for host %" AFS_PTR_FMT " (%s:%d) failed %d\n",
+                           ViceLog(0,("CB: ProbeUuid for host %p (%s:%d) failed %d\n",
                                         oldHost,
                                         afs_inet_ntoa_r(oldHost->z.host, hoststr),
                                         ntohs(oldHost->z.port),code2));
@@ -2297,7 +2297,7 @@ h_GetHost_r(struct rx_connection *tcon)
                        struct rx_connection *rxconn;
 
                        ViceLog(25,
-                                ("CB: Host %" AFS_PTR_FMT " (%s:%d) has new addr %s:%d\n",
+                                ("CB: Host %p (%s:%d) has new addr %s:%d\n",
                                   oldHost,
                                   afs_inet_ntoa_r(oldHost->z.host, hoststr2),
                                   ntohs(oldHost->z.port),
@@ -2367,7 +2367,7 @@ h_GetHost_r(struct rx_connection *tcon)
                    H_LOCK;
                    if (code == 0) {
                        ViceLog(25,
-                               ("InitCallBackState3 success on host %" AFS_PTR_FMT " (%s:%d)\n",
+                               ("InitCallBackState3 success on host %p (%s:%d)\n",
                                 host, afs_inet_ntoa_r(host->z.host, hoststr),
                                 ntohs(host->z.port)));
                    }
@@ -2375,12 +2375,12 @@ h_GetHost_r(struct rx_connection *tcon)
            }
            if (code) {
                ViceLog(0,
-                       ("CB: RCallBackConnectBack failed for %" AFS_PTR_FMT " (%s:%d)\n",
+                       ("CB: RCallBackConnectBack failed for %p (%s:%d)\n",
                         host, afs_inet_ntoa_r(host->z.host, hoststr), ntohs(host->z.port)));
                host->z.hostFlags |= VENUSDOWN;
            } else {
                ViceLog(125,
-                       ("CB: RCallBackConnectBack succeeded for %" AFS_PTR_FMT " (%s:%d)\n",
+                       ("CB: RCallBackConnectBack succeeded for %p (%s:%d)\n",
                         host, afs_inet_ntoa_r(host->z.host, hoststr), ntohs(host->z.port)));
                host->z.hostFlags |= RESETDONE;
            }
@@ -2732,7 +2732,7 @@ h_FindClient_r(struct rx_connection *tcon, afs_int32 *a_viceid)
            if (code) {
                char hoststr[16];
                ViceLog(0,
-                       ("pr_GetCPS failed(%d) for user %d, host %" AFS_PTR_FMT " (%s:%d)\n",
+                       ("pr_GetCPS failed(%d) for user %d, host %p (%s:%d)\n",
                         code, viceid, client->z.host,
                         afs_inet_ntoa_r(client->z.host->z.host,hoststr),
                         ntohs(client->z.host->z.port)));
@@ -4286,7 +4286,7 @@ h_DeleteHostFromAddrHashTable_r(afs_uint32 addr, afs_uint16 port,
         hp = &th->next) {
         opr_Assert(th->hostPtr);
         if (th->hostPtr == host && th->addr == addr && th->port == port) {
-           ViceLog(125, ("h_DeleteHostFromAddrHashTable_r: host %" AFS_PTR_FMT " (%s:%d)\n",
+           ViceLog(125, ("h_DeleteHostFromAddrHashTable_r: host %p (%s:%d)\n",
                          host, afs_inet_ntoa_r(host->z.host, hoststr),
                          ntohs(host->z.port)));
             *hp = th->next;
@@ -4295,7 +4295,7 @@ h_DeleteHostFromAddrHashTable_r(afs_uint32 addr, afs_uint16 port,
         }
     }
     ViceLog(125,
-           ("h_DeleteHostFromAddrHashTable_r: host %" AFS_PTR_FMT " (%s:%d) not found\n",
+           ("h_DeleteHostFromAddrHashTable_r: host %p (%s:%d) not found\n",
             host, afs_inet_ntoa_r(host->z.host, hoststr),
             ntohs(host->z.port)));
     return 0;
index 10974fd..d5b8dbb 100644 (file)
@@ -259,7 +259,7 @@ openFile(char * path)
        goto done;
     }
 
-    printf("mapped %lu bytes at %"AFS_PTR_FMT"\n", (unsigned long)map_len, map);
+    printf("mapped %lu bytes at %p\n", (unsigned long)map_len, map);
 
  done:
     if (ret) {
@@ -700,7 +700,7 @@ print_cb_help(void)
     do { \
         char * _p = (char *)addr; \
         char * _m = (char *)map; \
-        printf("loading structure from address %"AFS_PTR_FMT" (offset %ld)\n", \
+       printf("loading structure from address %p (offset %ld)\n", \
                addr, (long)(_p-_m)); \
     } while (0)
 
index dfe67ca..deaa079 100644 (file)
@@ -256,7 +256,7 @@ DoCloneIndex(Volume * rwvp, Volume * clvp, VnodeClass class, int reclone)
            } else if (rwinode) {
                if (IH_INC(V_linkHandle(rwvp), rwinode, V_parentId(rwvp)) ==
                    -1) {
-                   Log("IH_INC failed: %"AFS_PTR_FMT", %s, %" AFS_VOLID_FMT " errno %d\n",
+                   Log("IH_INC failed: %p, %s, %" AFS_VOLID_FMT " errno %d\n",
                        V_linkHandle(rwvp), PrintInode(stmp, rwinode),
                        afs_printable_VolumeId_lu(V_parentId(rwvp)), errno);
                    VForceOffline(rwvp);
@@ -309,7 +309,7 @@ DoCloneIndex(Volume * rwvp, Volume * clvp, VnodeClass class, int reclone)
            if (inodeinced) {
                if (IH_DEC(V_linkHandle(rwvp), rwinode, V_parentId(rwvp)) ==
                    -1) {
-                   Log("IH_DEC failed: %"AFS_PTR_FMT", %s, %" AFS_VOLID_FMT " errno %d\n",
+                   Log("IH_DEC failed: %p, %s, %" AFS_VOLID_FMT " errno %d\n",
                        V_linkHandle(rwvp), PrintInode(stmp, rwinode),
                        afs_printable_VolumeId_lu(V_parentId(rwvp)), errno);
                    VForceOffline(rwvp);
index 5a53a2f..5e8ca75 100644 (file)
@@ -508,7 +508,7 @@ _VVGC_hash_entry_add(struct DiskPartition64 * dp,
        if (ent != nent) {
            ViceLog(0, ("_VVGC_hash_entry_add: tried to add a duplicate "
                        " nonmatching entry for vol %lu: original "
-                       "(%"AFS_PTR_FMT",%lu) new (%"AFS_PTR_FMT",%lu)\n",
+                       "(%p,%lu) new (%p,%lu)\n",
                        afs_printable_uint32_lu(volid),
                        nent, afs_printable_uint32_lu(nent->rw),
                        ent, afs_printable_uint32_lu(ent->rw)));
@@ -891,8 +891,8 @@ _VVGC_entry_purge_r(struct DiskPartition64 * dp,
        if (parent_ent != child_ent) {
            ViceLog(0,
                    ("VVGCache_entry_del: trying to delete vol %lu from VG %lu, "
-                    "but vol %lu points to VGC entry %" AFS_PTR_FMT
-                    " and VG %lu " "points to VGC entry %" AFS_PTR_FMT "\n",
+                    "but vol %lu points to VGC entry %p"
+                    " and VG %lu " "points to VGC entry %p\n",
                     afs_printable_uint32_lu(child),
                     afs_printable_uint32_lu(parent),
                     afs_printable_uint32_lu(child), child_ent,
index 472e2e4..dd50c5f 100644 (file)
@@ -1376,7 +1376,7 @@ VPutVnode_r(Error * ec, Vnode * vnp)
                                                changed_oldTime) << 1) | vnp->
              delete, 0, 0);
        if (thisProcess != vnp->writer)
-           Abort("VPutVnode: Vnode at %"AFS_PTR_FMT" locked by another process!\n",
+           Abort("VPutVnode: Vnode at %p locked by another process!\n",
                  vnp);
 
 
@@ -1434,7 +1434,7 @@ VPutVnode_r(Error * ec, Vnode * vnp)
        if (vnp->changed_newTime || vnp->changed_oldTime || vnp->delete)
            Abort
                ("VPutVnode: Change or delete flag for vnode "
-                "%"AFS_PTR_FMT" is set but vnode is not write locked!\n",
+                "%p is set but vnode is not write locked!\n",
                 vnp);
 #ifdef AFS_DEMAND_ATTACH_FS
        VnEndRead_r(vnp);
@@ -1521,8 +1521,7 @@ VVnodeWriteToRead_r(Error * ec, Vnode * vnp)
     LWP_CurrentProcess(&thisProcess);
 #endif /* AFS_PTHREAD_ENV */
     if (thisProcess != vnp->writer)
-       Abort("VPutVnode: Vnode at %"AFS_PTR_FMT
-             " locked by another process!\n", vnp);
+       Abort("VPutVnode: Vnode at %p locked by another process!\n", vnp);
 
     if (vnp->delete) {
        return 0;
index 45e05f3..4189cb7 100644 (file)
@@ -258,12 +258,12 @@ xstat_cm_LWP(void *unused)
 
                    if (xstat_cm_debug) {
                        printf
-                           ("%s: Calling RXAFSCB_GetXStats, conn=%" AFS_PTR_FMT ", clientVersionNumber=%d, collectionNumber=%d, srvVersionNumberP=%" AFS_PTR_FMT ", timeP=%" AFS_PTR_FMT ", dataP=%" AFS_PTR_FMT "\n",
+                           ("%s: Calling RXAFSCB_GetXStats, conn=%p, clientVersionNumber=%d, collectionNumber=%d, srvVersionNumberP=%p, timeP=%p, dataP=%p\n",
                             rn, curr_conn->rxconn, clientVersionNumber,
                             *currCollIDP, &srvVersionNumber,
                             &(xstat_cm_Results.probeTime),
                             &(xstat_cm_Results.data));
-                       printf("%s: [bufflen=%d, buffer at %" AFS_PTR_FMT "]\n", rn,
+                       printf("%s: [bufflen=%d, buffer at %p]\n", rn,
                               xstat_cm_Results.data.AFSCB_CollData_len,
                               xstat_cm_Results.data.AFSCB_CollData_val);
                    }
@@ -552,7 +552,7 @@ xstat_cm_Init(int a_numServers, struct sockaddr_in *a_socketArray,
            conn_err = 1;
        }
        if (xstat_cm_debug)
-           printf("[%s] New connection at %" AFS_PTR_FMT "\n", rn, curr_conn->rxconn);
+           printf("[%s] New connection at %p\n", rn, curr_conn->rxconn);
 
        /*
         * Bump the current xstat_cm connection to set up.
index 99fdf63..a025abe 100644 (file)
@@ -277,12 +277,12 @@ xstat_fs_LWP(void *unused)
 
                    if (xstat_fs_debug) {
                        printf
-                           ("%s: Calling RXAFS_GetXStats, conn=%" AFS_PTR_FMT ", clientVersionNumber=%d, collectionNumber=%d, srvVersionNumberP=%" AFS_PTR_FMT ", timeP=%" AFS_PTR_FMT ", dataP=%" AFS_PTR_FMT "\n",
+                           ("%s: Calling RXAFS_GetXStats, conn=%p, clientVersionNumber=%d, collectionNumber=%d, srvVersionNumberP=%p, timeP=%p, dataP=%p\n",
                             rn, curr_conn->rxconn, clientVersionNumber,
                             *currCollIDP, &srvVersionNumber,
                             &(xstat_fs_Results.probeTime),
                             &(xstat_fs_Results.data));
-                       printf("%s: [bufflen=%d, buffer at %" AFS_PTR_FMT "]\n", rn,
+                       printf("%s: [bufflen=%d, buffer at %p]\n", rn,
                               xstat_fs_Results.data.AFS_CollData_len,
                               xstat_fs_Results.data.AFS_CollData_val);
                    }
@@ -585,7 +585,7 @@ xstat_fs_Init(int a_numServers, struct sockaddr_in *a_socketArray,
            conn_err = 1;
        }
        if (xstat_fs_debug)
-           printf("[%s] New connection at %" AFS_PTR_FMT "\n", rn, curr_conn->rxconn);
+           printf("[%s] New connection at %p\n", rn, curr_conn->rxconn);
 
        /*
         * Bump the current xstat_fs connection to set up.
index a066e8b..7f9f822 100644 (file)
@@ -123,7 +123,7 @@ PrintCallInfo(void)
           printableTime);
 
     if (debugging_on)
-       printf("\n[%u entries returned at %" AFS_PTR_FMT "]\n\n", numInt32s, currInt32);
+       printf("\n[%u entries returned at %p]\n\n", numInt32s, currInt32);
 
     for (i = 0; i < numInt32s; i++)
        printf("%u ", *currInt32++);