«Це просто сміття!»: Лінус Торвальдс розкритикував код, запропонований інженером Google

Творець та головний розробник ядра Linux Лінус Торвальдс публічно і різко відкинув внесок у код для підтримки архітектури RISC-V у майбутньому релізі Linux 6.17. Патч, надісланий інженером Google Палмером Даббелтом із команди Android, був відхилений під час обговорення у списках розсилки Linux через дві причини: низьку якість і запізніле подання.

Торвальдс наголосив, що цей внесок надійшов під кінець «вікна злиття» для версії 6.17, попри його попереднє прохання надсилати всі pull request’и завчасно, оскільки він перебував у відрядженні.

«Ні. Це сміття, і воно надійшло надто пізно. Якщо ви не можете дотриматися правила надсилати завчасно, хоча б робіть pull request’и якісними», — написав він.

Однією з найбільших претензій стала поява допоміжної функції make_u32_from_two_u16(), яку Торвальдс назвав такою, що «робить кодову базу гіршим місцем для життя». На його думку, ця функція приховує порядок обробки даних і вносить зайву складність у загальні заголовні файли, виходячи далеко за межі коду, специфічного для RISC-V.

Він підкреслив, що подібні зміни «забруднюють» ключові файли ядра, погіршуючи його підтримуваність, а також можуть створити ризики помилок у непов’язаних частинах системи. Тому, на його думку, будь-які внески мають бути максимально локалізованими в межах архітектури, для якої вони призначені, і лише у виняткових випадках зачіпати загальні файли — з чітким обґрунтуванням та завчасною подачею.

Публічна відмова супроводжувалася попередженням:

«Ви на прицілі: жодних запізнілих pull request’ів і жодного сміття поза деревом RISC-V».

Торвальдс також закликав розробника підготувати чистий і своєчасний внесок для версії ядра 6.18, натякнувши, що лише так він матиме шанси на схвалення.

Як вважаєте, реакція Торвальдса у таких випадках виправдана, чи він уже переходить межу й варто було б висловлюватися стриманіше?