<html><head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body>
    <p><br>
    </p>
    <div class="moz-cite-prefix">On 2021-11-01 8:55 a.m., Felix Kuehling
      wrote:<br>
    </div>
    <blockquote type="cite" cite="mid:20211101125513.999187-1-Felix.Kuehling@amd.com">
      <pre class="moz-quote-pre" wrap="">The preferred location should be used as the migration destination
whenever it is accessible by the faulting GPU. System memory is always
accessible. Peer memory is accessible if it's in the same XGMI hive.</pre>
    </blockquote>
    <p>This patch series is</p>
    <p>Reviewed-by: Philip Yang <a class="moz-txt-link-rfc2396E" href="mailto:Philip.Yang@amd.com"><Philip.Yang@amd.com></a><br>
    </p>
    <blockquote type="cite" cite="mid:20211101125513.999187-1-Felix.Kuehling@amd.com">
      <pre class="moz-quote-pre" wrap="">

Signed-off-by: Felix Kuehling <a class="moz-txt-link-rfc2396E" href="mailto:Felix.Kuehling@amd.com"><Felix.Kuehling@amd.com></a>
---
 drivers/gpu/drm/amd/amdkfd/kfd_svm.c | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_svm.c b/drivers/gpu/drm/amd/amdkfd/kfd_svm.c
index 9a46ba190ea7..8d37ee755b34 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_svm.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_svm.c
@@ -2250,7 +2250,7 @@ svm_range_from_addr(struct svm_range_list *svms, unsigned long addr,
  * migration if actual loc is not best location, then update GPU page table
  * mapping to the best location.
  *
- * If vm fault gpu is range preferred loc, the best_loc is preferred loc.
+ * If the preferred loc is accessible by faulting GPU, use preferred loc.
  * If vm fault gpu idx is on range ACCESSIBLE bitmap, best_loc is vm fault gpu
  * If vm fault gpu idx is on range ACCESSIBLE_IN_PLACE bitmap, then
  *    if range actual loc is cpu, best_loc is cpu
@@ -2267,7 +2267,7 @@ svm_range_best_restore_location(struct svm_range *prange,
                                struct amdgpu_device *adev,
                                int32_t *gpuidx)
 {
-       struct amdgpu_device *bo_adev;
+       struct amdgpu_device *bo_adev, *preferred_adev;
        struct kfd_process *p;
        uint32_t gpuid;
        int r;
@@ -2280,8 +2280,16 @@ svm_range_best_restore_location(struct svm_range *prange,
                return -1;
        }
 
-       if (prange->preferred_loc == gpuid)
+       if (prange->preferred_loc == gpuid ||
+           prange->preferred_loc == KFD_IOCTL_SVM_LOCATION_SYSMEM) {
                return prange->preferred_loc;
+       } else if (prange->preferred_loc != KFD_IOCTL_SVM_LOCATION_UNDEFINED) {
+               preferred_adev = svm_range_get_adev_by_id(prange,
+                                                       prange->preferred_loc);
+               if (amdgpu_xgmi_same_hive(adev, preferred_adev))
+                       return prange->preferred_loc;
+               /* fall through */
+       }
 
        if (test_bit(*gpuidx, prange->bitmap_access))
                return gpuid;
</pre>
    </blockquote>
  </body>
</html>