+ prevent cudaFree after cudaDeviceReset

+ check included
chrxh · Aug 21, 2024 · 923705a · 923705a
1 parent 1b7b339
commit 923705a
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/source/EngineGpuKernels/CudaMemoryManager.cuh b/source/EngineGpuKernels/CudaMemoryManager.cuh
@@ -40,7 +40,7 @@ public:
         }
         auto findResult = _pointerToSizeMap.find(reinterpret_cast<void*>(memory));
         if (findResult != _pointerToSizeMap.end()) {
-            cudaFree(memory);
+            CHECK_FOR_CUDA_ERROR(cudaFree(memory));
             _bytes -= sizeof(T) * findResult->second;
             _pointerToSizeMap.erase(findResult->first);
         }

diff --git a/source/EngineGpuKernels/SimulationCudaFacade.cu b/source/EngineGpuKernels/SimulationCudaFacade.cu
@@ -91,15 +91,22 @@ _SimulationCudaFacade::~_SimulationCudaFacade()
     _cudaSimulationStatistics->free();
     _cudaSelectionResult->free();
 
+    _simulationKernels.reset();
+    _dataAccessKernels.reset();
+    _garbageCollectorKernels.reset();
+    _renderingKernels.reset();
+    _editKernels.reset();
+    _statisticsKernels.reset();
+
     CudaMemoryManager::getInstance().freeMemory(_cudaAccessTO->cells);
     CudaMemoryManager::getInstance().freeMemory(_cudaAccessTO->particles);
     CudaMemoryManager::getInstance().freeMemory(_cudaAccessTO->auxiliaryData);
     CudaMemoryManager::getInstance().freeMemory(_cudaAccessTO->numCells);
     CudaMemoryManager::getInstance().freeMemory(_cudaAccessTO->numParticles);
     CudaMemoryManager::getInstance().freeMemory(_cudaAccessTO->numAuxiliaryData);
 
-    cudaDeviceReset();
-    log(Priority::Important, "close simulation");
+    CHECK_FOR_CUDA_ERROR(cudaDeviceReset());
+    log(Priority::Important, "simulation closed");
 }
 
 void* _SimulationCudaFacade::registerImageResource(GLuint image)