October 2022 - Devel - Nfs Ganesha List Archives

[S] Change in ...nfs-ganesha[next]: libganesha_nfsd.ver: Export symbol convert_ipv6_to_ipv4

by Name of user not set (GerritHub)

shaharhoch(a)gmail.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545498 ) Change subject: libganesha_nfsd.ver: Export symbol convert_ipv6_to_ipv4 ...................................................................... libganesha_nfsd.ver: Export symbol convert_ipv6_to_ipv4 We need to use this function in our FSAL Change-Id: I6494ce00d55f3ed858e0d8576d2083a811635770 Signed-off-by: Shahar Hochma <shaharhoch(a)google.com> --- M src/MainNFSD/libganesha_nfsd.ver 1 file changed, 13 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/98/545498/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545498 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: I6494ce00d55f3ed858e0d8576d2083a811635770 Gerrit-Change-Number: 545498 Gerrit-PatchSet: 1 Gerrit-Owner: shaharhoch(a)gmail.com Gerrit-MessageType: newchange

10 months, 4 weeks

1
0
0 / 0

[S] Change in ...nfs-ganesha[next]: FSAL_PROXY_V3: Support POSIX attributes instead of ATTRS_NFS3

by Assaf Yaari (GerritHub)

Assaf Yaari has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545455 ) Change subject: FSAL_PROXY_V3: Support POSIX attributes instead of ATTRS_NFS3 ...................................................................... FSAL_PROXY_V3: Support POSIX attributes instead of ATTRS_NFS3 The only difference is the addition of ATTR_CHANGE which is mandatory in the NFSv4.1 spec. Without it, NFS clients missed directory modifications. Signed-off-by: Assaf Yaari <assafy(a)google.com> Change-Id: I0e9bc42aac7fa2506d7d3ae616a3897d80da3606 --- M src/FSAL/FSAL_PROXY_V3/main.c M src/FSAL/FSAL_PROXY_V3/proxyv3_fsal_methods.h M src/FSAL/FSAL_PROXY_V3/utils.c 3 files changed, 30 insertions(+), 10 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/55/545455/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545455 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: I0e9bc42aac7fa2506d7d3ae616a3897d80da3606 Gerrit-Change-Number: 545455 Gerrit-PatchSet: 1 Gerrit-Owner: Assaf Yaari <assafy(a)google.com> Gerrit-MessageType: newchange

11 months

1
0
0 / 0

Announce Push of V4.0.11

by Frank Filz

Branch next Tag:V4.0.11 Merge Highlights * Numerous fixes of Coverity issues * Enable Protocols config in EXPORT_DEFAULTS to work as expected * ganesha: Build with Modern C * Check whether request is a real replay to warn kernel's bug * NLM: nlm send async ipv4 handling. * Fix crash at inc/dec_state_t_ref by getting open_state from hashtable Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: adbb6ec2d Frank S. Filz V4.0.11 8effd5703 liyinshu Fix crash at inc/dec_state_t_ref by getting open_state from hashtable 6c1fb888f Gaurav Gangalwar NLM: nlm send async ipv4 handling. c5cf8ca1a liyinshu Check whether request is a real replay to warn kernel's bug 708ffe297 Kaleb S. KEITHLEY ganesha: Build with Modern C dedb81de0 Paul Moyer fixed coverity issue 356423 - improved handling of pidfile d192cd604 Paul Moyer fixed coverity issue 313020 - i_acl should be e_acl instead eeaa567a7 Frank S. Filz Enable Protocols config in EXPORT_DEFAULTS to work as expected b9a0be0b4 Frank S. Filz RGW: fix memory leak of stream and streambuf on error db3a91861 Frank S. Filz RGW: Be safe with allocations when parsing rados URL 0ff2d9259 Frank S. Filz RGW: Disable xattrs 51ef99e5c Frank S. Filz RGW: construct_handle only returns 0, consistently ignore return 685f7f9ec Frank S. Filz proxyv3_mknode: cleanly return ERR_FSAL_INVAL if nodetype is invalid d8f97cb79 Frank S. Filz get_nfsmon_export_io: remove dead code 8e4a167e5 Frank S. Filz Logging: fix a type issue with server epoch 1e4d3032c Frank S. Filz mdcache_populate_dir_chunk: state.prev_chunk can't be NULL 5ed3c7dbd Frank S. Filz config_parsing: remove extraneous parameters from config_proc_error call 3b507127a Frank S. Filz encode_posix_acl make count signed so error detection works b87dd87df Frank S. Filz state_lock: Check return code of HashTable_Del 355acb2fb Frank S. Filz WARNING - BROKEN CODE FOR EVENT_TAKE_NODEID - Make this a bit more obvious 369176ebe Frank S. Filz recovery_fs: Properly account for size of node sub-string 0b9e96a88 Frank S. Filz nfs_dupreq: release t->mtx if we bail out for in-progress dupe req. fe4c85ead Frank S. Filz NFSv3: Fix unitialized status 2399a8211 Frank S. Filz FSAL: Display ACE needs to use IS_FSAL_ACE_GROUP_ID if not IS_FSAL_ACE_SPECIAL_ID d27445911 Frank S. Filz VFS: u32 is a pair of uint32_t need to display with PRIx32 59a4e2f36 Frank S. Filz VFS: fix use of return of vfs_fsal_open and remove dead code 28e98a7b1 Frank S. Filz VFS: fix error path in vfs_get_root_fd that can leak fd 18fcbd279 Frank S. Filz FSAL: pds->fsal should not be NULL 1e3ae15f1 Frank S. Filz server_stats - clean up casting to double and division 4e2321b9d Frank S. Filz NFS3: Cleanup handing of immediate return of NFS_REQ_OK fddeec928 Frank S. Filz EXPORTS: Initialize NULL to old_fullpath and old_pseudopath

