Durante la discussione su un altro insieme di correzioni proposte per l'inclusione nel kernel 6.11-rc5 da parte dell'autore di Bcachefs, Linus Torvalds ha scritto di iniziare a rimpiangere di aver accolto il filesystem Bcachefs nel kernel. Il malcontento di Torvalds è legato al fatto che Kent Overstreet, l'autore di Bcachefs, durante la fase dei candidati finali, invia correzioni troppo ampie che, in sostanza, non solo risolvono errori, ma continuano anche lo sviluppo delle funzionalità, nonostante le modifiche funzionali siano ammesse solo nelle fasi iniziali dello sviluppo di un nuovo ramo; nella fase rc5 sono accettate solo piccole correzioni singole, la cui dimensione non dovrebbe idealmente superare le 100 righe di codice.
Il patch inviato per Bcachefs, che ha aggiunto 1309 righe e rimosso 671 righe in 39 file, oltre a correggere errori, includeva due modifiche significative: supporto per una nuova struttura dati per gestire l'elenco degli elementi liberi nella cache e la possibilità di convertire le tabelle hash per la cache. La nuova struttura è stata aggiunta per eliminare blocchi di cache non necessari ed escludere situazioni che potrebbero portare a conflitti di blocco sotto carichi multithreading.
Non è la prima volta che grandi correzioni per Bcachefs vengono inviate dopo la finestra di accettazione delle modifiche, e non è la prima volta che invece di semplici correzioni di errori vengono promosse modifiche funzionali più pesanti, presentate tramite richieste di modifica nelle fasi finali dello sviluppo di un nuovo ramo. Linus Torvalds ha risposto alla pull request inviata:
«Basta. L'ultima pull request è stata grande. Anche questa pull request è troppo voluminoso, tocca cose non correlate a Bcachefs e non è nemmeno lontanamente una regressione. A un certo punto la «correzione di qualcosa» si trasforma semplicemente in sviluppo, e questo è quel momento. Nessuno in sano giudizio usa Bcachefs e si aspetta stabilità da esso, perciò ogni utente è un campo di esperimenti. Le patch di Bcachefs sono diventate una sorta di «grande sviluppo durante il ciclo di preparazione del rilascio, e non prima», al punto che inizio a rimanere deluso per aver accettato Bcachefs. Se Bcachefs non può funzionare normalmente secondo il programma di sviluppo del kernel upstream, forse non dovrebbe far parte del kernel principale. Questo supera già i limiti del ridicolo».
Kent Overstreet ha risposto che Bcachefs è decisamente più affidabile di Btrfs e sta lavorando per farla diventare più resistente e affidabile di XFS ed EXT4, grazie all'implementazione di un meccanismo di integrità dei dati end-to-end. Kent ha anche sottolineato che le modifiche proposte riguardano direttamente il miglioramento dell'affidabilità e che, se un errore in altri sottosistemi del kernel potrebbe limitarsi a un crash e un riavvio, negli filesystem il costo di un errore è molto alto e può portare a danni dati nascosti che potrebbero essere notati solo settimane dopo.
Torvalds ha obiettato che finora non ci sono grandi distribuzioni Linux che utilizzano Bcachefs e ha sottolineato che è necessario seguire le regole di sviluppo delle versioni del kernel Linux, poiché c'è il rischio di introdurre nuovi errori a causa di patch di questo tipo, contenenti più di 1000 righe di codice. Al momento, le correzioni discusse per Bcachefs non sono state accettate nel kernel Linux e lo sviluppatore di Bcachefs non ha ancora preparato una versione aggiornata della pull request, priva di nuove funzionalità e concentrata solo sulla correzione di errori.
Fonte: opennet.ru
