Page MenuHomeFreeBSD
Differential D10217

Increase number of L2 tables required for kernel bootstrap
ClosedPublic
Actions

Authored by mw_semihalf.com on Mar 31 2017, 2:23 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Oct 30, 5:12 AM
Unknown Object (File)
Tue, Oct 28, 12:43 AM
Unknown Object (File)
Fri, Oct 24, 11:27 AM
Unknown Object (File)
Fri, Oct 24, 1:21 AM
Unknown Object (File)
Thu, Oct 9, 7:17 PM
Unknown Object (File)
Thu, Oct 9, 6:51 AM
Unknown Object (File)
Tue, Oct 7, 12:26 AM
Unknown Object (File)
Mon, Oct 6, 2:14 PM
View All 91 Files
Subscribers
fabient
imp
loos
ARM

Details

Reviewers
• ian
zbb
andrew
mmel
Commits
rS317085: Increase number of L2 tables required for kernel bootstrap
Summary

Memory space reserved for pmap_kernel_l2dtable_kva and
pmap_kernel_l2ptp_kva has not been taken into account in
original code.

Submitted by: Grzegorz Bernacki <gjb@semihalf.com>
Obtained from: Semihalf
Sponsored by: Stormshield
Reviewed by:
Differential revision:

Diff Detail

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

Event Timeline

mw_semihalf.com created this revision.Mar 31 2017, 2:23 PM
Herald added a subscriber: imp. · View Herald TranscriptMar 31 2017, 2:23 PM
loos added a subscriber: loos.Mar 31 2017, 6:25 PM
mmel edited edge metadata.Apr 2 2017, 7:12 AM

Can you be, please, little more verbose? I'm unable to detect where exactly the problem is.

mw_semihalf.com added a comment.Apr 11 2017, 6:26 AM

Here's an explanation from gber, who is the commit author:
If my understanding of code responsible for initial mapping is correct, then all the memory reserved from kernel space by pmap_alloc_specials() function called in pmap_bootstrap() should be mapped initially by initarm(). To create initial mapping initarm() function reserves proper number of l2 page tables. However the number of the l2 page tables does not take into account memory for: pmap_kernel_l2ptp_kva, pmap_kernel_l2dtable_kva, crashdumpmap, etc.

mmel accepted this revision.Apr 16 2017, 1:24 PM

Thanks for clarification, I see it now.

This revision is now accepted and ready to land.Apr 16 2017, 1:24 PM
Closed by commit rS317085: Increase number of L2 tables required for kernel bootstrap (authored by zbb). · Explain WhyApr 18 2017, 10:26 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
arm/
arm/
5 lines

Diff 27507

View Options

head/sys/arm/arm/machdep.c

Loading...