Ça y est, je viens de contribuer à Linux ! Je ne parle pas du système d'exploitation GNU/Linux, que l'on raccourci souvent en "Linux" chez le grand public (la confusion est volontaire), mais bel et bien du noyau Linux, à savoir la partie qui gère directement le matériel (pour l'exporter aux logiciels sus-jacent). Comme vous le savez, il n'y a pas que les barbus intégristes vivant dans une cave qui ont un noyau Linux, mais bel et bien 95% des utilisateurs de smartphones (grâce à Android). On peut aussi rajouter plus de 90% des supercalculateurs, une bonne 20% des serveurs et quelques pourcent chez les utilisateurs finaux.
Le bug
Le bug corrigé (car oui, c'était un bug), fait partie du sous-système mmc (MultiMediaCard). Il s'agit d'une mémoire de stockage contenue dans les cartes SD, mais aussi intégrée dans la plupart des systèmes embarqués tels que les téléphones. Bref, il est possible de partitionner une mmc physiquement (sans possibilité de retour arrière). Cette opération se fait en deux temps : écriture des paramètres et finalisation (bit EXT _ CSD _ PARTITION _ SETTING _ COMPLETED). Le noyau précédent ne vérifiait pas la finalisation et utilisait les données de partitionnement si elles étaient présentes (ce qui engendre des erreurs par la suite).
Pourquoi ce problème est-il passé à la trappe ? Parce que la plupart des mémoires sortent d'usine déjà partitionnées. Pour le reproduire, il faut avoir une mmc vierge et faire l'opération manuellement.
Développement Linux
Les choses commencent à devenir drôle à partir de maintenant. La communauté qui développe le noyau est composée de développeurs, de mainteneurs et d'UN intégrateur. Vu le travail monstre qui est réalisé, l'intégrateur à savoir Linus Torvald en personne (l'initiateur du projet) doit faire confiance à des mainteneurs pour chaque sous-système du noyau. Ces mainteneurs sont chargés de récolter les modifications (patchs) des développeurs et participent éventuellement eux-mêmes à ces modifications. La plupart travaillent pour une entreprise ou sont rémunérés par une fondation.
Autant le dire tout de suite : le niveau technique de toutes ces personnes est très important. En plus de cela, il faut respecter scrupuleusement les normes de codage du dictateur sous peine de se faire jeter lamentablement. Pour ma part, j'ai eu affaire à Ulf Hansson (un des mainteneurs du sous-système mmc) qui travaille pour la fondation Linaro. Ce qui est drôle c'est que, suite aux échanges, je suis passé d'une simple modification (~ 17 lignes) à une série de trois patchs (~ 176 lignes) + une lettre de couverture (cover letter) expliquant en détail la nature du patch. Il m'a fallu pas moins de 6 versions sur deux mois (principalement des questions de forme) pour arriver à un résultat parfait, ce qui peut être vraiment décourageant.
Déroulement des opérations :
- Mardi 17 juillet 2014 : Permière version envoyée sur la liste
- Mercredi 13 août 2014 : Réponse d'Ulf
- Lundi 18 août 2014 : Après quelques discussions, soumission de la deuxième version (améliorée)
- Lundi 18 août 2014 : Réponse d'Ulf, Les patchs ont le même entête et il manque l'historique.
- Lundi 18 août 2014 : Soumission de la troisième version
- Lundi 8 septembre 2014 : Réponse d'Ulf : c'est cool, mais le patch bouge du code ET fixe un bug en même temps
- Jeudi 11 septembre 2014 : Soumission de la quatrième version, trois patchs
- Jeudi 11 septembre 2014 : Réponse d'Ulf, le second patch ne compile pas. Il faudrait être plus clair dans les explications et il y a des accolades inutiles
- Vendredi 12 septembre 2014 : Soumission de la cinquième version avec une lettre de couverture contenant tous les détails du patchset
- Lundi 15 septembre 2014 : Réponse de Jaehoon Chung : j'ai oublié de supprimer les accolades
- Lundi 15 septembre 2014 : Soumission de la sixième version
- Jeudi 18 septembre 2014 : Acception du patch par Ulf
Que ce fut long ! Deux mois pour pousser un patch. Pour autant, ce n'est pas fini. Le patch a été intégré dans la branche 'next' d'Ulf. Celle-ci sera intégrée le lendemain dans la branche next du noyau qui servira de base pour la prochaine version (3.18).
Outils
Il y a plusieurs façons de générer un patch. Au début, je n'avais pas envie de récupérer les gigas octets que représentent le git du noyau, donc c'était à la main à partir d'un tar. On se rend vite compte que pour un patch un peu plus évolué sur un noyau qui bouge tout le temps, il faut absolument passer par git et ses commandes magiques : git-format-patch/git-apply/git-am. En plus d'être efficaces, elles ont été conçues pour ça (et ne parlons pas de la possibilité de créer des branches de développement pour tester/valider ses patchs). Il faut juste avoir le courage de faire le clone initial...
Conclusion
Envoyer un patch pour la première fois relève du parcours du combattant ! La modification est maintenant intégrée dans le noyau 3.18 (sortie définitive décembre 2014). Mes quelques lignes de code seront donc exécutées au démarrage des prochains smartphones équipés d'un noyau d'une version égale ou supérieure à 3.18. Donc, si ça plante, ce sera peut-être un peu de ma faute :)