Page MenuHomeFreeBSD

x86: Don't bother copying the FPU state for cpu_copy_thread().
AbandonedPublic

Authored by jhb on Mar 3 2021, 12:12 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 14, 2:07 PM
Unknown Object (File)
Thu, Nov 14, 9:33 AM
Unknown Object (File)
Sat, Oct 19, 8:01 PM
Unknown Object (File)
Oct 10 2024, 11:14 PM
Unknown Object (File)
Oct 2 2024, 6:56 AM
Unknown Object (File)
Sep 29 2024, 8:30 PM
Unknown Object (File)
Sep 27 2024, 9:25 AM
Unknown Object (File)
Sep 27 2024, 3:19 AM
Subscribers

Details

Reviewers
kib
Summary

The copied state never gets used since the FPU INITDONE flags are
always cleared in the new threads causing the first DNA trap to
overwrite the saved state with the appropriate initial state.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 37516
Build 34405: arc lint + arc unit

Event Timeline

jhb requested review of this revision.Mar 3 2021, 12:12 AM
This revision is now accepted and ready to land.Mar 3 2021, 3:25 AM

So now I'm a bit torn on this due to the language for pthread_create() I found in the other review. If the "floating point environment" includes register values, then what we should be doing here instead is avoid clearing the INITDONE flags in the pcb for cpu_copy_thread().