Messages in this thread | | | Subject | Re: [PATCH v4 10/10] x86/xsaves: Re-enable XSAVES | From | Dave Hansen <> | Date | Fri, 29 Apr 2016 13:32:15 -0700 |
| |
On 03/04/2016 10:12 AM, Yu-cheng Yu wrote: > We did not handle XSAVES* instructions correctly. There were issues in > converting between standard and compacted format when interfacing with > user-space. These issues have been corrected. > > Add a WARN_ONCE() to make it clear that XSAVES supervisor states are not > yet implemented.
The reason I haven't acked this patch is that I want to be _sure_ that we've audited all of the call paths that access the XSAVE buffer to ensure that they can all either handle the XSAVES format *or* don't care for whatever reason.
Could you share the steps that you've taken to assure yourself that all of the call paths are handled and we don't have more bugs?
FWIW, this was the single biggest lesson I learned from the failure the last time this got turned on: we simply didn't go look for all the places that the new format had to be handled. Let's be sure we don't repeat that.
If we get this *wrong* in another user/kernel interface like we did for ptrace and the signal save/restore and we ever enable a supervisor state we've got an almost certain immediate root hole of some kind. I think we need to exercise some serious caution here. Thank $DEITY we don't have any supported supervisor states at the moment.
| |