Commit 430cc5d3 authored by Christoph Hellwig's avatar Christoph Hellwig Committed by Jens Axboe

block: cleanup the GENHD_FL_* definitions

Switch to an enum and tidy up the documentation.
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
Link: https://lore.kernel.org/r/20211122130625.1136848-14-hch@lst.deSigned-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent 9f18db57
...@@ -39,28 +39,24 @@ struct partition_meta_info { ...@@ -39,28 +39,24 @@ struct partition_meta_info {
/** /**
* DOC: genhd capability flags * DOC: genhd capability flags
* *
* ``GENHD_FL_REMOVABLE`` (0x0001): indicates that the block device * ``GENHD_FL_REMOVABLE``: indicates that the block device gives access to
* gives access to removable media. * removable media. When set, the device remains present even when media is not
* When set, the device remains present even when media is not * inserted. Shall not be set for devices which are removed entirely when the
* inserted.
* Must not be set for devices which are removed entirely when the
* media is removed. * media is removed.
* *
* ``GENHD_FL_NO_PART`` (0x0200): partition support is disabled. * ``GENHD_FL_HIDDEN``: the block device is hidden; it doesn't produce events,
* The kernel will not scan for partitions from add_disk, and users * doesn't appear in sysfs, and can't be opened from userspace or using
* can't add partitions manually. * blkdev_get*. Used for the underlying components of multipath devices.
*
* ``GENHD_FL_NO_PART``: partition support is disabled. The kernel will not
* scan for partitions from add_disk, and users can't add partitions manually.
* *
* ``GENHD_FL_HIDDEN`` (0x0400): the block device is hidden; it
* doesn't produce events, doesn't appear in sysfs, and doesn't have
* an associated ``bdev``.
* Implies ``GENHD_FL_NO_PART``.
* Used for multipath devices.
*/ */
#define GENHD_FL_REMOVABLE 0x0001 enum {
/* 2 is unused (used to be GENHD_FL_DRIVERFS) */ GENHD_FL_REMOVABLE = 1 << 0,
/* 4 is unused (used to be GENHD_FL_MEDIA_CHANGE_NOTIFY) */ GENHD_FL_HIDDEN = 1 << 1,
#define GENHD_FL_NO_PART 0x0200 GENHD_FL_NO_PART = 1 << 2,
#define GENHD_FL_HIDDEN 0x0400 };
enum { enum {
DISK_EVENT_MEDIA_CHANGE = 1 << 0, /* media changed */ DISK_EVENT_MEDIA_CHANGE = 1 << 0, /* media changed */
......
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