November 2022 - Devel - NFS Ganesha List Archives

[M] Change in ...nfs-ganesha[next]: Add long term obj handle (ultimately mdcache) references
by Frank Filz (GerritHub) 08 Nov '22

08 Nov '22

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545906 ) Change subject: Add long term obj handle (ultimately mdcache) references ...................................................................... Add long term obj handle (ultimately mdcache) references Change-Id: Ida17075b35a2fde6a76af233bd6462ff48485310 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_handle.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_int.h M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.h M src/FSAL/default_methods.c M src/include/fsal_api.h M src/include/gsh_lttng/mdcache.h 7 files changed, 214 insertions(+), 15 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/06/545906/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545906 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Ida17075b35a2fde6a76af233bd6462ff48485310 Gerrit-Change-Number: 545906 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

[L] Change in ...nfs-ganesha[next]: MDCACHE: Clean up refcounting and LRU
by Frank Filz (GerritHub) 08 Nov '22

08 Nov '22

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545905 ) Change subject: MDCACHE: Clean up refcounting and LRU ...................................................................... MDCACHE: Clean up refcounting and LRU Streamline refcounting so all callers use mdcache_lru_ref and unref. Promote from L2 to MRU of L1 (old code promoted to LRU of L1 which was problematic). Eliminate unused and confusing flags and return values. Change-Id: I8e3fcbcb4f4bf3c4ee4e762cea814fd375b9801d 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_export.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_file.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_handle.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_hash.h M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_helpers.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_int.h M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.h M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_up.c 10 files changed, 141 insertions(+), 191 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/05/545905/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545905 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I8e3fcbcb4f4bf3c4ee4e762cea814fd375b9801d Gerrit-Change-Number: 545905 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

[L] Change in ...nfs-ganesha[next]: INCOMPLETE FSAL_GLUSTER - Convert to new fsal_fd management
by Frank Filz (GerritHub) 08 Nov '22

08 Nov '22

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545904 ) Change subject: INCOMPLETE FSAL_GLUSTER - Convert to new fsal_fd management ...................................................................... INCOMPLETE FSAL_GLUSTER - Convert to new fsal_fd management This code has some work still to be done, but we need to move forward to some degree. Change-Id: I55816931e0af8f7079efb2f70b5a944056a1e963 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/FSAL/FSAL_GLUSTER/export.c M src/FSAL/FSAL_GLUSTER/gluster_internal.h M src/FSAL/FSAL_GLUSTER/handle.c 3 files changed, 358 insertions(+), 348 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/04/545904/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545904 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I55816931e0af8f7079efb2f70b5a944056a1e963 Gerrit-Change-Number: 545904 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

[L] Change in ...nfs-ganesha[next]: INCOMPLETE - FSAL_LIZARDFS - Convert to new fsal_fd management
by Frank Filz (GerritHub) 08 Nov '22

08 Nov '22

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545903 ) Change subject: INCOMPLETE - FSAL_LIZARDFS - Convert to new fsal_fd management ...................................................................... INCOMPLETE - FSAL_LIZARDFS - Convert to new fsal_fd management This conversion may be incomplete. Unfortunately it must be made so we can remove the old fsal_fd management code. For now, this FSAL compiles and is being included, however if maintanence becomes too much of an issue, it may be deprecated and removed. Change-Id: I4a0588f2c96c9538eaff9e9d1c2abddf2003941a Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/FSAL/FSAL_LIZARDFS/handle.c M src/FSAL/FSAL_LIZARDFS/lzfs_internal.h 2 files changed, 429 insertions(+), 274 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/03/545903/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545903 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I4a0588f2c96c9538eaff9e9d1c2abddf2003941a Gerrit-Change-Number: 545903 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

[L] Change in ...nfs-ganesha[next]: FSAL_KVSFS: Use new fsal_fd management
by Frank Filz (GerritHub) 08 Nov '22

08 Nov '22

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545902 ) Change subject: FSAL_KVSFS: Use new fsal_fd management ...................................................................... FSAL_KVSFS: Use new fsal_fd management Change-Id: Ib0a5d1804bf4c0561bb516884f761a186b587ce2 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/FSAL/FSAL_KVSFS/kvsfs_export.c M src/FSAL/FSAL_KVSFS/kvsfs_file.c M src/FSAL/FSAL_KVSFS/kvsfs_handle.c M src/FSAL/FSAL_KVSFS/kvsfs_methods.h 4 files changed, 304 insertions(+), 453 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/02/545902/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545902 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Ib0a5d1804bf4c0561bb516884f761a186b587ce2 Gerrit-Change-Number: 545902 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

