x86: initialize initial FPU state earlier
ClosedPublic
Actions

Authored by kib on May 27 2021, 4:37 PM.

Tags

None

Referenced Files

	F97483264: D30512.diff
	Sun, Sep 29, 3:32 PM

	F97410326: D30512.diff
	Sun, Sep 29, 3:34 AM

	Unknown Object (File)
	Mon, Sep 23, 11:00 AM

	Unknown Object (File)
	Sat, Sep 7, 4:20 AM

	Unknown Object (File)
	Aug 23 2024, 8:39 PM

	Unknown Object (File)
	Aug 22 2024, 6:00 AM

	Unknown Object (File)
	Aug 11 2024, 1:42 AM

	Unknown Object (File)
	Aug 9 2024, 5:33 PM

View All 29 Files

Subscribers

None

Details

Reviewers

jhb

Commits

rGc56de177d282: x86: initialize initial FPU state earlier

Summary

Make it under SI_SUB_CPU sysinit, instead of much later SI_SUB_DRIVERS. The SI_SUB_DRIVERS survived from times when FPU used real ISA attachment, now it is only pnp stub claiming id.

PR: 255997

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

kib requested review of this revision.May 27 2021, 4:37 PM

kib created this revision.

jhb added inline comments.May 27 2021, 9:57 PM

sys/amd64/amd64/fpu.c
498	Can this be SI_ORDER_ANY? I can't recall if there is a specific order to the SI_SUB_CPU inits.

Use SI_ORDER_ANY on amd64, same as on i386.

jhb accepted this revision.May 28 2021, 5:49 PM

This revision is now accepted and ready to land.May 28 2021, 5:49 PM

Closed by commit rGc56de177d282: x86: initialize initial FPU state earlier (authored by kib). · Explain WhyMay 28 2021, 6:39 PM

This revision was automatically updated to reflect the committed changes.

kib added a commit: rGc56de177d282: x86: initialize initial FPU state earlier.

Revision Contents
Changeset List

Path

Size

sys/

amd64/

fpu.c

2 lines

i386/

npx.c

2 lines

Diff 90020

View Options

sys/amd64/amd64/fpu.c