Linus keményen kiosztotta az ex-Google-most-Meta-mérnököt: a kódod egy szemét ...

Kernel

A napokban Linus egy ex-Google-most-Meta-mérnökkel közölte, hogy kódja "szemét" és "rosszabb hellyé teszi a világot". Ez az eset is jól példázza Linus pull request-eket érintő kemény, de következetes hozzáállását, ami ugyan egyeseknek nem tetszik, de tény, hogy ezzel a szemlélettel évtizedek óta sikerrel egyengeti a világ legnagyobb FLOSS projektjét, ami nélkül az IT világ ma már kb. működésképtelen lenne.

( bennyh | 2025. 08. 14., cs – 09:09 )

Szerkesztve: 2025. 08. 14., cs – 09:11

Akkor, ha jól értem az ürge csinált egy felesleges függvényt egy sornyi kódnak, aminek a neve jobban elviszi a sűrűbe a többi fejlesztőt, mintha csak használta volna az egysoros bal shift + összeadást, amiről lehet is látni, hogy micsoda?

Ha ilonka valami jót is akarna adni a világnak AI fronton, inkább a Linus tudását pakolnák bele a grokba :D

Színes vászon, színes vászon, fúj!

Kérem a Fiátot..

( kikadff v | 2025. 08. 14., cs – 09:29 )

Fura nekem Linus. Kiosztja az nvidiat, bcaches fejlesztőt, stb, kiosztja ezt is, de mi változik? Annyira nem látok bele a linux fejlődésébe, de van bármi eredménye ezeknek a Linus kitöréseknek, vagy már mindenki beárazta, hogy Linus ilyen, amúgy megy minden a maga útján? Valaki linkelte másik topikban, hogy a requestek kicsiny százaléka megy át Linuson, mindenki tudja, hogy ha hozzá kerül és olyanja van épp, akkor majd lesz egy kis műsor?

Régebben még itt is hír volt, hogy el volt ájulva valami új kódon, hogy milyen kevés sorban oldja meg a régi kódhoz képest a fejlesztő, aztán meg mások szóltak, hogy kilométeres sorok vannak benne, meg valami feature szegénység is talán.

Nem tudom hová tenni Linust, erős kezű toppon lévő vezető, vagy csak egy brand.

Ha azt gondolja, hogy szemét a kód, akkor valószínűleg az is.

De a make_u32_from_two_u16-ból nagyon úgy tűnik, hogy igaza van az (a << 16) + b -vel és gondolom azon csesződött fel, hogy meg kellene ütni egy szintet. 

Abban is igaza van, ha rutinokat nem a megfelelő "általános"  header-be tettek. Plusz a szósorrendet, ami nem lesz mindenkinek megfelelő, elfedi a rutin (a nevét látod a rutinnak).

Gondolom nem az első ilyen a kernel történetében, és a linux most nem ilyen lenne, ha minden ilyet beengedne. Meg én elhiszem, hogy frusztrált emiatt, próbál tartani egy szintet, és akkor jönnek ilyenek.

Igen, azért általában (vagy mindig) van oka a kifakadásra, csak mi fog változni? Jó, gondolom ha megfogja ezeket a vitatható (szar) kódokat, akkor a megszólított fejlesztő változtat rajta, de a nagy képet nézve mi változik? Nem akarom becsmérelni Linust, méltán írta bele magát a történelem könyvekbe, azért még az ilyen-olyan fanboyoknak is egyértelmű, hogy a világ legelterjedtebb kernele az a linux, mégha nem is uralja az összes szegmenst. Ez hatalmas és elvitathatatlan eredmény Linustól. Viszont pont ezért a linux már óriási project, kizárt hogy egy ember toppon legyen az egészben, rengeteg kód kerül bele, amit Linus nem látott (#FIXME), szóval mi jelentősége van ezeknek a Linus kitöréseknek? Vagy van mástól is ilyen csak ők nem Linus és nincs visszhangja?

Évről évre olvasom a Linus kiosztotta ezt, kiosztotta azt híreket, már az a benyomásom, hogy Linus így működik alapjáraton és ez be van árazva. A fejlesztő is tudja, hogy majd előbb-utóbb eljön az idő, amikor Linus kiosztja, ő majd az aktuális szarságon módosít, aztán megy minden tovább.

Ha már látott egy ilyet, akkor nem lehet felette szemet hunyni, mert problémákat okoz. Gondolom az pluszban frusztráló lehet, ha nem tud minden kódot megnézni, és közben ilyenek is érkeznek.

Szerintem van értelme "beszólni". Akinek ezt írta, annak most az lenne a helyes hozzáállás, hogy "húú, erre nem is gondoltam, és igaza van", és beküldeni normálisan, legközelebb pedig figyelembe venni ezeket a szempontokat is. Ahelyett, hogy azt elemezgetni, hogy miért így kellett válaszolni. Tehát gyakorlatilag próbál átadni egy szemléletet annak, aki sz*r kódot küldött be. (És Linus elég jó kóder lehet, jó kis szemlélettel ami a kernelhez kell, mert jó-pár éve csinálja. Érdemes lehet hallgatni rá.).

Amúgy felmerül a kérdés, hogy jutott ez el Linusig? Ugye a normális flow szerint az egyéni fejlesztők patchei előbb a subsystem maintainerek reviewján mennek keresztül, ők signoff-olják és úgy megy tovább Linus-hoz végső approve-ra.

Van egy gyanúm, hogy a csávó a késve beküldés miatt (hogy mégis beférjen) megpróbálta megkerülni az alsóbb review köröket és direktbe küldte Linushoz.

Régóta vágyok én, az androidok mezonkincsére már!

Szerintem olyan , amilyennek lennie kell. Nem az embert nevezte szemétnek, hanem a munkáját. Aki ezt nem bírja, annak nem való ez a szakma. 
 

Ha tartós rendszert építesz és okos csapatot nevelsz, akkor száz kiadásban sem érheti baj; ha csak a gépekre hagyatkozol, akkor egyszer jól jársz, máskor rosszul; de ha sem a rendszer nem bírja a terhet, sem a csapat nem tanul a hibákból, akkor minden egyes kiadás kockázat.

Melyik alapelvet szegte ezzel meg? 

Ha tartós rendszert építesz és okos csapatot nevelsz, akkor száz kiadásban sem érheti baj; ha csak a gépekre hagyatkozol, akkor egyszer jól jársz, máskor rosszul; de ha sem a rendszer nem bírja a terhet, sem a csapat nem tanul a hibákból, akkor minden egyes kiadás kockázat.

Nem tudna. Ugyanis nem a személyt kritizálta, hanem a kódot - a CoC pedig a személyeket védi, nem a szarul elvégzett munkát. 

Ha tartós rendszert építesz és okos csapatot nevelsz, akkor száz kiadásban sem érheti baj; ha csak a gépekre hagyatkozol, akkor egyszer jól jársz, máskor rosszul; de ha sem a rendszer nem bírja a terhet, sem a csapat nem tanul a hibákból, akkor minden egyes kiadás kockázat.

Mivel nem ez az első "szemét a kódod" kirohanása, talán a CoC-ba be lehetne tenni erre is egy "szabvány" szöveget, ami határozott, de kevésbé élesen fogalmazott.
Aztán ha valakinek azt a szöveget küldené el, arról mindenki tudná, hogy Szemét a kódod haver! :)
De az ilyen vitákat és fölösleges szócséplést el lehetne kerülni vele.

