Page MenuHomeFreeBSD

Revert the IPI startup sequence to match what is described in the Intel Multiprocessor Specification v1.4.
ClosedPublic

Authored by jhb on Jan 29 2015, 4:23 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 8, 6:07 PM
Unknown Object (File)
Mon, Apr 8, 6:07 PM
Unknown Object (File)
Mon, Mar 18, 2:43 PM
Unknown Object (File)
Mon, Mar 18, 6:46 AM
Unknown Object (File)
Mar 8 2024, 12:41 AM
Unknown Object (File)
Mar 7 2024, 10:14 PM
Unknown Object (File)
Mar 4 2024, 7:13 AM
Unknown Object (File)
Jan 21 2024, 10:27 AM
Subscribers

Details

Reviewers
jhb
avg
Summary

Revert the IPI startup sequence to match what is described in the
Intel Multiprocessor Specification v1.4. The Intel SDM claims that
the INIT IPIs here are invalid, but other systems follow the MP
spec instead.

While here, fix the IPI wait routine to accept a timeout in microseconds
instead of a raw spin count, and don't spin forever during AP startup.
Instead, panic if a STARTUP IPI is not delivered after 20 us.

PR: 196542

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

jhb retitled this revision from to Revert the IPI startup sequence to match what is described in the Intel Multiprocessor Specification v1.4..
jhb updated this object.
jhb edited the test plan for this revision. (Show Details)
jhb added a reviewer: avg.
  • Compile.
  • Always set the level mode for all except the INIT de-assert.

Updating D1719: Revert the IPI startup sequence to match what is described in the

Intel Multiprocessor Specification v1.4.

emaste added inline comments.
sys/amd64/amd64/mp_machdep.c
1068

missing space theIntel
(and in i386)

  • Whitespace nit noted by emaste@.

Updating D1719: Revert the IPI startup sequence to match what is described in the

Intel Multiprocessor Specification v1.4.

  • Merge branch 'master' into x86_mp_startup

Updating D1719: Revert the IPI startup sequence to match what is described in the

Intel Multiprocessor Specification v1.4.

jhb added a reviewer: jhb.
This revision is now accepted and ready to land.Feb 6 2015, 6:20 PM