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

2014-09-15 Thread Steven Whitehouse
Hi, Here are a number of small fixes for GFS2. There is a fix for FIEMAP on large sparse files, a negative dentry hashing fix, a fix for flock, and a bug fix relating to d_splice_alias usage. There are also (patches 1 and 5) a couple of updates which are less critical, but small and low risk.

[Cluster-devel] [PATCH 1/6] GFS2: fs/gfs2/super.c: replace seq_printf by seq_puts

2014-09-15 Thread Steven Whitehouse
From: Fabian Frederick f...@skynet.be fix checkpatch warnings: WARNING: Prefer seq_puts to seq_printf Cc: cluster-devel@redhat.com Signed-off-by: Fabian Frederick f...@skynet.be Signed-off-by: Steven Whitehouse swhit...@redhat.com diff --git a/fs/gfs2/super.c b/fs/gfs2/super.c index

[Cluster-devel] [PATCH 4/6] GFS2: Hash the negative dentry during inode lookup

2014-09-15 Thread Steven Whitehouse
From: Benjamin Coddington bcodd...@redhat.com Fix a regression introduced by: 6d4ade986f9c8df31e68 GFS2: Add atomic_open support where an early return misses d_splice_alias() which had been adding the negative dentry. Signed-off-by: Benjamin Coddington bcodd...@redhat.com Signed-off-by: Bob

[Cluster-devel] [PATCH 5/6] GFS2: Don't use MAXQUOTAS value

2014-09-15 Thread Steven Whitehouse
From: Jan Kara j...@suse.cz MAXQUOTAS value defines maximum number of quota types VFS supports. This isn't necessarily the number of types gfs2 supports and with addition of project quotas these two numbers stop matching. So make gfs2 use its private definition. CC: cluster-devel@redhat.com

[Cluster-devel] [PATCH 6/6] GFS2: fix d_splice_alias() misuses

2014-09-15 Thread Steven Whitehouse
From: Al Viro v...@zeniv.linux.org.uk Callers of d_splice_alias(dentry, inode) don't need iput(), neither on success nor on failure. Either the reference to inode is stored in a previously negative dentry, or it's dropped. In either case inode reference the caller used to hold is consumed.

[Cluster-devel] [PATCH 2/6] GFS2: Change maxlen variables to size_t

2014-09-15 Thread Steven Whitehouse
From: Bob Peterson rpete...@redhat.com This patch changes some variables (especially maxlen in function gfs2_block_map) from unsigned int to size_t. We need 64-bit arithmetic for very large files (e.g. 1PB) where the variables otherwise get shifted to all 0's. Signed-off-by: Bob Peterson