1.. include:: ../disclaimer-ita.rst 2 3:Original: :ref:`Documentation/process/7.AdvancedTopics.rst <development_advancedtopics>` 4:Translator: Federico Vaga <federico.vaga@vaga.pv.it> 5 6.. _it_development_advancedtopics: 7 8Argomenti avanzati 9================== 10 11A questo punto, si spera, dovreste avere un'idea su come funziona il processo 12di sviluppo. Ma rimane comunque molto da imparare! Questo capitolo copre 13alcuni argomenti che potrebbero essere utili per gli sviluppatori che stanno 14per diventare parte integrante del processo di sviluppo del kernel. 15 16Gestire le modifiche con git 17----------------------------- 18 19L'uso di un sistema distribuito per il controllo delle versioni del kernel 20ebbe iniziò nel 2002 quando Linux iniziò a provare il programma proprietario 21BitKeeper. Nonostante l'uso di BitKeeper fosse opinabile, di certo il suo 22approccio alla gestione dei sorgenti non lo era. Un sistema distribuito per 23il controllo delle versioni accelerò immediatamente lo sviluppo del kernel. 24Oggigiorno, ci sono diverse alternative libere a BitKeeper. Per il meglio o il 25peggio, il progetto del kernel ha deciso di usare git per gestire i sorgenti. 26 27Gestire le modifiche con git può rendere la vita dello sviluppatore molto 28più facile, specialmente quando il volume delle modifiche cresce. 29Git ha anche i suoi lati taglienti che possono essere pericolosi; è uno 30strumento giovane e potente che è ancora in fase di civilizzazione da parte 31dei suoi sviluppatori. Questo documento non ha lo scopo di insegnare l'uso 32di git ai suoi lettori; ci sarebbe materiale a sufficienza per un lungo 33documento al riguardo. Invece, qui ci concentriamo in particolare su come 34git è parte del processo di sviluppo del kernel. Gli sviluppatori che 35desiderassero diventare agili con git troveranno più informazioni ai 36seguenti indirizzi: 37 38 https://git-scm.com/ 39 40 https://www.kernel.org/pub/software/scm/git/docs/user-manual.html 41 42e su varie guide che potrete trovare su internet. 43 44La prima cosa da fare prima di usarlo per produrre patch che saranno 45disponibili ad altri, è quella di leggere i siti qui sopra e di acquisire una 46base solida su come funziona git. Uno sviluppatore che sappia usare git 47dovrebbe essere capace di ottenere una copia del repositorio principale, 48esplorare la storia della revisione, registrare le modifiche, usare i rami, 49eccetera. Una certa comprensione degli strumenti git per riscrivere la storia 50(come ``rebase``) è altrettanto utile. Git ha i propri concetti e la propria 51terminologia; un nuovo utente dovrebbe conoscere *refs*, *remote branch*, 52*index*, *fast-forward merge*, *push* e *pull*, *detached head*, eccetera. 53Il tutto potrebbe essere un po' intimidatorio visto da fuori, ma con un po' 54di studio i concetti non saranno così difficili da capire. 55 56Utilizzare git per produrre patch da sottomettere via email può essere 57un buon esercizio da fare mentre si sta prendendo confidenza con lo strumento. 58 59Quando sarete in grado di creare rami git che siano guardabili da altri, 60vi servirà, ovviamente, un server dal quale sia possibile attingere le vostre 61modifiche. Se avete un server accessibile da Internet, configurarlo per 62eseguire git-daemon è relativamente semplice . Altrimenti, iniziano a 63svilupparsi piattaforme che offrono spazi pubblici, e gratuiti (Github, 64per esempio). Gli sviluppatori permanenti possono ottenere un account 65su kernel.org, ma non è proprio facile da ottenere; per maggiori informazioni 66consultate la pagina web https://kernel.org/faq/. 67 68In git è normale avere a che fare con tanti rami. Ogni linea di sviluppo 69può essere separata in "rami per argomenti" e gestiti indipendentemente. 70In git i rami sono facilissimi, per cui non c'è motivo per non usarli 71in libertà. In ogni caso, non dovreste sviluppare su alcun ramo dal 72quale altri potrebbero attingere. I rami disponibili pubblicamente dovrebbero 73essere creati con attenzione; integrate patch dai rami di sviluppo 74solo quando sono complete e pronte ad essere consegnate - non prima. 75 76Git offre alcuni strumenti che vi permettono di riscrivere la storia del 77vostro sviluppo. Una modifica errata (diciamo, una che rompe la bisezione, 78oppure che ha un qualche tipo di baco evidente) può essere corretta sul posto 79o fatta sparire completamente dalla storia. Una serie di patch può essere 80riscritta come se fosse stata scritta in cima al ramo principale di oggi, 81anche se ci avete lavorato per mesi. Le modifiche possono essere spostate 82in modo trasparente da un ramo ad un altro. E così via. Un uso giudizioso 83di git per revisionare la storia può aiutare nella creazione di una serie 84di patch pulite e con meno problemi. 85 86Un uso eccessivo può portare ad altri tipi di problemi, tuttavia, oltre 87alla semplice ossessione per la creazione di una storia del progetto che sia 88perfetta. Riscrivere la storia riscriverà le patch contenute in quella 89storia, trasformando un kernel verificato (si spera) in uno da verificare. 90Ma, oltre a questo, gli sviluppatori non possono collaborare se non condividono 91la stessa vista sulla storia del progetto; se riscrivete la storia dalla quale 92altri sviluppatori hanno attinto per i loro repositori, renderete la loro vita 93molto più difficile. Quindi tenete conto di questa semplice regola generale: 94la storia che avete esposto ad altri, generalmente, dovrebbe essere vista come 95immutabile. 96 97Dunque, una volta che il vostro insieme di patch è stato reso disponibile 98pubblicamente non dovrebbe essere più sovrascritto. Git tenterà di imporre 99questa regola, e si rifiuterà di pubblicare nuove patch che non risultino 100essere dirette discendenti di quelle pubblicate in precedenza (in altre parole, 101patch che non condividono la stessa storia). È possibile ignorare questo 102controllo, e ci saranno momenti in cui sarà davvero necessario riscrivere 103un ramo già pubblicato. Un esempio è linux-next dove le patch vengono 104spostate da un ramo all'altro al fine di evitare conflitti. Ma questo tipo 105d'azione dovrebbe essere un'eccezione. Questo è uno dei motivi per cui lo 106sviluppo dovrebbe avvenire in rami privati (che possono essere sovrascritti 107quando lo si ritiene necessario) e reso pubblico solo quando è in uno stato 108avanzato. 109 110Man mano che il ramo principale (o altri rami su cui avete basato le 111modifiche) avanza, diventa allettante l'idea di integrare tutte le patch 112per rimanere sempre aggiornati. Per un ramo privato, il *rebase* può essere 113un modo semplice per rimanere aggiornati, ma questa non è un'opzione nel 114momento in cui il vostro ramo è stato esposto al mondo intero. 115*Merge* occasionali possono essere considerati di buon senso, ma quando 116diventano troppo frequenti confondono inutilmente la storia. La tecnica 117suggerita in questi casi è quella di fare *merge* raramente, e più in generale 118solo nei momenti di rilascio (per esempio gli -rc del ramo principale). 119Se siete nervosi circa alcune patch in particolare, potete sempre fare 120dei *merge* di test in un ramo privato. In queste situazioni git "rerere" 121può essere utile; questo strumento si ricorda come i conflitti di *merge* 122furono risolti in passato cosicché non dovrete fare lo stesso lavoro due volte. 123 124Una delle lamentele più grosse e ricorrenti sull'uso di strumenti come git 125è il grande movimento di patch da un repositorio all'altro che rende 126facile l'integrazione nel ramo principale di modifiche mediocri, il tutto 127sotto il naso dei revisori. Gli sviluppatori del kernel tendono ad essere 128scontenti quando vedono succedere queste cose; preparare un ramo git con 129patch che non hanno ricevuto alcuna revisione o completamente avulse, potrebbe 130influire sulla vostra capacita di proporre, in futuro, l'integrazione dei 131vostri rami. Citando Linus 132 133:: 134 135 Potete inviarmi le vostre patch, ma per far si che io integri una 136 vostra modifica da git, devo sapere che voi sappiate cosa state 137 facendo, e ho bisogno di fidarmi *senza* dover passare tutte 138 le modifiche manualmente una per una. 139 140(https://lwn.net/Articles/224135/). 141 142Per evitare queste situazioni, assicuratevi che tutte le patch in un ramo 143siano strettamente correlate al tema delle modifiche; un ramo "driver fixes" 144non dovrebbe fare modifiche al codice principale per la gestione della memoria. 145E, più importante ancora, non usate un repositorio git per tentare di 146evitare il processo di revisione. Pubblicate un sommario di quello che il 147vostro ramo contiene sulle liste di discussione più opportune, e , quando 148sarà il momento, richiedete che il vostro ramo venga integrato in linux-next. 149 150Se e quando altri inizieranno ad inviarvi patch per essere incluse nel 151vostro repositorio, non dovete dimenticare di revisionarle. Inoltre 152assicuratevi di mantenerne le informazioni di paternità; al riguardo git "am" 153fa del suo meglio, ma potreste dover aggiungere una riga "From:" alla patch 154nel caso in cui sia arrivata per vie traverse. 155 156Quando richiedete l'integrazione, siate certi di fornire tutte le informazioni: 157dov'è il vostro repositorio, quale ramo integrare, e quali cambiamenti si 158otterranno dall'integrazione. Il comando git request-pull può essere d'aiuto; 159preparerà una richiesta nel modo in cui gli altri sviluppatori se l'aspettano, 160e verificherà che vi siate ricordati di pubblicare quelle patch su un 161server pubblico. 162 163Revisionare le patch 164-------------------- 165 166Alcuni lettori potrebbero avere obiezioni sulla presenza di questa sezione 167negli "argomenti avanzati" sulla base che anche gli sviluppatori principianti 168dovrebbero revisionare le patch. É certamente vero che non c'è modo 169migliore di imparare come programmare per il kernel che guardare il codice 170pubblicato dagli altri. In aggiunta, i revisori sono sempre troppo pochi; 171guardando il codice potete apportare un significativo contributo all'intero 172processo. 173 174Revisionare il codice potrebbe risultare intimidatorio, specialmente per i 175nuovi arrivati che potrebbero sentirsi un po' nervosi nel questionare 176il codice - in pubblico - pubblicato da sviluppatori più esperti. Perfino 177il codice scritto dagli sviluppatori più esperti può essere migliorato. 178Forse il suggerimento migliore per i revisori (tutti) è questo: formulate 179i commenti come domande e non come critiche. Chiedere "Come viene rilasciato 180il *lock* in questo percorso?" funziona sempre molto meglio che 181"qui la sincronizzazione è sbagliata". 182 183Diversi sviluppatori revisioneranno il codice con diversi punti di vista. 184Alcuni potrebbero concentrarsi principalmente sullo stile del codice e se 185alcune linee hanno degli spazio bianchi di troppo. Altri si chiederanno 186se accettare una modifica interamente è una cosa positiva per il kernel 187o no. E altri ancora si focalizzeranno sui problemi di sincronizzazione, 188l'uso eccessivo di *stack*, problemi di sicurezza, duplicazione del codice 189in altri contesti, documentazione, effetti negativi sulle prestazioni, cambi 190all'ABI dello spazio utente, eccetera. Qualunque tipo di revisione è ben 191accetta e di valore, se porta ad avere un codice migliore nel kernel. 192