Commit 0054eeb7 authored by Rob Clark's avatar Rob Clark

drm/msm: Fix spelling "purgable" -> "purgeable"

The previous patch fixes the user visible spelling.  This one fixes the
code.  Oops.
Signed-off-by: default avatarRob Clark <robdclark@chromium.org>
Link: https://lore.kernel.org/r/20210406151816.1515329-1-robdclark@gmail.comSigned-off-by: default avatarRob Clark <robdclark@chromium.org>
parent f1902c6b
...@@ -821,14 +821,14 @@ static void update_inactive(struct msm_gem_object *msm_obj) ...@@ -821,14 +821,14 @@ static void update_inactive(struct msm_gem_object *msm_obj)
WARN_ON(msm_obj->active_count != 0); WARN_ON(msm_obj->active_count != 0);
if (msm_obj->dontneed) if (msm_obj->dontneed)
mark_unpurgable(msm_obj); mark_unpurgeable(msm_obj);
list_del(&msm_obj->mm_list); list_del(&msm_obj->mm_list);
if (msm_obj->madv == MSM_MADV_WILLNEED) { if (msm_obj->madv == MSM_MADV_WILLNEED) {
list_add_tail(&msm_obj->mm_list, &priv->inactive_willneed); list_add_tail(&msm_obj->mm_list, &priv->inactive_willneed);
} else if (msm_obj->madv == MSM_MADV_DONTNEED) { } else if (msm_obj->madv == MSM_MADV_DONTNEED) {
list_add_tail(&msm_obj->mm_list, &priv->inactive_dontneed); list_add_tail(&msm_obj->mm_list, &priv->inactive_dontneed);
mark_purgable(msm_obj); mark_purgeable(msm_obj);
} else { } else {
WARN_ON(msm_obj->madv != __MSM_MADV_PURGED); WARN_ON(msm_obj->madv != __MSM_MADV_PURGED);
list_add_tail(&msm_obj->mm_list, &priv->inactive_purged); list_add_tail(&msm_obj->mm_list, &priv->inactive_purged);
...@@ -901,8 +901,8 @@ void msm_gem_describe(struct drm_gem_object *obj, struct seq_file *m, ...@@ -901,8 +901,8 @@ void msm_gem_describe(struct drm_gem_object *obj, struct seq_file *m,
madv = " purged"; madv = " purged";
break; break;
case MSM_MADV_DONTNEED: case MSM_MADV_DONTNEED:
stats->purgable.count++; stats->purgeable.count++;
stats->purgable.size += obj->size; stats->purgeable.size += obj->size;
madv = " purgeable"; madv = " purgeable";
break; break;
case MSM_MADV_WILLNEED: case MSM_MADV_WILLNEED:
...@@ -984,7 +984,7 @@ void msm_gem_describe_objects(struct list_head *list, struct seq_file *m) ...@@ -984,7 +984,7 @@ void msm_gem_describe_objects(struct list_head *list, struct seq_file *m)
seq_printf(m, "Active: %4d objects, %9zu bytes\n", seq_printf(m, "Active: %4d objects, %9zu bytes\n",
stats.active.count, stats.active.size); stats.active.count, stats.active.size);
seq_printf(m, "Purgeable: %4d objects, %9zu bytes\n", seq_printf(m, "Purgeable: %4d objects, %9zu bytes\n",
stats.purgable.count, stats.purgable.size); stats.purgeable.count, stats.purgeable.size);
seq_printf(m, "Purged: %4d objects, %9zu bytes\n", seq_printf(m, "Purged: %4d objects, %9zu bytes\n",
stats.purged.count, stats.purged.size); stats.purged.count, stats.purged.size);
} }
...@@ -1003,7 +1003,7 @@ void msm_gem_free_object(struct drm_gem_object *obj) ...@@ -1003,7 +1003,7 @@ void msm_gem_free_object(struct drm_gem_object *obj)
mutex_lock(&priv->mm_lock); mutex_lock(&priv->mm_lock);
if (msm_obj->dontneed) if (msm_obj->dontneed)
mark_unpurgable(msm_obj); mark_unpurgeable(msm_obj);
list_del(&msm_obj->mm_list); list_del(&msm_obj->mm_list);
mutex_unlock(&priv->mm_lock); mutex_unlock(&priv->mm_lock);
......
...@@ -163,7 +163,7 @@ struct msm_gem_stats { ...@@ -163,7 +163,7 @@ struct msm_gem_stats {
struct { struct {
unsigned count; unsigned count;
size_t size; size_t size;
} all, active, purgable, purged; } all, active, purgeable, purged;
}; };
void msm_gem_describe(struct drm_gem_object *obj, struct seq_file *m, void msm_gem_describe(struct drm_gem_object *obj, struct seq_file *m,
...@@ -207,8 +207,8 @@ static inline bool is_active(struct msm_gem_object *msm_obj) ...@@ -207,8 +207,8 @@ static inline bool is_active(struct msm_gem_object *msm_obj)
return msm_obj->active_count; return msm_obj->active_count;
} }
/* imported/exported objects are not purgable: */ /* imported/exported objects are not purgeable: */
static inline bool is_unpurgable(struct msm_gem_object *msm_obj) static inline bool is_unpurgeable(struct msm_gem_object *msm_obj)
{ {
return msm_obj->base.dma_buf && msm_obj->base.import_attach; return msm_obj->base.dma_buf && msm_obj->base.import_attach;
} }
...@@ -216,7 +216,7 @@ static inline bool is_unpurgable(struct msm_gem_object *msm_obj) ...@@ -216,7 +216,7 @@ static inline bool is_unpurgable(struct msm_gem_object *msm_obj)
static inline bool is_purgeable(struct msm_gem_object *msm_obj) static inline bool is_purgeable(struct msm_gem_object *msm_obj)
{ {
return (msm_obj->madv == MSM_MADV_DONTNEED) && msm_obj->sgt && return (msm_obj->madv == MSM_MADV_DONTNEED) && msm_obj->sgt &&
!is_unpurgable(msm_obj); !is_unpurgeable(msm_obj);
} }
static inline bool is_vunmapable(struct msm_gem_object *msm_obj) static inline bool is_vunmapable(struct msm_gem_object *msm_obj)
...@@ -225,13 +225,13 @@ static inline bool is_vunmapable(struct msm_gem_object *msm_obj) ...@@ -225,13 +225,13 @@ static inline bool is_vunmapable(struct msm_gem_object *msm_obj)
return (msm_obj->vmap_count == 0) && msm_obj->vaddr; return (msm_obj->vmap_count == 0) && msm_obj->vaddr;
} }
static inline void mark_purgable(struct msm_gem_object *msm_obj) static inline void mark_purgeable(struct msm_gem_object *msm_obj)
{ {
struct msm_drm_private *priv = msm_obj->base.dev->dev_private; struct msm_drm_private *priv = msm_obj->base.dev->dev_private;
WARN_ON(!mutex_is_locked(&priv->mm_lock)); WARN_ON(!mutex_is_locked(&priv->mm_lock));
if (is_unpurgable(msm_obj)) if (is_unpurgeable(msm_obj))
return; return;
if (WARN_ON(msm_obj->dontneed)) if (WARN_ON(msm_obj->dontneed))
...@@ -241,13 +241,13 @@ static inline void mark_purgable(struct msm_gem_object *msm_obj) ...@@ -241,13 +241,13 @@ static inline void mark_purgable(struct msm_gem_object *msm_obj)
msm_obj->dontneed = true; msm_obj->dontneed = true;
} }
static inline void mark_unpurgable(struct msm_gem_object *msm_obj) static inline void mark_unpurgeable(struct msm_gem_object *msm_obj)
{ {
struct msm_drm_private *priv = msm_obj->base.dev->dev_private; struct msm_drm_private *priv = msm_obj->base.dev->dev_private;
WARN_ON(!mutex_is_locked(&priv->mm_lock)); WARN_ON(!mutex_is_locked(&priv->mm_lock));
if (is_unpurgable(msm_obj)) if (is_unpurgeable(msm_obj))
return; return;
if (WARN_ON(!msm_obj->dontneed)) if (WARN_ON(!msm_obj->dontneed))
......
...@@ -49,7 +49,7 @@ msm_gem_shrinker_scan(struct shrinker *shrinker, struct shrink_control *sc) ...@@ -49,7 +49,7 @@ msm_gem_shrinker_scan(struct shrinker *shrinker, struct shrink_control *sc)
/* /*
* Now that we own a reference, we can drop mm_lock for the * Now that we own a reference, we can drop mm_lock for the
* rest of the loop body, to reduce contention with the * rest of the loop body, to reduce contention with the
* retire_submit path (which could make more objects purgable) * retire_submit path (which could make more objects purgeable)
*/ */
mutex_unlock(&priv->mm_lock); mutex_unlock(&priv->mm_lock);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment