drm/i915/gem: fix typos in i915/gem files
authorNitin Gote <nitin.r.gote@intel.com>
Mon, 20 Jan 2025 08:15:12 +0000 (13:45 +0530)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
Thu, 23 Jan 2025 10:48:23 +0000 (05:48 -0500)
Fix all typos in files under drm/i915/gem reported by codespell tool.

v2: Codespell won't catch it, but it should be
    "user defined" and not "use defined". <Krzysztof Niemiec>

Signed-off-by: Nitin Gote <nitin.r.gote@intel.com>
Reviewed-by: Krzysztof Niemiec <krzysztof.niemiec@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20250120081517.3237326-4-nitin.r.gote@intel.com
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
drivers/gpu/drm/i915/gem/i915_gem_context.c
drivers/gpu/drm/i915/gem/i915_gem_context_types.h
drivers/gpu/drm/i915/gem/i915_gem_domain.c
drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c
drivers/gpu/drm/i915/gem/i915_gem_region.c
drivers/gpu/drm/i915/gem/i915_gem_shrinker.c
drivers/gpu/drm/i915/gem/i915_gem_tiling.c
drivers/gpu/drm/i915/gem/i915_gem_ttm.c
drivers/gpu/drm/i915/gem/i915_gem_ttm_move.c
drivers/gpu/drm/i915/gem/selftests/huge_pages.c
drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c

