> +void vfio_device_unmap_mapping_range(struct vfio_device *device, > + loff_t start, loff_t len) > +{ > + unmap_mapping_range(device->inode->i_mapping, start, len, true); > +} > +EXPORT_SYMBOL_GPL(vfio_device_unmap_mapping_range); Instead of mirroring the name of unmap_mapping_range maybe give this a name to document the use case? > +extern void vfio_device_unmap_mapping_range(struct vfio_device *device, > + loff_t start, loff_t len); No need for the extern.