Skip to content

Reduce FUSE transport log level.#200

Closed
nrath-js wants to merge 1 commit intocloud-hypervisor:masterfrom
nrath-js:adjust-log-level
Closed

Reduce FUSE transport log level.#200
nrath-js wants to merge 1 commit intocloud-hypervisor:masterfrom
nrath-js:adjust-log-level

Conversation

@nrath-js
Copy link
Copy Markdown
Contributor

@nrath-js nrath-js commented Dec 4, 2024

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.

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.
@flokli
Copy link
Copy Markdown
Contributor

flokli commented Mar 20, 2025

This is the same as #195.

I'm fine with merging this or the other, but it'd be nice to make a decision on this.

cc @imeoer @jiangliu @bergwolf

@flokli flokli mentioned this pull request Mar 20, 2025
@georgyo
Copy link
Copy Markdown

georgyo commented Mar 17, 2026

@bergwolf @eryugey @imeoer @liubogithub We still think these patches are important. Is there any chance to get these reviewed?

@imeoer
Copy link
Copy Markdown
Collaborator

imeoer commented Mar 18, 2026

@nrath-js @georgyo Sorry for the late reply, there are some conflicts that need to be resolved.

@georgyo
Copy link
Copy Markdown

georgyo commented Mar 19, 2026

Thanks for looking at this! Unfortunately @nrath-js is an account that no one has access to any more, so I created #225 which is a re-base of this PR.

@imeoer imeoer closed this Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants