Page MenuHomeFreeBSD
Differential D33645

Fix an accounting error from 9b0c0769b2b2e1d6c917c0799f2a0
ClosedPublic
Actions

Authored by dougm on Dec 24 2021, 6:05 AM.
Tags
None
Referenced Files
F140256526: D33645.id100527.diff
Sun, Dec 21, 11:04 PM
Unknown Object (File)
Sun, Dec 21, 3:02 AM
Unknown Object (File)
Fri, Dec 19, 10:25 AM
Unknown Object (File)
Thu, Dec 18, 1:34 PM
Unknown Object (File)
Wed, Dec 17, 6:22 PM
Unknown Object (File)
Tue, Dec 16, 12:34 AM
Unknown Object (File)
Sat, Dec 13, 8:33 PM
Unknown Object (File)
Sat, Dec 13, 8:30 PM
View All Files
Subscribers
imp

Details

Reviewers
alc
kib
markj
Commits
rGb52d35520c51: Fix clerical error in page alloc
rG184c63db3c94: Fix clerical error in page alloc
Summary

A tardy review suggests that there's an accounting error in the most recent change to vm_page.c. This seeks to fix it.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dougm requested review of this revision.Dec 24 2021, 6:05 AM
dougm created this revision.
alc accepted this revision.Dec 24 2021, 8:44 AM
This revision is now accepted and ready to land.Dec 24 2021, 8:44 AM
alc added a reviewer: markj.Dec 24 2021, 8:45 AM
Closed by commit rG184c63db3c94: Fix clerical error in page alloc (authored by dougm). · Explain WhyDec 24 2021, 8:52 AM
This revision was automatically updated to reflect the committed changes.
dougm added a commit: rG184c63db3c94: Fix clerical error in page alloc.
Herald added a subscriber: imp. · View Herald TranscriptDec 24 2021, 8:52 AM
alc added inline comments.Dec 24 2021, 7:04 PM
sys/vm/vm_page.c
2244–2247

This comment no longer makes sense here. It belongs before the call to vm_domain_allocate().

dougm added a commit: rGb52d35520c51: Fix clerical error in page alloc.Jul 11 2022, 5:43 AM

Revision Contents
Changeset List

PathSize
sys/
vm/
4 lines

Diff 100527

View Options

sys/vm/vm_page.c

Loading...