Announce Push of V2.7-dev.14
by Frank Filz
Branch next
Tag:V2.7-dev.14
Release Highlights
* Shared code for gtests
* Several new gtests including the first batch of NFS v4 layer tests
* MDCACHE - Close an unexport race and Fix some subcalls
* FSAL_VFS: fix deadlock in vfs_read2
* Avoid ABBA kind deadlock while IP release in grace period
Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com>
Contents:
e34c16d Frank S. Filz V2.7-dev.14
d75c7a0 grajoria gtest/test_handle_to_key_latency: handle_to_key latency
microbenchmark
4865874 Girjesh Rajoria gtest/test_lock_op2_latency: lock_op2 latency
microbenchmark
6fdccc1 Sachin Punadikar Avoid ABBA kind deadlock while IP release in grace
period
7bc99d7 Fatih Acar FSAL_VFS: fix deadlock in vfs_read2
03e4aa1 Supriti Singh Man page: Add a new line to make it readable
7bd8708 Daniel Gryniewicz MDCACHE - Fix some subcalls
d287a4e Daniel Gryniewicz MDCACHE - Close an unexport race
cc12f36 Frank S. Filz gtest: nfs4_op_putfh
2e9c202 Frank S. Filz gtest: nfs4_op_lookup
27176b3 Frank S. Filz gtest: Add GaeshaNFS4BaseTest to collect common NFS v4
code
589bc08 Frank S. Filz gtest: Add GaeshaFSALBaseTest to collect common FSAL
code
3170725 Frank S. Filz NFSv4: Pull op_ctx->ctx_export cleanup out of
compound_data_free
2b65741 Frank S. Filz Update .gitignore to ignore some generated files
84b48f0 Frank S. Filz Make nfs_file_handle.h compile under C++
55d41cf Daniel Gryniewicz Add library for gtest microbenchmarks
5 years, 11 months
Change in ffilz/nfs-ganesha[next]: MDCACHE - Fix some subcalls
by GerritHub
From Daniel Gryniewicz <dang(a)redhat.com>:
Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/412483
Change subject: MDCACHE - Fix some subcalls
......................................................................
MDCACHE - Fix some subcalls
Change-Id: Id3c60f80c9579771aad6a704664d5532e472a780
Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_helpers.c
1 file changed, 14 insertions(+), 5 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/83/412483/1
--
To view, visit https://review.gerrithub.io/412483
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: Id3c60f80c9579771aad6a704664d5532e472a780
Gerrit-Change-Number: 412483
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>
5 years, 11 months
Change in ffilz/nfs-ganesha[next]: FSAL_VFS: fix deadlock in vfs_read2
by GerritHub
From <fatih(a)gandi.net>:
fatih(a)gandi.net has uploaded this change for review. ( https://review.gerrithub.io/412451
Change subject: FSAL_VFS: fix deadlock in vfs_read2
......................................................................
FSAL_VFS: fix deadlock in vfs_read2
this fixes commit c11eb9e421ac90fb788144dc1144888e6f20f7f0
it is easily triggered by `nfstest_io -d /mnt/ -n 5 --minfiles=10`
Change-Id: Ifebc243ccf80e1b9ca0c648551b8d0746c6183d2
Signed-off-by: Fatih Acar <fatih.acar(a)gandi.net>
---
M src/FSAL/FSAL_VFS/file.c
1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/51/412451/1
--
To view, visit https://review.gerrithub.io/412451
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: Ifebc243ccf80e1b9ca0c648551b8d0746c6183d2
Gerrit-Change-Number: 412451
Gerrit-PatchSet: 1
Gerrit-Owner: fatih(a)gandi.net
5 years, 11 months
Change in ffilz/nfs-ganesha[next]: gtest: nfs4_op_putfh
by GerritHub
From Frank Filz <ffilzlnx(a)mindspring.com>:
Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/412419
Change subject: gtest: nfs4_op_putfh
......................................................................
gtest: nfs4_op_putfh
Change-Id: Id88c4ed7080b71466989bdc5863332090a02609a
Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com>
---
M src/gtest/CMakeLists.txt
A src/gtest/test_nfs4_putfh_latency.cc
2 files changed, 292 insertions(+), 0 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/19/412419/1
--
To view, visit https://review.gerrithub.io/412419
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: Id88c4ed7080b71466989bdc5863332090a02609a
Gerrit-Change-Number: 412419
Gerrit-PatchSet: 1
Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>
5 years, 11 months
Change in ffilz/nfs-ganesha[next]: NFSv4: Pull op_ctx->ctx_export cleanup out of compound_data_free
by GerritHub
From Frank Filz <ffilzlnx(a)mindspring.com>:
Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/412407
Change subject: NFSv4: Pull op_ctx->ctx_export cleanup out of compound_data_free
......................................................................
NFSv4: Pull op_ctx->ctx_export cleanup out of compound_data_free
It sort of doesn't belong there and with gtest latency tests
calling compound_data_free it works better to not be there.
Change-Id: I73880d05a04cfccda39a440c4d53a4cb05a395dd
Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com>
---
M src/Protocols/NFS/nfs4_Compound.c
1 file changed, 7 insertions(+), 7 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/07/412407/1
--
To view, visit https://review.gerrithub.io/412407
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: I73880d05a04cfccda39a440c4d53a4cb05a395dd
Gerrit-Change-Number: 412407
Gerrit-PatchSet: 1
Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>
5 years, 11 months
Change in ffilz/nfs-ganesha[next]: Man page: Add a new line to make it readable
by GerritHub
From <supriti.singh(a)suse.com>:
supriti.singh(a)suse.com has uploaded this change for review. ( https://review.gerrithub.io/412403
Change subject: Man page: Add a new line to make it readable
......................................................................
Man page: Add a new line to make it readable
Change-Id: I9665699a5a8e54f7467395f011f409e299865a8f
Signed-off-by: Supriti Singh <supriti.singh(a)suse.com>
---
M src/doc/man/ganesha-core-config.rst
1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/03/412403/1
--
To view, visit https://review.gerrithub.io/412403
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: I9665699a5a8e54f7467395f011f409e299865a8f
Gerrit-Change-Number: 412403
Gerrit-PatchSet: 1
Gerrit-Owner: supriti.singh(a)suse.com
5 years, 11 months
Change in ffilz/nfs-ganesha[next]: RADOS_KV: Check if ceph_conf is provided
by GerritHub
From <supriti.singh(a)suse.com>:
supriti.singh(a)suse.com has uploaded this change for review. ( https://review.gerrithub.io/412402
Change subject: RADOS_KV: Check if ceph_conf is provided
......................................................................
RADOS_KV: Check if ceph_conf is provided
Change-Id: I2a62e4871502de58fd80cb6a73030ffed52cfc61
Signed-off-by: Supriti Singh <supriti.singh(a)suse.com>
---
M src/SAL/recovery/recovery_rados_kv.c
1 file changed, 7 insertions(+), 1 deletion(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/02/412402/1
--
To view, visit https://review.gerrithub.io/412402
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: I2a62e4871502de58fd80cb6a73030ffed52cfc61
Gerrit-Change-Number: 412402
Gerrit-PatchSet: 1
Gerrit-Owner: supriti.singh(a)suse.com
5 years, 11 months
Change in ffilz/nfs-ganesha[next]: Avoid ABBA kind deadlock while IP release in grace period
by GerritHub
From Sachin Punadikar <psachin(a)in.ibm.com>:
Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/412321
Change subject: Avoid ABBA kind deadlock while IP release in grace period
......................................................................
Avoid ABBA kind deadlock while IP release in grace period
During IP release event ABBA deadlock situation ariases as below.
In case of IP release event, nfs4_start_grace (grace_mutex already
locked) calls nfs_release_v4_client. In this it gets the list of
confirmed client records associated with that IP. It travels that
list and attempts to grab lock on client_record->cr_mutex.
Now in parallel, there was a new client approaching Ganesha for the
IP to be released. As part of response to SETCLIENTID_CONFIRM,
nfs4_op_setclientid_confirm, gets the unconfirmed client record, locks
it (client_record->cr_mutex) and after validations, makes it part of
confirmed clients. As next processing it calls nfs4_chk_clid which
needs lock on grace_mutex.
As grace_mutex is for protecting clid_list, the locking can be done
when it is required. This solves this deadlock issue.
Change-Id: Ibdaab87c91b9c78e1b30aa38909032eb7c41f5b6
Signed-off-by: Sachin Punadikar <psachin(a)in.ibm.com>
---
M src/SAL/nfs4_recovery.c
1 file changed, 5 insertions(+), 7 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/21/412321/1
--
To view, visit https://review.gerrithub.io/412321
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: Ibdaab87c91b9c78e1b30aa38909032eb7c41f5b6
Gerrit-Change-Number: 412321
Gerrit-PatchSet: 1
Gerrit-Owner: Sachin Punadikar <psachin(a)in.ibm.com>
5 years, 11 months
Change in ffilz/nfs-ganesha[next]: gtest: Add GaeshaNFS4BaseTest to collect common NFS v4 code
by GerritHub
From Frank Filz <ffilzlnx(a)mindspring.com>:
Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/412300
Change subject: gtest: Add GaeshaNFS4BaseTest to collect common NFS v4 code
......................................................................
gtest: Add GaeshaNFS4BaseTest to collect common NFS v4 code
Change-Id: I28650769f2ac9e538f584979524463f73f9e3dd6
Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com>
---
A src/gtest/gtest_nfs4.hh
1 file changed, 98 insertions(+), 0 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/00/412300/1
--
To view, visit https://review.gerrithub.io/412300
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: I28650769f2ac9e538f584979524463f73f9e3dd6
Gerrit-Change-Number: 412300
Gerrit-PatchSet: 1
Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>
5 years, 11 months
Change in ffilz/nfs-ganesha[next]: gtest: Add GaeshaFSALBaseTest to collect common FSAL code
by GerritHub
From Frank Filz <ffilzlnx(a)mindspring.com>:
Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/412299
Change subject: gtest: Add GaeshaFSALBaseTest to collect common FSAL code
......................................................................
gtest: Add GaeshaFSALBaseTest to collect common FSAL code
Change-Id: Ie70227cb5bb42f1cfa5537ba08a0057d86eb3a86
Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com>
---
M src/gtest/gtest.hh
1 file changed, 152 insertions(+), 3 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/99/412299/1
--
To view, visit https://review.gerrithub.io/412299
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: Ie70227cb5bb42f1cfa5537ba08a0057d86eb3a86
Gerrit-Change-Number: 412299
Gerrit-PatchSet: 1
Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>
5 years, 11 months