On Fri, Jul 23, 2021 at 09:22:27AM -0300, Jason Gunthorpe wrote: > > But do we even need the else part? Assingning &mc_dev->dev is > > equivalent to the default per-device set anyway, isn't it? > > Not quite, the default is this: > > if (!device->dev_set) > vfio_assign_device_set(device, device); > > Where 'device' is the vfio_device itself, the above is connecting to > the struct fsl_mc_device. Isn't there a 1:1 relation?