change default rocprof version to v3 when not setting env variable (#673)

Author: ywang103-amd

CHANGELOG.md

@@ -47,6 +47,7 @@ Full documentation for ROCm Compute Profiler is available at [https://rocm.docs.
 
 ### Changed
 
+* Change the default rocprof version to v3 when environment variable "ROCPROF" is not set
 * Change the rocprof version for unit tests to rocprofv3 on all SoCs except MI100
 * Change normal_unit default to per_kernel
 * Change dependency from rocm-smi to amd-smi

src/utils/utils.py

@@ -70,14 +70,13 @@ def get_base_spi_pipe_counter(counter):
 
 
 def using_v1():
-
-    return "ROCPROF" not in os.environ.keys() or (
-        "ROCPROF" in os.environ.keys() and os.environ["ROCPROF"].endswith("rocprof")
-    )
+    return "ROCPROF" in os.environ.keys() and os.environ["ROCPROF"].endswith("rocprof")
 
 
 def using_v3():
-    return "ROCPROF" in os.environ.keys() and os.environ["ROCPROF"].endswith("rocprofv3")
+    return "ROCPROF" not in os.environ.keys() or (
+        "ROCPROF" in os.environ.keys() and os.environ["ROCPROF"].endswith("rocprofv3")
+    )
 
 
 def get_version(rocprof_compute_home) -> dict:
@@ -141,15 +140,15 @@ def detect_rocprof():
     global rocprof_cmd
     # detect rocprof
     if not "ROCPROF" in os.environ.keys():
-        rocprof_cmd = "rocprof"
+        rocprof_cmd = "rocprofv3"
     else:
         rocprof_cmd = os.environ["ROCPROF"]
 
     # resolve rocprof path
     rocprof_path = shutil.which(rocprof_cmd)
 
     if not rocprof_path:
-        rocprof_cmd = "rocprof"
+        rocprof_cmd = "rocprofv3"
         console_warning(
             "Unable to resolve path to %s binary. Reverting to default." % rocprof_cmd
         )