Differential D32023

aio_fsync_vnode: handle ERELOOKUP after VOP_FSYNC()
ClosedPublic
Actions

Authored by kib on Sep 20 2021, 9:36 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Apr 15, 1:54 PM

	Unknown Object (File)
	Mon, Apr 13, 2:16 PM

	Unknown Object (File)
	Thu, Apr 9, 4:08 PM

	Unknown Object (File)
	Thu, Apr 9, 8:43 AM

	Unknown Object (File)
	Mon, Apr 6, 7:05 AM

	Unknown Object (File)
	Sat, Apr 4, 7:49 PM

	Unknown Object (File)
	Mar 12 2026, 4:05 PM

	Unknown Object (File)
	Mar 12 2026, 3:59 AM

View All Files

Subscribers

None

Details

Reviewers

tmunro
mckusick
jhb

Commits

rG922bee44e400: aio_fsync_vnode: use for(;;) loop instead of label
rG2933a7ca03f1: aio_fsync_vnode: handle ERELOOKUP after VOP_FSYNC()

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kib requested review of this revision.Sep 20 2021, 9:36 AM

kib created this revision.

LGTM. I tried it with my aio_fsync/aio_waitcomplete-heavy workload and saw no problem (though that isn't saying much, as I've only ever seen ERELOOKUP/panic twice).

This revision is now accepted and ready to land.Sep 20 2021, 11:55 AM

jhb accepted this revision.Sep 20 2021, 5:01 PM

Closed by commit rG2933a7ca03f1: aio_fsync_vnode: handle ERELOOKUP after VOP_FSYNC() (authored by kib). · Explain WhySep 20 2021, 8:27 PM

This revision was automatically updated to reflect the committed changes.

kib added a commit: rG2933a7ca03f1: aio_fsync_vnode: handle ERELOOKUP after VOP_FSYNC().

kib added a commit: rG922bee44e400: aio_fsync_vnode: use for(;;) loop instead of label.

Revision Contents
Changeset List

Path

Size

sys/

kern/

vfs_aio.c

3 lines

Diff 95398

View Options

aio_fsync_vnode: handle ERELOOKUP after VOP_FSYNC()ClosedPublicActions