[M] Change in ...nfs-ganesha[next]: clientid: fix Coverity CID 502248 - mutex ordering
by Frank Filz (GerritHub)
Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1200700?usp=email )
Change subject: clientid: fix Coverity CID 502248 - mutex ordering
......................................................................
clientid: fix Coverity CID 502248 - mutex ordering
Fix ordering of cid_mutex and expired_client_ids_list_lock to
be consistent.
Change-Id: I81eda13e8bad1d3d170dcedc57d12c9e2129b95d
Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com>
---
M src/FSAL_UP/fsal_up_top.c
M src/Protocols/NFS/nfs4_Compound.c
M src/Protocols/NFS/nfs4_op_lockt.c
M src/Protocols/NFS/nfs4_op_open.c
M src/Protocols/NFS/nfs4_op_release_lockowner.c
M src/SAL/nfs4_clientid.c
M src/SAL/nfs4_lease.c
M src/SAL/nfs4_state_id.c
M src/include/sal_functions.h
9 files changed, 67 insertions(+), 46 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/00/1200700/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1200700?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: I81eda13e8bad1d3d170dcedc57d12c9e2129b95d
Gerrit-Change-Number: 1200700
Gerrit-PatchSet: 1
Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>
2 weeks
[M] Change in ...nfs-ganesha[next]: Fix coverity issues by altering branching logic.
by Sachin Punadikar (GerritHub)
Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1200492?usp=email )
Change subject: Fix coverity issues by altering branching logic.
......................................................................
Fix coverity issues by altering branching logic.
For optimized branching, "unlikely" was used, but coverity scan reported
lot of issues for same check again. Changed/altered the logic to make
use of "likely" to have similar effect in branching optimization and to
satisfy coverity scans.
Solving issues - CID 502229, 502246, 502163, 502220, 502066, 502211,
502055, 502209, 502178, 502051, 502119
Change-Id: I5950eb363289e67f1fb2ae649fa0987197e4a9f4
Signed-off-by: Sachin Punadikar <sachin.punadikar(a)ibm.com>
---
M src/support/server_stats.c
1 file changed, 47 insertions(+), 23 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/92/1200492/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1200492?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: I5950eb363289e67f1fb2ae649fa0987197e4a9f4
Gerrit-Change-Number: 1200492
Gerrit-PatchSet: 1
Gerrit-Owner: Sachin Punadikar <sachin.punadikar(a)ibm.com>
2 weeks, 3 days