| Message ID | 20240624135820.516893-1-perex@perex.cz (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v4] ALSA: compress_offload: introduce passthrough operation mode | expand |
Hi Jaroslav, On 24-06-24, 15:58, Jaroslav Kysela wrote: > There is a requirement to expose the audio hardware that accelerates various > tasks for user space such as sample rate converters, compressed > stream decoders, etc. Can you please tell me more about this requirement. The initial view of compressed API was data only and use alsa kcontrols to handle the DSP functions? I would like to understand why we need a new API. > > This is description for the API extension for the compress ALSA API which > is able to handle "tasks" that are not bound to real-time operations > and allows for the serialization of operations. > > For details, refer to "compress-passthrough.rst" document. > > Cc: Mark Brown <broonie@kernel.org> > Cc: Shengjiu Wang <shengjiu.wang@gmail.com> > Cc: Nicolas Dufresne <nicolas@ndufresne.ca> > Cc: Amadeusz Sławiński <amadeuszx.slawinski@linux.intel.com> > Cc: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> > Cc: Vinod Koul <vkoul@kernel.org> > Signed-off-by: Jaroslav Kysela <perex@perex.cz> > > --- > v3..v4: > - resolved Takashi's requests: > - make CONFIG_SND_COMPRESS_PASSTHROUGH as bool, remove wrong DMA_BUF selection > - use EXPORT_SYMBOL_GPL for snd_compr_task_finished function > - fix snd_compr_find_task indentation > - add more CONFIG_SND_COMPRESS_PASSTHROUGH #if blocks > > v2..v3: > - fix missing runtime->tasks initialization (thanks Shengjiu Wang) > - fix missing seqno initialization in task_new (thanks Shengjiu Wang) > - fix reference counting for allocated dma buffers (thanks Shengjiu Wang) > - use origin_seqno to reuse the already allocated buffers for new task > > v1..v2: > - fix some documentation typos (thanks Amadeusz Sławiński) > - fix memdup_user() error handling (thanks Takashi) > - use one state variable instead multiple (thanks Takashi) > - handle task limit (set to 64 - mentioned in documentation, NIY) > - fix file release (free all tasks) > --- > .../sound/designs/compress-passthrough.rst | 125 +++++++ > include/sound/compress_driver.h | 34 ++ > include/uapi/sound/compress_offload.h | 51 ++- > sound/core/Kconfig | 3 + > sound/core/compress_offload.c | 346 +++++++++++++++++- What about the user of this API, i would like to see that as well > 5 files changed, 550 insertions(+), 9 deletions(-) > create mode 100644 Documentation/sound/designs/compress-passthrough.rst > > diff --git a/Documentation/sound/designs/compress-passthrough.rst b/Documentation/sound/designs/compress-passthrough.rst > new file mode 100644 > index 000000000000..975462500c33 > --- /dev/null > +++ b/Documentation/sound/designs/compress-passthrough.rst > @@ -0,0 +1,125 @@ > +================================= > +ALSA Co-processor Passthrough API > +================================= > + > +Jaroslav Kysela <perex@perex.cz> > + > + > +Overview > +======== > + > +There is a requirement to expose the audio hardware that accelerates various > +tasks for user space such as sample rate converters, compressed > +stream decoders, etc. > + > +This is description for the API extension for the compress ALSA API which > +is able to handle "tasks" that are not bound to real-time operations > +and allows for the serialization of operations. > + > +Requirements > +============ > + > +The main requirements are: > + > +- serialization of multiple tasks for user space to allow multiple > + operations without user space intervention > + > +- separate buffers (input + output) for each operation > + > +- expose buffers using mmap to user space > + > +- signal user space when the task is finished (standard poll mechanism) > + > +Design > +====== > + > +A new direction SND_COMPRESS_PASSTHROUGH is introduced to identify > +the passthrough API. Is passthrough really a new good new name, this suggests data being passed thru but that is not the case... Wouldn't a control device be better or something else? > + > +The API extension shares device enumeration and parameters handling from > +the main compressed API. All other realtime streaming ioctls are deactivated > +and a new set of task related ioctls are introduced. The standard > +read/write/mmap I/O operations are not supported in the passthrough device. > + > +Device ("stream") state handling is reduced to OPEN/SETUP. All other > +states are not available for the passthrough mode. > + > +Data I/O mechanism is using standard dma-buf interface with all advantages > +like mmap, standard I/O, buffer sharing etc. One buffer is used for the > +input data and second (separate) buffer is used for the output data. Each task > +have separate I/O buffers. > + > +For the buffering parameters, the fragments means a limit of allocated tasks > +for given device. The fragment_size limits the input buffer size for the given > +device. The output buffer size is determined by the driver (may be different > +from the input buffer size). > + > +State Machine > +============= > + > +The passthrough audio stream state machine is described below : > + > + +----------+ > + | | > + | OPEN | > + | | > + +----------+ > + | > + | > + | compr_set_params() > + | > + v > + all passthrough task ops +----------+ > + +------------------------------------| | > + | | SETUP | > + | | > + | +----------+ > + | | > + +------------------------------------------+ > + > + > +Passthrough operations (ioctls) > +=============================== > + > +CREATE > +------ > +Creates a set of input/output buffers. The input buffer size is > +fragment_size. Allocates unique seqno. > + > +The hardware drivers allocate internal 'struct dma_buf' for both input and > +output buffers (using 'dma_buf_export()' function). The anonymous > +file descriptors for those buffers are passed to user space. > + > +FREE > +---- > +Free a set of input/output buffers. If a task is active, the stop > +operation is executed before. If seqno is zero, operation is executed for all > +tasks. > + > +START > +----- > +Starts (queues) a task. There are two cases of the task start - right after > +the task is created. In this case, origin_seqno must be zero. > +The second case is for reusing of already finished task. The origin_seqno > +must identify the task to be reused. In both cases, a new seqno value > +is allocated and returned to user space. > + > +The prerequisite is that application filled input dma buffer with > +new source data and set input_size to pass the real data size to the driver. > + > +The order of data processing is preserved (first started job must be > +finished at first). > + > +STOP > +---- > +Stop (dequeues) a task. If seqno is zero, operation is executed for all > +tasks. > + > +STATUS > +------ > +Obtain the task status (active, finished). Also, the driver will set > +the real output data size (valid area in the output buffer). > + > +Credits > +======= > +- ... > diff --git a/include/sound/compress_driver.h b/include/sound/compress_driver.h > index bcf872c17dd3..433ad897f054 100644 > --- a/include/sound/compress_driver.h > +++ b/include/sound/compress_driver.h > @@ -19,6 +19,22 @@ > > struct snd_compr_ops; > > +/** > + * struct snd_compr_task_runtime: task runtime description Can we add the description here for these.. > + * > + */ > +struct snd_compr_task_runtime { > + struct list_head list; > + struct dma_buf *input; > + struct dma_buf *output; > + u64 seqno; > + u64 input_size; > + u64 output_size; > + u8 state; > + void *private_value; > +}; > + > + > /** > * struct snd_compr_runtime: runtime stream description > * @state: stream state here as well, am sure it gives a warning now for missing description
On 6/24/2024 3:58 PM, Jaroslav Kysela wrote: > There is a requirement to expose the audio hardware that accelerates various > tasks for user space such as sample rate converters, compressed > stream decoders, etc. > > This is description for the API extension for the compress ALSA API which > is able to handle "tasks" that are not bound to real-time operations > and allows for the serialization of operations. > > For details, refer to "compress-passthrough.rst" document. > > Cc: Mark Brown <broonie@kernel.org> > Cc: Shengjiu Wang <shengjiu.wang@gmail.com> > Cc: Nicolas Dufresne <nicolas@ndufresne.ca> > Cc: Amadeusz Sławiński <amadeuszx.slawinski@linux.intel.com> > Cc: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> > Cc: Vinod Koul <vkoul@kernel.org> > Signed-off-by: Jaroslav Kysela <perex@perex.cz> > > --- > diff --git a/include/uapi/sound/compress_offload.h b/include/uapi/sound/compress_offload.h > index d185957f3fe0..5fed1979522b 100644 > --- a/include/uapi/sound/compress_offload.h > +++ b/include/uapi/sound/compress_offload.h > @@ -1,4 +1,4 @@ > -/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */ > + /* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */ Accidental indentation change?
On 24. 06. 24 16:47, Vinod Koul wrote: > Hi Jaroslav, Hi Vinod, > On 24-06-24, 15:58, Jaroslav Kysela wrote: >> There is a requirement to expose the audio hardware that accelerates various >> tasks for user space such as sample rate converters, compressed >> stream decoders, etc. > > Can you please tell me more about this requirement. The initial view of > compressed API was data only and use alsa kcontrols to handle the DSP > functions? I would like to understand why we need a new API. There are very long threads for v4l audio support - last v15 thread: https://lore.kernel.org/linux-media/1710834674-3285-1-git-send-email-shengjiu.wang@nxp.com/ So the goal is to create something more efficient for the offload work, when the data (decoded/converted) should be returned back to the user space. > What about the user of this API, i would like to see that as well Any audio streaming framework like gstreamer or ffmpeg who can accelerate stream conversions in hardware for capable devices. >> +A new direction SND_COMPRESS_PASSTHROUGH is introduced to identify >> +the passthrough API. > > Is passthrough really a new good new name, this suggests data being > passed thru but that is not the case... It's something like "PASS data THROUGH kernel/driver". So it makes sense. My alternate name may be ACCEL (like acceleration). > Wouldn't a control device be better or something else? Compress API handles already enumeration and capabilities, so they can be extended for this purpose. Only I/O is different. Basically, we can reuse half of API from the current compress-offload API. It's really close thing (and it's also offload work). >> +/** >> + * struct snd_compr_task_runtime: task runtime description > > Can we add the description here for these.. Yes, I'll do. > >> + * >> + */ >> +struct snd_compr_task_runtime { ... >> /** >> * struct snd_compr_runtime: runtime stream description >> * @state: stream state > > here as well, am sure it gives a warning now for missing description I'll do. Thanks, Jaroslav
On 24. 06. 24 17:01, Amadeusz Sławiński wrote: >> --- a/include/uapi/sound/compress_offload.h >> +++ b/include/uapi/sound/compress_offload.h >> @@ -1,4 +1,4 @@ >> -/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */ >> + /* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */ > > Accidental indentation change? Yes, I'll fix that. Thanks, Jaroslav
Hi Jaroslav, On 24-06-24, 17:31, Jaroslav Kysela wrote: > On 24. 06. 24 16:47, Vinod Koul wrote: > > On 24-06-24, 15:58, Jaroslav Kysela wrote: > > > There is a requirement to expose the audio hardware that accelerates various > > > tasks for user space such as sample rate converters, compressed > > > stream decoders, etc. > > > > Can you please tell me more about this requirement. The initial view of > > compressed API was data only and use alsa kcontrols to handle the DSP > > functions? I would like to understand why we need a new API. > > There are very long threads for v4l audio support - last v15 thread: > > https://lore.kernel.org/linux-media/1710834674-3285-1-git-send-email-shengjiu.wang@nxp.com/ Very long indeed but very interesting. I think going compressed audio way seems reasonable choice to me > > So the goal is to create something more efficient for the offload work, when > the data (decoded/converted) should be returned back to the user space. Not rendered? so we are using it as an accelerator...? > > > What about the user of this API, i would like to see that as well > > Any audio streaming framework like gstreamer or ffmpeg who can accelerate > stream conversions in hardware for capable devices. I meant to see driver users along with this patch :-) That also reminds me to ask about usermode support for this, are you planning to support it in tinycompress? > > > > +A new direction SND_COMPRESS_PASSTHROUGH is introduced to identify > > > +the passthrough API. > > > > Is passthrough really a new good new name, this suggests data being > > passed thru but that is not the case... > > It's something like "PASS data THROUGH kernel/driver". So it makes sense. My > alternate name may be ACCEL (like acceleration). I like that.. Also do we change the device name if the passthru is enabled? it should not be called a playback or capture compressed device
On Mon, Jun 24, 2024 at 9:58 PM Jaroslav Kysela <perex@perex.cz> wrote: > > There is a requirement to expose the audio hardware that accelerates various > tasks for user space such as sample rate converters, compressed > stream decoders, etc. > > This is description for the API extension for the compress ALSA API which > is able to handle "tasks" that are not bound to real-time operations > and allows for the serialization of operations. > > For details, refer to "compress-passthrough.rst" document. > > Cc: Mark Brown <broonie@kernel.org> > Cc: Shengjiu Wang <shengjiu.wang@gmail.com> > Cc: Nicolas Dufresne <nicolas@ndufresne.ca> > Cc: Amadeusz Sławiński <amadeuszx.slawinski@linux.intel.com> > Cc: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> > Cc: Vinod Koul <vkoul@kernel.org> > Signed-off-by: Jaroslav Kysela <perex@perex.cz> Gentle ping. Looking forward to the new version. I have tested the ASRC function based on this API, basically can work. Best regards Shengjiu Wang > > --- > v3..v4: > - resolved Takashi's requests: > - make CONFIG_SND_COMPRESS_PASSTHROUGH as bool, remove wrong DMA_BUF selection > - use EXPORT_SYMBOL_GPL for snd_compr_task_finished function > - fix snd_compr_find_task indentation > - add more CONFIG_SND_COMPRESS_PASSTHROUGH #if blocks > > v2..v3: > - fix missing runtime->tasks initialization (thanks Shengjiu Wang) > - fix missing seqno initialization in task_new (thanks Shengjiu Wang) > - fix reference counting for allocated dma buffers (thanks Shengjiu Wang) > - use origin_seqno to reuse the already allocated buffers for new task > > v1..v2: > - fix some documentation typos (thanks Amadeusz Sławiński) > - fix memdup_user() error handling (thanks Takashi) > - use one state variable instead multiple (thanks Takashi) > - handle task limit (set to 64 - mentioned in documentation, NIY) > - fix file release (free all tasks) > --- > .../sound/designs/compress-passthrough.rst | 125 +++++++ > include/sound/compress_driver.h | 34 ++ > include/uapi/sound/compress_offload.h | 51 ++- > sound/core/Kconfig | 3 + > sound/core/compress_offload.c | 346 +++++++++++++++++- > 5 files changed, 550 insertions(+), 9 deletions(-) > create mode 100644 Documentation/sound/designs/compress-passthrough.rst > > diff --git a/Documentation/sound/designs/compress-passthrough.rst b/Documentation/sound/designs/compress-passthrough.rst > new file mode 100644 > index 000000000000..975462500c33 > --- /dev/null > +++ b/Documentation/sound/designs/compress-passthrough.rst > @@ -0,0 +1,125 @@ > +================================= > +ALSA Co-processor Passthrough API > +================================= > + > +Jaroslav Kysela <perex@perex.cz> > + > + > +Overview > +======== > + > +There is a requirement to expose the audio hardware that accelerates various > +tasks for user space such as sample rate converters, compressed > +stream decoders, etc. > + > +This is description for the API extension for the compress ALSA API which > +is able to handle "tasks" that are not bound to real-time operations > +and allows for the serialization of operations. > + > +Requirements > +============ > + > +The main requirements are: > + > +- serialization of multiple tasks for user space to allow multiple > + operations without user space intervention > + > +- separate buffers (input + output) for each operation > + > +- expose buffers using mmap to user space > + > +- signal user space when the task is finished (standard poll mechanism) > + > +Design > +====== > + > +A new direction SND_COMPRESS_PASSTHROUGH is introduced to identify > +the passthrough API. > + > +The API extension shares device enumeration and parameters handling from > +the main compressed API. All other realtime streaming ioctls are deactivated > +and a new set of task related ioctls are introduced. The standard > +read/write/mmap I/O operations are not supported in the passthrough device. > + > +Device ("stream") state handling is reduced to OPEN/SETUP. All other > +states are not available for the passthrough mode. > + > +Data I/O mechanism is using standard dma-buf interface with all advantages > +like mmap, standard I/O, buffer sharing etc. One buffer is used for the > +input data and second (separate) buffer is used for the output data. Each task > +have separate I/O buffers. > + > +For the buffering parameters, the fragments means a limit of allocated tasks > +for given device. The fragment_size limits the input buffer size for the given > +device. The output buffer size is determined by the driver (may be different > +from the input buffer size). > + > +State Machine > +============= > + > +The passthrough audio stream state machine is described below : > + > + +----------+ > + | | > + | OPEN | > + | | > + +----------+ > + | > + | > + | compr_set_params() > + | > + v > + all passthrough task ops +----------+ > + +------------------------------------| | > + | | SETUP | > + | | > + | +----------+ > + | | > + +------------------------------------------+ > + > + > +Passthrough operations (ioctls) > +=============================== > + > +CREATE > +------ > +Creates a set of input/output buffers. The input buffer size is > +fragment_size. Allocates unique seqno. > + > +The hardware drivers allocate internal 'struct dma_buf' for both input and > +output buffers (using 'dma_buf_export()' function). The anonymous > +file descriptors for those buffers are passed to user space. > + > +FREE > +---- > +Free a set of input/output buffers. If a task is active, the stop > +operation is executed before. If seqno is zero, operation is executed for all > +tasks. > + > +START > +----- > +Starts (queues) a task. There are two cases of the task start - right after > +the task is created. In this case, origin_seqno must be zero. > +The second case is for reusing of already finished task. The origin_seqno > +must identify the task to be reused. In both cases, a new seqno value > +is allocated and returned to user space. > + > +The prerequisite is that application filled input dma buffer with > +new source data and set input_size to pass the real data size to the driver. > + > +The order of data processing is preserved (first started job must be > +finished at first). > + > +STOP > +---- > +Stop (dequeues) a task. If seqno is zero, operation is executed for all > +tasks. > + > +STATUS > +------ > +Obtain the task status (active, finished). Also, the driver will set > +the real output data size (valid area in the output buffer). > + > +Credits > +======= > +- ... > diff --git a/include/sound/compress_driver.h b/include/sound/compress_driver.h > index bcf872c17dd3..433ad897f054 100644 > --- a/include/sound/compress_driver.h > +++ b/include/sound/compress_driver.h > @@ -19,6 +19,22 @@ > > struct snd_compr_ops; > > +/** > + * struct snd_compr_task_runtime: task runtime description > + * > + */ > +struct snd_compr_task_runtime { > + struct list_head list; > + struct dma_buf *input; > + struct dma_buf *output; > + u64 seqno; > + u64 input_size; > + u64 output_size; > + u8 state; > + void *private_value; > +}; > + > + > /** > * struct snd_compr_runtime: runtime stream description > * @state: stream state > @@ -54,6 +70,13 @@ struct snd_compr_runtime { > dma_addr_t dma_addr; > size_t dma_bytes; > struct snd_dma_buffer *dma_buffer_p; > + > +#if IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) > + u32 active_tasks; > + u32 total_tasks; > + u64 task_seqno; > + struct list_head tasks; > +#endif > }; > > /** > @@ -132,6 +155,12 @@ struct snd_compr_ops { > struct snd_compr_caps *caps); > int (*get_codec_caps) (struct snd_compr_stream *stream, > struct snd_compr_codec_caps *codec); > +#if IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) > + int (*task_create) (struct snd_compr_stream *stream, struct snd_compr_task_runtime *task); > + int (*task_start) (struct snd_compr_stream *stream, struct snd_compr_task_runtime *task); > + int (*task_stop) (struct snd_compr_stream *stream, struct snd_compr_task_runtime *task); > + int (*task_free) (struct snd_compr_stream *stream, struct snd_compr_task_runtime *task); > +#endif > }; > > /** > @@ -242,4 +271,9 @@ int snd_compr_free_pages(struct snd_compr_stream *stream); > int snd_compr_stop_error(struct snd_compr_stream *stream, > snd_pcm_state_t state); > > +#if IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) > +void snd_compr_task_finished(struct snd_compr_stream *stream, > + struct snd_compr_task_runtime *task); > +#endif > + > #endif > diff --git a/include/uapi/sound/compress_offload.h b/include/uapi/sound/compress_offload.h > index d185957f3fe0..5fed1979522b 100644 > --- a/include/uapi/sound/compress_offload.h > +++ b/include/uapi/sound/compress_offload.h > @@ -1,4 +1,4 @@ > -/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */ > + /* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */ > /* > * compress_offload.h - compress offload header definations > * > @@ -14,7 +14,7 @@ > #include <sound/compress_params.h> > > > -#define SNDRV_COMPRESS_VERSION SNDRV_PROTOCOL_VERSION(0, 2, 0) > +#define SNDRV_COMPRESS_VERSION SNDRV_PROTOCOL_VERSION(0, 3, 0) > /** > * struct snd_compressed_buffer - compressed buffer > * @fragment_size: size of buffer fragment in bytes > @@ -68,7 +68,8 @@ struct snd_compr_avail { > > enum snd_compr_direction { > SND_COMPRESS_PLAYBACK = 0, > - SND_COMPRESS_CAPTURE > + SND_COMPRESS_CAPTURE, > + SND_COMPRESS_PASSTHROUGH > }; > > /** > @@ -127,6 +128,42 @@ struct snd_compr_metadata { > __u32 value[8]; > } __attribute__((packed, aligned(4))); > > +/** > + * struct snd_compr_task - task primitive for non-realtime operation > + * @seqno: sequence number (task identifier) > + * @origin_seqno: previous sequence number (task identifier) - for reuse > + * @input_fd: data input file descriptor (dma-buf) > + * @output_fd: data output file descriptor (dma-buf) > + * @input_size: filled data in bytes (from caller, must not exceed fragment size) > + */ > +struct snd_compr_task { > + __u64 seqno; > + __u64 origin_seqno; > + int input_fd; > + int output_fd; > + __u64 input_size; > + __u8 reserved[16]; > +} __attribute__((packed, aligned(4))); > + > +enum snd_compr_state { > + SND_COMPRESS_TASK_STATE_IDLE = 0, > + SND_COMPRESS_TASK_STATE_ACTIVE, > + SND_COMPRESS_TASK_STATE_FINISHED > +}; > + > +/** > + * struct snd_compr_task_status - task status > + * @seqno: sequence number (task identifier) > + * @output_size: filled data in bytes (from driver) > + * @state: actual task state (SND_COMPRESS_TASK_STATE_*) > + */ > +struct snd_compr_task_status { > + __u64 seqno; > + __u64 output_size; > + __u8 state; > + __u8 reserved[15]; > +} __attribute__((packed, aligned(4))); > + > /* > * compress path ioctl definitions > * SNDRV_COMPRESS_GET_CAPS: Query capability of DSP > @@ -164,6 +201,14 @@ struct snd_compr_metadata { > #define SNDRV_COMPRESS_DRAIN _IO('C', 0x34) > #define SNDRV_COMPRESS_NEXT_TRACK _IO('C', 0x35) > #define SNDRV_COMPRESS_PARTIAL_DRAIN _IO('C', 0x36) > + > + > +#define SNDRV_COMPRESS_TASK_CREATE _IOWR('C', 0x60, struct snd_compr_task) > +#define SNDRV_COMPRESS_TASK_FREE _IOW('C', 0x61, __u64) > +#define SNDRV_COMPRESS_TASK_START _IOWR('C', 0x62, struct snd_compr_task) > +#define SNDRV_COMPRESS_TASK_STOP _IOW('C', 0x63, __u64) > +#define SNDRV_COMPRESS_TASK_STATUS _IOWR('C', 0x68, struct snd_compr_task_status) > + > /* > * TODO > * 1. add mmap support > diff --git a/sound/core/Kconfig b/sound/core/Kconfig > index 8077f481d84f..309cfae3b8b2 100644 > --- a/sound/core/Kconfig > +++ b/sound/core/Kconfig > @@ -59,6 +59,9 @@ config SND_CORE_TEST > config SND_COMPRESS_OFFLOAD > tristate > > +config SND_COMPRESS_PASSTHROUGH > + bool > + > config SND_JACK > bool > > diff --git a/sound/core/compress_offload.c b/sound/core/compress_offload.c > index f0008fa2d839..7b5e2e9e737e 100644 > --- a/sound/core/compress_offload.c > +++ b/sound/core/compress_offload.c > @@ -24,6 +24,7 @@ > #include <linux/types.h> > #include <linux/uio.h> > #include <linux/uaccess.h> > +#include <linux/dma-buf.h> > #include <linux/module.h> > #include <linux/compat.h> > #include <sound/core.h> > @@ -54,6 +55,12 @@ struct snd_compr_file { > > static void error_delayed_work(struct work_struct *work); > > +#if IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) > +static void snd_compr_task_free_all(struct snd_compr_stream *stream); > +#else > +static inline void snd_compr_task_free_all(struct snd_compr_stream *stream) { } > +#endif > + > /* > * a note on stream states used: > * we use following states in the compressed core > @@ -85,6 +92,8 @@ static int snd_compr_open(struct inode *inode, struct file *f) > dirn = SND_COMPRESS_PLAYBACK; > else if ((f->f_flags & O_ACCMODE) == O_RDONLY) > dirn = SND_COMPRESS_CAPTURE; > + else if ((f->f_flags & O_ACCMODE) == O_RDWR) > + dirn = SND_COMPRESS_PASSTHROUGH; > else > return -EINVAL; > > @@ -112,6 +121,9 @@ static int snd_compr_open(struct inode *inode, struct file *f) > } > > INIT_DELAYED_WORK(&data->stream.error_work, error_delayed_work); > +#if IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) > + INIT_LIST_HEAD(&runtime->tasks); > +#endif > > data->stream.ops = compr->ops; > data->stream.direction = dirn; > @@ -154,6 +166,8 @@ static int snd_compr_free(struct inode *inode, struct file *f) > break; > } > > + snd_compr_task_free_all(&data->stream); > + > data->stream.ops->free(&data->stream); > if (!data->stream.runtime->dma_buffer_p) > kfree(data->stream.runtime->buffer); > @@ -226,6 +240,9 @@ snd_compr_ioctl_avail(struct snd_compr_stream *stream, unsigned long arg) > struct snd_compr_avail ioctl_avail; > size_t avail; > > + if (stream->direction == SND_COMPRESS_PASSTHROUGH) > + return -EBADFD; > + > avail = snd_compr_calc_avail(stream, &ioctl_avail); > ioctl_avail.avail = avail; > > @@ -287,6 +304,8 @@ static ssize_t snd_compr_write(struct file *f, const char __user *buf, > return -EFAULT; > > stream = &data->stream; > + if (stream->direction == SND_COMPRESS_PASSTHROUGH) > + return -EBADFD; > guard(mutex)(&stream->device->lock); > /* write is allowed when stream is running or has been steup */ > switch (stream->runtime->state) { > @@ -336,6 +355,8 @@ static ssize_t snd_compr_read(struct file *f, char __user *buf, > return -EFAULT; > > stream = &data->stream; > + if (stream->direction == SND_COMPRESS_PASSTHROUGH) > + return -EBADFD; > guard(mutex)(&stream->device->lock); > > /* read is allowed when stream is running, paused, draining and setup > @@ -385,6 +406,7 @@ static __poll_t snd_compr_poll(struct file *f, poll_table *wait) > { > struct snd_compr_file *data = f->private_data; > struct snd_compr_stream *stream; > + struct snd_compr_runtime *runtime; > size_t avail; > __poll_t retval = 0; > > @@ -392,6 +414,7 @@ static __poll_t snd_compr_poll(struct file *f, poll_table *wait) > return EPOLLERR; > > stream = &data->stream; > + runtime = stream->runtime; > > guard(mutex)(&stream->device->lock); > > @@ -405,6 +428,20 @@ static __poll_t snd_compr_poll(struct file *f, poll_table *wait) > > poll_wait(f, &stream->runtime->sleep, wait); > > +#if IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) > + if (stream->direction == SND_COMPRESS_PASSTHROUGH) { > + struct snd_compr_task_runtime *task; > + if (runtime->fragments > runtime->active_tasks) > + retval |= EPOLLOUT | EPOLLWRNORM; > + task = list_first_entry_or_null(&runtime->tasks, > + struct snd_compr_task_runtime, > + list); > + if (task && task->state == SND_COMPRESS_TASK_STATE_FINISHED) > + retval |= EPOLLIN | EPOLLRDNORM; > + return retval; > + } > +#endif > + > avail = snd_compr_get_avail(stream); > pr_debug("avail is %ld\n", (unsigned long)avail); > /* check if we have at least one fragment to fill */ > @@ -521,6 +558,9 @@ static int snd_compr_allocate_buffer(struct snd_compr_stream *stream, > unsigned int buffer_size; > void *buffer = NULL; > > + if (stream->direction == SND_COMPRESS_PASSTHROUGH) > + goto params; > + > buffer_size = params->buffer.fragment_size * params->buffer.fragments; > if (stream->ops->copy) { > buffer = NULL; > @@ -543,18 +583,30 @@ static int snd_compr_allocate_buffer(struct snd_compr_stream *stream, > if (!buffer) > return -ENOMEM; > } > - stream->runtime->fragment_size = params->buffer.fragment_size; > - stream->runtime->fragments = params->buffer.fragments; > + > stream->runtime->buffer = buffer; > stream->runtime->buffer_size = buffer_size; > +params: > + stream->runtime->fragment_size = params->buffer.fragment_size; > + stream->runtime->fragments = params->buffer.fragments; > return 0; > } > > -static int snd_compress_check_input(struct snd_compr_params *params) > +static int > +snd_compress_check_input(struct snd_compr_stream *stream, struct snd_compr_params *params) > { > + u32 max_fragments; > + > /* first let's check the buffer parameter's */ > - if (params->buffer.fragment_size == 0 || > - params->buffer.fragments > U32_MAX / params->buffer.fragment_size || > + if (params->buffer.fragment_size == 0) > + return -EINVAL; > + > + if (stream->direction == SND_COMPRESS_PASSTHROUGH) > + max_fragments = 64; /* safe value */ > + else > + max_fragments = U32_MAX / params->buffer.fragment_size; > + > + if (params->buffer.fragments > max_fragments || > params->buffer.fragments == 0) > return -EINVAL; > > @@ -583,7 +635,7 @@ snd_compr_set_params(struct snd_compr_stream *stream, unsigned long arg) > if (IS_ERR(params)) > return PTR_ERR(no_free_ptr(params)); > > - retval = snd_compress_check_input(params); > + retval = snd_compress_check_input(stream, params); > if (retval) > return retval; > > @@ -939,6 +991,262 @@ static int snd_compr_partial_drain(struct snd_compr_stream *stream) > return snd_compress_wait_for_drain(stream); > } > > +#if IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) > + > +static struct snd_compr_task_runtime * > +snd_compr_find_task(struct snd_compr_stream *stream, __u64 seqno) > +{ > + struct snd_compr_task_runtime *task; > + > + list_for_each_entry(task, &stream->runtime->tasks, list) { > + if (task->seqno == seqno) > + return task; > + } > + return NULL; > +} > + > +static void snd_compr_task_free(struct snd_compr_task_runtime *task) > +{ > + if (task->output) > + dma_buf_put(task->output); > + if (task->input) > + dma_buf_put(task->input); > + kfree(task); > +} > + > +static u64 snd_compr_seqno_next(struct snd_compr_stream *stream) > +{ > + u64 seqno = ++stream->runtime->task_seqno; > + if (seqno == 0) > + seqno = ++stream->runtime->task_seqno; > + return seqno; > +} > + > +static int snd_compr_task_new(struct snd_compr_stream *stream, struct snd_compr_task *utask) > +{ > + struct snd_compr_task_runtime *task; > + int retval; > + > + if (stream->runtime->total_tasks >= stream->runtime->fragments) > + return -EBUSY; > + if (utask->origin_seqno != 0 || utask->input_size != 0) > + return -EINVAL; > + task = kzalloc(sizeof(*task), GFP_KERNEL); > + if (task == NULL) > + return -ENOMEM; > + task->seqno = utask->seqno = snd_compr_seqno_next(stream); > + task->input_size = utask->input_size; > + retval = stream->ops->task_create(stream, task); > + if (retval < 0) > + goto cleanup; > + utask->input_fd = dma_buf_fd(task->input, O_WRONLY|O_CLOEXEC); > + if (utask->input_fd < 0) { > + retval = utask->input_fd; > + goto cleanup; > + } > + utask->output_fd = dma_buf_fd(task->output, O_RDONLY|O_CLOEXEC); > + if (utask->output_fd < 0) { > + retval = utask->output_fd; > + goto cleanup; > + } > + /* keep dmabuf reference until freed with task free ioctl */ > + dma_buf_get(utask->input_fd); > + dma_buf_get(utask->output_fd); > + list_add_tail(&task->list, &stream->runtime->tasks); > + stream->runtime->total_tasks++; > + return 0; > +cleanup: > + snd_compr_task_free(task); > + return retval; > +} > + > +static int snd_compr_task_create(struct snd_compr_stream *stream, unsigned long arg) > +{ > + struct snd_compr_task *task __free(kfree) = NULL; > + int retval; > + > + if (stream->runtime->state != SNDRV_PCM_STATE_SETUP) > + return -EPERM; > + task = memdup_user((void __user *)arg, sizeof(*task)); > + if (IS_ERR(task)) > + return PTR_ERR(no_free_ptr(task)); > + retval = snd_compr_task_new(stream, task); > + if (retval >= 0) > + if (copy_to_user((void __user *)arg, task, sizeof(*task))) > + retval = -EFAULT; > + return retval; > +} > + > +static int snd_compr_task_start_prepare(struct snd_compr_task_runtime *task, > + struct snd_compr_task *utask) > +{ > + if (task == NULL) > + return -EINVAL; > + if (task->state >= SND_COMPRESS_TASK_STATE_FINISHED) > + return -EBUSY; > + if (utask->input_size > task->input->size) > + return -EINVAL; > + task->input_size = utask->input_size; > + task->state = SND_COMPRESS_TASK_STATE_IDLE; > + return 0; > +} > + > +static int snd_compr_task_start(struct snd_compr_stream *stream, struct snd_compr_task *utask) > +{ > + struct snd_compr_task_runtime *task; > + int retval; > + > + if (utask->origin_seqno > 0) { > + task = snd_compr_find_task(stream, utask->origin_seqno); > + retval = snd_compr_task_start_prepare(task, utask); > + if (retval < 0) > + return retval; > + task->seqno = utask->seqno = snd_compr_seqno_next(stream); > + utask->origin_seqno = 0; > + list_move_tail(&task->list, &stream->runtime->tasks); > + } else { > + task = snd_compr_find_task(stream, utask->seqno); > + if (task && task->state != SND_COMPRESS_TASK_STATE_IDLE) > + return -EBUSY; > + retval = snd_compr_task_start_prepare(task, utask); > + if (retval < 0) > + return retval; > + } > + retval = stream->ops->task_start(stream, task); > + if (retval >= 0) { > + task->state = SND_COMPRESS_TASK_STATE_ACTIVE; > + stream->runtime->active_tasks++; > + } > + return retval; > +} > + > +static int snd_compr_task_start_ioctl(struct snd_compr_stream *stream, unsigned long arg) > +{ > + struct snd_compr_task *task __free(kfree) = NULL; > + int retval; > + > + if (stream->runtime->state != SNDRV_PCM_STATE_SETUP) > + return -EPERM; > + task = memdup_user((void __user *)arg, sizeof(*task)); > + if (IS_ERR(task)) > + return PTR_ERR(no_free_ptr(task)); > + retval = snd_compr_task_start(stream, task); > + if (retval >= 0) > + if (copy_to_user((void __user *)arg, task, sizeof(*task))) > + retval = -EFAULT; > + return retval; > +} > + > +static void snd_compr_task_stop_one(struct snd_compr_stream *stream, > + struct snd_compr_task_runtime *task) > +{ > + if (task->state != SND_COMPRESS_TASK_STATE_ACTIVE) > + return; > + stream->ops->task_stop(stream, task); > + if (!snd_BUG_ON(stream->runtime->active_tasks == 0)) > + stream->runtime->active_tasks--; > + list_move_tail(&task->list, &stream->runtime->tasks); > + task->state = SND_COMPRESS_TASK_STATE_IDLE; > +} > + > +static void snd_compr_task_free_one(struct snd_compr_stream *stream, > + struct snd_compr_task_runtime *task) > +{ > + snd_compr_task_stop_one(stream, task); > + stream->ops->task_free(stream, task); > + list_del(&task->list); > + snd_compr_task_free(task); > + stream->runtime->total_tasks--; > +} > + > +static void snd_compr_task_free_all(struct snd_compr_stream *stream) > +{ > + struct snd_compr_task_runtime *task; > + > + list_for_each_entry(task, &stream->runtime->tasks, list) > + snd_compr_task_free_one(stream, task); > +} > + > +typedef void (*snd_compr_seq_func_t)(struct snd_compr_stream *stream, > + struct snd_compr_task_runtime *task); > + > +static int snd_compr_task_seq(struct snd_compr_stream *stream, unsigned long arg, > + snd_compr_seq_func_t fcn) > +{ > + struct snd_compr_task_runtime *task; > + __u64 seqno; > + int retval; > + > + if (stream->runtime->state != SNDRV_PCM_STATE_SETUP) > + return -EPERM; > + retval = get_user(seqno, (__u64 __user *)arg); > + if (retval < 0) > + return retval; > + retval = 0; > + if (seqno == 0) { > + list_for_each_entry(task, &stream->runtime->tasks, list) > + fcn(stream, task); > + } else { > + task = snd_compr_find_task(stream, seqno); > + if (task == NULL) { > + retval = -EINVAL; > + } else { > + fcn(stream, task); > + } > + } > + return retval; > +} > + > +static int snd_compr_task_status(struct snd_compr_stream *stream, > + struct snd_compr_task_status *status) > +{ > + struct snd_compr_task_runtime *task; > + > + task = snd_compr_find_task(stream, status->seqno); > + if (task == NULL) > + return -EINVAL; > + status->output_size = task->output_size; > + status->state = task->state; > + return 0; > +} > + > +static int snd_compr_task_status_ioctl(struct snd_compr_stream *stream, unsigned long arg) > +{ > + struct snd_compr_task_status *status __free(kfree) = NULL; > + int retval; > + > + if (stream->runtime->state != SNDRV_PCM_STATE_SETUP) > + return -EPERM; > + status = memdup_user((void __user *)arg, sizeof(*status)); > + if (IS_ERR(status)) > + return PTR_ERR(no_free_ptr(status)); > + retval = snd_compr_task_status(stream, status); > + if (retval >= 0) > + if (copy_to_user((void __user *)arg, status, sizeof(*status))) > + retval = -EFAULT; > + return retval; > +} > + > +/** > + * snd_compr_task_finished: Notify that the task was finished > + * @stream: pointer to stream > + * @task: runtime task structure > + * > + * Set the finished task state and notify waiters. > + */ > +void snd_compr_task_finished(struct snd_compr_stream *stream, > + struct snd_compr_task_runtime *task) > +{ > + guard(mutex)(&stream->device->lock); > + if (!snd_BUG_ON(stream->runtime->active_tasks == 0)) > + stream->runtime->active_tasks--; > + task->state = SND_COMPRESS_TASK_STATE_FINISHED; > + wake_up(&stream->runtime->sleep); > +} > +EXPORT_SYMBOL_GPL(snd_compr_task_finished); > + > +#endif /* CONFIG_COMPRESS_PASSTHROUGH */ > + > static long snd_compr_ioctl(struct file *f, unsigned int cmd, unsigned long arg) > { > struct snd_compr_file *data = f->private_data; > @@ -968,6 +1276,27 @@ static long snd_compr_ioctl(struct file *f, unsigned int cmd, unsigned long arg) > return snd_compr_set_metadata(stream, arg); > case _IOC_NR(SNDRV_COMPRESS_GET_METADATA): > return snd_compr_get_metadata(stream, arg); > + } > + > + if (stream->direction == SND_COMPRESS_PASSTHROUGH) { > +#if IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) > + switch (_IOC_NR(cmd)) { > + case _IOC_NR(SNDRV_COMPRESS_TASK_CREATE): > + return snd_compr_task_create(stream, arg); > + case _IOC_NR(SNDRV_COMPRESS_TASK_FREE): > + return snd_compr_task_seq(stream, arg, snd_compr_task_free_one); > + case _IOC_NR(SNDRV_COMPRESS_TASK_START): > + return snd_compr_task_start_ioctl(stream, arg); > + case _IOC_NR(SNDRV_COMPRESS_TASK_STOP): > + return snd_compr_task_seq(stream, arg, snd_compr_task_stop_one); > + case _IOC_NR(SNDRV_COMPRESS_TASK_STATUS): > + return snd_compr_task_status_ioctl(stream, arg); > + } > +#endif > + return -ENOTTY; > + } > + > + switch (_IOC_NR(cmd)) { > case _IOC_NR(SNDRV_COMPRESS_TSTAMP): > return snd_compr_tstamp(stream, arg); > case _IOC_NR(SNDRV_COMPRESS_AVAIL): > @@ -1140,6 +1469,11 @@ int snd_compress_new(struct snd_card *card, int device, > }; > int ret; > > +#if !IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) > + if (snd_BUG_ON(dirn == SND_COMPRESS_PASSTHROUGH)) > + return -EINVAL; > +#endif > + > compr->card = card; > compr->device = device; > compr->direction = dirn; > -- > 2.45.2 >
On 02. 07. 24 15:51, Vinod Koul wrote: > > Hi Jaroslav, > > On 24-06-24, 17:31, Jaroslav Kysela wrote: >> On 24. 06. 24 16:47, Vinod Koul wrote: >>> On 24-06-24, 15:58, Jaroslav Kysela wrote: >>>> There is a requirement to expose the audio hardware that accelerates various >>>> tasks for user space such as sample rate converters, compressed >>>> stream decoders, etc. >>> >>> Can you please tell me more about this requirement. The initial view of >>> compressed API was data only and use alsa kcontrols to handle the DSP >>> functions? I would like to understand why we need a new API. >> >> There are very long threads for v4l audio support - last v15 thread: >> >> https://lore.kernel.org/linux-media/1710834674-3285-1-git-send-email-shengjiu.wang@nxp.com/ > > Very long indeed but very interesting. I think going compressed audio > way seems reasonable choice to me > >> >> So the goal is to create something more efficient for the offload work, when >> the data (decoded/converted) should be returned back to the user space. > > Not rendered? so we are using it as an accelerator...? Yes, it's just to accelerate decoding/encoding in the user space chain. >>> What about the user of this API, i would like to see that as well >> >> Any audio streaming framework like gstreamer or ffmpeg who can accelerate >> stream conversions in hardware for capable devices. > > I meant to see driver users along with this patch :-) > > That also reminds me to ask about usermode support for this, are you > planning to support it in tinycompress? Yes, I do, but my time is limited in the next weeks, but it's on my to-do list. >>>> +A new direction SND_COMPRESS_PASSTHROUGH is introduced to identify >>>> +the passthrough API. >>> >>> Is passthrough really a new good new name, this suggests data being >>> passed thru but that is not the case... >> >> It's something like "PASS data THROUGH kernel/driver". So it makes sense. My >> alternate name may be ACCEL (like acceleration). > > I like that.. To clarify, are you speaking about "passthrough" or "accel" here? > Also do we change the device name if the passthru is > enabled? it should not be called a playback or capture compressed device I tried to follow playback/capture scheme - O_ACCMODE for open determines the direction - in the passthrough mode the matched value is O_RDWR. Also, the range for static minor numbers is shared so different device name may be problematic. Jaroslav
On 12. 07. 24 5:38, Shengjiu Wang wrote: > On Mon, Jun 24, 2024 at 9:58 PM Jaroslav Kysela <perex@perex.cz> wrote: >> >> There is a requirement to expose the audio hardware that accelerates various >> tasks for user space such as sample rate converters, compressed >> stream decoders, etc. >> >> This is description for the API extension for the compress ALSA API which >> is able to handle "tasks" that are not bound to real-time operations >> and allows for the serialization of operations. >> >> For details, refer to "compress-passthrough.rst" document. >> >> Cc: Mark Brown <broonie@kernel.org> >> Cc: Shengjiu Wang <shengjiu.wang@gmail.com> >> Cc: Nicolas Dufresne <nicolas@ndufresne.ca> >> Cc: Amadeusz Sławiński <amadeuszx.slawinski@linux.intel.com> >> Cc: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> >> Cc: Vinod Koul <vkoul@kernel.org> >> Signed-off-by: Jaroslav Kysela <perex@perex.cz> > > Gentle ping. Looking forward to the new version. Please, give me more time to include Vinod's requests. It's on my to-do list. > I have tested the ASRC function based on this API, basically can work. That sounds nice. Jaroslav
On 12-07-24, 18:45, Jaroslav Kysela wrote: > On 02. 07. 24 15:51, Vinod Koul wrote: > > > > What about the user of this API, i would like to see that as well > > > > > > Any audio streaming framework like gstreamer or ffmpeg who can accelerate > > > stream conversions in hardware for capable devices. > > > > I meant to see driver users along with this patch :-) > > > > That also reminds me to ask about usermode support for this, are you > > planning to support it in tinycompress? > > Yes, I do, but my time is limited in the next weeks, but it's on my to-do list. > > > > > > +A new direction SND_COMPRESS_PASSTHROUGH is introduced to identify > > > > > +the passthrough API. > > > > > > > > Is passthrough really a new good new name, this suggests data being > > > > passed thru but that is not the case... > > > > > > It's something like "PASS data THROUGH kernel/driver". So it makes sense. My > > > alternate name may be ACCEL (like acceleration). > > > > I like that.. > > To clarify, are you speaking about "passthrough" or "accel" here? Confirming that I would prefer 'accel' here and probably not to confuse with new 'accel' subsystem maybe compr_accel or something.. Better suggestions?
diff --git a/Documentation/sound/designs/compress-passthrough.rst b/Documentation/sound/designs/compress-passthrough.rst new file mode 100644 index 000000000000..975462500c33 --- /dev/null +++ b/Documentation/sound/designs/compress-passthrough.rst @@ -0,0 +1,125 @@ +================================= +ALSA Co-processor Passthrough API +================================= + +Jaroslav Kysela <perex@perex.cz> + + +Overview +======== + +There is a requirement to expose the audio hardware that accelerates various +tasks for user space such as sample rate converters, compressed +stream decoders, etc. + +This is description for the API extension for the compress ALSA API which +is able to handle "tasks" that are not bound to real-time operations +and allows for the serialization of operations. + +Requirements +============ + +The main requirements are: + +- serialization of multiple tasks for user space to allow multiple + operations without user space intervention + +- separate buffers (input + output) for each operation + +- expose buffers using mmap to user space + +- signal user space when the task is finished (standard poll mechanism) + +Design +====== + +A new direction SND_COMPRESS_PASSTHROUGH is introduced to identify +the passthrough API. + +The API extension shares device enumeration and parameters handling from +the main compressed API. All other realtime streaming ioctls are deactivated +and a new set of task related ioctls are introduced. The standard +read/write/mmap I/O operations are not supported in the passthrough device. + +Device ("stream") state handling is reduced to OPEN/SETUP. All other +states are not available for the passthrough mode. + +Data I/O mechanism is using standard dma-buf interface with all advantages +like mmap, standard I/O, buffer sharing etc. One buffer is used for the +input data and second (separate) buffer is used for the output data. Each task +have separate I/O buffers. + +For the buffering parameters, the fragments means a limit of allocated tasks +for given device. The fragment_size limits the input buffer size for the given +device. The output buffer size is determined by the driver (may be different +from the input buffer size). + +State Machine +============= + +The passthrough audio stream state machine is described below : + + +----------+ + | | + | OPEN | + | | + +----------+ + | + | + | compr_set_params() + | + v + all passthrough task ops +----------+ + +------------------------------------| | + | | SETUP | + | | + | +----------+ + | | + +------------------------------------------+ + + +Passthrough operations (ioctls) +=============================== + +CREATE +------ +Creates a set of input/output buffers. The input buffer size is +fragment_size. Allocates unique seqno. + +The hardware drivers allocate internal 'struct dma_buf' for both input and +output buffers (using 'dma_buf_export()' function). The anonymous +file descriptors for those buffers are passed to user space. + +FREE +---- +Free a set of input/output buffers. If a task is active, the stop +operation is executed before. If seqno is zero, operation is executed for all +tasks. + +START +----- +Starts (queues) a task. There are two cases of the task start - right after +the task is created. In this case, origin_seqno must be zero. +The second case is for reusing of already finished task. The origin_seqno +must identify the task to be reused. In both cases, a new seqno value +is allocated and returned to user space. + +The prerequisite is that application filled input dma buffer with +new source data and set input_size to pass the real data size to the driver. + +The order of data processing is preserved (first started job must be +finished at first). + +STOP +---- +Stop (dequeues) a task. If seqno is zero, operation is executed for all +tasks. + +STATUS +------ +Obtain the task status (active, finished). Also, the driver will set +the real output data size (valid area in the output buffer). + +Credits +======= +- ... diff --git a/include/sound/compress_driver.h b/include/sound/compress_driver.h index bcf872c17dd3..433ad897f054 100644 --- a/include/sound/compress_driver.h +++ b/include/sound/compress_driver.h @@ -19,6 +19,22 @@ struct snd_compr_ops; +/** + * struct snd_compr_task_runtime: task runtime description + * + */ +struct snd_compr_task_runtime { + struct list_head list; + struct dma_buf *input; + struct dma_buf *output; + u64 seqno; + u64 input_size; + u64 output_size; + u8 state; + void *private_value; +}; + + /** * struct snd_compr_runtime: runtime stream description * @state: stream state @@ -54,6 +70,13 @@ struct snd_compr_runtime { dma_addr_t dma_addr; size_t dma_bytes; struct snd_dma_buffer *dma_buffer_p; + +#if IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) + u32 active_tasks; + u32 total_tasks; + u64 task_seqno; + struct list_head tasks; +#endif }; /** @@ -132,6 +155,12 @@ struct snd_compr_ops { struct snd_compr_caps *caps); int (*get_codec_caps) (struct snd_compr_stream *stream, struct snd_compr_codec_caps *codec); +#if IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) + int (*task_create) (struct snd_compr_stream *stream, struct snd_compr_task_runtime *task); + int (*task_start) (struct snd_compr_stream *stream, struct snd_compr_task_runtime *task); + int (*task_stop) (struct snd_compr_stream *stream, struct snd_compr_task_runtime *task); + int (*task_free) (struct snd_compr_stream *stream, struct snd_compr_task_runtime *task); +#endif }; /** @@ -242,4 +271,9 @@ int snd_compr_free_pages(struct snd_compr_stream *stream); int snd_compr_stop_error(struct snd_compr_stream *stream, snd_pcm_state_t state); +#if IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) +void snd_compr_task_finished(struct snd_compr_stream *stream, + struct snd_compr_task_runtime *task); +#endif + #endif diff --git a/include/uapi/sound/compress_offload.h b/include/uapi/sound/compress_offload.h index d185957f3fe0..5fed1979522b 100644 --- a/include/uapi/sound/compress_offload.h +++ b/include/uapi/sound/compress_offload.h @@ -1,4 +1,4 @@ -/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */ + /* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */ /* * compress_offload.h - compress offload header definations * @@ -14,7 +14,7 @@ #include <sound/compress_params.h> -#define SNDRV_COMPRESS_VERSION SNDRV_PROTOCOL_VERSION(0, 2, 0) +#define SNDRV_COMPRESS_VERSION SNDRV_PROTOCOL_VERSION(0, 3, 0) /** * struct snd_compressed_buffer - compressed buffer * @fragment_size: size of buffer fragment in bytes @@ -68,7 +68,8 @@ struct snd_compr_avail { enum snd_compr_direction { SND_COMPRESS_PLAYBACK = 0, - SND_COMPRESS_CAPTURE + SND_COMPRESS_CAPTURE, + SND_COMPRESS_PASSTHROUGH }; /** @@ -127,6 +128,42 @@ struct snd_compr_metadata { __u32 value[8]; } __attribute__((packed, aligned(4))); +/** + * struct snd_compr_task - task primitive for non-realtime operation + * @seqno: sequence number (task identifier) + * @origin_seqno: previous sequence number (task identifier) - for reuse + * @input_fd: data input file descriptor (dma-buf) + * @output_fd: data output file descriptor (dma-buf) + * @input_size: filled data in bytes (from caller, must not exceed fragment size) + */ +struct snd_compr_task { + __u64 seqno; + __u64 origin_seqno; + int input_fd; + int output_fd; + __u64 input_size; + __u8 reserved[16]; +} __attribute__((packed, aligned(4))); + +enum snd_compr_state { + SND_COMPRESS_TASK_STATE_IDLE = 0, + SND_COMPRESS_TASK_STATE_ACTIVE, + SND_COMPRESS_TASK_STATE_FINISHED +}; + +/** + * struct snd_compr_task_status - task status + * @seqno: sequence number (task identifier) + * @output_size: filled data in bytes (from driver) + * @state: actual task state (SND_COMPRESS_TASK_STATE_*) + */ +struct snd_compr_task_status { + __u64 seqno; + __u64 output_size; + __u8 state; + __u8 reserved[15]; +} __attribute__((packed, aligned(4))); + /* * compress path ioctl definitions * SNDRV_COMPRESS_GET_CAPS: Query capability of DSP @@ -164,6 +201,14 @@ struct snd_compr_metadata { #define SNDRV_COMPRESS_DRAIN _IO('C', 0x34) #define SNDRV_COMPRESS_NEXT_TRACK _IO('C', 0x35) #define SNDRV_COMPRESS_PARTIAL_DRAIN _IO('C', 0x36) + + +#define SNDRV_COMPRESS_TASK_CREATE _IOWR('C', 0x60, struct snd_compr_task) +#define SNDRV_COMPRESS_TASK_FREE _IOW('C', 0x61, __u64) +#define SNDRV_COMPRESS_TASK_START _IOWR('C', 0x62, struct snd_compr_task) +#define SNDRV_COMPRESS_TASK_STOP _IOW('C', 0x63, __u64) +#define SNDRV_COMPRESS_TASK_STATUS _IOWR('C', 0x68, struct snd_compr_task_status) + /* * TODO * 1. add mmap support diff --git a/sound/core/Kconfig b/sound/core/Kconfig index 8077f481d84f..309cfae3b8b2 100644 --- a/sound/core/Kconfig +++ b/sound/core/Kconfig @@ -59,6 +59,9 @@ config SND_CORE_TEST config SND_COMPRESS_OFFLOAD tristate +config SND_COMPRESS_PASSTHROUGH + bool + config SND_JACK bool diff --git a/sound/core/compress_offload.c b/sound/core/compress_offload.c index f0008fa2d839..7b5e2e9e737e 100644 --- a/sound/core/compress_offload.c +++ b/sound/core/compress_offload.c @@ -24,6 +24,7 @@ #include <linux/types.h> #include <linux/uio.h> #include <linux/uaccess.h> +#include <linux/dma-buf.h> #include <linux/module.h> #include <linux/compat.h> #include <sound/core.h> @@ -54,6 +55,12 @@ struct snd_compr_file { static void error_delayed_work(struct work_struct *work); +#if IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) +static void snd_compr_task_free_all(struct snd_compr_stream *stream); +#else +static inline void snd_compr_task_free_all(struct snd_compr_stream *stream) { } +#endif + /* * a note on stream states used: * we use following states in the compressed core @@ -85,6 +92,8 @@ static int snd_compr_open(struct inode *inode, struct file *f) dirn = SND_COMPRESS_PLAYBACK; else if ((f->f_flags & O_ACCMODE) == O_RDONLY) dirn = SND_COMPRESS_CAPTURE; + else if ((f->f_flags & O_ACCMODE) == O_RDWR) + dirn = SND_COMPRESS_PASSTHROUGH; else return -EINVAL; @@ -112,6 +121,9 @@ static int snd_compr_open(struct inode *inode, struct file *f) } INIT_DELAYED_WORK(&data->stream.error_work, error_delayed_work); +#if IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) + INIT_LIST_HEAD(&runtime->tasks); +#endif data->stream.ops = compr->ops; data->stream.direction = dirn; @@ -154,6 +166,8 @@ static int snd_compr_free(struct inode *inode, struct file *f) break; } + snd_compr_task_free_all(&data->stream); + data->stream.ops->free(&data->stream); if (!data->stream.runtime->dma_buffer_p) kfree(data->stream.runtime->buffer); @@ -226,6 +240,9 @@ snd_compr_ioctl_avail(struct snd_compr_stream *stream, unsigned long arg) struct snd_compr_avail ioctl_avail; size_t avail; + if (stream->direction == SND_COMPRESS_PASSTHROUGH) + return -EBADFD; + avail = snd_compr_calc_avail(stream, &ioctl_avail); ioctl_avail.avail = avail; @@ -287,6 +304,8 @@ static ssize_t snd_compr_write(struct file *f, const char __user *buf, return -EFAULT; stream = &data->stream; + if (stream->direction == SND_COMPRESS_PASSTHROUGH) + return -EBADFD; guard(mutex)(&stream->device->lock); /* write is allowed when stream is running or has been steup */ switch (stream->runtime->state) { @@ -336,6 +355,8 @@ static ssize_t snd_compr_read(struct file *f, char __user *buf, return -EFAULT; stream = &data->stream; + if (stream->direction == SND_COMPRESS_PASSTHROUGH) + return -EBADFD; guard(mutex)(&stream->device->lock); /* read is allowed when stream is running, paused, draining and setup @@ -385,6 +406,7 @@ static __poll_t snd_compr_poll(struct file *f, poll_table *wait) { struct snd_compr_file *data = f->private_data; struct snd_compr_stream *stream; + struct snd_compr_runtime *runtime; size_t avail; __poll_t retval = 0; @@ -392,6 +414,7 @@ static __poll_t snd_compr_poll(struct file *f, poll_table *wait) return EPOLLERR; stream = &data->stream; + runtime = stream->runtime; guard(mutex)(&stream->device->lock); @@ -405,6 +428,20 @@ static __poll_t snd_compr_poll(struct file *f, poll_table *wait) poll_wait(f, &stream->runtime->sleep, wait); +#if IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) + if (stream->direction == SND_COMPRESS_PASSTHROUGH) { + struct snd_compr_task_runtime *task; + if (runtime->fragments > runtime->active_tasks) + retval |= EPOLLOUT | EPOLLWRNORM; + task = list_first_entry_or_null(&runtime->tasks, + struct snd_compr_task_runtime, + list); + if (task && task->state == SND_COMPRESS_TASK_STATE_FINISHED) + retval |= EPOLLIN | EPOLLRDNORM; + return retval; + } +#endif + avail = snd_compr_get_avail(stream); pr_debug("avail is %ld\n", (unsigned long)avail); /* check if we have at least one fragment to fill */ @@ -521,6 +558,9 @@ static int snd_compr_allocate_buffer(struct snd_compr_stream *stream, unsigned int buffer_size; void *buffer = NULL; + if (stream->direction == SND_COMPRESS_PASSTHROUGH) + goto params; + buffer_size = params->buffer.fragment_size * params->buffer.fragments; if (stream->ops->copy) { buffer = NULL; @@ -543,18 +583,30 @@ static int snd_compr_allocate_buffer(struct snd_compr_stream *stream, if (!buffer) return -ENOMEM; } - stream->runtime->fragment_size = params->buffer.fragment_size; - stream->runtime->fragments = params->buffer.fragments; + stream->runtime->buffer = buffer; stream->runtime->buffer_size = buffer_size; +params: + stream->runtime->fragment_size = params->buffer.fragment_size; + stream->runtime->fragments = params->buffer.fragments; return 0; } -static int snd_compress_check_input(struct snd_compr_params *params) +static int +snd_compress_check_input(struct snd_compr_stream *stream, struct snd_compr_params *params) { + u32 max_fragments; + /* first let's check the buffer parameter's */ - if (params->buffer.fragment_size == 0 || - params->buffer.fragments > U32_MAX / params->buffer.fragment_size || + if (params->buffer.fragment_size == 0) + return -EINVAL; + + if (stream->direction == SND_COMPRESS_PASSTHROUGH) + max_fragments = 64; /* safe value */ + else + max_fragments = U32_MAX / params->buffer.fragment_size; + + if (params->buffer.fragments > max_fragments || params->buffer.fragments == 0) return -EINVAL; @@ -583,7 +635,7 @@ snd_compr_set_params(struct snd_compr_stream *stream, unsigned long arg) if (IS_ERR(params)) return PTR_ERR(no_free_ptr(params)); - retval = snd_compress_check_input(params); + retval = snd_compress_check_input(stream, params); if (retval) return retval; @@ -939,6 +991,262 @@ static int snd_compr_partial_drain(struct snd_compr_stream *stream) return snd_compress_wait_for_drain(stream); } +#if IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) + +static struct snd_compr_task_runtime * +snd_compr_find_task(struct snd_compr_stream *stream, __u64 seqno) +{ + struct snd_compr_task_runtime *task; + + list_for_each_entry(task, &stream->runtime->tasks, list) { + if (task->seqno == seqno) + return task; + } + return NULL; +} + +static void snd_compr_task_free(struct snd_compr_task_runtime *task) +{ + if (task->output) + dma_buf_put(task->output); + if (task->input) + dma_buf_put(task->input); + kfree(task); +} + +static u64 snd_compr_seqno_next(struct snd_compr_stream *stream) +{ + u64 seqno = ++stream->runtime->task_seqno; + if (seqno == 0) + seqno = ++stream->runtime->task_seqno; + return seqno; +} + +static int snd_compr_task_new(struct snd_compr_stream *stream, struct snd_compr_task *utask) +{ + struct snd_compr_task_runtime *task; + int retval; + + if (stream->runtime->total_tasks >= stream->runtime->fragments) + return -EBUSY; + if (utask->origin_seqno != 0 || utask->input_size != 0) + return -EINVAL; + task = kzalloc(sizeof(*task), GFP_KERNEL); + if (task == NULL) + return -ENOMEM; + task->seqno = utask->seqno = snd_compr_seqno_next(stream); + task->input_size = utask->input_size; + retval = stream->ops->task_create(stream, task); + if (retval < 0) + goto cleanup; + utask->input_fd = dma_buf_fd(task->input, O_WRONLY|O_CLOEXEC); + if (utask->input_fd < 0) { + retval = utask->input_fd; + goto cleanup; + } + utask->output_fd = dma_buf_fd(task->output, O_RDONLY|O_CLOEXEC); + if (utask->output_fd < 0) { + retval = utask->output_fd; + goto cleanup; + } + /* keep dmabuf reference until freed with task free ioctl */ + dma_buf_get(utask->input_fd); + dma_buf_get(utask->output_fd); + list_add_tail(&task->list, &stream->runtime->tasks); + stream->runtime->total_tasks++; + return 0; +cleanup: + snd_compr_task_free(task); + return retval; +} + +static int snd_compr_task_create(struct snd_compr_stream *stream, unsigned long arg) +{ + struct snd_compr_task *task __free(kfree) = NULL; + int retval; + + if (stream->runtime->state != SNDRV_PCM_STATE_SETUP) + return -EPERM; + task = memdup_user((void __user *)arg, sizeof(*task)); + if (IS_ERR(task)) + return PTR_ERR(no_free_ptr(task)); + retval = snd_compr_task_new(stream, task); + if (retval >= 0) + if (copy_to_user((void __user *)arg, task, sizeof(*task))) + retval = -EFAULT; + return retval; +} + +static int snd_compr_task_start_prepare(struct snd_compr_task_runtime *task, + struct snd_compr_task *utask) +{ + if (task == NULL) + return -EINVAL; + if (task->state >= SND_COMPRESS_TASK_STATE_FINISHED) + return -EBUSY; + if (utask->input_size > task->input->size) + return -EINVAL; + task->input_size = utask->input_size; + task->state = SND_COMPRESS_TASK_STATE_IDLE; + return 0; +} + +static int snd_compr_task_start(struct snd_compr_stream *stream, struct snd_compr_task *utask) +{ + struct snd_compr_task_runtime *task; + int retval; + + if (utask->origin_seqno > 0) { + task = snd_compr_find_task(stream, utask->origin_seqno); + retval = snd_compr_task_start_prepare(task, utask); + if (retval < 0) + return retval; + task->seqno = utask->seqno = snd_compr_seqno_next(stream); + utask->origin_seqno = 0; + list_move_tail(&task->list, &stream->runtime->tasks); + } else { + task = snd_compr_find_task(stream, utask->seqno); + if (task && task->state != SND_COMPRESS_TASK_STATE_IDLE) + return -EBUSY; + retval = snd_compr_task_start_prepare(task, utask); + if (retval < 0) + return retval; + } + retval = stream->ops->task_start(stream, task); + if (retval >= 0) { + task->state = SND_COMPRESS_TASK_STATE_ACTIVE; + stream->runtime->active_tasks++; + } + return retval; +} + +static int snd_compr_task_start_ioctl(struct snd_compr_stream *stream, unsigned long arg) +{ + struct snd_compr_task *task __free(kfree) = NULL; + int retval; + + if (stream->runtime->state != SNDRV_PCM_STATE_SETUP) + return -EPERM; + task = memdup_user((void __user *)arg, sizeof(*task)); + if (IS_ERR(task)) + return PTR_ERR(no_free_ptr(task)); + retval = snd_compr_task_start(stream, task); + if (retval >= 0) + if (copy_to_user((void __user *)arg, task, sizeof(*task))) + retval = -EFAULT; + return retval; +} + +static void snd_compr_task_stop_one(struct snd_compr_stream *stream, + struct snd_compr_task_runtime *task) +{ + if (task->state != SND_COMPRESS_TASK_STATE_ACTIVE) + return; + stream->ops->task_stop(stream, task); + if (!snd_BUG_ON(stream->runtime->active_tasks == 0)) + stream->runtime->active_tasks--; + list_move_tail(&task->list, &stream->runtime->tasks); + task->state = SND_COMPRESS_TASK_STATE_IDLE; +} + +static void snd_compr_task_free_one(struct snd_compr_stream *stream, + struct snd_compr_task_runtime *task) +{ + snd_compr_task_stop_one(stream, task); + stream->ops->task_free(stream, task); + list_del(&task->list); + snd_compr_task_free(task); + stream->runtime->total_tasks--; +} + +static void snd_compr_task_free_all(struct snd_compr_stream *stream) +{ + struct snd_compr_task_runtime *task; + + list_for_each_entry(task, &stream->runtime->tasks, list) + snd_compr_task_free_one(stream, task); +} + +typedef void (*snd_compr_seq_func_t)(struct snd_compr_stream *stream, + struct snd_compr_task_runtime *task); + +static int snd_compr_task_seq(struct snd_compr_stream *stream, unsigned long arg, + snd_compr_seq_func_t fcn) +{ + struct snd_compr_task_runtime *task; + __u64 seqno; + int retval; + + if (stream->runtime->state != SNDRV_PCM_STATE_SETUP) + return -EPERM; + retval = get_user(seqno, (__u64 __user *)arg); + if (retval < 0) + return retval; + retval = 0; + if (seqno == 0) { + list_for_each_entry(task, &stream->runtime->tasks, list) + fcn(stream, task); + } else { + task = snd_compr_find_task(stream, seqno); + if (task == NULL) { + retval = -EINVAL; + } else { + fcn(stream, task); + } + } + return retval; +} + +static int snd_compr_task_status(struct snd_compr_stream *stream, + struct snd_compr_task_status *status) +{ + struct snd_compr_task_runtime *task; + + task = snd_compr_find_task(stream, status->seqno); + if (task == NULL) + return -EINVAL; + status->output_size = task->output_size; + status->state = task->state; + return 0; +} + +static int snd_compr_task_status_ioctl(struct snd_compr_stream *stream, unsigned long arg) +{ + struct snd_compr_task_status *status __free(kfree) = NULL; + int retval; + + if (stream->runtime->state != SNDRV_PCM_STATE_SETUP) + return -EPERM; + status = memdup_user((void __user *)arg, sizeof(*status)); + if (IS_ERR(status)) + return PTR_ERR(no_free_ptr(status)); + retval = snd_compr_task_status(stream, status); + if (retval >= 0) + if (copy_to_user((void __user *)arg, status, sizeof(*status))) + retval = -EFAULT; + return retval; +} + +/** + * snd_compr_task_finished: Notify that the task was finished + * @stream: pointer to stream + * @task: runtime task structure + * + * Set the finished task state and notify waiters. + */ +void snd_compr_task_finished(struct snd_compr_stream *stream, + struct snd_compr_task_runtime *task) +{ + guard(mutex)(&stream->device->lock); + if (!snd_BUG_ON(stream->runtime->active_tasks == 0)) + stream->runtime->active_tasks--; + task->state = SND_COMPRESS_TASK_STATE_FINISHED; + wake_up(&stream->runtime->sleep); +} +EXPORT_SYMBOL_GPL(snd_compr_task_finished); + +#endif /* CONFIG_COMPRESS_PASSTHROUGH */ + static long snd_compr_ioctl(struct file *f, unsigned int cmd, unsigned long arg) { struct snd_compr_file *data = f->private_data; @@ -968,6 +1276,27 @@ static long snd_compr_ioctl(struct file *f, unsigned int cmd, unsigned long arg) return snd_compr_set_metadata(stream, arg); case _IOC_NR(SNDRV_COMPRESS_GET_METADATA): return snd_compr_get_metadata(stream, arg); + } + + if (stream->direction == SND_COMPRESS_PASSTHROUGH) { +#if IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) + switch (_IOC_NR(cmd)) { + case _IOC_NR(SNDRV_COMPRESS_TASK_CREATE): + return snd_compr_task_create(stream, arg); + case _IOC_NR(SNDRV_COMPRESS_TASK_FREE): + return snd_compr_task_seq(stream, arg, snd_compr_task_free_one); + case _IOC_NR(SNDRV_COMPRESS_TASK_START): + return snd_compr_task_start_ioctl(stream, arg); + case _IOC_NR(SNDRV_COMPRESS_TASK_STOP): + return snd_compr_task_seq(stream, arg, snd_compr_task_stop_one); + case _IOC_NR(SNDRV_COMPRESS_TASK_STATUS): + return snd_compr_task_status_ioctl(stream, arg); + } +#endif + return -ENOTTY; + } + + switch (_IOC_NR(cmd)) { case _IOC_NR(SNDRV_COMPRESS_TSTAMP): return snd_compr_tstamp(stream, arg); case _IOC_NR(SNDRV_COMPRESS_AVAIL): @@ -1140,6 +1469,11 @@ int snd_compress_new(struct snd_card *card, int device, }; int ret; +#if !IS_ENABLED(CONFIG_SND_COMPRESS_PASSTHROUGH) + if (snd_BUG_ON(dirn == SND_COMPRESS_PASSTHROUGH)) + return -EINVAL; +#endif + compr->card = card; compr->device = device; compr->direction = dirn;
There is a requirement to expose the audio hardware that accelerates various tasks for user space such as sample rate converters, compressed stream decoders, etc. This is description for the API extension for the compress ALSA API which is able to handle "tasks" that are not bound to real-time operations and allows for the serialization of operations. For details, refer to "compress-passthrough.rst" document. Cc: Mark Brown <broonie@kernel.org> Cc: Shengjiu Wang <shengjiu.wang@gmail.com> Cc: Nicolas Dufresne <nicolas@ndufresne.ca> Cc: Amadeusz Sławiński <amadeuszx.slawinski@linux.intel.com> Cc: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> Cc: Vinod Koul <vkoul@kernel.org> Signed-off-by: Jaroslav Kysela <perex@perex.cz> --- v3..v4: - resolved Takashi's requests: - make CONFIG_SND_COMPRESS_PASSTHROUGH as bool, remove wrong DMA_BUF selection - use EXPORT_SYMBOL_GPL for snd_compr_task_finished function - fix snd_compr_find_task indentation - add more CONFIG_SND_COMPRESS_PASSTHROUGH #if blocks v2..v3: - fix missing runtime->tasks initialization (thanks Shengjiu Wang) - fix missing seqno initialization in task_new (thanks Shengjiu Wang) - fix reference counting for allocated dma buffers (thanks Shengjiu Wang) - use origin_seqno to reuse the already allocated buffers for new task v1..v2: - fix some documentation typos (thanks Amadeusz Sławiński) - fix memdup_user() error handling (thanks Takashi) - use one state variable instead multiple (thanks Takashi) - handle task limit (set to 64 - mentioned in documentation, NIY) - fix file release (free all tasks) --- .../sound/designs/compress-passthrough.rst | 125 +++++++ include/sound/compress_driver.h | 34 ++ include/uapi/sound/compress_offload.h | 51 ++- sound/core/Kconfig | 3 + sound/core/compress_offload.c | 346 +++++++++++++++++- 5 files changed, 550 insertions(+), 9 deletions(-) create mode 100644 Documentation/sound/designs/compress-passthrough.rst