Skip to content

Commit 0bdc0a0

Browse files
matt-auldtursulin
authored andcommitted
drm/i915/ttm: tweak priority hint selection
For some reason we are selecting PRIO_HAS_PAGES when we don't have mm.pages, and vice versa. v2(Thomas): - Add missing fixes tag Fixes: 213d509 ("drm/i915/ttm: Introduce a TTM i915 gem object backend") Signed-off-by: Matthew Auld <matthew.auld@intel.com> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> Reviewed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20220209111652.468762-1-matthew.auld@intel.com (cherry picked from commit ba2c5d1) Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
1 parent 8d9d2a7 commit 0bdc0a0

1 file changed

Lines changed: 2 additions & 4 deletions

File tree

drivers/gpu/drm/i915/gem/i915_gem_ttm.c

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -842,11 +842,9 @@ void i915_ttm_adjust_lru(struct drm_i915_gem_object *obj)
842842
} else if (obj->mm.madv != I915_MADV_WILLNEED) {
843843
bo->priority = I915_TTM_PRIO_PURGE;
844844
} else if (!i915_gem_object_has_pages(obj)) {
845-
if (bo->priority < I915_TTM_PRIO_HAS_PAGES)
846-
bo->priority = I915_TTM_PRIO_HAS_PAGES;
845+
bo->priority = I915_TTM_PRIO_NO_PAGES;
847846
} else {
848-
if (bo->priority > I915_TTM_PRIO_NO_PAGES)
849-
bo->priority = I915_TTM_PRIO_NO_PAGES;
847+
bo->priority = I915_TTM_PRIO_HAS_PAGES;
850848
}
851849

852850
ttm_bo_move_to_lru_tail(bo, bo->resource, NULL);

0 commit comments

Comments
 (0)