From 884fed4f045d6dcc5f67cb1245fbe4bc818f9f1e Mon Sep 17 00:00:00 2001 From: Sean Keely Date: Thu, 26 Mar 2020 16:22:25 -0500 Subject: [PATCH] Correct initial kfd_open_count increment. Don't set kfd_open_count=1 unless hsaKmtOpenKFD actually succeeds. This prevents returning HSAKMT_STATUS_KERNEL_ALREADY_OPENED in subsequent calls when KFD is actually closed. Signed-off-by: Sean Keely Change-Id: Ia870b5faa8626826a6c8795aa10784d376cf2e80 --- src/openclose.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/openclose.c b/src/openclose.c index eb9b6080a0..72dfd66a1f 100644 --- a/src/openclose.c +++ b/src/openclose.c @@ -194,7 +194,6 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtOpenKFD(void) } kfd_fd = fd; - kfd_open_count = 1; init_page_size(); @@ -210,6 +209,8 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtOpenKFD(void) if (result != HSAKMT_STATUS_SUCCESS) goto init_doorbell_failed; + kfd_open_count = 1; + if (init_device_debugging_memory(sys_props.NumNodes) != HSAKMT_STATUS_SUCCESS) pr_warn("Insufficient Memory. Debugging unavailable\n");