diff options
author | Niklas Cassel <cassel@kernel.org> | 2024-06-29 14:42:14 +0200 |
---|---|---|
committer | Niklas Cassel <cassel@kernel.org> | 2024-06-30 22:23:39 +0200 |
commit | eeb25a09c5e0805d92e4ebd12c4b0ad0df1b0295 (patch) | |
tree | a4b79f48deebf97ec082c40a97f40e5ab5177903 /drivers/ata/ahci_imx.c | |
parent | ab9e0c529eb7cafebdd31fe1644524e80a48b05d (diff) | |
download | linux-eeb25a09c5e0805d92e4ebd12c4b0ad0df1b0295.tar.gz linux-eeb25a09c5e0805d92e4ebd12c4b0ad0df1b0295.tar.bz2 linux-eeb25a09c5e0805d92e4ebd12c4b0ad0df1b0295.zip |
ata: ahci: Clean up sysfs file on error
.probe() (ahci_init_one()) calls sysfs_add_file_to_group(), however,
if probe() fails after this call, we currently never call
sysfs_remove_file_from_group().
(The sysfs_remove_file_from_group() call in .remove() (ahci_remove_one())
does not help, as .remove() is not called on .probe() error.)
Thus, if probe() fails after the sysfs_add_file_to_group() call, the next
time we insmod the module we will get:
sysfs: cannot create duplicate filename '/devices/pci0000:00/0000:00:04.0/remapped_nvme'
CPU: 11 PID: 954 Comm: modprobe Not tainted 6.10.0-rc5 #43
Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 1.16.3-2.fc40 04/01/2014
Call Trace:
<TASK>
dump_stack_lvl+0x5d/0x80
sysfs_warn_dup.cold+0x17/0x23
sysfs_add_file_mode_ns+0x11a/0x130
sysfs_add_file_to_group+0x7e/0xc0
ahci_init_one+0x31f/0xd40 [ahci]
Fixes: 894fba7f434a ("ata: ahci: Add sysfs attribute to show remapped NVMe device count")
Cc: stable@vger.kernel.org
Reviewed-by: Damien Le Moal <dlemoal@kernel.org>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Link: https://lore.kernel.org/r/20240629124210.181537-10-cassel@kernel.org
Signed-off-by: Niklas Cassel <cassel@kernel.org>
Diffstat (limited to 'drivers/ata/ahci_imx.c')
0 files changed, 0 insertions, 0 deletions