Page MenuHomeFreeBSD
Differential D42566

arm64: Use pmap_early_vtophys in pmap_bootstrap_san
ClosedPublic
Actions

Authored by andrew on Nov 13 2023, 5:18 PM.
Tags
None
Referenced Files
F171137692: D42566.diff
Tue, Sep 8, 10:52 PM
Unknown Object (File)
Mon, Sep 7, 12:24 PM
Unknown Object (File)
Sun, Sep 6, 5:28 PM
Unknown Object (File)
Sun, Sep 6, 2:59 PM
Unknown Object (File)
Sat, Sep 5, 7:30 AM
Unknown Object (File)
Sat, Sep 5, 7:16 AM
Unknown Object (File)
Fri, Sep 4, 7:55 AM
Unknown Object (File)
Fri, Sep 4, 12:49 AM
View All Files
Subscribers
emaste
imp

Details

Reviewers
manu
Group Reviewers
arm64
Commits
rG88227ddc7536: arm64: Use pmap_early_vtophys in pmap_bootstrap_san
rG5fae53589eac: arm64: Use pmap_early_vtophys in pmap_bootstrap_san
Summary

Use pmap_early_vtophys to find the physical address of the kernel base
rather than using the calculated offset as it will be removed in a
latter commit.

Sponsored by: Arm Ltd

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

andrew created this revision.Nov 13 2023, 5:18 PM
Herald added a reviewer: manu. · View Herald TranscriptNov 13 2023, 5:18 PM
Herald added subscribers: emaste, imp. · View Herald Transcript
andrew requested review of this revision.Nov 13 2023, 5:18 PM
Harbormaster completed remote builds in B54389: Diff 130020.Nov 13 2023, 5:18 PM
andrew added a child revision: D42567: arm64: Make kern_delta unneeded in the boot params.Nov 13 2023, 5:18 PM
andrew added a comment.Nov 13 2023, 5:20 PM

I tried testing this locally, however I don't get very far into the kernel before it hangs with KASAN both with and without this change.

andrew added a comment.Nov 15 2023, 5:45 PM

Tested on an the Arm AEM Model (architecture simulator). I think KASAN has issues with my local VM.

This revision was not accepted when it landed; it landed in state Needs Review.Nov 15 2023, 6:13 PM
Closed by commit rG5fae53589eac: arm64: Use pmap_early_vtophys in pmap_bootstrap_san (authored by andrew). · Explain Why
This revision was automatically updated to reflect the committed changes.
andrew added a commit: rG5fae53589eac: arm64: Use pmap_early_vtophys in pmap_bootstrap_san.
andrew added a commit: rG88227ddc7536: arm64: Use pmap_early_vtophys in pmap_bootstrap_san.Feb 19 2024, 4:47 PM

Revision Contents
Changeset List

PathSize
sys/
arm64/
arm64/
2 lines
5 lines
include/
2 lines

Diff 130154

View Options

sys/arm64/arm64/machdep.c

Loading...
View Options

sys/arm64/arm64/pmap.c

Loading...
View Options

sys/arm64/include/pmap.h

Loading...