The branch main has been updated by mjg: URL: https://cgit.FreeBSD.org/src/commit/?id=31d1c080ab7b6d713f7b5fd8ff78e07d63645bc6
commit 31d1c080ab7b6d713f7b5fd8ff78e07d63645bc6 Author: Mateusz Guzik <m...@freebsd.org> AuthorDate: 2025-07-16 08:38:22 +0000 Commit: Mateusz Guzik <m...@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);