> From: Nicolin Chen <nicolinc@xxxxxxxxxx> > Sent: Saturday, March 11, 2023 7:38 AM > > On Fri, Mar 10, 2023 at 11:53:56AM +0000, Tian, Kevin wrote: > > External email: Use caution opening links or attachments > > > > > > > From: Nicolin Chen <nicolinc@xxxxxxxxxx> > > > Sent: Wednesday, March 8, 2023 10:26 PM > > > > > > Now both the physical path and the emulated path should support an IO > > > page > > > table replacement. > > > > > > Call iommufd_device_replace() when vdev->iommufd_attached is true. > > > > > > > why is replace enabled only in physical path in this patch? > > The emulated pathway does not call iommufd_device_attach() but > iommufd_access_set_ioas() in the other patch, which internally > takes care of the replacement for the access pointer. > I thought there is a similar check as in physical path which should be removed: if (vdev->iommufd_attached) - return -EBUSY; but looks it's not the case for the emulated path.
