From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4317F28E609 for ; Wed, 4 Jun 2025 13:30:01 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749043801; cv=none; b=Lr0g4FWQe5/l4eEytT44+RjS1foVTE+Qee6pFQpqXxeBWH77gaJ8ASeNCf6i5Vw6qrtxFmtjItgdt9Eb3r/Tz0iihMUfmgjNKaoMiqjazBdurxSrRCvVU6xiDyIuI1ArS3V2d9+5bUyqMusgIF86MoHiLZeRfLgZMXTLg1XQ88I= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749043801; c=relaxed/simple; bh=uAGF1rGgwXCCFTMw3y6E5QwiVbOIGvouMM0eNw+XRz0=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=eWfLi4Jr5ySDJWqnOOepyw4EROS9Qh0MoBZezXbw14Nf2uJ1/nLURP0MjeRSWMhewg1WkgINxWEbMxyP72RVTjtBJcgATV7cTC7nvxBjU0VPa+j4lXT5Xf9NRifGtBO8W+52F9DwiDyvnCxK18UnNo3PDF0XosS3ZmS/0QgX5wk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=qU1R/kXK; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="qU1R/kXK" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9BAD0C4CEF0; Wed, 4 Jun 2025 13:30:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1749043801; bh=uAGF1rGgwXCCFTMw3y6E5QwiVbOIGvouMM0eNw+XRz0=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=qU1R/kXKp9D6+jx5VYTlP7L0TWBw4HwzAe/nBG7841zDc0hNJnT/FgXj0h3Sy5Syx VLic0jaMZDl6jLnrJPzQc/m19ZOHyxAsvhF67K0xV5MR0NAqRRJ3lKUU+sfVwl7Uw7 x5rkOTr8SInBk1mby7fViWxyxBJLqPPocmLapX7U= Date: Wed, 4 Jun 2025 15:29:58 +0200 From: Greg KH To: Alex Deucher Cc: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= , Aurabindo Pillai , stable@vger.kernel.org, Alex Deucher , David Airlie , Simona Vetter , dri-devel@lists.freedesktop.org Subject: Re: [PATCH] Revert "drm/amd/display: more liberal vmin/vmax update for freesync" Message-ID: <2025060410-skinning-unguided-a3de@gregkh> References: <20250530200918.391912-1-aurabindo.pillai@amd.com> Precedence: bulk X-Mailing-List: stable@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: On Wed, Jun 04, 2025 at 09:15:14AM -0400, Alex Deucher wrote: > On Wed, Jun 4, 2025 at 5:40 AM Uwe Kleine-König > wrote: > > > > Hello Alex, > > > > On Fri, May 30, 2025 at 04:14:09PM -0400, Alex Deucher wrote: > > > On Fri, May 30, 2025 at 4:09 PM Aurabindo Pillai > > > wrote: > > > > > > > > This reverts commit 219898d29c438d8ec34a5560fac4ea8f6b8d4f20 since it > > > > causes regressions on certain configs. Revert until the issue can be > > > > isolated and debugged. > > > > > > > > Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/4238 > > > > Signed-off-by: Aurabindo Pillai > > > > Acked-by: Alex Deucher > > > > > > Already included in my -fixes PR for this week: > > > https://lists.freedesktop.org/archives/amd-gfx/2025-May/125350.html > > > > Note the way this was done isn't maximally friendly to our stable > > maintainers though. > > > > The commit in your tree (1b824eef269db44d068bbc0de74c94a8e8f9ce02) is a > > tad better than the patch that Aurabindo sent as it has: > > > > This reverts commit cfb2d41831ee5647a4ae0ea7c24971a92d5dfa0d ... > > > > which at least is a known commit and has Cc: stable. > > > > However this is still a bit confusing as commit cfb2d41831ee has no Cc: > > stable, but a duplicate in mainline: f1c6be3999d2 that has Cc: stable. > > > > So f1c6be3999d2 was backported to 6.14.7 (commit > > 4ec308a4104bc71a431c75cc9babf49303645617), 6.12.29 (commit > > 468034a06a6e8043c5b50f9cd0cac730a6e497b5) and 6.6.91 (commit > > c8a91debb020298c74bba0b9b6ed720fa98dc4a9). But it might not be obvious > > that 1b824eef269db44d068bbc0de74c94a8e8f9ce02 needs backporting to trees > > that don't contain cfb2d41831ee (or a backport of it). > > > > Please keep an eye on that change that it gets properly backported. > > DRM patches land in -next first since that is where the developers > work and then bug fixes get cherry-picked to -fixes. When a patch is > cherry-picked to -fixes, we use cherry-pick -x to keep the reference > to the original commit and then add stable CC's as needed. See this > thread for background: > https://lore.kernel.org/dri-devel/871px5iwbx.fsf@intel.com/T/#t And that thread shows how the confusion between git ids that are reverted and committed to the tree cause reverts to get missed with our automatic tools, so you HAVE to explicitly tag them as cc: stable, which is not always done :( thanks, greg k-h