Page MenuHomeFreeBSD

Remove the header comment in vm_page.c.
ClosedPublic

Authored by markj on Jun 3 2019, 3:34 PM.

Details

Summary

It is outdated, and there is a much more thorough comment in vm_page.h.

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 created this revision.Jun 3 2019, 3:34 PM
markj added reviewers: alc, kib.Jun 3 2019, 3:34 PM
kib accepted this revision.Jun 3 2019, 4:24 PM
This revision is now accepted and ready to land.Jun 3 2019, 4:24 PM
alc added inline comments.Jun 3 2019, 5:07 PM
sys/vm/vm_page.c
82–84 ↗(On Diff #58193)

I'm fine with deleting the GENERAL RULES ON VM_PAGE MANIPULATION comment as stale.

However, there are similar one-line comments to "Resident memory management module." at the start of other files that we inherited from Mach. If you're going to delete one, please delete them all.

markj updated this revision to Diff 58198.Jun 3 2019, 5:25 PM

Restore the "Resident memory management module" comment.

This revision now requires review to proceed.Jun 3 2019, 5:25 PM
alc accepted this revision.Jun 3 2019, 7:10 PM
This revision is now accepted and ready to land.Jun 3 2019, 7:10 PM
kib accepted this revision.Jun 3 2019, 7:21 PM
This revision was automatically updated to reflect the committed changes.