linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* stop using ioctl_by_bdev for file system access to CDROMs v2
@ 2020-04-25  7:56 Christoph Hellwig
  2020-04-25  7:57 ` [PATCH 1/7] block: add a cdrom_device_info pointer to struct gendisk Christoph Hellwig
                   ` (8 more replies)
  0 siblings, 9 replies; 26+ messages in thread
From: Christoph Hellwig @ 2020-04-25  7:56 UTC (permalink / raw)
  To: Jens Axboe
  Cc: Tim Waugh, Borislav Petkov, Jan Kara, linux-block, linux-ide,
	linux-scsi, linux-fsdevel, linux-kernel

Hi Jens,

except for the DASD case under discussion the last users of ioctl_by_bdev
are the file system drivers that want to query CDROM information using
ioctls.  This series switches them to use function calls directly into
the CDROM midlayer instead, which implies:

 - adding a cdrom_device_info pointer to the gendisk, so that file systems
   can find it without going to the low-level driver first
 - ensuring that the CDROM midlayer (which isn't a lot of code) is built
   in if the file systems are built in so that they can actually call the
   exported functions

Changes since v1:
 - fix up the no-CDROM error case in isofs_get_last_session to return 0
   instead of -EINVAL.

^ permalink raw reply	[flat|nested] 26+ messages in thread
* stop using ioctl_by_bdev for file system access to CDROMs
@ 2020-04-23  7:12 Christoph Hellwig
  2020-04-23  7:12 ` [PATCH 6/7] isofs: stop using ioctl_by_bdev Christoph Hellwig
  0 siblings, 1 reply; 26+ messages in thread
From: Christoph Hellwig @ 2020-04-23  7:12 UTC (permalink / raw)
  To: Jens Axboe
  Cc: Tim Waugh, Borislav Petkov, Jan Kara, linux-block, linux-ide,
	linux-scsi, linux-fsdevel, linux-kernel

Hi Jens,

except for the DASD case under discussion the last users of ioctl_by_bdev
are the file system drivers that want to query CDROM information using
ioctls.  This series switches them to use function calls directly into
the CDROM midlayer instead, which implies:

 - adding a cdrom_device_info pointer to the gendisk, so that file systems
   can find it without going to the low-level driver first
 - ensuring that the CDROM midlayer (which isn't a lot of code) is built
   in if the file systems are built in so that they can actually call the
   exported functions

^ permalink raw reply	[flat|nested] 26+ messages in thread

end of thread, other threads:[~2020-05-04 16:42 UTC | newest]

Thread overview: 26+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-25  7:56 stop using ioctl_by_bdev for file system access to CDROMs v2 Christoph Hellwig
2020-04-25  7:57 ` [PATCH 1/7] block: add a cdrom_device_info pointer to struct gendisk Christoph Hellwig
2020-04-27  6:15   ` Hannes Reinecke
2020-04-25  7:57 ` [PATCH 2/7] ide-cd: rename cdrom_read_tocentry Christoph Hellwig
2020-04-27  6:16   ` Hannes Reinecke
2020-04-25  7:57 ` [PATCH 3/7] cdrom: factor out a cdrom_read_tocentry helper Christoph Hellwig
2020-04-27  6:17   ` Hannes Reinecke
2020-04-25  7:57 ` [PATCH 4/7] cdrom: factor out a cdrom_multisession helper Christoph Hellwig
2020-04-27  6:17   ` Hannes Reinecke
2020-04-25  7:57 ` [PATCH 5/7] hfsplus: stop using ioctl_by_bdev Christoph Hellwig
2020-04-27  6:18   ` Hannes Reinecke
2020-05-04 16:16   ` Jens Axboe
2020-05-04 16:21     ` Christoph Hellwig
2020-05-04 16:41       ` Jens Axboe
2020-04-25  7:57 ` [PATCH 6/7] isofs: " Christoph Hellwig
2020-04-27  6:18   ` Hannes Reinecke
2020-04-27  9:50   ` Jan Kara
2020-04-25  7:57 ` [PATCH 7/7] udf: " Christoph Hellwig
2020-04-27  6:18   ` Hannes Reinecke
2020-04-28  6:53 ` stop using ioctl_by_bdev for file system access to CDROMs v2 Christoph Hellwig
2020-05-04 16:42 ` Jens Axboe
  -- strict thread matches above, loose matches on Subject: below --
2020-04-23  7:12 stop using ioctl_by_bdev for file system access to CDROMs Christoph Hellwig
2020-04-23  7:12 ` [PATCH 6/7] isofs: stop using ioctl_by_bdev Christoph Hellwig
2020-04-23  7:42   ` Damien Le Moal
2020-04-23 11:03   ` Jan Kara
2020-04-24  6:52     ` Christoph Hellwig
2020-04-24  9:21       ` Jan Kara

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).