On Tue, Apr 4, 2017 at 5:20 PM, Burakov, Anatoly <anatoly.bura...@intel.com> wrote:
> Hi Andrew, > > > Please, take a look at http://dpdk.org/dev/patchwork/patch/23202/ > > I took a quick look. It should fix the problem (closing the group will > cause container detachment, so we can always assume that if there's no > container associated with the group, there are no devices within that > group). The issue of vfio_release_device() still remains. I think the fix > for that would closely follow what you did here: close the dev_fd, check if > the group still has a container associated with it, and if not, close the > group_fd and clear the group. > > That could work. When implementing the hotplug I realize the kernel could give more info about a particular group status. Maybe I propose some changes for facilitating this, even if we can fix the issue just checking if there is a container there. > I cannot thoroughly test it right this moment, however I'll have time > later during the week. > > Thanks, > Anatoly >