On 22.01.2021 18:44, Paul Gofman wrote:
xsave is part of SSE2, not AVX, and it should ignore unsupported
requested features, so the patch should be fine as is on hardware
without AVX. xsave needs, however, to be enabled by OS, so we may need
a feature check if we want to support OSes without xsave enabled.

There was a real bug with that:
https://bugs.winehq.org/show_bug.cgi?id=50271.


Looking at the bug report, I think it was about missing AVX support, not missing XSAVE support, so it should be fine with my patches.


I would need to do testing to be sure, but I noticed that our feature detection looks suspicious. I would expect that we should do something like the attached patch (but if it's right, it would also need fixes in ntdll).


Jacek