Page MenuHomeFreeBSD

Strengthen warnings about ext4
AcceptedPublic

Authored by ceri on Jul 22 2021, 11:56 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 6, 1:04 AM
Unknown Object (File)
Wed, Nov 27, 10:29 AM
Unknown Object (File)
Oct 17 2024, 9:01 AM
Unknown Object (File)
Oct 4 2024, 7:25 PM
Unknown Object (File)
Oct 3 2024, 10:18 PM
Unknown Object (File)
Oct 2 2024, 6:08 AM
Unknown Object (File)
Oct 2 2024, 6:04 AM
Unknown Object (File)
Oct 2 2024, 1:04 AM

Details

Reviewers
bcr
pauamma_gundo.com
Group Reviewers
docs
Summary

Helder Novais pointed out on July 9th via the doc@freebsd.org list that the note about journalling on ext4 was not strong enough.

Fix that (and, while here, add ext4 to the FAQ).

Diff Detail

Repository
R9 FreeBSD doc repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

ceri requested review of this revision.Jul 22 2021, 11:56 AM
ceri created this revision.
pauamma_gundo.com added inline comments.
documentation/content/en/books/handbook/filesystems/_index.adoc
96

Is that warning applicable to base system support, to sysutils/fusefs-ext2, or both? (Given its position in the text and with no indication of scope, it reads as applicable to sysutils/fusefs-ext2 only.)

Clarify situation muddied by mention of the port.

ceri marked an inline comment as done.Jul 22 2021, 4:00 PM
ceri added inline comments.
documentation/content/en/books/handbook/filesystems/_index.adoc
96

Clarified; thanks. It's supposed to apply to both. I want to discourage people from using read-write support at all, I think.

documentation/content/en/books/handbook/filesystems/_index.adoc
96

I see you also removed "Starting with FreeBSD 12.1-RELEASE, a DTrace provider will be available as well." but the review summary doesn't explain why. Accidental overdeletion?

ceri marked 2 inline comments as done.Jul 22 2021, 4:25 PM
ceri added inline comments.
documentation/content/en/books/handbook/filesystems/_index.adoc
96

It wasn't appropriate for a WARNING, so I moved it higher up - see new line 89.

documentation/content/en/books/handbook/filesystems/_index.adoc
96

Oh, right, missed it.

bcr added a subscriber: bcr.

Looks good to me now.

This revision is now accepted and ready to land.Mar 11 2023, 2:09 PM

I completely forgot about that. Sorry.