On 09/06/2021 10:28, Baokun Li wrote:

Using list_move_tail() instead of list_del() + list_add_tail() in msm_gem.c.

Reported-by: Hulk Robot hulkci@huawei.com Signed-off-by: Baokun Li libaokun1@huawei.com

Reviewed-by: Dmitry Baryshkov dmitry.baryshkov@linaro.org

---

V1->V2: CC mailist

drivers/gpu/drm/msm/msm_gem.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/msm/msm_gem.c b/drivers/gpu/drm/msm/msm_gem.c index 1865919368f2..5b7d63d3750a 100644 --- a/drivers/gpu/drm/msm/msm_gem.c +++ b/drivers/gpu/drm/msm/msm_gem.c @@ -854,8 +854,7 @@ void msm_gem_active_get(struct drm_gem_object *obj, struct msm_gpu *gpu) mutex_lock(&priv->mm_lock); if (msm_obj->evictable) mark_unevictable(msm_obj);

• list_del(&msm_obj->mm_list);
  

• list_add_tail(&msm_obj->mm_list, &gpu->active_list);
  

• list_move_tail(&msm_obj->mm_list, &gpu->active_list);
  

  mutex_unlock(&priv->mm_lock); } }