On Wed, 28 Aug 2019 15:25:44 -0600 Alex Williamson <alex.williamson@xxxxxxxxxx> wrote: > On Tue, 27 Aug 2019 14:16:50 -0500 > Parav Pandit <parav@xxxxxxxxxxxx> wrote: > > module_init(mdev_init) > > diff --git a/drivers/vfio/mdev/mdev_private.h b/drivers/vfio/mdev/mdev_private.h > > index 7d922950caaf..cf1c0d9842c6 100644 > > --- a/drivers/vfio/mdev/mdev_private.h > > +++ b/drivers/vfio/mdev/mdev_private.h > > @@ -33,6 +33,7 @@ struct mdev_device { > > struct kobject *type_kobj; > > struct device *iommu_device; > > bool active; > > + const char *alias; Nit, put this above active to avoid creating a hole in the structure. Thanks, Alex