From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [PATCH RESEND] xen/sndif: Sync up with the canonical definition in Xen Date: Thu, 12 Apr 2018 14:30:52 -0400 Message-ID: <20180412183052.GS350@char.us.oracle.com> References: <20180412172627.18065-1-oleksandr_andrushchenko@epam.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from aserp2120.oracle.com (aserp2120.oracle.com [141.146.126.78]) by alsa0.perex.cz (Postfix) with ESMTP id 978BC2671E9 for ; Thu, 12 Apr 2018 20:30:59 +0200 (CEST) Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Boris Ostrovsky Cc: jgross@suse.com, alsa-devel@alsa-project.org, Oleksandr Andrushchenko , iurii.konovalenko@globallogic.com, linux-kernel@vger.kernel.org, tiwai@suse.de, Oleksandr Grytsov , xen-devel@lists.xenproject.org List-Id: alsa-devel@alsa-project.org On Thu, Apr 12, 2018 at 01:46:33PM -0400, Boris Ostrovsky wrote: > On 04/12/2018 01:26 PM, Oleksandr Andrushchenko wrote: > > This is the sync up with the canonical definition of the sound > > protocol in Xen: > > > > 1. Protocol version was referenced in the protocol description, > > but missed its definition. Fixed by adding a constant > > for current protocol version. > > > > 2. Some of the request descriptions have "reserved" fields > > missed: fixed by adding corresponding entries. > > > > 3. Extend the size of the requests and responses to 64 octets. > > Bump protocol version to 2. > > > > 4. Add explicit back and front synchronization > > In order to provide explicit synchronization between backend and > > frontend the following changes are introduced in the protocol: > > - add new ring buffer for sending asynchronous events from > > backend to frontend to report number of bytes played by the > > frontend (XENSND_EVT_CUR_POS) > > - introduce trigger events for playback control: start/stop/pause/resume > > - add "req-" prefix to event-channel and ring-ref to unify naming > > of the Xen event channels for requests and events > > > > 5. Add explicit back and front parameter negotiation > > In order to provide explicit stream parameter negotiation between > > backend and frontend the following changes are introduced in the protocol: > > add XENSND_OP_HW_PARAM_QUERY request to read/update > > configuration space for the parameters given: request passes > > desired parameter's intervals/masks and the response to this request > > returns allowed min/max intervals/masks to be used. > > > > Signed-off-by: Oleksandr Andrushchenko > > Signed-off-by: Oleksandr Grytsov > > Cc: Konrad Rzeszutek Wilk > > Cc: Takashi Iwai > > --- > > Reviewed-by: Boris Ostrovsky > Reviewed-by: Konrad Rzeszutek Wilk Thank you!