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

	F147589491: D32023.id.diff
	Thu, Mar 12, 3:59 AM

	F147556002: D32023.id.diff
	Wed, Mar 11, 9:15 PM

	Unknown Object (File)
	Mon, Mar 2, 9:00 PM

	Unknown Object (File)
	Sat, Feb 28, 9:01 AM

	Unknown Object (File)
	Mon, Feb 23, 7:42 PM

	Unknown Object (File)
	Sun, Feb 22, 2:07 AM

	Unknown Object (File)
	Sun, Feb 22, 2:07 AM

	Unknown Object (File)
	Sun, Feb 22, 2:07 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