Commit 5e8f6920 authored by Peng Tao's avatar Peng Tao Committed by Greg Kroah-Hartman
Browse files

staging/lustre: remove HIPQUAD



Stephen Rothwell reported below error on powerpc:

In file included from drivers/staging/lustre/include/linux/libcfs/libcfs.h:203:0,
                 from drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.h:67,
                 from drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c:41:
drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c: In function 'kiblnd_dev_need_failover':
drivers/staging/lustre/include/linux/libcfs/libcfs_debug.h:215:16: error: implicit declaration of function 'NIPQUAD' [-Werror=implicit-function-declaration]
  static struct libcfs_debug_msg_data msgdata;      \
                ^
We should just remove HIPQUAD and replace it with %pI4h.

Reported-by: default avatarStephen Rothwell <sfr@canb.auug.org.au>
Cc: Joe Perches <joe@perches.com>
Signed-off-by: default avatarPeng Tao <bergwolf@gmail.com>
Signed-off-by: default avatarAndreas Dilger <andreas.dilger@intel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent c1042ed8
Loading
Loading
Loading
Loading
+0 −15
Original line number Diff line number Diff line
@@ -48,21 +48,6 @@

#include <net/sock.h>

#ifndef HIPQUAD
// XXX Should just kill all users
#if defined(__LITTLE_ENDIAN)
#define HIPQUAD(addr) \
	((unsigned char *)&addr)[3], \
	((unsigned char *)&addr)[2], \
	((unsigned char *)&addr)[1], \
	((unsigned char *)&addr)[0]
#elif defined(__BIG_ENDIAN)
#define HIPQUAD NIPQUAD
#else
#error "Please fix asm/byteorder.h"
#endif /* __LITTLE_ENDIAN */
#endif

typedef struct socket   socket_t;

