From b04aa8f36dde2a8dcffc472a36bf4e8402361ca8 Mon Sep 17 00:00:00 2001
From: Manjunath-Jakaraddi <manjunath.jakaraddi@amd.com>
Date: Fri, 2 Feb 2024 13:12:24 -0800
Subject: [PATCH] SWDEV-419965: Enabling and fixing AyncCopyTest in
 runTracerFeatureTests

Change-Id: Iceeaae11b5367483ac03db0c500dd27f4037e53e


[ROCm/rocprofiler commit: a1f9b244c8050e56817627758a5ff9cb750e01ac]
---
 .../tests-v2/featuretests/tracer/apps/copy_on_engine.cpp      | 4 ++--
 .../rocprofiler/tests-v2/featuretests/tracer/tracer_gtest.cpp | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/projects/rocprofiler/tests-v2/featuretests/tracer/apps/copy_on_engine.cpp b/projects/rocprofiler/tests-v2/featuretests/tracer/apps/copy_on_engine.cpp
index 185a060efd..337d5a3533 100644
--- a/projects/rocprofiler/tests-v2/featuretests/tracer/apps/copy_on_engine.cpp
+++ b/projects/rocprofiler/tests-v2/featuretests/tracer/apps/copy_on_engine.cpp
@@ -194,7 +194,7 @@ static hsa_status_t AsyncCpyTest(async_mem_cpy_agent* dst, async_mem_cpy_agent*
 
   // Initialize the system and destination buffers with a value so we can later
   // validate it has been overwritten
-  void* sysPtr = args->cpu.ptr;
+  void* sysPtr = dst->ptr;
   err = hsa_amd_memory_fill(src->ptr, val, sz / sizeof(uint32_t));
   RET_IF_HSA_ERR(err);
 
@@ -303,7 +303,7 @@ int main() {
                                        reinterpret_cast<void**>(&args.gpu2.ptr));
     RET_IF_HSA_ERR(err);
     fprintf(stdout, "Copying %lu bytes from gpu1 memory to gpu2 memory...\n", sz);
-    err = AsyncCpyTest(&args.gpu1, &args.gpu2, &args, sz, kTestFillValue3);
+    err = AsyncCpyTest(&args.gpu2, &args.gpu1, &args, sz, kTestFillValue3);
     RET_IF_HSA_ERR(err);
     fprintf(stdout, "Success!\n");
   }
diff --git a/projects/rocprofiler/tests-v2/featuretests/tracer/tracer_gtest.cpp b/projects/rocprofiler/tests-v2/featuretests/tracer/tracer_gtest.cpp
index 704456ac0f..cf5a1ca1ee 100644
--- a/projects/rocprofiler/tests-v2/featuretests/tracer/tracer_gtest.cpp
+++ b/projects/rocprofiler/tests-v2/featuretests/tracer/tracer_gtest.cpp
@@ -303,7 +303,7 @@ class AsyncCopyTest : public Tracertest {
 
 // Test:1 Compares total num of kernel-names in golden output against current
 // tracer output
-TEST_F(AsyncCopyTest, DISABLED_WhenRunningTracerWithAppThenAsyncCopyOutputIsGenerated) {
+TEST_F(AsyncCopyTest, WhenRunningTracerWithAppThenAsyncCopyOutputIsGenerated) {
   // kernel info in current profler run
   std::vector<tracer_kernel_info_t> current_kernel_info;
 
@@ -312,7 +312,7 @@ TEST_F(AsyncCopyTest, DISABLED_WhenRunningTracerWithAppThenAsyncCopyOutputIsGene
 }
 
 // Test:2 Matches coelation Ids
-TEST_F(AsyncCopyTest, DISABLED_WhenRunningTracerWithAppThenAsyncCorelationCountIsCorrect) {
+TEST_F(AsyncCopyTest, WhenRunningTracerWithAppThenAsyncCorelationCountIsCorrect) {
   // kernel info in current profler run
   std::vector<tracer_kernel_info_t> current_kernel_info;