Re: [PATCH v7 3/3] dm: add DM_INTERPOSED_FLAG

From: Sergei Shtepa
Date: Mon Mar 15 2021 - 09:26:14 EST


The 03/14/2021 12:30, Christoph Hellwig wrote:
> On Fri, Mar 12, 2021 at 06:44:55PM +0300, Sergei Shtepa wrote:
> > DM_INTERPOSED_FLAG allow to create DM targets on "the fly".
> > Underlying block device opens without a flag FMODE_EXCL.
> > DM target receives bio from the original device via
> > bdev_interposer.
>
> This is more of a philopical comment, but the idea of just letting the
> interposed reopen the device by itself seems like a bad idea. I think
> that is probably better hidden in the block layer interposer attachment
> function, which could do the extra blkdev_get_by_dev for the caller.

I suppose this cannot be implemented, since we need to change the behavior
for block devices that already have been opened.

--
Sergei Shtepa
Veeam Software developer.