Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 15 Dec 2025 17:26:29 +0000
From:      Mateusz Piotrowski <0mp@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: faf64970ac80 - stable/13 - vfs_cache: Fix the SDT definition of vfs:fplookup:lookup:done
Message-ID:  <694044c5.fe78.2a68be91@gitrepo.freebsd.org>

index | next in thread | raw e-mail

The branch stable/13 has been updated by 0mp:

URL: https://cgit.FreeBSD.org/src/commit/?id=faf64970ac80caa5293ee58fe8614aa68d4c9b8e

commit faf64970ac80caa5293ee58fe8614aa68d4c9b8e
Author:     Mateusz Piotrowski <0mp@FreeBSD.org>
AuthorDate: 2025-07-14 23:09:24 +0000
Commit:     Mateusz Piotrowski <0mp@FreeBSD.org>
CommitDate: 2025-12-15 17:26:14 +0000

    vfs_cache: Fix the SDT definition of vfs:fplookup:lookup:done
    
    1. The definition lists struct nameidata as the type of the first
       argument. However, the actual probes always pass a variable of type
       struct nameidata* to SDT_PROBE3.
    2. The third argument (args[2]) is actually enum cache_fpl_status.
    
    Reviewed by:    markj
    Approved by:    markj (mentor)
    Fixes:          07d2145a1717 vfs: add the infrastructure for lockless lookup
    MFC after:      1 week
    Differential Revision:  https://reviews.freebsd.org/D51315
    
    (cherry picked from commit 6567623f831daaffa67777d17780e8f424c1bb01)
---
 sys/kern/vfs_cache.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/sys/kern/vfs_cache.c b/sys/kern/vfs_cache.c
index e6cf39c09f19..0afb65c55c5a 100644
--- a/sys/kern/vfs_cache.c
+++ b/sys/kern/vfs_cache.c
@@ -245,7 +245,8 @@ SDT_PROBE_DEFINE2(vfs, namecache, evict_negative, done, "struct vnode *",
     "char *");
 SDT_PROBE_DEFINE1(vfs, namecache, symlink, alloc__fail, "size_t");
 
-SDT_PROBE_DEFINE3(vfs, fplookup, lookup, done, "struct nameidata", "int", "bool");
+SDT_PROBE_DEFINE3(vfs, fplookup, lookup, done, "struct nameidata *", "int",
+    "enum cache_fpl_status");
 SDT_PROBE_DECLARE(vfs, namei, lookup, entry);
 SDT_PROBE_DECLARE(vfs, namei, lookup, return);
 


help

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?694044c5.fe78.2a68be91>