libkvm: add kvm_walk_pages API.

[will]

Summary

This API allows callers to enumerate all known pages, including any
direct map & kernel map virtual addresses, physical addresses, size,
offset into the core, & protection configured.

For architectures that support direct map addresses, also generate pages
for any direct map only addresses that are not associated with kernel
map addresses.

Fix page size portability issue left behind from previous kvm page table
lookup interface.

Test Plan

Install libkvm, make sure kgdb et al still work (should be a no-op).

[will]

• kvm: pass up better error messages.

[jhb]

lib/libkvm/kvm.c

503 ↗

(On Diff #32820)

Minor style nit: parens around return value.

lib/libkvm/kvm_aarch64.h

47 ↗

(On Diff #32820)

Tab after #define instead of space

lib/libkvm/kvm_arm.h

56 ↗

(On Diff #32820)

Another tab vs space.

lib/libkvm/kvm_minidump_amd64.c

69 ↗

(On Diff #32820)

I found it somewhat confusing at first that there was both pde_get() and pte_get() for amd64, it might warrant a short block comment above these functions explaining that version 1 dumps use pte_get() and version 2 use pde_get()

342 ↗

(On Diff #32820)

Since this always reads pdes, does it only work for version 2? If so, we should check the version?

[will]

• libkvm: fix minor nits.

[will]

lib/libkvm/kvm_minidump_amd64.c

342 ↗

(On Diff #32820)

You're right.

Version 2 was added in 2010, so all versions after that point only generate v2 minidumps. Would it be acceptable to simply leave it out of this change?

[jhb]

lib/libkvm/kvm_minidump_amd64.c

342 ↗

(On Diff #32820)

I think just adding a version check and failing for version 1 dumps is probably fine. I'd just rather have an explicit failure than undefined behavior.

[will]

• libkvm: require v2 minidumps for walk_pages on amd64.

[will]

On amd64, this now requires v2 minidumps (generated since 2010).

[will]

@jhb Any further feedback? Does this look ready for HEAD?

[jhb]

This should be fine. I think the diff in phab is just a diff relative to the previous rather than the full diff FWIW.