[Cluster-devel] [PATCH] Fixed hep for 'addservice' command

2010-12-17 Thread Dmitry Mishin
Sorry, missed it in 'addservice' addition patch series. Signed-off-by: Dmitry Mishin d...@parallels.com --- config/tools/ccs_tool/editconf.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/config/tools/ccs_tool/editconf.c b/config/tools/ccs_tool/editconf.c index

Re: [Cluster-devel] [PATCH] Fixed hep for 'addservice' command

2010-12-17 Thread Fabio M. Di Nitto
On 12/17/2010 12:54 PM, Dmitry Mishin wrote: Sorry, missed it in 'addservice' addition patch series. No problem :) merged now. Fabio

[Cluster-devel] [PATCH 14/15] GFS2: fsck.gfs2 reported statfs error after gfs2_grow

2010-12-17 Thread Steven Whitehouse
From: Bob Peterson rpete...@redhat.com When you do gfs2_grow it failed to take the very last rgrp into account when adding up the new free space due to an off-by-one error. It was not reading the last rgrp from the rindex because of a check for = that should have been . Therefore, fsck.gfs2 was

[Cluster-devel] [PATCH 07/15] GFS2: Allow gfs2 to update quota usage values through the quotactl interface

2010-12-17 Thread Steven Whitehouse
From: Abhijith Das a...@redhat.com With this patch the gfs2_set_dqblk() function will be able to update the quota usage block count (FS_DQ_BCOUNT) in addition to the already supported FS_DQ_BHARD (limit) and FS_DQ_BSOFT (warn) fields of the dquot structure. Signed-off-by: Abhi Das

[Cluster-devel] [PATCH 12/15] GFS2: Fix uninitialised error value in previous patch

2010-12-17 Thread Steven Whitehouse
Signed-off-by: Steven Whitehouse swhit...@redhat.com diff --git a/fs/gfs2/bmap.c b/fs/gfs2/bmap.c index ae7d205..3c4039d 100644 --- a/fs/gfs2/bmap.c +++ b/fs/gfs2/bmap.c @@ -763,7 +763,7 @@ static int do_strip(struct gfs2_inode *ip, struct buffer_head *dibh, int metadata;

[Cluster-devel] [PATCH 01/15] GFS2: Change two WQ_RESCUERs into WQ_MEM_RECLAIM

2010-12-17 Thread Steven Whitehouse
The WQ_RESCUER flag should only be used internally to the workqueue implementation. Signed-off-by: Steven Whitehouse swhit...@redhat.com Acked-by: Tejun Heo t...@kernel.org diff --git a/fs/gfs2/glock.c b/fs/gfs2/glock.c index f92c177..5a56568 100644 --- a/fs/gfs2/glock.c +++ b/fs/gfs2/glock.c @@

[Cluster-devel] [PATCH 03/15] GFS2: Remove unreachable calls to vmtruncate

2010-12-17 Thread Steven Whitehouse
Suggested-by: Christoph Hellwig h...@infradead.org Signed-off-by: Steven Whitehouse swhit...@redhat.com diff --git a/fs/gfs2/inode.c b/fs/gfs2/inode.c index e1213f7..14e682d 100644 --- a/fs/gfs2/inode.c +++ b/fs/gfs2/inode.c @@ -916,17 +916,8 @@ static int __gfs2_setattr_simple(struct gfs2_inode

[Cluster-devel] [PATCH 08/15] GFS2: Clean up of gdlm_lock function

2010-12-17 Thread Steven Whitehouse
The DLM never returns -EAGAIN in response to dlm_lock(), and even if it did, the test in gdlm_lock() was wrong anyway. Once that test is removed, it is possible to greatly simplify this code by simply using a normal error return code (0 for success). We then no longer need the LM_OUT_ASYNC return

[Cluster-devel] [PATCH 15/15] GFS2: Don't flush delete workqueue when releasing the transaction lock

2010-12-17 Thread Steven Whitehouse
There is no requirement to flush the delete workqueue before a gfs2 filesystem is suspended. The workqueue's work will just be suspended along with the rest of the tasks on the filesystem. The resolves a deadlock situation where the transaction lock's demotion code was trying to flush the delete

[Cluster-devel] GFS2: Pre-pull patch posting (-nmw)

2010-12-17 Thread Steven Whitehouse
Hi, This is a bit early this time, but I'm not expecting to add any more features to this patch set before the merge window. In fact it is mostly clean up and fixes anyway. So I'm pushing it out now to let everybody see it ahead of time. The only user visible new feature this time is the

[Cluster-devel] [PATCH 05/15] GFS2: fs/gfs2/glock.c: Use printf extension %pV

2010-12-17 Thread Steven Whitehouse
From: Joe Perches j...@perches.com Using %pV reduces the number of printk calls and eliminates any possible message interleaving from other printk calls. Signed-off-by: Joe Perches j...@perches.com Signed-off-by: Steven Whitehouse swhit...@redhat.com diff --git a/fs/gfs2/glock.c

[Cluster-devel] [PATCH 04/15] GFS2: Clean up duplicated setattr code

2010-12-17 Thread Steven Whitehouse
While preparing the last patch I noticed that the gfs2_setattr_simple code had been duplicated into two other places. This patch updates those to call gfs2_setattr_simple rather than open coding it. Signed-off-by: Steven Whitehouse swhit...@redhat.com diff --git a/fs/gfs2/ops_inode.c

[Cluster-devel] [PATCH 13/15] GFS2: Merge glock state fields into a bitfield

2010-12-17 Thread Steven Whitehouse
We can only merge the fields into a bitfield if the locking rules for them are the same. In this case gl_spin covers all of the fields (write side) but a couple of them are used with GLF_LOCK as the read side lock, which should be ok since we know that the field in question won't be changing at

[Cluster-devel] [PATCH 06/15] GFS2: fs/gfs2/glock.h: Add __attribute__((format(printf, 2, 3)) to gfs2_print_dbg

2010-12-17 Thread Steven Whitehouse
From: Joe Perches j...@perches.com Functions that use printf formatting, especially those that use %pV, should have their uses of printf format and arguments checked by the compiler. Signed-off-by: Joe Perches j...@perches.com Signed-off-by: Steven Whitehouse swhit...@redhat.com diff --git