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);

Reply via email to