Re: [Xen-devel] [PATCH v4] sndif: add ABI for Para-virtual sound

[Oleksandr Dmytryshyn <oleksandr.dmytryshyn@xxxxxxxxxxxxxxx>]

On Thu, Jan 22, 2015 at 3:58 PM, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx> wrote:
> Oleksandr Dmytryshyn writes ("[PATCH v4] sndif: add ABI for Para-virtual 
> sound"):
>> This is ABI for the two halves of a Para-virtual
>> sound driver to communicate with each to other.
> ...
>
> I think the unnecessary difference between request and response
> headers is undesirable.  I would put the id at the front of both.
I'll do this in the next patch-set.

> Where do the fixed packet size of 80 bytes and the fixed maximum
> channel of 15 come from ?  What if we want to change these later ?
> Is that likely ?  What's our extension plan ?
One of the possible ways - add an extended version of the SET/GET volume
which will be similar to read/write operation (using additional shared pages).
And in this case there will be 2 sets of commands SET/GET volume.

> Ian.

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