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
F166072454: D33645.diff
Tue, Aug 11, 12:52 PM
Unknown Object (File)
Mon, Aug 10, 6:17 AM
Unknown Object (File)
Sun, Aug 9, 9:21 PM
Unknown Object (File)
Sun, Aug 9, 4:50 PM
Unknown Object (File)
Sun, Aug 9, 4:11 PM
Unknown Object (File)
Sun, Aug 9, 4:10 PM
Unknown Object (File)
Sat, Aug 8, 2:36 AM
Unknown Object (File)
Sat, Aug 8, 2:09 AM
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...