Received-SPF: pass (google.com: domain of linux-kernel+bounces-43645-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <76af5d2e-7598-42b4-9ed9-8fec25ece057@intel.com>
Date: Mon, 29 Jan 2024 15:56:29 -0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH bpf v2 1/3] x86/mm: Move is_vsyscall_vaddr() into
 asm/vsyscall.h
Content-Language: en-US
To: Hou Tao <houtao@huaweicloud.com>, <x86@kernel.org>, <bpf@vger.kernel.org>
CC: Dave Hansen <dave.hansen@linux.intel.com>, Andy Lutomirski
	<luto@kernel.org>, Peter Zijlstra <peterz@infradead.org>, Thomas Gleixner
	<tglx@linutronix.de>, Ingo Molnar <mingo@redhat.com>, Borislav Petkov
	<bp@alien8.de>, "H . Peter Anvin" <hpa@zytor.com>,
	<linux-kernel@vger.kernel.org>, xingwei lee <xrivendell7@gmail.com>, "Jann
 Horn" <jannh@google.com>, Yonghong Song <yonghong.song@linux.dev>,
	<houtao1@huawei.com>
References: <20240126115423.3943360-1-houtao@huaweicloud.com>
 <20240126115423.3943360-2-houtao@huaweicloud.com>
From: Sohil Mehta <sohil.mehta@intel.com>
In-Reply-To: <20240126115423.3943360-2-houtao@huaweicloud.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?UEhZL3JFUy9QT1owK2VQQ3hiU2FjWjQxMFZhcC9UMHRSYXpOU0Z0dlRkc3Iy?=
 =?utf-8?B?SkxBalVMemJTVGo0YmtmWThmNFd2V3NteFI0OTFIQ2FjQk1BOWloN2lMcDgw?=
 =?utf-8?B?ZHVVdnNXMkh6bU9OQmxyOVN1TU8vcGwyNjVBRy9ITWF6QjBIVExWeGtac0tN?=
 =?utf-8?B?dzZPVGo4MGh5dXVxOThNMUw0QlNndTlOMzdXc3dvQ05ZV0F2S2pxQjA1WCtD?=
 =?utf-8?B?T29qQmtUSHZpUGRLdkVVaEJ3R0Z1ZnEzR29aRGtlMlBabnV3N1dSMUxjQ0ZM?=
 =?utf-8?B?RnIzVFpzcjArU1hBTnRJZWdqN2R5bll5TnBpVUdGVk85V2NTb0NOYjd1SytB?=
 =?utf-8?B?SHV0b2xLQ2l6cVV3Y1owOXlzcGFTUHM3amIwTHRMNU4zVXJvQ0NZbVBVWFVL?=
 =?utf-8?B?STE1MkRjb3FTeisrZjRJRUw1Y3ZoNkpWWnUzUG1BTG9PV3diOE5GdDJiaFZI?=
 =?utf-8?B?eSt5QzBsWVFHTEVHT1V5WWFFdnJBeTlKeXJ5dEduYzFaVTcvWnE4OTJ3enZY?=
 =?utf-8?B?TUVELzVGVzhMQ1NCUDlUV1VoNUNzNUdYSE5CVkRGMUpmSldYQTlJaU5KMkpK?=
 =?utf-8?B?Q0R2NTY4TkNBWEd3STFtOG45UzBsTkhjZThEU0ZrdnRGNlllK0VaVVp0TSt5?=
 =?utf-8?B?Tjd6WWhFY2ZUUVgxdGZrK1FwVzNqNFJYMU1JcXNGQ3hnbDR0bEN2YVhUampW?=
 =?utf-8?B?UnRuNDgwZUE2aDBxWkF4MEtULzJWWDlOSkNOem9GOGNZcjRyR3R4bEtYMkE5?=
 =?utf-8?B?aGtjYWNzemhJN1c0RW9UaCt0NTU0aGtSZzNCcUFVYjQ0WCtiNncwZHhic0Qr?=
 =?utf-8?B?c1lUNFJuUkhhQUpScFBiUk9LOGZmczN3ajBaU0M1eWU2MEprSUJxZm93RVhU?=
 =?utf-8?B?M2pDaHZoeTZBdHZIdE91MGdIeFo1OCtrNW1nR3gvUTMxN2E1UWxRTVJtU3pR?=
 =?utf-8?B?M0MybFdvUDBZejU3TzRFRytmM2RoMW5pTU1lWmpJZzZocGZZazNmNGNGUUhY?=
 =?utf-8?B?cmdmU3pxQm1xeU04TXpPN3lCOThhdHJwbm8vOGwrVGJvOUxGZGRtOEtWUkpP?=
 =?utf-8?B?MHI4ZFR1RE5CbE5FZTJzd0dhQnRpVXh1VzRMMFUzbkUwNUQvbmZraHl0SkFs?=
 =?utf-8?B?UCtlelBNY1FxNEM4VnUvQk1mMkE2MXRIZmt6MnJScUxpSExPRGdVSzJQVnZY?=
 =?utf-8?B?Z3Vub1B6QVRsQnV0ampkQTV1Wk9HVGptRDV1KzdxVk1vZkFBME9qQmIzM1Uw?=
 =?utf-8?B?T0tWVUxnbXlWNzNBQTJ1QTRpMTAxckFuRW9TenBhNUdKelU4TFdhK2liYlpT?=
 =?utf-8?B?QW1IRTBneThvOEJ4WDh5YzJPL3YrWmdUaHRIMXJpSk5VcTZGc0o5MHJFSzZS?=
 =?utf-8?B?Y0FQRUZqV0ZQbGdGdTBCVW9mRllWR1cxSnI3T3J2NlB2S3hqRU9pZlNPN2ha?=
 =?utf-8?B?QkVvUUpEY053enpzNTF6VEp5YTd3VTkwU0YvTEdWbHM3d2tsOTdob1BFT1o1?=
 =?utf-8?B?SWFONnZJRitCTzZSOUVGOEpjREx6RjNubU1pc0I5RXEyNkZpUGRjRmRMSkVZ?=
 =?utf-8?B?K1huekJtV2J2SkMzRVJJZGlOZkczSytXVFRoWWZ6czRXOVYzRUs4eGQ4Smly?=
 =?utf-8?B?bzJPR2pOMUZnRThlUWlLTEd5TmhoUjRjTE9sbFN3NVVKR1ZLRGMydTVpNlo1?=
 =?utf-8?B?cTVNZ0pOUlJZVFgyNWRVNUtsd1ZDUUlDSFVISk9hVVNKZGlrK0hUMW5lZWt2?=
 =?utf-8?B?RTlCeHBqclpDYmpldXBNcU41ZjAra2lHL0dZSGI2ZUh5bVVWNUxvYTNCQzF3?=
 =?utf-8?B?YTE2bVAzYVU2SFdZSUNuZkgydDFIZzlUaEN3RVFTaFoxbEIrMDRPcVFLK2Nw?=
 =?utf-8?B?ZHNoQ2tibUtaK3o4T1I3eDZIYjYrNHVFNHI1ZTRRWFF2aFp0YVR0OEI3bGZp?=
 =?utf-8?B?WVQ3ZnZMYmxqeE53anlHSGJCTDVMR2h6dXZITFcyL1R0d0NTTDRrbmtQYStF?=
 =?utf-8?B?RFNvWkpHbGxBdWI1NVpJQU4rNS9UZjk5emRRUXFvcUd0WVFyNWNxRFN4Rmhs?=
 =?utf-8?B?WS9ncVlUSE9rdDkzR1ZiRWJDT29UZDlneVNzMzdBd1hORFN0WTc4ZkVYVWY0?=
 =?utf-8?Q?1tdx7ij6/UplBdH9bqsarrfHb?=
X-MS-Exchange-CrossTenant-Network-Message-Id: c14897dd-c7eb-4d86-5795-08dc2125ea58
X-MS-Exchange-CrossTenant-AuthSource: BYAPR11MB3320.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Jan 2024 23:56:31.1387
 (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: 5k9fa7lxy4ffF9j5wornGOrFbV+cRDTJonLojiOdi8rIoro98QWjZvhpolovJWSsGvtLhbxLJqyRaMIrnmUbMg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR11MB5199
X-OriginatorOrg: intel.com

On 1/26/2024 3:54 AM, Hou Tao wrote:
> From: Hou Tao <houtao1@huawei.com>
> 
> Moving is_vsyscall_vaddr() into asm/vsyscall.h to make it available for

s/Moving/Move

> copy_from_kernel_nofault_allowed() in arch/x86/mm/maccess.c.
> 
> Signed-off-by: Hou Tao <houtao1@huawei.com>
> ---
>  arch/x86/include/asm/vsyscall.h | 10 ++++++++++
>  arch/x86/mm/fault.c             |  9 ---------
>  2 files changed, 10 insertions(+), 9 deletions(-)
> 


Reviewed-by: Sohil Mehta <sohil.mehta@intel.com>