Change in ffilz/nfs-ganesha[next]: MDCACHE: make all dirent related functions use LogDebugAlt/LogFullDeb...
by GerritHub
From Frank Filz <ffilzlnx(a)mindspring.com>:
Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/416265
Change subject: MDCACHE: make all dirent related functions use LogDebugAlt/LogFullDebugAlt
......................................................................
MDCACHE: make all dirent related functions use LogDebugAlt/LogFullDebugAlt
Change-Id: Ie424eb21c3bb91b12247e62c1cef7ce807953214
Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_avl.c
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_helpers.c
2 files changed, 194 insertions(+), 171 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/65/416265/1
--
To view, visit https://review.gerrithub.io/416265
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie424eb21c3bb91b12247e62c1cef7ce807953214
Gerrit-Change-Number: 416265
Gerrit-PatchSet: 1
Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>
6 years, 5 months
Change in ffilz/nfs-ganesha[next]: MDCACHE: For dirent chunking, prevent prev_chunk from being reaped
by GerritHub
From Frank Filz <ffilzlnx(a)mindspring.com>:
Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/416227
Change subject: MDCACHE: For dirent chunking, prevent prev_chunk from being reaped
......................................................................
MDCACHE: For dirent chunking, prevent prev_chunk from being reaped
Change-Id: I16d021e103a67315c4c3c60c68903079030ef221
Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_helpers.c
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.h
3 files changed, 32 insertions(+), 26 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/27/416227/1
--
To view, visit https://review.gerrithub.io/416227
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: I16d021e103a67315c4c3c60c68903079030ef221
Gerrit-Change-Number: 416227
Gerrit-PatchSet: 1
Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>
6 years, 5 months
Change in ffilz/nfs-ganesha[next]: MDCACHE: Add some next_ck debugging for dirent chunks
by GerritHub
From Frank Filz <ffilzlnx(a)mindspring.com>:
Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/416226
Change subject: MDCACHE: Add some next_ck debugging for dirent chunks
......................................................................
MDCACHE: Add some next_ck debugging for dirent chunks
Change-Id: I0048811ccddf6cf98f4912dc6de24bc2eccf18af
Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_helpers.c
1 file changed, 23 insertions(+), 0 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/26/416226/1
--
To view, visit https://review.gerrithub.io/416226
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: I0048811ccddf6cf98f4912dc6de24bc2eccf18af
Gerrit-Change-Number: 416226
Gerrit-PatchSet: 1
Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>
6 years, 5 months
state_lock detects conflict independetly of the async lock opt
by ntvietvn@gmail.com
Hello,
While reading the state_lock() fct i see this comment
/* We have already returned if:
* + we have found an identical blocking lock
* + we have found an entirely overlapping lock with the same lock owner
* + this was not a supported blocking lock and we found a conflict
*
* So at this point, we are either going to do one of the following (all
* descriptions below assume no problems occur):
*
* (1) FSAL supports async blocking locks, we know there is a conflict,
( and this is a supported blocking lock request
*
* Make FSAL_OP_LOCKB call anyway, we will rely on FSAL to grant
* blocking locks. We will return the conflict we know about rather
* than what the FSAL returns. Insert blocking lock into queue.
It seems that it is contradictory to the code above the comment which is in case of conflict lock, the fct returns immediately and does not make any call to FSAL, even if it supports async blocking lock.
Can you clarify this point?
Thank you
6 years, 5 months
Change in ffilz/nfs-ganesha[next]: FSAL_GLUSTER : extend support for glfs_ftruncate and glfs_fsync
by GerritHub
From Jiffin Tony Thottan <jthottan(a)redhat.com>:
Jiffin Tony Thottan has uploaded this change for review. ( https://review.gerrithub.io/416109
Change subject: FSAL_GLUSTER : extend support for glfs_ftruncate and glfs_fsync
......................................................................
FSAL_GLUSTER : extend support for glfs_ftruncate and glfs_fsync
The https://github.com/gluster/glusterfs/issues/389 add support to fetch stat in fsync and
truncate call in gfapi.
Change-Id: I5db1d36ab3f98119f5de43792d7ce560a2e3dda8
Signed-off-by: Jiffin Tony Thottan <jthottan(a)redhat.com>
---
M src/FSAL/FSAL_GLUSTER/ds.c
M src/FSAL/FSAL_GLUSTER/handle.c
M src/tools/multilock/ml_glusterfs_client.c
3 files changed, 4 insertions(+), 4 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/09/416109/1
--
To view, visit https://review.gerrithub.io/416109
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5db1d36ab3f98119f5de43792d7ce560a2e3dda8
Gerrit-Change-Number: 416109
Gerrit-PatchSet: 1
Gerrit-Owner: Jiffin Tony Thottan <jthottan(a)redhat.com>
6 years, 5 months
Change in ffilz/nfs-ganesha[next]: SAL - Allow v4 blocking locks
by GerritHub
From Daniel Gryniewicz <dang(a)redhat.com>:
Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/416029
Change subject: SAL - Allow v4 blocking locks
......................................................................
SAL - Allow v4 blocking locks
Change-Id: I88b9ade4b3f9e66f3f9693bdcfd3c5f5efb819ce
Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com>
---
M src/SAL/state_lock.c
M src/include/sal_data.h
2 files changed, 7 insertions(+), 1 deletion(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/29/416029/1
--
To view, visit https://review.gerrithub.io/416029
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: I88b9ade4b3f9e66f3f9693bdcfd3c5f5efb819ce
Gerrit-Change-Number: 416029
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>
6 years, 5 months
Change in ffilz/nfs-ganesha[next]: MEM - Limit dirent readahead to 1 chunk
by GerritHub
From Daniel Gryniewicz <dang(a)redhat.com>:
Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/416028
Change subject: MEM - Limit dirent readahead to 1 chunk
......................................................................
MEM - Limit dirent readahead to 1 chunk
Change-Id: I5e9b95e1dd657f766e5bf28d8ee88cba2f2739ee
Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com>
---
M src/FSAL/FSAL_MEM/mem_handle.c
1 file changed, 7 insertions(+), 0 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/28/416028/1
--
To view, visit https://review.gerrithub.io/416028
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5e9b95e1dd657f766e5bf28d8ee88cba2f2739ee
Gerrit-Change-Number: 416028
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>
6 years, 5 months
Change in ffilz/nfs-ganesha[next]: specfile: make ganesha-rados-grace part of base package
by GerritHub
From Jeff Layton <jlayton(a)redhat.com>:
Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/416027
Change subject: specfile: make ganesha-rados-grace part of base package
......................................................................
specfile: make ganesha-rados-grace part of base package
The RADOS recovery backend code is compiled into the main binary, so
we should package ganesha-rados-grace with it.
Change-Id: I7f607b0fea7a720aa0c8a6be31bc18e1caeb68a7
Signed-off-by: Jeff Layton <jlayton(a)redhat.com>
---
M src/nfs-ganesha.spec-in.cmake
1 file changed, 11 insertions(+), 3 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/27/416027/1
--
To view, visit https://review.gerrithub.io/416027
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: I7f607b0fea7a720aa0c8a6be31bc18e1caeb68a7
Gerrit-Change-Number: 416027
Gerrit-PatchSet: 1
Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>
6 years, 5 months
Change in ffilz/nfs-ganesha[next]: rados-grace-tool: underscore to dash conversion
by GerritHub
From Jeff Layton <jlayton(a)redhat.com>:
Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/416026
Change subject: rados-grace-tool: underscore to dash conversion
......................................................................
rados-grace-tool: underscore to dash conversion
Sage W. pointed out that we usually use dashes in command names, rather
than underscores, so switch to using that. While we're in there, let's
make it clear that this is for ganesha, so rename it to
ganesha-rados-grace-tool.
Change-Id: I26281450f71943c50d0c5ee80f69b9287cb94a36
Signed-off-by: Jeff Layton <jlayton(a)redhat.com>
---
M src/doc/man/CMakeLists.txt
R src/doc/man/ganesha-rados-grace.rst
M src/doc/man/index.rst
M src/nfs-ganesha.spec-in.cmake
M src/tools/CMakeLists.txt
R src/tools/ganesha-rados-grace.c
6 files changed, 15 insertions(+), 15 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/26/416026/1
--
To view, visit https://review.gerrithub.io/416026
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: I26281450f71943c50d0c5ee80f69b9287cb94a36
Gerrit-Change-Number: 416026
Gerrit-PatchSet: 1
Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>
6 years, 5 months
Change in ffilz/nfs-ganesha[next]: config_samples: remove obsolete config params from sample ceph.conf
by GerritHub
From Jeff Layton <jlayton(a)redhat.com>:
Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/416025
Change subject: config_samples: remove obsolete config params from sample ceph.conf
......................................................................
config_samples: remove obsolete config params from sample ceph.conf
These config parameters were removed recently. Remove them from the
sample config as well.
Change-Id: I482b740804f2b2a3b7c6b24c9a7ae4daefb9ea3b
Signed-off-by: Jeff Layton <jlayton(a)redhat.com>
---
M src/config_samples/ceph.conf
1 file changed, 0 insertions(+), 4 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/25/416025/1
--
To view, visit https://review.gerrithub.io/416025
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: I482b740804f2b2a3b7c6b24c9a7ae4daefb9ea3b
Gerrit-Change-Number: 416025
Gerrit-PatchSet: 1
Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>
6 years, 5 months