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 F1B52ECAAD3 for ; Thu, 1 Sep 2022 15:40:01 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 4BFB48D000F; Thu, 1 Sep 2022 11:40:01 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 448ED8D0006; Thu, 1 Sep 2022 11:40:01 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 2C1D88D000F; Thu, 1 Sep 2022 11:40:01 -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 1768D8D0006 for ; Thu, 1 Sep 2022 11:40:01 -0400 (EDT) Received: from smtpin28.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id CDD8780EC3 for ; Thu, 1 Sep 2022 15:40:00 +0000 (UTC) X-FDA: 79863927360.28.7858C7A Received: from mail-yb1-f171.google.com (mail-yb1-f171.google.com [209.85.219.171]) by imf31.hostedemail.com (Postfix) with ESMTP id 8814A20057 for ; Thu, 1 Sep 2022 15:40:00 +0000 (UTC) Received: by mail-yb1-f171.google.com with SMTP id c9so9153143ybf.5 for ; Thu, 01 Sep 2022 08:40:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date; bh=3cE853tliEpPWpHjkdjBsm55bguibJ1NOZ2l+ytqvIs=; b=CNR1NtmysqZN43aw5siPQxQB15yoyKxn0UtAt1eFrCVSTrqVXrck0MJQ5rF0EE8Iqy FECYDZNtaf4xFqIfRFeGfncBEzgsXCtgfPFQ9NnPdVRFkAf7PTRy+isa8Q/3u9s63jFF Bkv08GJGttRtk8ihR4u9G9sfgOvnbQBOURRSewgh7NzkPEQgVbW8k9NY5ToRU0fk16sO KDQeEwKDvn2AZgblfu2tmDll7zNnUGlKeUzAPAS8k31d+CGKhmmtzNe2LpeGY2e9A3T2 OyHR4dbvshaM9SSwkyP8U36LLaknoM2MMTtTCDpp5f48zXa2ecNcsCYbrEUZ3+gtqLOE C5XA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date; bh=3cE853tliEpPWpHjkdjBsm55bguibJ1NOZ2l+ytqvIs=; b=AsC6nTt5XEfAG1vka++CoxKDgsNRhOJga7OsgCVbfW/UUSD23gy0POnkOOEDRjT15M hpJPTrPVju9vOA79zwIIFB8HCb+M0TgVUq27lqUUj8r2AWhuUhGRSfECR2NekerJ2pOs XX180IJifGbL3XGDjdrdO94JuXnmNDs0X6y07KVPyebMe/6K8K7I1SO9yW3vW6JJ/Aa3 JalLECzHn4OoIR5W5QEyKw4smiY1ZO61PGzjjCVhGl5NwIHU7jIb2UUcazQSbFxQG6P9 LCj1UD2kiIJG2zu/H1RNg5lnZEDydgt81b+/1MynIHh7K/AC3Y6bhkhuQr09Ub0yckkx rhmA== X-Gm-Message-State: ACgBeo00d8EIXLdsVSojFsr3avojtTyLqjFPIen/R9It8oKQZ38vK6X2 fFoqGCuPzWL58031lLaLHlmJi/6Ye3DrK9uwOQDyKw== X-Google-Smtp-Source: AA6agR5aFHV5oWy3bOo2eVd0PIajtlCyUMrAF/y9Oh7oqrhvwnrJM9jnSE6k3ZmfRfh8uK89oJZlft1zkj3aGLobpWw= X-Received: by 2002:a25:b983:0:b0:695:d8b4:a5a3 with SMTP id r3-20020a25b983000000b00695d8b4a5a3mr20405655ybg.553.1662046799565; Thu, 01 Sep 2022 08:39:59 -0700 (PDT) MIME-Version: 1.0 References: <20220830214919.53220-1-surenb@google.com> <20220831084230.3ti3vitrzhzsu3fs@moria.home.lan> <20220831101948.f3etturccmp5ovkl@suse.de> <20220831190154.qdlsxfamans3ya5j@moria.home.lan> <404e947a-e1b2-0fae-8b4f-6f2e3ba6328d@redhat.com> <20220901142345.agkfp2d5lijdp6pt@moria.home.lan> <78e55029-0eaf-b4b3-7e86-1086b97c60c6@redhat.com> In-Reply-To: <78e55029-0eaf-b4b3-7e86-1086b97c60c6@redhat.com> From: Suren Baghdasaryan Date: Thu, 1 Sep 2022 08:39:48 -0700 Message-ID: Subject: Re: [RFC PATCH 00/30] Code tagging framework and applications To: David Hildenbrand Cc: Kent Overstreet , Michal Hocko , Mel Gorman , Peter Zijlstra , Andrew Morton , Vlastimil Babka , Johannes Weiner , Roman Gushchin , Davidlohr Bueso , Matthew Wilcox , "Liam R. Howlett" , David Vernet , Juri Lelli , Laurent Dufour , Peter Xu , Jens Axboe , mcgrof@kernel.org, masahiroy@kernel.org, nathan@kernel.org, changbin.du@intel.com, ytcoode@gmail.com, Vincent Guittot , Dietmar Eggemann , Steven Rostedt , Benjamin Segall , Daniel Bristot de Oliveira , Valentin Schneider , Christopher Lameter , Pekka Enberg , Joonsoo Kim , 42.hyeyoo@gmail.com, Alexander Potapenko , Marco Elver , Dmitry Vyukov , Shakeel Butt , Muchun Song , arnd@arndb.de, jbaron@akamai.com, David Rientjes , Minchan Kim , Kalesh Singh , kernel-team , linux-mm , iommu@lists.linux.dev, kasan-dev@googlegroups.com, io-uring@vger.kernel.org, linux-arch@vger.kernel.org, xen-devel@lists.xenproject.org, linux-bcache@vger.kernel.org, linux-modules@vger.kernel.org, LKML Content-Type: text/plain; charset="UTF-8" ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1662046800; a=rsa-sha256; cv=none; b=KTo6OblYTsqF0VF0H8gun20m46/uOYJpE2xHqpgUQdkTFcVDsgOaRSXxfnoMkrPLFWsAs5 7Y6yQcGMrBM+8gfJV5evus3m6oC3UVB7kUfyMP0gfW63BTWz+1PSmUGCeGkAaK75sO+nwH /WrEjgXbs2knNOKsegmOSB8DbJmXdzI= ARC-Authentication-Results: i=1; imf31.hostedemail.com; dkim=pass header.d=google.com header.s=20210112 header.b=CNR1Ntmy; dmarc=pass (policy=reject) header.from=google.com; spf=pass (imf31.hostedemail.com: domain of surenb@google.com designates 209.85.219.171 as permitted sender) smtp.mailfrom=surenb@google.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1662046800; 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: in-reply-to:in-reply-to:references:references:dkim-signature; bh=3cE853tliEpPWpHjkdjBsm55bguibJ1NOZ2l+ytqvIs=; b=D9Eaw8LxriyAXTI+oatg14S1GN+7QmlDByY3VdumKgw1qAkrrp7CYQnu64qCv1EU4nePXq G/pXQmmihVoDjNpPna1p2gwozIUjAF41HY40JVX+HEq+QvlXd331tRCxSNSb8lIXq/+36H zGGocAD+bRIVM4mX9nQbqO7Nil8u5R4= X-Rspamd-Server: rspam03 X-Rspam-User: X-Rspamd-Queue-Id: 8814A20057 X-Stat-Signature: kmwr4kwsf1nmxgrso6m71913jojayjz7 Authentication-Results: imf31.hostedemail.com; dkim=pass header.d=google.com header.s=20210112 header.b=CNR1Ntmy; dmarc=pass (policy=reject) header.from=google.com; spf=pass (imf31.hostedemail.com: domain of surenb@google.com designates 209.85.219.171 as permitted sender) smtp.mailfrom=surenb@google.com X-HE-Tag: 1662046800-842774 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, Sep 1, 2022 at 8:07 AM David Hildenbrand wrote: > > On 01.09.22 16:23, Kent Overstreet wrote: > > On Thu, Sep 01, 2022 at 10:05:03AM +0200, David Hildenbrand wrote: > >> On 31.08.22 21:01, Kent Overstreet wrote: > >>> On Wed, Aug 31, 2022 at 12:47:32PM +0200, Michal Hocko wrote: > >>>> On Wed 31-08-22 11:19:48, Mel Gorman wrote: > >>>>> Whatever asking for an explanation as to why equivalent functionality > >>>>> cannot not be created from ftrace/kprobe/eBPF/whatever is reasonable. > >>>> > >>>> Fully agreed and this is especially true for a change this size > >>>> 77 files changed, 3406 insertions(+), 703 deletions(-) > >>> > >>> In the case of memory allocation accounting, you flat cannot do this with ftrace > >>> - you could maybe do a janky version that isn't fully accurate, much slower, > >>> more complicated for the developer to understand and debug and more complicated > >>> for the end user. > >>> > >>> But please, I invite anyone who's actually been doing this with ftrace to > >>> demonstrate otherwise. > >>> > >>> Ftrace just isn't the right tool for the job here - we're talking about adding > >>> per callsite accounting to some of the fastest fast paths in the kernel. > >>> > >>> And the size of the changes for memory allocation accounting are much more > >>> reasonable: > >>> 33 files changed, 623 insertions(+), 99 deletions(-) > >>> > >>> The code tagging library should exist anyways, it's been open coded half a dozen > >>> times in the kernel already. > >> > >> Hi Kent, > >> > >> independent of the other discussions, if it's open coded already, does > >> it make sense to factor that already-open-coded part out independently > >> of the remainder of the full series here? > > > > It's discussed in the cover letter, that is exactly how the patch series is > > structured. > > Skimming over the patches (that I was CCed on) and skimming over the > cover letter, I got the impression that everything after patch 7 is > introducing something new instead of refactoring something out. Hi David, Yes, you are right, the RFC does incorporate lots of parts which can be considered separately. They are sent together to present the overall scope of the proposal but I do intend to send them separately once we decide if it's worth working on. Thanks, Suren. > > > > >> [I didn't immediately spot if this series also attempts already to > >> replace that open-coded part] > > > > Uh huh. > > > > Honestly, some days it feels like lkml is just as bad as slashdot, with people > > wanting to get in their two cents without actually reading... > > ... and of course you had to reply like that. I should just have learned > from my last upstream experience with you and kept you on my spam list. > > Thanks, bye > > -- > Thanks, > > David / dhildenb >