Received-SPF: pass (google.com: domain of linux-kernel+bounces-84226-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Message-ID: <b5045aac-e0d9-4ef4-b06e-0e7c01c2d63d@intel.com>
Date: Wed, 28 Feb 2024 12:33:02 +1300
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCHv8 06/17] x86/mm: Make
 x86_platform.guest.enc_status_change_*() return errno
Content-Language: en-US
To: "Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>, Thomas Gleixner
	<tglx@linutronix.de>, Ingo Molnar <mingo@redhat.com>, Borislav Petkov
	<bp@alien8.de>, Dave Hansen <dave.hansen@linux.intel.com>, <x86@kernel.org>
CC: "Rafael J. Wysocki" <rafael@kernel.org>, Peter Zijlstra
	<peterz@infradead.org>, Adrian Hunter <adrian.hunter@intel.com>, "Kuppuswamy
 Sathyanarayanan" <sathyanarayanan.kuppuswamy@linux.intel.com>, Elena
 Reshetova <elena.reshetova@intel.com>, Jun Nakajima <jun.nakajima@intel.com>,
	"Rick Edgecombe" <rick.p.edgecombe@intel.com>, Tom Lendacky
	<thomas.lendacky@amd.com>, "Kalra, Ashish" <ashish.kalra@amd.com>, "Sean
 Christopherson" <seanjc@google.com>, Baoquan He <bhe@redhat.com>,
	<kexec@lists.infradead.org>, <linux-coco@lists.linux.dev>,
	<linux-kernel@vger.kernel.org>, Dave Hansen <dave.hansen@intel.com>
References: <20240227212452.3228893-1-kirill.shutemov@linux.intel.com>
 <20240227212452.3228893-7-kirill.shutemov@linux.intel.com>
