vdpa_sim: make 'config' generic and usable for any device type
commit f37cbbc65178e0a45823d281d290c4c02da9631c upstream. Add new 'config_size' attribute in 'vdpasim_dev_attr' and allocates 'config' dynamically to support any device types. Acked-by: Jason Wang <jasowang@redhat.com> Signed-off-by: Stefano Garzarella <sgarzare@redhat.com> Link: https://lore.kernel.org/r/20201215144256.155342-12-sgarzare@redhat.com Signed-off-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Stefano Garzarella <sgarzare@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
0fb67eda6a
commit
8faf3ea122
@ -70,6 +70,7 @@ static u64 vdpasim_features = (1ULL << VIRTIO_F_ANY_LAYOUT) |
|
||||
(1ULL << VIRTIO_NET_F_MAC);
|
||||
|
||||
struct vdpasim_dev_attr {
|
||||
size_t config_size;
|
||||
int nvqs;
|
||||
};
|
||||
|
||||
@ -81,7 +82,8 @@ struct vdpasim {
|
||||
struct vdpasim_dev_attr dev_attr;
|
||||
/* spinlock to synchronize virtqueue state */
|
||||
spinlock_t lock;
|
||||
struct virtio_net_config config;
|
||||
/* virtio config according to device type */
|
||||
void *config;
|
||||
struct vhost_iotlb *iommu;
|
||||
void *buffer;
|
||||
u32 status;
|
||||
@ -380,6 +382,10 @@ static struct vdpasim *vdpasim_create(struct vdpasim_dev_attr *dev_attr)
|
||||
goto err_iommu;
|
||||
set_dma_ops(dev, &vdpasim_dma_ops);
|
||||
|
||||
vdpasim->config = kzalloc(dev_attr->config_size, GFP_KERNEL);
|
||||
if (!vdpasim->config)
|
||||
goto err_iommu;
|
||||
|
||||
vdpasim->vqs = kcalloc(dev_attr->nvqs, sizeof(struct vdpasim_virtqueue),
|
||||
GFP_KERNEL);
|
||||
if (!vdpasim->vqs)
|
||||
@ -518,7 +524,8 @@ static u64 vdpasim_get_features(struct vdpa_device *vdpa)
|
||||
static int vdpasim_set_features(struct vdpa_device *vdpa, u64 features)
|
||||
{
|
||||
struct vdpasim *vdpasim = vdpa_to_sim(vdpa);
|
||||
struct virtio_net_config *config = &vdpasim->config;
|
||||
struct virtio_net_config *config =
|
||||
(struct virtio_net_config *)vdpasim->config;
|
||||
|
||||
/* DMA mapping must be done by driver */
|
||||
if (!(features & (1ULL << VIRTIO_F_ACCESS_PLATFORM)))
|
||||
@ -588,8 +595,8 @@ static void vdpasim_get_config(struct vdpa_device *vdpa, unsigned int offset,
|
||||
{
|
||||
struct vdpasim *vdpasim = vdpa_to_sim(vdpa);
|
||||
|
||||
if (offset + len < sizeof(struct virtio_net_config))
|
||||
memcpy(buf, (u8 *)&vdpasim->config + offset, len);
|
||||
if (offset + len < vdpasim->dev_attr.config_size)
|
||||
memcpy(buf, vdpasim->config + offset, len);
|
||||
}
|
||||
|
||||
static void vdpasim_set_config(struct vdpa_device *vdpa, unsigned int offset,
|
||||
@ -676,6 +683,7 @@ static void vdpasim_free(struct vdpa_device *vdpa)
|
||||
if (vdpasim->iommu)
|
||||
vhost_iotlb_free(vdpasim->iommu);
|
||||
kfree(vdpasim->vqs);
|
||||
kfree(vdpasim->config);
|
||||
}
|
||||
|
||||
static const struct vdpa_config_ops vdpasim_net_config_ops = {
|
||||
@ -736,6 +744,7 @@ static int __init vdpasim_dev_init(void)
|
||||
struct vdpasim_dev_attr dev_attr = {};
|
||||
|
||||
dev_attr.nvqs = VDPASIM_VQ_NUM;
|
||||
dev_attr.config_size = sizeof(struct virtio_net_config);
|
||||
|
||||
vdpasim_dev = vdpasim_create(&dev_attr);
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user