Skip to content

GitLab

Unverified Commit 6771015c authored by gilbertlee-amd's avatar gilbertlee-amd Committed by GitHub
Browse files

Fixing bug with non-default cpu memory types (#5) 


Co-authored-by: default avatarGilbert Lee <gilbertlee-amd@github.com>
parent be0db711
Show whitespace changes
Inline Side-by-side
Showing with 6 additions and 2 deletions
CHANGELOG.md
View file @ 6771015c
# Changelog for TransferBench # Changelog for TransferBench
## v1.07
### Changed
- Fix bug with allocations involving non-default CPU memory types
## v1.06 ## v1.06
### Added ### Added
- Added unpinned CPU memory type ('U'). May require HSA_XNACK=1 in order to access via GPU executors - Added unpinned CPU memory type ('U'). May require HSA_XNACK=1 in order to access via GPU executors
......
EnvVars.hpp
View file @ 6771015c
...@@ -26,7 +26,7 @@ THE SOFTWARE. ...@@ -26,7 +26,7 @@ THE SOFTWARE.
#include <algorithm> #include <algorithm>
#include <random> #include <random>
#include <time.h> #include <time.h>
#define TB_VERSION "1.06" #define TB_VERSION "1.07"
extern char const MemTypeStr[]; extern char const MemTypeStr[];
......
TransferBench.cpp
View file @ 6771015c
...@@ -546,7 +546,7 @@ int RemappedIndex(int const origIdx, MemType const memType) ...@@ -546,7 +546,7 @@ int RemappedIndex(int const origIdx, MemType const memType)
static std::vector<int> remapping; static std::vector<int> remapping;
// No need to re-map CPU devices // No need to re-map CPU devices
if (memType == MEM_CPU) return origIdx; if (IsCpuType(memType)) return origIdx;
// Build remapping on first use // Build remapping on first use
if (remapping.empty()) if (remapping.empty())
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment