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)
	Thu, May 14, 12:32 PM

	Unknown Object (File)
	Thu, May 14, 10:41 AM

	Unknown Object (File)
	Thu, May 14, 4:44 AM

	Unknown Object (File)
	Wed, May 13, 2:00 AM

	Unknown Object (File)
	Mon, May 11, 3:49 PM

	Unknown Object (File)
	Fri, May 1, 8:03 PM

	Unknown Object (File)
	Thu, Apr 23, 3:12 AM

	Unknown Object (File)
	Wed, Apr 22, 11:52 PM

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