Re: [dev] [PATCH 2/2] slstatus: determine the number of cpu's in runtime

From: Aaron Marcher <>
Date: Sat, 16 Sep 2017 13:45:26 +0200


> Why do we need to get the number of cpus in runtime? It's a static
> value, which in worst case changes after a reboot when disabling
> hyperthreading.
> Also, using ncpu for more than one thing (first it's flag, then it's a
> number) is a really bad idea.
> I don't think this should be merged, neither the whole 100% vs
> per-core thing: I think that it gives little to no information, and it
> doesn't seems to play nice with multi-threaded CPUs (in a 2:2 setup,
> does having over 200% means that the cpu is over-the-top? Maybe it's
> the same core, so it can handle more load, but maybe not, so, what
> info did it gave to me?)

I totally agree to that. I will neither merge this patch, nor the 100%
thing. Thank you very much for your work, Kurt, but I think that these
two patches do not fit well into slstatus.


Web: or http://drkhsh5rv6pnahas.onion/
Gopher: gopher:// or gopher://drkhsh5rv6pnahas.onion
GPG: 0x09e71697435bf54b
Fingerprint: 57D2 5F2C 9402 A6BD FEF9 B3B6 09E7 1697 435B F54B
Received on Sat Sep 16 2017 - 13:45:26 CEST

This archive was generated by hypermail 2.3.0 : Sat Sep 16 2017 - 13:48:43 CEST