[Cluster-devel] GFS2: Pre-pull patch posting

2013-04-05 Thread Steven Whitehouse
Hi, Here are a few GFS2 fixes which are pending. There are two patches which fix up a couple of minor issues in the DLM interface code, a missing error path in gfs2_rs_alloc(), two patches which fix problems during withdraw and a fix for discards/FITRIM when using 4k sector sized devices, Steve.

[Cluster-devel] [PATCH 1/6] GFS2: use kmalloc for lvb bitmap

2013-04-05 Thread Steven Whitehouse
From: David Teigland teigl...@redhat.com The temp lvb bitmap was on the stack, which could be an alignment problem for __set_bit_le. Use kmalloc for it instead. Signed-off-by: David Teigland teigl...@redhat.com Signed-off-by: Steven Whitehouse swhit...@redhat.com diff --git a/fs/gfs2/incore.h

[Cluster-devel] [PATCH 5/6] GFS2: Fix unlock of fcntl locks during withdrawn state

2013-04-05 Thread Steven Whitehouse
From: David Teigland teigl...@redhat.com This reminded me of another old patch I had sitting around which I never had a chance to test. I copied this idea from the nfs code. The problem is that when the kernel clears flocks/plocks during close, it calls posix unlock even if there are no posix

[Cluster-devel] [PATCH 2/6] GFS2: use memchr_inv

2013-04-05 Thread Steven Whitehouse
From: Akinobu Mita akinobu.m...@gmail.com Use memchr_inv to verify that the specified memory range is cleared. Signed-off-by: Akinobu Mita akinobu.m...@gmail.com Cc: Steven Whitehouse swhit...@redhat.com Cc: cluster-devel@redhat.com Cc: Christine Caulfield ccaul...@redhat.com Cc: David Teigland

[Cluster-devel] [PATCH 3/6] GFS2: return error if malloc failed in gfs2_rs_alloc()

2013-04-05 Thread Steven Whitehouse
From: Wei Yongjun yongjun_...@trendmicro.com.cn The error code in gfs2_rs_alloc() is set to ENOMEM when error but never be used, instead, gfs2_rs_alloc() always return 0. Fix to return 'error'. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn Signed-off-by: Steven Whitehouse

[Cluster-devel] [PATCH 6/6] GFS2: Issue discards in 512b sectors

2013-04-05 Thread Steven Whitehouse
From: Bob Peterson rpete...@redhat.com This patch changes GFS2's discard issuing code so that it calls function sb_issue_discard rather than blkdev_issue_discard. The code was calling blkdev_issue_discard and specifying the correct sector offset and sector size, but blkdev_issue_discard expects

[Cluster-devel] [PATCH 4/6] GFS2: Fix unlock of fcntl locks during withdrawn state

2013-04-05 Thread Steven Whitehouse
When withdraw occurs, we need to continue to allow unlocks of fcntl locks to occur, however these will only be local, since the node has withdrawn from the cluster. This prevents triggering a VFS level bug trap due to locks remaining when a file is closed. Signed-off-by: Steven Whitehouse

[Cluster-devel] GFS2: Pull request (fixes)

2013-04-05 Thread Steven Whitehouse
Hi, Please consider pulling the following changes, Steve. There are two patches which fix up a couple of minor issues in the DLM interface code, a missing error path in gfs2_rs_alloc(), two patches which fix problems

Re: [Cluster-devel] GFS2: Pull request (fixes)

2013-04-05 Thread David Teigland
On Fri, Apr 05, 2013 at 11:34:45AM +0100, Steven Whitehouse wrote: Please consider pulling the following changes, There's some mixup here that should be cleared up first. David Teigland (2): GFS2: Fix unlock of fcntl locks during withdrawn state Steven Whitehouse (1): GFS2:

Re: [Cluster-devel] GFS2: Pull request (fixes)

2013-04-05 Thread Steven Whitehouse
Hi, On Fri, 2013-04-05 at 12:27 -0400, David Teigland wrote: On Fri, Apr 05, 2013 at 11:34:45AM +0100, Steven Whitehouse wrote: Please consider pulling the following changes, There's some mixup here that should be cleared up first. David Teigland (2): GFS2: Fix unlock of fcntl

Re: [Cluster-devel] GFS2: Pull request (fixes)

2013-04-05 Thread Steven Whitehouse
Hi, I've dropped out the patch which shouldn't have made it into the previous pull request, so this set should be ok now. Apologies for not spotting that issue sooner, Steve. - There are two patches which fix up a couple of

[Cluster-devel] [PATCH] GFS2: replace gfs2_ail structure with gfs2_trans

2013-04-05 Thread Benjamin Marzinski
In order to allow transactions and log flushes to happen at the same time, gfs2 needs to move the transaction accounting and active items list code into the gfs2_trans structure. As a first step toward this, this patch removes the gfs2_ail structure, and handles the active items list in the