Prevent race condition in accelerator copies management

Prevent the owner device from repurposing a GPU copy while another GPU is planning to use it as a candidate for a d2d transfer. Signed-off-by: George Bosilca <[email protected]>
ICLDisco · Oct 18, 2023 · f69460c · f69460c
1 parent ac4799d
commit f69460c
Show file tree

Hide file tree

Showing 2 changed files with 114 additions and 84 deletions.
diff --git a/parsec/data.c b/parsec/data.c
@@ -329,11 +329,11 @@ int parsec_data_start_transfer_ownership_to_copy(parsec_data_t* data,
 
     copy = data->device_copies[device];
     assert( NULL != copy );
-    
+
     PARSEC_DEBUG_VERBOSE(10, parsec_debug_output,
                          "DEV[%d]: start transfer ownership of data %p to copy %p in mode %d",
                          device, data, copy, access_mode);
-    
+
     switch( copy->coherency_state ) {
     case PARSEC_DATA_COHERENCY_INVALID:
         transfer_required = 1;
@@ -415,8 +415,6 @@ int parsec_data_start_transfer_ownership_to_copy(parsec_data_t* data,
         }
     }
 
-    assert( (!transfer_required) || (data->device_copies[valid_copy]->version >= copy->version) );
-
     if( PARSEC_FLOW_ACCESS_READ & access_mode ) {
         copy->readers++;
     }