From mboxrd@z Thu Jan  1 00:00:00 1970
From: Wendy Cheng <wcheng@redhat.com>
Date: Wed, 31 Jan 2007 19:22:48 -0500
Subject: [Cluster-devel] Fix for unlink deadlock
In-Reply-To: <1170289078.3832.5.camel@localhost.localdomain>
References: <1170112424.28100.124.camel@xenon.msp.redhat.com>
	<1170163127.11001.164.camel@quoit.chygwyn.com>
	<1170289078.3832.5.camel@localhost.localdomain>
Message-ID: <1170289369.3832.7.camel@localhost.localdomain>
List-Id: <cluster-devel.redhat.com>
To: cluster-devel.redhat.com
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit

On Wed, 2007-01-31 at 19:17 -0500, Wendy Cheng wrote:
> On Tue, 2007-01-30 at 13:18 +0000, Steven Whitehouse wrote:
> > Hi,
> > 
> > On Mon, 2007-01-29 at 17:13 -0600, Russell Cattelan wrote:
> > > Note the current glock code in the tree is completely busted
> > > and will dead lock almost immediately.
> > > 
> > > I have reverted several changes to glock.c and have 
> > > tested this patch with the older glock code.
> > > dbench will now run through to completion.
> > > 
> > > 
> > > https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=217356
> > 
> > Now applied to the -nmw git tree.
> 
> hmm.. just untar Steve's git tree he sent me this morning few minutes
> ago and spot this - sorry I should have read this post earlier ... Look
> to me this will deadlock gfs2_rename() *again* (since it tries to grab
> rindex lock multiple times). Will test this out tomorrow when I get the
> connectathon cluster up.
> 
> 

BTW, Is it too late to NACK this patch ? :) ... Wendy