[PATCH] virtio_blk: corrent types for status handling

Stefan Hajnoczi
Mon, 25 Oct 2021 08:22:08 -0000

On Mon, Oct 25, 2021 at 03:58:54AM -0400, Michael S. Tsirkin
wrote:
> virtblk_setup_cmd returns blk_status_t in an int, callers then assign it
> back to a blk_status_t variable. blk_status_t is either u32 or (more
> typically) u8 so it works, but is inelegant and causes sparse warnings.
>
> Pass the status in blk_status_t in a consistent way.
>
> Reported-by: kernel test robot <lkp at intel.com>
> Fixes: b2c5221fd074 ("virtio-blk: avoid preallocating big SGL for
data")
> Cc: Max Gurtovoy <mgurtovoy at nvidia.com>
> Signed-off-by: Michael S. Tsirkin <mst at redhat.com>
> ---
>  drivers/block/virtio_blk.c | 14 ++++++++------
>  1 file changed, 8 insertions(+), 6 deletions(-)

Reviewed-by: Stefan Hajnoczi <stefanha at redhat.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL:
<http://lists.linuxfoundation.org/pipermail/virtualization/attachments/20211025/9a069f54/attachment.sig>