Page MenuHomeFreeBSD
Differential D7273

Declare aio requests on files from local filesystems safe.
ClosedPublic
Actions

Authored by kib on Jul 21 2016, 4:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Oct 24, 5:02 AM
Unknown Object (File)
Thu, Oct 23, 10:22 AM
Unknown Object (File)
Thu, Oct 23, 1:50 AM
Unknown Object (File)
Thu, Oct 23, 1:50 AM
Unknown Object (File)
Wed, Oct 22, 6:04 PM
Unknown Object (File)
Sun, Oct 12, 5:32 AM
Unknown Object (File)
Sat, Oct 11, 3:32 PM
Unknown Object (File)
Thu, Oct 2, 3:26 AM
View All 92 Files
Subscribers
imp

Details

Reviewers
jhb
Commits
rS303154: Declare aio requests on files from local filesystems safe.
Summary

There are two fine moments:

  • I allow AIO on reclaimed vnodes, since it is deterministically terminated fast.
  • devfs mounts are marked as MNT_LOCAL, but device vnodes have type VCHR, so the slow device io is not allowed.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib updated this revision to Diff 18636.Jul 21 2016, 4:50 PM
kib retitled this revision from to Declare aio requests on files from local filesystems safe..
kib updated this object.
kib edited the test plan for this revision. (Show Details)
kib added a reviewer: jhb.
kib set the repository for this revision to rS FreeBSD src repository - subversion.
Herald added a subscriber: imp. · View Herald TranscriptJul 21 2016, 4:50 PM
jhb accepted this revision.Jul 21 2016, 4:56 PM
jhb edited edge metadata.

This should probably go into 11 if possible. It probably fixes the bulk of the use cases that folks like jmg@ really care about (since samba was using aio for local filesystem access).

This revision is now accepted and ready to land.Jul 21 2016, 4:56 PM
jhb added a comment.Jul 21 2016, 4:59 PM

One other note is that some of the aio tests in test/sys/aio might be able to remove the unsafe check.

Closed by commit rS303154: Declare aio requests on files from local filesystems safe. (authored by kib). · Explain WhyJul 21 2016, 5:07 PM
This revision was automatically updated to reflect the committed changes.
kib added a comment.Jul 21 2016, 5:11 PM
In D7273#151301, @jhb wrote:

One other note is that some of the aio tests in test/sys/aio might be able to remove the unsafe check.

This requires more work, like statfs(2)-ing the operational mount point. E.g., tests might be performed on NFS.

jhb added a comment.Jul 21 2016, 9:15 PM
In D7273#151311, @kib wrote:
In D7273#151301, @jhb wrote:

One other note is that some of the aio tests in test/sys/aio might be able to remove the unsafe check.

This requires more work, like statfs(2)-ing the operational mount point. E.g., tests might be performed on NFS.

Yes, I might get around to fixing that (I have certainly run them both locally and over NFS myself).

I do need to update the language in aio(4) to document the change of this commit, though I will do that after my pending aio_warn change.

Revision Contents
Changeset List

PathSize
head/
sys/
kern/
14 lines

Diff 18638

View Options

head/sys/kern/vfs_aio.c

Loading...