Názory k článku
Oprava regrese, kterou našel Linus Torvalds

Proc zrovna 25%? Je to proto, ze je to zaroven bitovy posun o 2? A nemely by takove "magic constants" byt spis na jednom miste v kodu toho scheduleru?

No pokud ti to moc vadí, tak můžeš přispět ke zlepšení kvality kernelu a napsat do kernel mailing listu :) Tady s tím nikdo nic neudělá...

Vzhledem k tomu, že vývojáři kernelu jsou pragmatici, neudělají s tím pravděpodobně nic ani kdyby zaslal patch. Takovéhle různé hodnoty v optimalizacích jsou prostě často určené heuristicky a dát tam hodnotu, která jde snadno spočítat, je rozumné řešení. A magické konstanty – někdy je podstatně větší magie nějaká složitě pojmenovaná konstanta, jejíž hodnotu musím hledat, než když na první pohled vidím, že se to dělí 4 – a v komentáři nad tím je napsáno proč.

No vzhleldem k tomu ze je to v *cpufreq_sche­dutil.c* na jedinem miste a to v *get_capacity_ref_freq* nedava vubec smysl to vytahovat do nejake "magic constanty"

Vytažení do magic konstanty, nebo dokonce do konfigurace, by mohlo mít smysl v tom, že by to pak někdo mohl ladit a dospět k tomu, že třeba ideální hodnota není 25% ale 31,2%, nebo i různě pro různé typy zátěže. Takhle se tím nikdo zabývat nebude.

Ne, to s laděním vůbec nesouvisí, ladit je to možné už teď (a snáz, než kdyby to bylo někde mimo). Pokud by se ukázalo, že to má být konfigurovatelné, předělá se to.

dopředná optimalizace je také chyba. Aplikovat patch pro ladění výkonu nebo konfigurací je legitimní způsob jak to udělat. Není potřeba na vše dávat parametr a takhle to je správně, konstanta v kódu dokud nedává smysl to dát někam jinám.

Nezapomeň i na takovou drobnost, že cokoliv vystrčíš ven je prostě veřejný interface a jen tak se toho nezbavíš, musíš to udržovat i s případným refaktorem a při jiné implementaci scheduleru to nemusí už dávat smysl. Tohle je prostě interní věc dané implementace, ta má zůstat interní.

První, co mne napadá: má někdo ze zdejších diskutujících vůbec představu, jak často a v jakém kontextu se tahle funkce vlastně volá? Ono násobit 0.312 a posunout o dva bity může být v některých případech zatracený rozdíl.

A pak také bývá dobré podívat se i na reakce na ten patch. A, ejhle, co to nevidím: I think we can do better, but this does re-instate the previous behavior at least for this merge window. So FWIW Reviewed-and-tested-by: Qais Yousef <...>