Port r324665 and r325285 to arm64.
ClosedPublic
Actions

Authored by markj on Jul 17 2018, 6:10 PM.

Details

Reviewers

manu
alc
kib
andrew

Commits

rS336460: Port r324665 and r325285 to arm64.

Test Plan

I added a fail point to get_pv_entry(), causing it to fail to allocate
a page 33% of the time and resort to pv_chunk reclamation.
Without the patch I can reproduce the bug (which manifests
as a panic due to tailq corruption).

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 18133
Build 17868: arc lint + arc unit

Event Timeline

markj created this revision.Jul 17 2018, 6:10 PM

Herald added a reviewer: manu. · View Herald TranscriptJul 17 2018, 6:10 PM

Herald added subscribers: emaste, andrew. · View Herald Transcript

Harbormaster completed remote builds in B18101: Diff 45422.Jul 17 2018, 6:10 PM

markj edited the test plan for this revision. (Show Details)Jul 17 2018, 6:12 PM

markj added reviewers: alc, kib, andrew.

alc added inline comments.Jul 18 2018, 3:56 AM

sys/arm64/arm64/pmap.c
1890–1894	`new_tail` is no longer used.
2040–2044	This is suspiciously different from amd64. Please double-check it.

alc added inline comments.Jul 18 2018, 3:58 PM

sys/arm64/arm64/pmap.c
1874	Could you please swap the pmap and next_pmap definitions so that this line is identical to the amd64 version and the diff between the two is smaller.

Address feedback.

Harbormaster completed remote builds in B18133: Diff 45481.Jul 18 2018, 5:21 PM

markj added inline comments.Jul 18 2018, 5:23 PM

sys/arm64/arm64/pmap.c
2040–2044	The difference comes from the fact that _pmap_unwire_l3() adjusts v_wire_count, while _pmap_unwire_ptp() does not. I think the code here is correct, but it would be nice to resolve that discrepancy in a future revision.