> From: Jason Gunthorpe <jgg@xxxxxxxxxx>
> Sent: Thursday, March 30, 2023 7:52 PM
>
> On Thu, Mar 30, 2023 at 07:09:31AM +0000, Liu, Yi L wrote:
>
> > > > +long vfio_device_ioctl_bind_iommufd(struct vfio_device_file *df,
> > > > + struct vfio_device_bind_iommufd __user *arg)
> > > > +{
> > > > + struct vfio_device *device = df->device;
> > > > + struct vfio_device_bind_iommufd bind;
> > > > + struct iommufd_ctx *iommufd = NULL;
> > > > + unsigned long minsz;
> > > > + int ret;
> > > > +
> > > > + static_assert(__same_type(arg->out_devid, bind.out_devid));
> > >
> > > They're the same field in the same structure, how could they be
> > > otherwise?
> >
> > @Jason, should I remove this check?
>
> Yes, it was from something that looked very different from this
ok, I'll remove it here and next patch. 😊
