diff options
author | Parav Pandit <parav@mellanox.com> | 2019-05-01 00:49:32 +0200 |
---|---|---|
committer | Alex Williamson <alex.williamson@redhat.com> | 2019-05-07 19:23:13 +0200 |
commit | a6d6f4f160f76d840e59affe664b8d3159e23056 (patch) | |
tree | 4f78d5852a56fd70d40b96d53fd78968d4c91446 /drivers/vfio | |
parent | vfio/mdev: Avoid masking error code to EBUSY (diff) | |
download | linux-a6d6f4f160f76d840e59affe664b8d3159e23056.tar.xz linux-a6d6f4f160f76d840e59affe664b8d3159e23056.zip |
vfio/mdev: Follow correct remove sequence
mdev_remove_sysfs_files() should follow exact mirror sequence of a
create, similar to what is followed in error unwinding path of
mdev_create_sysfs_files().
Fixes: 6a62c1dfb5c7 ("vfio/mdev: Re-order sysfs attribute creation")
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
Reviewed-by: Maxim Levitsky <mlevitsk@redhat.com>
Signed-off-by: Parav Pandit <parav@mellanox.com>
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Diffstat (limited to 'drivers/vfio')
-rw-r--r-- | drivers/vfio/mdev/mdev_sysfs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/vfio/mdev/mdev_sysfs.c b/drivers/vfio/mdev/mdev_sysfs.c index 5193a0e0ce5a..cbf94b8165ea 100644 --- a/drivers/vfio/mdev/mdev_sysfs.c +++ b/drivers/vfio/mdev/mdev_sysfs.c @@ -280,7 +280,7 @@ type_link_failed: void mdev_remove_sysfs_files(struct device *dev, struct mdev_type *type) { + sysfs_remove_files(&dev->kobj, mdev_device_attrs); sysfs_remove_link(&dev->kobj, "mdev_type"); sysfs_remove_link(type->devices_kobj, dev_name(dev)); - sysfs_remove_files(&dev->kobj, mdev_device_attrs); } |