drm/amdgpu: rename the files for HMM handling
authorChristian König <christian.koenig@amd.com>
Wed, 9 Nov 2022 11:28:46 +0000 (12:28 +0100)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 17 Nov 2022 05:23:36 +0000 (00:23 -0500)
Clean that up a bit, no functional change.

Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Reviewed-by: Felix Kuehling <Felix.Kuehling@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/Makefile
drivers/gpu/drm/amd/amdgpu/amdgpu.h
drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
drivers/gpu/drm/amd/amdgpu/amdgpu_hmm.c [new file with mode: 0644]
drivers/gpu/drm/amd/amdgpu/amdgpu_hmm.h [new file with mode: 0644]
drivers/gpu/drm/amd/amdgpu/amdgpu_mn.c [deleted file]
drivers/gpu/drm/amd/amdgpu/amdgpu_mn.h [deleted file]
drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
drivers/gpu/drm/amd/amdkfd/kfd_svm.c

index 6ad39cf71bdd90728a9253e39ef4208a763da582..712075a491f2e3be4a679cb04e61fa79786d46a2 100644 (file)
@@ -250,7 +250,7 @@ endif
 amdgpu-$(CONFIG_COMPAT) += amdgpu_ioc32.o
 amdgpu-$(CONFIG_VGA_SWITCHEROO) += amdgpu_atpx_handler.o
 amdgpu-$(CONFIG_ACPI) += amdgpu_acpi.o
-amdgpu-$(CONFIG_HMM_MIRROR) += amdgpu_mn.o
+amdgpu-$(CONFIG_HMM_MIRROR) += amdgpu_hmm.o
 
 include $(FULL_AMD_PATH)/pm/Makefile
 
index 1f3a4d596d0d1a6d0ec43d6665dd64c37beb28e2..6b74df446694b3b5911bdb5c3d771e8d21ff246e 100644 (file)
@@ -82,7 +82,6 @@
 #include "amdgpu_vce.h"
 #include "amdgpu_vcn.h"
 #include "amdgpu_jpeg.h"
-#include "amdgpu_mn.h"
 #include "amdgpu_gmc.h"
 #include "amdgpu_gfx.h"
 #include "amdgpu_sdma.h"
index e44d740022bce0098b33e3142dd9342abfe063c8..3a763916a5a10cec31851e4123f1ec76154e9eae 100644 (file)
@@ -29,6 +29,7 @@
 #include "amdgpu_object.h"
 #include "amdgpu_gem.h"
 #include "amdgpu_vm.h"
+#include "amdgpu_hmm.h"
 #include "amdgpu_amdkfd.h"
 #include "amdgpu_dma_buf.h"
 #include <uapi/linux/kfd_ioctl.h>
@@ -949,7 +950,7 @@ static int init_user_pages(struct kgd_mem *mem, uint64_t user_addr,
                goto out;
        }
 
