Page MenuHomeFreeBSD

amd64: correct size of the SSE area in the xsave layout
AcceptedPublic

Authored by kib on Dec 12 2021, 2:52 AM.
Tags
None
Referenced Files
F132365450: D33390.diff
Thu, Oct 16, 6:29 AM
Unknown Object (File)
Mon, Oct 13, 1:14 PM
Unknown Object (File)
Mon, Oct 13, 12:36 AM
Unknown Object (File)
Mon, Oct 13, 12:36 AM
Unknown Object (File)
Thu, Sep 18, 12:02 AM
Unknown Object (File)
Aug 26 2025, 3:16 AM
Unknown Object (File)
Aug 26 2025, 2:56 AM
Unknown Object (File)
Aug 25 2025, 9:13 PM

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

kib requested review of this revision.Dec 12 2021, 2:52 AM
emaste added inline comments.
sys/amd64/amd64/fpu.c
487

Indeed, 16 128-bit XMM registers occupy 256 bytes.

I wonder if xsave_area_desc[1].size = 256 would be more clear here.

This revision is now accepted and ready to land.Dec 12 2021, 6:18 PM
mpratt_google.com added inline comments.
sys/amd64/amd64/fpu.c
487

Alternatively/additionally, a reference to the documentation source for these values would be helpful. e.g., Intel SDM Vol 1, Ch. 13.4.1 "Legacy Region of an XSAVE Area", Table 13-1 "Format of the Legacy Region of an XSAVE Area" shows these sizes and offsets.