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
F168712351: D3675.diff
Sat, Aug 29, 6:42 PM
Unknown Object (File)
Tue, Aug 25, 8:12 PM
Unknown Object (File)
Tue, Aug 25, 1:50 PM
Unknown Object (File)
Mon, Aug 24, 11:36 AM
Unknown Object (File)
Mon, Aug 24, 12:48 AM
Unknown Object (File)
Mon, Aug 10, 10:43 AM
Unknown Object (File)
Mon, Aug 10, 10:38 AM
Unknown Object (File)
Mon, Aug 10, 10:30 AM
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...