drm/amdgpu: drop testing module parameter
authorAlex Deucher <alexander.deucher@amd.com>
Fri, 18 Feb 2022 21:50:26 +0000 (16:50 -0500)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 23 Feb 2022 19:02:51 +0000 (14:02 -0500)
This test is not particularly useful now that GTT and GART
are decoupled in the driver.

Reviewed-by: Christian König <christian.koenig@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_device.c
drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
drivers/gpu/drm/amd/amdgpu/amdgpu_test.c [deleted file]

index 7fedbb725e179e700513c35aea6dacf949bf964f..5dfe08cb045eb2ef6335870449d2241bc21ab50e 100644 (file)
@@ -46,7 +46,7 @@ amdgpu-y += amdgpu_device.o amdgpu_kms.o \
        atom.o amdgpu_fence.o amdgpu_ttm.o amdgpu_object.o amdgpu_gart.o \
        amdgpu_encoders.o amdgpu_display.o amdgpu_i2c.o \
        amdgpu_gem.o amdgpu_ring.o \
-       amdgpu_cs.o amdgpu_bios.o amdgpu_benchmark.o amdgpu_test.o \
+       amdgpu_cs.o amdgpu_bios.o amdgpu_benchmark.o \
        atombios_dp.o amdgpu_afmt.o amdgpu_trace_points.o \
        atombios_encoders.o amdgpu_sa.o atombios_i2c.o \
        amdgpu_dma_buf.o amdgpu_vm.o amdgpu_ib.o amdgpu_pll.o \
index d6cf2e343a43b6294affe3c3e848ecb86db04fdf..a117b40f03a0953506465c02c363c9be41702ad8 100644 (file)
@@ -155,7 +155,6 @@ extern int amdgpu_vis_vram_limit;
 extern int amdgpu_gart_size;
 extern int amdgpu_gtt_size;
 extern int amdgpu_moverate;
-extern int amdgpu_testing;
 extern int amdgpu_audio;
 extern int amdgpu_disp_priority;
 extern int amdgpu_hw_i2c;
@@ -587,12 +586,6 @@ void amdgpu_device_wb_free(struct amdgpu_device *adev, u32 wb);
  */
 int amdgpu_benchmark(struct amdgpu_device *adev, int test_number);
 
-
-/*
- * Testing
- */
-void amdgpu_test_moves(struct amdgpu_device *adev);
-
 /*
  * ASIC specific register table accessible by UMD
  */
index bf7e648fd4217cc6cd3956a495d2d515a584f8cb..17ed7fd2a5c29bb143f44fa8d7d3d804b9514fc5 100644 (file)
@@ -3803,13 +3803,6 @@ fence_driver_init:
        } else
                adev->ucode_sysfs_en = true;
 
-       if ((amdgpu_testing & 1)) {
-               if (adev->accel_working)
-                       amdgpu_test_moves(adev);
-               else
-                       DRM_INFO("amdgpu: acceleration disabled, skipping move tests\n");
-       }
-
        /*
         * Register gpu instance before amdgpu_device_enable_mgpu_fan_boost.
         * Otherwise the mgpu fan boost feature will be skipped due to the
index 196d131e9a80458e4c26c8f826ba9dcd55a66daa..e18356fff7104a63dbb6e2f27e00825f1b9031c0 100644 (file)
@@ -110,7 +110,6 @@ int amdgpu_vis_vram_limit;
 int amdgpu_gart_size = -1; /* auto */
 int amdgpu_gtt_size = -1; /* auto */
 int amdgpu_moverate = -1; /* auto */
-int amdgpu_testing;
 int amdgpu_audio = -1;
 int amdgpu_disp_priority;
 int amdgpu_hw_i2c;
@@ -230,13 +229,6 @@ module_param_named(gttsize, amdgpu_gtt_size, int, 0600);
 MODULE_PARM_DESC(moverate, "Maximum buffer migration rate in MB/s. (32, 64, etc., -1=auto, 0=1=disabled)");
 module_param_named(moverate, amdgpu_moverate, int, 0600);
 
