From f571c5ec7cca23d91c13222497863a9042c9aad4 Mon Sep 17 00:00:00 2001
From: Aditya Atluri <adityaavinash1@gmail.com>
Date: Mon, 25 Apr 2016 11:05:30 -0500
Subject: [PATCH 1/2] fixed hipmemset to use native ihip api

---
 hipamd/src/hip_memory.cpp | 49 ++++++++++++++++++++++++++++++++++++---
 1 file changed, 46 insertions(+), 3 deletions(-)

diff --git a/hipamd/src/hip_memory.cpp b/hipamd/src/hip_memory.cpp
index 01b24a9e13..4107be44ab 100644
--- a/hipamd/src/hip_memory.cpp
+++ b/hipamd/src/hip_memory.cpp
@@ -419,10 +419,53 @@ hipError_t hipMemsetAsync(void* dst, int  value, size_t sizeBytes, hipStream_t s
 
 hipError_t hipMemset(void* dst, int  value, size_t sizeBytes )
 {
-    HIP_INIT_API(dst, value, sizeBytes);
-
+    hipStream_t stream = hipStreamNull;
     // TODO - call an ihip memset so HIP_TRACE is correct.
-    return hipMemsetAsync(dst, value, sizeBytes, hipStreamNull);
+    HIP_INIT_API(dst, value, sizeBytes, stream);
+
+    hipError_t e = hipSuccess;
+
+    stream =  ihipSyncAndResolveStream(stream);
+
+    if (stream) {
+        stream->lockopen_preKernelCommand();
+
+        hc::completion_future cf ;
+
+        if ((sizeBytes & 0x3) == 0) {
+            // use a faster dword-per-workitem copy:
+            try {
+                value = value & 0xff;
+                unsigned value32 = (value << 24) | (value << 16) | (value << 8) | (value) ;
+                cf = ihipMemsetKernel<unsigned> (stream, static_cast<unsigned*> (dst), value32, sizeBytes/sizeof(unsigned));
+            }
+            catch (std::exception &ex) {
+                e = hipErrorInvalidValue;
+            }
+        } else {
+            // use a slow byte-per-workitem copy:
+            try {
+                cf = ihipMemsetKernel<char> (stream, static_cast<char*> (dst), value, sizeBytes);
+            }
+            catch (std::exception &ex) {
+                e = hipErrorInvalidValue;
+            }
+        }
+        cf.wait();
+
+        stream->lockclose_postKernelCommand(cf);
+
+
+        if (HIP_LAUNCH_BLOCKING) {
+            tprintf (DB_SYNC, "'%s' LAUNCH_BLOCKING wait for memset [stream:%p].\n", __func__, (void*)stream);
+            cf.wait();
+            tprintf (DB_SYNC, "'%s' LAUNCH_BLOCKING memset completed [stream:%p].\n", __func__, (void*)stream);
+        }
+    } else {
+        e = hipErrorInvalidValue;
+    }
+
+    return ihipLogStatus(e);
 }
 
 

From c46e03de962bee7b8677bb6585b6ece9167e78fc Mon Sep 17 00:00:00 2001
From: Aditya Atluri <adityaavinash1@gmail.com>
Date: Sat, 30 Apr 2016 12:11:04 -0500
Subject: [PATCH 2/2] removed warnings

---
 hipamd/src/hip_peer.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/hipamd/src/hip_peer.cpp b/hipamd/src/hip_peer.cpp
index e81b27c31a..cec8017b0c 100644
--- a/hipamd/src/hip_peer.cpp
+++ b/hipamd/src/hip_peer.cpp
@@ -24,7 +24,7 @@ THE SOFTWARE.
 #include "hcc_detail/trace_helper.h"
 
 /**
- * @warning HCC returns 0 in *canAccessPeer ; Need to update this function when RT supports P2P
+ * HCC returns 0 in *canAccessPeer ; Need to update this function when RT supports P2P
  */
 //---
 hipError_t hipDeviceCanAccessPeer (int* canAccessPeer, int  deviceId, int peerDeviceId)
@@ -145,7 +145,7 @@ hipError_t hipMemcpyPeer (void* dst, int  dstDevice, const void* src, int  srcDe
 
 
 /**
- * @bug This function uses a synchronous copy
+ * This function uses a synchronous copy
  */
 //---
 hipError_t hipMemcpyPeerAsync (void* dst, int  dstDevice, const void* src, int  srcDevice, size_t sizeBytes, hipStream_t stream)