[L] Change in ...nfs-ganesha[next]: FSAL_GPFS: Use new fsal_fd management
by Frank Filz (GerritHub) 08 Nov '22

08 Nov '22

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545901 ) Change subject: FSAL_GPFS: Use new fsal_fd management ...................................................................... FSAL_GPFS: Use new fsal_fd management Also implement seek2 instead of seek Change-Id: I3369996f83ca29a737b8697192a21752c4a7b534 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/FSAL/FSAL_GPFS/export.c M src/FSAL/FSAL_GPFS/file.c M src/FSAL/FSAL_GPFS/fsal_internal.h M src/FSAL/FSAL_GPFS/gpfs_methods.h M src/FSAL/FSAL_GPFS/handle.c 5 files changed, 393 insertions(+), 315 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/01/545901/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545901 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I3369996f83ca29a737b8697192a21752c4a7b534 Gerrit-Change-Number: 545901 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

[M] Change in ...nfs-ganesha[next]: FSAL_RGW: Don't use old fsal_find_fd just do share stuff
by Frank Filz (GerritHub) 08 Nov '22

08 Nov '22

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545900 ) Change subject: FSAL_RGW: Don't use old fsal_find_fd just do share stuff ...................................................................... FSAL_RGW: Don't use old fsal_find_fd just do share stuff We were only calling fsal_find_fd for share reservation stuff and then not cleaning it up. Just directly do share reservation stuff. FSAL_RGW doesn't need any fsal_fd stuff. Change-Id: I7236e1a5cc8621043e85cad5166e78129c9f785b Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/FSAL/FSAL_RGW/handle.c 1 file changed, 49 insertions(+), 19 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/00/545900/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545900 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I7236e1a5cc8621043e85cad5166e78129c9f785b Gerrit-Change-Number: 545900 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

[S] Change in ...nfs-ganesha[next]: build: use CMAKE_DL_LIBS
by Adrian Freihofer (GerritHub) 07 Nov '22

07 Nov '22

Adrian Freihofer has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545824 ) Change subject: build: use CMAKE_DL_LIBS ...................................................................... build: use CMAKE_DL_LIBS CMake Error: The following variables are used in this project, but they are set to NOTFOUND. | Please set them or make sure they are set and tested correctly in the CMake files: | LIBDL | linked by target... | Change-Id: Ie574e2020fa8fe90e7e37cd9530f6d6f0c8b9ea2 Signed-off-by: Adrian Freihofer <adrian.freihofer(a)siemens.com> Signed-off-by: Adrian Freihofer <adrian.freihofer(a)gmail.com> --- M src/CMakeLists.txt 1 file changed, 18 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/24/545824/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545824 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Ie574e2020fa8fe90e7e37cd9530f6d6f0c8b9ea2 Gerrit-Change-Number: 545824 Gerrit-PatchSet: 1 Gerrit-Owner: Adrian Freihofer <adrian.freihofer(a)gmail.com> Gerrit-MessageType: newchange

1 0

[S] Change in ...nfs-ganesha[next]: Merge branch 'next' of ssh://review.gerrithub.io:29418/ffilz/nfs-gane...
by Name of user not set (GerritHub) 06 Nov '22

06 Nov '22

1140520975(a)qq.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545765 ) Change subject: Merge branch 'next' of ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha into next ...................................................................... Merge branch 'next' of ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha into next Signed-off-by: iamtiger51 <1140520975(a)qq.com> Change-Id: I137c78ea57603f7498cff3ea0fc83e8a8cca304f --- M src/CMakeLists.txt 2 files changed, 14 insertions(+), 4 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/65/545765/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545765 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I137c78ea57603f7498cff3ea0fc83e8a8cca304f Gerrit-Change-Number: 545765 Gerrit-PatchSet: 1 Gerrit-Owner: 1140520975(a)qq.com Gerrit-MessageType: newchange

1 0

[M] Change in ...nfs-ganesha[next]: V4.0.12
by Name of user not set (GerritHub) 06 Nov '22

06 Nov '22

Attention is currently required from: Frank Filz. Hello Frank Filz, I'd like you to do a code review. Please visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545764 to review the following change. Change subject: V4.0.12 ...................................................................... V4.0.12 Change-Id: I743c81921f81bbd2677de11e50023a3c4f44c69a Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/CMakeLists.txt M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_ext.h 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 M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_read_conf.c M src/doc/man/ganesha-cache-config.rst 7 files changed, 80 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/64/545764/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545764 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I743c81921f81bbd2677de11e50023a3c4f44c69a Gerrit-Change-Number: 545764 Gerrit-PatchSet: 1 Gerrit-Owner: 1140520975(a)qq.com Gerrit-Reviewer: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-Attention: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0