index c0543c35cd6ac4a337858e34a89489e519d7a588..ab1af978911b520057b86a5838d4940c020359a2 100644 (file)
@@ -238,7 +238,7 @@ static int proto_context_set_persistence(struct drm_i915_private *i915,
                 *
                 * However, if we cannot reset an engine by itself, we cannot
                 * cleanup a hanging persistent context without causing
-                * colateral damage, and we should not pretend we can by
+                * collateral damage, and we should not pretend we can by
                 * exposing the interface.
                 */
                if (!intel_has_reset_engine(to_gt(i915)))
@@ -1589,7 +1589,7 @@ static int __context_set_persistence(struct i915_gem_context *ctx, bool state)
                 *
                 * However, if we cannot reset an engine by itself, we cannot
                 * cleanup a hanging persistent context without causing
-                * colateral damage, and we should not pretend we can by
+                * collateral damage, and we should not pretend we can by
                 * exposing the interface.
                 */
                if (!intel_has_reset_engine(to_gt(ctx->i915)))
@@ -2328,7 +2328,7 @@ finalize_create_context_locked(struct drm_i915_file_private *file_priv,
 
        /*
         * One for the xarray and one for the caller.  We need to grab
-        * the reference *prior* to making the ctx visble to userspace
+        * the reference *prior* to making the ctx visible to userspace
         * in gem_context_register(), as at any point after that
         * userspace can try to race us with another thread destroying
         * the context under our feet.
index b6d97da63d1fa9ac869ecdc7d5d9045a1be9ca27..67ac2586a0f3f262e256906882b1d1a08d660b46 100644 (file)
@@ -245,9 +245,9 @@ struct i915_gem_context {
         * Execbuf uses the I915_EXEC_RING_MASK as an index into this
         * array to select which HW context + engine to execute on. For
         * the default array, the user_ring_map[] is used to translate
-        * the legacy uABI onto the approprate index (e.g. both
+        * the legacy uABI onto the appropriate index (e.g. both
         * I915_EXEC_DEFAULT and I915_EXEC_RENDER select the same
-        * context, and I915_EXEC_BSD is weird). For a use defined
+        * context, and I915_EXEC_BSD is weird). For a user defined
         * array, execbuf uses I915_EXEC_RING_MASK as a plain index.
         *
         * User defined by I915_CONTEXT_PARAM_ENGINE (when the
@@ -276,7 +276,7 @@ struct i915_gem_context {
         * @vm: unique address space (GTT)
         *
         * In full-ppgtt mode, each context has its own address space ensuring
-        * complete seperation of one client from all others.
+        * complete separation of one client from all others.
         *
         * In other modes, this is a NULL pointer with the expectation that
         * the caller uses the shared global GTT.
index 3770828f2eaf22fa720f600559fb64ce33b38a4a..ee55caca67a11b7e892ad1b465eff58e75a77b24 100644 (file)
@@ -276,7 +276,7 @@ int i915_gem_object_set_cache_level(struct drm_i915_gem_object *obj,
         * For objects created by userspace through GEM_CREATE with pat_index
         * set by set_pat extension, simply return 0 here without touching
         * the cache setting, because such objects should have an immutable
-        * cache setting by desgin and always managed by userspace.
+        * cache setting by design and always managed by userspace.
         */
        if (i915_gem_object_has_cache_level(obj, cache_level))
                return 0;
index f151640c1d13ccffa87c5577b8bf6e0f9edba192..c8107502190d23eff88b1390fd1c4a4d852cfe99 100644 (file)
@@ -303,7 +303,7 @@ struct i915_execbuffer {
        struct intel_gt_buffer_pool_node *batch_pool; /** pool node for batch buffer */
 
        /**
-        * Indicate either the size of the hastable used to resolve
+        * Indicate either the size of the hashtable used to resolve
         * relocation handles, or if negative that we are using a direct
         * index into the execobj[].
         */
@@ -2543,7 +2543,7 @@ static int eb_pin_timeline(struct i915_execbuffer *eb, struct intel_context *ce,
 
                        /*
                         * Error path, cannot use intel_context_timeline_lock as
-                        * that is user interruptable and this clean up step
+                        * that is user interruptible and this clean up step
                         * must be done.
                         */
                        mutex_lock(&ce->timeline->mutex);
index b09b74a2448b55ec6d704eecf3a717e2e2a228d3..636768d0f57eb5a70b832885f9aae6c16f46a024 100644 (file)
@@ -82,7 +82,7 @@ __i915_gem_object_create_region(struct intel_memory_region *mem,
 
        /*
         * Anything smaller than the min_page_size can't be freely inserted into
-        * the GTT, due to alignemnt restrictions. For such special objects,
+        * the GTT, due to alignment restrictions. For such special objects,
         * make sure we force memcpy based suspend-resume. In the future we can
         * revisit this, either by allowing special mis-aligned objects in the
         * migration path, or by mapping all of LMEM upfront using cheap 1G
index 9117e94228440b1f83312f7c527694eb0dbeb743..aec41f0f098f63b432088ba201b26820b43998ed 100644 (file)
@@ -25,7 +25,7 @@ static bool swap_available(void)
 
 static bool can_release_pages(struct drm_i915_gem_object *obj)
 {
-       /* Consider only shrinkable ojects. */
+       /* Consider only shrinkable objects. */
        if (!i915_gem_object_is_shrinkable(obj))
                return false;
 
@@ -261,7 +261,7 @@ skip:
  * i915_gem_shrink_all - Shrink buffer object caches completely
  * @i915: i915 device
  *
- * This is a simple wraper around i915_gem_shrink() to aggressively shrink all
+ * This is a simple wrapper around i915_gem_shrink() to aggressively shrink all
  * caches completely. It also first waits for and retires all outstanding
  * requests to also be able to release backing storage for active objects.
  *
index d9eb84c1d2f11668b4b6bf722b6efe7899e9c9b8..5ac23ff3feff2550246d49df6cfcd7e191ca7898 100644 (file)
@@ -39,7 +39,7 @@
  * Since neither of this applies for new tiling layouts on modern platforms like
  * W, Ys and Yf tiling GEM only allows object tiling to be set to X or Y tiled.
  * Anything else can be handled in userspace entirely without the kernel's
- * invovlement.
+ * involvement.
  */
 
 /**
index 10d8673641f74feaf9f45c91770355e5858f778b..1f4814968868457ba8570cbcaecb94c7d757ea87 100644 (file)
@@ -994,7 +994,7 @@ void i915_ttm_adjust_lru(struct drm_i915_gem_object *obj)
                 * If we need to place an LMEM resource which doesn't need CPU
                 * access then we should try not to victimize mappable objects
                 * first, since we likely end up stealing more of the mappable
-                * portion. And likewise when we try to find space for a mappble
+                * portion. And likewise when we try to find space for a mappable
                 * object, we know not to ever victimize objects that don't
                 * occupy any mappable pages.
                 */
index 041dab543b78efdb0e60bdd7d6122b33c58d7af2..2f6b33edb9c9f6f728b700ab7693fe034753f007 100644 (file)
@@ -603,7 +603,7 @@ int i915_ttm_move(struct ttm_buffer_object *bo, bool evict,
                 * sequence, where at the end we can do the move for real.
                 *
                 * The special case here is when the dst_mem is TTM_PL_SYSTEM,
-                * which doens't require any kind of move, so it should be safe
+                * which doesn't require any kind of move, so it should be safe
                 * to skip all the below and call ttm_bo_move_null() here, where
                 * the caller in __i915_ttm_get_pages() will take care of the
                 * rest, since we should have a valid ttm_tt.
index 84d41e6ccf0561e45d6ead12b125ebca0651d4af..bd08605a1611fe73c1b2458028d315b5d3a70fd7 100644 (file)
@@ -1781,7 +1781,7 @@ static int igt_tmpfs_fallback(void *arg)
 
        /*
         * Make sure that we don't burst into a ball of flames upon falling back
-        * to tmpfs, which we rely on if on the off-chance we encouter a failure
+        * to tmpfs, which we rely on if on the off-chance we encounter a failure
         * when setting up gemfs.
         */
 
index 99a9ade739562ed8c2e9a9de32bb7b5655795cdf..804f74084bd4f18e0d8b93b856cf747cc583d8a9 100644 (file)
@@ -1342,7 +1342,7 @@ static int igt_mmap_migrate(void *arg)
                }
 
                /*
-                * Allocate in the mappable portion, should be no suprises here.
+                * Allocate in the mappable portion, should be no surprises here.
                 */
                err = __igt_mmap_migrate(mixed, ARRAY_SIZE(mixed), mr, 0);
                if (err)