Bug 2211420
| Summary: | e2fsck on a mounted filesystem complains about clean orphan file | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Tomáš Bžatek <tbzatek> |
| Component: | e2fsprogs | Assignee: | Eric Sandeen <esandeen> |
| Status: | CLOSED NOTABUG | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | rawhide | CC: | esandeen, josef, kasal, kzak, lczerner, vtrefny |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | Unspecified | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | If docs needed, set a value | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2023-06-26 14:00:31 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Tomáš Bžatek
2023-05-31 14:21:57 UTC
From the e2fsck man page: "it is not safe to run e2fsck on mounted file systems ... The only exception is if the -n option is specified ... However, even if it is safe to do so, the results printed by e2fsck are not valid if the file system is mounted." It really is not valid to check a mounted filesystem, ever. There is no guarantee of consistency while the filesystem is live. It may be "safe" to do so with -n, but as mentioned above, the results are not valid. Building this into some other process should be considered an error, even if it has been doing so for years. Where is this code in libblockdev and UDisks that invoke the check? Okay, thanks for such authoritative answer! At the end I found existing were extra checks for a mounted filesystem in UDisks, it was just the libblockdev side that was unrestricted and carried test cases over a mounted filesystem. I've added clarifications to docs and removed the libblockdev test: https://github.com/storaged-project/libblockdev/pull/907 https://github.com/storaged-project/udisks/pull/1130 This was all about the traditional fsck. For an online consistency check I think we should use scrubbing-like functionality - filed an RFE for UDisks: https://github.com/storaged-project/udisks/issues/1131 |