fix bugs in P2P implementation

- addPeers polarity reversed, would never add.
- check allow_access return value, pipe error to hipMalloc.


[ROCm/clr commit: 7ca06d2fb1]

projects/clr/hipamd/src/hip_hcc.cpp

@@ -214,7 +214,8 @@ template<>
 bool ihipDeviceCriticalBase_t<DeviceMutex>::addPeer(ihipDevice_t *peer) 
 {
     auto match = std::find(_peers.begin(), _peers.end(), peer);
-    if (match != std::end(_peers)) {
+    if (match == std::end(_peers)) {
+        // Not already a peer, let's update the list:
         _peers.push_back(peer);
         recomputePeerAgents();
         return true;
@@ -230,6 +231,7 @@ bool ihipDeviceCriticalBase_t<DeviceMutex>::removePeer(ihipDevice_t *peer)
 {
     auto match = std::find(_peers.begin(), _peers.end(), peer);
     if (match != std::end(_peers)) {
+        // Found a valid peer, let's remove it.
         _peers.remove(peer);
         recomputePeerAgents();
         return true;