From mboxrd@z Thu Jan 1 00:00:00 1970 From: Milan Broz Subject: Re: [PATCH UPDATED] block: restore multiple bd_link_disk_holder() support Date: Fri, 14 Jan 2011 22:09:12 +0100 Message-ID: <4D30BB78.5020101@redhat.com> References: <16069.1294853673@localhost> <4D2E4611.90002@redhat.com> <4D2E6129.8000700@ce.jp.nec.com> <20110113172133.GE14096@htj.dyndns.org> <4D2F4799.5030901@redhat.com> <4D2FFBED.6010406@ce.jp.nec.com> <20110114161043.GB978@htj.dyndns.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: "Jun'ichi Nomura" , Jens Axboe , Valdis.Kletnieks@vt.edu, Alexander Viro , Neil Brown , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-raid@vger.kernel.org, device-mapper development , Kay Sievers , Alasdair G Kergon To: Tejun Heo Return-path: In-Reply-To: <20110114161043.GB978@htj.dyndns.org> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org 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 > Reported-by: Milan Broz > Cc: Jun'ichi Nomura > Cc: Neil Brown > Cc: linux-raid@vger.kernel.org > Cc: Kay Sievers > --- > 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 Milan