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)
Fri, Dec 6, 11:34 AM
Unknown Object (File)
Nov 26 2024, 8:44 AM
Unknown Object (File)
Nov 22 2024, 12:32 PM
Unknown Object (File)
Nov 19 2024, 12:59 PM
Unknown Object (File)
Nov 18 2024, 12:42 AM
Unknown Object (File)
Nov 16 2024, 4:23 AM
Unknown Object (File)
Nov 16 2024, 2:24 AM
Unknown Object (File)
Nov 14 2024, 2:07 PM
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().