From: "Huang, Kai" <kai.huang@intel.com>
In-Reply-To: <20240227212452.3228893-7-kirill.shutemov@linux.intel.com>
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?d01kdVovVUdlM3JkQkhwdHY4Q3Z6eTk4TnQyUXRJSkVETWU4ZFE3MFZSdzhO?=
 =?utf-8?B?SzlGbCtrV1lsMzdVUmV4cDlMSWJJWmdZTmdkWWkxVEQxNGZjUnNZNnN6cWtw?=
 =?utf-8?B?WWpocWlMWTVvQ1A3VVVQSTBCY0lReFl4czc3azlBc2pzekp2MTFId0l0SUVl?=
 =?utf-8?B?RkxUSjVINlBsUC8wRGlTRFFpOU10T2x1M1BPaVF2clljK0ZpUWppWG9vVnN6?=
 =?utf-8?B?NlRlbm90QjJrdHM4Tk9oT2cyT0FwOE8wU0JuYVMyYzdQLytsTHNYa0VxWUx4?=
 =?utf-8?B?M2gxYmtmSGJGU29rOWkvSENUSHkvVHRsQzFlZUJoMHkxeldSSytwMzFGQ1My?=
 =?utf-8?B?dExLeTVERHVNSEREa0RoU0tIVnAzQWtIbkZUNjBMMDg3aFdFTmRWRG9lYjU2?=
 =?utf-8?B?T3l4M3VGWGNnRGZHVUxqUnJ5SC95V1N0b2ZWSnRxVGkyVXpwdVlLbVo1SkJ6?=
 =?utf-8?B?WjdqalovSXp2UzJmODZuMUN3NjdjUlZEajljZ2FRdGNmREU2c3Q2bVN1VytO?=
 =?utf-8?B?V3pGQWJoVDNQWDMzNjdnemtzUnc5NUExUExmd3Y5Wk9udDBQejhRUGlXRFJ2?=
 =?utf-8?B?ZTFiVVF1VHBLNjRnZFh0N1VvTmMzTDhvZGg3WjlrQlJtQVdzdVMyWmhtYUFz?=
 =?utf-8?B?TGNEaVhHWFpvMElYVFAzMlVvOHJhOTRVOXlGY2hZc01rRFRuRUVTT1pZMmJx?=
 =?utf-8?B?NmZsa0xlZjBwNXdNczU4WTlnRGNmTFNOMFVyN0pSd1lQc29YRVVhVXVmdEky?=
 =?utf-8?B?OWQrbWQ2WDFwUU40QmNtUW9RYU1PWEcwbW9Jbmx0dm5xdVVqZC80d3dFNjNF?=
 =?utf-8?B?QVB1TXZSbExEbnZOeWxWb0IzQVBtVkNwZzUzTFlnbnZTZzNRSzVMR0EwajU5?=
 =?utf-8?B?RkJISy9aQmRmU2FMUFFKOWlCVjc1OWx4K3hadWpnaEE5eWJpQ0FPN1ExTDhy?=
 =?utf-8?B?MWZIcUNQNUhVOVMzTkYxbG91b01DVGtNNzh4YnNnY3QwOC83Z0RxcGkrZEM1?=
 =?utf-8?B?MnZKa3dobHZBS2lQMzNBK2w2eEtWOTVFQ1lnOU5uMVFGdlZNZmdBSW1PRkkz?=
 =?utf-8?B?VmU0UElSRVpZaU54MUt1YnYvbDdzaFBQQ0FLQVlXQloyMXpTYlE5TDE5cVBM?=
 =?utf-8?B?WTdQdjdBT0JPZGxFdlh1c0tRSlp3RFZ2U0dEK1ZrTkQ0MnpRQ3UxUUR1a3B3?=
 =?utf-8?B?VjJITjZCaG5iQko2YXFsYitNcWNDWmxVeFU3T2x0cldDZ3g3OWcrdHZ0cWxn?=
 =?utf-8?B?c094czdsbzdCNVVNaHJHRkNPM2kzVElma2tucTB6SFRoRDdBeE5FNXdlNmpn?=
 =?utf-8?B?UlZlcG5NbExkbjkraSt6WFYwTEFOWmIza0dtNmVDVnRZeENBR2UxWkxwV0hZ?=
 =?utf-8?B?dkovdCtrOE5lYnNPK2huZEx3bkFQWFFvWThuY0JaS3cvK1hHc2VGN0paN2ND?=
 =?utf-8?B?VEVDMEdzQ3NGbXBXZFM0NTZWTWpnOU9zY2NyZC9QS2IwWm1UdXczWitlaDBu?=
 =?utf-8?B?elBtWnh4NWxUcEl5cDBQeU5wV0N1anhWTko0Zk0yMGJtU0FFbjFBd0I1czFS?=
 =?utf-8?B?L2xXUEo3Rjlhd29rU3RFYnIwbmx5OWVLQWlRNFpCUXd2UkpHL1g3MzROMVFk?=
 =?utf-8?B?cjN2ZE9HcmZwaWF0L3V2R3lmVGFzcVZKSW13NmxtcTR0SXBoejRFWUtoZS8y?=
 =?utf-8?B?QTY2UlRDUUQzMXo0QURMY1U3SzlSVkkyWmJIc01zWDA2bTZwUmpzaHFSdFNj?=
 =?utf-8?B?VFY3Wmt4UlB0azdDMytPOWJoUjYvck5QWjMvQjB4OVlhWVBxYjZmRUZJYk1t?=
 =?utf-8?B?RWg5b3NlSHJtYWRJNnZjTVYydnhEdEYwVGtRVS8yWWxGZEk4cXA5TUM1YTly?=
 =?utf-8?B?bzM2Rnk5NXQ2dlZiLzV6a3VNU01hMnkzbTIyb1o3MDhRSUJqR1B4M20rbkxD?=
 =?utf-8?B?cTN1MjlYWTZhWkhWS0dJaWQvRUViM2pITFlmNEx1TUVPN3RkSGRIY2R2dFg2?=
 =?utf-8?B?R3pVMHJJcE1yd2ZFak44YmJwYzB0MGJoODh0Zk9obzdBcFF2ZjVpZW5IY0Uy?=
 =?utf-8?B?RGpQTithZWNWMGNiQWZsMW1BQ0dTSEN4MXFMSHYxWSt3aEwxelJHUDEva25B?=
 =?utf-8?Q?A92Nf+0RHHBJynAeVO0yCMKAJ?=
X-MS-Exchange-CrossTenant-Network-Message-Id: ba0c4f77-106e-49ca-9567-08dc37ec773b
X-MS-Exchange-CrossTenant-AuthSource: BL1PR11MB5978.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Feb 2024 23:33:13.3495
 (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: ibvxqTcwZsik6yA+usxEEk4bKLIwRMM9sQLJvvO6ybQ9C9UX3MHs5U9+bg0iZQnfsV7AX1XxfghUqnrqybyhFw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA2PR11MB5084
X-OriginatorOrg: intel.com


On 28/02/2024 10:24 am, Kirill A. Shutemov wrote:
> TDX is going to have more than one reason to fail
> enc_status_change_prepare().
> 
> Change the callback to return errno instead of assuming -EIO;
> enc_status_change_finish() changed too to keep the interface symmetric.
> 
> Signed-off-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
> Reviewed-by: Dave Hansen <dave.hansen@intel.com>
> ---

Acked-by: Kai Huang <kai.huang@intel.com>