Re: [PATCH spice 14/18] Add new spice-gl stubs API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




----- Original Message -----
> > 
> > - spice_gl_scanout() to take the current scanout
> > 
> > - spice_gl_draw_async() to draw the scanout, is like other Spice async
> >   functions, it takes a cookie and will return in the
> >   QXLInterface.async_complete()
> > 
> > Two new fields are also added to QXLState, in order to save the current
> > scanout, and the pending async.
> > 
> > Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxx>
> > ---
> >  server/red-dispatcher.c  | 34 ++++++++++++++++++++++++++++++++++
> >  server/reds.c            |  1 +
> >  server/reds.h            |  2 ++
> >  server/spice-qxl.h       | 11 +++++++++++
> >  server/spice-server.syms |  6 ++++++
> >  5 files changed, 54 insertions(+)
> > 
> > diff --git a/server/red-dispatcher.c b/server/red-dispatcher.c
> > index 0810798..a1e4e1e 100644
> > --- a/server/red-dispatcher.c
> > +++ b/server/red-dispatcher.c
> > @@ -959,6 +959,40 @@ void spice_qxl_driver_unload(QXLInstance *instance)
> >      red_dispatcher_driver_unload(instance->st->dispatcher);
> >  }
> >  
> > +SPICE_GNUC_VISIBLE
> > +void spice_gl_scanout(QXLInstance *qxl,
> > +                      int fd,
> > +                      uint32_t width, uint32_t height,
> > +                      uint32_t stride, uint32_t format,
> > +                      int y_0_top)
> > +{
> > +    spice_return_if_fail(qxl != NULL);
> > +
> > +    if (qxl->st->scanout.fd != -1) {
> > +        close(qxl->st->scanout.fd);
> > +    }
> > +
> > +    qxl->st->scanout = (SpiceMsgDisplayGlScanoutUnix) {
> > +        .flags = y_0_top ? SPICE_GL_SCANOUT_FLAGS_Y0TOP : 0,
> > +        .fd = fd,
> > +        .width = width,
> > +        .height = height,
> > +        .stride = stride,
> > +        .format = format
> > +    };
> > +}
> > +
> 
> Where the format enumeration came?
> Which kind of file descriptor is fd?

As explained in cover it's from spice-common & spice-protocol (imho it's best to discuss the protocol first, but I got no feedback so far)
 
> Is there a patch set for spice-gtk and/or Qemu?

There is a wip qemu branch: https://github.com/elmarco/qemu/tree/virgl
and wip spice-gtk branch: https://github.com/elmarco/spice-gtk/tree/virgl

(I am waiting for protocol & server bits before sending those)

> 
> > +SPICE_GNUC_VISIBLE
> > +void spice_gl_draw_async(QXLInstance *qxl,
> > +                         uint32_t x, uint32_t y,
> > +                         uint32_t w, uint32_t h,
> > +                         uint64_t cookie)
> > +{
> > +    spice_return_if_fail(qxl != NULL);
> > +    spice_return_if_fail(qxl->st->scanout.fd != -1);
> > +    spice_return_if_fail(qxl->st->gl_draw_async == NULL);
> > +}
> > +
> >  void red_dispatcher_async_complete(struct RedDispatcher *dispatcher,
> >                                     AsyncCommand *async_command)
> >  {
> > diff --git a/server/reds.c b/server/reds.c
> > index e8cf168..fafa135 100644
> > --- a/server/reds.c
> > +++ b/server/reds.c
> > @@ -3155,6 +3155,7 @@ SPICE_GNUC_VISIBLE int
> > spice_server_add_interface(SpiceServer *s,
> >  
> >          qxl = SPICE_CONTAINEROF(sin, QXLInstance, base);
> >          qxl->st = spice_new0(QXLState, 1);
> > +        qxl->st->scanout.fd = -1;
> >          qxl->st->qif = SPICE_CONTAINEROF(interface, QXLInterface, base);
> >          red_dispatcher_init(qxl);
> >  
> > diff --git a/server/reds.h b/server/reds.h
> > index 4d1b631..07ad8f9 100644
> > --- a/server/reds.h
> > +++ b/server/reds.h
> > @@ -33,6 +33,8 @@
> >  struct QXLState {
> >      QXLInterface          *qif;
> >      struct RedDispatcher  *dispatcher;
> > +    SpiceMsgDisplayGlScanoutUnix scanout;
> > +    AsyncCommand *gl_draw_async;
> >  };
> >  
> >  struct TunnelWorker;
> > diff --git a/server/spice-qxl.h b/server/spice-qxl.h
> > index e1f14e7..466b95c 100644
> > --- a/server/spice-qxl.h
> > +++ b/server/spice-qxl.h
> > @@ -193,4 +193,15 @@ struct QXLInstance {
> >      QXLState           *st;
> >  };
> >  
> > +void spice_gl_scanout(QXLInstance *instance,
> > +                      int fd,
> > +                      uint32_t width, uint32_t height,
> > +                      uint32_t stride, uint32_t format,
> > +                      int y_0_top);
> > +
> > +void spice_gl_draw_async(QXLInstance *instance,
> > +                         uint32_t x, uint32_t y,
> > +                         uint32_t w, uint32_t h,
> > +                         uint64_t cookie);
> > +
> >  #endif /* SPICE_QXL_H_ */
> > diff --git a/server/spice-server.syms b/server/spice-server.syms
> > index d65e14d..09ed19f 100644
> > --- a/server/spice-server.syms
> > +++ b/server/spice-server.syms
> > @@ -162,3 +162,9 @@ global:
> >      spice_replay_next_cmd;
> >      spice_replay_free_cmd;
> >  } SPICE_SERVER_0.12.5;
> > +
> > +SPICE_SERVER_0.13 {
> > +global:
> > +    spice_gl_scanout;
> > +    spice_gl_draw_async;
> > +} SPICE_SERVER_0.12.6;
> 
> Frediano
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
> 
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]