On Thu, 14 Nov 2013, Bodo Stroesser wrote: ... > From: Bodo Stroesser <bstroes...@ts.fujitsu.com> > Date: Thu, 14 Nov 2013 14:35:00 +0100 > Subject: [PATCH] sg: fix blk_get_queue usage > > If blk_queue_get() in st_probe fails, disk->queue must not > be set to SDp->request_queue, as that would result in > put_disk() dropping a not taken reference. > > Thus, disk->queue should be set only after a successful > blk_queue_get(). > Revised patch due to a hint from Kai Makisara. > > Signed-off-by: Bodo Stroesser <bstroes...@ts.fujitsu.com>
Acked-by: Kai Mäkisara <kai.makis...@kolumbus.fi> > > --- > > --- a/drivers/scsi/st.c 2013-11-14 14:10:40.000000000 +0100 > +++ b/drivers/scsi/st.c 2013-11-14 14:10:57.000000000 +0100 > @@ -4107,11 +4107,11 @@ > kref_init(&tpnt->kref); > tpnt->disk = disk; > disk->private_data = &tpnt->driver; > - disk->queue = SDp->request_queue; > /* SCSI tape doesn't register this gendisk via add_disk(). Manually > * take queue reference that release_disk() expects. */ > - if (blk_get_queue(disk->queue)) > + if (blk_get_queue(SDp->request_queue)) > goto out_put_disk; > + disk->queue = SDp->request_queue; > tpnt->driver = &st_template; > > tpnt->device = SDp; Thanks for the fix, Kai
