alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
From: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
To: Liam Girdwood <liam.r.girdwood@linux.intel.com>
Cc: Takashi Iwai <tiwai@suse.de>,
	alsa-devel@alsa-project.org, Mark Brown <broonie@kernel.org>,
	Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>,
	sound-open-firmware@alsa-project.org
Subject: Re: [Sound-open-firmware] [PATCH v2 00/12] [RESEND] ASoC: SOF DSP virtualisation
Date: Fri, 3 Apr 2020 20:09:19 +0200	[thread overview]
Message-ID: <20200403180918.GA25646@ubuntu> (raw)
In-Reply-To: <29be8be8d9b8700588da693d93b9f10adbb10016.camel@linux.intel.com>

Hi Liam,

Thanks for the clarifications.

On Fri, Apr 03, 2020 at 05:10:20PM +0100, Liam Girdwood wrote:
> On Fri, 2020-04-03 at 13:04 +0200, Guennadi Liakhovetski wrote:
> > Hi Mark,
> > 
> > On Fri, Apr 03, 2020 at 10:28:42AM +0100, Mark Brown wrote:
> > > On Fri, Apr 03, 2020 at 11:13:54AM +0200, Guennadi Liakhovetski
> > > wrote:
> > > > This patch series extends the SOF driver to add support for DSP
> > > > virtualisation to ASoC. It is built on top of VirtIO, contains a
> > > > guest driver and a vhost in-kernel guest driver. This version
> > > > supports a single playback and a single capture interface on the
> > > > guest. The specific guest audio topology is supplied by the host
> > > 
> > > I've asked a couple of times for documentation of the protocol here
> > > but
> > > don't think I've seen anything yet?
> 
> Sorry Mark, the whole series should not have been sent since we are
> still pending on some OASIS standards being ratified. Guennadi, please
> just send the patches that add the protocol independent dependencies to
> ASoC and SOF driver only for review atm (patches 1,2,3,5,6 & 7).
> 
> The full series is blocking on
> 
> 1) the virto-snd patches being merged. This will then allow the code
> (when modified) to run HDA like audio on SOF DSPs.
> 
> 2) virtio DMA buffer sharing being concluded for "zero copy" usage.

I don't think we're blocked by this. If I understand correctly it is our 
intention to first upstream the present copying solution and then 
implement zero-copy as a next step.

> 3) rpmsg integration. The SOF IPC will use rpmsg virtio transport
> between host and guests.

We started discussing this on github, unfortunately this didn't come to a 
conclusion. From what I've read in the kernel, RPMSG is currencly used 
there in scenarios, that are very different from ours. Typically you have 
a Linux host, that uses RPMSG to communicate with an "embedded" 
counterpart, where that communication includes boooting ELF firmware on 
that counterpart, and then using RPMSG on top of Virtual Queues to 
communicate with it. Our case is quite different. I'm not saying, that it 
is impossible to use the Linux RPMSG subsystem for use-cases like ours, 
but it seems to me, that this would require a significant effort on the 
Linux RPMSG core implementation, and we would be the first use-case for 
this.

The rest has to be discussed.

Thanks
Guennadi

> This series does not mandate a DSP IPC standard, since this may differ
> between DSP vendors, but it will use OASIS standards for virtio-snd,
> DMA buffers and rpmsg.
> 
> I would say that parts of the "SOF protocol" may be useful for other
> vendors in the DAPM/topology areas where users want to connect guest
> topologies to host topologies (connected internally with DAPM).
> 
> Thanks
> 
> Liam
> 
> 
> 

  parent reply	other threads:[~2020-04-03 18:10 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-03  9:13 [PATCH v2 00/12] [RESEND] ASoC: SOF DSP virtualisation Guennadi Liakhovetski
2020-04-03  9:13 ` [PATCH v2 01/12] [RESEND] ASoC: add function parameters to enable forced path pruning Guennadi Liakhovetski
2020-04-03  9:13 ` [PATCH v2 02/12] [RESEND] ASoC: SOF: extract firmware-related operation into a function Guennadi Liakhovetski
2020-04-03  9:13 ` [PATCH v2 03/12] [RESEND] ASoC: SOF: VirtIO: make a function global Guennadi Liakhovetski
2020-04-03  9:13 ` [PATCH v2 04/12] [RESEND] vhost: convert VHOST_VSOCK_SET_RUNNING to a generic ioctl Guennadi Liakhovetski
2020-04-03  9:13 ` [PATCH v2 05/12] [RESEND] ASoC: SOF: support IPC with immediate response Guennadi Liakhovetski
2020-04-03  9:14 ` [PATCH v2 06/12] [RESEND] ASoC: SOF: add a power status IPC Guennadi Liakhovetski
2020-04-03  9:14 ` [PATCH v2 07/12] [RESEND] ASoC: SOF: add two helper lookup functions Guennadi Liakhovetski
2020-04-03  9:14 ` [PATCH v2 08/12] [RESEND] ASoC: SOF: add a VirtIO DSP driver Guennadi Liakhovetski
2020-04-03  9:14 ` [PATCH v2 09/12] [RESEND] ASoC: SOF: add a vhost driver: sound part Guennadi Liakhovetski
2020-04-03  9:14 ` [PATCH v2 10/12] [RESEND] vhost: add an SOF DSP driver Guennadi Liakhovetski
2020-04-03  9:14 ` [PATCH v2 11/12] [RESEND] ASoC: SOF: VirtIO: free guest pipelines upon termination Guennadi Liakhovetski
2020-04-03  9:14 ` [PATCH v2 12/12] [RESEND] ASoC: SOF: VirtIO: enable simultaneous playback and capture Guennadi Liakhovetski
2020-04-03  9:28 ` [PATCH v2 00/12] [RESEND] ASoC: SOF DSP virtualisation Mark Brown
2020-04-03 11:04   ` Guennadi Liakhovetski
2020-04-03 11:26     ` Mark Brown
2020-04-03 16:10     ` [Sound-open-firmware] " Liam Girdwood
2020-04-03 16:21       ` Mark Brown
2020-04-03 18:09       ` Guennadi Liakhovetski [this message]
2020-04-07 11:24         ` Guennadi Liakhovetski
2020-04-07 11:30           ` Mark Brown
2020-04-07 12:44             ` Guennadi Liakhovetski

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=20200403180918.GA25646@ubuntu \
    --to=guennadi.liakhovetski@linux.intel.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@kernel.org \
    --cc=liam.r.girdwood@linux.intel.com \
    --cc=pierre-louis.bossart@linux.intel.com \
    --cc=sound-open-firmware@alsa-project.org \
    --cc=tiwai@suse.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).