diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2019-06-07 00:27:48 +0200 |
---|---|---|
committer | Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com> | 2019-06-27 04:13:10 +0200 |
commit | 64cfaa5092715a00a5cdbd3d804d16a38554fe40 (patch) | |
tree | 5a1c109a4cd9a0162fa83bd0763e9d0ea7d9b3b5 | |
parent | drm/vkms: flush crc workers earlier in commit flow (diff) | |
download | linux-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.c | 3 |
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); |