Re: linux-next - WARNING: at fs/block_dev.c:824 bd_link_disk_holder+0x92/0x1ac()

Tejun Heo
2011-01-13T11:06:57Z

Hello,

On Thu, Jan 13, 2011 at 11:19:21AM +0900, Jun'ichi Nomura wrote:
> > "block: simplify holder symlink handling"
> >
> > dm linear just claims device in table constructor, I don't think it is bug in DM code.
>
> The patch assumes only one holder disk for a claimed dev, which is not true.
> E.g. if there are multiple LVs on a PV.
>
> In addition to that, since claiming is done in table constructor,
> there can be 2 claim instances for a slave/holder pair at a time
> when you load a table while there's already an active one.
> E.g. if you do lvresize.
> We need consideration for this, too.

Urgh... gees, so there actually was a user using all that cruft.
Sorry about the breakage, I'll see how multiple symlinks can be
restored. I'm curious why this was added at all tho. What was the
rationalization? It's not like two subsystems can share the same
block device so marking the currently owning subsystem should have
been enough at the block layer. There is no reason for block devices
to present information which is of no use to itself. All that's
necessary is "this is taken by dm or md for more information, query
those". dm and md need their own conf/representation layer anyway.

Thanks.

--
tejun