提交 a7529b33 编写于 作者: S Stefano Garzarella 提交者: Pengyuan Zhao

vdpa_sim: add supported_features field in vdpasim_dev_attr

stable inclusion
from stable-v5.11
commit 011c35ba
category: feature
bugzilla: https://gitee.com/openeuler/kernel/issues/I5WXCZ
CVE: NA

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=011c35bac5ef25f701d9a79bc731782889c0ff58

----------------------------------------------------------------------

Introduce a new VDPASIM_FEATURES macro with the generic features
supported by the vDPA simulator, and VDPASIM_NET_FEATURES macro with
vDPA-net features.

Add 'supported_features' field in vdpasim_dev_attr, to allow devices
to specify their features.
Co-developed-by: NMax Gurtovoy <mgurtovoy@nvidia.com>
Signed-off-by: NMax Gurtovoy <mgurtovoy@nvidia.com>
Acked-by: NJason Wang <jasowang@redhat.com>
Signed-off-by: NStefano Garzarella <sgarzare@redhat.com>
Link: https://lore.kernel.org/r/20201215144256.155342-9-sgarzare@redhat.comSigned-off-by: NMichael S. Tsirkin <mst@redhat.com>
Signed-off-by: NPengyuan Zhao <zhaopengyuan@hisilicon.com>
上级 9ed77dbf
...@@ -64,14 +64,16 @@ struct vdpasim_virtqueue { ...@@ -64,14 +64,16 @@ struct vdpasim_virtqueue {
#define VDPASIM_VQ_NUM 0x2 #define VDPASIM_VQ_NUM 0x2
#define VDPASIM_NAME "vdpasim-netdev" #define VDPASIM_NAME "vdpasim-netdev"
static u64 vdpasim_features = (1ULL << VIRTIO_F_ANY_LAYOUT) | #define VDPASIM_FEATURES ((1ULL << VIRTIO_F_ANY_LAYOUT) | \
(1ULL << VIRTIO_F_VERSION_1) | (1ULL << VIRTIO_F_VERSION_1) | \
(1ULL << VIRTIO_F_ACCESS_PLATFORM) | (1ULL << VIRTIO_F_ACCESS_PLATFORM))
(1ULL << VIRTIO_NET_F_MAC);
#define VDPASIM_NET_FEATURES (VDPASIM_FEATURES | \
(1ULL << VIRTIO_NET_F_MAC))
struct vdpasim; struct vdpasim;
struct vdpasim_dev_attr { struct vdpasim_dev_attr {
u64 supported_features;
size_t config_size; size_t config_size;
int nvqs; int nvqs;
u32 id; u32 id;
...@@ -135,7 +137,7 @@ static void vdpasim_queue_ready(struct vdpasim *vdpasim, unsigned int idx) ...@@ -135,7 +137,7 @@ static void vdpasim_queue_ready(struct vdpasim *vdpasim, unsigned int idx)
{ {
struct vdpasim_virtqueue *vq = &vdpasim->vqs[idx]; struct vdpasim_virtqueue *vq = &vdpasim->vqs[idx];
vringh_init_iotlb(&vq->vring, vdpasim_features, vringh_init_iotlb(&vq->vring, vdpasim->dev_attr.supported_features,
VDPASIM_QUEUE_MAX, false, VDPASIM_QUEUE_MAX, false,
(struct vring_desc *)(uintptr_t)vq->desc_addr, (struct vring_desc *)(uintptr_t)vq->desc_addr,
(struct vring_avail *) (struct vring_avail *)
...@@ -144,7 +146,8 @@ static void vdpasim_queue_ready(struct vdpasim *vdpasim, unsigned int idx) ...@@ -144,7 +146,8 @@ static void vdpasim_queue_ready(struct vdpasim *vdpasim, unsigned int idx)
(uintptr_t)vq->device_addr); (uintptr_t)vq->device_addr);
} }
static void vdpasim_vq_reset(struct vdpasim_virtqueue *vq) static void vdpasim_vq_reset(struct vdpasim *vdpasim,
struct vdpasim_virtqueue *vq)
{ {
vq->ready = false; vq->ready = false;
vq->desc_addr = 0; vq->desc_addr = 0;
...@@ -152,8 +155,8 @@ static void vdpasim_vq_reset(struct vdpasim_virtqueue *vq) ...@@ -152,8 +155,8 @@ static void vdpasim_vq_reset(struct vdpasim_virtqueue *vq)
vq->device_addr = 0; vq->device_addr = 0;
vq->cb = NULL; vq->cb = NULL;
vq->private = NULL; vq->private = NULL;
vringh_init_iotlb(&vq->vring, vdpasim_features, VDPASIM_QUEUE_MAX, vringh_init_iotlb(&vq->vring, vdpasim->dev_attr.supported_features,
false, NULL, NULL, NULL); VDPASIM_QUEUE_MAX, false, NULL, NULL, NULL);
} }
static void vdpasim_reset(struct vdpasim *vdpasim) static void vdpasim_reset(struct vdpasim *vdpasim)
...@@ -161,7 +164,7 @@ static void vdpasim_reset(struct vdpasim *vdpasim) ...@@ -161,7 +164,7 @@ static void vdpasim_reset(struct vdpasim *vdpasim)
int i; int i;
for (i = 0; i < vdpasim->dev_attr.nvqs; i++) for (i = 0; i < vdpasim->dev_attr.nvqs; i++)
vdpasim_vq_reset(&vdpasim->vqs[i]); vdpasim_vq_reset(vdpasim, &vdpasim->vqs[i]);
spin_lock(&vdpasim->iommu_lock); spin_lock(&vdpasim->iommu_lock);
vhost_iotlb_reset(vdpasim->iommu); vhost_iotlb_reset(vdpasim->iommu);
...@@ -528,7 +531,9 @@ static u32 vdpasim_get_vq_align(struct vdpa_device *vdpa) ...@@ -528,7 +531,9 @@ static u32 vdpasim_get_vq_align(struct vdpa_device *vdpa)
static u64 vdpasim_get_features(struct vdpa_device *vdpa) static u64 vdpasim_get_features(struct vdpa_device *vdpa)
{ {
return vdpasim_features; struct vdpasim *vdpasim = vdpa_to_sim(vdpa);
return vdpasim->dev_attr.supported_features;
} }
static int vdpasim_set_features(struct vdpa_device *vdpa, u64 features) static int vdpasim_set_features(struct vdpa_device *vdpa, u64 features)
...@@ -539,7 +544,7 @@ static int vdpasim_set_features(struct vdpa_device *vdpa, u64 features) ...@@ -539,7 +544,7 @@ static int vdpasim_set_features(struct vdpa_device *vdpa, u64 features)
if (!(features & (1ULL << VIRTIO_F_ACCESS_PLATFORM))) if (!(features & (1ULL << VIRTIO_F_ACCESS_PLATFORM)))
return -EINVAL; return -EINVAL;
vdpasim->features = features & vdpasim_features; vdpasim->features = features & vdpasim->dev_attr.supported_features;
return 0; return 0;
} }
...@@ -776,6 +781,7 @@ static int __init vdpasim_dev_init(void) ...@@ -776,6 +781,7 @@ static int __init vdpasim_dev_init(void)
struct vdpasim_dev_attr dev_attr = {}; struct vdpasim_dev_attr dev_attr = {};
dev_attr.id = VIRTIO_ID_NET; dev_attr.id = VIRTIO_ID_NET;
dev_attr.supported_features = VDPASIM_NET_FEATURES;
dev_attr.nvqs = VDPASIM_VQ_NUM; dev_attr.nvqs = VDPASIM_VQ_NUM;
dev_attr.work_fn = vdpasim_net_work; dev_attr.work_fn = vdpasim_net_work;
dev_attr.config_size = sizeof(struct virtio_net_config); dev_attr.config_size = sizeof(struct virtio_net_config);
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册