Page MenuHomeFreeBSD
Differential D46945

vm_object: Assert that managed pages are on pagequeues when freeing
AcceptedPublic
Actions

Authored by markj on Fri, Oct 4, 3:08 PM.
Tags
None
Referenced Files
F98834473: D46945.diff
Fri, Oct 4, 10:15 PM
F98834439: D46945.id.diff
Fri, Oct 4, 10:15 PM
F98834435: D46945.id144252.diff
Fri, Oct 4, 10:15 PM
Subscribers
imp

Details

Reviewers
alc
kib
dougm

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 59759
Build 56645: arc lint + arc unit

Event Timeline

markj created this revision.Fri, Oct 4, 3:08 PM
Herald added a subscriber: imp. · View Herald TranscriptFri, Oct 4, 3:08 PM
markj requested review of this revision.Fri, Oct 4, 3:08 PM
Harbormaster completed remote builds in B59759: Diff 144252.Fri, Oct 4, 3:08 PM
markj added a parent revision: D46944: vm_page: Fix a logic bug in vm_page_unwire_managed().Fri, Oct 4, 3:08 PM
dougm accepted this revision.Fri, Oct 4, 6:17 PM
This revision is now accepted and ready to land.Fri, Oct 4, 6:17 PM
kib added a comment.Fri, Oct 4, 11:48 PM

Consider vm_page_grab_valid(). It allocates a page, and tries to read the page' content from pager. If the pager failed, the page is freed. I do not believe that the freed invalid page is enqueued there.

Revision Contents
Changeset List

PathSize
sys/
vm/
3 lines

Diff 144252

View Options

sys/vm/vm_object.c

Loading...