On Mon, Oct 07, 2024 at 08:23:54PM +0200, Christian König wrote: > Am 07.10.24 um 19:59 schrieb Greg KH: > > On Mon, Oct 07, 2024 at 07:53:26PM +0200, Christian König wrote: > > > Hi Greg, > > > > > > please drop this patch from all backports. It turned out to be broken and > > > the old handling has been restored by a revert. > > > > > > Sorry for the noise. The revert should show up in Linus tree by the end of > > > the week. > > What is the revert's git id? I'd rather take that so we don't have any > > "why wasn't this patch applied" stuff later on. > > Good point. I was also not 100% sure which ones of Tvrtko's patches we had > to revert. > > Here is the git Id from the drm-misc-fixes branch: > > commit abf201f6ce14c4ceeccde5471bdf59614b83a3d8 > Author: Christian König <christian.koenig@xxxxxxx> > Date: Mon Sep 30 15:07:49 2024 +0200 > > drm/sched: revert "Always increment correct scheduler score" > > This reverts commit 087913e0ba2b3b9d7ccbafb2acf5dab9e35ae1d5. > > It turned out that the original code was correct since the rq can only > change when there is no armed job for an entity. > > This change here broke the logic since we only incremented the counter > for the first job, so revert it. > > Signed-off-by: Christian König <christian.koenig@xxxxxxx> > Acked-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxxx> > Link: https://patchwork.freedesktop.org/patch/msgid/20240930131451.536150-1-christian.koenig@xxxxxxx > > Sorry for the noise, we should have catched that during the review and put > the stable tag on the revert as well. Thanks for the info, I've picked this revert up now as well. greg k-h