Page MenuHomeFreeBSD
Differential D3675

kevent(2): Trigger notes on DOOMED vnodes
ClosedPublic
Actions

Authored by cem on Sep 15 2015, 6:15 PM.
Tags
None
Referenced Files
F138641799: D3675.id8778.diff
Wed, Dec 3, 5:45 PM
Unknown Object (File)
Sun, Nov 30, 2:42 PM
Unknown Object (File)
Fri, Nov 28, 10:02 AM
Unknown Object (File)
Sun, Nov 23, 9:11 PM
Unknown Object (File)
Sat, Nov 22, 11:59 PM
Unknown Object (File)
Fri, Nov 21, 5:48 PM
Unknown Object (File)
Thu, Nov 20, 10:24 PM
Unknown Object (File)
Thu, Nov 20, 10:24 PM
View All Files
Subscribers
imp

Details

Reviewers
kib
markj
Commits
rS287831: kevent(2): Note DOOMED vnodes with NOTE_REVOKE
Summary

Always wake callers who ask for events on DOOMED files.

Diff Detail

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

Event Timeline

cem updated this revision to Diff 8774.Sep 15 2015, 6:15 PM
cem retitled this revision from to kevent(2): Trigger notes on DOOMED vnodes.
cem updated this object.
cem edited the test plan for this revision. (Show Details)
cem added reviewers: kib, markj.
Herald added a subscriber: imp. · View Herald TranscriptSep 15 2015, 6:15 PM
cem updated this revision to Diff 8776.Sep 15 2015, 7:13 PM

Also trigger notes when vnodes are reclaimed.

Limit VBAD check to poll mode (hint == 0).

kib accepted this revision.Sep 15 2015, 7:17 PM
kib edited edge metadata.

Looks fine to me. I would also added a comment explaining the hint == 0 check.

This revision is now accepted and ready to land.Sep 15 2015, 7:17 PM
markj accepted this revision.Sep 15 2015, 7:23 PM
markj edited edge metadata.
Closed by commit rS287831: kevent(2): Note DOOMED vnodes with NOTE_REVOKE (authored by cem). · Explain WhySep 15 2015, 8:22 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
kern/
15 lines
1 line
sys/
1 line

Diff 8778

View Options

head/sys/kern/vfs_subr.c

Loading...
View Options

head/sys/kern/vnode_if.src

Loading...
View Options

head/sys/sys/vnode.h

Loading...