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=-6.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,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 9C712C11F67 for ; Wed, 14 Jul 2021 17:34:19 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 3C5EC613C1 for ; Wed, 14 Jul 2021 17:34:19 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 3C5EC613C1 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linuxfoundation.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 6913D6B0078; Wed, 14 Jul 2021 13:34:19 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 6686C6B007E; Wed, 14 Jul 2021 13:34:19 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 509F56B0080; Wed, 14 Jul 2021 13:34:19 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0114.hostedemail.com [216.40.44.114]) by kanga.kvack.org (Postfix) with ESMTP id 2F2F16B0078 for ; Wed, 14 Jul 2021 13:34:19 -0400 (EDT) Received: from smtpin23.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay02.hostedemail.com (Postfix) with ESMTP id 12E9F2775A for ; Wed, 14 Jul 2021 17:34:18 +0000 (UTC) X-FDA: 78361892196.23.A375064 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by imf04.hostedemail.com (Postfix) with ESMTP id 75ADE50000AD for ; Wed, 14 Jul 2021 17:34:17 +0000 (UTC) Received: by mail.kernel.org (Postfix) with ESMTPSA id 0C24E613BE; Wed, 14 Jul 2021 17:34:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1626284056; bh=VXOK22zekk8d0nAluf7/iFpjMB0tQ21s1pyi6ngOZa4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=lL8g74uPTYRpKc+z4V1/510oVl+jgZ6caECWjeZuzaqzCzn3uR+r5aMaiMJ4Oo5We UgTfe7AkQash+rWa6L0oPD8D7bQlCdtklmuL90kCB5q7CApXJ+xwHqcUtCiddQkFSe jZVpmLGMEe5nZCmTpDw3gra83zFzeJRMcOU8aAmI= Date: Wed, 14 Jul 2021 19:34:14 +0200 From: Greg Kroah-Hartman To: "Theodore Y. Ts'o" 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: 75ADE50000AD X-Stat-Signature: 9jiee89xju3hue5potffjbo6mzcaurjh Authentication-Results: imf04.hostedemail.com; dkim=pass header.d=linuxfoundation.org header.s=korg header.b=lL8g74uP; spf=temperror (imf04.hostedemail.com: error in processing during lookup of gregkh@linuxfoundation.org: DNS error) smtp.mailfrom=gregkh@linuxfoundation.org; dmarc=temperror reason="server fail" header.from=linuxfoundation.org (policy=temperror) X-HE-Tag: 1626284057-563491 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 01:21:59PM -0400, Theodore Y. Ts'o wrote: > 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? I would love to not have to look at the Fixes: tag, but today we have to because not all subsystems DO use cc: stable. We miss loads of real fixes if we only go by cc: stable right now. If you can go and fix those subsystems to actually remember to do this "properly", wonderful, we will not have to mess with only Fixes: tags again. But until that happens, we have to live with what we have. And all we have are "hints" like Fixes: to go off of. > 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. That's wonderful, I would love to see that more, and we do see that on some commits. And we mostly catch them (I miss them at times, but that's my fault, not the developer/maintainers fault.) > 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. Because cc: stable came first, and for some reason people think that it is all that is necessary to get patches committed to the stable tree, despite it never being documented or that way. I have to correct someone about this about 2x a month on the stable@vger list. thanks, greg k-h