X-Git-Url: https://git.whamcloud.com/?a=blobdiff_plain;f=lustre%2Fptlrpc%2Fevents.c;h=4cc7a746dbc2d39e852a5be558057faba97057b9;hb=c0ab3403465d75c80c5486347741172dca9e0f2f;hp=ca4e566d97edbc6ea9eeb0f580dd63ca3565a1c2;hpb=4cab34af56c645cfbe2e20ded908e7a871b5cc1a;p=fs%2Flustre-release.git diff --git a/lustre/ptlrpc/events.c b/lustre/ptlrpc/events.c index ca4e566d..4cc7a74 100644 --- a/lustre/ptlrpc/events.c +++ b/lustre/ptlrpc/events.c @@ -37,6 +37,8 @@ static int request_out_callback(ptl_event_t *ev) { ENTRY; + LASSERT ((ev->mem_desc.options & PTL_MD_IOV) == 0); /* requests always contiguous */ + if (ev->type != PTL_EVENT_SENT) { // XXX make sure we understand all events, including ACK's CERROR("Unknown event %d\n", ev->type); @@ -54,6 +56,8 @@ static int reply_out_callback(ptl_event_t *ev) { ENTRY; + LASSERT ((ev->mem_desc.options & PTL_MD_IOV) == 0); /* replies always contiguous */ + if (ev->type == PTL_EVENT_SENT) { OBD_FREE(ev->mem_desc.start, ev->mem_desc.length); } else { @@ -73,7 +77,9 @@ static int reply_in_callback(ptl_event_t *ev) struct ptlrpc_request *req = ev->mem_desc.user_ptr; ENTRY; - if (req->rq_xid == 0x5a5a5a5a) { + LASSERT ((ev->mem_desc.options & PTL_MD_IOV) == 0); /* replies always contiguous */ + + if (req->rq_xid == 0x5a5a5a5a5a5a5a5a) { CERROR("Reply received for freed request! Probably a missing " "ptlrpc_abort()\n"); LBUG(); @@ -101,6 +107,8 @@ int request_in_callback(ptl_event_t *ev) { struct ptlrpc_service *service = ev->mem_desc.user_ptr; + LASSERT ((ev->mem_desc.options & PTL_MD_IOV) == 0); /* requests always contiguous */ + if (ev->rlength != ev->mlength) CERROR("Warning: Possibly truncated rpc (%d/%d)\n", ev->mlength, ev->rlength); @@ -115,22 +123,30 @@ int request_in_callback(ptl_event_t *ev) static int bulk_source_callback(ptl_event_t *ev) { - struct ptlrpc_bulk_page *bulk = ev->mem_desc.user_ptr; - struct ptlrpc_bulk_desc *desc = bulk->b_desc; + struct ptlrpc_bulk_desc *desc = ev->mem_desc.user_ptr; + struct ptlrpc_bulk_page *bulk; + struct list_head *tmp; + struct list_head *next; ENTRY; + /* 1 fragment for each page always */ + LASSERT (ev->mem_desc.niov == desc->b_page_count); + 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"); - 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(&desc->b_waitq); - if (desc->b_cb != NULL) - desc->b_cb(desc, desc->b_cb_data); + + list_for_each_safe(tmp, next, &desc->b_page_list) { + bulk = list_entry(tmp, struct ptlrpc_bulk_page, b_link); + + if (bulk->b_cb != NULL) + bulk->b_cb(bulk); } + desc->b_flags |= PTL_BULK_FL_SENT; + wake_up(&desc->b_waitq); + if (desc->b_cb != NULL) + desc->b_cb(desc, desc->b_cb_data); } else { CERROR("Unexpected event type!\n"); LBUG(); @@ -141,21 +157,36 @@ static int bulk_source_callback(ptl_event_t *ev) static int bulk_sink_callback(ptl_event_t *ev) { - struct ptlrpc_bulk_page *bulk = ev->mem_desc.user_ptr; - struct ptlrpc_bulk_desc *desc = bulk->b_desc; + struct ptlrpc_bulk_desc *desc = ev->mem_desc.user_ptr; + struct ptlrpc_bulk_page *bulk; + struct list_head *tmp; + struct list_head *next; + ptl_size_t total = 0; ENTRY; if (ev->type == PTL_EVENT_PUT) { - if (bulk->b_buf != ev->mem_desc.start + ev->offset) - CERROR("bulkbuf != mem_desc -- why?\n"); - if (bulk->b_cb != NULL) - bulk->b_cb(bulk); - if (atomic_dec_and_test(&desc->b_pages_remaining)) { - desc->b_flags |= PTL_BULK_FL_RCVD; - wake_up(&desc->b_waitq); - if (desc->b_cb != NULL) - desc->b_cb(desc, desc->b_cb_data); + /* put with zero offset */ + LASSERT (ev->offset == 0); + /* used iovs */ + LASSERT ((ev->mem_desc.options & PTL_MD_IOV) != 0); + /* 1 fragment for each page always */ + LASSERT (ev->mem_desc.niov == desc->b_page_count); + + list_for_each_safe (tmp, next, &desc->b_page_list) { + bulk = list_entry(tmp, struct ptlrpc_bulk_page, b_link); + + total += bulk->b_buflen; + + if (bulk->b_cb != NULL) + bulk->b_cb(bulk); } + + LASSERT (ev->mem_desc.length == total); + + desc->b_flags |= PTL_BULK_FL_RCVD; + wake_up(&desc->b_waitq); + if (desc->b_cb != NULL) + desc->b_cb(desc, desc->b_cb_data); } else { CERROR("Unexpected event type!\n"); LBUG(); @@ -182,23 +213,23 @@ int ptlrpc_init_portals(void) else ni = *socknal_nip; - rc = PtlEQAlloc(ni, 128, request_out_callback, &request_out_eq); + rc = PtlEQAlloc(ni, 1024, request_out_callback, &request_out_eq); if (rc != PTL_OK) CERROR("PtlEQAlloc failed: %d\n", rc); - rc = PtlEQAlloc(ni, 128, reply_out_callback, &reply_out_eq); + rc = PtlEQAlloc(ni, 1024, reply_out_callback, &reply_out_eq); if (rc != PTL_OK) CERROR("PtlEQAlloc failed: %d\n", rc); - rc = PtlEQAlloc(ni, 128, reply_in_callback, &reply_in_eq); + rc = PtlEQAlloc(ni, 1024, reply_in_callback, &reply_in_eq); if (rc != PTL_OK) CERROR("PtlEQAlloc failed: %d\n", rc); - rc = PtlEQAlloc(ni, 128, bulk_source_callback, &bulk_source_eq); + rc = PtlEQAlloc(ni, 1024, bulk_source_callback, &bulk_source_eq); if (rc != PTL_OK) CERROR("PtlEQAlloc failed: %d\n", rc); - rc = PtlEQAlloc(ni, 128, bulk_sink_callback, &bulk_sink_eq); + rc = PtlEQAlloc(ni, 1024, bulk_sink_callback, &bulk_sink_eq); if (rc != PTL_OK) CERROR("PtlEQAlloc failed: %d\n", rc);