Dopo aver lavorato con Linux (per lo più come un utente avanzato) per anni, ho deciso di postare la mia prima patch per il kernel Linux (anche se banale, ma hey è necessario iniziare da qualche parte!) e sono rimasto basito quando in primo luogo ha ottenuto accettata dal manutentore sottosistema e poi in ultima analisi, scorreva nel kernel mainline.

Quello che segue è una guida passo-passo su come presentare un kernel Linux di patch e spero che venga accettato nel kernel mainline. Questo HOW-TO si basa su una patch reale che ho scritto e presentato e che stato accettato nel kernel mainline ieri. La guida mette in evidenza soprattutto la meccanica di invio di patch e non necessariamente gli aspetti tecnici di progettazione del kernel Linux.

Passo 1: Installare Git Tools

La prima cosa che dobbiamo fare è assicurarsi di avere gli strumenti necessari per creare e presentare la nostra patch per il kernel Linux. Sto usando il mio sistema Debian Lenny per questo HOW-TO.

# apt-get update
# apt-get install git git-email gitk

Inoltre è una buona idea per configurare alcuni pochi parametri che Git utilizzerà quando si genera il cerotto:

# git config --add user.name "Kushal Koolwal"
# git config --add user.email "kushalkoolwal@gmail.com"

Fase 2: Clone Linus 'Git Albero

La prossima cosa che si potrebbe voler fare è quello di clonare Git albero di Linus preferibilmente sulla vostra macchina Linux. Assicurarsi di avere spazio sufficiente (almeno 2 GB) prima di clonare l'albero. Al momento della stesura di questo la dimensione totale della struttura è stata circa 950 + MB.

debian:~# cd /usr/src/
debian:/usr/src/# git clone git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6 mylinux-2.6

Ora aspetta per 30 minuti o giù di lì, a seconda della velocità di connessione a Internet, prima che l'intero albero viene scaricato nella directory tua mylinux-2.6 (directory verrà creata automaticamente in quanto è stato specificato nel comando). Qualsiasi patchwork dovrebbe sempre essere fatto contro questo albero per fare in modo che le modifiche potrebbero non entrare in conflitto con le modifiche apportate da altri sviluppatori.

debian:/usr/src/# cd mylinux-2.6
debian:/usr/src/mylinux-2.6# ls
arch COPYING crypto drivers fs init Kbuild kernel MAINTAINERS mm README samples security tools virt block CREDITS Documentation firmware include ipc Kconfig lib Makefile net REPORTING-BUGS scripts sound usr
debian:/usr/src/mylinux-2.6#

Da qui in poi tutto il lavoro sarà fatto nella directory / usr/src/mylinux-2.6.

Passo 3: Ora create una filiale locale per apportare le modifiche

Per impostazione predefinita, quando si clona l'albero Git siete nel ramo master.

# git branch
* master
#

Il simbolo * indica sopra la vostra filiale corrente.

Creiamo una nuova filiale miei MenlowRebootFix in cui faremo le nostre modifiche al codice:

# git branch MenlowRebootFix

Allora avete bisogno di cassa sopra ramo appena creato:

# git checkout MenlowRebootFix

Confermare che siete nel ramo sopra:

# git branch
*MenlowRebootFix
master
#

Passo 4: Apportare le modifiche

Ora sto andando a fare modifiche al file arch/x86/kernel/reboot.c. Dopo aver effettuato le modifiche al file e salvato dare il seguente comando:

# git commit -a

Il comando di cui sopra si aprirà un editor di testo nella stessa finestra sullo schermo del terminale (il mio editor di testo predefinito è nano) come questo:

# Please enter the commit message for your changes. Lines starting
# with '#' will be ignored, and an empty message aborts the commit.
# On branch Test1
# Changes to be committed:
# (use "git reset HEAD <file>..." to unstage)
#
# modified: arch/x86/kernel/reboot.c
#

Ora dobbiamo entrare nel nostro messaggio di commit log. State molto attenti a ciò che si digita nel messaggio di log perché questi messaggi entreranno a far parte della Linux Git albero commettere log e la gente sarà cercare il vostro impegno in base a questi messaggi. Ecco cosa ho scritto e qui è quello che la mia finestra dell'editor finale sembrava che:

x86: Fix reboot problem on VersaLogic Menlow boards

VersaLogic Menlow based boards hang on reboot unless reboot=bios is used. Add quirk to reboot through the BIOS.

Tested on at least four boards.
# Please enter the commit message for your changes. Lines starting
# with '#' will be ignored, and an empty message aborts the commit.
# On branch Test1
# Changes to be committed:
# (use "git reset HEAD <file>..." to unstage)
#
# modified: arch/x86/kernel/reboot.c
#

