Re: [PATCH -tip 1/4] x86/idle: Fix argument type for MONITOR{,X} and MWAIT{,X} instructions

[Ingo Molnar <mingo@kernel.org>]

* Uros Bizjak <ubizjak@gmail.com> wrote:

> MONITOR and MONITORX expect 32-bit unsigned integer argument in %ecx
> and %edx registers. MWAIT and MWAITX expect 32-bit usigned int
> argument in %eax and %ecx registers.

Please always include a 0/4 cover letter as well for such series, which 
gives people a chance to reply to the whole series, instead of having 
to awkwardly pick a patch to reply to. :-)

Such as this general feedback:

I've applied this series with edits to the changelogs, note in 
particular:

 patch #1:

 - Changed verbiage from 'fix' to 'standardize to u32'. There was no 
   bug to fix, using 'long' instead of 'int' is at worst an ineffiency.

 patch #3:

 - Provided much needed historic context behind ;, \t, \n beautifiers 
   used in asm() statements. These aren't just random noise added in.

 patch #4:

 - Added in the current binutils version cutoff, to make it really 
   clear why we cannot use the mnemonic yet, and how far away we are 
   from doing so.

I've also extended the Cc: list in the commit to give people a chance 
to object, to patch #3 in particular:

  8ad521aaf743 ("x86/idle: Remove .s output beautifying delimiters from simpler asm() templates")

Thanks,

	Ingo