From: Oleksandr Dmytryshyn <oleksandr.dmytryshyn@globallogic.com>
To: Ian Jackson <Ian.Jackson@eu.citrix.com>
Cc: Iurii Konovalenko <iurii.konovalenko@globallogic.com>,
Keir Fraser <keir@xen.org>,
Ian Campbell <ian.campbell@citrix.com>, Tim Deegan <tim@xen.org>,
xen-devel <xen-devel@lists.xen.org>,
Jan Beulich <jbeulich@suse.com>
Subject: Re: [PATCH v4] sndif: add ABI for Para-virtual sound
Date: Thu, 22 Jan 2015 16:22:42 +0200 [thread overview]
Message-ID: <CAN58jitpSNCvzrD-rTqGfYt=UkETv6hJpKxuLSDEVC0docjjSg@mail.gmail.com> (raw)
In-Reply-To: <21697.531.830005.516477@mariner.uk.xensource.com>
On Thu, Jan 22, 2015 at 3:58 PM, Ian Jackson <Ian.Jackson@eu.citrix.com> 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.
prev parent reply other threads:[~2015-01-22 14:22 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-22 13:04 [PATCH v4] sndif: add ABI for Para-virtual sound Oleksandr Dmytryshyn
2015-01-22 13:58 ` Ian Jackson
2015-01-22 14:22 ` Oleksandr Dmytryshyn [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAN58jitpSNCvzrD-rTqGfYt=UkETv6hJpKxuLSDEVC0docjjSg@mail.gmail.com' \
--to=oleksandr.dmytryshyn@globallogic.com \
--cc=Ian.Jackson@eu.citrix.com \
--cc=ian.campbell@citrix.com \
--cc=iurii.konovalenko@globallogic.com \
--cc=jbeulich@suse.com \
--cc=keir@xen.org \
--cc=tim@xen.org \
--cc=xen-devel@lists.xen.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.