[Spice-devel] [PATCH spice-protocol] support multiple monitors on a single display channel

Alon Levy alevy at redhat.com
Mon Jul 16 00:43:21 PDT 2012


On Mon, Jul 16, 2012 at 09:27:59AM +0200, Christophe Fergeau wrote:
> On Sat, Jul 14, 2012 at 05:08:24PM +0300, Alon Levy wrote:
> > On Fri, Jul 13, 2012 at 11:39:06AM +0200, Christophe Fergeau wrote:
> > > On Thu, Jul 12, 2012 at 07:49:45PM +0300, Alon Levy wrote:
> > > > Adds on device:
> > > > 
> > > > RAM
> > > >  Header
> > > >   monitors_config - pointer
> > > > 
> > > >  QXLMonitorsConfig
> > > >   count == n
> > > >   max_allowed = N >= 0
> > > >   QXLHead 1
> > > >   ...
> > > >   QXLHead n
> > > >    id, surface_id, x, y, width, height, flags
> > > 
> > > Do we want to have some header size fields in QXLMonitorsConfig and QXLHead
> > > to be able to add data to them in the future while keeping compat? Assuming
> > > there's any point in being compatible there...
> > 
> > We can always bump the version of the device in this case. We'll have to
> > anyway to let the driver know the device expects it to use any new
> > fields.
> 
> Yes, but old drivers would still work with the new layout, no idea if this
> is something that we are interested in.

... and after sending, I actually understood what you were talking
about. Yes, we would have to support both the old monitors_config and a
new one in that case. I'm not sure if it is worth it to change or not.

> 
> Christophe



> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel



More information about the Spice-devel mailing list