Linus Torvalds figyelmeztetése a RISC-V patch-ek kapcsán: a késés és a nem releváns kód nem elfogadható

Segítséget kaptál? Szívesen töltöd itt az idődet? Visszajársz hozzánk? Támogasd a munkákat: Ko-fi és Paypal!

kami911 képe

Linus Torvalds a 6.17-es Linux kernel merge window végén határozottan reagált a RISC-V patch-ek késedelmes beküldésére és a nem releváns kód hozzáadására. A RISC-V fejlesztő, Palmer Dabbelt, későn küldte be a patch-eket a 6.17-es merge window végére. Linus kiemelte, hogy a késői pull request nem csak szabályellenes, de szemétkódot (garbage) is tartalmaz, ami nem RISC-V-specifikus és a generikus header fájlokat szennyezi.

Külön kiemelte a make_u32_from_two_u16() segédfüggvényt, amely szerinte értelmetlen és a kód olvashatóságát rontja:

„Ha kódot írsz (a << 16) + b formában, pontosan tudod, mi történik, és melyik a magas szó. A make_u32_from_two_u16(a,b) használatával viszont nem tudod, melyik a sorrend. Ezzel a kódot csak rosszabbá tetted.”

Linus hangsúlyozta, hogy nem szabad generikus header fájlokban ilyen „helper” függvényeket elhelyezni, és a merge ablak utolsó napján beküldött nagy patch-ek nem elfogadhatóak.

Torvalds üzenete a fejlesztőnek

Linus világosan jelezte:

  • Nincs több késői pull request.
  • Nem releváns kód nem kerülhet a RISC-V tree-n kívüli részekbe.
  • A jövőbeli patch-eket korán kell beküldeni a 6.18-as merge window-ban.

Palmer Dabbelt reakciója

A fejlesztő elismerte a hibát és ígéretet tett:

„Oké, sajnálom. Mostantól nem küldök késői patch-eket, és remélhetőleg ez javítja a minőséget.”

Ez az eset rávilágít a kernel-fejlesztés szabályaira és minőségi elvárásaira: a kódnak nem csak működőképesnek, hanem érthetőnek és relevánsnak kell lennie, a határidők betartása pedig elengedhetetlen a kernel stabil fejlődése érdekében.