Skip to content

Commit bfcf145

Browse files
committed
Refactor the request test and wait functions.
1 parent d9fb59b commit bfcf145

File tree

2 files changed

+119
-121
lines changed

2 files changed

+119
-121
lines changed

ompi/request/req_test.c

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
/* -*- Mode: C; c-basic-offset:4 ; indent-tabs-mode:nil -*- */
12
/*
23
* Copyright (c) 2004-2007 The Trustees of Indiana University and Indiana
34
* University Research and Technology
@@ -27,11 +28,12 @@
2728

2829
#include "ompi/mca/crcp/crcp.h"
2930

30-
int ompi_request_default_test( ompi_request_t ** rptr,
31-
int *completed,
32-
ompi_status_public_t * status )
31+
int ompi_request_default_test(ompi_request_t ** rptr,
32+
int *completed,
33+
ompi_status_public_t * status )
3334
{
3435
ompi_request_t *request = *rptr;
36+
3537
#if OPAL_ENABLE_PROGRESS_THREADS == 0
3638
int do_it_once = 0;
3739

@@ -46,7 +48,7 @@ int ompi_request_default_test( ompi_request_t ** rptr,
4648
return OMPI_SUCCESS;
4749
}
4850

49-
if (request->req_complete) {
51+
if( REQUEST_COMPLETE(request) ) {
5052
OMPI_CRCP_REQUEST_COMPLETE(request);
5153

5254
*completed = true;
@@ -117,7 +119,7 @@ int ompi_request_default_test_any(
117119
continue;
118120
}
119121

120-
if( request->req_complete ) {
122+
if( REQUEST_COMPLETE(request) ) {
121123
OMPI_CRCP_REQUEST_COMPLETE(request);
122124

123125
*index = i;
@@ -193,8 +195,7 @@ int ompi_request_default_test_all(
193195
request = *rptr;
194196

195197
if( request->req_state == OMPI_REQUEST_INACTIVE ||
196-
request->req_complete) {
197-
OMPI_CRCP_REQUEST_COMPLETE(request);
198+
REQUEST_COMPLETE(request) ) {
198199
num_completed++;
199200
}
200201
}
@@ -225,6 +226,7 @@ int ompi_request_default_test_all(
225226
if (OMPI_REQUEST_GEN == request->req_type) {
226227
ompi_grequest_invoke_query(request, &request->req_status);
227228
}
229+
OMPI_CRCP_REQUEST_COMPLETE(request);
228230
statuses[i] = request->req_status;
229231
if( request->req_persistent ) {
230232
request->req_state = OMPI_REQUEST_INACTIVE;
@@ -255,6 +257,7 @@ int ompi_request_default_test_all(
255257
if (OMPI_REQUEST_GEN == request->req_type) {
256258
ompi_grequest_invoke_query(request, &request->req_status);
257259
}
260+
OMPI_CRCP_REQUEST_COMPLETE(request);
258261
if( request->req_persistent ) {
259262
request->req_state = OMPI_REQUEST_INACTIVE;
260263
continue;

0 commit comments

Comments
 (0)