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 2E2C7D7876C for ; Thu, 21 Nov 2024 14:43:45 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id B539B6B0093; Thu, 21 Nov 2024 09:43:44 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id B03B46B0095; Thu, 21 Nov 2024 09:43:44 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 9A3866B0096; Thu, 21 Nov 2024 09:43:44 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id 7A3936B0093 for ; Thu, 21 Nov 2024 09:43:44 -0500 (EST) Received: from smtpin17.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id 2BCDE1A13D1 for ; Thu, 21 Nov 2024 14:43:44 +0000 (UTC) X-FDA: 82810367430.17.CDCC7F8 Received: from mail-pj1-f49.google.com (mail-pj1-f49.google.com [209.85.216.49]) by imf10.hostedemail.com (Postfix) with ESMTP id E73B2C0008 for ; Thu, 21 Nov 2024 14:43:18 +0000 (UTC) Authentication-Results: imf10.hostedemail.com; dkim=pass header.d=gmail.com header.s=20230601 header.b=MkM7G3BG; spf=pass (imf10.hostedemail.com: domain of andrii.nakryiko@gmail.com designates 209.85.216.49 as permitted sender) smtp.mailfrom=andrii.nakryiko@gmail.com; dmarc=pass (policy=none) header.from=gmail.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1732200038; 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:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=lKfc6WK70Ujcv4qyYOGzHtW3+I+/alF7xv4s8D+Od9A=; b=H8qUOKbS4hk+aWwf7fLqZRRDzNsz7ZOOEm9rXEsnxAeM9buuzwq8uLU050j/Af9Tge+Vbh pFlpWJonmEU+ATxfgY+aXIUjeOxO4I2c71zSLhXiMYde/rybsmYQFgUMyyxeCPpiNthoey XJfyY7H5lt3aB+RAYjRx+1XhBWQ+dHE= ARC-Authentication-Results: i=1; imf10.hostedemail.com; dkim=pass header.d=gmail.com header.s=20230601 header.b=MkM7G3BG; spf=pass (imf10.hostedemail.com: domain of andrii.nakryiko@gmail.com designates 209.85.216.49 as permitted sender) smtp.mailfrom=andrii.nakryiko@gmail.com; dmarc=pass (policy=none) header.from=gmail.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1732200038; a=rsa-sha256; cv=none; b=lP/+sacvHoBJwmsRmxDYzGp2+YOuoztFeeHR02myulRFLVi5vYDGyqiaG+uE3hEsXpc8Ui LdxnJNvuDYHlp0tWqqwi9/j4QDlkwy78/n6uwZW69AB83F9jBMwmAwi3YqjvS9td4qESGL XbakvLCYhtzWU2QrKJMQlbqUk1yb1NM= Received: by mail-pj1-f49.google.com with SMTP id 98e67ed59e1d1-2e59746062fso856858a91.2 for ; Thu, 21 Nov 2024 06:43:41 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1732200221; x=1732805021; darn=kvack.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=lKfc6WK70Ujcv4qyYOGzHtW3+I+/alF7xv4s8D+Od9A=; b=MkM7G3BGc6LWCgd1b+d+Wqjim53tAXfA4CNMxWDeJRuQx9XN5+F9LEE5LesrPOS7BE sdzsxt2kyWaoR5mR80Ba3FlHttDsaSlRpP1xe5kKJAf4ZDS+KRPQ2Gbsng7XR6s2+l9N PeeSwNI+IHY+doU91r32ZePDgRJVhPTefoikwM9Qb9x7b8WS3N9X5OTY0Y2YYQDHP4MU AIaW2orzXKuLvLT8Xysq0nVl+y2zHAWykHM+4nlyMGZ1aZYB8X2Cpk1a43Z8tJqFr2Kc G1Yg2Xpz23GJyuxRZ5ERHfqBnxXOKl9cPo8vyOP0N9aXiunVGKzR87KVoivwz0RTmwhQ OAgA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732200221; x=1732805021; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=lKfc6WK70Ujcv4qyYOGzHtW3+I+/alF7xv4s8D+Od9A=; b=jmKPgLfFMxaOLwtpGnSR2lLRhBqDX0M+P9krdV6axezUiKx59muJgdjuXvweOB9+ZB D/WdwITLdkTqYaVnX76Goeon9ti6jLqLXhZy2UAgiEcWyWUo6t8jU8YKavOZXSR1Q7LF cwGzRq82klR4AdhiD6cjg3X1P9BgG3NyGlR2JZqllHcGapCxpSAyZELGxXdfZZHIyKpP TDAv0JZD04ocWLGBhPo6nofF1i76x585J9lwvxp8Q10bKsmH1gH4IeiW+h7X2g2jnbAp Hgc5miYFtYmpDBQq6cY+qsw4KkEb2uxE5ZUTLcB/dhXDwy2f2daFUDt4ddLPOVrDk33c c7ig== X-Forwarded-Encrypted: i=1; AJvYcCULEclWI1BBOvAyoFRP4bvEMQEBVflqXxmCZJwyYrjKsE3UdDuBT/850IGRFCVasRzDTR62zOZtsQ==@kvack.org X-Gm-Message-State: AOJu0Yx1Slm3+yhWVs+/lED7bmFdNidAkCVRThtewZt3O7qvJ3ESuicR GhfDOxR4WiP8qlGeDk5ynkBkUki8HDH8/d+RT4roGRUTtLDEczYbZWiUXTj6abQxlpLmMOxos7u //fDxFnVg0AvYeTJljIOeGDmWMsE= X-Gm-Gg: ASbGncvzGasL51KBE6xXuvtn0s4hQl7cOO/5zpgjc7092F5sspAOJNt4fmKzfK0RaWz Iy0Lis0r1AwjiYxd6Hoxx9Cmdb83SqSwYtecSpAuT8A4S3s8= X-Google-Smtp-Source: AGHT+IHI1sNYh8TGwsXDad2aw7zBPtwV39Yiz8NDiu1l/E1ipvCw/26XGQRMk0sEM8UETmQ5TWxeG9UsU+oCg/GqQQg= X-Received: by 2002:a17:90b:4a51:b0:2ea:525e:14a7 with SMTP id 98e67ed59e1d1-2eaca7c675bmr8652406a91.29.1732200220751; Thu, 21 Nov 2024 06:43:40 -0800 (PST) MIME-Version: 1.0 References: <20241028010818.2487581-1-andrii@kernel.org> <20241120154323.GA24774@noisy.programming.kicks-ass.net> In-Reply-To: From: Andrii Nakryiko Date: Thu, 21 Nov 2024 06:43:28 -0800 Message-ID: Subject: Re: [PATCH v4 tip/perf/core 0/4] uprobes,mm: speculative lockless VMA-to-uprobe lookup To: Ingo Molnar Cc: Peter Zijlstra , Linus Torvalds , linux-trace-kernel@vger.kernel.org, linux-mm@kvack.org, akpm@linux-foundation.org, oleg@redhat.com, rostedt@goodmis.org, mhiramat@kernel.org, bpf@vger.kernel.org, linux-kernel@vger.kernel.org, jolsa@kernel.org, paulmck@kernel.org, willy@infradead.org, surenb@google.com, mjguzik@gmail.com, brauner@kernel.org, jannh@google.com, mhocko@kernel.org, Andrii Nakryiko , vbabka@suse.cz, shakeel.butt@linux.dev, hannes@cmpxchg.org, Liam.Howlett@oracle.com, lorenzo.stoakes@oracle.com, david@redhat.com, arnd@arndb.de, richard.weiyang@gmail.com, zhangpeng.00@bytedance.com, linmiaohe@huawei.com, viro@zeniv.linux.org.uk, hca@linux.ibm.com, Mark Rutland , Will Deacon , linux-arm-kernel , Catalin Marinas , Kernel Team , Liao Chang Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Rspamd-Server: rspam09 X-Rspamd-Queue-Id: E73B2C0008 X-Stat-Signature: d8chg6tw4ho4dt16rhrysyxap1bpqxey X-Rspam-User: X-HE-Tag: 1732200198-996493 X-HE-Meta: U2FsdGVkX18z9gJvQ/Ul6f5Eyg/S2mesJTZot+qHBzFLZFdKewhJx5QN9fo3NBywUyJr4+z9UWCEio7wRGq4h1nO/6cO3I/75a4iNkxGKOp8o+krrvyiAJiQVrjkQUMAY/gnUtNeKwYIly1DT/oTJTfxvacekAfUVQK0xBA7y6T6RJEmMtRXUcvyAcEsu7m7xe192Sp+5E3TCKoF7wUpd98KbdQsnJUzlt7YG2PehB1GIe1AflXcRt3nWNzbQCnRj0E3zO6f7PdYA8Ybz4+ChkMQylTl0bdkloVUyoD+NUhHSM2OR4fhKWrTg0EQUB9LzSo1qQcq0W1iQsVFb5whgyEJ0geZivhUTUyk0UagSdwh3SXZan+gmnrSsQo46Uekp3C9g1IwuJZ+Y6YAnNy8ADubfxgUE+bqyuHUxHqao7wuddzjRCFZrPy5OG/QvNohS90RdX6LUB1ScYKb+dYnLh8S2lGbT8VItKq/zBksjLiptuCRh7/R3fVO1FCIsZDqbolfxqF2JeumHrxkxaRUqz+50Sz1pVo5522EhktyPMpAkMVLAznYQYl8rfVsFhAvmNcJyM/x7HxavQaeFkihNQmKfAXhF94wcTEOnXNzyU9m/XoGcuOhor0IFNObgBDmUHuyBT/SD2yw8UYUu7Og9LxF0mSiBFYyp9VybukZFSbsZH1qspslq9Z97akJk1qArIZXece1wU2Oh1eu1oRxkOVPkUT3uA+6w/xwVQNXBlZ9uAZLBfQ0CfV/IUPKElHyJiJEzngBQIJBQG55Cqf9Yi78Fe2fMDbHtTXTSHJGfE4XnoG1GAXe4IElrqdxMVP0rGOECgaTzCLS3eycd03FWL9i4ejUg4kAyghqp3ynwbj4IPvpympviHGl3qUp45FgBN3DmiikDbIokNqNcV/6zINgfZEaA+9oc9EgNC51Ae3Hftse8ZSnsLdFTMdEuNPTgbINAaH2NYrTDKYGkMg 7UqijFaB rj8GvuxLT2JT/PoxI+0avL3dA4HYHtyMeUC8Unh4yPCneOgrTLiUWy4iFqzVvb9o8CqgfwEbhSFjxhWLbE2r6OpVutCXuVKpub4jMJUicHHp6prWBtDDxcXdM4ZNCWvOoIjejpW93Iklie9Bq/JJvIjiaOEzwOo4A8hIawT7d4N5d36M6ZMMi1svw541H4VL+WU+DjYBIYaeG+gg1Q/CvZmxJbwXCTBK2Pv6vhN29NMXu8Jmsuzs26mkNXqIxzq4Hog+I7R0jhOKZ37+TD0e+ed4TpE8yDycmfYGw1eX2k7nAO5CqyQLM7NYVSlqMOCbvlXOtoSs2R+5V2mmAC25H+FIV+zXD+1ARdp0jIfKFTQ8cDzMzdhrW0DMqwN8a6jSYJYixxL6wM9SNHlJ7GkO2fvweOb7mc3yRN9NU3ekjC5I5TNHcDwhBuaYTv8W2BtXLUgCPEIG7kclK4S2b0xHNAetVJew3Z6/nu8t11HM8BGuEnQU0fNiIQuw9UOcTzo0IMZjo 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: List-Subscribe: List-Unsubscribe: cc'ing Liao for awareness (should have done it earlier, sorry) On Thu, Nov 21, 2024 at 1:33=E2=80=AFAM Ingo Molnar wrot= e: > > > * Andrii Nakryiko wrote: > > > Is this about Liao's siglock patch set? We are at v4 (!) already (see > > [0]) with Oleg's Acked-by added. > > AFAICS you didn't Cc: me to -v3 and -v4 - and while I'll generally see > those patches too, eventually, there's a delay. Ok, I think we are now switching to my patch set here ([0]), because Liao's v4 ([1]) does have mingo@redhat.com in CC. So, on Liao's behalf, there wasn't really anything specific pointed out that would explain a month's delay. But let's switch to my patch set. Yes, my bad, I didn't CC you directly, that wasn't in any way intentional, and that's my bad and I will make sure to CC you on every patch for uprobes subsystem, even though you are not explicitly listed in UPROBES section of MAINTAINERS ([2]), and Peter was the one who was handling all the uprobe-related stuff since before this summer. But let's please not randomly jump between discussing two separate patch sets here, it's confusing. > > > > Andrii did get some other uprobes scalability work merged in v6.13: > > > > > > - Switch to RCU Tasks Trace flavor for better performance > > > (Andrii Nakryiko) > > > > > > - Massively increase uretprobe SMP scalability by > > > SRCU-protecting the uretprobe lifetime (Andrii Nakryiko) > > > > > > So we've certainly not been ignoring his patches, to the contrary > > > ... > > > > Yes, and as I mentioned, this one is a) ready, reviewed, tested and > > b) complements the other work you mention. > > Sorry, but patchsets that didn't even build a few weeks before the > development window closed are generally pushed further down the > backlog. Think of this as rate-limiting the risk of potentially broken > code entering the kernel. You can avoid this problem by doing more > testing, or by accepting that sometimes one more cycle is needed to get > your patchsets merged. Those build failures were happening in stub implementations, which were used only on !CONFIG_PER_VMA_LOCK configuration, which I'm not even sure if possible to get on x86-64. When I tried to reproduce this locally I couldn't even get such configuration. Thankfully we have a kernel test robot that would test multiple architectures and configurations and it did catch it on i386 and loongarch64. And was fixed quickly. It doesn't seem fair or reasonable to penalize patch sets for *weeks*, silently and without any notice just for this, IMO. The patch set was very thoroughly tested, actually. Not just building, but also running various unit tests (BPF selftests in particular). But even more so, I built an entire uprobe stress-testing tool just to test all my uprobe-related. I deployed custom kernels and ran these stress tests on all uprobe patch sets and their revisions, over many hours. Sure, my main platform is x86-64, so that's where all the testing was done. But you can't accuse me of negligence. > > > [...] It removes mmap_lock which limits scalability of the rest of > > the work. Is there some rule that I get to land only two patch sets > > in a single release? > > Your facetous question and the hostile tone of your emails is not > appreciated. I'm sticking to the facts in these emails. And when I get a response in the style of "you got two patch sets in, why are you complaining", that's not exactly friendly and fair. I put a lot of effort and time not just into producing and testing all those patches, but also into the logistics of it, coordinating with other people working within uprobes subsystem. And instead of accusations, I'd like to get an understanding of expectations I can have in terms of handling patches. Being ignored for many weeks is not OK. If you don't like something about what I do or how, procedurally or technically, please call it out and I'll try to fix whatever the problem might be. Silent treatment is not productive. But while on the topic. Those two patch sets you mentioned didn't go in smoothly and quickly either. "Switch to RCU Tasks Trace flavor for better performance" in particular should have gone in with the original patch set one release earlier. But instead that patch was dropped from the tree after applying it. Silently. I was not notified at all (5 days that passed before I asked would be plenty of time to mention this, IMO). It's good I noticed this, inquired with an email reply (after making sure it's not some transient patch handling issue), and only after that I got a reply that there was a build failure I had to fix. You can see the thread at [4]. > > Me pointing out that two other patchsets of yours got integrated simply > demonstrates how your original complaint of an 'ignore list' is not > just unprofessional on its face, but also demonstrably unfair: > > > > > > I'm not sure what's going on here, this patch set seems to be > > > > > in some sort of "ignore list" on Peter's side with no > > > > > indication on its destiny. My "ignore list" complaint is specifically about this patch set, which I explicitly stated above in the quote you provided. So yes, it's a professional, and demonstrably fair statement, and I provided the timeline and supporting links. > > Trying to pressure maintainers over a patchset that recently had build > failures isn't going to get your patches applied faster. > I'm not asking to apply my patches blindly without critical review or anything like that. I'm not expecting reviews or even just email replies within a few days of posting. I *do* expect some sort of reaction, though, and not after many weeks of inactivity and pinging from my side, yes. And note, I got replies only after sending an email straight to Linus. I'm not pressuring anyone into anything. But as a maintainer myself, I do think that being a maintainer is not just about having rights, it's also about responsibilities. Let's please stop with the excuses and instead discuss constructive solutio= ns. > Thanks, > > Ingo [0] https://lore.kernel.org/linux-trace-kernel/20241028010818.2487581-1-and= rii@kernel.org/ [1] https://lore.kernel.org/linux-trace-kernel/20241022073141.3291245-1-lia= ochang1@huawei.com/ [2] https://lore.kernel.org/all/172074397710.247544.17045299807723238107.st= git@devnote2/ [3] https://github.com/libbpf/libbpf-bootstrap/commit/2f88cef90f9728ec8c7be= e7bd48fdbcf197806c3 [4] https://lore.kernel.org/all/CAEf4BzZihPPiReE3anhrVOzjoZW5v4vFVouK_Arm8v= JexCTT4g@mail.gmail.com/