11 months

1
0
0 / 0

Support for xattrs, where client is macOS (darwin and ventura) .... and Ganesha runs on same machine

by David Rieber

In my system setup I have the NFS client and NFS server (Ganesha) running on the same machine, running macOS (darwin and soon ventura). Most things are working, but xattrs do not work. Does anybody has pointers on how to get this to work? I am aware of RFC8276 "File System Extended Attributes in NFSv4" and it looks like Ganesha has added support for RFC8276 (right?). But even after mounting the NFS filesystem with "namedattr" it looks like this does not work: "xattr -l README" returns empty output.

11 months

2
1
0 / 0

[M] Change in ...nfs-ganesha[next]: ganesha: Build with Modern C

by Kaleb KEITHLEY (GerritHub)

Kaleb KEITHLEY has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545319 ) Change subject: ganesha: Build with Modern C ...................................................................... ganesha: Build with Modern C GCC and Clang communities are hinting that in versions 14 and 16 respectively they will deprecate or disable legacy C89 features, e.g. K&R1 style function definitions, among others. In parallel, Fedora is going to start enforcing C99 as the minimum, expected to land in F40. (I.e. around Spring 2024 IIRC.) Currently Fedora is recommending that use of -Werror=implicit-int, -Werror=implicit-function-declaration, -Werror=int-conversion, -Werror=strict-prototypes, and -Werror=old-style-definition a set of options to build with in the mean time to clean up code. Change-Id: I94e1c6d1442524d4e9e594605b20479fb3d5fd62 Signed-off-by: Kaleb S. KEITHLEY <kkeithle(a)redhat.com> --- M src/Protocols/NFS/nfs3_readdir.c M src/Protocols/NFS/nfs3_readdirplus.c M src/Protocols/NFS/nfs_proto_tools.c M src/Protocols/XDR/xdr_mount.c 4 files changed, 52 insertions(+), 54 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/19/545319/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545319 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: I94e1c6d1442524d4e9e594605b20479fb3d5fd62 Gerrit-Change-Number: 545319 Gerrit-PatchSet: 1 Gerrit-Owner: Kaleb KEITHLEY <kaleb(a)redhat.com> Gerrit-MessageType: newchange

11 months

1
0
0 / 0

[L] Change in ...nfs-ganesha[next]: Enable Protocols config in EXPORT_DEFAULTS to work as expected

