Page MenuHomeFreeBSD

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

Authored by markj on Tue, Jul 28, 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
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.Tue, Jul 28, 8:03 PM
markj created this revision.
kib added inline comments.Wed, Jul 29, 11:38 AM
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 updated this revision to Diff 75125.Wed, Jul 29, 2:22 PM
markj marked 2 inline comments as done.

Address feedback.

kib accepted this revision.Wed, Jul 29, 2:56 PM
This revision is now accepted and ready to land.Wed, Jul 29, 2:56 PM
This revision was automatically updated to reflect the committed changes.