On Tue, Oct 17, 2023 at 08:48:21PM +0200, Christoph Hellwig wrote: > disk_check_media_change is mostly called from ->open where it makes > little sense to mark the file system on the device as dead, as we > are just opening it. So instead of calling bdev_mark_dead from > disk_check_media_change move it into the few callers that are not > in an open instance. This avoid calling into bdev_mark_dead and > thus taking s_umount with open_mutex held. ->open() is called when opening bdev every times, and there can be existed openers, so not sure if it makes little sense here. Thanks, Ming
