diff options
author | Parav Pandit | 2019-04-30 17:49:33 -0500 |
---|---|---|
committer | Alex Williamson | 2019-05-07 11:23:13 -0600 |
commit | 6093e348a5e2475c5bb2e571346460f939998670 (patch) | |
tree | f170540e8a72a08261f0f18a359adeb8880b9382 /crypto/internal.h | |
parent | a6d6f4f160f76d840e59affe664b8d3159e23056 (diff) |
vfio/mdev: Fix aborting mdev child device removal if one fails
device_for_each_child() stops executing callback function for remaining
child devices, if callback hits an error.
Each child mdev device is independent of each other.
While unregistering parent device, mdev core must remove all child mdev
devices.
Therefore, mdev_device_remove_cb() always returns success so that
device_for_each_child doesn't abort if one child removal hits error.
While at it, improve remove and unregister functions for below simplicity.
There isn't need to pass forced flag pointer during mdev parent
removal which invokes mdev_device_remove(). So simplify the flow.
mdev_device_remove() is called from two paths.
1. mdev_unregister_driver()
mdev_device_remove_cb()
mdev_device_remove()
2. remove_store()
mdev_device_remove()
Fixes: 7b96953bc640 ("vfio: Mediated device Core driver")
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 'crypto/internal.h')
0 files changed, 0 insertions, 0 deletions