From 64dae113b12a77ce314b19b4ee8d1f7f9e7c0c83 Mon Sep 17 00:00:00 2001 From: Sean Keely Date: Fri, 22 Apr 2022 21:03:58 -0500 Subject: [PATCH] Minor typo fixes. Declare the type of HSA_AMD_AGENT_INFO_COOPERATIVE_COMPUTE_UNIT_COUNT and add a missing break statement. Change-Id: I86ce8a2e620438e046b60cee991ce1fbe07a3e88 --- runtime/hsa-runtime/core/runtime/amd_gpu_agent.cpp | 1 + runtime/hsa-runtime/inc/hsa_ext_amd.h | 1 + 2 files changed, 2 insertions(+) diff --git a/runtime/hsa-runtime/core/runtime/amd_gpu_agent.cpp b/runtime/hsa-runtime/core/runtime/amd_gpu_agent.cpp index 0bddcc4e2c..d937651be4 100644 --- a/runtime/hsa-runtime/core/runtime/amd_gpu_agent.cpp +++ b/runtime/hsa-runtime/core/runtime/amd_gpu_agent.cpp @@ -1015,6 +1015,7 @@ hsa_status_t GpuAgent::GetInfo(hsa_agent_info_t attribute, void* value) const { case HSA_AMD_AGENT_INFO_SVM_DIRECT_HOST_ACCESS: assert(regions_.size() != 0 && "No device local memory found!"); *((bool*)value) = properties_.Capability.ui32.CoherentHostAccess == 1; + break; case HSA_AMD_AGENT_INFO_COOPERATIVE_COMPUTE_UNIT_COUNT: if (core::Runtime::runtime_singleton_->flag().coop_cu_count() && (isa_->GetMajorVersion() == 9) && (isa_->GetMinorVersion() == 0) && diff --git a/runtime/hsa-runtime/inc/hsa_ext_amd.h b/runtime/hsa-runtime/inc/hsa_ext_amd.h index 6c1a936e5f..fb4facfb46 100644 --- a/runtime/hsa-runtime/inc/hsa_ext_amd.h +++ b/runtime/hsa-runtime/inc/hsa_ext_amd.h @@ -316,6 +316,7 @@ typedef enum hsa_amd_agent_info_s { * Some processors support more CUs than can reliably be used in a cooperative * dispatch. This queries the count of CUs which are fully enabled for * cooperative dispatch. + * The type of this attribute is uint32_t. */ HSA_AMD_AGENT_INFO_COOPERATIVE_COMPUTE_UNIT_COUNT = 0xA014 } hsa_amd_agent_info_t;