Page MenuHomeFreeBSD

x86: Rename {stop,start}_emulating to fpu_{enable,disable}
ClosedPublic

Authored by jhb on Oct 9 2023, 4:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 2, 6:59 AM
Unknown Object (File)
Mon, Oct 21, 1:40 PM
Unknown Object (File)
Mon, Oct 21, 6:10 AM
Unknown Object (File)
Sep 23 2024, 8:01 PM
Unknown Object (File)
Sep 22 2024, 1:53 AM
Unknown Object (File)
Sep 20 2024, 1:03 AM
Unknown Object (File)
Sep 18 2024, 11:12 AM
Unknown Object (File)
Sep 18 2024, 8:18 AM

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 53902
Build 50793: arc lint + arc unit

Event Timeline

sys/amd64/amd64/fpu.c
803

fpu_enable()?

sys/amd64/amd64/fpu.c
803

I chose clts() here due to the comment above explicitly referencing %cr0.TS, but perhaps fpu_enable() is more correct?

sys/amd64/amd64/fpu.c
803

I do not think the specific name there matters. Perhaps the comment should stop referencing lazy mode.

Might be, it is time to remove the workaround as well (I am not too sure about it).

markj added inline comments.
sys/amd64/amd64/fpu.c
803

I slightly prefer to be consistent and write fpu_enable().

This revision is now accepted and ready to land.Oct 11 2023, 1:38 PM