From nobody Fri Mar 31 19:07:58 2023 X-Original-To: freebsd-hackers@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4Pp8v66L1mz437tH for ; Fri, 31 Mar 2023 19:08:02 +0000 (UTC) (envelope-from markjdb@gmail.com) Received: from mail-qt1-x82b.google.com (mail-qt1-x82b.google.com [IPv6:2607:f8b0:4864:20::82b]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "GTS CA 1D4" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4Pp8v64gZkz3Hcj for ; Fri, 31 Mar 2023 19:08:02 +0000 (UTC) (envelope-from markjdb@gmail.com) Authentication-Results: mx1.freebsd.org; none Received: by mail-qt1-x82b.google.com with SMTP id p2so17555245qtw.13 for ; Fri, 31 Mar 2023 12:08:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; t=1680289682; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:sender:from:to:cc:subject:date:message-id :reply-to; bh=I7wivi/BupONQGpatWw+ZGulI+q/KEqq7EYbH2LYjho=; b=m6vCVaaa6ld0y+NlO2pRRhAckWifOBpBkSjVw70IjLtkVr9+MTR6x441DqbjFloZ/c fDlxWChRhx+OpL2W4iefukrQWPx3Rimkwm3SLPfDWb1xecjyP03AbPnh1/yb8bdKVa3F L6O7IMBazi4barqXcn3XJNyziLTGTQtO8QOrXffavdUza+PnETyz1KUyx0XUMXKwxONq Pfv6c9LOVnEmCmjJZWKyLDjgDqmI4RKfYHh854u69TKyOg0S33FqawogC7Bvs4tgwMjn PhAblpuiU1J5auetBPdbAFArUCK59pmttbtxpaT4u7e4JbYc1aXhNnSDZ63SZIo9Pj5W pOZg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1680289682; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:sender:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=I7wivi/BupONQGpatWw+ZGulI+q/KEqq7EYbH2LYjho=; b=x7y1g2s1vD3Wp+LUgJ2/D83AR/M8bvVPG2zWLHpm2a5+JfZ/Pr+yIHPHDegOzYFflN qB6huUDhQYxtM61KdyxXHaOKZ7XUvYrs6Q6+aQZyLQDiMTK2fGeoK6lda1YsdTbHS20D QLOsm3G7PySVwFqyHJ3aiKcFEyoM6XsOCuvJEHHuK2vHNMd5hFsoN5ElwugYTci5qfg9 B6yarpGXr9TgbqhpsCoAfe5J2VGdSmpJvJ5EMeprPP8rD6WgpfM0QtC079xe3ceeH5EG iHiCKdyrbngLqRRo4GtS52MlvkkAioV/OK9DUh107qIqdIfX7F5b4zuFIH4Dk2kXmTes fiWQ== X-Gm-Message-State: AO0yUKUPJMWowFRaL+k2LLQQOP5pccYe7zaJam6eBELu1z+SfWILrgCd S9rOHXc5Gp1UfLeOE2+7RoN9woJfKmQ= X-Google-Smtp-Source: AKy350Z/AleDbdlqDwJBJYE86C1ycpNpWNfRTpmDtfsVd/cV8QUFwZ5mpOpeeCLa+GkUUVgGMxIZ0A== X-Received: by 2002:a05:622a:214:b0:3e3:902a:a084 with SMTP id b20-20020a05622a021400b003e3902aa084mr45698794qtx.6.1680289681835; Fri, 31 Mar 2023 12:08:01 -0700 (PDT) Received: from framework ([24.114.52.162]) by smtp.gmail.com with ESMTPSA id y3-20020a37f603000000b007468733cd1fsm860506qkj.58.2023.03.31.12.08.00 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 31 Mar 2023 12:08:01 -0700 (PDT) Date: Fri, 31 Mar 2023 15:07:58 -0400 From: Mark Johnston To: Mateusz Guzik Cc: freebsd-hackers@freebsd.org Subject: Re: Periodic rant about SCHED_ULE Message-ID: References: List-Id: Technical discussions relating to FreeBSD List-Archive: https://lists.freebsd.org/archives/freebsd-hackers List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-freebsd-hackers@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Rspamd-Queue-Id: 4Pp8v64gZkz3Hcj X-Spamd-Bar: ---- X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[]; ASN(0.00)[asn:15169, ipnet:2607:f8b0::/32, country:US] X-Rspamd-Pre-Result: action=no action; module=replies; Message is reply to one we originated X-ThisMailContainsUnwantedMimeParts: N On Fri, Mar 31, 2023 at 08:41:41PM +0200, Mateusz Guzik wrote: > On 3/30/23, Mark Johnston wrote: > > On Thu, Mar 30, 2023 at 05:36:54PM +0200, Mateusz Guzik wrote: > >> I looked into it a little more, below you can find summary and steps > >> forward. > >> > >> First a general statement: while ULE does have performance bugs, it > >> has better basis than 4BSD to make scheduling decisions. Most notably > >> it understands CPU topology, at least for cases which don't involve > >> big.LITTLE. For any non-freak case where 4BSD performs better, it is a > >> bug in ULE if this is for any reason other than a tradeoff which can > >> be tweaked to line them up. Or more to the point, there should not be > >> any legitimate reason to use 4BSD these days and modulo the bugs > >> below, you are probably losing on performance for doing so. > >> > >> Bugs reported in this thread by others and confirmed by me: > >> 1. failure to load-balance when having n CPUs and n + 1 workers -- the > >> excess one stays on one the same CPU thread continuously penalizing > >> the same victim. as a result total real time to execute a finite > >> computation is longer than in the case of 4BSD > >> 2. unfairness of nice -n 20 threads vs threads going frequently off > >> CPU (e.g., due to I/O) -- after using only a fraction of the slice the > >> victim has to wait for the cpu hog to use up its entire slice, rinse > >> and repeat. This extends a 7+ minute buildkernel to over 67 minutes, > >> not an issue on 4BSD > >> > >> I did not put almost any effort into investigating no 1. There is code > >> which is supposed to rebalance load across CPUs, someone(tm) will have > >> to sit through it -- for all I know the fix is trivial. > >> > >> Fixing number 2 makes *another* bug more acute and it complicates the > >> whole ordeal. > >> > >> Thus, bug reported by me: > >> 3. interactivity scoring is bogus -- originally introduced to detect > >> "interactive" behavior by equating being off CPU with waiting for user > >> input. One part of the problem is that it puts *all* non-preempted off > >> CPU time into one bag: a voluntary sleep. This includes suffering from > >> lock contention in the kernel, lock contention in the program itself, > > > > Note that time spent off-CPU on a turnstile is not counted as sleeping > > for the purpose of interactivity scoring, so this observation applies > > only to sx, lockmgr and sleepable rm locks. That's not to say that this > > behaviour is correct, but it doesn't apply to some of the most contended > > locks unless I'm missing something. > > > > page busy (massively contested for fork/exec), pipe_lock and even > not-locks like waitpid(!) A program that spends most of its time blocked in waitpid, like a shell, interactive or not, should indeed have a higher scheduling priority... > >> file I/O and so on, none of which has bearing on how interactive or > >> not the program might happen to be. A bigger part of the problem is > >> that at least today, the graphical programs don't even act this way to > >> begin with -- they stay on CPU *a lot*. > > > > I think this statement deserves more nuance. I was on a video call just > > now and firefox was consuming about the equivalent of 20-30% of a CPU > > across all threads. What kind of graphical programs are you talking > > about specifically? > > > > you don't consider 20-30% a lot? I would expect a program consuming 20-30% of a CPU to be prioritized higher than a CPU hog. And in my experience, running builds while on a call doesn't hurt anything (usually). Again, there is room for improvement, I don't claim the scheduler is perfect. > >> I asked people to provide me with the output of: dtrace -n > >> 'sched:::on-cpu { @[execname] = lquantize(curthread->td_priority, 0, > >> 224, 1); }' from their laptops/desktops. > >> > >> One finding is that most people (at least those who reported) use > >> firefox. > >> > >> Another finding is that the browser is above the threshold which would > >> be considered "interactive" for vast majority of the time in all > >> reported cases. > > > > That is not true of the output that I sent. There, most of the firefox > > thread samples are in the interactive range [88-135]. Some show an even > > higher priority, maybe due to priority propagation. > > > > That's not the interactive range. 88 is PRI_MIN_BATCH 88 is PRI_MIN_TIMESHARE (on main, stable/13 ranges are different I think). PRI_MIN_BATCH is PRI_MIN_TIMESHARE + PRI_INTERACT_RANGE = 88 + 48 = 136. Everything in [88-135] goes into the realtime queue. > >> I booted a 2 thread vm with xfce and decided to click around. Spawned > >> firefox, opened a file manager (Thunar) and from there I opened a > >> movie to play with mpv. As root I spawned make -j 2 buildkernel. it > >> was not particularly good :) > >> > >> I found that mpv spawns a bunch of threads, most notably 2 distinct > >> threads for audio and video output. The one for video got a priority > >> of 175, while the rest had either 88 or 89 -- the lowest for > >> timesharing not considered interactive [note lower is considered > >> better]. > > > > Presumably all of the video decoding was done in software, since you're > > running in a VM? On my desktop, mpv does not consume much CPU and is > > entirely interactive. Your test suggests that you expect ULE to > > prioritize a CPU hog, which doesn't seem realistic absent some > > scheduling hints from the user or the program itself. Problem 2 is the > > opposite problem: timesharing CPU hogs are allowed to starve other > > timesharing threads. > > > > Now that I pointed out anything >= 88 is *NOT* interactive, are you > sure your mpv was considered interactive anyway? Yes. > I don't expect ULE to prioritize CPU hogs. I'm pointing out how a hog > which was a part of an interactive program got shafted, further > showing how the method based on counting off cpu time does not work. You're saying that interactivity scoring should take into account overall process behaviour instead of just thread behviour? Sure, that could be reasonable. > >> At the same time the file manager who was left in the background kept > >> doing evil syscall usage, which as a result bouncing between a regular > >> timesharing priority and one which made it "interactive", even though > >> the program was not touched for minutes. > >> > >> Or to put it differently, the scheduler failed to recognize that mpv > >> is the program to prioritize, all while thinking the background time > >> waster is the thing to look after (so to speak). > >> > >> This brings us to fixing problem 2: currently, due to the existence of > >> said problem, the interactivity scoring woes are less acute -- the > >> venerable make -j example is struggling to get CPU time, as a result > >> messing with real interactive programs to a lesser extent. If that > >> gets fixed, we are in a different boat altogether. > >> > >> I don't see a clean solution. > >> > >> Right now I'm toying with the idea of either: > >> 1. having programs explicitly tell the kernel they are interactive > > > > I don't see how this can work. It's not just traditional "interactive" > > programs that benefit from this scoring, it applies also to network > > servers and other programs which spend most of their time sleeping but > > want to handle requests with low latency. > > > > Such an interface would also let any program request soft realtime > > scheduling without giving up the ability to monopolize CPU time, which > > goes against ULE's fairness goals. > > > > Clearly it would be gated with some permission, so only available on a > desktop for example. > > Then again see my response else in the thread: x server could be > patched to mark threads. To do what? > And it does not go against any fairness goals -- it very much can be > achieved, but one has information who can be put off cpu for a longer > time without introducing issues. > > >> 2. adding a scheduler hook to /dev/dsp -- the observation is that if a > >> program is producing sound it probably should get some cpu time in a > >> timely manner. this would cover audio/video players and web browsers, > > > > On my system at least firefox doesn't open /dev/dsp, it sends audio > > streams to pulseaudio. > > > > I think I noted elsewhere in the thread that pulseaudio may need the > same treatment as the x server. > > >> but would not cover other programs (say a pdf reader). it may be it is > >> good enough though > > > > I think some more thorough analysis, using tools like schedgraph or > > KUtrace[1], is needed to characterize the problems you are reporting > > with interactivity scoring. It's also not clear how any of this would > > address the problem that started this thread, wherein two competing > > timesharing (i.e., non-interactive) workloads get uneven amounts of CPU > > time. > > > > I explicitly stated I have not looked into this bit. > > > There is absolutely room for improvement in ULE's scheduling decisions. > > It seems to be common practice to tune various ULE parameters to get > > better interactive performance, but in general I see no analysis > > explaining /why/ exactly they help and what goes wrong with the default > > parameter values in specific workloads. schedgraph is a very useful > > tool for this sort of thing. > > > > I tried schedgraph in the past to look at buildkernel and found it > does not cope with the amount of threads, at least on my laptop. > > > Such tools also required to rule out bugs in ULE itself, when looking at > > abnormal scheduling behaviour. Last year some scheduler races[2] were > > fixed that apparently hurt system performance on EPYC quite a bit. I > > was told privately that applying those patches to 13.1 improved IPSec > > throughput by ~25% on EPYC, and I wouldn't be surprised if there are > > more improvements to be had which don't involve modifying core > > heuristics of the scheduler. Either way this requires deeper analysis > > of ULE's micro-level behaviour; I don't think "interactivity scoring is > > bogus" is a useful starting point. > > > > I provided explicit examples how it marked a background thread as > interactive, while the real hard worker (if you will) as not > interactive, because said worker was not acting the way ULE expects. > > A bandaid for the time being will stop shafting processes giving up > their time slice early in the batch queue, along with some fairness > for the rest who does not (like firefox). I'll hack it up for testing. > > -- > Mateusz Guzik