Re: [PATCH v25 09/16] ALSA: virtio: bound PCM transfers and retain DMA buffers

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Low] ALSA: virtio: missing Fixes trailer for PCM DMA page retention
  and bounds fix
--

commit 119548f8da39346df59cd856937f2c76e6900cb9
Author: Joonho Lee <mynameisjune111@gmail.com>

> ALSA: virtio: bound PCM transfers and retain DMA buffers
>
> Validate ACK geometry, conversion bounds and queued-byte accounting
> before submitting transfers. Retain the backing DMA pages until message
> retirement. Check RX status and upper payload bounds while accepting
> bounded short responses, including at STOP and PAUSE. Preserve the
> legacy RX completion and ACK policy until the request-owned conversion,
> including the legacy handling of empty PAUSE returns. The fixed-offset
> layout still has short-response data gaps; request-owned capture and its
> queue/accounting contract arrive together in patch 12. Non-MMU copies
> use only the validated returned payload length.
>
> Assisted-by: LLM
> Signed-off-by: Joonho Lee <mynameisjune111@gmail.com>

[Severity: Low]
This fixes DMA page lifetime and transfer bounds handling in
virtsnd_pcm_msg_alloc() and virtsnd_pcm_msg_complete().  Should this
include:

  Fixes: f40a28679e0b ("ALSA: virtio: handling control and I/O messages for the PCM device")

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261111025655.63951-1-mynameisjune111@gmail.com?part=9