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)
Sat, Mar 16, 9:59 PM
Unknown Object (File)
Dec 23 2023, 12:43 AM
Unknown Object (File)
Dec 13 2023, 1:23 AM
Unknown Object (File)
Nov 20 2023, 7:56 PM
Unknown Object (File)
Nov 15 2023, 8:33 AM
Unknown Object (File)
Nov 10 2023, 4:11 AM
Unknown Object (File)
Oct 14 2023, 7:35 AM
Unknown Object (File)
Oct 9 2023, 3:04 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().