On 2 May 2017 at 11:52, Daniel Stone <[email protected]> wrote: > ... > > Signed-off-by: Daniel Stone <[email protected]> > --- > drm-atomic.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drm-atomic.c b/drm-atomic.c > index 25e04b1..6fb40fd 100644 > --- a/drm-atomic.c > +++ b/drm-atomic.c > @@ -368,6 +368,9 @@ const struct drm * init_drm_atomic(const char *device) > drm.crtc = calloc(1, sizeof(*drm.crtc)); > drm.connector = calloc(1, sizeof(*drm.connector)); > > + drm.kms_in_fence_fd = -1; > + drm.kms_out_fence_fd = -1; > + Seems to be the correct place to handle here, despite that both are already handled.
Patches 1/4 + 4/4 are Reviewed-by: Emil Velikov <[email protected]> As a follow-up can we: - drop atomic_run's "drm.kms_in_fence_fd = -1;" + git revert a46366c9903efc8b63eb4e78ae72f869be0b0a3d - in drm_atomic_commit() use "(flags & DRM_MODE_ATOMIC_ALLOW_MODESET) == 0" over "drm.kms_in_fence_fd != -1" -Emil _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
