[Cluster-devel] [PATCH] dlm/user.c: static initialization improvements

2008-01-28 Thread Denis Cheng
also change name_prefix from char pointer to char array. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/dlm/user.c | 13 +++-- 1 files changed, 7 insertions(+), 6 deletions(-) diff --git a/fs/dlm/user.c b/fs/dlm/user.c index 4f74154..2cc5415 100644 --- a/fs/dlm/user.c +++ b/fs/d

[Cluster-devel] [GFS2 PATCH] Get rid of gl_waiters2

2008-01-28 Thread Bob Peterson
Hi, This patch reduces memory by replacing the int variable gl_waiters2 by a single bit in the gl_flags. Regards, Bob Peterson Red Hat GFS Signed-off-by: Bob Peterson <[EMAIL PROTECTED]> -- fs/gfs2/glock.c |7 --- fs/gfs2/incore.h |4 ++-- 2 files changed, 6 insertions(+), 5 dele

[Cluster-devel] [GFS2 PATCH] Reduce size of version numbers

2008-01-28 Thread Bob Peterson
Hi, This patch changes the version number values in three structures from 64-bit to 16-bit. These values are never compared in a greater-than or less-than, so wrap-around shouldn't be an issue. Eventually we can get rid of these altogether in favor of a bit or two. The locations were moved to fa

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

2008-01-28 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.

[Cluster-devel] Re: [PATCH 1/3] NLM add resume procfs file