by Frank Filz (GerritHub)

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545233 ) Change subject: Enable Protocols config in EXPORT_DEFAULTS to work as expected ...................................................................... Enable Protocols config in EXPORT_DEFAULTS to work as expected An EXPORT need not have a Protocols config, in which case the value from EXPORT_DEFAULTS will be used and if that isn't present, then the value from NFS_CORE_PARAM will be used. Also fixup some checking of exports. Now that an export can be exported NFSv4 by default, we make it NOT an error to not have a pseudo path, instead we fix up the export to not be NFSv4. To make everything work, the Protocols in EXPORT needs to default to nothing. With this, if Protocols is not specified in CLIENT, EXPORT, or EXPORT_DEFAULTS, it will defer to the next layer up, ultimately to the NFS_CORE_PARAM Protocols, which defaults to NFSv3, NFSv4. Change-Id: If932a602b5dacd266ac2813f84936a97f8ba0f7e Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/Protocols/NFS/nfs4_pseudo.c M src/config_samples/config.txt M src/doc/man/ganesha-export-config.rst M src/include/export_mgr.h M src/support/exports.c 5 files changed, 262 insertions(+), 48 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/33/545233/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545233 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: If932a602b5dacd266ac2813f84936a97f8ba0f7e Gerrit-Change-Number: 545233 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

11 months

1
0
0 / 0

[M] Change in ...nfs-ganesha[next]: NFS: Add a config to limit the number of entries returned in readdir

by Frank Filz (GerritHub)

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545070 ) Change subject: NFS: Add a config to limit the number of entries returned in readdir ...................................................................... NFS: Add a config to limit the number of entries returned in readdir Also, fix up the maximum size of the response. Don't have the special value 0 any more and make the maximum 1 MB (which is close to what the Linux NFSv4 client asks for). Change-Id: I5e0c6a93003212efb0ec9ed9c9f62d5b90394bf3 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/Protocols/NFS/nfs3_readdir.c M src/Protocols/NFS/nfs3_readdirplus.c M src/Protocols/NFS/nfs4_op_readdir.c M src/config_samples/config.txt M src/doc/man/ganesha-core-config.rst M src/include/gsh_config.h M src/support/nfs_read_conf.c 7 files changed, 101 insertions(+), 21 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/70/545070/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545070 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: I5e0c6a93003212efb0ec9ed9c9f62d5b90394bf3 Gerrit-Change-Number: 545070 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

11 months, 1 week

1
0
0 / 0

[S] Change in ...nfs-ganesha[next]: recovery_rados_cluster: check for error on sysconf(_SC_HOST_NAME_MAX);

by Frank Filz (GerritHub)

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545030 ) Change subject: recovery_rados_cluster: check for error on sysconf(_SC_HOST_NAME_MAX); ...................................................................... recovery_rados_cluster: check for error on sysconf(_SC_HOST_NAME_MAX); Change-Id: I72154959163c7664796f7d8aa7ee993ae9838d06 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/SAL/recovery/recovery_rados_cluster.c 1 file changed, 18 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/30/545030/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545030 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: I72154959163c7664796f7d8aa7ee993ae9838d06 Gerrit-Change-Number: 545030 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

11 months, 1 week

1
0
0 / 0

[S] Change in ...nfs-ganesha[next]: RGW: fix memory leak of stream and streambuf on error

by Frank Filz (GerritHub)

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545029 ) Change subject: RGW: fix memory leak of stream and streambuf on error ...................................................................... RGW: fix memory leak of stream and streambuf on error Change-Id: Ib46bac10dea1e02782034d7d00a8c1c816b46e66 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/config_parsing/conf_url_rados.c 1 file changed, 21 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/29/545029/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545029 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: Ib46bac10dea1e02782034d7d00a8c1c816b46e66 Gerrit-Change-Number: 545029 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

11 months, 1 week

1
0
0 / 0

[S] Change in ...nfs-ganesha[next]: RGW: Be safe with allocations when parsing rados URL

by Frank Filz (GerritHub)

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545028 ) Change subject: RGW: Be safe with allocations when parsing rados URL ...................................................................... RGW: Be safe with allocations when parsing rados URL Just in case any of x1, x2, or x3 are not actually consumed, free them. Change-Id: Ib4264488ef6a4d8314a2acde6bb2aabae13575b1 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/config_parsing/conf_url_rados.c 1 file changed, 32 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/28/545028/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/545028 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: Ib4264488ef6a4d8314a2acde6bb2aabae13575b1 Gerrit-Change-Number: 545028 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

11 months, 1 week

1
0
0 / 0

2023

2022

2021

2020

2019

2018

Devel October 2022