X-Git-Url: https://git.whamcloud.com/?a=blobdiff_plain;f=lustre%2Fptlrpc%2Fevents.c;h=b2ae37e842d7ace819a9d0af8f6bbf0ec82807da;hb=c2f0e8398d39660a0c0ac02c1fa12faa8b360c5b;hp=5cf37fbd283ed799052dd2c77c57a6ba203ba789;hpb=726c9cf39547151a7cf85118c8a23b9ecf8220f5;p=fs%2Flustre-release.git diff --git a/lustre/ptlrpc/events.c b/lustre/ptlrpc/events.c index 5cf37fb..b2ae37e 100644 --- a/lustre/ptlrpc/events.c +++ b/lustre/ptlrpc/events.c @@ -20,41 +20,25 @@ * */ -#define EXPORT_SYMTAB - -#include -#include -#include - #define DEBUG_SUBSYSTEM S_RPC -#include -#include +#include #include -ptl_handle_eq_t request_out_eq, - reply_in_eq, - reply_out_eq, - bulk_source_eq, - bulk_sink_eq; +ptl_handle_eq_t request_out_eq, reply_in_eq, reply_out_eq, bulk_source_eq, + bulk_sink_eq; static const ptl_handle_ni_t *socknal_nip = NULL, *qswnal_nip = NULL; /* * Free the packet when it has gone out */ -static int request_out_callback(ptl_event_t *ev, void *data) +static int request_out_callback(ptl_event_t *ev) { - struct ptlrpc_request *req = ev->mem_desc.user_ptr; - struct ptlrpc_client *cl = req->rq_client; - ENTRY; - if (ev->type == PTL_EVENT_SENT) { - list_del(&req->rq_list); - list_add(&req->rq_list, &cl->cli_sent_head); - } else { + if (ev->type != PTL_EVENT_SENT) { // XXX make sure we understand all events, including ACK's - CERROR("Unknown event %d\n", ev->type); + CERROR("Unknown event %d\n", ev->type); LBUG(); } @@ -65,15 +49,15 @@ static int request_out_callback(ptl_event_t *ev, void *data) /* * Free the packet when it has gone out */ -static int reply_out_callback(ptl_event_t *ev, void *data) +static int reply_out_callback(ptl_event_t *ev) { ENTRY; if (ev->type == PTL_EVENT_SENT) { OBD_FREE(ev->mem_desc.start, ev->mem_desc.length); - } else { + } else { // XXX make sure we understand all events, including ACK's - CERROR("Unknown event %d\n", ev->type); + CERROR("Unknown event %d\n", ev->type); LBUG(); } @@ -83,61 +67,43 @@ static int reply_out_callback(ptl_event_t *ev, void *data) /* * Wake up the thread waiting for the reply once it comes in. */ -static int reply_in_callback(ptl_event_t *ev, void *data) +static int reply_in_callback(ptl_event_t *ev) { - struct ptlrpc_request *rpc = ev->mem_desc.user_ptr; + struct ptlrpc_request *req = ev->mem_desc.user_ptr; ENTRY; + if (req->rq_xid == 0x5a5a5a5a) { + CERROR("Reply received for freed request! Probably a missing " + "ptlrpc_abort()\n"); + LBUG(); + } + + if (req->rq_xid != ev->match_bits) { + CERROR("Reply packet for wrong request\n"); + LBUG(); + } + if (ev->type == PTL_EVENT_PUT) { - rpc->rq_repbuf = ev->mem_desc.start + ev->offset; + req->rq_repmsg = ev->mem_desc.start + ev->offset; barrier(); - wake_up_interruptible(&rpc->rq_wait_for_rep); - } else { + wake_up_interruptible(&req->rq_wait_for_rep); + } else { // XXX make sure we understand all events, including ACK's - CERROR("Unknown event %d\n", ev->type); + CERROR("Unknown event %d\n", ev->type); LBUG(); } RETURN(1); } -int request_in_callback(ptl_event_t *ev, void *data) +int request_in_callback(ptl_event_t *ev) { - struct ptlrpc_service *service = data; - int index; + struct ptlrpc_service *service = ev->mem_desc.user_ptr; if (ev->rlength != ev->mlength) CERROR("Warning: Possibly truncated rpc (%d/%d)\n", ev->mlength, ev->rlength); - spin_lock(&service->srv_lock); - for (index = 0; index < service->srv_ring_length; index++) - if ( service->srv_buf[index] == ev->mem_desc.start) - break; - - if (index == service->srv_ring_length) - LBUG(); - - service->srv_ref_count[index]++; - - if (ptl_is_valid_handle(&ev->unlinked_me)) { - int idx; - - for (idx = 0; idx < service->srv_ring_length; idx++) - if (service->srv_me_h[idx].handle_idx == - ev->unlinked_me.handle_idx) - break; - if (idx == service->srv_ring_length) - LBUG(); - - CDEBUG(D_NET, "unlinked %d\n", idx); - ptl_set_inv_handle(&(service->srv_me_h[idx])); - - if (service->srv_ref_count[idx] == 0) - ptlrpc_link_svc_me(service, idx); - } - - spin_unlock(&service->srv_lock); if (ev->type == PTL_EVENT_PUT) wake_up(&service->srv_waitq); else @@ -146,51 +112,55 @@ int request_in_callback(ptl_event_t *ev, void *data) return 0; } -static int bulk_source_callback(ptl_event_t *ev, void *data) +static int bulk_source_callback(ptl_event_t *ev) { - struct ptlrpc_bulk_desc *bulk = ev->mem_desc.user_ptr; - + struct ptlrpc_bulk_page *bulk = ev->mem_desc.user_ptr; + struct ptlrpc_bulk_desc *desc = bulk->b_desc; ENTRY; if (ev->type == PTL_EVENT_SENT) { CDEBUG(D_NET, "got SENT event\n"); } else if (ev->type == PTL_EVENT_ACK) { CDEBUG(D_NET, "got ACK event\n"); - bulk->b_flags = PTL_BULK_SENT; - wake_up_interruptible(&bulk->b_waitq); + if (bulk->b_cb != NULL) + bulk->b_cb(bulk); + if (atomic_dec_and_test(&desc->b_pages_remaining)) { + desc->b_flags |= PTL_BULK_FL_SENT; + wake_up_interruptible(&desc->b_waitq); + if (desc->b_cb != NULL) + desc->b_cb(desc, desc->b_cb_data); + } } else { CERROR("Unexpected event type!\n"); LBUG(); } - EXIT; - return 1; + RETURN(1); } -static int bulk_sink_callback(ptl_event_t *ev, void *data) +static int bulk_sink_callback(ptl_event_t *ev) { - struct ptlrpc_bulk_desc *bulk = ev->mem_desc.user_ptr; - + struct ptlrpc_bulk_page *bulk = ev->mem_desc.user_ptr; + struct ptlrpc_bulk_desc *desc = bulk->b_desc; ENTRY; if (ev->type == PTL_EVENT_PUT) { if (bulk->b_buf != ev->mem_desc.start + ev->offset) CERROR("bulkbuf != mem_desc -- why?\n"); - bulk->b_flags = PTL_BULK_RCVD; if (bulk->b_cb != NULL) - bulk->b_cb(bulk, data); - wake_up_interruptible(&bulk->b_waitq); + bulk->b_cb(bulk); + if (atomic_dec_and_test(&desc->b_pages_remaining)) { + desc->b_flags |= PTL_BULK_FL_RCVD; + wake_up_interruptible(&desc->b_waitq); + if (desc->b_cb != NULL) + desc->b_cb(desc, desc->b_cb_data); + } } else { CERROR("Unexpected event type!\n"); LBUG(); } - /* FIXME: This should happen unconditionally */ - if (bulk->b_cb != NULL) - OBD_FREE(bulk, sizeof(*bulk)); - - EXIT; - return 1; + RETURN(1); } int ptlrpc_init_portals(void) @@ -211,23 +181,23 @@ int ptlrpc_init_portals(void) else ni = *socknal_nip; - rc = PtlEQAlloc(ni, 128, request_out_callback, NULL, &request_out_eq); + rc = PtlEQAlloc(ni, 128, request_out_callback, &request_out_eq); if (rc != PTL_OK) CERROR("PtlEQAlloc failed: %d\n", rc); - rc = PtlEQAlloc(ni, 128, reply_out_callback, NULL, &reply_out_eq); + rc = PtlEQAlloc(ni, 128, reply_out_callback, &reply_out_eq); if (rc != PTL_OK) CERROR("PtlEQAlloc failed: %d\n", rc); - rc = PtlEQAlloc(ni, 128, reply_in_callback, NULL, &reply_in_eq); + rc = PtlEQAlloc(ni, 128, reply_in_callback, &reply_in_eq); if (rc != PTL_OK) CERROR("PtlEQAlloc failed: %d\n", rc); - rc = PtlEQAlloc(ni, 128, bulk_source_callback, NULL, &bulk_source_eq); + rc = PtlEQAlloc(ni, 128, bulk_source_callback, &bulk_source_eq); if (rc != PTL_OK) CERROR("PtlEQAlloc failed: %d\n", rc); - rc = PtlEQAlloc(ni, 128, bulk_sink_callback, NULL, &bulk_sink_eq); + rc = PtlEQAlloc(ni, 128, bulk_sink_callback, &bulk_sink_eq); if (rc != PTL_OK) CERROR("PtlEQAlloc failed: %d\n", rc);