Re: [PATCH 03/15] sched/fair: Add lag based placement
From: Mike Galbraith
Date: Sat Oct 14 2023 - 04:24:35 EST
On Fri, 2023-10-13 at 18:35 +0200, Peter Zijlstra wrote:
> On Fri, Oct 13, 2023 at 12:34:28AM +0200, Peter Zijlstra wrote:
>
> > Right, so I do have this:
> >
> > https://git.kernel.org/pub/scm/linux/kernel/git/peterz/queue.git/commit/?h=sched/eevdf&id=344944e06f11da25b49328825ed15fedd63036d3
> >
> > That allows tasks to sleep away the lag -- with all the gnarly bits that
> > sleep time has. And it reliably fixes the above. However, it also
> > depresses a bunch of other stuff. Never a free lunch etc.
> >
> > It is so far the least horrible of the things I've tried.
>
> So the below is one I conceptually like more -- except I hate the code,
> nor does it work as well as the one linked above.
>
> (Mike, this isn't the same one you saw before -- it's been 'improved')
Still improves high frequency switchers vs hogs nicely.
tbench vs massive_intr
6.4.16-stable avg
2353.57 2311.77 2399.27 2354.87 1.00
6.4.16-eevdf avg
2037.93 2014.57 2026.84 2026.44 .86 1.00 DELAY_DEQUEUE v1
1893.53 1903.45 1851.57 1882.85 .79 .92 NO_DELAY_DEQUEUE
2193.33 2165.35 2201.82 2186.83 .92 1.07 DELAY_DEQUEUE v2
It's barely visible in mixed youtube vs compute in i7-4790 box,
squinting required, and completely invisible in dinky rpi4.
A clear win along with no harm to the mundane mix is a good start in my
book. Here's hoping canned benchmark bots don't grumble.
-Mike