From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Teigland Date: Thu, 5 Jan 2012 10:21:05 -0500 Subject: [Cluster-devel] [PATCH 5/5] gfs2: dlm based recovery coordination In-Reply-To: References: <1324072991-30729-6-git-send-email-teigland@redhat.com> Message-ID: <20120105152105.GA22610@redhat.com> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit On Thu, Jan 05, 2012 at 10:08:15AM -0500, Bob Peterson wrote: > ----- Original Message ----- > | This new method of managing recovery is an alternative to > | the previous approach of using the userland gfs_controld. > | > | - use dlm slot numbers to assign journal id's > | - use dlm recovery callbacks to initiate journal recovery > | - use a dlm lock to determine the first node to mount fs > | - use a dlm lock to track journals that need recovery > | > | Signed-off-by: David Teigland > | --- > | --- a/fs/gfs2/lock_dlm.c > | +++ b/fs/gfs2/lock_dlm.c > (snip) > | +#include > | #include > > Hi, > > Dave, are you going to post a replacement patch or addendum patch > that addresses Steve's concerns, such as the above? > I'd like to review this, but I want the review the latest/greatest. I haven't resent the patches after making the changes (which were fairly minor.) I'll resend them shortly for another check before a pull request. Dave