From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on archive.lwn.net X-Spam-Level: X-Spam-Status: No, score=-6.0 required=5.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI autolearn=ham autolearn_force=no version=3.4.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by archive.lwn.net (Postfix) with ESMTP id 46D867D08A for ; Wed, 7 Nov 2018 15:35:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727564AbeKHBFx (ORCPT ); Wed, 7 Nov 2018 20:05:53 -0500 Received: from usa-sjc-mx-foss1.foss.arm.com ([217.140.101.70]:53216 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727387AbeKHBFx (ORCPT ); Wed, 7 Nov 2018 20:05:53 -0500 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 8E55880D; Wed, 7 Nov 2018 07:35:01 -0800 (PST) Received: from brain-police (usa-sjc-mx-foss1.foss.arm.com [217.140.101.70]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id D5FD63F5BD; Wed, 7 Nov 2018 07:34:59 -0800 (PST) Date: Wed, 7 Nov 2018 15:34:57 +0000 From: Will Deacon 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 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: User-Agent: Mutt/1.9.4 (2018-02-28) Sender: linux-doc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-doc@vger.kernel.org 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