2008-01-28 Thread J. Bruce Fields
On Mon, Jan 28, 2008 at 09:19:10PM -0500, bfields wrote: > On Fri, Jan 25, 2008 at 12:15:05AM -0500, Wendy Cheng wrote: > > +static ssize_t failover_resume_ip(struct file *file, char *buf, size_t > > size) > > +{ > > + __be32 server_ip; > > + int len; > > + > > + len = failover_parse_ip(file

[Cluster-devel] Re: [PATCH 1/3] NLM add resume procfs file

2008-01-28 Thread J. Bruce Fields
On Fri, Jan 25, 2008 at 12:15:05AM -0500, Wendy Cheng wrote: > Add a new nfsd procfs file "resume_ip" to enable per-ip base lockd grace > period: > > Example Usage: > root-shell> echo 10.1.1.64 > /proc/fs/nfsd/resume_ip > > Only support IPV4 address for this patch submission. > > -- Wendy > > Ad

[Cluster-devel] [GFS2 PATCH] Combine rg_flags and rd_flags

2008-01-28 Thread Bob Peterson
Hi, This patch reduces the memory required by GFS2 by combining the rd_flags and rg_flags (in core only). Regards, Bob Peterson Red Hat GFS Signed-off-by: Bob Peterson <[EMAIL PROTECTED]> -- fs/gfs2/incore.h |6 +++--- fs/gfs2/rgrp.c | 36 +++- 2 files

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

2008-01-28 Thread Bob Peterson
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 Hat GFS Signed-off-by: Bob Peterson <[EMAIL PROTECTED]> -- fs/gfs2/main.c | 10 ++ fs/gfs2/rgrp.c |

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

2008-01-28 Thread Bob Peterson
-- 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 +++ b/fs/gfs2/lops.c @@ -404,8 +404,10 @@ static int revoke_lo_scan_elements(struct gfs2_jdesc *jd, unsigned int start,

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

2008-01-28 Thread Adrian Bunk
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 deletions(-) f04fbd054a051a564a3138e0e02b7423acf45a06 diff --git a/fs/gfs2/glock.c b/fs/gfs2/glock.c index 80e09c

[Cluster-devel] [GFS2 PATCH] Only wake the reclaim daemon if we need to

2008-01-28 Thread Bob Peterson
Hi, This patch only wakes up the glock reclaim daemon if there is actually something to be reclaimed. Regards, Bob Peterson Red Hat GFS Signed-off-by: Bob Peterson <[EMAIL PROTECTED]> -- fs/gfs2/glock.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/fs/gfs2/gl

[Cluster-devel] cluster/rgmanager ChangeLog

2008-01-28 Thread lhh
CVSROOT:/cvs/cluster Module name:cluster Branch: RHEL4 Changes by: [EMAIL PROTECTED] 2008-01-28 19:42:56 Modified files: rgmanager : ChangeLog Log message: Fix #292861 - nfsclient doesn't unexport correctly Patches: http://sourceware.org/cgi-bi

[Cluster-devel] cluster/rgmanager/src/resources nfsclient.sh

2008-01-28 Thread lhh
CVSROOT:/cvs/cluster Module name:cluster Branch: RHEL4 Changes by: [EMAIL PROTECTED] 2008-01-28 19:42:43 Modified files: rgmanager/src/resources: nfsclient.sh Log message: Fix #292861 - nfsclient doesn't unexport correctly Patches: http://sourceware

[Cluster-devel] cluster/rgmanager/src/utils clulog.c

2008-01-28 Thread lhh
CVSROOT:/cvs/cluster Module name:cluster Branch: RHEL4 Changes by: [EMAIL PROTECTED] 2008-01-28 19:34:32 Modified files: rgmanager/src/utils: clulog.c Log message: Fix #289521 - clulog not respecting rgmanager log level Patches: http://sourceware.or

[Cluster-devel] cluster/rgmanager ChangeLog include/rg_locks.h ...

2008-01-28 Thread lhh
CVSROOT:/cvs/cluster Module name:cluster Branch: RHEL4 Changes by: [EMAIL PROTECTED] 2008-01-28 19:27:20 Modified files: rgmanager : ChangeLog rgmanager/include: rg_locks.h rgmanager/src/daemons: groups.c main.c rg_locks.c rgman

[Cluster-devel] [GFS2 PATCH] Misc fixups

2008-01-28 Thread Bob Peterson
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. Regards, Bob Peterson Red Hat GFS Signed-off-by: Bob Peterson <[EMAIL PROTECTED]> -- fs/gfs2/bmap.c

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

2008-01-28 Thread Bob Peterson
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 same function and does the same thing in both cases, there should be no reason to call it twice. Since we o

[Cluster-devel] [GFS2 PATCH]

2008-01-28 Thread Bob Peterson
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 <[EMAIL PROTECTED]> -- fs/gfs2/inode.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --

[Cluster-devel] [GFS2 PATCH] Get rid of unneeded parameter in gfs2_rlist_alloc

2008-01-28 Thread Bob Peterson
Hi, This patch removed the unnecessary parameter from function gfs2_rlist_alloc. The parameter was always passed in as 0. Regards, Bob Peterson Red Hat GFS Signed-off-by: Bob Peterson <[EMAIL PROTECTED]> -- fs/gfs2/bmap.c|1 +- fs/gfs2/dir.c |2 +- fs/gfs2/eattr.c

Re: [Cluster-devel] Re: [PATCH 1/2] NLM failover unlock commands

2008-01-28 Thread Felix Blyakher
On Jan 28, 2008, at 9:56 AM, Wendy Cheng wrote: Felix Blyakher wrote: (I think Wendy's pretty close to that api already after adding the second method to start grace?) For reclaiming grace period issues, maybe we should move to https://www.redhat.com/archives/cluster-devel/2008-January/ ms

Re: [Cluster-devel] Re: [PATCH 1/2] NLM failover unlock commands

2008-01-28 Thread Wendy Cheng
Felix Blyakher wrote: (I think Wendy's pretty close to that api already after adding the second method to start grace?) For reclaiming grace period issues, maybe we should move to https://www.redhat.com/archives/cluster-devel/2008-January/msg00340.html thread ? I view this (unlock) patch se

[Cluster-devel] GFS2 git tree

2008-01-28 Thread Steven Whitehouse
Hi, Since Linus pulled the patches from the -nmw tree on Friday, I've rebased the tree today from the current upstream and added a couple of patches that didn't make the merge window, Steve.

[Cluster-devel] [GFS2] Streamline indirect pointer tree height calculation

2008-01-28 Thread Steven Whitehouse
Hi, This patch improves the calculation of the tree height in order to reduce the number of operations which are carried out on each call to gfs2_block_map. In the common case, we now make a single comparison, rather than calculating the required tree height from scratch each time. Also in the cas