Re: [PATCH] virtio_ring: fix stalls for packed rings

[Jason Wang]

On 2019/10/28 äå2:59, Sasha Levin wrote:
> Hi,
>
> [This is an automated email]
>
> This commit has been processed because it contains a -stable tag.
> The stable tag indicates that it's relevant for the following trees: all
>
> The bot has tested the following trees: v5.3.7, v4.19.80, v4.14.150, v4.9.197, v4.4.197.
>
> v5.3.7: Build OK!
> v4.19.80: Failed to apply! Possible dependencies:
>      138fd25148638 ("virtio_ring: add _split suffix for split ring functions")
>      1ce9e6055fa0a ("virtio_ring: introduce packed ring support")
>      cbeedb72b97ad ("virtio_ring: allocate desc state for split ring separately")
>      d79dca75c7968 ("virtio_ring: extract split ring handling from ring creation")
>      e593bf9751566 ("virtio_ring: put split ring fields in a sub struct")
>      e6f633e5beab6 ("virtio_ring: put split ring functions together")
>      f51f982682e2a ("virtio_ring: leverage event idx in packed ring")
>      fb3fba6b162aa ("virtio_ring: cache whether we will use DMA API")
>
> v4.14.150: Failed to apply! Possible dependencies:
>      138fd25148638 ("virtio_ring: add _split suffix for split ring functions")
>      1ce9e6055fa0a ("virtio_ring: introduce packed ring support")
>      cbeedb72b97ad ("virtio_ring: allocate desc state for split ring separately")
>      d79dca75c7968 ("virtio_ring: extract split ring handling from ring creation")
>      e593bf9751566 ("virtio_ring: put split ring fields in a sub struct")
>      e6f633e5beab6 ("virtio_ring: put split ring functions together")
>      f51f982682e2a ("virtio_ring: leverage event idx in packed ring")
>      fb3fba6b162aa ("virtio_ring: cache whether we will use DMA API")
>
> v4.9.197: Failed to apply! Possible dependencies:
>      0c7eaf5930e14 ("virtio_ring: fix description of virtqueue_get_buf")
>      138fd25148638 ("virtio_ring: add _split suffix for split ring functions")
>      1ce9e6055fa0a ("virtio_ring: introduce packed ring support")
>      44ed8089e991a ("scsi: virtio: Reduce BUG if total_sg > virtqueue size to WARN.")
>      5a08b04f63792 ("virtio: allow extra context per descriptor")
>      c60923cb9cb5e ("virtio_ring: fix complaint by sparse")
>      cbeedb72b97ad ("virtio_ring: allocate desc state for split ring separately")
>      e593bf9751566 ("virtio_ring: put split ring fields in a sub struct")
>      e6f633e5beab6 ("virtio_ring: put split ring functions together")
>      f51f982682e2a ("virtio_ring: leverage event idx in packed ring")
>      fb3fba6b162aa ("virtio_ring: cache whether we will use DMA API")
>
> v4.4.197: Failed to apply! Possible dependencies:
>      0c7eaf5930e14 ("virtio_ring: fix description of virtqueue_get_buf")
>      138fd25148638 ("virtio_ring: add _split suffix for split ring functions")
>      1ce9e6055fa0a ("virtio_ring: introduce packed ring support")
>      44ed8089e991a ("scsi: virtio: Reduce BUG if total_sg > virtqueue size to WARN.")
>      5a08b04f63792 ("virtio: allow extra context per descriptor")
>      780bc7903a32e ("virtio_ring: Support DMA APIs")
>      c60923cb9cb5e ("virtio_ring: fix complaint by sparse")
>      d26c96c810254 ("vring: Introduce vring_use_dma_api()")
>      e6f633e5beab6 ("virtio_ring: put split ring functions together")
>      f51f982682e2a ("virtio_ring: leverage event idx in packed ring")
>      fb3fba6b162aa ("virtio_ring: cache whether we will use DMA API")
>
>
> NOTE: The patch will not be queued to stable trees until it is upstream.
>
> How should we proceed with this patch?


It is only needed for kernel > 4.20.

Thanks


>