Page MenuHomeFreeBSD
Differential D16893

Add pmap_activate_boot() on i386.
ClosedPublic
Actions

Authored by kib on Aug 24 2018, 11:24 PM.
Tags
None
Referenced Files
F146628091: D16893.id47270.diff
Wed, Mar 4, 5:57 AM
F146627123: D16893.id47270.diff
Wed, Mar 4, 5:45 AM
Unknown Object (File)
Fri, Feb 27, 3:13 AM
Unknown Object (File)
Mon, Feb 16, 9:44 AM
Unknown Object (File)
Mon, Feb 9, 3:14 PM
Unknown Object (File)
Sun, Feb 8, 3:25 PM
Unknown Object (File)
Jan 27 2026, 5:48 AM
Unknown Object (File)
Jan 18 2026, 12:03 AM
View All Files
Subscribers
imp
rpokala

Details

Reviewers
alc
markj
Commits
rS338313: Remove dead code in i386 cpu_throw().
rS338312: Unify amd64 and i386 vmspace0 pmap activation.
Summary

Move pmap_activate_boot() invocation to mp_x86.c:init_secondary_tail().
Remove dead code from i386 cpu_throw().

Suggested by: alc

Test Plan

Boot SMP i386.

Diff Detail

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

Event Timeline

kib created this revision.Aug 24 2018, 11:24 PM
Herald added subscribers: rpokala, imp. · View Herald TranscriptAug 24 2018, 11:24 PM
Harbormaster completed remote builds in B19161: Diff 47270.Aug 24 2018, 11:24 PM
kib updated this revision to Diff 47271.Aug 24 2018, 11:29 PM

Remove unused local label.

Harbormaster completed remote builds in B19162: Diff 47271.Aug 24 2018, 11:29 PM
markj accepted this revision.Aug 24 2018, 11:57 PM
markj added inline comments.
sys/i386/i386/pmap.c
5810 ↗(On Diff #47271)

Extra whitespace? Looks like there's a mix of both styles in this file.

This revision is now accepted and ready to land.Aug 24 2018, 11:57 PM
kib marked an inline comment as done.Aug 25 2018, 12:13 AM
kib added inline comments.
sys/i386/i386/pmap.c
5810 ↗(On Diff #47271)

It is tab.

kib updated this revision to Diff 47272.Aug 25 2018, 12:14 AM
kib marked an inline comment as done.

Use space instead of tab.

This revision now requires review to proceed.Aug 25 2018, 12:14 AM
Harbormaster completed remote builds in B19163: Diff 47272.Aug 25 2018, 12:14 AM
alc accepted this revision.Aug 25 2018, 3:09 AM
This revision is now accepted and ready to land.Aug 25 2018, 3:09 AM
Closed by commit rS338312: Unify amd64 and i386 vmspace0 pmap activation. (authored by kib). · Explain WhyAug 25 2018, 3:21 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
amd64/
amd64/
2 lines
i386/
i386/
16 lines
include/
1 line
x86/
x86/
3 lines

Diff 47287

View Options

head/sys/amd64/amd64/mp_machdep.c

Loading...
View Options

head/sys/i386/i386/pmap.c

Loading...
View Options

head/sys/i386/include/pmap.h

Loading...
View Options

head/sys/x86/x86/mp_x86.c

Loading...