Change in ...nfs-ganesha[next]: FSAL: Implement Linux replacements for POSIX ACL FD get/set functions
by Martin Schwenke (GerritHub)
Martin Schwenke has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536254 )
Change subject: FSAL: Implement Linux replacements for POSIX ACL FD get/set functions
......................................................................
FSAL: Implement Linux replacements for POSIX ACL FD get/set functions
These functions exist on FreeBSD (and probably other platforms),
allowing default ACLs to be set via a file descriptor. libacl on
Linux does not have this, so implement a replacement.
Signed-off-by: Martin Schwenke <mschwenke(a)ddn.com>
Change-Id: Ib9fc0d95c21742b43ec25780d5fcb46d50fa9041
---
M src/FSAL/posix_acls.c
M src/include/posix_acls.h
2 files changed, 105 insertions(+), 0 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/54/536254/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536254
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: Ib9fc0d95c21742b43ec25780d5fcb46d50fa9041
Gerrit-Change-Number: 536254
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Schwenke <martin(a)meltin.net>
Gerrit-MessageType: newchange
2 years, 5 months
Change in ...nfs-ganesha[next]: build: Add checks for POSIX ACL FD get/set extensions
by Martin Schwenke (GerritHub)
Martin Schwenke has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536253 )
Change subject: build: Add checks for POSIX ACL FD get/set extensions
......................................................................
build: Add checks for POSIX ACL FD get/set extensions
These are FreeBSD functions that will need to be replaced on Linux to
allow setting/getting default ACLs.
Signed-off-by: Martin Schwenke <mschwenke(a)ddn.com>
Change-Id: I60a2242ece073e3d43cc40e76bd17fac02dd4c0a
---
M src/CMakeLists.txt
M src/include/config-h.in.cmake
2 files changed, 20 insertions(+), 0 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/53/536253/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536253
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: I60a2242ece073e3d43cc40e76bd17fac02dd4c0a
Gerrit-Change-Number: 536253
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Schwenke <martin(a)meltin.net>
Gerrit-MessageType: newchange
2 years, 5 months
Change in ...nfs-ganesha[next]: build: Add check for libacl
by Martin Schwenke (GerritHub)
Martin Schwenke has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536252 )
Change subject: build: Add check for libacl
......................................................................
build: Add check for libacl
While this is useful on its own, it seems to be required for a
subsequent commit that uses check_symbol_exists().
Unfortunately, this triggers a developer warning about this policy:
https://cmake.org/cmake/help/latest/policy/CMP0075.html#policy:CMP0075
This seems to suggest that it may no longer work with cmake 3.23.1.
My testing is with cmake 3.23.0 (Debian testing), 2.8.12.2 (CentOS 7)
and 3.22.2 (FreeBSD 13).
Signed-off-by: Martin Schwenke <mschwenke(a)ddn.com>
Change-Id: Icfbc76117298a60cb106950c80311d483251e9e9
---
M src/CMakeLists.txt
1 file changed, 26 insertions(+), 0 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/52/536252/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536252
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: Icfbc76117298a60cb106950c80311d483251e9e9
Gerrit-Change-Number: 536252
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Schwenke <martin(a)meltin.net>
Gerrit-MessageType: newchange
2 years, 5 months
Change in ...nfs-ganesha[next]: build: Improve POSIX ACL include file checking
by Martin Schwenke (GerritHub)
Martin Schwenke has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536251 )
Change subject: build: Improve POSIX ACL include file checking
......................................................................
build: Improve POSIX ACL include file checking
Linux libacl has both sys/acl.h and acl/libacl.h, while FreeBSD has
only sys/acl.h. Check separately for both of these and use the
appropriate cache variable in tests to improve readability.
Includes related to FSAL_CEPH and USE_NFSACL3 already indirectly
depend on HAVE_ACL_LIBACL_H, so nothing needs to change there.
It might be possible to allow USE_NFSACL3 on FreeBSD, with just
sys/acl.h rather than acl/libacl.h, but consider solving that later.
Signed-off-by: Martin Schwenke <mschwenke(a)ddn.com>
Change-Id: Ia22beffab4cf77275eaa50410a3c05194c30272e
---
M src/CMakeLists.txt
1 file changed, 30 insertions(+), 9 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/51/536251/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536251
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: Ia22beffab4cf77275eaa50410a3c05194c30272e
Gerrit-Change-Number: 536251
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Schwenke <martin(a)meltin.net>
Gerrit-MessageType: newchange
2 years, 5 months
Announce Push of V4.0.4
by Frank Filz
Sorry about the long delay since V4.0.3
Branch next
Tag:V4.0.4
NOTE: Please make sure the checkpatch.pl that you run on commit is updated
Merge Highlights
* New version of checkpatch.pl from kernel
* Fixups for new checkpatch
* Add SPDX licence tags to most files, also add some license text to files
* Fix typos found by codespell
* Delete some obsolete files
* FSAL: Fix regression in POSIX ACL mapping code
* systemd: eliminate a (benign?) error when enabling nfs-ganesha.service
* Fix release dupreq_entry_t before access.
* Exit immediately when LogFatal
Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com>
Contents:
38c545e68 Frank S. Filz V4.0.4
5d23a4d1b liyinshu Exit immediately when LogFatal
2ec3d6497 Gaurav Gangalwar Fix release dupreq_entry_t before access.
52372fa44 Kaleb S. KEITHLEY systemd: eliminate a (benign?) error when
enabling nfs-ganesha.service
53721b6de Martin Schwenke FSAL: Fix regression in POSIX ACL mapping code
9fdedc180 Dimitri Papadopoulos Orfanos Delete obsolete files
af948e11c Dimitri Papadopoulos Orfanos Fix typos not found by codespell
e49a07bdf Dimitri Papadopoulos Orfanos MINOR update to the latest version of
LGPL v3
42b4c8d49 Dimitri Papadopoulos Orfanos Fix typos found by codespell
41955f6f3 Frank S. Filz Based on src/cmake/COPYING-CMAKE-SCRIPTS add
BSD-3-Clause license to cmake files
f7fb407ce Frank S. Filz Add SPDX Licence Identifier to several cmake files
c1b5a0631 Frank S. Filz Add missing LGPL license and SPDX license identifier
to CMakeLists.txt files
2c0ab5d0f Frank S. Filz Remove some useless files
ecc5528d9 Frank S. Filz Add SPDX Licence Identifier to source files with
unknown license
2d1452da7 Frank S. Filz Add missing license to original files also add SPDX
licence tag
94c9969f0 Frank S. Filz Add missing LGPL license and SPDX license identifier
to contributed files
d41b67245 Frank S. Filz Add missing license to files to match files they
were contributed with
ba37ffe7b Frank S. Filz Add license to support/strnlen.c
f8dc7eae5 Frank S. Filz FSAL: Add missing license to files derrived from
other files
759a00c1a Frank S. Filz cidr: Add license to all source files from LICENSE
file
b096b94d8 Frank S. Filz Add GPL-2.0 or later with special exception to
Red-Black Tree headers
6fbe0f663 Frank S. Filz Add SPDX Licence Identifier to match the license in
the source files
272c895aa Frank S. Filz checkpatch fixups
57215287f Frank S. Filz Update checkpatch.pl
2 years, 5 months
Change in ...nfs-ganesha[next]: FSAL_MDCACHE: add parameter `Try_Force_Reuse_Entry`
by vicente (GerritHub)
vicente has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536037 )
Change subject: FSAL_MDCACHE: add parameter `Try_Force_Reuse_Entry`
......................................................................
FSAL_MDCACHE: add parameter `Try_Force_Reuse_Entry`
- That parameter could make reclaim early not matter
the number of entire entries
Change-Id: I4c57f033e16f2b9d3b6560ae0b732ad692b63393
Signed-off-by: Vicente Cheng <vicente_cheng(a)bigtera.com>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_read_conf.c
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_ext.h
3 files changed, 28 insertions(+), 4 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/37/536037/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536037
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: I4c57f033e16f2b9d3b6560ae0b732ad692b63393
Gerrit-Change-Number: 536037
Gerrit-PatchSet: 1
Gerrit-Owner: vicente <freeze.bilsted(a)gmail.com>
Gerrit-MessageType: newchange
2 years, 5 months
Change in ...nfs-ganesha[next]: FSAL_MDCACHE: avoid redundant QLOCK acquiring
by vicente (GerritHub)
vicente has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536036 )
Change subject: FSAL_MDCACHE: avoid redundant QLOCK acquiring
......................................................................
FSAL_MDCACHE: avoid redundant QLOCK acquiring
- also fix qid incorrect when adjust lru from L2 to L1
Change-Id: I9ae8de4af78eb2916862363e5785e7fe333831f0
Signed-off-by: Vicente Cheng <vicente_cheng(a)bigtera.com>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c
1 file changed, 24 insertions(+), 7 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/36/536036/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536036
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: I9ae8de4af78eb2916862363e5785e7fe333831f0
Gerrit-Change-Number: 536036
Gerrit-PatchSet: 1
Gerrit-Owner: vicente <freeze.bilsted(a)gmail.com>
Gerrit-MessageType: newchange
2 years, 5 months
Change in ...nfs-ganesha[next]: DBus: add dbus interface to show the ratio of entry reusing
by vicente (GerritHub)
vicente has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536035 )
Change subject: DBus: add dbus interface to show the ratio of entry reusing
......................................................................
DBus: add dbus interface to show the ratio of entry reusing
- That would be helpful to know the reclaimed efficiency
on MDCache
Change-Id: I7a66cb69ca38a15cf86b830d0a3f6a01f438eabf
Signed-off-by: Vicente Cheng <vicente_cheng(a)bigtera.com>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.h
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_main.c
M src/include/server_stats_private.h
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_int.h
M src/support/mdcache_mgr.c
6 files changed, 120 insertions(+), 2 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/35/536035/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536035
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: I7a66cb69ca38a15cf86b830d0a3f6a01f438eabf
Gerrit-Change-Number: 536035
Gerrit-PatchSet: 1
Gerrit-Owner: vicente <freeze.bilsted(a)gmail.com>
Gerrit-MessageType: newchange
2 years, 5 months
Change in ...nfs-ganesha[next]: FSAL_MDCACHE: add statistics counter for reusing entry
by vicente (GerritHub)
vicente has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536034 )
Change subject: FSAL_MDCACHE: add statistics counter for reusing entry
......................................................................
FSAL_MDCACHE: add statistics counter for reusing entry
- That we could know the rate of reusing obviously
Change-Id: I3da43b98a78d3d82654940ee21628085fec618ba
Signed-off-by: Vicente Cheng <vicente_cheng(a)bigtera.com>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_main.c
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_int.h
3 files changed, 51 insertions(+), 4 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/34/536034/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536034
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: I3da43b98a78d3d82654940ee21628085fec618ba
Gerrit-Change-Number: 536034
Gerrit-PatchSet: 1
Gerrit-Owner: vicente <freeze.bilsted(a)gmail.com>
Gerrit-MessageType: newchange
2 years, 5 months
Change in ...nfs-ganesha[next]: Statistics: split mdcache information from export_mgr
by vicente (GerritHub)
vicente has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536033 )
Change subject: Statistics: split mdcache information from export_mgr
......................................................................
Statistics: split mdcache information from export_mgr
- To add the individual dbus command with
/org/ganesha/nfsd/MDCMgr, interface
org.ganesha.nfsd.mdcstats
Change-Id: I695fe8785168c9c6541f3d6d88116f1c39b06508
Signed-off-by: Vicente Cheng <vicente_cheng(a)bigtera.com>
---
M src/MainNFSD/nfs_init.c
M src/support/export_mgr.c
M src/support/CMakeLists.txt
A src/include/mdcache_mgr.h
A src/support/mdcache_mgr.c
5 files changed, 195 insertions(+), 31 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/33/536033/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536033
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: I695fe8785168c9c6541f3d6d88116f1c39b06508
Gerrit-Change-Number: 536033
Gerrit-PatchSet: 1
Gerrit-Owner: vicente <freeze.bilsted(a)gmail.com>
Gerrit-MessageType: newchange
2 years, 5 months