shinys000114/linux
1
0
Fork 0
mirror of https://github.com/hardkernel/linux.git synced 2026-06-05 18:41:58 +09:00
Code Issues Packages Projects Releases Wiki Activity

vduse: validate block features only with block devices

Browse Source 
[ Upstream commit a115b5716fc9a64652aa9cb332070087178ffafa ]

This patch is preliminary work to enable network device
type support to VDUSE.

As VIRTIO_BLK_F_CONFIG_WCE shares the same value as
VIRTIO_NET_F_HOST_TSO4, we need to restrict its check
to Virtio-blk device type.

Acked-by: Jason Wang <jasowang@redhat.com>
Reviewed-by: Xie Yongji <xieyongji@bytedance.com>
Reviewed-by: Eugenio Pérez <eperezma@redhat.com>
Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>
Message-Id: <20240109111025.1320976-2-maxime.coquelin@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
Maxime Coquelin
2024-01-09 12:10:23 +01:00
committed by Greg Kroah-Hartman
parent 5f41401219
commit d99a4e147b
1 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
drivers/vdpa/vdpa_user/vduse_dev.c
View File

@@ -1416,13 +1416,14 @@ static bool device_is_allowed(u32 device_id)
return false; return false;
} }
static bool features_is_valid(u64 features) static bool features_is_valid(struct vduse_dev_config *config)
{ {
if (!(features & (1ULL << VIRTIO_F_ACCESS_PLATFORM))) if (!(config->features & BIT_ULL(VIRTIO_F_ACCESS_PLATFORM)))
return false; return false;
/* Now we only support read-only configuration space */ /* Now we only support read-only configuration space */
if (features & (1ULL << VIRTIO_BLK_F_CONFIG_WCE)) if ((config->device_id == VIRTIO_ID_BLOCK) &&
(config->features & BIT_ULL(VIRTIO_BLK_F_CONFIG_WCE)))
return false; return false;
return true; return true;
@@ -1449,7 +1450,7 @@ static bool vduse_validate_config(struct vduse_dev_config *config)
if (!device_is_allowed(config->device_id)) if (!device_is_allowed(config->device_id))
return false; return false;
if (!features_is_valid(config->features)) if (!features_is_valid(config))
return false; return false;
return true; return true;