projects
/
platform
/
kernel
/
linux-rpi.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
3cfd070
)
drm/msm: Fix dirtyfb refcounting
author
Rob Clark
<robdclark@chromium.org>
Fri, 4 Mar 2022 20:21:45 +0000
(12:21 -0800)
committer
Greg Kroah-Hartman
<gregkh@linuxfoundation.org>
Wed, 17 Aug 2022 12:24:30 +0000
(14:24 +0200)
commit
9225b337072a10bf9b09df8bf281437488dd8a26
upstream.
refcount_t complains about 0->1 transitions, which isn't *quite* what we
wanted. So use dirtyfb==1 to mean that the fb is not connected to any
output that requires dirtyfb flushing, so that we can keep the underflow
and overflow checking.
Fixes:
9e4dde28e9cd
("drm/msm: Avoid dirtyfb stalls on video mode displays (v2)")
Signed-off-by: Rob Clark <robdclark@chromium.org>
Link:
https://lore.kernel.org/r/20220304202146.845566-1-robdclark@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/gpu/drm/msm/msm_fb.c
patch
|
blob
|
history
diff --git
a/drivers/gpu/drm/msm/msm_fb.c
b/drivers/gpu/drm/msm/msm_fb.c
index
96b379a
..
7137492
100644
(file)
--- a/
drivers/gpu/drm/msm/msm_fb.c
+++ b/
drivers/gpu/drm/msm/msm_fb.c
@@
-37,7
+37,7
@@
static int msm_framebuffer_dirtyfb(struct drm_framebuffer *fb,
/* If this fb is not used on any display requiring pixel data to be
* flushed, then skip dirtyfb
*/
- if (refcount_read(&msm_fb->dirtyfb) ==
0
)
+ if (refcount_read(&msm_fb->dirtyfb) ==
1
)
return 0;
return drm_atomic_helper_dirtyfb(fb, file_priv, flags, color,
@@
-221,6
+221,8
@@
static struct drm_framebuffer *msm_framebuffer_init(struct drm_device *dev,
goto fail;
}
+ refcount_set(&msm_fb->dirtyfb, 1);
+
drm_dbg_state(dev, "create: FB ID: %d (%p)", fb->base.id, fb);
return fb;