From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-oi1-f199.google.com (mail-oi1-f199.google.com [209.85.167.199]) by kanga.kvack.org (Postfix) with ESMTP id C9A8A6B051B for ; Wed, 7 Nov 2018 10:35:03 -0500 (EST) Received: by mail-oi1-f199.google.com with SMTP id m129-v6so10827479oif.22 for ; Wed, 07 Nov 2018 07:35:03 -0800 (PST) Received: from foss.arm.com (foss.arm.com. [217.140.101.70]) by mx.google.com with ESMTP id j9-v6si262642oif.167.2018.11.07.07.35.02 for ; Wed, 07 Nov 2018 07:35:02 -0800 (PST) Date: Wed, 7 Nov 2018 15:34:57 +0000 From: Will Deacon Subject: Re: [PATCH v10 00/22] kasan: add software tag-based mode for arm64 Message-ID: <20181107153456.GE2623@brain-police> References: <20181107145922.GD2623@brain-police> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: owner-linux-mm@kvack.org List-ID: To: Andrey Konovalov Cc: Andrew Morton , Kostya Serebryany , Evgeniy Stepanov , Lee Smith , Ramana Radhakrishnan , Jacob Bramley , Ruben Ayrapetyan , Jann Horn , Mark Brand , Chintan Pandya , Vishwath Mohan , Andrey Ryabinin , Alexander Potapenko , Dmitry Vyukov , Catalin Marinas , Christoph Lameter , Mark Rutland , Nick Desaulniers , Marc Zyngier , Dave Martin , Ard Biesheuvel , "Eric W . Biederman" , Ingo Molnar , Paul Lawrence , Geert Uytterhoeven , Arnd Bergmann , "Kirill A . Shutemov" , Greg Kroah-Hartman , Kate Stewart , Mike Rapoport , kasan-dev@googlegroups.com, "open list:DOCUMENTATION" , LKML , Linux ARM , linux-sparse@vger.kernel.org, Linux Memory Management List , Linux Kbuild mailing list On Wed, Nov 07, 2018 at 04:11:35PM +0100, Andrey Konovalov wrote: > On Wed, Nov 7, 2018 at 3:59 PM, Will Deacon wrote: > > On Wed, Nov 07, 2018 at 03:56:03PM +0100, Andrey Konovalov wrote: > >> On Tue, Nov 6, 2018 at 6:30 PM, Andrey Konovalov wrote: > >> > This patchset adds a new software tag-based mode to KASAN [1]. > >> > (Initially this mode was called KHWASAN, but it got renamed, > >> > see the naming rationale at the end of this section). > >> > >> [...] > >> > >> > Reviewed-by: Andrey Ryabinin > >> > Reviewed-by: Dmitry Vyukov > >> > Signed-off-by: Andrey Konovalov > >> > >> Hi Andrew, > >> > >> This patchset has now been reviewed-by KASAN maintainers. Could you > >> take a look and consider taking this into the -mm tree? > > > > I would much prefer to take the arm64 parts (which still need to be reviewed > > by Catalin afaict) via the arm64 tree, so please can you split those out > > separately? > > Which parts do you mean exactly, which patches? I don't think it makes > sense to split this patchset, as one part won't function without the > other. I would like the patches that touch code under arch/arm64/ to be reviewed by somebody from the arm64 community. Since the core parts have already been reviewed, I was suggesting that you could split them out so that they are not blocked by the architecture code. Is it not possible to preserve the existing KASAN behaviour for arm64 with the core parts merged? I figured it must be, since you're not touching any other architectures here and they assumedly continue to function correctly. However, if you'd rather keep everything together, please can we give it a couple of weeks so we can at least get the architecture bits reviewed? Most people are out at LPC next week (and I'm at another conference this week). Will