[PATCH weston] compositor-drm: reset KMS state on VT-switch in

Pekka Paalanen ppaalanen at gmail.com
Thu Jul 27 11:07:31 UTC 2017


On Thu, 27 Jul 2017 11:54:43 +0100
Daniel Stone <daniel at fooishbar.org> wrote:

> Hi,
> 
> On 27 July 2017 at 11:51, Pekka Paalanen <ppaalanen at gmail.com> wrote:
> > @@ -2810,6 +2820,7 @@ create_output_for_connector(struct drm_backend *b,
> >         if (output == NULL)
> >                 goto err;
> >
> > +       output->state_invalid = true;
> >         output->connector = connector;
> >         output->crtc_id = resources->crtcs[i];
> >         output->pipe = i;
> > @@ -3130,6 +3141,10 @@ session_notify(struct wl_listener *listener, void *data)
> >                 weston_log("activating session\n");
> >                 weston_compositor_wake(compositor);
> >                 weston_compositor_damage_all(compositor);
> > +
> > +               wl_list_for_each(output, &compositor->output_list, base.link)
> > +                       output->state_invalid = true;
> > +  
> 
> In addition to output creation, and on VT enter, should we also be
> resetting the state on output enable? Or maybe just moving it there
> from create_output_for_connector, rather than both. Either way:

Hi,

yes, we should indeed. But I don't think enable is called on VT enter,
so we still need it on VT enter explicitly. I'll remove it from create
output, as that path always goes through enable, and have it in enable.

> Reviewed-by: Daniel Stone <daniels at collabora.com>

Will test and push.


Thanks,
pq
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20170727/13876c70/attachment.sig>


More information about the wayland-devel mailing list