On Thu, Jun 02, 2022 at 07:19:40PM +0200, Eric Farman wrote: > @@ -278,7 +269,12 @@ static void vfio_ccw_sch_remove(struct subchannel *sch) > > static void vfio_ccw_sch_shutdown(struct subchannel *sch) > { > - vfio_ccw_sch_quiesce(sch); > + struct vfio_ccw_private *private = dev_get_drvdata(&sch->dev); > + > + if (!private) > + return; WARN_ON here too Looks Ok otherwise Reviewed-by: Jason Gunthorpe <jgg@xxxxxxxxxx> Jason