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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id CB0C0C61DFD for ; Wed, 2 Sep 2026 09:18:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Type:MIME-Version: Message-ID:Date:References:In-Reply-To:Subject:Cc:To:From:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=KT+Ce48Xpd0kjMmEXV03S5Jcy8w12n7wdIchht7SYMo=; b=aYcvSV8Pum0SSJNJtOYQEnS7ML /vmQuuDs6+43n4brpJKxQTrShnrM7+uB8tV1ONxQbSKudzjuMH4p/uuiT6rBwWvQw1ik8GP6T9n9U xw8aqppaVLmNVQqYNxWNcNX+O7TCM/PNy23H1YAk71BdfB9+XdofOTY4yJgpAGeAPTlGS4GsQAdlo cjrbkSBblM0MkfJ8Hn+z686RVBxmxASneF4ls8OYe0Oo65L71GdnZt4V53Fck9Qn9LVeArP4FwY4d 0wbLF35QAvKMeTAm+WxfaFKqnO1DiWl7t9QoRUhxLau5i3DYXxdwBP23BMaznmq9qTi7LB5JpPpvl YbMR42jg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1x1h6p-0000000ECVN-0LCY; Wed, 02 Sep 2026 09:18:19 +0000 Received: from tor.source.kernel.org ([172.105.4.254]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1x1h6n-0000000ECVD-06bG for kexec@lists.infradead.org; Wed, 02 Sep 2026 09:18:17 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by tor.source.kernel.org (Postfix) with ESMTP id 111BE600D0; Wed, 2 Sep 2026 09:18:16 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 0FBBE1F000E9; Wed, 2 Sep 2026 09:18:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1788340695; bh=KT+Ce48Xpd0kjMmEXV03S5Jcy8w12n7wdIchht7SYMo=; h=From:To:Cc:Subject:In-Reply-To:References:Date; b=KfgtVI29FS0imhdh0+W+batqoDbsjCXJXqvkpFBsgtpqvH8W8luOsa9A6Pi/qs/Br dEWZpE9IuTz6LbygxWHYRzBJ1RklP6MNOtZtRUoY00WdKmWUnhLjrZq6L+v+csXwrH sFq91TZfJSglUG9M2/kJWusV8YdtVYFvUJmuGibPf+I8Oqc47ZIoPBjjbgzkOPHRAN iBmJjMDFonm01p/CoaokGqyPFoUn7kiLzW6sWHERLjv5NwEjJ0sJIPf1ejMGbyjHlG G5Ad4pGxOZkr81uSfQREHrnQQtHJXcNaSBPv/CWSQdXhWChBz4QUSqf8PHWeMVcRMw zHGvyDgoHZyZw== From: Pratyush Yadav To: Baoquan He Cc: Mike Rapoport , David Matlack , kexec@lists.infradead.org, Andrew Morton , Pasha Tatashin , Pratyush Yadav , Dave Young , Alexander Graf , Roman Gushchin Subject: Re: RFC: Enable Sashiko code reviews for kexec@lists.infradead.org In-Reply-To: (Baoquan He's message of "Wed, 2 Sep 2026 16:25:42 +0800") References: Date: Wed, 02 Sep 2026 11:18:12 +0200 Message-ID: <2vxzqzjcxb8r.fsf@kernel.org> User-Agent: Gnus/5.13 (Gnus v5.13) MIME-Version: 1.0 Content-Type: text/plain X-BeenThere: kexec@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "kexec" Errors-To: kexec-bounces+kexec=archiver.kernel.org@lists.infradead.org +Cc Roman because I am complaining about Sashiko ;-) On Wed, Sep 02 2026, Baoquan He wrote: > On 09/02/26 at 11:04am, Mike Rapoport wrote: >> Hi Baoquan, >> >> On Wed, Sep 02, 2026 at 01:52:51PM +0800, Baoquan He wrote: >> > On 09/02/26 at 07:34am, Mike Rapoport wrote: >> > > On Tue, Sep 01, 2026 at 01:27:34PM -0700, David Matlack wrote: >> > > > Hi kexec@ mailing list and maintainers, >> > > > >> > > > I would like to enable Sashiko code reviews for patches sent to the >> > > > kexec mailing list. I have found it useful for patches I have sent and >> > > > reviewed on the kvm and linux-pci mailing lists, which have Sashiko >> > > > enabled. >> > > > >> > > > I sent a pull request to Sashiko to add kexec here: >> > > > >> > > > https://github.com/sashiko-dev/sashiko/pull/474 >> > > >> > > diff --git a/sashiko.dev/email_policy.toml b/sashiko.dev/email_policy.toml >> > > index e6e4fb57f..e9ad97708 100644 >> > > --- a/sashiko.dev/email_policy.toml >> > > +++ b/sashiko.dev/email_policy.toml >> > > @@ -175,6 +175,12 @@ reply_to_author = false >> > > cc_individuals = false >> > > cc = ["Chuck Lever ", "Jeff Layton ", "Anna Schumaker "] >> > > >> > > +[subsystems.kexec] >> > > +lists = ["kexec@lists.infradead.org"] >> > > +reply_to_author = true >> > > +send_positive_review = true >> > > >> > > Do we really want those? >> > >> > Some components don't want to CC list, then other people are welcome to >> > drop any comment during reviewing, while cc to maintainer and author >> > looks good to me. Just an input, no objection to any move. >> >> Sorry, I wasn't clear, I meant do we really want extra emails saying >> sashiko is happy? > > Oh, sorry. I misunderstood, I thought you had agreed to introduce > sashiko to part of list/author/maintainer or all. > > I think there are two different ways and they have different impact: > > 1) CC list (CC author and maintainer can be ignored beause all people > can see it) This is a side topic, but if we only Cc the list then Sashiko won't Cc the other people in the thread, like the usual "reply to all" every mail clients does. It will _only_ Cc the list. See [0] for example. I don't think that is a good idea, because Sashiko might complain about something, and then the patch author can reply to the complaint. People who are not subscribed to the list will neither see Sashiko's complaint nor the author's response. So they lose a useful part of the patch review process. I am one of those people. I don't subscribe to any mailing lists and instead use lei [1] to fetch them. I treat my inbox as the primary stream of patches, and then glance at the lists every now and then. If I get some patches in my inbox but then I don't get any of the follow-ups from the author to Sashiko, that's annoying. Sure, I might see them on the list later, but it is still annoying. And of course, there might be patches that touch multiple subsystems and maintainers of that subsystem might not subscribe to or track kexec@ at all. So they will entirely miss the conversation. If Sashiko is useful enough to send replies on the list, then I think it is useful enough to reply to everyone. IIRC some people have complained in the past about automatically getting replies from Sashiko so maybe that is why this behaviour exists? Roman, is that correct? Is there any appetite yet to change that behaviour? Or could we perhaps have an unsubscribe mechanism where people who don't want to see anything from Sashiko can unsubscribe themselves and Sashiko will skip them when replying? [0] https://lore.kernel.org/kvm/20260902072059.90ADC1F000E9@smtp.kernel.org/ [1] https://public-inbox.org/lei.txt > This equals to cc author and push other reviewers away. And no > reviewers will be patient and curious enough to check the sashiko > report except of maintainers. Because maintainers need pick patch > and may check if all reported issues are handled; > > 2) Cc maintainers when patch is cooked > Maintainers tell patch author to check the report. Maybe CC patch > author directly can save maintainers' effort. > > I personally think 2) is good. About whether introducing sashiko, > I would vote yes. I buy token by myself, sashiko would be a plus. FWIW, Sashiko already tracks the kexec@ list and you can see all the patches in the web interface [2]. So you don't need to burn your tokens on reviewing kexec patches ;-) What David is proposing is to send those reviews to the list. [2] https://sashiko.dev/#/?list=org.infradead.lists.kexec > > Thanks > Baoquan -- Regards, Pratyush Yadav