The registration code first registers the mdev device, and then
proceeds to populate sysfs. An userspace application that listens
for the ADD uevent is therefore likely to look for sysfs entries
that have not yet been created.

The canonical way to fix this is to use attribute groups that are
registered by the driver core before it sends the ADD uevent; I
unfortunately did not find a way to make this work in this case,
though.

An alternative approach is to suppress uevents before we register
with the core and generate the ADD uevent ourselves after the
sysfs infrastructure is in place.

Signed-off-by: Cornelia Huck <coh...@redhat.com>
---

This feels like a band-aid, but I can't figure out how to handle creating
attribute groups when there's a callback in the parent involved.

This should address the issue with libvirt's processing of mdevs raised in
https://www.redhat.com/archives/libvir-list/2018-February/msg00023.html
- although libvirt will still need to deal with older kernels, of course.

Best to consider this an untested patch :)

---
 drivers/vfio/mdev/mdev_core.c | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/drivers/vfio/mdev/mdev_core.c b/drivers/vfio/mdev/mdev_core.c
index 126991046eb7..942e880d8e7f 100644
--- a/drivers/vfio/mdev/mdev_core.c
+++ b/drivers/vfio/mdev/mdev_core.c
@@ -335,6 +335,8 @@ int mdev_device_create(struct kobject *kobj, struct device 
*dev, uuid_le uuid)
        mdev->dev.release = mdev_device_release;
        dev_set_name(&mdev->dev, "%pUl", uuid.b);
 
+       /* don't notify userspace until we're ready */
+       dev_set_uevent_suppress(&mdev->dev, 1);
        ret = device_register(&mdev->dev);
        if (ret) {
                put_device(&mdev->dev);
@@ -350,6 +352,9 @@ int mdev_device_create(struct kobject *kobj, struct device 
*dev, uuid_le uuid)
                mdev_device_remove_ops(mdev, true);
                goto create_failed;
        }
+       /* all done, notify userspace */
+       dev_set_uevent_suppress(&mdev->dev, 0);
+       kobject_uevent(&mdev->dev.kobj, KOBJ_ADD);
 
        mdev->type_kobj = kobj;
        dev_dbg(&mdev->dev, "MDEV: created\n");
@@ -363,6 +368,10 @@ int mdev_device_create(struct kobject *kobj, struct device 
*dev, uuid_le uuid)
        return ret;
 
 create_failed:
+       /*
+        * If we reach this, uevents are still suppressed for mdev->dev,
+        * so we don't get a KOBJ_DEL uevent without a previous KOBJ_ADD.
+        */
        device_unregister(&mdev->dev);
 
 create_err:
-- 
2.13.6

Reply via email to