From mboxrd@z Thu Jan 1 00:00:00 1970 Message-ID: <5148940F.3020600@hoster-ok.com> Date: Tue, 19 Mar 2013 19:36:31 +0300 From: Vladislav Bogdanov MIME-Version: 1.0 References: <1363699970-10002-1-git-send-email-bubble@hoster-ok.com> <1363699970-10002-2-git-send-email-bubble@hoster-ok.com> <20130319152347.GA986@redhat.com> <51488550.5070201@hoster-ok.com> <514887FA.4000104@hoster-ok.com> <20130319160319.GD986@redhat.com> In-Reply-To: <20130319160319.GD986@redhat.com> Content-Transfer-Encoding: 7bit Subject: Re: [linux-lvm] [PATCH 01/10] lvchange: Allow cluster lock conversion Reply-To: LVM general discussion and development List-Id: LVM general discussion and development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , List-Id: Content-Type: text/plain; charset="us-ascii" To: David Teigland Cc: linux-lvm@redhat.com 19.03.2013 19:03, David Teigland wrote: > On Tue, Mar 19, 2013 at 06:44:58PM +0300, Vladislav Bogdanov wrote: >>>> I'm trying to figure out why the code doesn't already use CONVERT when >>>> possible; I would have expected it to. >>> >>>> What happens now without this force/CONVERT >>>> patch? a new lock created/requested? >>>> > >> After new lock creation is requested ;) > > Yeah, it doesn't make much sense for clvmd-corosync.c to do that. > I'm hoping that we can make clvmd-corosync.c aware that it already > holds that lock and attempt to convert it. But that could be not always expected...