diff options
| author |   Rob Clark <robdclark@chromium.org> | 2021-07-27 18:06:15 -0700 |
|---|---|---|
| committer | Rob Clark <robdclark@chromium.org> | 2021-07-28 09:19:00 -0700 |
| commit | bd0b8e9f9c3c0f7f29884f62eec585a6396ba575 (patch) | |
| tree | c7e794df7070620d3be1826ed2766a1f1cb685f4 /drivers/gpu/drm/msm/msm_gem.c | |
| parent | drm/msm: Conversion to drm scheduler (diff) | |
| download | linux-dev-bd0b8e9f9c3c0f7f29884f62eec585a6396ba575.tar.xz linux-dev-bd0b8e9f9c3c0f7f29884f62eec585a6396ba575.zip | |
drm/msm: Drop submit bo_list
This was only used to detect userspace including the same bo multiple
times in a submit. But ww_mutex can already tell us this.
When we drop struct_mutex around the submit ioctl, we'd otherwise need
to lock the bo before adding it to the bo_list. But since ww_mutex can
already tell us this, it is simpler just to remove the bo_list.
Signed-off-by: Rob Clark <robdclark@chromium.org>
Link: https://lore.kernel.org/r/20210728010632.2633470-11-robdclark@gmail.com
Signed-off-by: Rob Clark <robdclark@chromium.org>
Diffstat (limited to 'drivers/gpu/drm/msm/msm_gem.c')
| -rw-r--r-- | drivers/gpu/drm/msm/msm_gem.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/gpu/drm/msm/msm_gem.c b/drivers/gpu/drm/msm/msm_gem.c index 3ab5c8620893..cfcca3975055 100644 --- a/drivers/gpu/drm/msm/msm_gem.c +++ b/drivers/gpu/drm/msm/msm_gem.c @@ -1148,7 +1148,6 @@ static int msm_gem_new_impl(struct drm_device *dev, msm_obj->flags = flags; msm_obj->madv = MSM_MADV_WILLNEED; - INIT_LIST_HEAD(&msm_obj->submit_entry); INIT_LIST_HEAD(&msm_obj->vmas); *obj = &msm_obj->base; |