Re: [Devel] [PATCH hci-8.1 0/3] drivers/vhost/blk: fix virtio-blk request header and payload parsing

[Andrey Zhadchenko <andrey.zhadchenko@virtuozzo.com>]

Reviewed-by: Andrey Zhadchenko <andrey.zhadchenko@virtuozzo.com>

On 10/21/26 10:46, Alexander Ivanov wrote:
> vhost-blk parsed guest requests from the whole descriptor chain without
> telling the device-readable part from the device-writable one, and used
> the header fields without byte order conversion.
> 
> Patch 1 converts hdr->type and hdr->sector to CPU byte order.
> 
> Patch 2 takes the request header only from the readable part and drops
> requests whose readable part is shorter than the header.
> 
> Patch 3 takes the write payload from the readable part and the payload
> of other requests from the writable part, reads the status byte from
> the end of the writable part, and fails reads and writes with an empty,
> oversized or misaligned payload with VIRTIO_BLK_S_IOERR instead of
> leaving them hanging.
> 
> https://virtuozzo.atlassian.net/browse/VSTOR-142755
> 
> Alexander Ivanov (3):
>    drivers/vhost/blk: convert virtio_blk_outhdr fields to CPU byte order
>    drivers/vhost/blk: take virtio_blk_outhdr only from the readable part
>    drivers/vhost/blk: take req->len from one side of the descriptor chain
> 
>   drivers/vhost/blk.c | 56 ++++++++++++++++++++++++++++++++-------------
>   1 file changed, 40 insertions(+), 16 deletions(-)
> 

_______________________________________________
Devel mailing list
Devel@openvz.org
https://lists.openvz.org/mailman/listinfo/devel