Re: [PATCH 3/4] x86/umip: Identify the str and sldt instructions
- Date: Mon, 13 Nov 2017 09:12:03 +0100
- From: Ingo Molnar <mingo@xxxxxxxxxx>
- Subject: Re: [PATCH 3/4] x86/umip: Identify the str and sldt instructions
* Ricardo Neri <ricardo.neri-calderon@xxxxxxxxxxxxxxx> wrote:
> The instructions str and sldt are not emulated in any case. Thus, it made
> sense to not implement functionality to identify them. However, a
> subsequent commit will introduce functionality to warn about the use of
> all the instructions that UMIP protect, not only those that are emulated.
> A first step for that is the ability to identify them.
> Plus, now that str and sldt are identified, we need to explicitly avoid
> their emulation (i.e., not rely on unsuccessful identification). Group
> togehter all the cases that we do not want to emulate: str, sldt and user
> long mode processes.
Did you notice how in all your previous patches (both in the code and in the
changelogs) I have manually fixed up the capitalization of these instruction
The capitalized form is much more readable, especially with seriously overloaded
acronyms such as 'str' ...
You now repeat the same bad pattern, in fact you regress existing code:
> - /* SLDT AND STR are not emulated */
> + /* Do not emulate sldt, str or user long mode processes. */
Please be more careful with such details, and please fix & resend this series.