On 01/14/2011 05:10 PM, Tejun Heo wrote: > Commit e09b457b (block: simplify holder symlink handling) incorrectly > assumed that there is only one link at maximum. dm may use multiple > links and expects block layer to track reference count for each link, > which is different from and unrelated to the exclusive device holder > identified by @holder when the device is opened. > > Remove the single holder assumption and automatic removal of the link > and revive the per-link reference count tracking. The code > essentially behaves the same as before commit e09b457b sans the > unnecessary kobject reference count dancing. > > While at it, note that this facility should not be used by anyone else > than the current ones. Sysfs symlinks shouldn't be abused like this > and the whole thing doesn't belong in the block layer at all. > > Signed-off-by: Tejun Heo <tj@xxxxxxxxxx> > Reported-by: Milan Broz <mbroz@xxxxxxxxxx> > Cc: Jun'ichi Nomura <j-nomura@xxxxxxxxxxxxx> > Cc: Neil Brown <neilb@xxxxxxx> > Cc: linux-raid@xxxxxxxxxxxxxxx > Cc: Kay Sievers <kay.sievers@xxxxxxxx> > --- > Thanks for the test commands. They were very helpful. Can you please > test this one? Hi, yes, this one works for me. I run full lvm2 testsuite and no warnings. Thanks! Tested-by: Milan Broz <mbroz@xxxxxxxxxx> Milan -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
