summaryrefslogtreecommitdiffstats
path: root/drivers/ata/ahci_imx.c
diff options
context:
space:
mode:
authorNiklas Cassel <cassel@kernel.org>2024-06-29 14:42:14 +0200
committerNiklas Cassel <cassel@kernel.org>2024-06-30 22:23:39 +0200
commiteeb25a09c5e0805d92e4ebd12c4b0ad0df1b0295 (patch)
treea4b79f48deebf97ec082c40a97f40e5ab5177903 /drivers/ata/ahci_imx.c
parentab9e0c529eb7cafebdd31fe1644524e80a48b05d (diff)
downloadlinux-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