Re: [Cluster-devel] [GFS2 PATCH] Allocate gfs2_rgrpd from slab memory

2008-01-29 Thread Steven Whitehouse
Hi, Now in the -nmw git tree. Thanks, Steve. On Mon, 2008-01-28 at 17:20 -0600, Bob Peterson wrote: Hi, This patch moves the gfs2_rgrpd structure to its own slab memory. This makes it easier to control and monitor, and yields less memory fragmentation. Regards, Bob Peterson Red

Re: [Cluster-devel] [GFS2 PATCH]

2008-01-29 Thread Steven Whitehouse
Hi, Now in the -nmw tree. Thanks, Steve. On Mon, 2008-01-28 at 11:15 -0600, Bob Peterson wrote: Hi, I noticed that the latest change to i_height got rid of the value from the inode dump. This patch adds it back. Regards, Bob Peterson Red Hat GFS Signed-off-by: Bob Peterson

[Cluster-devel] Re: [2.6 patch] make gfs2_glock_hold() static

2008-01-29 Thread Steven Whitehouse
Hi, Now in the GFS2 -nmw git tree. Thanks, Steve. On Tue, 2008-01-29 at 00:11 +0200, Adrian Bunk wrote: gfs2_glock_hold() can now become static. Signed-off-by: Adrian Bunk [EMAIL PROTECTED] --- fs/gfs2/glock.c |2 +- fs/gfs2/glock.h |1 - 2 files changed, 1 insertion(+), 2

Re: [Cluster-devel] [GFS2 PATCH] Only do lo_incore_commit once

2008-01-29 Thread Steven Whitehouse
Hi, Now in the -nmw tree. Thanks, Steve. On Mon, 2008-01-28 at 11:20 -0600, Bob Peterson wrote: Hi, This patch is performance related. When we're doing a log flush, I noticed we were calling buf_lo_incore_commit twice: once for data bufs and once for metadata bufs. Since this is the

Re: [Cluster-devel] [GFS2 PATCH] Misc fixups

2008-01-29 Thread Steven Whitehouse
Hi, Now in the -nmw tree. Thanks, Steve. On Mon, 2008-01-28 at 11:24 -0600, Bob Peterson wrote: Hi, This patch contains two small fixups that didn't fit elsewhere. They are: (1) get rid of temp variable in find_metapath. (2) Remove vestigial ret variable from gfs2_writepage_common.

Re: [Cluster-devel] [GFS2 PATCH] Plug an unlikely leak

2008-01-29 Thread Steven Whitehouse
Hi, Now in the -nme git tree. Thanks, Steve. On Mon, 2008-01-28 at 16:35 -0600, Bob Peterson wrote: -- fs/gfs2/lops.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/fs/gfs2/lops.c b/fs/gfs2/lops.c index 7138737..4390f6f 100644 --- a/fs/gfs2/lops.c +++

[Cluster-devel] Re: [PATCH 2/3] NLM per-ip grace period - core

2008-01-29 Thread J. Bruce Fields
On Tue, Jan 29, 2008 at 11:33:08AM -0500, Chuck Lever wrote: On Jan 28, 2008, at 9:56 PM, J. Bruce Fields wrote: Also, is that the right choice of types? Maybe we should just use struct in6_addr? I agree with Bruce -- just use in_addr, and leave out the unused IPv6 stuff for now. In

[Cluster-devel] [GFS2] Introduce array of buffers to struct metapath

2008-01-29 Thread Steven Whitehouse
From 9fdea9ec9922417b9c884edbf27db9f83600cf95 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse [EMAIL PROTECTED] Date: Tue, 29 Jan 2008 09:12:55 + Subject: [PATCH] [GFS2] Introduce array of buffers to struct metapath The reason for doing this is to allow all the block mapping code to share

[Cluster-devel] [GFS2] Move part of gfs2_block_map into a separate function

2008-01-29 Thread Steven Whitehouse
From cea26d2d43d69e88298154da7d023ab2af5eae7a Mon Sep 17 00:00:00 2001 From: Steven Whitehouse [EMAIL PROTECTED] Date: Mon, 28 Jan 2008 15:10:29 + Subject: [PATCH] [GFS2] Move part of gfs2_block_map into a separate function This is required to enable future changes to the block mapping code.

[Cluster-devel] Re: [PATCH 2/3] NLM per-ip grace period - core

2008-01-29 Thread Wendy Cheng
Chuck Lever wrote: On Jan 28, 2008, at 9:56 PM, J. Bruce Fields wrote: On Fri, Jan 25, 2008 at 12:17:30AM -0500, Wendy Cheng wrote: The logic is implemented on top of linux nfsd procfs with core functions residing in lockd kernel module. Entry function is nlmsvc_resume_ip() where it stores

[Cluster-devel] [GFS2] Add consts to various bits of rgrp.c

2008-01-29 Thread Steven Whitehouse
From 15792c4b1735f6859edb81a2038af39c298250d1 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse [EMAIL PROTECTED] Date: Tue, 29 Jan 2008 13:30:20 + Subject: [PATCH] [GFS2] Add consts to various bits of rgrp.c There are a couple of routines which scan bitmaps where we can mark the bitmaps

Re: [Cluster-devel] [GFS2] Add consts to various bits of rgrp.c

2008-01-29 Thread Bob Peterson
On Tue, 2008-01-29 at 14:32 +, Steven Whitehouse wrote: From 15792c4b1735f6859edb81a2038af39c298250d1 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse [EMAIL PROTECTED] Date: Tue, 29 Jan 2008 13:30:20 + Subject: [PATCH] [GFS2] Add consts to various bits of rgrp.c There are a couple