Ora il testo in colore blu diventerà la vostra linea soggetto principale della patch che verrà posta elettronica e diventerà anche l'identificatore del Git commit log.

Nota: E 'molto importante che prima di digitare il nome del sottosistema a cui il cerotto appartiene. Nel mio caso, dal momento che sto modificando il file reboot.c nella directory di architettura x86, comincio dalla linea soggetto con x86:.

Dopo di che è necessario lasciare una riga vuota. e quindi digitare una breve descrizione del tuo cambiamento (di colore marrone), che diventerà il changelog del Git commit log.

Non preoccuparti per il testo di colore arancione che inizia con #. Coloro che verrà ignorato dal Git commit.

and description , your description will become part of the subject line and it will be all mess. Nota: Se non lasciare una riga vuota tra il soggetto e la descrizione, la tua descrizione diventerà parte della riga dell'oggetto e sarà tutto pasticcio.

Fase 5: Genera il cerotto


Così ora avete commesso il cerotto nel vostro repository Git locale, è il momento di generare la patch che vi e-mail ai rispettivi manutentori e mailing list. Per generare cerotto semplicemente dare il seguente comando:

# git format-patch -s -n master..MenlowRebootFix

Questo dovrebbe creare un file di 0001-x86-Fix-riavvio-problemi-on-VERSALOGIC-Menlow-boards.patch nella directory corrente il cui contenuto sarà:

Da 04056e508c996b023857f8076da7fb54096d10e9 Lun 17 Set 2001 00:00:00
Da: Kushal Koolwal <kushalkoolwal@gmail.com>
Data: Sat, 19 Feb 2011 13:14:03 -0800
Oggetto: [PATCH 1/1] x86: Risolvere problema di riavvio sulle schede VERSALOGIC Menlow.
Schede basate VERSALOGIC Menlow appendere al riavvio a meno che non viene utilizzato reboot = bios.

Aggiungi stranezza di riavviare tramite il BIOS.
Testato su almeno quattro tavole.

Firmato-off-by: Kushal Koolwal <kushalkoolwal@gmail.com>
-
arch/x86/kernel/reboot.c | 8 + + + + + + + +
1 file modificati, 8 inserimenti (+), 0 delezioni (-)
diff-git a/arch/x86/kernel/reboot.cb/arch/x86/kernel/reboot.c
Indice fc7aae1 .. 715037c 100.644
- A/arch/x86/kernel/reboot.c
+ + + B/arch/x86/kernel/reboot.c
@ @ -285,6 +285,14 @ @ Static struct dmi_system_id __ initdata reboot_dmi_table [] = {
DMI_MATCH (DMI_BOARD_NAME, "P4S800"),
},
},
+ {/ * Problemi maniglia con il riavvio sulle schede VERSALOGIC Menlow * /
+. Callback = set_bios_reboot,
+. Ident = "board basata VERSALOGIC Menlow",
+. Partite = {
+ DMI_MATCH (DMI_BOARD_VENDOR, "VERSALOGIC Corporation"),
+ DMI_MATCH (DMI_BOARD_NAME, "board VERSALOGIC Menlow"),
+},
+},
{}
};
-
1.7.2.3

Si noti come il nome del file è stato prelevato dalla prima riga del Git commit log. Anche l'opzione-n, aggiunge il numero di patch [PATCH 1/1] per il soggetto e l'opzione-s aggiunge la Signed-off-by: linea. L'e-mail che viene prelevato in linea firmata off viene prelevato dalla configurazione del Git che nella fase 1.

Fase 6: Controlla la tua patch per errori

Poi abbiamo bisogno di fare in modo che la patch stiamo cercando di presentare non contiene errori evidenti come spazi bianchi, superiore a 80 limite di colonna, ecc C'è uno script perl chiamato checkpath.pl che viene fornito con i sorgenti del kernel per questo scopo :

# scripts/checkpatch.pl 0001-x86-Fix-reboot-problem-on-VersaLogic-Menlow-boards.patch

e si dovrebbe vedere qualcosa di simile a questo:

Uscita:

total: 0 errors, 0 warnings, 14 lines checked
0001-x86-Fix-reboot-problem-on-VersaLogic-Menlow-boards.patch has no obvious style problems and is ready for submission.

Ci sono altri script forniti anche all'interno del sorgente del kernel per riparare / pulire il cerotto come:

scripts/cleanfile
scripts/cleanpatch

Fase 7: prova il cerotto

