Re: [PATCH 2/6] block: remove the NULL bdev check in bdev_read_only

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH 2/6] block: remove the NULL bdev check in bdev_read_only
From: "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>
Date: Tue, 08 Dec 2020 00:23:59 -0500
Cc: Jens Axboe <axboe@xxxxxxxxx>, Sagi Grimberg <sagi@xxxxxxxxxxx>, Mike Snitzer <snitzer@xxxxxxxxxx>, Oleksii Kurochko <olkuroch@xxxxxxxxx>, Dongsheng Yang <dongsheng.yang@xxxxxxxxxxxx>, linux-block@xxxxxxxxxxxxxxx, dm-devel@xxxxxxxxxx, linux-nvme@xxxxxxxxxxxxxxxxxxx, "Martin K . Petersen" <martin.petersen@xxxxxxxxxx>, Ilya Dryomov <idryomov@xxxxxxxxx>, ceph-devel@xxxxxxxxxxxxxxx
In-reply-to: <20201207131918.2252553-3-hch@lst.de> (Christoph Hellwig's message of "Mon, 7 Dec 2020 14:19:14 +0100")
Organization: Oracle Corporation
References: <20201207131918.2252553-1-hch@lst.de> <20201207131918.2252553-3-hch@lst.de>

Christoph,

> Only a single caller can end up in bdev_read_only, so move the check there.

Reviewed-by: Martin K. Petersen <martin.petersen@xxxxxxxxxx>

-- 
Martin K. Petersen	Oracle Linux Engineering

--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel

References:
- split hard read-only vs read-only policy v2
  - From: Christoph Hellwig
- [PATCH 2/6] block: remove the NULL bdev check in bdev_read_only
  - From: Christoph Hellwig

Prev by Date: Re: [PATCH 1/6] dm: use bdev_read_only to check if a device is read-only
Next by Date: Re: [PATCH 3/6] block: add a hard-readonly flag to struct gendisk
Previous by thread: [PATCH 2/6] block: remove the NULL bdev check in bdev_read_only
Next by thread: Re: [PATCH 2/6] block: remove the NULL bdev check in bdev_read_only
Index(es):
- Date
- Thread

[Index of Archives] [DM Crypt] [Fedora Desktop] [ATA RAID] [Fedora Marketing] [Fedora Packaging] [Fedora SELinux] [Yosemite Discussion] [KDE Users] [Fedora Docs]

Powered by Linux