Commit 03264ddd authored by Adam Williamson's avatar Adam Williamson Committed by Martin K. Petersen

scsi: compat_ioctl: cdrom: Replace .ioctl with .compat_ioctl in four appropriate places

Arnd Bergmann inadvertently typoed these in d320a955 and 64cbfa96;
they seem to be the cause of
https://bugzilla.redhat.com/show_bug.cgi?id=1801353 , invalid SCSI commands
when udev tries to query a DVD drive.

[arnd] Found another instance of the same bug, also introduced in my
compat_ioctl series.

Link: https://bugzilla.redhat.com/show_bug.cgi?id=1801353
Link: https://lore.kernel.org/r/20200219165139.3467320-1-arnd@arndb.de
Fixes: c103d6ee ("compat_ioctl: ide: floppy: add handler")
Fixes: 64cbfa96 ("compat_ioctl: move cdrom commands into cdrom.c")
Fixes: d320a955 ("compat_ioctl: scsi: move ioctl handling into drivers")
Bisected-by: default avatarChris Murphy <bugzilla@colorremedies.com>
Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
Signed-off-by: default avatarAdam Williamson <awilliam@redhat.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent a3fd4bfe
...@@ -276,7 +276,7 @@ static const struct block_device_operations pcd_bdops = { ...@@ -276,7 +276,7 @@ static const struct block_device_operations pcd_bdops = {
.release = pcd_block_release, .release = pcd_block_release,
.ioctl = pcd_block_ioctl, .ioctl = pcd_block_ioctl,
#ifdef CONFIG_COMPAT #ifdef CONFIG_COMPAT
.ioctl = blkdev_compat_ptr_ioctl, .compat_ioctl = blkdev_compat_ptr_ioctl,
#endif #endif
.check_events = pcd_block_check_events, .check_events = pcd_block_check_events,
}; };
......
...@@ -519,7 +519,7 @@ static const struct block_device_operations gdrom_bdops = { ...@@ -519,7 +519,7 @@ static const struct block_device_operations gdrom_bdops = {
.check_events = gdrom_bdops_check_events, .check_events = gdrom_bdops_check_events,
.ioctl = gdrom_bdops_ioctl, .ioctl = gdrom_bdops_ioctl,
#ifdef CONFIG_COMPAT #ifdef CONFIG_COMPAT
.ioctl = blkdev_compat_ptr_ioctl, .compat_ioctl = blkdev_compat_ptr_ioctl,
#endif #endif
}; };
......
...@@ -361,7 +361,7 @@ static const struct block_device_operations ide_gd_ops = { ...@@ -361,7 +361,7 @@ static const struct block_device_operations ide_gd_ops = {
.release = ide_gd_release, .release = ide_gd_release,
.ioctl = ide_gd_ioctl, .ioctl = ide_gd_ioctl,
#ifdef CONFIG_COMPAT #ifdef CONFIG_COMPAT
.ioctl = ide_gd_compat_ioctl, .compat_ioctl = ide_gd_compat_ioctl,
#endif #endif
.getgeo = ide_gd_getgeo, .getgeo = ide_gd_getgeo,
.check_events = ide_gd_check_events, .check_events = ide_gd_check_events,
......
...@@ -688,7 +688,7 @@ static const struct block_device_operations sr_bdops = ...@@ -688,7 +688,7 @@ static const struct block_device_operations sr_bdops =
.release = sr_block_release, .release = sr_block_release,
.ioctl = sr_block_ioctl, .ioctl = sr_block_ioctl,
#ifdef CONFIG_COMPAT #ifdef CONFIG_COMPAT
.ioctl = sr_block_compat_ioctl, .compat_ioctl = sr_block_compat_ioctl,
#endif #endif
.check_events = sr_block_check_events, .check_events = sr_block_check_events,
.revalidate_disk = sr_block_revalidate_disk, .revalidate_disk = sr_block_revalidate_disk,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment