[PATCH] x86: don't rely on VMWare emulating PAT MSR correctly

[PATCH] x86: don't rely on VMWare emulating PAT MSR correctly

[Juergen Gross]

VMWare seems not to emulate the PAT MSR correctly: reaeding
MSR_IA32_CR_PAT returns 0 even after writing another value to it.

Detect this bug and don't use the read value if it is 0.

Commit bd809af16e3ab1f8d55b3e2928c47c67e2a865d2 ("x86: Enable PAT to
use cache mode translation tables") triggers this VMWare bug when the
kernel is booted as a VMWare guest.

Reported-by: Jongman Heo <jongman.heo@samsung.com>
Signed-off-by: Juergen Gross <jgross@suse.com>
Tested-by: Jongman Heo <jongman.heo@samsung.com>
---
 arch/x86/mm/pat.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/arch/x86/mm/pat.c b/arch/x86/mm/pat.c
index edf299c..7ac6869 100644
--- a/arch/x86/mm/pat.c
+++ b/arch/x86/mm/pat.c
@@ -234,8 +234,13 @@ void pat_init(void)
 	      PAT(4, WB) | PAT(5, WC) | PAT(6, UC_MINUS) | PAT(7, UC);
 
 	/* Boot CPU check */
-	if (!boot_pat_state)
+	if (!boot_pat_state) {
 		rdmsrl(MSR_IA32_CR_PAT, boot_pat_state);
+		if (!boot_pat_state) {
+			pat_disable("PAT read returns always zero, disabled.");
+			return;
+		}
+	}
 
 	wrmsrl(MSR_IA32_CR_PAT, pat);
 
-- 
2.1.2

Re: [PATCH] x86: don't rely on VMWare emulating PAT MSR correctly

[H. Peter Anvin]

On 12/16/2014 01:58 AM, Juergen Gross wrote:
> VMWare seems not to emulate the PAT MSR correctly: reaeding
> MSR_IA32_CR_PAT returns 0 even after writing another value to it.
> 
> Detect this bug and don't use the read value if it is 0.
> 
> Commit bd809af16e3ab1f8d55b3e2928c47c67e2a865d2 ("x86: Enable PAT to
> use cache mode translation tables") triggers this VMWare bug when the
> kernel is booted as a VMWare guest.
> 
> Reported-by: Jongman Heo <jongman.heo@samsung.com>
> Signed-off-by: Juergen Gross <jgross@suse.com>
> Tested-by: Jongman Heo <jongman.heo@samsung.com>

I presume cachability control is irrelevant for a VMware guest?  I'm
wondering if it would be better to just plain ignore the failure rather
than disabling PAT.

	-hpa

Re: [PATCH] x86: don't rely on VMWare emulating PAT MSR correctly

[Juergen Gross]

On 12/16/2014 05:47 PM, H. Peter Anvin wrote:
> On 12/16/2014 01:58 AM, Juergen Gross wrote:
>> VMWare seems not to emulate the PAT MSR correctly: reaeding
>> MSR_IA32_CR_PAT returns 0 even after writing another value to it.
>>
>> Detect this bug and don't use the read value if it is 0.
>>
>> Commit bd809af16e3ab1f8d55b3e2928c47c67e2a865d2 ("x86: Enable PAT to
>> use cache mode translation tables") triggers this VMWare bug when the
>> kernel is booted as a VMWare guest.
>>
>> Reported-by: Jongman Heo <jongman.heo@samsung.com>
>> Signed-off-by: Juergen Gross <jgross@suse.com>
>> Tested-by: Jongman Heo <jongman.heo@samsung.com>
>
> I presume cachability control is irrelevant for a VMware guest?  I'm
> wondering if it would be better to just plain ignore the failure rather
> than disabling PAT.

I'd rather disable PAT as this seems to be like a PAT erratum.

In case you want to keep PAT active I can redo the patch to not rely on
the read MSR value in the non-Xen case (under Xen reading the MSR is the
only way to obtain the correct settings).

Juergen

Re: [PATCH] x86: don't rely on VMWare emulating PAT MSR correctly

[Alok Kataria]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 1662 bytes --]

Hi,

On Tue, 2014-12-16 at 10:58 +0100, Juergen Gross wrote:
> VMWare seems not to emulate the PAT MSR correctly: reaeding
> MSR_IA32_CR_PAT returns 0 even after writing another value to it.
> 
> Detect this bug and don't use the read value if it is 0.
> 
> Commit bd809af16e3ab1f8d55b3e2928c47c67e2a865d2 ("x86: Enable PAT to
> use cache mode translation tables") triggers this VMWare bug when the
> kernel is booted as a VMWare guest.

Thanks for taking care of this, we are fine with this workaround. On
newer version of our product, where PAT is virtualized correctly on our
platform we will go back to using PAT, I will give it a spin sometime
later to verify it.

Acked-by: Alok N Kataria <akataria@vmware.com>

> Reported-by: Jongman Heo <jongman.heo@samsung.com>
> Signed-off-by: Juergen Gross <jgross@suse.com>
> Tested-by: Jongman Heo <jongman.heo@samsung.com>
> ---
>  arch/x86/mm/pat.c | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/x86/mm/pat.c b/arch/x86/mm/pat.c
> index edf299c..7ac6869 100644
> --- a/arch/x86/mm/pat.c
> +++ b/arch/x86/mm/pat.c
> @@ -234,8 +234,13 @@ void pat_init(void)
>  	      PAT(4, WB) | PAT(5, WC) | PAT(6, UC_MINUS) | PAT(7, UC);
>  
>  	/* Boot CPU check */
> -	if (!boot_pat_state)
> +	if (!boot_pat_state) {
>  		rdmsrl(MSR_IA32_CR_PAT, boot_pat_state);
> +		if (!boot_pat_state) {
> +			pat_disable("PAT read returns always zero, disabled.");
> +			return;
> +		}
> +	}
>  
>  	wrmsrl(MSR_IA32_CR_PAT, pat);
>  

Alok
ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥

Re: [PATCH] x86: don't rely on VMWare emulating PAT MSR correctly

[Juergen Gross]

x86 maintainers, could you please consider taking my patch?

There have been several reports now regarding this issue, all could be
fixed using my patch.

Juergen

On 12/17/2014 05:57 AM, Alok Kataria wrote:
> Hi,
>
> On Tue, 2014-12-16 at 10:58 +0100, Juergen Gross wrote:
>> VMWare seems not to emulate the PAT MSR correctly: reaeding
>> MSR_IA32_CR_PAT returns 0 even after writing another value to it.
>>
>> Detect this bug and don't use the read value if it is 0.
>>
>> Commit bd809af16e3ab1f8d55b3e2928c47c67e2a865d2 ("x86: Enable PAT to
>> use cache mode translation tables") triggers this VMWare bug when the
>> kernel is booted as a VMWare guest.
>
> Thanks for taking care of this, we are fine with this workaround. On
> newer version of our product, where PAT is virtualized correctly on our
> platform we will go back to using PAT, I will give it a spin sometime
> later to verify it.
>
> Acked-by: Alok N Kataria <akataria@vmware.com>
>
>> Reported-by: Jongman Heo <jongman.heo@samsung.com>
>> Signed-off-by: Juergen Gross <jgross@suse.com>
>> Tested-by: Jongman Heo <jongman.heo@samsung.com>
>> ---
>>   arch/x86/mm/pat.c | 7 ++++++-
>>   1 file changed, 6 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/x86/mm/pat.c b/arch/x86/mm/pat.c
>> index edf299c..7ac6869 100644
>> --- a/arch/x86/mm/pat.c
>> +++ b/arch/x86/mm/pat.c
>> @@ -234,8 +234,13 @@ void pat_init(void)
>>   	      PAT(4, WB) | PAT(5, WC) | PAT(6, UC_MINUS) | PAT(7, UC);
>>
>>   	/* Boot CPU check */
>> -	if (!boot_pat_state)
>> +	if (!boot_pat_state) {
>>   		rdmsrl(MSR_IA32_CR_PAT, boot_pat_state);
>> +		if (!boot_pat_state) {
>> +			pat_disable("PAT read returns always zero, disabled.");
>> +			return;
>> +		}
>> +	}
>>
>>   	wrmsrl(MSR_IA32_CR_PAT, pat);
>>
>
> Alok
> N�����r��y���b�X��ǧv�^�)޺{.n�+����{����zX��\x17��ܨ}���Ơz�&j:+v���\a����zZ+��+zf���h���~����i���z�\x1e�w���?����&�)ߢ^[f��^jǫy�m��@A�a��\x7f�\f0��h�\x0f�i\x7f
>