summaryrefslogtreecommitdiff
path: root/CHANGES
diff options
context:
space:
mode:
authorTayler Hetherington <[email protected]>2013-06-17 15:05:14 -0800
committerAndrew Boktor <[email protected]>2014-08-14 13:50:47 -0700
commitd79fab8098036dcad4628c21d61258195721e2a2 (patch)
treea0967e13942bd7ca6e9d53e61d1294b25bff5af0 /CHANGES
parent564de5650af4e54b02f954f47c363ca1da7732f9 (diff)
Review: 46001. Fixing a source of non-determinism in GPGPU-Sim (Bug 174). Not sure if this is the only issue, but this was definitely a source of non-determinism (checked with many of the <rodinia>-FT benchmarks). In gpgpu_sim_thread_concurrent() (gpgpusim_entrypoint.cc), the inner do{...}while(active); only breaks when the gpu is not active. As a result, the gpu is only initialized when the gpu becomes momentarily inactive. If one kernel completes while another kernel is currently pending in the stream's queue, the next kernel will start running immediately causing the GPU to not be reinitialized (which includes resetting per-kernel stats). The fix simply recognizes that a kernel has completed and breaks from the loop prior to starting the next operation.
[git-p4: depot-paths = "//depot/gpgpu_sim_research/fermi/distribution/": change = 16463]
Diffstat (limited to 'CHANGES')
-rw-r--r--CHANGES1
1 files changed, 1 insertions, 0 deletions
diff --git a/CHANGES b/CHANGES
index 37bc9c1..a7ea699 100644
--- a/CHANGES
+++ b/CHANGES
@@ -38,6 +38,7 @@ Version 3.2.1+edits (development branch) versus 3.2.1
- Average/min/max per-kernel powers not being reset at kernel boundaries causing
incorrect per-kernel values.
- Fixed a dependency error in the src/cuda-sim Makefile.
+ - Fixing a source of non-determinism in GPGPU-Sim (Bug 147).
Version 3.2.1 versus 3.2.0
- Added kernel name and launch uids to performance statistics log.