From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932387AbeENQJF (ORCPT ); Mon, 14 May 2018 12:09:05 -0400 Received: from mail-lf0-f65.google.com ([209.85.215.65]:43873 "EHLO mail-lf0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753582AbeENQJA (ORCPT ); Mon, 14 May 2018 12:09:00 -0400 X-Google-Smtp-Source: AB8JxZqbbWKrqq7g49WWC2IJs9jYCUgfFn6WKh2D/NdBWhfjEJUgTna4UozkWT4nhAsg7ApJukV7kw== Subject: Re: [alsa-devel] [PATCH v3 0/6] ALSA: xen-front: Add Xen para-virtualized frontend driver To: Takashi Iwai Cc: "Oleksandr_Andrushchenko@epam.com" , xen-devel@lists.xenproject.org, linux-kernel@vger.kernel.org, alsa-devel@alsa-project.org, jgross@suse.com, boris.ostrovsky@oracle.com, konrad.wilk@oracle.com, perex@perex.cz References: <20180514062742.25879-1-andr2000@gmail.com> From: Oleksandr Andrushchenko Message-ID: <096bee8c-69ad-b1ca-5eb6-370203d73c16@gmail.com> Date: Mon, 14 May 2018 19:08:56 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 05/14/2018 06:52 PM, Takashi Iwai wrote: > On Mon, 14 May 2018 08:27:36 +0200, > Oleksandr Andrushchenko wrote: >> From: Oleksandr Andrushchenko >> >> Please note: this patch series depends on [3]. >> >> This patch series adds support for Xen [1] para-virtualized >> sound frontend driver. It implements the protocol from >> include/xen/interface/io/sndif.h with the following limitations: >> - mute/unmute is not supported >> - get/set volume is not supported >> Volume control is not supported for the reason that most of the >> use-cases (at the moment) are based on scenarious where >> unprivileged OS (e.g. Android, AGL etc) use software mixers. >> >> Both capture and playback are supported. >> >> Corresponding backend, implemented as a user-space application, can be >> found at [2]. > Through a quick glance, all look OK for merge (assuming that they > really work on your systems :) Well, we have tested this at EPAM on Android/Linux, so this is it for now > > Of course, it'd be great if anyone else can review, test and give > reviewed-by or tested-by tags. I hope Juergen and Boris can review, at least Juergen had some comments that I have addressed in v3 > So I'll keep this floating for a > couple of days and merge to for-next branch after that. Oh, this is great news! Thank you > > thanks, > > Takashi