On 14/03/18 17:42, Punit Agrawal wrote: >> + region->vdev = vdev; >> + region->is_ioport = !!(bar & PCI_BASE_ADDRESS_SPACE_IO); >> region->info = (struct vfio_region_info) { >> .argsz = sizeof(*region), > > sizeof(region->info)? Woops, good catch Thanks, Jean