Linus Torvalds határozottan elutasította a RISC-V alrendszer első, a 6.17-es kernel összevonási időszakára szánt pull requestjét, mivel az késve érkezett és nem megfelelő a beküldött kód minősége. Torvalds korábban jelezte, hogy utazása miatt a kéréseket a szokásosnál korábban szeretné megkapni, így a határidő figyelmen kívül hagyása önmagában is problémásnak számított. A benyújtott változtatások között olyan elemek is szerepeltek, amelyek nem kifejezetten RISC-V specifikusak, hanem a kernel általános fejlécfájljaiba kerültek, ami különösen kifogásolható volt.
Tartalomjegyzék
A vitatott kódrészlet és annak kritikája
A legnagyobb felháborodást a make_u32_from_two_u16() nevű segédfüggvény bevezetése váltotta ki, amely két 16 bites értékből állít elő egy 32 bites számot. Torvalds szerint ez a megoldás nemcsak felesleges, de rontja a kód olvashatóságát is, mivel eltakarja az adattípusok és a bájtsorrend egyértelmű megjelenítését. Hagyományos biteltolással, például „(a << 16) + b” formában ugyanis azonnal látszik, melyik a magasabb rendű szó, és szükség esetén típuskonverzióval biztosítható a helyes működés. Torvalds hangsúlyozta, hogy az ilyen „segédek” csak összezavarják a fejlesztőket, és nem helyezhetők el generikus fejlécfájlokban, különösen nem az összevonási időszak végén.
Torvalds egyértelművé tette a szabályokat és a jövőbeli elvárásokat
Torvalds világossá tette, hogy a jövőben nem fogad el késve érkező pull requesteket, és szigorúan elutasít minden olyan változtatást, amely nem illeszkedik szorosan az adott architektúra kódjához. Bár reméli, hogy a RISC-V-specifikus részek nem tartalmaznak hasonló minőségű kódot, figyelmeztette a maintainer csapatot, hogy a közös fejlécek „szennyezése” elfogadhatatlan. A mostani benyújtást elutasította, és a RISC-V fejlesztőknek legközelebb a 6.18-as kernel összevonási időszakának elején lesz lehetőségük újra próbálkozni, kizárólag megfelelő, architektúrához illő változtatásokkal.