Tertilla; Tisztelem a botladozó embert és nem rokonszenvezem a tökéletessel! Hagyd már abba!; DropBox

  • Other conduct which could reasonably be considered inappropriate in a professional setting

Mondasz te a kollégádnak egy széles körben tartott meeting-en a munkahelyeden, hogy "tesó, a munkád egy szemét" és a "világot sokkal élhetetlenebb hellyé teszi"?

Ha igen, akkor ez teljesen rendben van a CoC szerint. Pont ez a probléma az ilyen erőltetett CoC szemetekkel egyébként. Gumiszabályok.

trey @ gépház

Alap esetben nem mondok, de simán elmondom négyszemközt valamire (ott is igyekszem nem nyersen, amennyire lehet). - Lehet, hogy Linus sem rögtön egy ilyen levéllel kezdi egy új fejlesztőnek, de erről ugye pont ezért nincs hír.

A levél gondolom azért sikerült nyersre, mert nem csak az adott esetre vonatkozik, hanem az összes múltbeli, és jövőbeli egyébként 1. elkésett pull requestekre, ami 2. szar kódot tartalmaz, ami 3. később még káros is, félreviszi a többi fejlesztőt. Tehát másnak is szólt, ha széles körben küldte szét.

Szerintem érhető Linus lelki világa is. Ezekre Linusnak kell odafigyelni, ha elcseszi abból probléma lesz. Arra van valami CoC pont, hogy normális munkát adunk ki a kezünkből a társunknak (nem szívatjuk meg?) ? Tehát hogy ne csak a commitoló lelki világát kelljen figyelembe venni.

Mondjuk itt vannak ezek:

https://www.kernel.org/doc/html/latest/process/submitting-patches.html

https://www.kernel.org/doc/html/latest/process/coding-style.html

https://docs.kernel.org/process/development-process.html

Ha a CoC-ra hivatkozna a fejlesztő, akkor ezeket is át kellene nézni.

( joco01 v | 2025. 08. 14., cs – 11:45 )

Szép hatásvadász címadás, de nem Google mérnök.

https://www.linkedin.com/in/palmerdabbelt/ szerint már egy ideje Meta. Pár éve volt Google-nél, de most tuti nincs, és biztos elfelejtette frissíteni a honlapját, amiben semmi meglepő nincs.

Amúgy ha Linus kioszt egy akármilyen random mérnököt, annak már nincs hírértéke, csak akkor, ha mellé lehet tenni valami plusz szaftos infót?

Egyébként tudom trey hogy te csak átvetted a címet, mint sokan mások, de szerintem minden újságíró saját felelőssége, hogy ellenőrzés nélkül ne tegyen ki fals infót a saját neve alatt.

https://www.dabbelt.com/~palmer/

Frissítse a resumé-ját. Blogger vagyok, nem oknyomozó újságíró. 

PS: az, ha az szerepelne a címben, hogy kiosztotta a Meta-mérnököt semmit sem vesz le a téma ínyencségéből. Az meg, hogy a RISC-V Foundation elnökét tették helyre egy azzal kapcsolatos patch nyomán, ... még kellemetlenebb lenne az úr számára.

trey @ gépház

( Ice | 2025. 08. 14., cs – 15:28 )

Már hiányoztak ezek a "rendberakások". :) Végre van újabb. :) 100%-ig igaza szokott lenni. Ha nincs rend, nagy káosz lesz az tuti!