Differential D38114

Handle ERELOOKUP from VOP_FSYNC()
ClosedPublic
Actions

Authored by kib on Jan 18 2023, 9:12 PM.

Tags

None

Referenced Files

	F151089440: D38114.id115281.diff
	Sun, Apr 5, 10:53 PM

	F151061944: D38114.diff
	Sun, Apr 5, 5:39 PM

	Unknown Object (File)
	Sun, Apr 5, 9:36 AM

	Unknown Object (File)
	Sat, Apr 4, 1:13 PM

	Unknown Object (File)
	Tue, Mar 31, 11:57 PM

	Unknown Object (File)
	Feb 8 2026, 2:43 PM

	Unknown Object (File)
	Feb 8 2026, 12:16 AM

	Unknown Object (File)
	Feb 7 2026, 7:41 PM

Subscribers

Details

Reviewers

markj
mckusick

Commits

rG6189672e6008: Handle ERELOOKUP from VOP_FSYNC() in several other places

Summary

We need to repeat the operation if the vnode was relocked.

Reported by: markj

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

kib created this revision.Jan 18 2023, 9:12 PM

Herald added a subscriber: imp. · View Herald TranscriptJan 18 2023, 9:12 PM

kib requested review of this revision.Jan 18 2023, 9:12 PM

Thanks, this fixes the panic I saw.

sys/vm/vm_object.c
1239	Why isn't it necessary to relock the vnode here like we do in md? And if we can simply call VOP_FSYNC() in a loop like this, why not handle the retry entirely in the VOP?

kib added inline comments.Jan 18 2023, 9:55 PM

sys/vm/vm_object.c
1239	Generally it is useful to unlock the vnode since it allows SU thread and bufdaemon to clean more dependencies that perhaps caused the relock during ffs_fsyncvnode(). I added the relock there.

Relock for vm_object_sync()

kib marked an inline comment as done.Jan 18 2023, 9:57 PM

markj accepted this revision.Jan 18 2023, 10:15 PM

markj added inline comments.

sys/vm/vm_object.c
1245

This revision is now accepted and ready to land.Jan 18 2023, 10:15 PM

Avoid duplicating VOP_FSYNC() call.
Move comment to the more logical place. Make it stylish.

This revision now requires review to proceed.Jan 18 2023, 10:31 PM

markj accepted this revision.Jan 19 2023, 3:49 PM

markj added inline comments.

sys/vm/vm_object.c
1250

This revision is now accepted and ready to land.Jan 19 2023, 3:49 PM

Closed by commit rG6189672e6008: Handle ERELOOKUP from VOP_FSYNC() in several other places (authored by kib). · Explain WhyJan 20 2023, 1:56 AM

This revision was automatically updated to reflect the committed changes.

kib added a commit: rG6189672e6008: Handle ERELOOKUP from VOP_FSYNC() in several other places.

Revision Contents
Changeset List

Path

Size

sys/

dev/

md/

12 lines

vm/

16 lines

Diff 115281

sys/dev/md/md.c

Loading...

sys/vm/vm_object.c

Loading...