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=-10.3 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,MAILING_LIST_MULTI, NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable 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 4EADEC433FE for ; Fri, 4 Dec 2020 11:55:16 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 1A1582251E for ; Fri, 4 Dec 2020 11:55:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2388102AbgLDLzP (ORCPT ); Fri, 4 Dec 2020 06:55:15 -0500 Received: from mail-wm1-f67.google.com ([209.85.128.67]:40198 "EHLO mail-wm1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728666AbgLDLzO (ORCPT ); Fri, 4 Dec 2020 06:55:14 -0500 Received: by mail-wm1-f67.google.com with SMTP id a3so6758222wmb.5 for ; Fri, 04 Dec 2020 03:54:58 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:reply-to:subject:to:cc:references:from :message-id:date:user-agent:mime-version:in-reply-to :content-language:content-transfer-encoding; bh=wHqmBeW2IFxVt4oHJizfcV3fQQX4Hgdaz39Wp4K7c0A=; b=qlp9PhdRghtfGxUhq6Yr02LvnLfDF8oruWwkFQQhz/iNOf3aHArn4YqYLIKck2FIcL /yx1KGpv+OpqPIfteu7Rjm/URHLYV/0cPj2AzWEfM+aNd7b7dRpuJ957oai1ARNGw4HO Oie+gJNx5I6hWIoJaWE7LPKSgKho7hj6VwTRMT0ryvxmLGXVKwMOkaMPTXmDr1iPfwHT sGe1hz8ul3W+bfYUzwmBnG/KB4QYM7d9ydWOEJZEMDRk87De7ACTMa0dR5bdtXhv/WT3 QOdlaA4xaVtStVrXw8huAWLzjt+CM5BZlG6GDsstFwpwCekscBYcrl957b0denBM+2WJ iuLQ== X-Gm-Message-State: AOAM5307tnpoNRasq8PBYZeZn3lIFuq1hOAe/6I+sCxDSOXkTV9Wgz92 DryxQuLGmsBPl2XIjWsNRxw= X-Google-Smtp-Source: ABdhPJz3k3/M4l7ovBWHXv3cerqd8tc5RbeKIr8k7pXjJ8RslEzOgI+FhEYt0rRwwhaYS4XH6t8M9g== X-Received: by 2002:a7b:c087:: with SMTP id r7mr3791014wmh.153.1607082872591; Fri, 04 Dec 2020 03:54:32 -0800 (PST) Received: from [10.9.0.26] ([185.248.161.177]) by smtp.gmail.com with ESMTPSA id u26sm3027316wmm.24.2020.12.04.03.54.28 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 04 Dec 2020 03:54:31 -0800 (PST) Reply-To: alex.popov@linux.com Subject: Re: [PATCH RFC v2 2/6] mm/slab: Perform init_on_free earlier To: Andrew Morton Cc: Alexander Potapenko , Kees Cook , Jann Horn , Will Deacon , Andrey Ryabinin , Dmitry Vyukov , Christoph Lameter , Pekka Enberg , David Rientjes , Joonsoo Kim , Masahiro Yamada , Masami Hiramatsu , Steven Rostedt , Peter Zijlstra , Krzysztof Kozlowski , Patrick Bellasi , David Howells , Eric Biederman , Johannes Weiner , Laura Abbott , Arnd Bergmann , Greg Kroah-Hartman , Daniel Micay , Andrey Konovalov , Matthew Wilcox , Pavel Machek , Valentin Schneider , kasan-dev , Linux Memory Management List , Kernel Hardening , LKML , notify@kernel.org References: <20200929183513.380760-1-alex.popov@linux.com> <20200929183513.380760-3-alex.popov@linux.com> <1772bc7d-e87f-0f62-52a8-e9d9ac99f5e3@linux.com> <20201203124914.25e63b013e9c69c79d481831@linux-foundation.org> From: Alexander Popov Message-ID: <9b9861c0-4c94-a51f-bbac-bd5e9b77d9e0@linux.com> Date: Fri, 4 Dec 2020 14:54:26 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.3.1 MIME-Version: 1.0 In-Reply-To: <20201203124914.25e63b013e9c69c79d481831@linux-foundation.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 03.12.2020 23:49, Andrew Morton wrote: > On Thu, 3 Dec 2020 22:50:27 +0300 Alexander Popov wrote: > >> On 30.09.2020 15:50, Alexander Potapenko wrote: >>> On Tue, Sep 29, 2020 at 8:35 PM Alexander Popov wrote: >>>> >>>> Currently in CONFIG_SLAB init_on_free happens too late, and heap >>>> objects go to the heap quarantine being dirty. Lets move memory >>>> clearing before calling kasan_slab_free() to fix that. >>>> >>>> Signed-off-by: Alexander Popov >>> Reviewed-by: Alexander Potapenko >> >> Hello! >> >> Can this particular patch be considered for the mainline kernel? > > All patches are considered ;) And merged if they're reviewed, tested, > judged useful, etc. > > If you think this particular patch should be fast-tracked then please > send it as a non-RFC, standalone patch. Please also enhance the > changelog so that it actually explains what goes wrong. Presumably > "objects go to the heap quarantine being dirty" causes some > user-visible problem? What is that problem? Ok, thanks! I'll improve the commit message and send the patch separately. Best regards, Alexander