From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 72931C001B0 for ; Sun, 23 Jul 2023 07:34:30 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id BAC1A6B0072; Sun, 23 Jul 2023 03:34:29 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id B364F6B0074; Sun, 23 Jul 2023 03:34:29 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 9AE8C6B0075; Sun, 23 Jul 2023 03:34:29 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id 7F9C16B0072 for ; Sun, 23 Jul 2023 03:34:29 -0400 (EDT) Received: from smtpin28.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id 36DE5C08CF for ; Sun, 23 Jul 2023 07:34:29 +0000 (UTC) X-FDA: 81042063858.28.90E94E3 Received: from mailgw.kylinos.cn (mailgw.kylinos.cn [124.126.103.232]) by imf24.hostedemail.com (Postfix) with ESMTP id 42DF3180004 for ; Sun, 23 Jul 2023 07:34:25 +0000 (UTC) Authentication-Results: imf24.hostedemail.com; dkim=none; dmarc=none; spf=pass (imf24.hostedemail.com: domain of lienze@kylinos.cn designates 124.126.103.232 as permitted sender) smtp.mailfrom=lienze@kylinos.cn ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1690097667; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=NTrjRsqDU0JPO/TksSAZQ2975dh9ntf+HqOxhoFiv8c=; b=lKyJmI4rMnqSQGMehHcTfK8IcTaLvvXEwJzAtZIw9XOe+N4FwyfbmMjzn8cyxD2P/cwi7E ijJn+jAkJj09sVm9imJ9EuGDGtGp6Yj2K/gMbM5HraGI3HzgYg3deiX1Os4jiXNAfulhI0 WMTX5xPKutZh9oGtcT3vRE33cQM6Nd0= ARC-Authentication-Results: i=1; imf24.hostedemail.com; dkim=none; dmarc=none; spf=pass (imf24.hostedemail.com: domain of lienze@kylinos.cn designates 124.126.103.232 as permitted sender) smtp.mailfrom=lienze@kylinos.cn ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1690097667; a=rsa-sha256; cv=none; b=bvA6U5EHyrK6hFBtEpLJWmpBFTVUATmmmw4OO+wC+eJRu8fG9mOHklYyOghv45fIh4O6Q+ 6qeBUOm73BGcagBc/4moSVLFDQCfePgO8X7VYb8PwCEwBcEuGo+PDfCOAC2sJRVCdL4Uw0 IgludKijtoKB/q/lqnZhf0aCDNaA0ZA= X-UUID: 03ea0833cfc34e699fa3ba5a09395578-20230723 X-CID-P-RULE: Release_Ham X-CID-O-INFO: VERSION:1.1.28,REQID:d0d69a12-911b-4bc0-b4f9-f8d6f42fbb37,IP:15, URL:0,TC:0,Content:0,EDM:0,RT:0,SF:-15,FILE:0,BULK:0,RULE:Release_Ham,ACTI ON:release,TS:0 X-CID-INFO: VERSION:1.1.28,REQID:d0d69a12-911b-4bc0-b4f9-f8d6f42fbb37,IP:15,UR L:0,TC:0,Content:0,EDM:0,RT:0,SF:-15,FILE:0,BULK:0,RULE:Release_Ham,ACTION :release,TS:0 X-CID-META: VersionHash:176cd25,CLOUDID:c15ef98e-7caa-48c2-8dbb-206f0389473c,B ulkID:230721111347J28IJWE1,BulkQuantity:2,Recheck:0,SF:24|17|19|44|102,TC: nil,Content:0,EDM:-3,IP:-2,URL:1,File:nil,Bulk:40,QS:nil,BEC:nil,COL:0,OSI :0,OSA:0,AV:0,LES:1,SPR:NO,DKR:0,DKP:0 X-CID-BVR: 0,NGT X-CID-BAS: 0,NGT,0,_ X-CID-FACTOR: TF_CID_SPAM_FAS,TF_CID_SPAM_FSD,TF_CID_SPAM_FSI,TF_CID_SPAM_ULS, TF_CID_SPAM_SNR X-UUID: 03ea0833cfc34e699fa3ba5a09395578-20230723 X-User: lienze@kylinos.cn Received: from ubuntu [(39.156.73.12)] by mailgw (envelope-from ) (Generic MTA with TLSv1.2 ECDHE-RSA-AES256-GCM-SHA384 256/256) with ESMTP id 1903500873; Sun, 23 Jul 2023 15:34:15 +0800 From: Enze Li To: Huacai Chen Cc: kernel@xen0n.name, loongarch@lists.linux.dev, glider@google.com, elver@google.com, akpm@linux-foundation.org, kasan-dev@googlegroups.com, linux-mm@kvack.org, zhangqing@loongson.cn, yangtiezhu@loongson.cn, dvyukov@google.com Subject: Re: [PATCH 4/4] LoongArch: Add KFENCE support In-Reply-To: (Huacai Chen's message of "Fri, 21 Jul 2023 11:19:10 +0800") References: <20230719082732.2189747-1-lienze@kylinos.cn> <20230719082732.2189747-5-lienze@kylinos.cn> <87lefaez31.fsf@kylinos.cn> Date: Sun, 23 Jul 2023 15:34:08 +0800 Message-ID: <87h6pvaxov.fsf@kylinos.cn> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Rspam-User: X-Stat-Signature: n1tpjmksruqqyyms318gignpobmr96xc X-Rspamd-Server: rspam07 X-Rspamd-Queue-Id: 42DF3180004 X-HE-Tag: 1690097665-107989 X-HE-Meta: U2FsdGVkX1+nCWKE8C/bISoMOhqvcuFUVjo1OY73EFXfZ2+4ALXM6+5bNewza8qdxg/ujo+wFh6KGm/ZFMUL5e6EV04zx4yeeCk1iRGgZk9L7+zJE8Ggj9j+WOwHOlwpjnCMeNN+/F/Dm2ff9hIX9Gn87et9utDyLQmWB3YinZEEUFGHS+hT7Kj24UImDR7OUAdtje9Hw4EXzH+0RV2mGlJy4x1mI1hecxGkH8g2alA1591fXAZxZE2sviE5Wr816tBAenIKLHxhmomNDDuPVAigp9cxhGdGbby8MSQfAT4+fD75sCSvB5/kXNIozIlnV2WyDtPDlxo/DqWpuYXjRGrww2k9oF7lYvCHtArFNMTkHPImkmsUOYExsovVdwLPjCIA3KM1hZjsrd1OckXXN0ul+7d6ZgVtB8caN23qcHKsen/8NQZ+jvsDYscizdu1tDRSdQToPWholq6bL5n2rgpGwRW3XfkvjM2YQkkHcVGl05jZRvzQhhmU94irVrbkAxnzUw6eFCnsj0u0kttyr1S7MVUYa+x3ebL9ZN3SDhq84zObEsPVLE4RdOp+MTGKUTR0VFQvqoQBHDGMVja9i9+Fw6uAt/cntbOLyPF/jLipByCw2c23lX42fXCghfw99TjetFECpR8qhnkCTtdlj7QcqA5BVxMtXmM/M0BdMWH4cwr2kUvlW76M4r5NKQpKgaRuVpmB7lyyTmNuyHNWwxakZlFc+w5E+3oV3pkPHQlNqHsrgZ4z1UEDtMFQ+k7q0dmnSKuXsR2jzWfpfUCeYhVutsDlYYSoaMklA2qdGrbBw9GFb0aWsx1C4oimuYAwMyXOAqm4x2DxcmkrDQEE0yuBJTcghBNDNnzxWhnoiZZYMIE3hYgMDRy9vWXSbmY4n3GKdnkJGFJeYEVVmQ4YxzW9GOJ5MgMyL5G7hfsyV2c7ehyBxYHpqit03BIJc5mzF6Enq95HAWs5rTd5Ih6 xVChR+IZ i+Umtw8ys+70EXnsMV8E57n7T9+LQ9qyTMxweUc2/OA42zevvvRJmoz9OFq25L4gAUJx8Ozev5VtKEFxapHBdBgPzHsRvw1Ws3pvh4j5iz6nOIPVwWxh2++K9Ec+kBClCpkMQ X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Fri, Jul 21 2023 at 11:19:10 AM +0800, Huacai Chen wrote: > Hi, Enze, > > On Fri, Jul 21, 2023 at 11:14=E2=80=AFAM Enze Li wrot= e: >> >> On Wed, Jul 19 2023 at 11:27:50 PM +0800, Huacai Chen wrote: >> >> > Hi, Enze, >> > >> > On Wed, Jul 19, 2023 at 4:34=E2=80=AFPM Enze Li wr= ote: >> >> >> >> The LoongArch architecture is quite different from other architecture= s. >> >> When the allocating of KFENCE itself is done, it is mapped to the dir= ect >> >> mapping configuration window [1] by default on LoongArch. It means t= hat >> >> it is not possible to use the page table mapped mode which required by >> >> the KFENCE system and therefore it should be remapped to the appropri= ate >> >> region. >> >> >> >> This patch adds architecture specific implementation details for KFEN= CE. >> >> In particular, this implements the required interface in . >> >> >> >> Tested this patch by using the testcases and all passed. >> >> >> >> [1] https://loongson.github.io/LoongArch-Documentation/LoongArch-Vol1= -EN.html#virtual-address-space-and-address-translation-mode >> >> >> >> Signed-off-by: Enze Li >> >> --- >> >> arch/loongarch/Kconfig | 1 + >> >> arch/loongarch/include/asm/kfence.h | 62 ++++++++++++++++++++++++++= ++ >> >> arch/loongarch/include/asm/pgtable.h | 6 +++ >> >> arch/loongarch/mm/fault.c | 22 ++++++---- >> >> 4 files changed, 83 insertions(+), 8 deletions(-) >> >> create mode 100644 arch/loongarch/include/asm/kfence.h >> >> >> >> diff --git a/arch/loongarch/Kconfig b/arch/loongarch/Kconfig >> >> index 5411e3a4eb88..db27729003d3 100644 >> >> --- a/arch/loongarch/Kconfig >> >> +++ b/arch/loongarch/Kconfig >> >> @@ -93,6 +93,7 @@ config LOONGARCH >> >> select HAVE_ARCH_JUMP_LABEL >> >> select HAVE_ARCH_JUMP_LABEL_RELATIVE >> >> select HAVE_ARCH_KASAN >> >> + select HAVE_ARCH_KFENCE if 64BIT >> > "if 64BIT" can be dropped here. >> > >> >> Fixed. >> >> >> select HAVE_ARCH_MMAP_RND_BITS if MMU >> >> select HAVE_ARCH_SECCOMP_FILTER >> >> select HAVE_ARCH_TRACEHOOK >> >> diff --git a/arch/loongarch/include/asm/kfence.h b/arch/loongarch/inc= lude/asm/kfence.h >> >> new file mode 100644 >> >> index 000000000000..2a85acc2bc70 >> >> --- /dev/null >> >> +++ b/arch/loongarch/include/asm/kfence.h >> >> @@ -0,0 +1,62 @@ >> >> +/* SPDX-License-Identifier: GPL-2.0 */ >> >> +/* >> >> + * KFENCE support for LoongArch. >> >> + * >> >> + * Author: Enze Li >> >> + * Copyright (C) 2022-2023 KylinSoft Corporation. >> >> + */ >> >> + >> >> +#ifndef _ASM_LOONGARCH_KFENCE_H >> >> +#define _ASM_LOONGARCH_KFENCE_H >> >> + >> >> +#include >> >> +#include >> >> +#include >> >> + >> >> +static inline char *arch_kfence_init_pool(void) >> >> +{ >> >> + char *__kfence_pool_orig =3D __kfence_pool; >> > I prefer kfence_pool than __kfence_pool_orig here. >> > >> >> Fixed. >> >> >> + struct vm_struct *area; >> >> + int err; >> >> + >> >> + area =3D __get_vm_area_caller(KFENCE_POOL_SIZE, VM_IOREMAP, >> >> + KFENCE_AREA_START, KFENCE_AREA_EN= D, >> >> + __builtin_return_address(0)); >> >> + if (!area) >> >> + return NULL; >> >> + >> >> + __kfence_pool =3D (char *)area->addr; >> >> + err =3D ioremap_page_range((unsigned long)__kfence_pool, >> >> + (unsigned long)__kfence_pool + KFENC= E_POOL_SIZE, >> >> + virt_to_phys((void *)__kfence_pool_o= rig), >> >> + PAGE_KERNEL); >> >> + if (err) { >> >> + free_vm_area(area); >> >> + return NULL; >> >> + } >> >> + >> >> + return __kfence_pool; >> >> +} >> >> + >> >> +/* Protect the given page and flush TLB. */ >> >> +static inline bool kfence_protect_page(unsigned long addr, bool prot= ect) >> >> +{ >> >> + pte_t *pte =3D virt_to_kpte(addr); >> >> + >> >> + if (WARN_ON(!pte) || pte_none(*pte)) >> >> + return false; >> >> + >> >> + if (protect) >> >> + set_pte(pte, __pte(pte_val(*pte) & ~(_PAGE_VALID | _P= AGE_PRESENT))); >> >> + else >> >> + set_pte(pte, __pte(pte_val(*pte) | (_PAGE_VALID | _PA= GE_PRESENT))); >> >> + >> >> + /* Flush this CPU's TLB. */ >> >> + preempt_disable(); >> >> + local_flush_tlb_one(addr); >> >> + preempt_enable(); >> >> + >> >> + return true; >> >> +} >> >> + >> >> +#endif /* _ASM_LOONGARCH_KFENCE_H */ >> >> diff --git a/arch/loongarch/include/asm/pgtable.h b/arch/loongarch/in= clude/asm/pgtable.h >> >> index 0fc074b8bd48..5a9c81298fe3 100644 >> >> --- a/arch/loongarch/include/asm/pgtable.h >> >> +++ b/arch/loongarch/include/asm/pgtable.h >> >> @@ -85,7 +85,13 @@ extern unsigned long zero_page_mask; >> >> #define MODULES_VADDR (vm_map_base + PCI_IOSIZE + (2 * PAGE_SIZE)) >> >> #define MODULES_END (MODULES_VADDR + SZ_256M) >> >> >> >> +#ifdef CONFIG_KFENCE >> >> +#define KFENCE_AREA_START MODULES_END >> >> +#define KFENCE_AREA_END (KFENCE_AREA_START + SZ_512M) >> > Why you choose 512M here? >> > >> >> One day I noticed that 512M can hold 16K (default 255) KFENCE objects, >> which should be more than enough and I think this should be appropriate. >> >> As far as I see, KFENCE system does not have the upper limit of this >> value(CONFIG_KFENCE_NUM_OBJECTS), which could theoretically be any >> number. There's another way, how about setting this value to be >> determined by the configuration, like this, >> >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D >> +#define KFENCE_AREA_END \ >> + (KFENCE_AREA_START + (CONFIG_KFENCE_NUM_OBJECTS + 1) * 2 * PAGE_SIZE) >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > How does other archs configure the size? > They all use the same one with a macro named KFENCE_POOL_SIZE defined like this during kernel startup, #define KFENCE_POOL_SIZE ((CONFIG_KFENCE_NUM_OBJECTS + 1) * 2 * PAGE_SIZE) For now, we do not need to consider the KASAN region, and get enough address space after vmemmap, this will not be a problem. >> >> >> +#define VMALLOC_START KFENCE_AREA_END >> >> +#else >> >> #define VMALLOC_START MODULES_END >> >> +#endif >> > I don't like to put KFENCE_AREA between module and vmalloc range (it >> > may cause some problems), can we put it after vmemmap? >> >> I found that there is not enough space after vmemmap and that these >> spaces are affected by KASAN. As follows, >> >> Without KASAN >> ###### module 0xffff800002008000~0xffff800012008000 >> ###### malloc 0xffff800032008000~0xfffffefffe000000 >> ###### vmemmap 0xffffff0000000000~0xffffffffffffffff >> >> With KASAN >> ###### module 0xffff800002008000~0xffff800012008000 >> ###### malloc 0xffff800032008000~0xffffbefffe000000 >> ###### vmemmap 0xffffbf0000000000~0xffffbfffffffffff >> >> What about put it before MODULES_START? > I temporarily drop KASAN in linux-next for you. You can update a new > patch version without KASAN (still, put KFENCE after vmemmap), and > then we can improve further. > > Huacai Thank you so much. :) The v2 of the patchset is on the way. Best Regards, Enze >> >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D >> --- a/arch/loongarch/include/asm/pgtable.h >> +++ b/arch/loongarch/include/asm/pgtable.h >> @@ -82,7 +82,14 @@ extern unsigned long zero_page_mask; >> * Avoid the first couple of pages so NULL pointer dereferences will >> * still reliably trap. >> */ >> +#ifdef CONFIG_KFENCE >> +#define KFENCE_AREA_START (vm_map_base + PCI_IOSIZE + (2 * PAGE_SI= ZE)) >> +#define KFENCE_AREA_END \ >> + (KFENCE_AREA_START + (CONFIG_KFENCE_NUM_OBJECTS + 1) * 2 * PAGE_= SIZE) >> +#define MODULES_VADDR KFENCE_AREA_END >> +#else >> #define MODULES_VADDR (vm_map_base + PCI_IOSIZE + (2 * PAGE_SIZE)) >> +#endif >> #define MODULES_END (MODULES_VADDR + SZ_256M) >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D >> >> Best Regards, >> Enze >> >> >> >> >> #ifndef CONFIG_KASAN >> >> #define VMALLOC_END \ >> >> diff --git a/arch/loongarch/mm/fault.c b/arch/loongarch/mm/fault.c >> >> index da5b6d518cdb..c0319128b221 100644 >> >> --- a/arch/loongarch/mm/fault.c >> >> +++ b/arch/loongarch/mm/fault.c >> >> @@ -23,6 +23,7 @@ >> >> #include >> >> #include >> >> #include >> >> +#include >> >> >> >> #include >> >> #include >> >> @@ -30,7 +31,8 @@ >> >> >> >> int show_unhandled_signals =3D 1; >> >> >> >> -static void __kprobes no_context(struct pt_regs *regs, unsigned long= address) >> >> +static void __kprobes no_context(struct pt_regs *regs, unsigned long= address, >> >> + unsigned long write) >> >> { >> >> const int field =3D sizeof(unsigned long) * 2; >> >> >> >> @@ -38,6 +40,9 @@ static void __kprobes no_context(struct pt_regs *re= gs, unsigned long address) >> >> if (fixup_exception(regs)) >> >> return; >> >> >> >> + if (kfence_handle_page_fault(address, write, regs)) >> >> + return; >> >> + >> >> /* >> >> * Oops. The kernel tried to access some bad page. We'll have= to >> >> * terminate things with extreme prejudice. >> >> @@ -51,14 +56,15 @@ static void __kprobes no_context(struct pt_regs *= regs, unsigned long address) >> >> die("Oops", regs); >> >> } >> >> >> >> -static void __kprobes do_out_of_memory(struct pt_regs *regs, unsigne= d long address) >> >> +static void __kprobes do_out_of_memory(struct pt_regs *regs, unsigne= d long address, >> >> + unsigned long write) >> >> { >> >> /* >> >> * We ran out of memory, call the OOM killer, and return the = userspace >> >> * (which will retry the fault, or kill us if we got oom-kill= ed). >> >> */ >> >> if (!user_mode(regs)) { >> >> - no_context(regs, address); >> >> + no_context(regs, address, write); >> >> return; >> >> } >> >> pagefault_out_of_memory(); >> >> @@ -69,7 +75,7 @@ static void __kprobes do_sigbus(struct pt_regs *reg= s, >> >> { >> >> /* Kernel mode? Handle exceptions or die */ >> >> if (!user_mode(regs)) { >> >> - no_context(regs, address); >> >> + no_context(regs, address, write); >> >> return; >> >> } >> >> >> >> @@ -90,7 +96,7 @@ static void __kprobes do_sigsegv(struct pt_regs *re= gs, >> >> >> >> /* Kernel mode? Handle exceptions or die */ >> >> if (!user_mode(regs)) { >> >> - no_context(regs, address); >> >> + no_context(regs, address, write); >> >> return; >> >> } >> >> >> >> @@ -149,7 +155,7 @@ static void __kprobes __do_page_fault(struct pt_r= egs *regs, >> >> */ >> >> if (address & __UA_LIMIT) { >> >> if (!user_mode(regs)) >> >> - no_context(regs, address); >> >> + no_context(regs, address, write); >> >> else >> >> do_sigsegv(regs, write, address, si_code); >> >> return; >> >> @@ -211,7 +217,7 @@ static void __kprobes __do_page_fault(struct pt_r= egs *regs, >> >> >> >> if (fault_signal_pending(fault, regs)) { >> >> if (!user_mode(regs)) >> >> - no_context(regs, address); >> >> + no_context(regs, address, write); >> >> return; >> >> } >> >> >> >> @@ -232,7 +238,7 @@ static void __kprobes __do_page_fault(struct pt_r= egs *regs, >> >> if (unlikely(fault & VM_FAULT_ERROR)) { >> >> mmap_read_unlock(mm); >> >> if (fault & VM_FAULT_OOM) { >> >> - do_out_of_memory(regs, address); >> >> + do_out_of_memory(regs, address, write); >> >> return; >> >> } else if (fault & VM_FAULT_SIGSEGV) { >> >> do_sigsegv(regs, write, address, si_code); >> >> -- >> >> 2.34.1 >> >> >> >> >>