Anche in questo caso è molto importante che si applica il cerotto per l'albero di Linus 'Git, compilarlo e testarlo prima di inviare la vostra patch upstream.

# git checkout master
# patch -p1 < 0001-x86-Fix-reboot-problem-on-VersaLogic-Menlow-boards.patch
# make-kpkg --append-to-version=test-patch kernel_image kernel_headers

Fase 8: Prendi la lista delle persone a presentare patch

Assumendo che il cerotto compilata, ha lavorato nel modo desiderato per e tue e-mail di prova sembrava buono il passo successivo è quello di ottenere l'elenco delle riguardano persone a cui si deve inviare la vostra patch. Ci sono due metodi per farlo:

Metodo 1: Utilizzare lo script fornito nei sorgenti del kernel

# scripts/get_maintainer.pl 0001-x86-Fix-reboot-problem-on-VersaLogic-Menlow-boards.patch

Uscita:

Thomas Gleixner <tglx@linutronix.de> (maintainer:X86 ARCHITECTURE...)
Ingo Molnar <mingo@redhat.com> (maintainer:X86 ARCHITECTURE...,commit_signer:5/8=62%)
"H. Peter Anvin" <hpa@zytor.com> (maintainer:X86 ARCHITECTURE...,commit_signer:3/8=38%)
x86@kernel.org (maintainer:X86 ARCHITECTURE...)
Don Zickus <dzickus@redhat.com> (commit_signer:2/8=25%)
Peter Zijlstra <apzijlstra@chello.nl> (commit_signer:2/8=25%)
Borislav Petkov <bp@alien8.de> (commit_signer:1/8=12%)
linux-kernel@vger.kernel.org (open list)
#

Metodo 2: Fare riferimento al file MAINTAINERS

L'altro metodo è quello di fare riferimento al MAINTAINERS file che viene fornito all'interno dei sorgenti del kernel. È necessario almeno con la posta elettronica il cerotto a tutte le persone che hanno "M" prima del loro nome.

Nota: Inoltre è necessario inviare la vostra patch per almeno una mailing list. Se non siete in grado trovare qualsiasi mailing list, sulla base dei due metodi di cui sopra, che si occupa con il sottosistema contro il quale si sta tentando di presentare la patch, quindi si dovrebbe almeno mail per Linux Kernel Mailing List (linux-kernel @ vger . kernel.org).

Passo 9: Verifica Email il cerotto

Anche se è possibile utilizzare qualsiasi client di posta elettronica e host SMTP che si desidera e avere accesso a, ho scoperto che l'uso di Gmail è stato il migliore dal momento che un sacco di sviluppatori del kernel usare quella. Ora è molto importante che prima di testare il cerotto inviando i propri indirizzi email (possibile account di posta elettronica diverso con diversi provider di posta elettronica). Io uso il seguente:

git send-email --smtp-encryption=tls --smtp-server=smtp.gmail.com --smtp-user=kushalkoolwal@gmail.com --smtp-server-port=587 --to "KKoolwal <myname@koolsolutions.com>" --to "Kushal Koolwal <myname@yahoo.com>" --cc "KK <myname@hotmail.com>" 0001-x86-Fix-reboot-problem-on-VersaLogic-Menlow-boards.patch

Dopo aver premuto invio, vi chiederà la password dell'account Gmail che dovete entrare per inviare effettivamente l'e-mail.

Ora controllare account di posta elettronica sopra elencati per verificare se hai l'email e tutto sembra a posto per la presentazione.

Fase 10: Finalmente mail il cerotto

. Se tutto sembra a destra e poi finalmente email il cerotto sulla base della lista delle persone che hai trovato nel passaggio 8 Questo è quello che ho effettivamente finito per usare:

# git send-email --smtp-encryption=tls --smtp-server=smtp.gmail.com --smtp-user=kushalkoolwal@gmail.com --smtp-server-port=587 --from "Kushal Koolwal <kushalkoolwal@gmail.com>" --to "Thomas Gleixner <tglx@linutronix.de>" --to "Ingo Molnar <mingo@redhat.com>" --to "H. Peter Anvin <hpa@zytor.com>" --to "x86@kernel.org" --cc "linux-kernel@vger.kernel.org" 0001-x86-Fix-reboot-problem-on-VersaLogic-Menlow-boards.patch

Ecco che commit log di accettazione della mia patch di Ingo Molnar che poi colava da un albero di Ingo Molnar a Stephen Rothwell linux-next albero e infine in di Linus Torvald mainline kernel Linux albero.

Questo è tutto! Buona fortuna con la vostra prima patch!

Essere socievole, trova!