> Hi
>
> ----- Original Message -----
> > >
> > > Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxx>
> > > ---
> > > server/dcc.c | 11 +++++++++++
> > > 1 file changed, 11 insertions(+)
> > >
> > > diff --git a/server/dcc.c b/server/dcc.c
> > > index 3161375..193d630 100644
> > > --- a/server/dcc.c
> > > +++ b/server/dcc.c
> > > @@ -445,6 +445,7 @@ void dcc_start(DisplayChannelClient *dcc)
> > > {
> > > DisplayChannel *display = DCC_TO_DC(dcc);
> > > RedChannelClient *rcc = RED_CHANNEL_CLIENT(dcc);
> > > + QXLInstance *qxl =
> > > red_worker_get_qxl(COMMON_CHANNEL(display)->worker);
> > >
> > > red_channel_client_push_set_ack(RED_CHANNEL_CLIENT(dcc));
> > >
> > > @@ -464,6 +465,16 @@ void dcc_start(DisplayChannelClient *dcc)
> > > red_pipe_add_verb(rcc, SPICE_MSG_DISPLAY_MARK);
> > > dcc_create_all_streams(dcc);
> > > }
> > > +
> > > + if (qxl->st->scanout.drm_dma_buf_fd >= 0) {
> > > + if (reds_stream_is_plain_unix(rcc->stream) &&
> > > + red_channel_client_test_remote_cap(rcc,
> > > SPICE_DISPLAY_CAP_GL_SCANOUT)) {
> > > + red_channel_client_pipe_add(rcc,
> > > dcc_gl_scanout_item_new(rcc,
> > > NULL, 0));
> > > + dcc_push_monitors_config(dcc);
> > > + } else {
> > > + spice_printerr("FIXME: GL not supported on this kind of
> > > connection");
> > > + }
> > > + }
> > > }
> > >
> > > static void dcc_destroy_stream_agents(DisplayChannelClient *dcc)
> >
> > Yes, I think definitively scanout should be in DisplayChannel
>
> See discussion in previous mail. (using definitively without arguments is
> moot you know)
>
That's because all display state is mainly in DisplayChannel (like monitors_config).
This patch make clear that DisplayChannel has to know RedWorker which has to know
QXL.
You will note that calls to red_worker_get_qxl are really limited. They should be
removed. You added 4 calls.
I think would be better if spice_gl_scanout call a function that make DisplayChannel
do the change for you.
Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel