[Cluster-devel] [PATCH 2/2 linux-next] gfs2: remove unused flags parameter

2017-05-03 Thread Fabian Frederick
We already assumed rename flags when calling gfs2_exchange() Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/gfs2/inode.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/fs/gfs2/inode.c b/fs/gfs2/inode.c index 9698796..b5858a3 100644 --- a/fs/gfs2/inode.c

[Cluster-devel] [PATCH 1/1 linux-next] GFS2: remove inline on static in c file

2016-07-31 Thread Fabian Frederick
' +static void gfs2_update_stats(struct gfs2_lkstats *s, unsigned index, Inspired-by: "David S. Miller" <da...@davemloft.net> Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/gfs2/bmap.c | 15 --- fs/gfs2/dir.c | 4 ++-- fs/gfs2/glock.c

[Cluster-devel] [PATCH 1/1 linux-next] GFS2: use BIT() macro

2016-07-24 Thread Fabian Frederick
Replace 1 << value shift by more explicit BIT() macro Also fixes two bare unsigned definitions: WARNING: Prefer 'unsigned int' to bare use of 'unsigned' + unsigned hsize = BIT(ip->i_depth); Signed-off-by: Fabian Frederick <f...@skynet.be> --- fs/gfs2/aops.c |

[Cluster-devel] [PATCH V3 linux-next] gfs2: convert simple_str to kstr

2015-05-05 Thread Fabian Frederick
-Remove obsolete simple_str functions. -Return error code when kstr failed. -This patch also calls functions corresponding to destination type. Thanks to Alexey Dobriyan for suggesting improvements in block_store() and wdack_store() Signed-off-by: Fabian Frederick f...@skynet.be --- V3: -Compare

Re: [Cluster-devel] [PATCH V2 linux-next] gfs2: convert simple_str to kstr

2015-05-05 Thread Fabian Frederick
Dobriyan for suggesting improvements in block_store() and wdack_store() Signed-off-by: Fabian Frederick f...@skynet.be ---   fs/gfs2/sys.c | 69   ++-   1 file changed, 49 insertions(+), 20 deletions(-) diff --git a/fs/gfs2/sys.c b/fs

[Cluster-devel] [PATCH 1/1 linux-next] gfs2: kerneldoc warning fixes

2015-05-02 Thread Fabian Frederick
): No description found for parameter 'copied' Signed-off-by: Fabian Frederick f...@skynet.be --- fs/gfs2/aops.c | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/fs/gfs2/aops.c b/fs/gfs2/aops.c index 5551fea..1caee05 100644 --- a/fs/gfs2/aops.c +++ b/fs/gfs2/aops.c

[Cluster-devel] [PATCH V2 linux-next] gfs2: convert simple_str to kstr

2015-04-30 Thread Fabian Frederick
-Remove obsolete simple_str functions. -Return error code when kstr failed. -This patch also calls functions corresponding to destination type. Thanks to Alexey Dobriyan for suggesting improvements in block_store() and wdack_store() Signed-off-by: Fabian Frederick f...@skynet.be --- fs/gfs2

[Cluster-devel] [RFC 1/1 linux-next] gfs2: convert simple_str to kstr

2015-04-29 Thread Fabian Frederick
such problems. By default I used destination type as a reference. Maybe it's wrong and we really want to read long, unsigned long from source ? Signed-off-by: Fabian Frederick f...@skynet.be --- fs/gfs2/sys.c | 56 1 file changed, 44 insertions

[Cluster-devel] [PATCH 1/1 linux-next] GFS2: directly return gfs2_dir_check()

2014-10-09 Thread Fabian Frederick
No need to store gfs2_dir_check result and test it before returning. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/gfs2/inode.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/fs/gfs2/inode.c b/fs/gfs2/inode.c index c4ed823..b41b5c7 100644 --- a/fs/gfs2/inode.c

[Cluster-devel] [PATCH 1/1] GFS2: use _RET_IP_ instead of (unsigned long)__builtin_return_address(0)

2014-10-03 Thread Fabian Frederick
use macro definition Signed-off-by: Fabian Frederick f...@skynet.be --- fs/gfs2/glock.c | 4 ++-- fs/gfs2/glops.c | 2 +- fs/gfs2/trans.c | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/fs/gfs2/glock.c b/fs/gfs2/glock.c index 7f513b1..8f0c19d 100644 --- a/fs/gfs2/glock.c

[Cluster-devel] [PATCH 1/1] fs/gfs2/rgrp.c: kernel-doc warning fixes

2014-07-02 Thread Fabian Frederick
Cc: Steven Whitehouse swhit...@redhat.com Cc: cluster-devel@redhat.com Signed-off-by: Fabian Frederick f...@skynet.be --- fs/gfs2/rgrp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/gfs2/rgrp.c b/fs/gfs2/rgrp.c index db629d1..f4cb9c0 100644 --- a/fs/gfs2/rgrp.c +++ b

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

2014-07-02 Thread Fabian Frederick
fix checkpatch warnings: WARNING: Prefer seq_puts to seq_printf Cc: Steven Whitehouse swhit...@redhat.com Cc: cluster-devel@redhat.com Signed-off-by: Fabian Frederick f...@skynet.be --- fs/gfs2/super.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/fs

[Cluster-devel] [PATCH 1/1] fs/dlm/debug_fs.c: remove unnecessary null test before debugfs_remove

2014-06-26 Thread Fabian Frederick
This fixes checkpatch warning: WARNING: debugfs_remove(NULL) is safe this check is probably not required Cc: Christine Caulfield ccaul...@redhat.com Cc: cluster-devel@redhat.com Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- fs/dlm/debug_fs.c | 15

[Cluster-devel] [PATCH 1/1] fs/gfs2/lock_dlm.c: replace count*size kzalloc by kcalloc

2014-06-25 Thread Fabian Frederick
kcalloc manages count*sizeof overflow. Cc: Steven Whitehouse swhit...@redhat.com Cc: cluster-devel@redhat.com Signed-off-by: Fabian Frederick f...@skynet.be --- fs/gfs2/lock_dlm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/gfs2/lock_dlm.c b/fs/gfs2/lock_dlm.c