Page MenuHomeFreeBSD

Remove most lingering references to the page lock in comments.
ClosedPublic

Authored by markj on Jul 28 2020, 8:03 PM.

Details

Summary

Finish updating comments to reflect new locking protocols introduced
over the past year. In particular, vm_page_lock is effectively unused
now. This diff does not modify comments relating to the busy lock,
which has also changed in scope.

There is one remaining comment that references the page lock, omitted
here because it corresponds to a (mostly harmless) bug in r356157. I
will fix it separately.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

markj requested review of this revision.Jul 28 2020, 8:03 PM
markj created this revision.
sys/vm/vm_page.c
3800 ↗(On Diff #75091)

xbusied

4259 ↗(On Diff #75091)

This comments literally reflects asserts, I am not sure if it is useful.

markj marked 2 inline comments as done.

Address feedback.

This revision is now accepted and ready to land.Jul 29 2020, 2:56 PM