#define SOCK_SNDBUF(so)	 ((so)->sk->sk_sndbuf)
+4 −4
Original line number Diff line number Diff line
@@ -2574,8 +2574,8 @@ kiblnd_dev_need_failover(kib_dev_t *dev)
	rc = rdma_resolve_addr(cmid, (struct sockaddr *)&srcaddr,
			       (struct sockaddr *)&dstaddr, 1);
	if (rc != 0 || cmid->device == NULL) {
		CERROR("Failed to bind %s:%u.%u.%u.%u to device(%p): %d\n",
		       dev->ibd_ifname, HIPQUAD(dev->ibd_ifip),
		CERROR("Failed to bind %s:%pI4h to device(%p): %d\n",
		       dev->ibd_ifname, &dev->ibd_ifip,
		       cmid->device, rc);
		rdma_destroy_id(cmid);
		return rc;
@@ -2647,8 +2647,8 @@ kiblnd_dev_failover(kib_dev_t *dev)
	/* Bind to failover device or port */
	rc = rdma_bind_addr(cmid, (struct sockaddr *)&addr);
	if (rc != 0 || cmid->device == NULL) {
		CERROR("Failed to bind %s:%u.%u.%u.%u to device(%p): %d\n",
		       dev->ibd_ifname, HIPQUAD(dev->ibd_ifip),
		CERROR("Failed to bind %s:%pI4h to device(%p): %d\n",
		       dev->ibd_ifname, &dev->ibd_ifip,
		       cmid->device, rc);
		rdma_destroy_id(cmid);
		goto out;
+6 −6
Original line number Diff line number Diff line
@@ -1319,9 +1319,9 @@ kiblnd_connect_peer (kib_peer_t *peer)
	}

	LASSERT (cmid->device != NULL);
	CDEBUG(D_NET, "%s: connection bound to %s:%u.%u.%u.%u:%s\n",
	CDEBUG(D_NET, "%s: connection bound to %s:%pI4h:%s\n",
	       libcfs_nid2str(peer->ibp_nid), dev->ibd_ifname,
	       HIPQUAD(dev->ibd_ifip), cmid->device->name);
	       &dev->ibd_ifip, cmid->device->name);

	return;

@@ -2209,8 +2209,8 @@ kiblnd_passive_connect (struct rdma_cm_id *cmid, void *priv, int priv_nob)
	if (*kiblnd_tunables.kib_require_priv_port &&
	    ntohs(peer_addr->sin_port) >= PROT_SOCK) {
		__u32 ip = ntohl(peer_addr->sin_addr.s_addr);
		CERROR("Peer's port (%u.%u.%u.%u:%hu) is not privileged\n",
		       HIPQUAD(ip), ntohs(peer_addr->sin_port));
		CERROR("Peer's port (%pI4h:%hu) is not privileged\n",
		       &ip, ntohs(peer_addr->sin_port));
		goto failed;
	}

@@ -2254,11 +2254,11 @@ kiblnd_passive_connect (struct rdma_cm_id *cmid, void *priv, int priv_nob)
	if (ni == NULL ||			 /* no matching net */
	    ni->ni_nid != reqmsg->ibm_dstnid ||   /* right NET, wrong NID! */
	    net->ibn_dev != ibdev) {	      /* wrong device */
		CERROR("Can't accept %s on %s (%s:%d:%u.%u.%u.%u): "
		CERROR("Can't accept %s on %s (%s:%d:%pI4h): "
		       "bad dst nid %s\n", libcfs_nid2str(nid),
		       ni == NULL ? "NA" : libcfs_nid2str(ni->ni_nid),
		       ibdev->ibd_ifname, ibdev->ibd_nnets,
		       HIPQUAD(ibdev->ibd_ifip),
		       &ibdev->ibd_ifip,
		       libcfs_nid2str(reqmsg->ibm_dstnid));

		goto failed;
+25 −25
Original line number Diff line number Diff line
@@ -334,17 +334,17 @@ ksocknal_associate_route_conn_locked(ksock_route_t *route, ksock_conn_t *conn)
	if (route->ksnr_myipaddr != conn->ksnc_myipaddr) {
		if (route->ksnr_myipaddr == 0) {
			/* route wasn't bound locally yet (the initial route) */
			CDEBUG(D_NET, "Binding %s %u.%u.%u.%u to %u.%u.%u.%u\n",
			CDEBUG(D_NET, "Binding %s %pI4h to %pI4h\n",
			       libcfs_id2str(peer->ksnp_id),
			       HIPQUAD(route->ksnr_ipaddr),
			       HIPQUAD(conn->ksnc_myipaddr));
			       &route->ksnr_ipaddr,
			       &conn->ksnc_myipaddr);
		} else {
			CDEBUG(D_NET, "Rebinding %s %u.%u.%u.%u from "
			       "%u.%u.%u.%u to %u.%u.%u.%u\n",
			CDEBUG(D_NET, "Rebinding %s %pI4h from "
			       "%pI4h to %pI4h\n",
			       libcfs_id2str(peer->ksnp_id),
			       HIPQUAD(route->ksnr_ipaddr),
			       HIPQUAD(route->ksnr_myipaddr),
			       HIPQUAD(conn->ksnc_myipaddr));
			       &route->ksnr_ipaddr,
			       &route->ksnr_myipaddr,
			       &conn->ksnc_myipaddr);

			iface = ksocknal_ip2iface(route->ksnr_peer->ksnp_ni,
						  route->ksnr_myipaddr);
@@ -384,9 +384,9 @@ ksocknal_add_route_locked (ksock_peer_t *peer, ksock_route_t *route)
		route2 = list_entry(tmp, ksock_route_t, ksnr_list);

		if (route2->ksnr_ipaddr == route->ksnr_ipaddr) {
			CERROR ("Duplicate route %s %u.%u.%u.%u\n",
			CERROR("Duplicate route %s %pI4h\n",
				libcfs_id2str(peer->ksnp_id),
				HIPQUAD(route->ksnr_ipaddr));
				&route->ksnr_ipaddr);
			LBUG();
		}
	}
@@ -982,8 +982,8 @@ ksocknal_accept (lnet_ni_t *ni, socket_t *sock)
	LIBCFS_ALLOC(cr, sizeof(*cr));
	if (cr == NULL) {
		LCONSOLE_ERROR_MSG(0x12f, "Dropping connection request from "
				   "%u.%u.%u.%u: memory exhausted\n",
				   HIPQUAD(peer_ip));
				   "%pI4h: memory exhausted\n",
				   &peer_ip);
		return -ENOMEM;
	}

@@ -1236,10 +1236,10 @@ ksocknal_create_conn (lnet_ni_t *ni, ksock_route_t *route,
	 * code below probably isn't going to work. */
	if (active &&
	    route->ksnr_ipaddr != conn->ksnc_ipaddr) {
		CERROR("Route %s %u.%u.%u.%u connected to %u.%u.%u.%u\n",
		CERROR("Route %s %pI4h connected to %pI4h\n",
		       libcfs_id2str(peer->ksnp_id),
		       HIPQUAD(route->ksnr_ipaddr),
		       HIPQUAD(conn->ksnc_ipaddr));
		       &route->ksnr_ipaddr,
		       &conn->ksnc_ipaddr);
	}

	/* Search for a route corresponding to the new connection and
@@ -1297,10 +1297,10 @@ ksocknal_create_conn (lnet_ni_t *ni, ksock_route_t *route,
	 *	socket callbacks.
	 */

	CDEBUG(D_NET, "New conn %s p %d.x %u.%u.%u.%u -> %u.%u.%u.%u/%d"
	CDEBUG(D_NET, "New conn %s p %d.x %pI4h -> %pI4h/%d"
	       " incarnation:"LPD64" sched[%d:%d]\n",
	       libcfs_id2str(peerid), conn->ksnc_proto->pro_version,
	       HIPQUAD(conn->ksnc_myipaddr), HIPQUAD(conn->ksnc_ipaddr),
	       &conn->ksnc_myipaddr, &conn->ksnc_ipaddr,
	       conn->ksnc_port, incarnation, cpt,
	       (int)(sched - &sched->kss_info->ksi_scheds[0]));

@@ -1648,10 +1648,10 @@ ksocknal_destroy_conn (ksock_conn_t *conn)
		last_rcv = conn->ksnc_rx_deadline -
			   cfs_time_seconds(*ksocknal_tunables.ksnd_timeout);
		CERROR("Completing partial receive from %s[%d]"
		       ", ip %d.%d.%d.%d:%d, with error, wanted: %d, left: %d, "
		       ", ip %pI4h:%d, with error, wanted: %d, left: %d, "
		       "last alive is %ld secs ago\n",
		       libcfs_id2str(conn->ksnc_peer->ksnp_id), conn->ksnc_type,
		       HIPQUAD(conn->ksnc_ipaddr), conn->ksnc_port,
		       &conn->ksnc_ipaddr, conn->ksnc_port,
		       conn->ksnc_rx_nob_wanted, conn->ksnc_rx_nob_left,
		       cfs_duration_sec(cfs_time_sub(cfs_time_current(),
					last_rcv)));
@@ -1661,25 +1661,25 @@ ksocknal_destroy_conn (ksock_conn_t *conn)
	case SOCKNAL_RX_LNET_HEADER:
		if (conn->ksnc_rx_started)
			CERROR("Incomplete receive of lnet header from %s"
			       ", ip %d.%d.%d.%d:%d, with error, protocol: %d.x.\n",
			       ", ip %pI4h:%d, with error, protocol: %d.x.\n",
			       libcfs_id2str(conn->ksnc_peer->ksnp_id),
			       HIPQUAD(conn->ksnc_ipaddr), conn->ksnc_port,
			       &conn->ksnc_ipaddr, conn->ksnc_port,
			       conn->ksnc_proto->pro_version);
		break;
	case SOCKNAL_RX_KSM_HEADER:
		if (conn->ksnc_rx_started)
			CERROR("Incomplete receive of ksock message from %s"
			       ", ip %d.%d.%d.%d:%d, with error, protocol: %d.x.\n",
			       ", ip %pI4h:%d, with error, protocol: %d.x.\n",
			       libcfs_id2str(conn->ksnc_peer->ksnp_id),
			       HIPQUAD(conn->ksnc_ipaddr), conn->ksnc_port,
			       &conn->ksnc_ipaddr, conn->ksnc_port,
			       conn->ksnc_proto->pro_version);
		break;
	case SOCKNAL_RX_SLOP:
		if (conn->ksnc_rx_started)
			CERROR("Incomplete receive of slops from %s"
			       ", ip %d.%d.%d.%d:%d, with error\n",
			       ", ip %pI4h:%d, with error\n",
			       libcfs_id2str(conn->ksnc_peer->ksnp_id),
			       HIPQUAD(conn->ksnc_ipaddr), conn->ksnc_port);
			       &conn->ksnc_ipaddr, conn->ksnc_port);
	       break;
	default:
		LBUG ();
+47 −47
Original line number Diff line number Diff line
@@ -553,21 +553,21 @@ ksocknal_process_transmit (ksock_conn_t *conn, ksock_tx_t *tx)
	if (!conn->ksnc_closing) {
		switch (rc) {
		case -ECONNRESET:
			LCONSOLE_WARN("Host %u.%u.%u.%u reset our connection "
			LCONSOLE_WARN("Host %pI4h reset our connection "
				      "while we were sending data; it may have "
				      "rebooted.\n",
				      HIPQUAD(conn->ksnc_ipaddr));
				      &conn->ksnc_ipaddr);
			break;
		default:
			LCONSOLE_WARN("There was an unexpected network error "
				      "while writing to %u.%u.%u.%u: %d.\n",
				      HIPQUAD(conn->ksnc_ipaddr), rc);
				      "while writing to %pI4h: %d.\n",
				      &conn->ksnc_ipaddr, rc);
			break;
		}
		CDEBUG(D_NET, "[%p] Error %d on write to %s"
		       " ip %d.%d.%d.%d:%d\n", conn, rc,
		       " ip %pI4h:%d\n", conn, rc,
		       libcfs_id2str(conn->ksnc_peer->ksnp_id),
		       HIPQUAD(conn->ksnc_ipaddr),
		       &conn->ksnc_ipaddr,
		       conn->ksnc_port);
	}

@@ -700,9 +700,9 @@ ksocknal_queue_tx_locked (ksock_tx_t *tx, ksock_conn_t *conn)
	 * ksnc_sock... */
	LASSERT(!conn->ksnc_closing);

	CDEBUG (D_NET, "Sending to %s ip %d.%d.%d.%d:%d\n",
	CDEBUG(D_NET, "Sending to %s ip %pI4h:%d\n",
		libcfs_id2str(conn->ksnc_peer->ksnp_id),
		HIPQUAD(conn->ksnc_ipaddr),
		&conn->ksnc_ipaddr,
		conn->ksnc_port);

	ksocknal_tx_prep(conn, tx);
@@ -801,9 +801,9 @@ ksocknal_find_connectable_route_locked (ksock_peer_t *peer)
		if (!(route->ksnr_retry_interval == 0 || /* first attempt */
		      cfs_time_aftereq(now, route->ksnr_timeout))) {
			CDEBUG(D_NET,
			       "Too soon to retry route %u.%u.%u.%u "
			       "Too soon to retry route %pI4h "
			       "(cnted %d, interval %ld, %ld secs later)\n",
			       HIPQUAD(route->ksnr_ipaddr),
			       &route->ksnr_ipaddr,
			       route->ksnr_connected,
			       route->ksnr_retry_interval,
			       cfs_duration_sec(route->ksnr_timeout - now));
@@ -1134,16 +1134,16 @@ ksocknal_process_receive (ksock_conn_t *conn)

			if (rc == 0)
				CDEBUG(D_NET, "[%p] EOF from %s"
					" ip %d.%d.%d.%d:%d\n", conn,
					" ip %pI4h:%d\n", conn,
					libcfs_id2str(conn->ksnc_peer->ksnp_id),
					HIPQUAD(conn->ksnc_ipaddr),
					&conn->ksnc_ipaddr,
					conn->ksnc_port);
			else if (!conn->ksnc_closing)
				CERROR("[%p] Error %d on read from %s"
					" ip %d.%d.%d.%d:%d\n",
					" ip %pI4h:%d\n",
					conn, rc,
					libcfs_id2str(conn->ksnc_peer->ksnp_id),
					HIPQUAD(conn->ksnc_ipaddr),
					&conn->ksnc_ipaddr,
					conn->ksnc_port);

			/* it's not an error if conn is being closed */
@@ -1722,8 +1722,8 @@ ksocknal_recv_hello (lnet_ni_t *ni, ksock_conn_t *conn,

	rc = libcfs_sock_read(sock, &hello->kshm_magic, sizeof (hello->kshm_magic), timeout);
	if (rc != 0) {
		CERROR ("Error %d reading HELLO from %u.%u.%u.%u\n",
			rc, HIPQUAD(conn->ksnc_ipaddr));
		CERROR("Error %d reading HELLO from %pI4h\n",
			rc, &conn->ksnc_ipaddr);
		LASSERT (rc < 0);
		return rc;
	}
@@ -1733,17 +1733,17 @@ ksocknal_recv_hello (lnet_ni_t *ni, ksock_conn_t *conn,
	    hello->kshm_magic != le32_to_cpu (LNET_PROTO_TCP_MAGIC)) {
		/* Unexpected magic! */
		CERROR("Bad magic(1) %#08x (%#08x expected) from "
			"%u.%u.%u.%u\n", __cpu_to_le32 (hello->kshm_magic),
			"%pI4h\n", __cpu_to_le32 (hello->kshm_magic),
			LNET_PROTO_TCP_MAGIC,
			HIPQUAD(conn->ksnc_ipaddr));
			&conn->ksnc_ipaddr);
		return -EPROTO;
	}

	rc = libcfs_sock_read(sock, &hello->kshm_version,
			      sizeof(hello->kshm_version), timeout);
	if (rc != 0) {
		CERROR ("Error %d reading HELLO from %u.%u.%u.%u\n",
			rc, HIPQUAD(conn->ksnc_ipaddr));
		CERROR("Error %d reading HELLO from %pI4h\n",
			rc, &conn->ksnc_ipaddr);
		LASSERT (rc < 0);
		return rc;
	}
@@ -1764,9 +1764,9 @@ ksocknal_recv_hello (lnet_ni_t *ni, ksock_conn_t *conn,
		}

		CERROR("Unknown protocol version (%d.x expected)"
			" from %u.%u.%u.%u\n",
			" from %pI4h\n",
			conn->ksnc_proto->pro_version,
			HIPQUAD(conn->ksnc_ipaddr));
			&conn->ksnc_ipaddr);

		return -EPROTO;
	}
@@ -1777,8 +1777,8 @@ ksocknal_recv_hello (lnet_ni_t *ni, ksock_conn_t *conn,
	/* receive the rest of hello message anyway */
	rc = conn->ksnc_proto->pro_recv_hello(conn, hello, timeout);
	if (rc != 0) {
		CERROR("Error %d reading or checking hello from from %u.%u.%u.%u\n",
		       rc, HIPQUAD(conn->ksnc_ipaddr));
		CERROR("Error %d reading or checking hello from from %pI4h\n",
		       rc, &conn->ksnc_ipaddr);
		LASSERT (rc < 0);
		return rc;
	}
@@ -1787,7 +1787,7 @@ ksocknal_recv_hello (lnet_ni_t *ni, ksock_conn_t *conn,

	if (hello->kshm_src_nid == LNET_NID_ANY) {
		CERROR("Expecting a HELLO hdr with a NID, but got LNET_NID_ANY"
		       "from %u.%u.%u.%u\n", HIPQUAD(conn->ksnc_ipaddr));
		       "from %pI4h\n", &conn->ksnc_ipaddr);
		return -EPROTO;
	}

@@ -1807,9 +1807,9 @@ ksocknal_recv_hello (lnet_ni_t *ni, ksock_conn_t *conn,
		/* peer determines type */
		conn->ksnc_type = ksocknal_invert_type(hello->kshm_ctype);
		if (conn->ksnc_type == SOCKLND_CONN_NONE) {
			CERROR ("Unexpected type %d from %s ip %u.%u.%u.%u\n",
			CERROR("Unexpected type %d from %s ip %pI4h\n",
				hello->kshm_ctype, libcfs_id2str(*peerid),
				HIPQUAD(conn->ksnc_ipaddr));
				&conn->ksnc_ipaddr);
			return -EPROTO;
		}

@@ -1819,11 +1819,11 @@ ksocknal_recv_hello (lnet_ni_t *ni, ksock_conn_t *conn,
	if (peerid->pid != recv_id.pid ||
	    peerid->nid != recv_id.nid) {
		LCONSOLE_ERROR_MSG(0x130, "Connected successfully to %s on host"
				   " %u.%u.%u.%u, but they claimed they were "
				   " %pI4h, but they claimed they were "
				   "%s; please check your Lustre "
				   "configuration.\n",
				   libcfs_id2str(*peerid),
				   HIPQUAD(conn->ksnc_ipaddr),
				   &conn->ksnc_ipaddr,
				   libcfs_id2str(recv_id));
		return -EPROTO;
	}
@@ -1834,9 +1834,9 @@ ksocknal_recv_hello (lnet_ni_t *ni, ksock_conn_t *conn,
	}

	if (ksocknal_invert_type(hello->kshm_ctype) != conn->ksnc_type) {
		CERROR ("Mismatched types: me %d, %s ip %u.%u.%u.%u %d\n",
		CERROR("Mismatched types: me %d, %s ip %pI4h %d\n",
			conn->ksnc_type, libcfs_id2str(*peerid),
			HIPQUAD(conn->ksnc_ipaddr),
			&conn->ksnc_ipaddr,
			hello->kshm_ctype);
		return -EPROTO;
	}
@@ -2208,8 +2208,8 @@ ksocknal_connd (void *arg)
				/* consecutive retry */
				if (cons_retry++ > SOCKNAL_INSANITY_RECONN) {
					CWARN("massive consecutive "
					      "re-connecting to %u.%u.%u.%u\n",
					      HIPQUAD(route->ksnr_ipaddr));
					      "re-connecting to %pI4h\n",
					      &route->ksnr_ipaddr);
					cons_retry = 0;
				}
			} else {
@@ -2274,26 +2274,26 @@ ksocknal_find_timed_out_conn (ksock_peer_t *peer)
			switch (error) {
			case ECONNRESET:
				CNETERR("A connection with %s "
					"(%u.%u.%u.%u:%d) was reset; "
					"(%pI4h:%d) was reset; "
					"it may have rebooted.\n",
					libcfs_id2str(peer->ksnp_id),
					HIPQUAD(conn->ksnc_ipaddr),
					&conn->ksnc_ipaddr,
					conn->ksnc_port);
				break;
			case ETIMEDOUT:
				CNETERR("A connection with %s "
					"(%u.%u.%u.%u:%d) timed out; the "
					"(%pI4h:%d) timed out; the "
					"network or node may be down.\n",
					libcfs_id2str(peer->ksnp_id),
					HIPQUAD(conn->ksnc_ipaddr),
					&conn->ksnc_ipaddr,
					conn->ksnc_port);
				break;
			default:
				CNETERR("An unexpected network error %d "
					"occurred with %s "
					"(%u.%u.%u.%u:%d\n", error,
					"(%pI4h:%d\n", error,
					libcfs_id2str(peer->ksnp_id),
					HIPQUAD(conn->ksnc_ipaddr),
					&conn->ksnc_ipaddr,
					conn->ksnc_port);
				break;
			}
@@ -2306,10 +2306,10 @@ ksocknal_find_timed_out_conn (ksock_peer_t *peer)
				     conn->ksnc_rx_deadline)) {
			/* Timed out incomplete incoming message */
			ksocknal_conn_addref(conn);
			CNETERR("Timeout receiving from %s (%u.%u.%u.%u:%d), "
			CNETERR("Timeout receiving from %s (%pI4h:%d), "
				"state %d wanted %d left %d\n",
				libcfs_id2str(peer->ksnp_id),
				HIPQUAD(conn->ksnc_ipaddr),
				&conn->ksnc_ipaddr,
				conn->ksnc_port,
				conn->ksnc_rx_state,
				conn->ksnc_rx_nob_wanted,
@@ -2324,10 +2324,10 @@ ksocknal_find_timed_out_conn (ksock_peer_t *peer)
			/* Timed out messages queued for sending or
			 * buffered in the socket's send buffer */
			ksocknal_conn_addref(conn);
			CNETERR("Timeout sending data to %s (%u.%u.%u.%u:%d) "
			CNETERR("Timeout sending data to %s (%pI4h:%d) "
				"the network or that node may be down.\n",
				libcfs_id2str(peer->ksnp_id),
				HIPQUAD(conn->ksnc_ipaddr),
				&conn->ksnc_ipaddr,
				conn->ksnc_port);
			return (conn);
		}
Loading