On Fri, Mar 30, 2018 at 11:39:05AM -0700, Eric Anholt wrote: > Boris Brezillon <[email protected]> writes: > > > ->atomic_async_update() requires that drivers update the plane->state > > object before returning. Make sure at least common properties have been > > updated. > > > > Cc: Gustavo Padovan <[email protected]> > > Signed-off-by: Boris Brezillon <[email protected]> > > --- > > Hello, > > > > This is a problem I had when debugging the VC4 ->atomic_async_update() > > implementation. The function was not updating plane->fb as it's > > supposed thus leaving plane->state->fb in an inconsistent state after > > each async update. > > > > Not sure if such WARN_ON_ONCE() are accepted in the core though, so > > I'll maintainers decide whether this is relevant or not and whether > > they prefer to have WARN_ON() or DRM_ERROR() messages. > > I'm a huge fan of this kind of safety check. FWIW, > > Acked-by: Eric Anholt <[email protected]>
Agreed, seems like a useful safety net to me. Acked-by: Sean Paul <[email protected]> > _______________________________________________ > dri-devel mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Sean Paul, Software Engineer, Google / Chromium OS _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
