search for: bobroff

From: Sam Bobroff <sbobroff at linux.ibm.com> [ Upstream commit 3c6b8625dde82600fd03ad1fcba223f1303ee535 ] When unloading the bochs-drm driver, a warning message is printed by drm_mode_config_cleanup() because a reference is still held to one of the drm_connector structs. Correct this by calling drm_atomic_...

From: Sam Bobroff <sbobroff at linux.ibm.com> [ Upstream commit 3c6b8625dde82600fd03ad1fcba223f1303ee535 ] When unloading the bochs-drm driver, a warning message is printed by drm_mode_config_cleanup() because a reference is still held to one of the drm_connector structs. Correct this by calling drm_atomic_...

It's not a core function, and the matching atomic functions are also not in the core. Plus the suspend/resume helper is also already there. Needs a tiny bit of open-coding, but less midlayer beats that I think. Cc: Sam Bobroff <sbobroff at linux.ibm.com> Signed-off-by: Daniel Vetter <daniel.vetter at intel.com> Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com> Cc: Maxime Ripard <maxime.ripard at bootlin.com> Cc: Sean Paul <sean at poorly.run> Cc: David Airlie <airlied at linux....

...wrote: > > > > It's not a core function, and the matching atomic functions are also > > not in the core. Plus the suspend/resume helper is also already there. > > > > Needs a tiny bit of open-coding, but less midlayer beats that I think. > > > > Cc: Sam Bobroff <sbobroff at linux.ibm.com> > > Signed-off-by: Daniel Vetter <daniel.vetter at intel.com> > > Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com> > > Cc: Maxime Ripard <maxime.ripard at bootlin.com> > > Cc: Sean Paul <sean at poorly.run&gt...

On Tue, May 21, 2019 at 10:10:29AM +0200, Gerd Hoffmann wrote: > Hi, > > The bug is in the driver, so ... > > > Bisecting the issue for commits to drivers/gpu/drm/bochs/ points to: > > 6579c39594ae ("drm/bochs: atomic: switch planes to atomic, wire up helpers.") > > ... but the issue also seems to be due to a change in the generic drm code > > ...

...ction of two previous commits. Both together are required to cause it: Fixes: 846c7dfc1193 ("drm/atomic: Try to preserve the crtc enabled state in drm_atomic_remove_fb, v2.") Fixes: 6579c39594ae ("drm/bochs: atomic: switch planes to atomic, wire up helpers.") Signed-off-by: Sam Bobroff <sbobroff at linux.ibm.com> --- Hello, This seems to be similar to an issue I recently fixed for the AST driver: 1e613f3c630c ("drm/ast: Fix connector leak during driver unload") Which is similar to at least one other recent fix: 192b4af6cd28 ("drm/tegra: Shutdown on driver un...

...ction of two previous commits. Both together are required to cause it: Fixes: 846c7dfc1193 ("drm/atomic: Try to preserve the crtc enabled state in drm_atomic_remove_fb, v2.") Fixes: 6579c39594ae ("drm/bochs: atomic: switch planes to atomic, wire up helpers.") Signed-off-by: Sam Bobroff <sbobroff at linux.ibm.com> --- Hello, This seems to be similar to an issue I recently fixed for the AST driver: 1e613f3c630c ("drm/ast: Fix connector leak during driver unload") Which is similar to at least one other recent fix: 192b4af6cd28 ("drm/tegra: Shutdown on driver un...

...<daniel.vetter at ffwll.ch> wrote: > > It's not a core function, and the matching atomic functions are also > not in the core. Plus the suspend/resume helper is also already there. > > Needs a tiny bit of open-coding, but less midlayer beats that I think. > > Cc: Sam Bobroff <sbobroff at linux.ibm.com> > Signed-off-by: Daniel Vetter <daniel.vetter at intel.com> > Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com> > Cc: Maxime Ripard <maxime.ripard at bootlin.com> > Cc: Sean Paul <sean at poorly.run> > Cc: David Air...

It's not a core function, and the matching atomic functions are also not in the core. Plus the suspend/resume helper is also already there. Needs a tiny bit of open-coding, but less midlayer beats that I think. v2: Rebase onto ast (which gained a new user). Cc: Sam Bobroff <sbobroff at linux.ibm.com> Reviewed-by: Alex Deucher <alexander.deucher at amd.com> Reviewed-by: Sean Paul <sean at poorly.run> Signed-off-by: Daniel Vetter <daniel.vetter at intel.com> Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com> Cc: Maxime Ripard &lt...

...x@renesas.com> Yuta Ando <yuta.and@gmail.com> Yuval Mintz <yuvalmin@broadcom.com> Yuvaraj CD <yuvaraj.cd@samsung.com> Yuvaraj Kumar C D <yuvaraj.cd@samsung.com> Yvan FILLION <yvan.fillion@stericsson.com> Yves-Alexis Perez <corsac@debian.org> Zach Bobroff <zacharyb@ami.com> Zach Brown <zab@redhat.com> Zach Brown <zab@zabbo.net> Zach Sadecki <zsadecki@itwatchdogs.com> Zachary Amsden <zamsden@gmail.com> Zang MingJie <zealot0630@gmail.com> Zbigniew Bodek <zbb@semihalf.com> Zdenek Kabelac <zkabe...

...x@renesas.com> Yuta Ando <yuta.and@gmail.com> Yuval Mintz <yuvalmin@broadcom.com> Yuvaraj CD <yuvaraj.cd@samsung.com> Yuvaraj Kumar C D <yuvaraj.cd@samsung.com> Yvan FILLION <yvan.fillion@stericsson.com> Yves-Alexis Perez <corsac@debian.org> Zach Bobroff <zacharyb@ami.com> Zach Brown <zab@redhat.com> Zach Brown <zab@zabbo.net> Zach Sadecki <zsadecki@itwatchdogs.com> Zachary Amsden <zamsden@gmail.com> Zang MingJie <zealot0630@gmail.com> Zbigniew Bodek <zbb@semihalf.com> Zdenek Kabelac <zkabe...