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)
	Sun, Sep 27, 10:41 PM

	Unknown Object (File)
	Thu, Sep 24, 3:34 AM

	Unknown Object (File)
	Thu, Sep 24, 3:32 AM

	Unknown Object (File)
	Thu, Sep 24, 3:30 AM

	Unknown Object (File)
	Sun, Sep 20, 12:42 AM

	Unknown Object (File)
	Mon, Sep 7, 3:31 PM

	Unknown Object (File)
	Mon, Sep 7, 3:31 PM

	Unknown Object (File)
	Mon, Sep 7, 3:31 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