Názor k článku Optimalizace pro Ryzeny s 3D V-Cache i podpora běhu Radeonů na LoongArch míří do jádra od Michal Kubeček - Jinak, ten patch, co tam v předmětném vlákně...

Jinak, ten patch, co tam v předmětném vlákně dotyčný poslal, nelze ani zkompilovat. Tomu teda říkám kvalitní práce.

On přeložit jde, ale jen na little endian architekturách (šlo o to, že jsou tam dvě verze deklarace struktury, jedna pro LE a druhá pro BE, a patch přejmenoval člen struktury jen v LE verzi, v BE zůstalo původní jméno). To je přehlédnutí, které se může stát každému. Horší je, že autor patche je zároveň maintainer bcachefs, který má ve zvyku ignorovat standardní procesy, takže patch neposlal do žádného mailing listu a nepobyl ani pár dní v nějakém veřejném stromu, než ho zařadil do pull requestu na samém konci merge window. (Což je samo o sobě nestandardní u patche, který vlastně neřeší žádný problém.) On sám si toho nevšiml, protože jeho testy běží jen na LE, a nikde jinde se to už před 6.12-rc1 nestihlo ohřát dost dlouho na to, aby to odchytil nějaký automat, který by zkusil aspoň build na nějaké BE architektuře.

A to je podstata celého problému, řeči o soudružkách a vločkách ji jen zamlžují a odvádějí pozornost jinam. Problém je v tom, že Kent Overstreet pro vlastní pohodlí ignoruje procesy a zvyklosti, které jsou nastavené určitým způsobem právě proto, aby se podobné chyby pokud možno objevily včas a minimalizoval se jejich dopad.

Tenhle případ je svým způsobem podobný. Michal Hocko rozhodně není nějaká sněhová vločka, ale zrovna tak nemá zapotřebí nechat se urážet jen proto, že si dovolí Kenta Overstreeta upozornit, že GFP_NOFAIL má určitou sémantiku a určité garance a že v jádře je hromada existujícího kódu, který na to spoléhá a neřeší chybu, která z definice nemůže nastat. A že není v pořádku tohle všechno rozbít jen proto, že se to Kentovi nehodí.