X-Git-Url: https://git.whamcloud.com/?a=blobdiff_plain;f=lustre%2Fptlrpc%2Fconnection.c;h=2e0d889da6fd6386ac9b2a8e072a3c061ffa39af;hb=e44e9bbc5804c33aabdd1a351f2123da72110c20;hp=8acb50421a56765324dbed6278626fdd410f34ae;hpb=981b793ba1757b480b95488575aa0bd27678eec8;p=fs%2Flustre-release.git diff --git a/lustre/ptlrpc/connection.c b/lustre/ptlrpc/connection.c index 8acb504..2e0d889 100644 --- a/lustre/ptlrpc/connection.c +++ b/lustre/ptlrpc/connection.c @@ -21,24 +21,54 @@ */ #define DEBUG_SUBSYSTEM S_RPC - +#ifdef __KERNEL__ #include +#include #include +#else +#include +#endif + +#include "ptlrpc_internal.h" static spinlock_t conn_lock; static struct list_head conn_list; static struct list_head conn_unused_list; -struct ptlrpc_connection *ptlrpc_get_connection(struct lustre_peer *peer) +void ptlrpc_dump_connections(void) +{ + char str[PTL_NALFMT_SIZE]; + struct list_head *tmp; + struct ptlrpc_connection *c; + ENTRY; + + list_for_each(tmp, &conn_list) { + c = list_entry(tmp, struct ptlrpc_connection, c_link); + CERROR("Connection %p/%s has refcount %d (nid=%s on %s)\n", + c, c->c_remote_uuid.uuid, atomic_read(&c->c_refcount), + ptlrpc_peernid2str(&c->c_peer, str), + c->c_peer.peer_ni->pni_name); + } + EXIT; +} + +struct ptlrpc_connection *ptlrpc_get_connection(struct ptlrpc_peer *peer, + struct obd_uuid *uuid) { + char str[PTL_NALFMT_SIZE]; struct list_head *tmp, *pos; struct ptlrpc_connection *c; ENTRY; + + CDEBUG(D_INFO, "peer is %s on %s\n", + ptlrpc_peernid2str(peer, str), peer->peer_ni->pni_name); + spin_lock(&conn_lock); list_for_each(tmp, &conn_list) { c = list_entry(tmp, struct ptlrpc_connection, c_link); - if (memcmp(peer, &c->c_peer, sizeof(*peer)) == 0) { + if (peer->peer_nid == c->c_peer.peer_nid && + peer->peer_ni == c->c_peer.peer_ni) { ptlrpc_connection_addref(c); GOTO(out, c); } @@ -46,7 +76,8 @@ struct ptlrpc_connection *ptlrpc_get_connection(struct lustre_peer *peer) list_for_each_safe(tmp, pos, &conn_unused_list) { c = list_entry(tmp, struct ptlrpc_connection, c_link); - if (memcmp(peer, &c->c_peer, sizeof(*peer)) == 0) { + if (peer->peer_nid == c->c_peer.peer_nid && + peer->peer_ni == c->c_peer.peer_ni) { ptlrpc_connection_addref(c); list_del(&c->c_link); list_add(&c->c_link, &conn_list); @@ -56,26 +87,18 @@ struct ptlrpc_connection *ptlrpc_get_connection(struct lustre_peer *peer) /* FIXME: this should be a slab once we can validate slab addresses * without OOPSing */ - OBD_ALLOC(c, sizeof(*c)); + OBD_ALLOC_GFP(c, sizeof(*c), GFP_ATOMIC); + if (c == NULL) GOTO(out, c); - c->c_level = LUSTRE_CONN_NEW; - c->c_xid_in = 1; - c->c_xid_out = 1; - c->c_generation = 1; - c->c_epoch = 1; - c->c_bootcount = 0; - INIT_LIST_HEAD(&c->c_delayed_head); - INIT_LIST_HEAD(&c->c_sending_head); - INIT_LIST_HEAD(&c->c_dying_head); - INIT_LIST_HEAD(&c->c_clients); - INIT_LIST_HEAD(&c->c_exports); + if (uuid && uuid->uuid) /* XXX ???? */ + obd_str2uuid(&c->c_remote_uuid, uuid->uuid); atomic_set(&c->c_refcount, 0); + memcpy(&c->c_peer, peer, sizeof(c->c_peer)); + ptlrpc_connection_addref(c); - spin_lock_init(&c->c_lock); - memcpy(&c->c_peer, peer, sizeof(c->c_peer)); list_add(&c->c_link, &conn_list); EXIT; @@ -86,11 +109,20 @@ struct ptlrpc_connection *ptlrpc_get_connection(struct lustre_peer *peer) int ptlrpc_put_connection(struct ptlrpc_connection *c) { + char str[PTL_NALFMT_SIZE]; int rc = 0; ENTRY; - CDEBUG(D_INFO, "connection=%p refcount %d\n", - c, atomic_read(&c->c_refcount) - 1); + if (c == NULL) { + CERROR("NULL connection\n"); + RETURN(0); + } + + CDEBUG (D_INFO, "connection=%p refcount %d to %s on %s\n", + c, atomic_read(&c->c_refcount) - 1, + ptlrpc_peernid2str(&c->c_peer, str), + c->c_peer.peer_ni->pni_name); + if (atomic_dec_and_test(&c->c_refcount)) { spin_lock(&conn_lock); list_del(&c->c_link); @@ -107,10 +139,13 @@ int ptlrpc_put_connection(struct ptlrpc_connection *c) struct ptlrpc_connection *ptlrpc_connection_addref(struct ptlrpc_connection *c) { + char str[PTL_NALFMT_SIZE]; ENTRY; - CDEBUG(D_INFO, "connection=%p refcount %d\n", - c, atomic_read(&c->c_refcount) + 1); atomic_inc(&c->c_refcount); + CDEBUG (D_INFO, "connection=%p refcount %d to %s on %s\n", + c, atomic_read(&c->c_refcount), + ptlrpc_peernid2str(&c->c_peer, str), + c->c_peer.peer_ni->pni_name); RETURN(c); } @@ -123,6 +158,7 @@ void ptlrpc_init_connection(void) void ptlrpc_cleanup_connection(void) { + char str[PTL_NALFMT_SIZE]; struct list_head *tmp, *pos; struct ptlrpc_connection *c; @@ -134,9 +170,10 @@ void ptlrpc_cleanup_connection(void) } list_for_each_safe(tmp, pos, &conn_list) { c = list_entry(tmp, struct ptlrpc_connection, c_link); - CERROR("Connection %p has refcount %d at cleanup (nid=%lu)!\n", - c, atomic_read(&c->c_refcount), - (unsigned long)c->c_peer.peer_nid); + CERROR("Connection %p/%s has refcount %d (nid=%s on %s)\n", + c, c->c_remote_uuid.uuid, atomic_read(&c->c_refcount), + ptlrpc_peernid2str(&c->c_peer, str), + c->c_peer.peer_ni->pni_name); list_del(&c->c_link); OBD_FREE(c, sizeof(*c)); }