From 4f2aa06688740e2df2685d14518f5ffce46c0a9c Mon Sep 17 00:00:00 2001
From: Michael LIAO <michael_liaocn@yahoo.com>
Date: Thu, 1 Aug 2019 14:50:03 -0400
Subject: [PATCH] Revise the previous fix to use the canonical path to HSA.

- This fix the build failures under certain environments.
---
 CMakeLists.txt   | 2 +-
 src/transport.cu | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 39a0cbcc94..237e9242b3 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -146,7 +146,7 @@ if("${HIP_COMPILER}" MATCHES "clang")
     PRIVATE --amdgpu-target=gfx908
     PRIVATE -fgpu-rdc)
   target_link_libraries(rccl PRIVATE -fgpu-rdc)
-  target_include_directories(rccl PRIVATE /opt/rocm/include)
+  target_include_directories(rccl PRIVATE /opt/rocm/hsa/include)
 endif()
 
 if("${HIP_COMPILER}" MATCHES "hcc")
diff --git a/src/transport.cu b/src/transport.cu
index 7d83ff4488..7d2003f267 100644
--- a/src/transport.cu
+++ b/src/transport.cu
@@ -7,7 +7,8 @@
 
 #include "core.h"
 #include "common_coll.h"
-#include <hsa.h>
+#include <hsa/hsa.h>
+#include <hsa/hsa_ext_amd.h>
 
 extern struct ncclTransport p2pTransport;
 extern struct ncclTransport shmTransport;