From ed25a920232cb659c2997c91c5bc92a659ff163a Mon Sep 17 00:00:00 2001 From: Jeffrey Altman Date: Mon, 20 Sep 2010 10:02:18 -0700 Subject: [PATCH 1/1] Rx: Permit MakeDebugCall() be be compiled when RXDEBUG is undefined If MAKEDEBUGCALL is defined, build MakeDebugCall() even if RXDEBUG is not defined. Change-Id: I98d3216e8bf1e1f3b6b62c962d33db316d0f51ac Reviewed-on: http://gerrit.openafs.org/2818 Tested-by: BuildBot Reviewed-by: Derrick Brashear --- src/rx/NTMakefile | 2 +- src/rx/rx.c | 28 ++++++++++++++-------------- 2 files changed, 15 insertions(+), 15 deletions(-) diff --git a/src/rx/NTMakefile b/src/rx/NTMakefile index b242bc0..e40c284 100644 --- a/src/rx/NTMakefile +++ b/src/rx/NTMakefile @@ -18,7 +18,7 @@ CML_PREFIX=rx_ $(C2OBJ) $< # Additional debugging flag for RX. -AFSDEV_AUXCDEFINES = +AFSDEV_AUXCDEFINES = -DMAKEDEBUGCALL !IF ("$(AFSDEV_BUILDTYPE)" == "CHECKED") AFSDEV_AUXCDEFINES = $(AFSDEV_AUXCDEFINES) -DRXDEBUG -DRXDEBUG_PACKET -DRX_TRACK_PACKETS -DRX_REFCOUNT_CHECK !ENDIF diff --git a/src/rx/rx.c b/src/rx/rx.c index afd6f7e..0f015c8 100644 --- a/src/rx/rx.c +++ b/src/rx/rx.c @@ -7093,7 +7093,7 @@ rx_PrintPeerStats(FILE * file, struct rx_peer *peer) #define UNLOCK_RX_DEBUG #endif /* AFS_PTHREAD_ENV */ -#ifdef RXDEBUG +#if defined(RXDEBUG) || defined(MAKEDEBUGCALL) static int MakeDebugCall(osi_socket socket, afs_uint32 remoteAddr, afs_uint16 remotePort, u_char type, void *inputData, size_t inputLength, @@ -7209,9 +7209,7 @@ rx_GetServerDebug(osi_socket socket, afs_uint32 remoteAddr, afs_uint16 remotePort, struct rx_debugStats * stat, afs_uint32 * supportedValues) { -#ifndef RXDEBUG - afs_int32 rc = -1; -#else +#if defined(RXDEBUG) || defined(MAKEDEBUGCALL) afs_int32 rc = 0; struct rx_debugIn in; @@ -7264,6 +7262,8 @@ rx_GetServerDebug(osi_socket socket, afs_uint32 remoteAddr, stat->nWaited = ntohl(stat->nWaited); stat->nPackets = ntohl(stat->nPackets); } +#else + afs_int32 rc = -1; #endif return rc; } @@ -7273,9 +7273,7 @@ rx_GetServerStats(osi_socket socket, afs_uint32 remoteAddr, afs_uint16 remotePort, struct rx_statistics * stat, afs_uint32 * supportedValues) { -#ifndef RXDEBUG - afs_int32 rc = -1; -#else +#if defined(RXDEBUG) || defined(MAKEDEBUGCALL) afs_int32 rc = 0; struct rx_debugIn in; afs_int32 *lp = (afs_int32 *) stat; @@ -7304,6 +7302,8 @@ rx_GetServerStats(osi_socket socket, afs_uint32 remoteAddr, *lp = ntohl(*lp); } } +#else + afs_int32 rc = -1; #endif return rc; } @@ -7313,7 +7313,7 @@ rx_GetServerVersion(osi_socket socket, afs_uint32 remoteAddr, afs_uint16 remotePort, size_t version_length, char *version) { -#ifdef RXDEBUG +#if defined(RXDEBUG) || defined(MAKEDEBUGCALL) char a[1] = { 0 }; return MakeDebugCall(socket, remoteAddr, remotePort, RX_PACKET_TYPE_VERSION, a, 1, version, @@ -7330,9 +7330,7 @@ rx_GetServerConnections(osi_socket socket, afs_uint32 remoteAddr, struct rx_debugConn * conn, afs_uint32 * supportedValues) { -#ifndef RXDEBUG - afs_int32 rc = -1; -#else +#if defined(RXDEBUG) || defined(MAKEDEBUGCALL) afs_int32 rc = 0; struct rx_debugIn in; int i; @@ -7406,6 +7404,8 @@ rx_GetServerConnections(osi_socket socket, afs_uint32 remoteAddr, conn->epoch = ntohl(conn->epoch); conn->natMTU = ntohl(conn->natMTU); } +#else + afs_int32 rc = -1; #endif return rc; } @@ -7416,9 +7416,7 @@ rx_GetServerPeers(osi_socket socket, afs_uint32 remoteAddr, afs_uint32 debugSupportedValues, struct rx_debugPeer * peer, afs_uint32 * supportedValues) { -#ifndef RXDEBUG - afs_int32 rc = -1; -#else +#if defined(RXDEBUG) || defined(MAKEDEBUGCALL) afs_int32 rc = 0; struct rx_debugIn in; @@ -7471,6 +7469,8 @@ rx_GetServerPeers(osi_socket socket, afs_uint32 remoteAddr, peer->bytesReceived.high = ntohl(peer->bytesReceived.high); peer->bytesReceived.low = ntohl(peer->bytesReceived.low); } +#else + afs_int32 rc = -1; #endif return rc; } -- 1.9.4