xref: /linux/Documentation/process/howto.rst (revision c532de5a67a70f8533d495f8f2aaa9a0491c3ad0)
1.. _process_howto:
2
3HOWTO do Linux kernel development
4=================================
5
6This is the be-all, end-all document on this topic.  It contains
7instructions on how to become a Linux kernel developer and how to learn
8to work with the Linux kernel development community.  It tries to not
9contain anything related to the technical aspects of kernel programming,
10but will help point you in the right direction for that.
11
12If anything in this document becomes out of date, please send in patches
13to the maintainer of this file, who is listed at the bottom of the
14document.
15
16
17Introduction
18------------
19
20So, you want to learn how to become a Linux kernel developer?  Or you
21have been told by your manager, "Go write a Linux driver for this
22device."  This document's goal is to teach you everything you need to
23know to achieve this by describing the process you need to go through,
24and hints on how to work with the community.  It will also try to
25explain some of the reasons why the community works like it does.
26
27The kernel is written mostly in C, with some architecture-dependent
28parts written in assembly. A good understanding of C is required for
29kernel development.  Assembly (any architecture) is not required unless
30you plan to do low-level development for that architecture.  Though they
31are not a good substitute for a solid C education and/or years of
32experience, the following books are good for, if anything, reference:
33
34 - "The C Programming Language" by Kernighan and Ritchie [Prentice Hall]
35 - "Practical C Programming" by Steve Oualline [O'Reilly]
36 - "C:  A Reference Manual" by Harbison and Steele [Prentice Hall]
37
38The kernel is written using GNU C and the GNU toolchain.  While it
39adheres to the ISO C11 standard, it uses a number of extensions that are
40not featured in the standard.  The kernel is a freestanding C
41environment, with no reliance on the standard C library, so some
42portions of the C standard are not supported.  Arbitrary long long
43divisions and floating point are not allowed.  It can sometimes be
44difficult to understand the assumptions the kernel has on the toolchain
45and the extensions that it uses, and unfortunately there is no
46definitive reference for them.  Please check the gcc info pages (`info
47gcc`) for some information on them.
48
49Please remember that you are trying to learn how to work with the
50existing development community.  It is a diverse group of people, with
51high standards for coding, style and procedure.  These standards have
52been created over time based on what they have found to work best for
53such a large and geographically dispersed team.  Try to learn as much as
54possible about these standards ahead of time, as they are well
55documented; do not expect people to adapt to you or your company's way
56of doing things.
57
58
59Legal Issues
60------------
61
62The Linux kernel source code is released under the GPL.  Please see the file
63COPYING in the main directory of the source tree. The Linux kernel licensing
64rules and how to use `SPDX <https://spdx.org/>`_ identifiers in source code are
65described in :ref:`Documentation/process/license-rules.rst <kernel_licensing>`.
66If you have further questions about the license, please contact a lawyer, and do
67not ask on the Linux kernel mailing list.  The people on the mailing lists are
68not lawyers, and you should not rely on their statements on legal matters.
69
70For common questions and answers about the GPL, please see:
71
72	https://www.gnu.org/licenses/gpl-faq.html
73
74
75Documentation
76-------------
77
78The Linux kernel source tree has a large range of documents that are
79invaluable for learning how to interact with the kernel community.  When
80new features are added to the kernel, it is recommended that new
81documentation files are also added which explain how to use the feature.
82When a kernel change causes the interface that the kernel exposes to
83userspace to change, it is recommended that you send the information or
84a patch to the manual pages explaining the change to the manual pages
85maintainer at alx@kernel.org, and CC the list linux-api@vger.kernel.org.
86
87Here is a list of files that are in the kernel source tree that are
88required reading:
89
90  :ref:`Documentation/admin-guide/README.rst <readme>`
91    This file gives a short background on the Linux kernel and describes
92    what is necessary to do to configure and build the kernel.  People
93    who are new to the kernel should start here.
94
95  :ref:`Documentation/process/changes.rst <changes>`
96    This file gives a list of the minimum levels of various software
97    packages that are necessary to build and run the kernel
98    successfully.
99
100  :ref:`Documentation/process/coding-style.rst <codingstyle>`
101    This describes the Linux kernel coding style, and some of the
102    rationale behind it. All new code is expected to follow the
103    guidelines in this document. Most maintainers will only accept
104    patches if these rules are followed, and many people will only
105    review code if it is in the proper style.
106
107  :ref:`Documentation/process/submitting-patches.rst <submittingpatches>`
108    This file describes in explicit detail how to successfully create
109    and send a patch, including (but not limited to):
110
111       - Email contents
112       - Email format
113       - Who to send it to
114
115    Following these rules will not guarantee success (as all patches are
116    subject to scrutiny for content and style), but not following them
117    will almost always prevent it.
118
119    Other excellent descriptions of how to create patches properly are:
120
121	"The Perfect Patch"
122		https://www.ozlabs.org/~akpm/stuff/tpp.txt
123
124	"Linux kernel patch submission format"
125		https://web.archive.org/web/20180829112450/http://linux.yyz.us/patch-format.html
126
127  :ref:`Documentation/process/stable-api-nonsense.rst <stable_api_nonsense>`
128    This file describes the rationale behind the conscious decision to
129    not have a stable API within the kernel, including things like:
130
131      - Subsystem shim-layers (for compatibility?)
132      - Driver portability between Operating Systems.
133      - Mitigating rapid change within the kernel source tree (or
134	preventing rapid change)
135
136    This document is crucial for understanding the Linux development
137    philosophy and is very important for people moving to Linux from
138    development on other Operating Systems.
139
140  :ref:`Documentation/process/security-bugs.rst <securitybugs>`
141    If you feel you have found a security problem in the Linux kernel,
142    please follow the steps in this document to help notify the kernel
143    developers, and help solve the issue.
144
145  :ref:`Documentation/process/management-style.rst <managementstyle>`
146    This document describes how Linux kernel maintainers operate and the
147    shared ethos behind their methodologies.  This is important reading
148    for anyone new to kernel development (or anyone simply curious about
149    it), as it resolves a lot of common misconceptions and confusion
150    about the unique behavior of kernel maintainers.
151
152  :ref:`Documentation/process/stable-kernel-rules.rst <stable_kernel_rules>`
153    This file describes the rules on how the stable kernel releases
154    happen, and what to do if you want to get a change into one of these
155    releases.
156
157  :ref:`Documentation/process/kernel-docs.rst <kernel_docs>`
158    A list of external documentation that pertains to kernel
159    development.  Please consult this list if you do not find what you
160    are looking for within the in-kernel documentation.
161
162  :ref:`Documentation/process/applying-patches.rst <applying_patches>`
163    A good introduction describing exactly what a patch is and how to
164    apply it to the different development branches of the kernel.
165
166The kernel also has a large number of documents that can be
167automatically generated from the source code itself or from
168ReStructuredText markups (ReST), like this one. This includes a
169full description of the in-kernel API, and rules on how to handle
170locking properly.
171
172All such documents can be generated as PDF or HTML by running::
173
174	make pdfdocs
175	make htmldocs
176
177respectively from the main kernel source directory.
178
179The documents that uses ReST markup will be generated at Documentation/output.
180They can also be generated on LaTeX and ePub formats with::
181
182	make latexdocs
183	make epubdocs
184
185Becoming A Kernel Developer
186---------------------------
187
188If you do not know anything about Linux kernel development, you should
189look at the Linux KernelNewbies project:
190
191	https://kernelnewbies.org
192
193It consists of a helpful mailing list where you can ask almost any type
194of basic kernel development question (make sure to search the archives
195first, before asking something that has already been answered in the
196past.)  It also has an IRC channel that you can use to ask questions in
197real-time, and a lot of helpful documentation that is useful for
198learning about Linux kernel development.
199
200The website has basic information about code organization, subsystems,
201and current projects (both in-tree and out-of-tree). It also describes
202some basic logistical information, like how to compile a kernel and
203apply a patch.
204
205If you do not know where you want to start, but you want to look for
206some task to start doing to join into the kernel development community,
207go to the Linux Kernel Janitor's project:
208
209	https://kernelnewbies.org/KernelJanitors
210
211It is a great place to start.  It describes a list of relatively simple
212problems that need to be cleaned up and fixed within the Linux kernel
213source tree.  Working with the developers in charge of this project, you
214will learn the basics of getting your patch into the Linux kernel tree,
215and possibly be pointed in the direction of what to go work on next, if
216you do not already have an idea.
217
218Before making any actual modifications to the Linux kernel code, it is
219imperative to understand how the code in question works.  For this
220purpose, nothing is better than reading through it directly (most tricky
221bits are commented well), perhaps even with the help of specialized
222tools.  One such tool that is particularly recommended is the Linux
223Cross-Reference project, which is able to present source code in a
224self-referential, indexed webpage format. An excellent up-to-date
225repository of the kernel code may be found at:
226
227	https://elixir.bootlin.com/
228
229
230The development process
231-----------------------
232
233Linux kernel development process currently consists of a few different
234main kernel "branches" and lots of different subsystem-specific kernel
235branches.  These different branches are:
236
237  - Linus's mainline tree
238  - Various stable trees with multiple major numbers
239  - Subsystem-specific trees
240  - linux-next integration testing tree
241
242Mainline tree
243~~~~~~~~~~~~~
244
245The mainline tree is maintained by Linus Torvalds, and can be found at
246https://kernel.org or in the repo.  Its development process is as follows:
247
248  - As soon as a new kernel is released a two week window is open,
249    during this period of time maintainers can submit big diffs to
250    Linus, usually the patches that have already been included in the
251    linux-next for a few weeks.  The preferred way to submit big changes
252    is using git (the kernel's source management tool, more information
253    can be found at https://git-scm.com/) but plain patches are also just
254    fine.
255  - After two weeks a -rc1 kernel is released and the focus is on making the
256    new kernel as rock solid as possible.  Most of the patches at this point
257    should fix a regression.  Bugs that have always existed are not
258    regressions, so only push these kinds of fixes if they are important.
259    Please note that a whole new driver (or filesystem) might be accepted
260    after -rc1 because there is no risk of causing regressions with such a
261    change as long as the change is self-contained and does not affect areas
262    outside of the code that is being added.  git can be used to send
263    patches to Linus after -rc1 is released, but the patches need to also be
264    sent to a public mailing list for review.
265  - A new -rc is released whenever Linus deems the current git tree to
266    be in a reasonably sane state adequate for testing.  The goal is to
267    release a new -rc kernel every week.
268  - Process continues until the kernel is considered "ready", the
269    process should last around 6 weeks.
270
271It is worth mentioning what Andrew Morton wrote on the linux-kernel
272mailing list about kernel releases:
273
274	*"Nobody knows when a kernel will be released, because it's
275	released according to perceived bug status, not according to a
276	preconceived timeline."*
277
278Various stable trees with multiple major numbers
279~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
280
281Kernels with 3-part versions are -stable kernels. They contain
282relatively small and critical fixes for security problems or significant
283regressions discovered in a given major mainline release. Each release
284in a major stable series increments the third part of the version
285number, keeping the first two parts the same.
286
287This is the recommended branch for users who want the most recent stable
288kernel and are not interested in helping test development/experimental
289versions.
290
291Stable trees are maintained by the "stable" team <stable@vger.kernel.org>, and
292are released as needs dictate.  The normal release period is approximately
293two weeks, but it can be longer if there are no pressing problems.  A
294security-related problem, instead, can cause a release to happen almost
295instantly.
296
297The file :ref:`Documentation/process/stable-kernel-rules.rst <stable_kernel_rules>`
298in the kernel tree documents what kinds of changes are acceptable for
299the -stable tree, and how the release process works.
300
301Subsystem-specific trees
302~~~~~~~~~~~~~~~~~~~~~~~~
303
304The maintainers of the various kernel subsystems --- and also many
305kernel subsystem developers --- expose their current state of
306development in source repositories.  That way, others can see what is
307happening in the different areas of the kernel.  In areas where
308development is rapid, a developer may be asked to base his submissions
309onto such a subsystem kernel tree so that conflicts between the
310submission and other already ongoing work are avoided.
311
312Most of these repositories are git trees, but there are also other SCMs
313in use, or patch queues being published as quilt series.  Addresses of
314these subsystem repositories are listed in the MAINTAINERS file.  Many
315of them can be browsed at https://git.kernel.org/.
316
317Before a proposed patch is committed to such a subsystem tree, it is
318subject to review which primarily happens on mailing lists (see the
319respective section below).  For several kernel subsystems, this review
320process is tracked with the tool patchwork.  Patchwork offers a web
321interface which shows patch postings, any comments on a patch or
322revisions to it, and maintainers can mark patches as under review,
323accepted, or rejected.  Most of these patchwork sites are listed at
324https://patchwork.kernel.org/.
325
326linux-next integration testing tree
327~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
328
329Before updates from subsystem trees are merged into the mainline tree,
330they need to be integration-tested.  For this purpose, a special
331testing repository exists into which virtually all subsystem trees are
332pulled on an almost daily basis:
333
334	https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
335
336This way, the linux-next gives a summary outlook onto what will be
337expected to go into the mainline kernel at the next merge period.
338Adventurous testers are very welcome to runtime-test the linux-next.
339
340
341Bug Reporting
342-------------
343
344The file 'Documentation/admin-guide/reporting-issues.rst' in the main kernel
345source directory describes how to report a possible kernel bug, and details
346what kind of information is needed by the kernel developers to help track
347down the problem.
348
349
350Managing bug reports
351--------------------
352
353One of the best ways to put into practice your hacking skills is by fixing
354bugs reported by other people. Not only will you help to make the kernel
355more stable, but you'll also learn to fix real-world problems and you will
356improve your skills, and other developers will be aware of your presence.
357Fixing bugs is one of the best ways to get merits among other developers,
358because not many people like wasting time fixing other people's bugs.
359
360To work on already reported bug reports, find a subsystem you are interested in.
361Check the MAINTAINERS file where bugs for that subsystem get reported to; often
362it will be a mailing list, rarely a bugtracker. Search the archives of said
363place for recent reports and help where you see fit. You may also want to check
364https://bugzilla.kernel.org for bug reports; only a handful of kernel subsystems
365use it actively for reporting or tracking, nevertheless bugs for the whole
366kernel get filed there.
367
368
369Mailing lists
370-------------
371
372As some of the above documents describe, the majority of the core kernel
373developers participate on the Linux Kernel Mailing list.  Details on how
374to subscribe and unsubscribe from the list can be found at:
375
376	https://subspace.kernel.org/subscribing.html
377
378There are archives of the mailing list on the web in many different
379places.  Use a search engine to find these archives.  For example:
380
381	https://lore.kernel.org/linux-kernel/
382
383It is highly recommended that you search the archives about the topic
384you want to bring up, before you post it to the list. A lot of things
385already discussed in detail are only recorded at the mailing list
386archives.
387
388Most of the individual kernel subsystems also have their own separate
389mailing list where they do their development efforts.  See the
390MAINTAINERS file for a list of what these lists are for the different
391groups.
392
393Many of the lists are hosted on kernel.org. Information on them can be
394found at:
395
396	https://subspace.kernel.org
397
398Please remember to follow good behavioral habits when using the lists.
399Though a bit cheesy, the following URL has some simple guidelines for
400interacting with the list (or any list):
401
402	https://subspace.kernel.org/etiquette.html
403
404If multiple people respond to your mail, the CC: list of recipients may
405get pretty large. Don't remove anybody from the CC: list without a good
406reason, or don't reply only to the list address. Get used to receiving the
407mail twice, one from the sender and the one from the list, and don't try
408to tune that by adding fancy mail-headers, people will not like it.
409
410Remember to keep the context and the attribution of your replies intact,
411keep the "John Kernelhacker wrote ...:" lines at the top of your reply, and
412add your statements between the individual quoted sections instead of
413writing at the top of the mail.
414
415If you add patches to your mail, make sure they are plain readable text
416as stated in :ref:`Documentation/process/submitting-patches.rst <submittingpatches>`.
417Kernel developers don't want to deal with
418attachments or compressed patches; they may want to comment on
419individual lines of your patch, which works only that way. Make sure you
420use a mail program that does not mangle spaces and tab characters. A
421good first test is to send the mail to yourself and try to apply your
422own patch by yourself. If that doesn't work, get your mail program fixed
423or change it until it works.
424
425Above all, please remember to show respect to other subscribers.
426
427
428Working with the community
429--------------------------
430
431The goal of the kernel community is to provide the best possible kernel
432there is.  When you submit a patch for acceptance, it will be reviewed
433on its technical merits and those alone.  So, what should you be
434expecting?
435
436  - criticism
437  - comments
438  - requests for change
439  - requests for justification
440  - silence
441
442Remember, this is part of getting your patch into the kernel.  You have
443to be able to take criticism and comments about your patches, evaluate
444them at a technical level and either rework your patches or provide
445clear and concise reasoning as to why those changes should not be made.
446If there are no responses to your posting, wait a few days and try
447again, sometimes things get lost in the huge volume.
448
449What should you not do?
450
451  - expect your patch to be accepted without question
452  - become defensive
453  - ignore comments
454  - resubmit the patch without making any of the requested changes
455
456In a community that is looking for the best technical solution possible,
457there will always be differing opinions on how beneficial a patch is.
458You have to be cooperative, and willing to adapt your idea to fit within
459the kernel.  Or at least be willing to prove your idea is worth it.
460Remember, being wrong is acceptable as long as you are willing to work
461toward a solution that is right.
462
463It is normal that the answers to your first patch might simply be a list
464of a dozen things you should correct.  This does **not** imply that your
465patch will not be accepted, and it is **not** meant against you
466personally.  Simply correct all issues raised against your patch and
467resend it.
468
469
470Differences between the kernel community and corporate structures
471-----------------------------------------------------------------
472
473The kernel community works differently than most traditional corporate
474development environments.  Here are a list of things that you can try to
475do to avoid problems:
476
477  Good things to say regarding your proposed changes:
478
479    - "This solves multiple problems."
480    - "This deletes 2000 lines of code."
481    - "Here is a patch that explains what I am trying to describe."
482    - "I tested it on 5 different architectures..."
483    - "Here is a series of small patches that..."
484    - "This increases performance on typical machines..."
485
486  Bad things you should avoid saying:
487
488    - "We did it this way in AIX/ptx/Solaris, so therefore it must be
489      good..."
490    - "I've being doing this for 20 years, so..."
491    - "This is required for my company to make money"
492    - "This is for our Enterprise product line."
493    - "Here is my 1000 page design document that describes my idea"
494    - "I've been working on this for 6 months..."
495    - "Here's a 5000 line patch that..."
496    - "I rewrote all of the current mess, and here it is..."
497    - "I have a deadline, and this patch needs to be applied now."
498
499Another way the kernel community is different than most traditional
500software engineering work environments is the faceless nature of
501interaction.  One benefit of using email and irc as the primary forms of
502communication is the lack of discrimination based on gender or race.
503The Linux kernel work environment is accepting of women and minorities
504because all you are is an email address.  The international aspect also
505helps to level the playing field because you can't guess gender based on
506a person's name. A man may be named Andrea and a woman may be named Pat.
507Most women who have worked in the Linux kernel and have expressed an
508opinion have had positive experiences.
509
510The language barrier can cause problems for some people who are not
511comfortable with English.  A good grasp of the language can be needed in
512order to get ideas across properly on mailing lists, so it is
513recommended that you check your emails to make sure they make sense in
514English before sending them.
515
516
517Break up your changes
518---------------------
519
520The Linux kernel community does not gladly accept large chunks of code
521dropped on it all at once.  The changes need to be properly introduced,
522discussed, and broken up into tiny, individual portions.  This is almost
523the exact opposite of what companies are used to doing.  Your proposal
524should also be introduced very early in the development process, so that
525you can receive feedback on what you are doing.  It also lets the
526community feel that you are working with them, and not simply using them
527as a dumping ground for your feature.  However, don't send 50 emails at
528one time to a mailing list, your patch series should be smaller than
529that almost all of the time.
530
531The reasons for breaking things up are the following:
532
5331) Small patches increase the likelihood that your patches will be
534   applied, since they don't take much time or effort to verify for
535   correctness.  A 5 line patch can be applied by a maintainer with
536   barely a second glance. However, a 500 line patch may take hours to
537   review for correctness (the time it takes is exponentially
538   proportional to the size of the patch, or something).
539
540   Small patches also make it very easy to debug when something goes
541   wrong.  It's much easier to back out patches one by one than it is
542   to dissect a very large patch after it's been applied (and broken
543   something).
544
5452) It's important not only to send small patches, but also to rewrite
546   and simplify (or simply re-order) patches before submitting them.
547
548Here is an analogy from kernel developer Al Viro:
549
550	*"Think of a teacher grading homework from a math student.  The
551	teacher does not want to see the student's trials and errors
552	before they came up with the solution. They want to see the
553	cleanest, most elegant answer.  A good student knows this, and
554	would never submit her intermediate work before the final
555	solution.*
556
557	*The same is true of kernel development. The maintainers and
558	reviewers do not want to see the thought process behind the
559	solution to the problem one is solving. They want to see a
560	simple and elegant solution."*
561
562It may be challenging to keep the balance between presenting an elegant
563solution and working together with the community and discussing your
564unfinished work. Therefore it is good to get early in the process to
565get feedback to improve your work, but also keep your changes in small
566chunks that they may get already accepted, even when your whole task is
567not ready for inclusion now.
568
569Also realize that it is not acceptable to send patches for inclusion
570that are unfinished and will be "fixed up later."
571
572
573Justify your change
574-------------------
575
576Along with breaking up your patches, it is very important for you to let
577the Linux community know why they should add this change.  New features
578must be justified as being needed and useful.
579
580
581Document your change
582--------------------
583
584When sending in your patches, pay special attention to what you say in
585the text in your email.  This information will become the ChangeLog
586information for the patch, and will be preserved for everyone to see for
587all time.  It should describe the patch completely, containing:
588
589  - why the change is necessary
590  - the overall design approach in the patch
591  - implementation details
592  - testing results
593
594For more details on what this should all look like, please see the
595ChangeLog section of the document:
596
597  "The Perfect Patch"
598      https://www.ozlabs.org/~akpm/stuff/tpp.txt
599
600
601All of these things are sometimes very hard to do. It can take years to
602perfect these practices (if at all). It's a continuous process of
603improvement that requires a lot of patience and determination. But
604don't give up, it's possible. Many have done it before, and each had to
605start exactly where you are now.
606
607
608
609
610----------
611
612Thanks to Paolo Ciarrocchi who allowed the "Development Process"
613(https://lwn.net/Articles/94386/) section
614to be based on text he had written, and to Randy Dunlap and Gerrit
615Huizenga for some of the list of things you should and should not say.
616Also thanks to Pat Mochel, Hanna Linder, Randy Dunlap, Kay Sievers,
617Vojtech Pavlik, Jan Kara, Josh Boyer, Kees Cook, Andrew Morton, Andi
618Kleen, Vadim Lobanov, Jesper Juhl, Adrian Bunk, Keri Harris, Frans Pop,
619David A. Wheeler, Junio Hamano, Michael Kerrisk, and Alex Shepard for
620their review, comments, and contributions.  Without their help, this
621document would not have been possible.
622
623
624
625Maintainer: Greg Kroah-Hartman <greg@kroah.com>
626