-       ret = amdgpu_mn_register(bo, user_addr);
+       ret = amdgpu_hmm_register(bo, user_addr);
        if (ret) {
                pr_err("%s: Failed to register MMU notifier: %d\n",
                       __func__, ret);
@@ -989,7 +990,7 @@ release_out:
        amdgpu_ttm_tt_get_user_pages_done(bo->tbo.ttm, range);
 unregister_out:
        if (ret)
-               amdgpu_mn_unregister(bo);
+               amdgpu_hmm_unregister(bo);
 out:
        mutex_unlock(&process_info->lock);
        return ret;
@@ -1773,7 +1774,7 @@ int amdgpu_amdkfd_gpuvm_free_memory_of_gpu(
        mutex_unlock(&process_info->lock);
 
        /* No more MMU notifiers */
-       amdgpu_mn_unregister(mem->bo);
+       amdgpu_hmm_unregister(mem->bo);
 
        ret = reserve_bo_and_cond_vms(mem, NULL, BO_VM_ALL, &ctx);
        if (unlikely(ret))
index 91571b1324f2feac9adaf3d907c6907e6e1786d4..a0780a4e3e618ee78b706ba24a4dff112cf58a70 100644 (file)
@@ -38,6 +38,7 @@
 #include "amdgpu.h"
 #include "amdgpu_display.h"
 #include "amdgpu_dma_buf.h"
+#include "amdgpu_hmm.h"
 #include "amdgpu_xgmi.h"
 
 static const struct drm_gem_object_funcs amdgpu_gem_object_funcs;
@@ -87,7 +88,7 @@ static void amdgpu_gem_object_free(struct drm_gem_object *gobj)
        struct amdgpu_bo *robj = gem_to_amdgpu_bo(gobj);
 
        if (robj) {
-               amdgpu_mn_unregister(robj);
+               amdgpu_hmm_unregister(robj);
                amdgpu_bo_unref(&robj);
        }
 }
@@ -414,7 +415,7 @@ int amdgpu_gem_userptr_ioctl(struct drm_device *dev, void *data,
        if (r)
                goto release_object;
 
-       r = amdgpu_mn_register(bo, args->addr);
+       r = amdgpu_hmm_register(bo, args->addr);
        if (r)
                goto release_object;
 
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_hmm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_hmm.c
new file mode 100644 (file)
index 0000000..a68072f
--- /dev/null
@@ -0,0 +1,245 @@
+/*
+ * Copyright 2014 Advanced Micro Devices, Inc.
+ * All Rights Reserved.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the
+ * "Software"), to deal in the Software without restriction, including
+ * without limitation the rights to use, copy, modify, merge, publish,
+ * distribute, sub license, and/or sell copies of the Software, and to
+ * permit persons to whom the Software is furnished to do so, subject to
+ * the following conditions:
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT. IN NO EVENT SHALL
+ * THE COPYRIGHT HOLDERS, AUTHORS AND/OR ITS SUPPLIERS BE LIABLE FOR ANY CLAIM,
+ * DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR
+ * OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE
+ * USE OR OTHER DEALINGS IN THE SOFTWARE.
+ *
+ * The above copyright notice and this permission notice (including the
+ * next paragraph) shall be included in all copies or substantial portions
+ * of the Software.
+ *
+ */
+/*
+ * Authors:
+ *    Christian König <christian.koenig@amd.com>
+ */
+
+/**
+ * DOC: MMU Notifier
+ *
+ * For coherent userptr handling registers an MMU notifier to inform the driver
+ * about updates on the page tables of a process.
+ *
+ * When somebody tries to invalidate the page tables we block the update until
+ * all operations on the pages in question are completed, then those pages are
+ * marked as accessed and also dirty if it wasn't a read only access.
+ *
+ * New command submissions using the userptrs in question are delayed until all
+ * page table invalidation are completed and we once more see a coherent process
+ * address space.
+ */
+
+#include <linux/firmware.h>
+#include <linux/module.h>
+#include <drm/drm.h>
+
+#include "amdgpu.h"
+#include "amdgpu_amdkfd.h"
+#include "amdgpu_hmm.h"
+
+/**
+ * amdgpu_hmm_invalidate_gfx - callback to notify about mm change
+ *
+ * @mni: the range (mm) is about to update
+ * @range: details on the invalidation
+ * @cur_seq: Value to pass to mmu_interval_set_seq()
+ *
+ * Block for operations on BOs to finish and mark pages as accessed and
+ * potentially dirty.
+ */
+static bool amdgpu_hmm_invalidate_gfx(struct mmu_interval_notifier *mni,
+                                     const struct mmu_notifier_range *range,
+                                     unsigned long cur_seq)
+{
+       struct amdgpu_bo *bo = container_of(mni, struct amdgpu_bo, notifier);
+       struct amdgpu_device *adev = amdgpu_ttm_adev(bo->tbo.bdev);
+       long r;
+
+       if (!mmu_notifier_range_blockable(range))
+               return false;
+
+       mutex_lock(&adev->notifier_lock);
+
+       mmu_interval_set_seq(mni, cur_seq);
+
+       r = dma_resv_wait_timeout(bo->tbo.base.resv, DMA_RESV_USAGE_BOOKKEEP,
+                                 false, MAX_SCHEDULE_TIMEOUT);
+       mutex_unlock(&adev->notifier_lock);
+       if (r <= 0)
+               DRM_ERROR("(%ld) failed to wait for user bo\n", r);
+       return true;
+}
+
+static const struct mmu_interval_notifier_ops amdgpu_hmm_gfx_ops = {
+       .invalidate = amdgpu_hmm_invalidate_gfx,
+};
+
+/**
+ * amdgpu_hmm_invalidate_hsa - callback to notify about mm change
+ *
+ * @mni: the range (mm) is about to update
+ * @range: details on the invalidation
+ * @cur_seq: Value to pass to mmu_interval_set_seq()
+ *
+ * We temporarily evict the BO attached to this range. This necessitates
+ * evicting all user-mode queues of the process.
+ */
+static bool amdgpu_hmm_invalidate_hsa(struct mmu_interval_notifier *mni,
+                                     const struct mmu_notifier_range *range,
+                                     unsigned long cur_seq)
+{
+       struct amdgpu_bo *bo = container_of(mni, struct amdgpu_bo, notifier);
+       struct amdgpu_device *adev = amdgpu_ttm_adev(bo->tbo.bdev);
+
+       if (!mmu_notifier_range_blockable(range))
+               return false;
+
+       mutex_lock(&adev->notifier_lock);
+
+       mmu_interval_set_seq(mni, cur_seq);
+
+       amdgpu_amdkfd_evict_userptr(bo->kfd_bo, bo->notifier.mm);
+       mutex_unlock(&adev->notifier_lock);
+
+       return true;
+}
+
+static const struct mmu_interval_notifier_ops amdgpu_hmm_hsa_ops = {
+       .invalidate = amdgpu_hmm_invalidate_hsa,
+};
+
+/**
+ * amdgpu_hmm_register - register a BO for notifier updates
+ *
+ * @bo: amdgpu buffer object
+ * @addr: userptr addr we should monitor
+ *
+ * Registers a mmu_notifier for the given BO at the specified address.
+ * Returns 0 on success, -ERRNO if anything goes wrong.
+ */
+int amdgpu_hmm_register(struct amdgpu_bo *bo, unsigned long addr)
+{
+       if (bo->kfd_bo)
+               return mmu_interval_notifier_insert(&bo->notifier, current->mm,
+                                                   addr, amdgpu_bo_size(bo),
+                                                   &amdgpu_hmm_hsa_ops);
+       return mmu_interval_notifier_insert(&bo->notifier, current->mm, addr,
+                                           amdgpu_bo_size(bo),
+                                           &amdgpu_hmm_gfx_ops);
+}
+
+/**
+ * amdgpu_hmm_unregister - unregister a BO for notifier updates
+ *
+ * @bo: amdgpu buffer object
+ *
+ * Remove any registration of mmu notifier updates from the buffer object.
+ */
+void amdgpu_hmm_unregister(struct amdgpu_bo *bo)
+{
+       if (!bo->notifier.mm)
+               return;
+       mmu_interval_notifier_remove(&bo->notifier);
+       bo->notifier.mm = NULL;
+}
+
+int amdgpu_hmm_range_get_pages(struct mmu_interval_notifier *notifier,
+                              struct mm_struct *mm, struct page **pages,
+                              uint64_t start, uint64_t npages,
+                              struct hmm_range **phmm_range, bool readonly,
+                              bool mmap_locked, void *owner)
+{
+       struct hmm_range *hmm_range;
+       unsigned long timeout;
+       unsigned long i;
+       unsigned long *pfns;
+       int r = 0;
+
+       hmm_range = kzalloc(sizeof(*hmm_range), GFP_KERNEL);
+       if (unlikely(!hmm_range))
+               return -ENOMEM;
+
+       pfns = kvmalloc_array(npages, sizeof(*pfns), GFP_KERNEL);
+       if (unlikely(!pfns)) {
+               r = -ENOMEM;
+               goto out_free_range;
+       }
+
+       hmm_range->notifier = notifier;
+       hmm_range->default_flags = HMM_PFN_REQ_FAULT;
+       if (!readonly)
+               hmm_range->default_flags |= HMM_PFN_REQ_WRITE;
+       hmm_range->hmm_pfns = pfns;
+       hmm_range->start = start;
+       hmm_range->end = start + npages * PAGE_SIZE;
+       hmm_range->dev_private_owner = owner;
+
+       /* Assuming 512MB takes maxmium 1 second to fault page address */
+       timeout = max(npages >> 17, 1ULL) * HMM_RANGE_DEFAULT_TIMEOUT;
+       timeout = jiffies + msecs_to_jiffies(timeout);
+
+retry:
+       hmm_range->notifier_seq = mmu_interval_read_begin(notifier);
+
+       if (likely(!mmap_locked))
+               mmap_read_lock(mm);
+
+       r = hmm_range_fault(hmm_range);
+
+       if (likely(!mmap_locked))
+               mmap_read_unlock(mm);
+       if (unlikely(r)) {
+               /*
+                * FIXME: This timeout should encompass the retry from
+                * mmu_interval_read_retry() as well.
+                */
+               if (r == -EBUSY && !time_after(jiffies, timeout))
+                       goto retry;
+               goto out_free_pfns;
+       }
+
+       /*
+        * Due to default_flags, all pages are HMM_PFN_VALID or
+        * hmm_range_fault() fails. FIXME: The pages cannot be touched outside
+        * the notifier_lock, and mmu_interval_read_retry() must be done first.
+        */
+       for (i = 0; pages && i < npages; i++)
+               pages[i] = hmm_pfn_to_page(pfns[i]);
+
+       *phmm_range = hmm_range;
+
+       return 0;
+
+out_free_pfns:
+       kvfree(pfns);
+out_free_range:
+       kfree(hmm_range);
+
+       return r;
+}
+
+int amdgpu_hmm_range_get_pages_done(struct hmm_range *hmm_range)
+{
+       int r;
+
+       r = mmu_interval_read_retry(hmm_range->notifier,
+                                   hmm_range->notifier_seq);
+       kvfree(hmm_range->hmm_pfns);
+       kfree(hmm_range);
+
+       return r;
+}
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_hmm.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_hmm.h
new file mode 100644 (file)
index 0000000..4e596a1
--- /dev/null
@@ -0,0 +1,53 @@
+/*
+ * Copyright 2017 Advanced Micro Devices, Inc.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR
+ * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
+ * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+ * OTHER DEALINGS IN THE SOFTWARE.
+ *
+ * Authors: Christian König
+ */
+#ifndef __AMDGPU_MN_H__
+#define __AMDGPU_MN_H__
+
+#include <linux/types.h>
+#include <linux/hmm.h>
+#include <linux/rwsem.h>
+#include <linux/workqueue.h>
+#include <linux/interval_tree.h>
+
+int amdgpu_hmm_range_get_pages(struct mmu_interval_notifier *notifier,
+                              struct mm_struct *mm, struct page **pages,
+                              uint64_t start, uint64_t npages,
+                              struct hmm_range **phmm_range, bool readonly,
+                              bool mmap_locked, void *owner);
+int amdgpu_hmm_range_get_pages_done(struct hmm_range *hmm_range);
+
+#if defined(CONFIG_HMM_MIRROR)
+int amdgpu_hmm_register(struct amdgpu_bo *bo, unsigned long addr);
+void amdgpu_hmm_unregister(struct amdgpu_bo *bo);
+#else
+static inline int amdgpu_hmm_register(struct amdgpu_bo *bo, unsigned long addr)
+{
+       DRM_WARN_ONCE("HMM_MIRROR kernel config option is not enabled, "
+                     "add CONFIG_ZONE_DEVICE=y in config file to fix this\n");
+       return -ENODEV;
+}
+static inline void amdgpu_hmm_unregister(struct amdgpu_bo *bo) {}
+#endif
+
+#endif
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_mn.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_mn.c
deleted file mode 100644 (file)
index b86c0b8..0000000
+++ /dev/null
@@ -1,244 +0,0 @@
-/*
- * Copyright 2014 Advanced Micro Devices, Inc.
- * All Rights Reserved.
- *
- * Permission is hereby granted, free of charge, to any person obtaining a
- * copy of this software and associated documentation files (the
- * "Software"), to deal in the Software without restriction, including
- * without limitation the rights to use, copy, modify, merge, publish,
- * distribute, sub license, and/or sell copies of the Software, and to
- * permit persons to whom the Software is furnished to do so, subject to
- * the following conditions:
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT. IN NO EVENT SHALL
- * THE COPYRIGHT HOLDERS, AUTHORS AND/OR ITS SUPPLIERS BE LIABLE FOR ANY CLAIM,
- * DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR
- * OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE
- * USE OR OTHER DEALINGS IN THE SOFTWARE.
- *
- * The above copyright notice and this permission notice (including the
- * next paragraph) shall be included in all copies or substantial portions
- * of the Software.
- *
- */
-/*
- * Authors:
- *    Christian König <christian.koenig@amd.com>
- */
-
-/**
- * DOC: MMU Notifier
- *
- * For coherent userptr handling registers an MMU notifier to inform the driver
- * about updates on the page tables of a process.
- *
- * When somebody tries to invalidate the page tables we block the update until
- * all operations on the pages in question are completed, then those pages are
- * marked as accessed and also dirty if it wasn't a read only access.
- *
- * New command submissions using the userptrs in question are delayed until all
- * page table invalidation are completed and we once more see a coherent process
- * address space.
- */
-
-#include <linux/firmware.h>
-#include <linux/module.h>
-#include <drm/drm.h>
-
-#include "amdgpu.h"
-#include "amdgpu_amdkfd.h"
-
-/**
- * amdgpu_mn_invalidate_gfx - callback to notify about mm change
- *
- * @mni: the range (mm) is about to update
- * @range: details on the invalidation
- * @cur_seq: Value to pass to mmu_interval_set_seq()
- *
- * Block for operations on BOs to finish and mark pages as accessed and
- * potentially dirty.
- */
-static bool amdgpu_mn_invalidate_gfx(struct mmu_interval_notifier *mni,
-                                    const struct mmu_notifier_range *range,
-                                    unsigned long cur_seq)
-{
-       struct amdgpu_bo *bo = container_of(mni, struct amdgpu_bo, notifier);
-       struct amdgpu_device *adev = amdgpu_ttm_adev(bo->tbo.bdev);
-       long r;
-
-       if (!mmu_notifier_range_blockable(range))
-               return false;
-
-       mutex_lock(&adev->notifier_lock);
-
-       mmu_interval_set_seq(mni, cur_seq);
-
-       r = dma_resv_wait_timeout(bo->tbo.base.resv, DMA_RESV_USAGE_BOOKKEEP,
-                                 false, MAX_SCHEDULE_TIMEOUT);
-       mutex_unlock(&adev->notifier_lock);
-       if (r <= 0)
-               DRM_ERROR("(%ld) failed to wait for user bo\n", r);
-       return true;
-}
-
-static const struct mmu_interval_notifier_ops amdgpu_mn_gfx_ops = {
-       .invalidate = amdgpu_mn_invalidate_gfx,
-};
-
-/**
- * amdgpu_mn_invalidate_hsa - callback to notify about mm change
- *
- * @mni: the range (mm) is about to update
- * @range: details on the invalidation
- * @cur_seq: Value to pass to mmu_interval_set_seq()
- *
- * We temporarily evict the BO attached to this range. This necessitates
- * evicting all user-mode queues of the process.
- */
-static bool amdgpu_mn_invalidate_hsa(struct mmu_interval_notifier *mni,
-                                    const struct mmu_notifier_range *range,
-                                    unsigned long cur_seq)
-{
-       struct amdgpu_bo *bo = container_of(mni, struct amdgpu_bo, notifier);
-       struct amdgpu_device *adev = amdgpu_ttm_adev(bo->tbo.bdev);
-
-       if (!mmu_notifier_range_blockable(range))
-               return false;
-
-       mutex_lock(&adev->notifier_lock);
-
-       mmu_interval_set_seq(mni, cur_seq);
-
-       amdgpu_amdkfd_evict_userptr(bo->kfd_bo, bo->notifier.mm);
-       mutex_unlock(&adev->notifier_lock);
-
-       return true;
-}
-
-static const struct mmu_interval_notifier_ops amdgpu_mn_hsa_ops = {
-       .invalidate = amdgpu_mn_invalidate_hsa,
-};
-
-/**
- * amdgpu_mn_register - register a BO for notifier updates
- *
- * @bo: amdgpu buffer object
- * @addr: userptr addr we should monitor
- *
- * Registers a mmu_notifier for the given BO at the specified address.
- * Returns 0 on success, -ERRNO if anything goes wrong.
- */
-int amdgpu_mn_register(struct amdgpu_bo *bo, unsigned long addr)
-{
-       if (bo->kfd_bo)
-               return mmu_interval_notifier_insert(&bo->notifier, current->mm,
-                                                   addr, amdgpu_bo_size(bo),
-                                                   &amdgpu_mn_hsa_ops);
-       return mmu_interval_notifier_insert(&bo->notifier, current->mm, addr,
-                                           amdgpu_bo_size(bo),
-                                           &amdgpu_mn_gfx_ops);
-}
-
-/**
- * amdgpu_mn_unregister - unregister a BO for notifier updates
- *
- * @bo: amdgpu buffer object
- *
- * Remove any registration of mmu notifier updates from the buffer object.
- */
-void amdgpu_mn_unregister(struct amdgpu_bo *bo)
-{
-       if (!bo->notifier.mm)
-               return;
-       mmu_interval_notifier_remove(&bo->notifier);
-       bo->notifier.mm = NULL;
-}
-
-int amdgpu_hmm_range_get_pages(struct mmu_interval_notifier *notifier,
-                              struct mm_struct *mm, struct page **pages,
-                              uint64_t start, uint64_t npages,
-                              struct hmm_range **phmm_range, bool readonly,
-                              bool mmap_locked, void *owner)
-{
-       struct hmm_range *hmm_range;
-       unsigned long timeout;
-       unsigned long i;
-       unsigned long *pfns;
-       int r = 0;
-
-       hmm_range = kzalloc(sizeof(*hmm_range), GFP_KERNEL);
-       if (unlikely(!hmm_range))
-               return -ENOMEM;
-
-       pfns = kvmalloc_array(npages, sizeof(*pfns), GFP_KERNEL);
-       if (unlikely(!pfns)) {
-               r = -ENOMEM;
-               goto out_free_range;
-       }
-
-       hmm_range->notifier = notifier;
-       hmm_range->default_flags = HMM_PFN_REQ_FAULT;
-       if (!readonly)
-               hmm_range->default_flags |= HMM_PFN_REQ_WRITE;
-       hmm_range->hmm_pfns = pfns;
-       hmm_range->start = start;
-       hmm_range->end = start + npages * PAGE_SIZE;
-       hmm_range->dev_private_owner = owner;
-
-       /* Assuming 512MB takes maxmium 1 second to fault page address */
-       timeout = max(npages >> 17, 1ULL) * HMM_RANGE_DEFAULT_TIMEOUT;
-       timeout = jiffies + msecs_to_jiffies(timeout);
-
-retry:
-       hmm_range->notifier_seq = mmu_interval_read_begin(notifier);
-
-       if (likely(!mmap_locked))
-               mmap_read_lock(mm);
-
-       r = hmm_range_fault(hmm_range);
-
-       if (likely(!mmap_locked))
-               mmap_read_unlock(mm);
-       if (unlikely(r)) {
-               /*
-                * FIXME: This timeout should encompass the retry from
-                * mmu_interval_read_retry() as well.
-                */
-               if (r == -EBUSY && !time_after(jiffies, timeout))
-                       goto retry;
-               goto out_free_pfns;
-       }
-
-       /*
-        * Due to default_flags, all pages are HMM_PFN_VALID or
-        * hmm_range_fault() fails. FIXME: The pages cannot be touched outside
-        * the notifier_lock, and mmu_interval_read_retry() must be done first.
-        */
-       for (i = 0; pages && i < npages; i++)
-               pages[i] = hmm_pfn_to_page(pfns[i]);
-
-       *phmm_range = hmm_range;
-
-       return 0;
-
-out_free_pfns:
-       kvfree(pfns);
-out_free_range:
-       kfree(hmm_range);
-
-       return r;
-}
-
-int amdgpu_hmm_range_get_pages_done(struct hmm_range *hmm_range)
-{
-       int r;
-
-       r = mmu_interval_read_retry(hmm_range->notifier,
-                                   hmm_range->notifier_seq);
-       kvfree(hmm_range->hmm_pfns);
-       kfree(hmm_range);
-
-       return r;
-}
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_mn.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_mn.h
deleted file mode 100644 (file)
index 14a3c18..0000000
+++ /dev/null
@@ -1,53 +0,0 @@
-/*
- * Copyright 2017 Advanced Micro Devices, Inc.
- *
- * Permission is hereby granted, free of charge, to any person obtaining a
- * copy of this software and associated documentation files (the "Software"),
- * to deal in the Software without restriction, including without limitation
- * the rights to use, copy, modify, merge, publish, distribute, sublicense,
- * and/or sell copies of the Software, and to permit persons to whom the
- * Software is furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice shall be included in
- * all copies or substantial portions of the Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
- * THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR
- * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
- * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
- * OTHER DEALINGS IN THE SOFTWARE.
- *
- * Authors: Christian König
- */
-#ifndef __AMDGPU_MN_H__
-#define __AMDGPU_MN_H__
-
-#include <linux/types.h>
-#include <linux/hmm.h>
-#include <linux/rwsem.h>
-#include <linux/workqueue.h>
-#include <linux/interval_tree.h>
-
-int amdgpu_hmm_range_get_pages(struct mmu_interval_notifier *notifier,
-                              struct mm_struct *mm, struct page **pages,
-                              uint64_t start, uint64_t npages,
-                              struct hmm_range **phmm_range, bool readonly,
-                              bool mmap_locked, void *owner);
-int amdgpu_hmm_range_get_pages_done(struct hmm_range *hmm_range);
-
-#if defined(CONFIG_HMM_MIRROR)
-int amdgpu_mn_register(struct amdgpu_bo *bo, unsigned long addr);
-void amdgpu_mn_unregister(struct amdgpu_bo *bo);
-#else
-static inline int amdgpu_mn_register(struct amdgpu_bo *bo, unsigned long addr)
-{
-       DRM_WARN_ONCE("HMM_MIRROR kernel config option is not enabled, "
-                     "add CONFIG_ZONE_DEVICE=y in config file to fix this\n");
-       return -ENODEV;
-}
-static inline void amdgpu_mn_unregister(struct amdgpu_bo *bo) {}
-#endif
-
-#endif
index 965808f4f6822b40e7db9bbe0d90a10d43104caa..ddb13b18ed7b1d47740253ed619304deb15c05ca 100644 (file)
@@ -58,6 +58,7 @@
 #include "amdgpu_amdkfd.h"
 #include "amdgpu_sdma.h"
 #include "amdgpu_ras.h"
+#include "amdgpu_hmm.h"
 #include "amdgpu_atomfirmware.h"
 #include "amdgpu_res_cursor.h"
 #include "bif/bif_4_1_d.h"
index 3723e90e3a9046bb9450a8cce43d70ee9f238675..77227761e669bd2d3d2c7ea4337fe944db81c953 100644 (file)
@@ -28,7 +28,6 @@
 #include "amdgpu_sync.h"
 #include "amdgpu_object.h"
 #include "amdgpu_vm.h"
-#include "amdgpu_mn.h"
 #include "amdgpu_res_cursor.h"
 #include "kfd_priv.h"
 #include "kfd_svm.h"
index afe7c499867687d6cb50b8935a4c2287141f899a..1cf7dcb22e3fb72bf4666cb17a2d725554e4b492 100644 (file)
@@ -26,7 +26,7 @@
 #include "amdgpu_sync.h"
 #include "amdgpu_object.h"
 #include "amdgpu_vm.h"
-#include "amdgpu_mn.h"
+#include "amdgpu_hmm.h"
 #include "amdgpu.h"
 #include "amdgpu_xgmi.h"
 #include "kfd_priv.h"