lkml.org 
[lkml]   [2019]   [Aug]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 1/4] mdev: Introduce sha1 based mdev alias
    On Mon, 26 Aug 2019 15:41:16 -0500
    Parav Pandit <parav@mellanox.com> wrote:

    > Whenever a parent requests to generate mdev alias, generate a mdev
    > alias.
    > It is an optional attribute that parent can request to generate
    > for each of its child mdev.
    > mdev alias is generated using sha1 from the mdev name.

    Maybe add some motivation here as well?

    "Some vendor drivers want an identifier for an mdev device that is
    shorter than the uuid, due to length restrictions in the consumers of
    that identifier.

    Add a callback that allows a vendor driver to request an alias of a
    specified length to be generated (via sha1) for an mdev device. If
    generated, that alias is checked for collisions."

    >
    > Signed-off-by: Parav Pandit <parav@mellanox.com>
    > ---
    > drivers/vfio/mdev/mdev_core.c | 98 +++++++++++++++++++++++++++++++-
    > drivers/vfio/mdev/mdev_private.h | 5 +-
    > drivers/vfio/mdev/mdev_sysfs.c | 13 +++--
    > include/linux/mdev.h | 4 ++
    > 4 files changed, 111 insertions(+), 9 deletions(-)
    >

    (...)

    > @@ -406,6 +495,10 @@ EXPORT_SYMBOL(mdev_get_iommu_device);
    >
    > static int __init mdev_init(void)
    > {
    > + alias_hash = crypto_alloc_shash("sha1", 0, 0);
    > + if (!alias_hash)
    > + return -ENOMEM;
    > +
    > return mdev_bus_register();

    Don't you need to call crypto_free_shash() if mdev_bus_register() fails?

    > }
    >
    > @@ -415,6 +508,7 @@ static void __exit mdev_exit(void)
    > class_compat_unregister(mdev_bus_compat_class);
    >
    > mdev_bus_unregister();
    > + crypto_free_shash(alias_hash);
    > }
    >
    > module_init(mdev_init)

    (...)

    > diff --git a/include/linux/mdev.h b/include/linux/mdev.h
    > index 0ce30ca78db0..f036fe9854ee 100644
    > --- a/include/linux/mdev.h
    > +++ b/include/linux/mdev.h
    > @@ -72,6 +72,9 @@ struct device *mdev_get_iommu_device(struct device *dev);
    > * @mmap: mmap callback
    > * @mdev: mediated device structure
    > * @vma: vma structure
    > + * @get_alias_length: Generate alias for the mdevs of this parent based on the
    > + * mdev device name when it returns non zero alias length.
    > + * It is optional.

    What about:

    * @get_alias_length: optional callback to specify length of the alias to create
    * Returns unsigned integer: length of the alias to be created,
    * 0 to not create an alias

    I also think it might be beneficial to add a device parameter here now
    (rather than later); that seems to be something that makes sense.

    > * Parent device that support mediated device should be registered with mdev
    > * module with mdev_parent_ops structure.
    > **/
    > @@ -92,6 +95,7 @@ struct mdev_parent_ops {
    > long (*ioctl)(struct mdev_device *mdev, unsigned int cmd,
    > unsigned long arg);
    > int (*mmap)(struct mdev_device *mdev, struct vm_area_struct *vma);
    > + unsigned int (*get_alias_length)(void);
    > };
    >
    > /* interface for exporting mdev supported type attributes */

    \
     
     \ /
      Last update: 2019-08-27 12:25    [W:4.158 / U:0.092 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site