Web lists-archives.com

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 
mnenonics?

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.

Thanks,

	Ingo