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 gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (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 08AE8C4828F for ; Fri, 2 Feb 2024 21:04:27 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 81B1810E178; Fri, 2 Feb 2024 21:04:27 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (1024-bit key; unprotected) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="FeV6pf/Y"; dkim-atps=neutral Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by gabe.freedesktop.org (Postfix) with ESMTPS id C330010E02F for ; Fri, 2 Feb 2024 21:04:16 +0000 (UTC) Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id EC7CF62862; Fri, 2 Feb 2024 21:04:12 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 82A11C433C7; Fri, 2 Feb 2024 21:04:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1706907852; bh=uJWZtIkntTroOirKsbD7/9ZTWGEScfywCm5pIdruVxI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=FeV6pf/Y36PqxsxQmhzuIh6XYkhogUC57yFGQ634Hkx6cxXyXeq9yvpB0Tx1tQmaW i8/YXvUV+OYB2+s616lZ8XgqOUo9t/NbiaZU4ZfB1canjVQ68r4RlULXvcAyywZXgj KCGa4VC6rXpsjI7UTlWUPAOaRoyVzXWtxoY2gw/w= Date: Fri, 2 Feb 2024 13:04:11 -0800 From: Greg KH To: Lucas De Marchi Cc: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= , Sasha Levin , stable-commits@vger.kernel.org, Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , intel-gfx@lists.freedesktop.org, Jani Nikula , Joonas Lahtinen , Rodrigo Vivi , Tvrtko Ursulin , Oded Gabbay , Thomas =?iso-8859-1?Q?Hellstr=F6m?= Subject: Re: Re: Patch "drm: Fix color LUT rounding" has been added to the 6.7-stable tree Message-ID: <2024020243-wriggly-percolate-23e6@gregkh> References: <20240201170320.69864-1-sashal@kernel.org> <2024020119-user-aviation-b3c5@gregkh> <2024020120-unclog-junction-8a5f@gregkh> <4rfubhgx5ro6uyril7labgxoovuj26funttdts6bbcrbdfjxlj@mrwxtts657g2> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <4rfubhgx5ro6uyril7labgxoovuj26funttdts6bbcrbdfjxlj@mrwxtts657g2> X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" On Fri, Feb 02, 2024 at 11:35:33AM -0600, Lucas De Marchi wrote: > On Fri, Feb 02, 2024 at 06:53:03PM +0200, Ville Syrjälä wrote: > > On Thu, Feb 01, 2024 at 11:17:28AM -0800, Greg KH wrote: > > > On Thu, Feb 01, 2024 at 08:35:24PM +0200, Ville Syrjälä wrote: > > > > On Thu, Feb 01, 2024 at 10:16:48AM -0800, Greg KH wrote: > > > > > On Thu, Feb 01, 2024 at 08:05:19PM +0200, Ville Syrjälä wrote: > > > > > > On Thu, Feb 01, 2024 at 12:03:20PM -0500, Sasha Levin wrote: > > > > > > > This is a note to let you know that I've just added the patch titled > > > > > > > > > > > > > > drm: Fix color LUT rounding > > > > > > > > > > > > > > to the 6.7-stable tree which can be found at: > > > > > > > http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary > > > > > > > > > > > > > > The filename of the patch is: > > > > > > > drm-fix-color-lut-rounding.patch > > > > > > > and it can be found in the queue-6.7 subdirectory. > > > > > > > > > > > > > > If you, or anyone else, feels it should not be added to the stable tree, > > > > > > > please let know about it. > > > > > > > > > > > > I guess I wasn't clear enough in the other mail... > > > > > > > > > > > > NAK for all of backports of this patch. > > > > > > > > > > Ok, but why? It seems that you are fixing a real issue here, right? If > > > > > not, the changelog is not clear with that at all... > > > > > > > > > > I'll go drop it now, thanks. > > > > > > > > Because backporting it would require other backports that depend on > > > > the rounding behaviour. > > > > > > > > Can I somehow fully opt out of these automagic backports? > > > > If I want my stuff backported I'll ask for it. > > > > > > You can, just let me know what exact files should be ignored, or you can > > > send a patch against this file: > > > https://git.kernel.org/pub/scm/linux/kernel/git/stable/stable-queue.git/tree/ignore_list > > > > I think we should add at least i915 and xe there. cc: maintainers > > It does feel a little wild to decide a patch needs to be backported > based on the commit title starting with "Fix", or whatever way was used > here. We always relied on patches being backported based on a) having a > Fixes: trailer and b) the commit pointed out in that trailer > being present in that stable version. Or the others options shown > in Documentation/process/stable-kernel-rules.rst > > Looking at the commit in question, c6fbb6bca10838485b820e8a26c23996f77ce580 > there's no such a trailer. Did I miss something from > Documentation/process/stable-kernel-rules.rst? No, this patch was picked up because of the text in the patch itself that said "Fix color...". We have a tool that does this to catch the patches that people forget to tag for stable inclusion. The patch really does seem like it needs to be backported, there's nothing in it that says otherwise as it is nice and self-contained and well documented. thanks, greg k-h