scsi/sg: don't grab scsi host module reference

In order to prevent request_queue to be freed before cleaning up
blktrace debugfs entries, commit db59133e9279 ("scsi: sg: fix blktrace
debugfs entries leakage") use scsi_device_get(), however,
scsi_device_get() will also grab scsi module reference and scsi module
can't be removed.

It's reported that blktests can't unload scsi_debug after block/001:

blktests (master) # ./check block
block/001 (stress device hotplugging) [failed]
+++ /root/blktests/results/nodev/block/001.out.bad 2023-06-19
Running block/001
Stressing sd
+modprobe: FATAL: Module scsi_debug is in use.

Fix this problem by grabbing request_queue reference directly, so that
scsi host module can still be unloaded while request_queue will be
pinged by sg device.

Reported-by: Chaitanya Kulkarni <chaitanyak@nvidia.com>
Link: https://lore.kernel.org/all/1760da91-876d-fc9c-ab51-999a6f66ad50@nvidia.com/
Fixes: db59133e9279 ("scsi: sg: fix blktrace debugfs entries leakage")
Signed-off-by: Yu Kuai <yukuai3@huawei.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Link: https://lore.kernel.org/r/20230621160111.1433521-1-yukuai1@huaweicloud.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>

authored by Yu Kuai and committed by Jens Axboe fcaa174a a42fb5a7

+3 -3
+3 -3
drivers/scsi/sg.c
··· 1497 1497 int error; 1498 1498 unsigned long iflags; 1499 1499 1500 - error = scsi_device_get(scsidp); 1500 + error = blk_get_queue(scsidp->request_queue); 1501 1501 if (error) 1502 1502 return error; 1503 1503 ··· 1558 1558 out: 1559 1559 if (cdev) 1560 1560 cdev_del(cdev); 1561 - scsi_device_put(scsidp); 1561 + blk_put_queue(scsidp->request_queue); 1562 1562 return error; 1563 1563 } 1564 1564 ··· 1575 1575 */ 1576 1576 1577 1577 blk_trace_remove(q); 1578 - scsi_device_put(sdp->device); 1578 + blk_put_queue(q); 1579 1579 1580 1580 write_lock_irqsave(&sg_index_lock, flags); 1581 1581 idr_remove(&sg_index_idr, sdp->index);