Page MenuHomeFreeBSD
Differential D20503

Remove the header comment in vm_page.c.
ClosedPublic
Actions

Authored by markj on Jun 3 2019, 3:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 8, 3:27 AM
Unknown Object (File)
Nov 23 2024, 6:35 AM
Unknown Object (File)
Nov 23 2024, 3:11 AM
Unknown Object (File)
Oct 6 2024, 6:57 AM
Unknown Object (File)
Oct 6 2024, 4:47 AM
Unknown Object (File)
Oct 5 2024, 11:46 AM
Unknown Object (File)
Oct 5 2024, 4:55 AM
Unknown Object (File)
Oct 4 2024, 11:36 PM
View All 68 Files
Subscribers
imp

Details

Reviewers
alc
kib
Commits
rS348660: Remove an outdated header comment for vm_page.c.
Summary

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

Diff Detail

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

Event Timeline

markj created this revision.Jun 3 2019, 3:34 PM
Harbormaster completed remote builds in B24664: Diff 58193.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
Harbormaster completed remote builds in B24666: Diff 58198.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
Closed by commit rS348660: Remove an outdated header comment for vm_page.c. (authored by markj). · Explain WhyJun 4 2019, 6:38 PM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptJun 4 2019, 6:38 PM

Revision Contents
Changeset List

PathSize
head/
sys/
vm/
17 lines

Diff 58241

View Options

head/sys/vm/vm_page.c

Loading...