[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 12/12] ui/console: call gfx_switch() even if the current s
From: |
Marc-André Lureau |
Subject: |
Re: [PATCH v3 12/12] ui/console: call gfx_switch() even if the current scanout is GL |
Date: |
Mon, 7 Mar 2022 15:50:24 +0400 |
Hi
On Mon, Mar 7, 2022 at 2:35 PM Akihiko Odaki <akihiko.odaki@gmail.com> wrote:
>
> On 2022/03/07 19:19, Marc-André Lureau wrote:
> > Hi Akihiko
> >
> > On Mon, Mar 7, 2022 at 12:09 PM Akihiko Odaki <akihiko.odaki@gmail.com>
> > wrote:
> >>
> >> On 2022/03/07 16:46, marcandre.lureau@redhat.com wrote:
> >>> From: Marc-André Lureau <marcandre.lureau@redhat.com>
> >>>
> >>> egl-headless depends on the backing surface to be set before texture are
> >>> set and updated. Display it (update=true) iff the current scanout kind
> >>> is SURFACE.
> >>
> >> egl-headless does not dynamically call register_displaychangelistener
> >> and has console associated (console_select would not affect egl-headless
> >> itself) so this should not be necessary.
> >
> > Could you help me understand, what should not be necessary?
>
> I read the description as it sets the backing surface for egl-headless
> when register_displaychangelistener or console_select is called. The
> change is not necessary.
Without it, gfx_switch is not called to set the new surface. Switching
console with VNC would fail (via ctrl+alt+num).
>
> >
> >> The remaining problem with egl-headless is that egl-headless renders the
> >> image to DisplaySurface, and a non-OpenGL display (namely vnc) has to
> >> consume it instead of texture even when con->scanout.kind is
> >> SCANOUT_TEXTURE or SCANOUT_DMABUF.
> >
> > This is already happening, because egl-headless calls dpy_gfx_update().
>
> It is not called when register_displaychangelistener or console_select
> is called by non-OpenGL display consuming the DisplaySurface.
When displaychangelistener_display_console() is called with
con->scanount.kind == SCANOUT_SURFACE, it calls gfx_update(update ==
TRUE), and thus calls gfx_update on the whole surface.
- [PATCH v3 06/12] ui/shader: fix potential leak of shader on error, (continued)
- [PATCH v3 06/12] ui/shader: fix potential leak of shader on error, marcandre . lureau, 2022/03/07
- [PATCH v3 07/12] ui/shader: free associated programs, marcandre . lureau, 2022/03/07
- [PATCH v3 08/12] ui/console: add a dpy_gfx_switch callback helper, marcandre . lureau, 2022/03/07
- [PATCH v3 09/12] ui/console: optionally update after gfx switch, marcandre . lureau, 2022/03/07
- [PATCH v3 10/12] ui/dbus: fix texture sharing, marcandre . lureau, 2022/03/07
- [PATCH v3 11/12] ui/dbus: do not send 2d scanout until gfx_update, marcandre . lureau, 2022/03/07
- [PATCH v3 12/12] ui/console: call gfx_switch() even if the current scanout is GL, marcandre . lureau, 2022/03/07
- Re: [PATCH v3 12/12] ui/console: call gfx_switch() even if the current scanout is GL, Akihiko Odaki, 2022/03/07
- Re: [PATCH v3 12/12] ui/console: call gfx_switch() even if the current scanout is GL, Marc-André Lureau, 2022/03/07
- Re: [PATCH v3 12/12] ui/console: call gfx_switch() even if the current scanout is GL, Akihiko Odaki, 2022/03/07
- Re: [PATCH v3 12/12] ui/console: call gfx_switch() even if the current scanout is GL,
Marc-André Lureau <=
- Re: [PATCH v3 12/12] ui/console: call gfx_switch() even if the current scanout is GL, Akihiko Odaki, 2022/03/07
- Re: [PATCH v3 12/12] ui/console: call gfx_switch() even if the current scanout is GL, Marc-André Lureau, 2022/03/07
- Re: [PATCH v3 12/12] ui/console: call gfx_switch() even if the current scanout is GL, Akihiko Odaki, 2022/03/07
- Re: [PATCH v3 12/12] ui/console: call gfx_switch() even if the current scanout is GL, Marc-André Lureau, 2022/03/08
- Re: [PATCH v3 12/12] ui/console: call gfx_switch() even if the current scanout is GL, Akihiko Odaki, 2022/03/08
- Re: [PATCH v3 12/12] ui/console: call gfx_switch() even if the current scanout is GL, Marc-André Lureau, 2022/03/09
- Re: [PATCH v3 12/12] ui/console: call gfx_switch() even if the current scanout is GL, Akihiko Odaki, 2022/03/09
- Re: [PATCH v3 12/12] ui/console: call gfx_switch() even if the current scanout is GL, Marc-André Lureau, 2022/03/09
- Re: [PATCH v3 12/12] ui/console: call gfx_switch() even if the current scanout is GL, Akihiko Odaki, 2022/03/09
- Re: [PATCH v3 12/12] ui/console: call gfx_switch() even if the current scanout is GL, Marc-André Lureau, 2022/03/09