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
F147319618: D33645.id100527.diff
Mon, Mar 9, 10:46 PM
F147238221: D33645.id100527.diff
Mon, Mar 9, 8:49 AM
F147237805: D33645.id.diff
Mon, Mar 9, 8:44 AM
F147229607: D33645.id100527.diff
Mon, Mar 9, 7:16 AM
Unknown Object (File)
Sat, Mar 7, 11:24 AM
Unknown Object (File)
Sat, Mar 7, 4:33 AM
Unknown Object (File)
Fri, Mar 6, 7:20 PM
Unknown Object (File)
Wed, Mar 4, 7: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...