passthroughfs: remove trace log in hot path

lookup() and forget() are two hot paths, we see performance hit even if
trace log is not enabled.

So remove trace log in hot paths.

Signed-off-by: Eryu Guan <[email protected]>

Author: eryugey

src/passthrough/async_io.rs

@@ -379,11 +379,6 @@ impl<S: BitmapSlice + Send + Sync> AsyncFileSystem for PassthroughFs<S> {
             // and results in EBADF.
             match InodeMap::get_alt_locked(inodes.deref(), &ids_altkey, handle_altkey.as_ref()) {
                 Some(data) => {
-                    trace!(
-                        "fuse: do_lookup sees existing inode {} ids_altkey {:?}",
-                        data.inode,
-                        ids_altkey
-                    );
                     data.refcount.fetch_add(1, Ordering::Relaxed);
                     data.inode
                 }
@@ -396,12 +391,6 @@ impl<S: BitmapSlice + Send + Sync> AsyncFileSystem for PassthroughFs<S> {
                             format!("max inode number reached: {}", VFS_MAX_INO),
                         ));
                     }
-                    trace!(
-                        "fuse: do_lookup adds new inode {} ids_altkey {:?} handle_altkey {:?}",
-                        inode,
-                        ids_altkey,
-                        handle_altkey
-                    );
 
                     InodeMap::insert_locked(
                         inodes.deref_mut(),

src/passthrough/mod.rs

@@ -939,11 +939,6 @@ impl<S: BitmapSlice + Send + Sync> PassthroughFs<S> {
             // and results in EBADF.
             match InodeMap::get_alt_locked(inodes.deref(), &ids_altkey, handle_opt) {
                 Some(data) => {
-                    trace!(
-                        "fuse: do_lookup sees existing inode {} ids_altkey {:?}",
-                        data.inode,
-                        ids_altkey
-                    );
                     data.refcount.fetch_add(1, Ordering::Relaxed);
                     data.inode
                 }
@@ -956,12 +951,6 @@ impl<S: BitmapSlice + Send + Sync> PassthroughFs<S> {
                             format!("max inode number reached: {VFS_MAX_INO}"),
                         ));
                     }
-                    trace!(
-                        "fuse: do_lookup adds new inode {} ids_altkey {:?} handle {:?}",
-                        inode,
-                        ids_altkey,
-                        handle_opt,
-                    );
 
                     InodeMap::insert_locked(
                         inodes.deref_mut(),
@@ -1012,14 +1001,6 @@ impl<S: BitmapSlice + Send + Sync> PassthroughFs<S> {
                 // we don't want misbehaving clients to cause integer overflow.
                 let new = curr.saturating_sub(count);
 
-                trace!(
-                    "fuse: forget inode {} refcount {}, count {}, new_count {}",
-                    inode,
-                    curr,
-                    count,
-                    new
-                );
-
                 // Synchronizes with the acquire load in `do_lookup`.
                 if data
                     .refcount