Received-SPF: pass (google.com: domain of linux-kernel+bounces-84258-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Message-ID: <3da6b6a5-dc66-47b7-b824-84e418695882@intel.com>
Date: Wed, 28 Feb 2024 08:07:28 +0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v2 0/2] x86/cpu: fix invalid MTRR mask values for SEV or
 TME
Content-Language: en-US
To: Dave Hansen <dave.hansen@intel.com>, Paolo Bonzini <pbonzini@redhat.com>
CC: <linux-kernel@vger.kernel.org>, <kvm@vger.kernel.org>, Zixi Chen
	<zixchen@redhat.com>, Adam Dunlap <acdunlap@google.com>, "Kirill A .
 Shutemov" <kirill.shutemov@linux.intel.com>, Xiaoyao Li
	<xiaoyao.li@intel.com>, Kai Huang <kai.huang@intel.com>, Dave Hansen
	<dave.hansen@linux.intel.com>, Thomas Gleixner <tglx@linutronix.de>, "Ingo
 Molnar" <mingo@kernel.org>, <x86@kernel.org>, <stable@vger.kernel.org>
References: <20240131230902.1867092-1-pbonzini@redhat.com>
 <2b5e6d68-007e-48bd-be61-9a354be2ccbf@intel.com>
 <CABgObfa_7ZAq1Kb9G=ehkzHfc5if3wnFi-kj3MZLE3oYLrArdQ@mail.gmail.com>
 <CABgObfbetwO=4whrCE+cFfCPJa0nsK=h6sQAaoamJH=UqaJqTg@mail.gmail.com>
 <CABgObfbUcG5NyKhLOnihWKNVM0OZ7zb9R=ADzq7mjbyOCg3tUw@mail.gmail.com>
 <eefbce80-18c5-42e7-8cde-3a352d5811de@intel.com>
 <CABgObfY=3msvJ2M-gHMqawcoaW5CDVDVxCO0jWi+6wrcrsEtAw@mail.gmail.com>
 <9c4ee2ca-007d-42f3-b23d-c8e67a103ad8@intel.com>
 <CABgObfYttER8yZBTReO+Cd5VqQCpEY9UdHH5E8BKuA1+2CsimA@mail.gmail.com>
 <7e118d89-3b7a-4e13-b3de-2acfbf712ad5@intel.com>
 <3807c397-2eef-4f1d-ae85-4259f061f08e@intel.com>
 <eff34df2-fdc1-4ee0-bb8d-90da386b7cb6@intel.com>
 <97b66d4d-f520-46c9-8164-ce5b2e4d5642@intel.com>
