[S] Change in ...nfs-ganesha[next]: log: skip conditional log check on isLevel when no targets configured
by Nishant Puri (GerritHub)
Nishant Puri has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1236743?usp=email )
Change subject: log: skip conditional log check on isLevel when no targets configured
......................................................................
log: skip conditional log check on isLevel when no targets configured
isLevel() is on the hot path, called for every log statement.
The call to is_op_context_conditional_flag_set() is always a no-op
when no exports or clients have conditional logging configured,
but still incurs full function call overhead on every invocation.
Introduce a bool conditional_logging_configured. Guard the
is_op_context_conditional_flag_set() call in isLevel() with
unlikely(conditional_logging_configured). In production,
where no conditional logging is active, the branch predictor
skips the call entirely at zero overhead.
Change-Id: Ia91dfedb143bcdf13c2cfd6cb58b8c2efc8b04ed
Signed-off-by: Nishant Puri <npuri(a)redhat.com>
---
M src/MainNFSD/libganesha_nfsd.ver
M src/include/log.h
M src/log/log_functions.c
3 files changed, 23 insertions(+), 2 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/43/1236743/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1236743?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: Ia91dfedb143bcdf13c2cfd6cb58b8c2efc8b04ed
Gerrit-Change-Number: 1236743
Gerrit-PatchSet: 1
Gerrit-Owner: Nishant Puri <npuri(a)redhat.com>
1 month
[XS] Change in ...nfs-ganesha[next]: Removed libnsl2-devel check in spec file
by Name of user not set (GerritHub)
Yogendra.Tangalapally(a)ibm.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1236710?usp=email )
Change subject: Removed libnsl2-devel check in spec file
......................................................................
Removed libnsl2-devel check in spec file
Removed libnsl2-devel, libnsl-devel BuildRequires
check as its dependecny removed in libntirpc in
below commit
Ref: cb308adf (Renmove libnsl dependency)
Change-Id: If505d807879f4ef33c45668ded659a066e2a5a94
Signed-off-by: Yogendra Charya Tangalapally <Yogendra.Tangalapally(a)ibm.com>
---
M src/nfs-ganesha.spec-in.cmake
1 file changed, 0 insertions(+), 7 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/10/1236710/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1236710?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: If505d807879f4ef33c45668ded659a066e2a5a94
Gerrit-Change-Number: 1236710
Gerrit-PatchSet: 1
Gerrit-Owner: Yogendra.Tangalapally(a)ibm.com
1 month