Page MenuHomeFreeBSD

Move the fault getpages code into its own function.
ClosedPublic

Authored by jeff on Wed, Jan 22, 3:03 AM.

Details

Summary

I left the readahead in the main loop because we don't need to repeat the dropbehind. However, storing the cached entry variables before unlocking the map was only done once before. I don't think this was important to the algorithm nor is it especially expensive.

Diff Detail

Repository
rS FreeBSD src repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

jeff created this revision.Wed, Jan 22, 3:03 AM
jeff updated this revision to Diff 67132.Wed, Jan 22, 3:08 AM

Fix a merge issue.

jeff edited the summary of this revision. (Show Details)Wed, Jan 22, 3:10 AM
jeff added reviewers: alc, dougm, kib, markj.
jeff set the repository for this revision to rS FreeBSD src repository.
kib added inline comments.Wed, Jan 22, 1:31 PM
sys/vm/vm_fault.c
1088 ↗(On Diff #67132)

Please do not do this. You are mixing VM_PAGER_XXX (pager errors namespace) and KERN_XXX (Mach errors).

This is arguably incorrect in particular because the numeric values intersect. Translate the rest of errors into KERN_ as well.

jeff added inline comments.Wed, Jan 22, 6:43 PM
sys/vm/vm_fault.c
1088 ↗(On Diff #67132)

ah yes. Will do.

I would argue that we should convert the whole mess to errno but not today.

jeff updated this revision to Diff 67159.Wed, Jan 22, 9:03 PM

Correct return values. Don't send pager errors back to vm_fault().

kib accepted this revision.Wed, Jan 22, 9:11 PM
This revision is now accepted and ready to land.Wed, Jan 22, 9:11 PM