git: 31d1c080ab7b - main - vfs cache: drop SDT_PROBES_ENABLED usage
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Wed, 16 Jul 2025 08:51:58 UTC
The branch main has been updated by mjg:
URL: https://cgit.FreeBSD.org/src/commit/?id=31d1c080ab7b6d713f7b5fd8ff78e07d63645bc6
commit 31d1c080ab7b6d713f7b5fd8ff78e07d63645bc6
Author: Mateusz Guzik <mjg@FreeBSD.org>
AuthorDate: 2025-07-16 08:38:22 +0000
Commit: Mateusz Guzik <mjg@FreeBSD.org>
CommitDate: 2025-07-16 08:50:35 +0000
vfs cache: drop SDT_PROBES_ENABLED usage
since sdt probes started being hot patched
This eliminates a now spurious branch on fpl.status
---
sys/kern/vfs_cache.c | 10 +++-------
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/sys/kern/vfs_cache.c b/sys/kern/vfs_cache.c
index ec3f947ffa55..89c1d779f04c 100644
--- a/sys/kern/vfs_cache.c
+++ b/sys/kern/vfs_cache.c
@@ -6421,15 +6421,11 @@ out:
cache_fpl_smr_assert_not_entered(&fpl);
cache_fpl_assert_status(&fpl);
*status = fpl.status;
- if (SDT_PROBES_ENABLED()) {
- SDT_PROBE3(vfs, fplookup, lookup, done, ndp, fpl.line, fpl.status);
- if (fpl.status == CACHE_FPL_STATUS_HANDLED)
- SDT_PROBE4(vfs, namei, lookup, return, error, ndp->ni_vp, true,
- ndp);
- }
-
+ SDT_PROBE3(vfs, fplookup, lookup, done, ndp, fpl.line, fpl.status);
if (__predict_true(fpl.status == CACHE_FPL_STATUS_HANDLED)) {
MPASS(error != CACHE_FPL_FAILED);
+ SDT_PROBE4(vfs, namei, lookup, return, error, ndp->ni_vp, true,
+ ndp);
if (error != 0) {
cache_fpl_cleanup_cnp(fpl.cnp);
MPASS(fpl.dvp == NULL);