Differential D29021

growfs: allow operation on RW-mounted filesystems
ClosedPublic
Actions

Authored by emaste on Mar 2 2021, 10:43 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Nov 17, 10:53 PM

	Unknown Object (File)
	Wed, Nov 5, 5:21 PM

	Unknown Object (File)
	Tue, Oct 28, 10:38 AM

	Unknown Object (File)
	Mon, Oct 27, 1:57 AM

	Unknown Object (File)
	Thu, Oct 23, 1:12 AM

	Unknown Object (File)
	Oct 19 2025, 11:54 AM

	Unknown Object (File)
	Oct 13 2025, 5:05 PM

	Unknown Object (File)
	Oct 13 2025, 5:05 PM

Subscribers

f0andrey_gmail.com

Details

Reviewers

Commits

rG8d25741b633a: growfs: allow operation on RW-mounted filesystems
rG56d80e35a333: growfs: allow operation on RW-mounted filesystems
rGb5920128c7eb: growfs: allow operation on RW-mounted filesystems
rG0dcde5cc1274: growfs: allow operation on RW-mounted filesystems

Summary

growfs supports growing mounted filesystems (writes are temporarily suspended while the grow happens). Drop the check for fs_clean == 0 to restore this case. Leave fs_flags check for FS_UNCLEAN or FS_NEEDSFSCK which represent the state of the filesystem when it was mounted, and fsck should be run first if they are set.

PR: 253754
Fixes: 6eb925f8450f ("Filesystem utilities that modify the...")

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

emaste requested review of this revision.Mar 2 2021, 10:43 PM

emaste created this revision.

emaste added inline comments.

sbin/growfs/growfs.c
1464	"Check for filesystem that was unclean at mount time." perhaps?

f0andrey_gmail.com added a subscriber: f0andrey_gmail.com.Mar 3 2021, 12:57 AM

Thanks, this fix live growfs

I agree that this change is appropriate.

This revision is now accepted and ready to land.Mar 3 2021, 9:02 PM

Closed by commit rG0dcde5cc1274: growfs: allow operation on RW-mounted filesystems (authored by emaste). · Explain WhyMar 3 2021, 11:37 PM

This revision was automatically updated to reflect the committed changes.

emaste added a commit: rG0dcde5cc1274: growfs: allow operation on RW-mounted filesystems.

emaste added a commit: rGb5920128c7eb: growfs: allow operation on RW-mounted filesystems.Mar 6 2021, 1:13 AM

emaste added a commit: rG56d80e35a333: growfs: allow operation on RW-mounted filesystems.

emaste added a commit: rG8d25741b633a: growfs: allow operation on RW-mounted filesystems.Mar 6 2021, 7:47 PM

Revision Contents
Changeset List

Path

Size

sbin/

growfs/

5 lines

Diff 85094

sbin/growfs/growfs.c

Loading...