Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV display device driver interface

[Ian Jackson <ian.jackson@xxxxxxxxxxxxx>]

Oleksandr Grytsov writes ("Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV display 
device driver interface"):
> I considered that frontend domain name and surface index is a unique surface
> identifier. Like following:
> 
> Surface with index 0 from DomU should be placed at x, y, display 0  etc.

Right, I see.

The difficulty with that is that as vdisplays might be added and
removed, there is not really a stable well-defined index.  At least,
if there is, then it would have to be specifiable in the config.

Eg, what if you decide to get rid of surface index 0 ?  Then there
would have to still be surface index 1.  So the indices have to be
settable.

> Surely, we can add identifier into xl.cfg to make it more generic or more
> readable from user point of view. In this case the config line could look 
> like:
> 
> vdispl = [ 'backend=0, devId=0, beAlloc=1,
> connectors=id0:800x600,id1:1024x768' ]
> 
> id0, id1 could be either integer id or string name. For example in our case it
> could be wayland surface id.

Right.  I think that would solve this problem.  I don't have an
opinion about whether these things should be integers or strings, but
you should pick one in your design.  If they are to be strings you
should specify a (fairly restrictive) character set.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel