Skip to content

Commit 4f7f459

Browse files
committed
Reduce FUSE transport log level.
The FUSE init flags are unlikely to be of use unless one is debugging a problem with the filesystem. The messages about channel status get printed for every thread, and it's rather confusing for end-users to have a FUSE filesystem terminate with: fuse filesystem umounted FUSE channel already closed! FUSE channel already closed! FUSE channel already closed! FUSE channel already closed! FUSE channel already closed! FUSE channel already closed! ...when everything went well.
1 parent 36cd288 commit 4f7f459

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

src/api/server/sync_io.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -717,7 +717,7 @@ impl<F: FileSystem + Sync> Server<F> {
717717
match self.fs.init(capable) {
718718
Ok(want) => {
719719
let enabled = capable & want;
720-
info!(
720+
debug!(
721721
"FUSE INIT major {} minor {}\n in_opts: {:?}\nout_opts: {:?}",
722722
major, minor, capable, enabled
723723
);

src/transport/fusedev/fuse_t_session.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -285,7 +285,7 @@ impl FuseChannel {
285285
return Err(IoError(e.into()));
286286
}
287287
Errno::ENODEV => {
288-
info!("fuse filesystem umounted");
288+
debug!("got ENODEV when reading fuse fd, assuming fuse filesystem was umounted.");
289289
return Ok(());
290290
}
291291
e => {

src/transport/fusedev/linux_session.rs

+3-3
Original file line numberDiff line numberDiff line change
@@ -351,7 +351,7 @@ impl FuseChannel {
351351
}
352352
}
353353
} else if event.is_error() {
354-
info!("FUSE channel already closed!");
354+
debug!("FUSE channel already closed!");
355355
return Err(SessionFailure("epoll error".to_string()));
356356
} else {
357357
// We should not step into this branch as other event is not registered.
@@ -362,7 +362,7 @@ impl FuseChannel {
362362
// Handle wake up event first. We don't read the event fd so that a LEVEL triggered
363363
// event can still be delivered to other threads/daemons.
364364
if need_exit {
365-
info!("Will exit from fuse service");
365+
debug!("Will exit from fuse service");
366366
return Ok(None);
367367
}
368368
if fusereq_available {
@@ -399,7 +399,7 @@ impl FuseChannel {
399399
continue;
400400
}
401401
Errno::ENODEV => {
402-
info!("fuse filesystem umounted");
402+
debug!("got ENODEV when reading fuse fd, assuming fuse filesystem was umounted.");
403403
return Ok(None);
404404
}
405405
e => {

0 commit comments

Comments
 (0)