[Cluster-devel] [PATCH] dlm: remove unnecessary error check

2015-06-09 Thread Guoqing Jiang
We don't need the redundant logic since send_message always returns 0. Signed-off-by: Guoqing Jiang --- fs/dlm/lock.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/fs/dlm/lock.c b/fs/dlm/lock.c index 35502d4..6fc3de9 100644 --- a/fs/dlm/lock.c +++ b/fs/dlm/lock.c

Re: [Cluster-devel] [PATCH] dlm: remove unnecessary error check

2015-06-09 Thread Bob Peterson
- Original Message - > We don't need the redundant logic since send_message always returns 0. > > Signed-off-by: Guoqing Jiang > --- > fs/dlm/lock.c | 10 ++ > 1 file changed, 2 insertions(+), 8 deletions(-) > > diff --git a/fs/dlm/lock.c b/fs/dlm/lock.c > index 35502d4..6fc3de9

Re: [Cluster-devel] [GFS2 PATCH] gfs2: Don't support fallocate on jdata files

2015-06-09 Thread Bob Peterson
- Original Message - > We cannot provide an efficient implementation due to the headers > on the data blocks, so there doesn't seem much point in having it. > > Resolves: rhbz#1221331 > Signed-off-by: Abhi Das > --- > fs/gfs2/file.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [Cluster-devel] [GFS2 PATCH] GFS2: Don't brelse rgrp buffer_heads every allocation

2015-06-09 Thread Bob Peterson
- Original Message - > Hi, > > > On 05/06/15 15:49, Bob Peterson wrote: > > Hi, > > > > This patch allows the block allocation code to retain the buffers > > for the resource groups so they don't need to be re-read from buffer > > cache with every request. This is a performance improvemen

Re: [Cluster-devel] [PATCH] dlm: remove unnecessary error check

2015-06-09 Thread Guoqing Jiang
Hi Bob, Bob Peterson wrote: > - Original Message - > >> We don't need the redundant logic since send_message always returns 0. >> >> Signed-off-by: Guoqing Jiang >> --- >> fs/dlm/lock.c | 10 ++ >> 1 file changed, 2 insertions(+), 8 deletions(-) >> >> diff --git a/fs/dlm/lock.

Re: [Cluster-devel] [PATCH] dlm: remove unnecessary error check

2015-06-09 Thread Bob Peterson
- Original Message - > Hi Bob, > > Bob Peterson wrote: > > - Original Message - > > > >> We don't need the redundant logic since send_message always returns 0. > >> > >> Signed-off-by: Guoqing Jiang > >> --- > >> fs/dlm/lock.c | 10 ++ > >> 1 file changed, 2 insertions

Re: [Cluster-devel] [PATCH] dlm: remove unnecessary error check

2015-06-09 Thread Guoqing Jiang
Bob Peterson wrote: > - Original Message - > >> Hi Bob, >> >> Bob Peterson wrote: >> >>> - Original Message - >>> >>> We don't need the redundant logic since send_message always returns 0. Signed-off-by: Guoqing Jiang --- fs/dlm/lock.c |