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 X-Spam-Level: X-Spam-Status: No, score=-14.4 required=3.0 tests=BAYES_00,DKIMWL_WL_MED, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_IN_DEF_DKIM_WL autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D700FC433E1 for ; Thu, 27 Aug 2020 14:36:33 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 80B342177B for ; Thu, 27 Aug 2020 14:36:33 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="g5T69mNO" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 80B342177B Authentication-Results: mail.kernel.org; dmarc=fail (p=reject dis=none) header.from=google.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 03E558E0009; Thu, 27 Aug 2020 10:36:33 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id F31448E0006; Thu, 27 Aug 2020 10:36:32 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id DF9118E0009; Thu, 27 Aug 2020 10:36:32 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0245.hostedemail.com [216.40.44.245]) by kanga.kvack.org (Postfix) with ESMTP id C68F98E0006 for ; Thu, 27 Aug 2020 10:36:32 -0400 (EDT) Received: from smtpin02.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id 8251C8245571 for ; Thu, 27 Aug 2020 14:36:32 +0000 (UTC) X-FDA: 77196599424.02.lace05_05030cf2706d Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin02.hostedemail.com (Postfix) with ESMTP id 3B21610097AA3 for ; Thu, 27 Aug 2020 14:36:32 +0000 (UTC) X-HE-Tag: lace05_05030cf2706d X-Filterd-Recvd-Size: 6109 Received: from mail-pf1-f194.google.com (mail-pf1-f194.google.com [209.85.210.194]) by imf36.hostedemail.com (Postfix) with ESMTP for ; Thu, 27 Aug 2020 14:36:31 +0000 (UTC) Received: by mail-pf1-f194.google.com with SMTP id k1so3661248pfu.2 for ; Thu, 27 Aug 2020 07:36:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=ESqQJ3ododn8j9sB4yCnSFUyTmI5ABpQzOrhit8Rb1k=; b=g5T69mNObCL2Grw6KpM8c7kJaXhDw6FnnWEnoUt2a2mXj7lmMnmDXfB0QgIi1Y2TxE ef2UhYnFSwkLTsWcaKGppKGv8SScFSWrN3gGiqB8FvSyYai9icB647Fjlocg9ADLoBpL 2R7jUYSXz5TotwCsvpSyZJsDOzb+me7pVDlGO/FZrFEVMOqzLFqGav9vW4IGmYNBMZZ7 tQKQeiMngbDbp6F8Ui4ywO5g48dojUg1e76DBPdtiyi6MgDh8BwLDMgKMpBTUS17hh0o jjl1XNzTRtP5LINd5Qrp3BIEMMdl3xKkofP7NDNFT1ZH9Zg45xxaSTSqoy/XivcEp1n7 AsWg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=ESqQJ3ododn8j9sB4yCnSFUyTmI5ABpQzOrhit8Rb1k=; b=X+FZmj9TO1sMbynsaOeD8lO1VnVes4PD56K7oSqv11Gt+RhC/tV4IREthLG/zNSEeI vBsqYRNFsLqz0k7THkbVxX60QC4yYfGXJ+mnXU9M0MUSRjLxz2sDTZJ4qZGtilZwE4UX ueit7UcoHiPZXt5dI1bMbOf5bbgpgDRuMMK6n4clDRnvSAqRAMuMhnGM9vOJ7tyNOKW3 nqp3bGPwyS/ruNENcpkw5kFirgHNKCp8Qx7iRLV/ow1aMHk/Zi+Bco62UEtJziGwVbPm A1Fz/cAdtutOZYalXs2hVYuRipGMZ16pzLIkKrKMPeez867v/MXOc6g5UObX5+pbBrsc Bilw== X-Gm-Message-State: AOAM533Ivp5Z1/SvQNDuRWfLnmWsBoH2ZMuKmFXwr0sY+ZvoF/cV8rCW TJ3JWq9cI3Su2Yp3ZGUoGPwFY6fFAmCJRc2xaAuoZg== X-Google-Smtp-Source: ABdhPJyOXXy6+yLSZX6CJbiVi7n4/xo7pJcVNWBmzm5+yLYznjEK7CI3vCIpoZLRNoCeI9h3o87WVSkoOug5TbvYApw= X-Received: by 2002:a62:2bcc:: with SMTP id r195mr8138438pfr.123.1598538990077; Thu, 27 Aug 2020 07:36:30 -0700 (PDT) MIME-Version: 1.0 References: <518da1e5169a4e343caa3c37feed5ad551b77a34.1597425745.git.andreyknvl@google.com> <20200827104033.GF29264@gaia> <9c53dfaa-119e-b12e-1a91-1f67f4aef503@arm.com> <20200827111344.GK29264@gaia> <30b90e66-2ac0-82b3-b590-5a2b35fad446@arm.com> In-Reply-To: <30b90e66-2ac0-82b3-b590-5a2b35fad446@arm.com> From: Andrey Konovalov Date: Thu, 27 Aug 2020 16:36:19 +0200 Message-ID: Subject: Re: [PATCH 26/35] kasan, arm64: Enable TBI EL1 To: Vincenzo Frascino Cc: Catalin Marinas , Dmitry Vyukov , kasan-dev , Andrey Ryabinin , Alexander Potapenko , Marco Elver , Evgenii Stepanov , Elena Petrova , Branislav Rankov , Kevin Brodsky , Will Deacon , Andrew Morton , Linux ARM , Linux Memory Management List , LKML Content-Type: text/plain; charset="UTF-8" X-Rspamd-Queue-Id: 3B21610097AA3 X-Spamd-Result: default: False [0.00 / 100.00] X-Rspamd-Server: rspam01 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 Thu, Aug 27, 2020 at 3:42 PM Vincenzo Frascino wrote: > > Hi Andrey, > > On 8/27/20 1:43 PM, Andrey Konovalov wrote: > > On Thu, Aug 27, 2020 at 1:15 PM Vincenzo Frascino > > wrote: > >> > >> > >> > >> On 8/27/20 12:13 PM, Catalin Marinas wrote: > >>> On Thu, Aug 27, 2020 at 12:05:55PM +0100, Vincenzo Frascino wrote: > >>>> On 8/27/20 11:40 AM, Catalin Marinas wrote: > >>>>> On Fri, Aug 14, 2020 at 07:27:08PM +0200, Andrey Konovalov wrote: > >>>>>> diff --git a/arch/arm64/mm/proc.S b/arch/arm64/mm/proc.S > >>>>>> index 152d74f2cc9c..6880ddaa5144 100644 > >>>>>> --- a/arch/arm64/mm/proc.S > >>>>>> +++ b/arch/arm64/mm/proc.S > >>>>>> @@ -38,7 +38,7 @@ > >>>>>> /* PTWs cacheable, inner/outer WBWA */ > >>>>>> #define TCR_CACHE_FLAGS TCR_IRGN_WBWA | TCR_ORGN_WBWA > >>>>>> > >>>>>> -#ifdef CONFIG_KASAN_SW_TAGS > >>>>>> +#if defined(CONFIG_KASAN_SW_TAGS) || defined(CONFIG_KASAN_HW_TAGS) > >>>>>> #define TCR_KASAN_FLAGS TCR_TBI1 > >>>>>> #else > >>>>>> #define TCR_KASAN_FLAGS 0 > >>>>> > >>>>> I prefer to turn TBI1 on only if MTE is present. So on top of the v8 > >>>>> user series, just do this in __cpu_setup. > >>>> > >>>> Not sure I understand... Enabling TBI1 only if MTE is present would break > >>>> KASAN_SW_TAGS which is based on TBI1 but not on MTE. > >>> > >>> You keep the KASAN_SW_TAGS as above but for HW_TAGS, only set TBI1 later > >>> in __cpu_setup(). > >>> > >> > >> Ok, sounds good. > > > > Sounds good to me too. > > > > Vincenzo, could you take care of Catalin's comments on your (arm64) > > patches, do the rebase onto user mte v8, and share it with me? I'll > > work on KASAN changes in the meantime, and then integrate everything > > together for v2. > > > > I am happy to do that. I will be on holiday though from this Saturday till the > September, 9. After that I will start the rebasing. Ah, OK. I'll see if I can do the rebase and fix some of Catalin's comments myself then. I'll let you know the current status once you're back. > > > Perhaps the best way to test only the arm64 part is writing a simple > > module that causes an MTE fault. (At least that's what I did when I > > was testing core in-kernel MTE patches separately.) Or reuse this > > series, all KASAN patches should rebase cleanly on top of the latest > > mainline. > > > > I can reuse the patches as they are, unless they require changes when I start > rebasing. In such a case to not duplicate the work I will scale back to use a > simple module. > > -- > Regards, > Vincenzo