> From: Shameer Kolothum <shameerali.kolothum.thodi@xxxxxxxxxx> > Sent: Thursday, February 29, 2024 5:12 PM > > The deferred_reset logic was added to vfio migration drivers to prevent > a circular locking dependency with respect to mm_lock and state mutex. > This is mainly because of the copy_to/from_user() functions(which takes > mm_lock) invoked under state mutex. But for HiSilicon driver, the only > place where we now hold the state mutex for copy_to_user is during the > PRE_COPY IOCTL. So for pre_copy, release the lock as soon as we have > updated the data and perform copy_to_user without state mutex. By this, > we can get rid of the deferred_reset logic. > > Link: https://lore.kernel.org/kvm/20240220132459.GM13330@xxxxxxxxxx/ > Signed-off-by: Shameer Kolothum <shameerali.kolothum.thodi@xxxxxxxxxx> Reviewed-by: Kevin Tian <kevin.tian@xxxxxxxxx>
