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=-3.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no 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 2FE66C07E9A for ; Wed, 14 Jul 2021 17:22:13 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id C19F0613B5 for ; Wed, 14 Jul 2021 17:22:12 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C19F0613B5 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=mit.edu Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id F2D0C6B0078; Wed, 14 Jul 2021 13:22:12 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id F03956B007E; Wed, 14 Jul 2021 13:22:12 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id DCC1A6B0080; Wed, 14 Jul 2021 13:22:12 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0132.hostedemail.com [216.40.44.132]) by kanga.kvack.org (Postfix) with ESMTP id B65D66B0078 for ; Wed, 14 Jul 2021 13:22:12 -0400 (EDT) Received: from smtpin21.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id A27C5185B9317 for ; Wed, 14 Jul 2021 17:22:11 +0000 (UTC) X-FDA: 78361861662.21.3CC0046 Received: from outgoing.mit.edu (outgoing-auth-1.mit.edu [18.9.28.11]) by imf09.hostedemail.com (Postfix) with ESMTP id AF7D73000104 for ; Wed, 14 Jul 2021 17:22:10 +0000 (UTC) Received: from callcc.thunk.org (96-65-121-81-static.hfc.comcastbusiness.net [96.65.121.81]) (authenticated bits=0) (User authenticated as tytso@ATHENA.MIT.EDU) by outgoing.mit.edu (8.14.7/8.12.4) with ESMTP id 16EHLxpS021655 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 14 Jul 2021 13:22:00 -0400 Received: by callcc.thunk.org (Postfix, from userid 15806) id 59F2A4202F5; Wed, 14 Jul 2021 13:21:59 -0400 (EDT) Date: Wed, 14 Jul 2021 13:21:59 -0400 From: "Theodore Y. Ts'o" To: Greg Kroah-Hartman Cc: Sasha Levin , Andrew Morton , Michal Hocko , Hugh Dickins , Linus Torvalds , Mike Kravetz , Miaohe Lin , linux-kernel@vger.kernel.org, linux-mm@kvack.org, stable@vger.kernel.org Subject: Re: 5.13.2-rc and others have many not for stable Message-ID: References: <2b1b798e-8449-11e-e2a1-daf6a341409b@google.com> <20210713182813.2fdd57075a732c229f901140@linux-foundation.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Rspamd-Server: rspam06 X-Rspamd-Queue-Id: AF7D73000104 X-Stat-Signature: ajdf6389p58r9yw6parsdd6keq1kmtpg Authentication-Results: imf09.hostedemail.com; dkim=none; spf=none (imf09.hostedemail.com: domain of tytso@mit.edu has no SPF policy when checking 18.9.28.11) smtp.mailfrom=tytso@mit.edu; dmarc=none X-HE-Tag: 1626283330-343432 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 Wed, Jul 14, 2021 at 05:46:22PM +0200, Greg Kroah-Hartman wrote: > > The number of valid cases where someone puts a "Fixes:" tag, and that > patch should NOT be backported is really really slim. Why would you put > that tag and not want to have known-broken kernels fixed? > > If it really is not an issue, just do not put the "Fixes:" tag? I think it really boils down to what the tags are supposed to mean and what do they imply. The argument from the other side is if the Stable maintainers are interpreting the Fixes: tag as an implicit "CC: stable", why should we have the "Cc: stable" tag at all in that case? We could also have the policy that in the case where you have a fix for a bug, but it's super subtle, and shouldn't be automatically backported, that the this should be explained in the commit, e.g., This commit fixes a bug in "1adeadbeef33: lorem ipsum dolor sit amet" but it is touching code which subtle and quick to anger, the bug isn't all that serious. So please don't backport it automatically; someone should manually do the backport and run the fooblat test before sumitting it to the stable maintainers. Andrew seems to be of the opinion that these sorts of cases are very common. I don't have enough data to have a strong opinion either way. But if you are right that it is a rare case, then sure, simply omitting the Fixes: tag and using text in the commit description would work. We just need to agree that this is the convention that we all shoulf be using. I still wonder though what's the point of having the "Cc: stable" tag if it's implicitly assumed to be there if there is a Fixes: tagle. Cheers, - Ted