Received-SPF: pass (google.com: domain of linux-kernel+bounces-145942-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Message-ID: <afe34183-4320-488f-bc03-075589bb6065@intel.com>
Date: Tue, 16 Apr 2024 09:43:06 +1200
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v3 1/5] x86/kexec: do unconditional WBINVD for bare-metal
 in stop_this_cpu()
To: Borislav Petkov <bp@alien8.de>
CC: "Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>,
	<linux-kernel@vger.kernel.org>, <x86@kernel.org>, <dave.hansen@intel.com>,
	<tglx@linutronix.de>, <mingo@redhat.com>, <hpa@zytor.com>, <luto@kernel.org>,
	<peterz@infradead.org>, <rick.p.edgecombe@intel.com>,
	<thomas.lendacky@amd.com>, <ashish.kalra@amd.com>, <chao.gao@intel.com>,
	<bhe@redhat.com>, <nik.borisov@suse.com>, <pbonzini@redhat.com>,
	<seanjc@google.com>
References: <cover.1712493366.git.kai.huang@intel.com>
 <33b985a8f4346f4bcf0944eaf37193a906b11af3.1712493366.git.kai.huang@intel.com>
 <a4o4wlaojm5am4hc5yhr2mpn7clm3sjy5vx3w76ahm52lhxvwr@msdcnkcj3i6k>
 <4875349a-90e5-4a04-8a56-7d172b17e245@intel.com>
 <20240415175912.GAZh1q8GgpY3tpJj5a@fat_crate.local>
