Re: [PATCH RESEND 00/11] ALSA: vsnd: Add Xen para-virtualized frontend driver

Oleksandr Andrushchenko
2017-08-07T12:18:37Z

On 08/07/2017 03:09 PM, Takashi Sakamoto wrote:
> Hi,
>
> On Aug 7 2017 20:50, Oleksandr Andrushchenko wrote:
>> From: Oleksandr Andrushchenko <oleksandr_andrushchenko@epam.com>
>>
>> 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.
>>
>> Thank you,
>> Oleksandr
>>
>> Resending because of rebase onto [2]
>>
>> [1] https://xenproject.org/
>> [2]
>> https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git/log/?h=for-next
>>
>> Oleksandr Andrushchenko (11):
>> ALSA: vsnd: Implement driver's probe/remove
>> ALSA: vsnd: Implement Xen bus state handling
>> ALSA: vsnd: Read sound driver configuration from Xen store
>> ALSA: vsnd: Implement Xen event channel handling
>> ALSA: vsnd: Implement handling of shared buffers
>> ALSA: vsnd: Introduce ALSA virtual sound driver
>> ALSA: vsnd: Initialize virtul sound card
>> ALSA: vsnd: Add timer for period interrupt emulation
>> ALSA: vsnd: Implement ALSA PCM operations
>> ALSA: vsnd: Implement communication with backend
>> ALSA: vsnd: Introduce Kconfig option to enable Xen PV sound
>>
>> sound/drivers/Kconfig | 12 +
>> sound/drivers/Makefile | 2 +
>> sound/drivers/xen-front.c | 2029
>> +++++++++++++++++++++++++++++++++++++++++++++
>> 3 files changed, 2043 insertions(+)
>
> This patchset is not proper yet.
>
really sorry for the mess :(
> Your first patch includes code difference under line 26, thus it
> expects existence of a file 'sound/drivers/xen-front.c' in tree.
> However, the file is not introduced in any upstream yet; at least,
> Linus's tree and Iwai-san's tree.
I missed the very first patch in the series, this is why, my bad
>
> You still work on your local branch, not for us.
>
sorry
>
> Regards
>
> Takashi Sakamoto