summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDaniel Vetter <daniel.vetter@ffwll.ch>2019-06-07 00:27:48 +0200
committerRodrigo Siqueira <rodrigosiqueiramelo@gmail.com>2019-06-27 04:13:10 +0200
commit64cfaa5092715a00a5cdbd3d804d16a38554fe40 (patch)
tree5a1c109a4cd9a0162fa83bd0763e9d0ea7d9b3b5
parentdrm/vkms: flush crc workers earlier in commit flow (diff)
downloadlinux-64cfaa5092715a00a5cdbd3d804d16a38554fe40.tar.xz
linux-64cfaa5092715a00a5cdbd3d804d16a38554fe40.zip
drm/vkms: Dont flush crc worker when we change crc status
The crc core code can cope with some late crc, the race is kinda unavoidable. So no need to flush pending workers, they'll complete in time. Cc: Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com> Cc: Haneen Mohammed <hamohammed.sa@gmail.com> Cc: Daniel Vetter <daniel@ffwll.ch> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com> Reviewed-by: Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com> Tested-by: Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com> Signed-off-by: Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com> Link: https://patchwork.freedesktop.org/patch/msgid/20190606222751.32567-8-daniel.vetter@ffwll.ch
-rw-r--r--drivers/gpu/drm/vkms/vkms_crc.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/gpu/drm/vkms/vkms_crc.c b/drivers/gpu/drm/vkms/vkms_crc.c
index 4bb08c3611fc..52964178acb4 100644
--- a/drivers/gpu/drm/vkms/vkms_crc.c
+++ b/drivers/gpu/drm/vkms/vkms_crc.c
@@ -258,9 +258,6 @@ int vkms_set_crc_source(struct drm_crtc *crtc, const char *src_name)
ret = vkms_crc_parse_source(src_name, &enabled);
- /* make sure nothing is scheduled on crtc workq */
- flush_workqueue(out->crc_workq);
-
spin_lock_irq(&out->lock);
out->crc_enabled = enabled;
spin_unlock_irq(&out->lock);