[Cluster-devel] cluster/cman/qdisk score.c

2008-01-29 Thread lhh
CVSROOT:/cvs/cluster Module name:cluster Branch: RHEL51 Changes by: [EMAIL PROTECTED] 2008-01-29 21:26:39 Modified files: cman/qdisk : score.c Log message: Fix #430574 - qdisk blocking signals in heuristics Patches:

[Cluster-devel] conga ./clustermon.spec.in.in ./conga.spec.in. ...

2008-01-29 Thread rmccabe
CVSROOT:/cvs/cluster Module name:conga Branch: RHEL5 Changes by: [EMAIL PROTECTED] 2008-01-29 22:02:13 Modified files: . : clustermon.spec.in.in conga.spec.in.in make : version.in ricci/modules/cluster:

[Cluster-devel] [PATCH] Make fence_xvm's TTL configurable

2008-01-29 Thread Lon Hohberger
fence_xvm's multicast TTL is statically set to 2. On machines where a router is in-between, this can cause failures due to the router dropping the packets. -- Lon Index: agents/xvm/fence_xvm.c === RCS file:

[Cluster-devel] [PATCH] Make fenced's retry time configurable;

2008-01-29 Thread Lon Hohberger
As seen here: https://bugzilla.redhat.com/show_bug.cgi?id=418531 Index: fenced/fd.h === RCS file: /cvs/cluster/cluster/fence/fenced/fd.h,v retrieving revision 1.27 diff -u -r1.27 fd.h --- fenced/fd.h 26 Oct 2007 20:36:49 -

Re: [Cluster-devel] [GFS2] Introduce array of buffers to struct metapath

2008-01-29 Thread Bob Peterson
On Tue, 2008-01-29 at 14:32 +, Steven Whitehouse wrote: From 9fdea9ec9922417b9c884edbf27db9f83600cf95 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse [EMAIL PROTECTED] Date: Tue, 29 Jan 2008 09:12:55 + Subject: [PATCH] [GFS2] Introduce array of buffers to struct metapath The

Re: [Cluster-devel] [GFS2] Move part of gfs2_block_map into a separate function

2008-01-29 Thread Bob Peterson
On Tue, 2008-01-29 at 14:31 +, Steven Whitehouse wrote: From cea26d2d43d69e88298154da7d023ab2af5eae7a Mon Sep 17 00:00:00 2001 From: Steven Whitehouse [EMAIL PROTECTED] Date: Mon, 28 Jan 2008 15:10:29 + Subject: [PATCH] [GFS2] Move part of gfs2_block_map into a separate function

[Cluster-devel] cluster/gfs-kernel/src/gfs glock.c

2008-01-29 Thread bmarzins
CVSROOT:/cvs/cluster Module name:cluster Branch: RHEL4 Changes by: [EMAIL PROTECTED] 2008-01-29 22:21:45 Modified files: gfs-kernel/src/gfs: glock.c Log message: Fix for bz #419391. gfs_glock_dq was traversing the gl_holders list without

[Cluster-devel] [GFS2 PATCH] Eliminate gh_state

2008-01-29 Thread Bob Peterson
Hi, This patch reduces the memory requirements of GFS2 by eliminating the gh_state variable from the gfs2_holder structure. With this patch the state is now held in the upper two bits of the gh_flags. There are new macros to fetch and/or set the new state flags. I hesitated sending this patch

[Cluster-devel] Re: [PATCH 2/3] NLM per-ip grace period - core

2008-01-29 Thread J. Bruce Fields
On Fri, Jan 25, 2008 at 12:17:30AM -0500, Wendy Cheng wrote: The logic is implemented on top of linux nfsd procfs with core functions residing in lockd kernel module. Entry function is nlmsvc_resume_ip() where it stores the requested ip interface into a linked-list nlm_failover_list. The

[Cluster-devel] Re: [PATCH 3/3] NLM enable per-ip base grace period

2008-01-29 Thread J. Bruce Fields
Thanks for the patches; a few more comments below: On Fri, Jan 25, 2008 at 12:19:07AM -0500, Wendy Cheng wrote: Hooks are added into the existing lockd global grace period checking to enable per-ip base grace period. -- Wendy Hooks are added into the existing lockd global grace period

Re: [Cluster-devel] [PATCH] Make fence_xvm's TTL configurable

2008-01-29 Thread Fabio M. Di Nitto
Hi Lon, On Tue, 29 Jan 2008, Lon Hohberger wrote: fence_xvm's multicast TTL is statically set to 2. On machines where a router is in-between, this can cause failures due to the router dropping the packets. do I understand right that we use multicast because we don't know where the target

[Cluster-devel] cluster/gfs-kernel/src/gfs eaops.c inode.c ops ...

2008-01-29 Thread fabbione
CVSROOT:/cvs/cluster Module name:cluster Changes by: [EMAIL PROTECTED] 2008-01-30 06:37:53 Modified files: gfs-kernel/src/gfs: eaops.c inode.c ops_file.c ops_file.h Log message: Red Hat bugzilla 244343: GFS supports two modes of locking -

[Cluster-devel] cluster/gfs-kernel/src/gfs rgrp.c

2008-01-29 Thread fabbione
CVSROOT:/cvs/cluster Module name:cluster Changes by: [EMAIL PROTECTED] 2008-01-30 06:57:41 Modified files: gfs-kernel/src/gfs: rgrp.c Log message: Bugzilla 227892: * Warn people about the RG corruption and request a gfs_fsck * Upon