mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-10-01 06:33:07 +00:00
scsi: core: sysfs: Remove comments that conflict with the actual logic
Christoph Hellwig Says:
=======================
I think we should just handle the error properly and remove the comment.
There's no good reason to ignore bsg registration errors.
In fact, after commit 92c4b58b15
("scsi: core: Register sysfs attributes
earlier"), we are already forced to return errno.
We discuss this issue in [1].
[1] https://lore.kernel.org/all/20211022010201.426746-1-liu.yun@linux.dev/
Link: https://lore.kernel.org/r/20220329021251.123805-1-liu.yun@linux.dev
Suggested-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jackie Liu <liuyun01@kylinos.cn>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
This commit is contained in:
parent
066f4c3194
commit
99241e119f
1 changed files with 0 additions and 4 deletions
|
@ -1384,10 +1384,6 @@ int scsi_sysfs_add_sdev(struct scsi_device *sdev)
|
||||||
if (IS_ENABLED(CONFIG_BLK_DEV_BSG)) {
|
if (IS_ENABLED(CONFIG_BLK_DEV_BSG)) {
|
||||||
sdev->bsg_dev = scsi_bsg_register_queue(sdev);
|
sdev->bsg_dev = scsi_bsg_register_queue(sdev);
|
||||||
if (IS_ERR(sdev->bsg_dev)) {
|
if (IS_ERR(sdev->bsg_dev)) {
|
||||||
/*
|
|
||||||
* We're treating error on bsg register as non-fatal, so
|
|
||||||
* pretend nothing went wrong.
|
|
||||||
*/
|
|
||||||
error = PTR_ERR(sdev->bsg_dev);
|
error = PTR_ERR(sdev->bsg_dev);
|
||||||
sdev_printk(KERN_INFO, sdev,
|
sdev_printk(KERN_INFO, sdev,
|
||||||
"Failed to register bsg queue, errno=%d\n",
|
"Failed to register bsg queue, errno=%d\n",
|
||||||
|
|
Loading…
Reference in a new issue