lkml.org 
[lkml]   [2019]   [Sep]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [RFC PATCH 2/2] mdev: introduce device specific ops
From
Date

On 2019/9/17 下午8:42, Cornelia Huck wrote:
> On Thu, 12 Sep 2019 17:40:12 +0800
> Jason Wang <jasowang@redhat.com> wrote:
>
>> Currently, except for the crate and remove. The rest fields of
>> mdev_parent_ops is just designed for vfio-mdev driver and may not help
>> for kernel mdev driver. So follow the device id support by previous
>> patch, this patch introduces device specific ops which points to
>> device specific ops (e.g vfio ops). This allows the future drivers
>> like virtio-mdev to implement its own device specific ops.
>>
>> Signed-off-by: Jason Wang <jasowang@redhat.com>
>> ---
>> drivers/gpu/drm/i915/gvt/kvmgt.c | 14 +++---
>> drivers/s390/cio/vfio_ccw_ops.c | 14 +++---
>> drivers/s390/crypto/vfio_ap_ops.c | 10 +++--
>> drivers/vfio/mdev/vfio_mdev.c | 30 +++++++------
>> include/linux/mdev.h | 72 ++++++++++++++++++-------------
>> samples/vfio-mdev/mbochs.c | 16 ++++---
>> samples/vfio-mdev/mdpy.c | 16 ++++---
>> samples/vfio-mdev/mtty.c | 14 +++---
>> 8 files changed, 113 insertions(+), 73 deletions(-)
>> diff --git a/include/linux/mdev.h b/include/linux/mdev.h
>> index f85045392120..3b8a76bc69cf 100644
>> --- a/include/linux/mdev.h
>> +++ b/include/linux/mdev.h
>> @@ -27,27 +27,9 @@ int mdev_set_iommu_device(struct device *dev, struct device *iommu_device);
>> struct device *mdev_get_iommu_device(struct device *dev);
>>
>> /**
>> - * struct mdev_parent_ops - Structure to be registered for each parent device to
>> - * register the device to mdev module.
>> + * struct vfio_mdev_parent_ops - Structure to be registered for each
>> + * parent device to register the device to vfio-mdev module.
>> *
>> - * @owner: The module owner.
>> - * @dev_attr_groups: Attributes of the parent device.
>> - * @mdev_attr_groups: Attributes of the mediated device.
>> - * @supported_type_groups: Attributes to define supported types. It is mandatory
>> - * to provide supported types.
>> - * @create: Called to allocate basic resources in parent device's
>> - * driver for a particular mediated device. It is
>> - * mandatory to provide create ops.
>> - * @kobj: kobject of type for which 'create' is called.
>> - * @mdev: mdev_device structure on of mediated device
>> - * that is being created
>> - * Returns integer: success (0) or error (< 0)
>> - * @remove: Called to free resources in parent device's driver for a
>> - * a mediated device. It is mandatory to provide 'remove'
>> - * ops.
>> - * @mdev: mdev_device device structure which is being
>> - * destroyed
>> - * Returns integer: success (0) or error (< 0)
>> * @open: Open mediated device.
>> * @mdev: mediated device.
>> * Returns integer: success (0) or error (< 0)
>> @@ -72,6 +54,43 @@ struct device *mdev_get_iommu_device(struct device *dev);
>> * @mmap: mmap callback
>> * @mdev: mediated device structure
>> * @vma: vma structure
>> + */
>> +struct vfio_mdev_parent_ops {
>> + int (*open)(struct mdev_device *mdev);
>> + void (*release)(struct mdev_device *mdev);
>> + ssize_t (*read)(struct mdev_device *mdev, char __user *buf,
>> + size_t count, loff_t *ppos);
>> + ssize_t (*write)(struct mdev_device *mdev, const char __user *buf,
>> + size_t count, loff_t *ppos);
>> + long (*ioctl)(struct mdev_device *mdev, unsigned int cmd,
>> + unsigned long arg);
>> + int (*mmap)(struct mdev_device *mdev, struct vm_area_struct *vma);
>> +};
>> +
>> +/**
>> + * struct mdev_parent_ops - Structure to be registered for each parent device to
>> + * register the device to mdev module.
>> + *
>> + * @owner: The module owner.
>> + * @dev_attr_groups: Attributes of the parent device.
>> + * @mdev_attr_groups: Attributes of the mediated device.
>> + * @supported_type_groups: Attributes to define supported types. It is mandatory
>> + * to provide supported types.
>> + * @create: Called to allocate basic resources in parent device's
>> + * driver for a particular mediated device. It is
>> + * mandatory to provide create ops.
>> + * @kobj: kobject of type for which 'create' is called.
>> + * @mdev: mdev_device structure on of mediated device
>> + * that is being created
>> + * Returns integer: success (0) or error (< 0)
>> + * @remove: Called to free resources in parent device's driver for a
>> + * a mediated device. It is mandatory to provide 'remove'
>> + * ops.
>> + * @mdev: mdev_device device structure which is being
>> + * destroyed
>> + * Returns integer: success (0) or error (< 0)
>> + * @device_ops: Device specific emulation callback.
>> + *
>> * Parent device that support mediated device should be registered with mdev
>> * module with mdev_parent_ops structure.
>> **/
>> @@ -83,15 +102,7 @@ struct mdev_parent_ops {
>>
>> int (*create)(struct kobject *kobj, struct mdev_device *mdev);
>> int (*remove)(struct mdev_device *mdev);
>> - int (*open)(struct mdev_device *mdev);
>> - void (*release)(struct mdev_device *mdev);
>> - ssize_t (*read)(struct mdev_device *mdev, char __user *buf,
>> - size_t count, loff_t *ppos);
>> - ssize_t (*write)(struct mdev_device *mdev, const char __user *buf,
>> - size_t count, loff_t *ppos);
>> - long (*ioctl)(struct mdev_device *mdev, unsigned int cmd,
>> - unsigned long arg);
>> - int (*mmap)(struct mdev_device *mdev, struct vm_area_struct *vma);
>> + const void *device_ops;
>> };
>>
>> /* interface for exporting mdev supported type attributes */
> This basically looks like a split between stuff that is always
> triggered from userspace (create and the like) and stuff that is
> triggered from userspace for vfio mdevs, but not necessarily for other
> mdevs.


Yes.


> Seems reasonable at a glance.
>
> If we decide to go forward with this, we should also update the
> documentation (split out stuff from driver-api/vfio-mediated-device.rst
> etc.)


Yes, I plan to do that.

Thanks

\
 
 \ /
  Last update: 2019-09-18 07:56    [W:0.635 / U:0.004 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site