Content-Language: en-US
From: "Huang, Kai" <kai.huang@intel.com>
In-Reply-To: <20240415175912.GAZh1q8GgpY3tpJj5a@fat_crate.local>
Content-Type: text/plain; charset="UTF-8"; format=flowed
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?cG1yRUNqdlpZSEhZL3RkQVZyd0JSSmFkdm1Ld3pnUnE1Sk50dkRHOG5hMGs0?=
 =?utf-8?B?MmoyYko4ZWh6UVd4UkpyNU5oTXduNGZPbG8yMlM5NHhydHl4Z2JqTUFpSkpn?=
 =?utf-8?B?cXNXS3pIU2VIZm9oK05tUjhVOGpKRnh4YWlVUElmbStqRE9IZFpSK3JPSnZ3?=
 =?utf-8?B?anRXbjQ3TVo2VjFYeEtvakNzYkY0SHNob1NMdlVscHBLTjlvUDVzKzVWNEI2?=
 =?utf-8?B?cktqRlJmeVAwQUhIZ2xQMVJoODczZkxqNFE0dFd0RHRKSkk2UjFtS3Y1V2Jq?=
 =?utf-8?B?dXVYNGNXdEkwMldqUENYaGcxUUFnT1NieEpybitPS2hrSS8zRFBBVGF4dlli?=
 =?utf-8?B?aG9hSFhYRkNveXZHY3BabjQwN3JpUGVSZWhoVjJHdEpMMlpJaGkzNFFMY2FM?=
 =?utf-8?B?YzZJeFU2MkJ2V0srR0dPb3RxSmE2RnlBdm5zQlhnMGNvaGh6a0NuR1B2YmE1?=
 =?utf-8?B?Nyswb2NKTHFjbkRaa0lTcEptS1VNOXhOb1lNWkdUUTVML0ZNcDBQNmFseHE5?=
 =?utf-8?B?OHkwS2Rkckt2UWlpakJBNi93cFoveXQ5YXE0WSt3d3NCZFVBTGIyQUxOZlB2?=
 =?utf-8?B?WWN1Mnk3UllMQ2lKT1QxdjNQUEZpQ2JlcTY5UnhYNGc0YjZJci9Ub3RlVlE0?=
 =?utf-8?B?cjdPQ2VhZTNPTVBCTHBKQTJhUWhvSGRaN25OYnpQaDQrR0dqdjhTZVhlT2JL?=
 =?utf-8?B?TitwSTMyb0ZBQW1OS29JSElVVTJsZ0MyQldVN3FMdFhyRHVRM1I5aEViSnd3?=
 =?utf-8?B?bEdyZ1RUS2NrQ1pEMVh4Wm9seXBtcUtWWnJpUjhsV2tWRmRLcnYrcklPWnNy?=
 =?utf-8?B?YnhlZHZzSjRraVlIeGcvV2c0UFQ4M0VvRDg1WjhiNEVPeEVNdzd2UFQyMXdL?=
 =?utf-8?B?Rno0NllZb28xbG5MY3lielg5ckxrS2h6YW5vMktNVWZPMm9HUEcxWmlHYkFn?=
 =?utf-8?B?RkQzYXlnSVRqSFRpM3F4eUdIL09uRzhncS80UmxPYnNmZGN6UVRFYVJEUUFX?=
 =?utf-8?B?KzVJUms2aUlpMVhBbTduWXVOSllFYWV3eTRYaWpid1g0SndGVk01dUVWSVZ5?=
 =?utf-8?B?Z1JkZGVrMFBsb1Vtc3pMbWhFTkJsRmgzbUhob1d5T202SXFuZUdpeXV4Qlc3?=
 =?utf-8?B?MlloQ1pLY3dPVlNOaWRXWHEydEZBNDlFbEhIVllraXZIZHovZm5jUzVvNkpw?=
 =?utf-8?B?VE9rUzZ5NWdMOGs2eWliUUdiRStmNWp4anJPV0oyaFAxcUVySEd3YmE5RjFB?=
 =?utf-8?B?WExNQXJHaVBzRmdURWgvYUl1VFRGblJRWUtTOUw4MGtUVy9PUEpBanUwbjhq?=
 =?utf-8?B?OTAzRGNkczVKL3VmYlpyeUYyM1h2YmxxTWtmODJXRmxnaHlDeDdvZTVYMnM3?=
 =?utf-8?B?aU1lUFJRcWp2czNaOXFRZk1lOVJuTTg0V1YxTFZDcWhPMUo5TnlQR1YvemVl?=
 =?utf-8?B?VTZWeUNvdWw3cVVHdllzci8vaDF5WTJ1MzZnTm5CQ0Nta2ZnejN3eXNOT1BS?=
 =?utf-8?B?eWc3ck9GckFINmdiM0JIZVVEVXI5UkJoSG84eTN5TDd6ajh3b0lOc3BTdWUv?=
 =?utf-8?B?RDNpWXdhSVE4YlBkZ3k0Qm8rNnBaWTd3UHlCWHVZcWRrY3g2ZEp4WENLQjZY?=
 =?utf-8?B?K2RJUkJUNDIvUzZ2RlVFWS9McG1FSitKSUszUm1sKzJrVzZoN0RxVzIvSXdz?=
 =?utf-8?B?a1R1WVdoLzNTRVhoZGlNRXJkREJYMThtejgvNlBTcU82a0ozMG5LYndhZjUz?=
 =?utf-8?B?ck9Ja0tKeTVQaWhMR2djTG50dW9xdkpzdlRZdVhzYjFTU1N1a2NRZXdxRGM4?=
 =?utf-8?B?VkhMQTJ4bFY2am9HYVZMNk9aaGdVSVpFVHZ0b0FJTHNsbG1MVG93Qy94MkNV?=
 =?utf-8?B?SzVnMmNSemVyNVFySGxKaVl1T1E5VUlueDhRV2dUMVBsNUh3b3JId05sNnNH?=
 =?utf-8?B?Z2dRY2x6cVB5R3FkTDJOSWJjTXpZVmxmQjFWc2xtc1Z4YmpZR0lXWkczUDRr?=
 =?utf-8?B?REpxVjk3TURZWGw2d3NhSTcvSEN3ZmRjTHdlTThhQmFldHpQeGVtZTZYUEZO?=
 =?utf-8?B?ZFVSOEhYejM1SzJvVnFFQ005UGhaRTdJaXdlMjVrSTZ0NGhoSzVIT1FMZFQw?=
 =?utf-8?Q?faAFknEAHabyTecChQ/ww86Jb?=
X-MS-Exchange-CrossTenant-Network-Message-Id: ea07818a-6aad-4fdb-fd00-08dc5d951053
X-MS-Exchange-CrossTenant-AuthSource: BL1PR11MB5978.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Apr 2024 21:43:18.6908
 (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: dbfBVW+M6bIG8xcl3F1M1s5AYC++mUDBtc58dK7XJT/zCzn87N8fn3g+KEawKifH8RoV+ZLsPtBZT/mKqcregg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH3PR11MB8443
X-OriginatorOrg: intel.com


On 16/04/2024 5:59 am, Borislav Petkov wrote:
> On Thu, Apr 11, 2024 at 09:54:13AM +1200, Huang, Kai wrote:
>> Could you be more specific?
>>
>> I was following Boris's suggestion to summerize all the discussion around
>> the "unconditional WBINVD" issue.
>>
>> https://lore.kernel.org/linux-kernel/20240228110207.GCZd8Sr8mXHA2KTiLz@fat_crate.local/
>>
>> I can try to improve if I can know specifically what should be trimmed down.
> 
> No, keep it this way. I've yet to see someone complaining from too
> verbose commit message while doing git archeology.
> 
> If it is too verbose to a reader, then that reader can jump over the
> paragraphs.
> 

Yeah as replied to Kirill I am keeping it.  Thanks for the feedback.