| Message ID | 20181129195838.222031-1-sean@poorly.run (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v2] drm: TODO: Add DRM_MODESET_LOCK_ALL* conversion to todo.rst | expand |
On Thu, Nov 29, 2018 at 02:58:33PM -0500, Sean Paul wrote: > From: Sean Paul <seanpaul@chromium.org> > > We should also get the drivers using the helpers. Seems like a good > intermediate TODO item. > > Changes in v2: > - Add line about replacing drm_modest_lock_all() (Daniel) > > Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> Pushed this to -misc-next > Signed-off-by: Sean Paul <seanpaul@chromium.org> > --- > Documentation/gpu/todo.rst | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst > index ab347dec50796..14191b64446df 100644 > --- a/Documentation/gpu/todo.rst > +++ b/Documentation/gpu/todo.rst > @@ -241,6 +241,21 @@ struct drm_gem_object_funcs > GEM objects can now have a function table instead of having the callbacks on the > DRM driver struct. This is now the preferred way and drivers can be moved over. > > +Use DRM_MODESET_LOCK_ALL_* helpers instead of boilerplate > +--------------------------------------------------------- > + > +For cases where drivers are attempting to grab the modeset locks with a local > +acquire context. Replace the boilerplate code surrounding > +drm_modeset_lock_all_ctx() with DRM_MODESET_LOCK_ALL_BEGIN() and > +DRM_MODESET_LOCK_ALL_END() instead. > + > +This should also be done for all places where drm_modest_lock_all() is still > +used. > + > +As a reference, take a look at the conversions already completed in drm core. > + > +Contact: Sean Paul, respective driver maintainers > + > Core refactorings > ================= > > -- > Sean Paul, Software Engineer, Google / Chromium OS >
diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst index ab347dec50796..14191b64446df 100644 --- a/Documentation/gpu/todo.rst +++ b/Documentation/gpu/todo.rst @@ -241,6 +241,21 @@ struct drm_gem_object_funcs GEM objects can now have a function table instead of having the callbacks on the DRM driver struct. This is now the preferred way and drivers can be moved over. +Use DRM_MODESET_LOCK_ALL_* helpers instead of boilerplate +--------------------------------------------------------- + +For cases where drivers are attempting to grab the modeset locks with a local +acquire context. Replace the boilerplate code surrounding +drm_modeset_lock_all_ctx() with DRM_MODESET_LOCK_ALL_BEGIN() and +DRM_MODESET_LOCK_ALL_END() instead. + +This should also be done for all places where drm_modest_lock_all() is still +used. + +As a reference, take a look at the conversions already completed in drm core. + +Contact: Sean Paul, respective driver maintainers + Core refactorings =================