-/**
- * DOC: test (int)
- * Test BO GTT->VRAM and VRAM->GTT GPU copies. The default is 0 (Skip test, only set 1 to run test).
- */
-MODULE_PARM_DESC(test, "Run tests");
-module_param_named(test, amdgpu_testing, int, 0444);
-
 /**
  * DOC: audio (int)
  * Set HDMI/DPAudio. Only affects non-DC display handling. The default is -1 (Enabled), set 0 to disabled it.
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c
deleted file mode 100644 (file)
index 909d830..0000000
+++ /dev/null
@@ -1,250 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0 OR MIT
-/*
- * Copyright 2009 VMware, 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: Michel Dänzer
- */
-
-#include <drm/amdgpu_drm.h>
-#include "amdgpu.h"
-#include "amdgpu_uvd.h"
-#include "amdgpu_vce.h"
-
-/* Test BO GTT->VRAM and VRAM->GTT GPU copies across the whole GTT aperture */
-static void amdgpu_do_test_moves(struct amdgpu_device *adev)
-{
-       struct amdgpu_ring *ring = adev->mman.buffer_funcs_ring;
-       struct amdgpu_bo *vram_obj = NULL;
-       struct amdgpu_bo **gtt_obj = NULL;
-       struct amdgpu_bo_param bp;
-       uint64_t gart_addr, vram_addr;
-       unsigned n, size;
-       int i, r;
-
-       size = 1024 * 1024;
-
-       /* Number of tests =
-        * (Total GTT - gart_pin_size - (2 transfer windows for buffer moves)) / test size
-        */
-       n = adev->gmc.gart_size - atomic64_read(&adev->gart_pin_size);
-       n -= AMDGPU_GTT_MAX_TRANSFER_SIZE * AMDGPU_GTT_NUM_TRANSFER_WINDOWS *
-               AMDGPU_GPU_PAGE_SIZE;
-       n /= size;
-
-       gtt_obj = kcalloc(n, sizeof(*gtt_obj), GFP_KERNEL);
-       if (!gtt_obj) {
-               DRM_ERROR("Failed to allocate %d pointers\n", n);
-               r = 1;
-               goto out_cleanup;
-       }
-       memset(&bp, 0, sizeof(bp));
-       bp.size = size;
-       bp.byte_align = PAGE_SIZE;
-       bp.domain = AMDGPU_GEM_DOMAIN_VRAM;
-       bp.flags = 0;
-       bp.type = ttm_bo_type_kernel;
-       bp.resv = NULL;
-       bp.bo_ptr_size = sizeof(struct amdgpu_bo);
-
-       r = amdgpu_bo_create(adev, &bp, &vram_obj);
-       if (r) {
-               DRM_ERROR("Failed to create VRAM object\n");
-               goto out_cleanup;
-       }
-       r = amdgpu_bo_reserve(vram_obj, false);
-       if (unlikely(r != 0))
-               goto out_unref;
-       r = amdgpu_bo_pin(vram_obj, AMDGPU_GEM_DOMAIN_VRAM);
-       if (r) {
-               DRM_ERROR("Failed to pin VRAM object\n");
-               goto out_unres;
-       }
-       vram_addr = amdgpu_bo_gpu_offset(vram_obj);
-       for (i = 0; i < n; i++) {
-               void *gtt_map, *vram_map;
-               void **gart_start, **gart_end;
-               void **vram_start, **vram_end;
-               struct dma_fence *fence = NULL;
-
-               bp.domain = AMDGPU_GEM_DOMAIN_GTT;
-               r = amdgpu_bo_create(adev, &bp, gtt_obj + i);
-               if (r) {
-                       DRM_ERROR("Failed to create GTT object %d\n", i);
-                       goto out_lclean;
-               }
-
-               r = amdgpu_bo_reserve(gtt_obj[i], false);
-               if (unlikely(r != 0))
-                       goto out_lclean_unref;
-               r = amdgpu_bo_pin(gtt_obj[i], AMDGPU_GEM_DOMAIN_GTT);
-               if (r) {
-                       DRM_ERROR("Failed to pin GTT object %d\n", i);
-                       goto out_lclean_unres;
-               }
-               r = amdgpu_ttm_alloc_gart(&gtt_obj[i]->tbo);
-               if (r) {
-                       DRM_ERROR("%p bind failed\n", gtt_obj[i]);
-                       goto out_lclean_unpin;
-               }
-               gart_addr = amdgpu_bo_gpu_offset(gtt_obj[i]);
-
-               r = amdgpu_bo_kmap(gtt_obj[i], &gtt_map);
-               if (r) {
-                       DRM_ERROR("Failed to map GTT object %d\n", i);
-                       goto out_lclean_unpin;
-               }
-
-               for (gart_start = gtt_map, gart_end = gtt_map + size;
-                    gart_start < gart_end;
-                    gart_start++)
-                       *gart_start = gart_start;
-
-               amdgpu_bo_kunmap(gtt_obj[i]);
-
-               r = amdgpu_copy_buffer(ring, gart_addr, vram_addr,
-                                      size, NULL, &fence, false, false, false);
-
-               if (r) {
-                       DRM_ERROR("Failed GTT->VRAM copy %d\n", i);
-                       goto out_lclean_unpin;
-               }
-
-               r = dma_fence_wait(fence, false);
-               if (r) {
-                       DRM_ERROR("Failed to wait for GTT->VRAM fence %d\n", i);
-                       goto out_lclean_unpin;
-               }
-
-               dma_fence_put(fence);
-               fence = NULL;
-
-               r = amdgpu_bo_kmap(vram_obj, &vram_map);
-               if (r) {
-                       DRM_ERROR("Failed to map VRAM object after copy %d\n", i);
-                       goto out_lclean_unpin;
-               }
-
-               for (gart_start = gtt_map, gart_end = gtt_map + size,
-                    vram_start = vram_map, vram_end = vram_map + size;
-                    vram_start < vram_end;
-                    gart_start++, vram_start++) {
-                       if (*vram_start != gart_start) {
-                               DRM_ERROR("Incorrect GTT->VRAM copy %d: Got 0x%p, "
-                                         "expected 0x%p (GTT/VRAM offset "
-                                         "0x%16llx/0x%16llx)\n",
-                                         i, *vram_start, gart_start,
-                                         (unsigned long long)
-                                         (gart_addr - adev->gmc.gart_start +
-                                          (void *)gart_start - gtt_map),
-                                         (unsigned long long)
-                                         (vram_addr - adev->gmc.vram_start +
-                                          (void *)gart_start - gtt_map));
-                               amdgpu_bo_kunmap(vram_obj);
-                               goto out_lclean_unpin;
-                       }
-                       *vram_start = vram_start;
-               }
-
-               amdgpu_bo_kunmap(vram_obj);
-
-               r = amdgpu_copy_buffer(ring, vram_addr, gart_addr,
-                                      size, NULL, &fence, false, false, false);
-
-               if (r) {
-                       DRM_ERROR("Failed VRAM->GTT copy %d\n", i);
-                       goto out_lclean_unpin;
-               }
-
-               r = dma_fence_wait(fence, false);
-               if (r) {
-                       DRM_ERROR("Failed to wait for VRAM->GTT fence %d\n", i);
-                       goto out_lclean_unpin;
-               }
-
-               dma_fence_put(fence);
-               fence = NULL;
-
-               r = amdgpu_bo_kmap(gtt_obj[i], &gtt_map);
-               if (r) {
-                       DRM_ERROR("Failed to map GTT object after copy %d\n", i);
-                       goto out_lclean_unpin;
-               }
-
-               for (gart_start = gtt_map, gart_end = gtt_map + size,
-                    vram_start = vram_map, vram_end = vram_map + size;
-                    gart_start < gart_end;
-                    gart_start++, vram_start++) {
-                       if (*gart_start != vram_start) {
-                               DRM_ERROR("Incorrect VRAM->GTT copy %d: Got 0x%p, "
-                                         "expected 0x%p (VRAM/GTT offset "
-                                         "0x%16llx/0x%16llx)\n",
-                                         i, *gart_start, vram_start,
-                                         (unsigned long long)
-                                         (vram_addr - adev->gmc.vram_start +
-                                          (void *)vram_start - vram_map),
-                                         (unsigned long long)
-                                         (gart_addr - adev->gmc.gart_start +
-                                          (void *)vram_start - vram_map));
-                               amdgpu_bo_kunmap(gtt_obj[i]);
-                               goto out_lclean_unpin;
-                       }
-               }
-
-               amdgpu_bo_kunmap(gtt_obj[i]);
-
-               DRM_INFO("Tested GTT->VRAM and VRAM->GTT copy for GTT offset 0x%llx\n",
-                        gart_addr - adev->gmc.gart_start);
-               continue;
-
-out_lclean_unpin:
-               amdgpu_bo_unpin(gtt_obj[i]);
-out_lclean_unres:
-               amdgpu_bo_unreserve(gtt_obj[i]);
-out_lclean_unref:
-               amdgpu_bo_unref(&gtt_obj[i]);
-out_lclean:
-               for (--i; i >= 0; --i) {
-                       amdgpu_bo_unpin(gtt_obj[i]);
-                       amdgpu_bo_unreserve(gtt_obj[i]);
-                       amdgpu_bo_unref(&gtt_obj[i]);
-               }
-               if (fence)
-                       dma_fence_put(fence);
-               break;
-       }
-
-       amdgpu_bo_unpin(vram_obj);
-out_unres:
-       amdgpu_bo_unreserve(vram_obj);
-out_unref:
-       amdgpu_bo_unref(&vram_obj);
-out_cleanup:
-       kfree(gtt_obj);
-       if (r) {
-               pr_warn("Error while testing BO move\n");
-       }
-}
-
-void amdgpu_test_moves(struct amdgpu_device *adev)
-{
-       if (adev->mman.buffer_funcs)
-               amdgpu_do_test_moves(adev);
-}