lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <Y2J4RFiOeJ5XymlI@e126311.manchester.arm.com> Date: Wed, 2 Nov 2022 14:01:40 +0000 From: Kajetan Puchalski <kajetan.puchalski@....com> To: Doug Smythies <dsmythies@...us.net> Cc: rafael@...nel.org, daniel.lezcano@...aro.org, lukasz.luba@....com, Dietmar.Eggemann@....com, yu.chen.surf@...il.com, linux-pm@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [RFC PATCH v3 2/2] cpuidle: teo: Introduce util-awareness On Mon, Oct 31, 2022 at 11:24:06PM -0700, Doug Smythies wrote: > Ignoring the metrics is not the correct thing to do. > Depending on the workflow, it can severely bias the idle states deeper > than they should be because most of the needed information to select > the appropriate shallow state is tossed out. Interesting, thanks for flagging this actually. Arguably it made sense originally but with v3 changes it probably causes more harm than good. >From testing it on my own side it still looks all right without the skipping metrics part so I'm fine with just dropping it from the patch. V4 it is. I'll include my updated numbers and more tests in the new cover letter. > Example 1: > 2 pairs of ping pongs = 4 threads > Parameters chosen such that idle state 2 would be a most used state. > CPU frequency governor: Schedutil. > CPU frequency scaling driver: intel_cpufreq. > HWP: Disabled > Processor: i5-10600K (6 cores 12 cpus). > Kernel: 6.1-rc3 > Run length: 1e8 cycles > Idle governor: > teo: 11.73 uSecs/loop ; idle state 1 ~3.5e6 exits/sec > menu: 12.1 uSecs/loop ; idle state 1 ~3.3e6 exits/sec > util-v3: 15.2 uSecs/loop ; idle state 1 ~200 exits/sec > util-v4: 11.63 uSecs/loop ; idle state 1 ~3.5e6 exits/sec > > Where util-v4 is the same as this patch (util-v3) with the above code reverted. > > Note: less time per loop is better. > > Example 2: Same but parameters selected such that idle state 0 would > be a most used idle state. > Run Length: 4e8 cycles > Idle governor: > teo: 3.1 uSecs/loop ; idle state 0 ~1.2e6 exits/sec > menu: 3.1 uSecs/loop ; idle state 0 ~1.3e6 exits/sec > util-v3: 5.1 uSecs/loop ; idle state 0 ~4 exits/sec > util-v4: ? uSecs/loop ; idle state 0 ~1.2e6 exits/sec (partial result) Thanks a lot for the testing! --- Kajetan
Powered by blists - more mailing lists