Page MenuHomeFreeBSD

Fix pmap_page_is_mapped() for arm64 and riscv.
ClosedPublic

Authored by markj on Sep 27 2019, 4:37 PM.
Tags
None
Referenced Files
F135335964: D21824.diff
Sat, Nov 8, 9:19 PM
Unknown Object (File)
Mon, Nov 3, 1:22 PM
Unknown Object (File)
Sat, Oct 25, 6:43 AM
Unknown Object (File)
Sat, Oct 25, 6:43 AM
Unknown Object (File)
Sat, Oct 25, 6:43 AM
Unknown Object (File)
Sat, Oct 25, 6:43 AM
Unknown Object (File)
Sat, Oct 25, 12:14 AM
Unknown Object (File)
Sat, Oct 18, 11:10 AM
Subscribers

Details

Summary

To be correct, this function needs to look at 2MB mappings as well. I
just copied the implementation from amd64.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

alc added inline comments.
sys/arm64/include/pmap.h
170–171 ↗(On Diff #62653)

You need to replace this by the function declaration from amd64:

boolean_t pmap_page_is_mapped(vm_page_t m);

This revision is now accepted and ready to land.Sep 27 2019, 4:57 PM
This revision now requires review to proceed.Sep 27 2019, 5:24 PM

Here is a take-it-or-leave comment: Consider conversion from "boolean_t" to "bool".

This revision is now accepted and ready to land.Sep 27 2019, 9:24 PM