From: Yin Fengwei <fengwei.yin@intel.com>
In-Reply-To: <97b66d4d-f520-46c9-8164-ce5b2e4d5642@intel.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?bXFjVHlWZnlpSHNaSUkzR3g1cGU5SUUzam8wSm9Gaml6M3F4c3pEem1qVzlQ?=
 =?utf-8?B?Zy9QLzkyQkRrNGtLdkJUSEhyV0oxWFk5VFVaQXN1UHByQnd1SkpTNUkxR3ZV?=
 =?utf-8?B?ZktKYzhwUkI0VG13SXBHOUlCVGM2Q3o0Ym15eDkzK0dBTW5FSWdUT1ovWWkx?=
 =?utf-8?B?VXhRZEh6NjRYb01yLytlYTMrRnp2ZndYYTBlYzdxVzcvYjBEdHFTVWFiR2hP?=
 =?utf-8?B?SHprRVFjYk4yTXg5Y0pQelNZVnBDdjdlRWZPZERSOHY4c1daZkhyMjdqVzJz?=
 =?utf-8?B?OFBKQTBKYVRmdFl1bTArL3diaXJCa0FKNWcxMXAxaldmMjZpRU1za0FvUUI0?=
 =?utf-8?B?VHFST3N0KytFRERWQ1NjajNQYmdKZCtVMlRYSTRNOTY2VDd5TEhwVjdvY1p3?=
 =?utf-8?B?Uk1teGlOSC91MEFRdm9UTjh0Y0xNOGNGNysrUk1hNFl2WnBFdjhJU3YrUUlm?=
 =?utf-8?B?S2lxR1VnZExiaFZYS1NOWlpZRDdSNmpRbGJqVGVzcFR0d1ZnTnZjYnBnSXpz?=
 =?utf-8?B?bW90aWJBbWNCaDVDRmtScmlOQThINUNpTjdReXRUcGNVRU1iK3ppK0IyNW9l?=
 =?utf-8?B?eUF0TDI0YVVmNmI1eFRKVFR2Vm94OUZFU3lBWkQybU5RaVZCaWtPakJxL1Nh?=
 =?utf-8?B?YlM4NzdYcXJGTHcrdVU2SWhubDNrS1NSaFZqUTlQZWdoWTVmUTJTcXAwWFo3?=
 =?utf-8?B?QldLQ1NWZm5uVXQwL2p4ZU1EZVowRjRLeUFGSDRyT3ZNVVpmcVkrZE5BQ2kw?=
 =?utf-8?B?QzRCSTl5UmdydDFGUWxweTByc1VWby9YOW5MREZ6RHN3K0VnalREeFpLaGlC?=
 =?utf-8?B?bTdoNytvcHpxaEtWU1BFeWk0bi9YWm1zdW5kTi9yRVhqdUpLb0Y1VFFOQXBi?=
 =?utf-8?B?MGh5bDJIeWdtNTN6NVJQOHpwaVF0TnFnVlNmQUJ3QXY2aUtqT1FTRXJMWmVN?=
 =?utf-8?B?SVZ1akVCL2pOS291WitYd2oxZi9qMkNyTm82b1dIMU9yaklPY29TQWRpME5J?=
 =?utf-8?B?Z0lFOHJrWXhPM0RnTmVWTGlrKzZRTmxFMzR2bXNMWGxaUkdaRllMbSs2cytx?=
 =?utf-8?B?YXFmUUFqNFNoajZrOXJQNFBqSFMzb2JiNnJ1bTYxT2FDeDc4UGdaNUwyQzZi?=
 =?utf-8?B?YVlwcTVFczF3TlZuV1N1MnNNSGcwcktJR2JXeVJQTXdjN25VZ3dTVkt2RTQ1?=
 =?utf-8?B?K0pMaTNScEFZTmZCMUZmZlBoUlo4Si9MMkkzUGlxb2N4R24xU2tGTFEwdHBs?=
 =?utf-8?B?YmM5M1RCb3FDUzdHQlJuMmJmc1hValB6NW15T3Vob0JMRUZsWGozbEFpM1dj?=
 =?utf-8?B?VTR4WlozSWV0M0JRMGNhVXlZc1A5aEYyZWpyaU5KZENEK01rQ3hOZVBDMXF3?=
 =?utf-8?B?TUdqQ0JKZWxacTJTYSs0elZNNXZ6ZkVvQ01MSnhZeE9MVTIrZGZ2Uk9nVU4y?=
 =?utf-8?B?cDB3VkhISDFhWkY2Q1lyKzlDeUMva3MwM05mZ3ZrK05LdEZFU2tNZzkrZVdv?=
 =?utf-8?B?eWpodXNnQi9SaEw5TVl1SVRvVHE5YTEzM29pTzA0MlViOHNIakI4RWJZZ0ZQ?=
 =?utf-8?B?ekc5MDRxcnBYOU81eVJWbTkxZSthYXNqSEExaFlFbEpDcm1vOWE5MG5vbmkv?=
 =?utf-8?B?NWIxVm13ckZLY1FqY00zZXVrK1NobElDZko5QWkzZUxNRW03bElwcWxTcWlK?=
 =?utf-8?B?QU1ZZlA0aVlsTHNKSWRCbGdHQ3lmRks0VDdUWFErS3VxazNVK3ZBcGN4c3Zl?=
 =?utf-8?B?WjkwZHBKaCtPd052NXJiSzVwSWVLcHN2U3JmWGMxQnkwVC9mQUFFaWgrYytm?=
 =?utf-8?B?SllsQnltcUZyM29HZnREZDNRUGZmNTlkcjhpS3JLRUkrdEszeUtseWtlUFhE?=
 =?utf-8?B?czJqUXVGNm0zazF5S0tSdDczQiswOWVJMHRORzlYVXNkbElCb2dVTmR3RlVY?=
 =?utf-8?B?RHpRWk5nUEs5ckQvSUhlVnQ3bndhT3FreEhjREZRNkFXc01Ob2l2M2Y4T3Aw?=
 =?utf-8?B?dW1mMzhSdWtnd2VieCtYQmxIdlFhc1lvY0krOEdHUzFBYjd6c0xUUnY4NVcy?=
 =?utf-8?B?dEdENmR1c0NSQ1crVFAvd3YrNUhvRWRHR2xQblRpOUlvY0p0cDhxRk0wQmQz?=
 =?utf-8?B?VjRFSkNWNWlXUmgwbW1PMGl5c0Q3OTBiS3laOEV5eWdmaXFRWEJ0UzRCc3Vk?=
 =?utf-8?B?UWc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: e2e84167-76dd-44aa-0cfb-08dc37f15993
X-MS-Exchange-CrossTenant-AuthSource: CO1PR11MB4820.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Feb 2024 00:08:11.2317
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: axibIoY6jz9SxtunwcowezpPDHB/bhJpde+V6Jwy/YTS0Nq8+yw8yh1G4RIJzboF8YNDPryy1WAqELpfqhHOvg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR11MB5311
X-OriginatorOrg: intel.com


On 2/27/24 21:30, Dave Hansen wrote:
> On 2/26/24 22:08, Yin Fengwei wrote:
>>>> https://lore.kernel.org/all/20240222183926.517AFCD2@davehans-spike.ostc.intel.com/
>>> If it _also_ fixes the problem, it'll be a strong indication that it's
>>> the right long-term approach.
>> I tried your patchset on a Sapphire machine which is the only one broken machine
>> I can access today. The base commit is 45ec2f5f6ed3 from the latest linus tree.
>>
>> Without your patchset, the system boot hang.
>> With your patchset, the system boot successfully.
> 
> Yay!  Thanks for testing.
My pleasure.


Regards
Yin, Fengwei