blob: c6875b1db56f3cac6bbdbe328f3bbc4e7421ea65 [file] [log] [blame]
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -08001HOWTO do Linux kernel development
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -03002=================================
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -08003
4This is the be-all, end-all document on this topic. It contains
5instructions on how to become a Linux kernel developer and how to learn
6to work with the Linux kernel development community. It tries to not
7contain anything related to the technical aspects of kernel programming,
8but will help point you in the right direction for that.
9
10If anything in this document becomes out of date, please send in patches
11to the maintainer of this file, who is listed at the bottom of the
12document.
13
14
15Introduction
16------------
17
18So, you want to learn how to become a Linux kernel developer? Or you
19have been told by your manager, "Go write a Linux driver for this
20device." This document's goal is to teach you everything you need to
21know to achieve this by describing the process you need to go through,
22and hints on how to work with the community. It will also try to
23explain some of the reasons why the community works like it does.
24
25The kernel is written mostly in C, with some architecture-dependent
26parts written in assembly. A good understanding of C is required for
27kernel development. Assembly (any architecture) is not required unless
28you plan to do low-level development for that architecture. Though they
29are not a good substitute for a solid C education and/or years of
30experience, the following books are good for, if anything, reference:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -030031
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -080032 - "The C Programming Language" by Kernighan and Ritchie [Prentice Hall]
33 - "Practical C Programming" by Steve Oualline [O'Reilly]
Robert P. J. Day4de0ca82007-01-17 04:54:07 -050034 - "C: A Reference Manual" by Harbison and Steele [Prentice Hall]
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -080035
36The kernel is written using GNU C and the GNU toolchain. While it
37adheres to the ISO C89 standard, it uses a number of extensions that are
38not featured in the standard. The kernel is a freestanding C
39environment, with no reliance on the standard C library, so some
40portions of the C standard are not supported. Arbitrary long long
41divisions and floating point are not allowed. It can sometimes be
42difficult to understand the assumptions the kernel has on the toolchain
43and the extensions that it uses, and unfortunately there is no
44definitive reference for them. Please check the gcc info pages (`info
45gcc`) for some information on them.
46
47Please remember that you are trying to learn how to work with the
48existing development community. It is a diverse group of people, with
49high standards for coding, style and procedure. These standards have
50been created over time based on what they have found to work best for
51such a large and geographically dispersed team. Try to learn as much as
52possible about these standards ahead of time, as they are well
53documented; do not expect people to adapt to you or your company's way
54of doing things.
55
56
57Legal Issues
58------------
59
60The Linux kernel source code is released under the GPL. Please see the
61file, COPYING, in the main directory of the source tree, for details on
62the license. If you have further questions about the license, please
63contact a lawyer, and do not ask on the Linux kernel mailing list. The
64people on the mailing lists are not lawyers, and you should not rely on
65their statements on legal matters.
66
67For common questions and answers about the GPL, please see:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -030068
Mauro Carvalho Chehabf1eebe92016-09-19 08:07:59 -030069 https://www.gnu.org/licenses/gpl-faq.html
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -080070
71
72Documentation
Philippe Loctauxce655082016-03-07 02:36:18 +010073-------------
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -080074
75The Linux kernel source tree has a large range of documents that are
76invaluable for learning how to interact with the kernel community. When
77new features are added to the kernel, it is recommended that new
78documentation files are also added which explain how to use the feature.
79When a kernel change causes the interface that the kernel exposes to
80userspace to change, it is recommended that you send the information or
81a patch to the manual pages explaining the change to the manual pages
Michael Kerrisk09b05f52008-10-03 15:23:45 -070082maintainer at mtk.manpages@gmail.com, and CC the list
83linux-api@vger.kernel.org.
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -080084
85Here is a list of files that are in the kernel source tree that are
86required reading:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -030087
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -080088 README
89 This file gives a short background on the Linux kernel and describes
90 what is necessary to do to configure and build the kernel. People
91 who are new to the kernel should start here.
92
Mauro Carvalho Chehab8c27ceff32016-10-18 10:12:27 -020093 :ref:`Documentation/process/changes.rst <changes>`
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -080094 This file gives a list of the minimum levels of various software
95 packages that are necessary to build and run the kernel
96 successfully.
97
Mauro Carvalho Chehab8c27ceff32016-10-18 10:12:27 -020098 :ref:`Documentation/process/coding-style.rst <codingstyle>`
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -080099 This describes the Linux kernel coding style, and some of the
100 rationale behind it. All new code is expected to follow the
101 guidelines in this document. Most maintainers will only accept
102 patches if these rules are followed, and many people will only
103 review code if it is in the proper style.
104
Mauro Carvalho Chehab8c27ceff32016-10-18 10:12:27 -0200105 :ref:`Documentation/process/submitting-patches.rst <submittingpatches>` and :ref:`Documentation/process/submitting-drivers.rst <submittingdrivers>`
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800106 These files describe in explicit detail how to successfully create
107 and send a patch, including (but not limited to):
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300108
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800109 - Email contents
110 - Email format
111 - Who to send it to
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300112
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800113 Following these rules will not guarantee success (as all patches are
114 subject to scrutiny for content and style), but not following them
115 will almost always prevent it.
116
117 Other excellent descriptions of how to create patches properly are:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300118
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800119 "The Perfect Patch"
Mauro Carvalho Chehabf1eebe92016-09-19 08:07:59 -0300120 https://www.ozlabs.org/~akpm/stuff/tpp.txt
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300121
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800122 "Linux kernel patch submission format"
123 http://linux.yyz.us/patch-format.html
124
Mauro Carvalho Chehab8c27ceff32016-10-18 10:12:27 -0200125 :ref:`Documentation/process/stable-api-nonsense.rst <stable_api_nonsense>`
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800126 This file describes the rationale behind the conscious decision to
127 not have a stable API within the kernel, including things like:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300128
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800129 - Subsystem shim-layers (for compatibility?)
130 - Driver portability between Operating Systems.
131 - Mitigating rapid change within the kernel source tree (or
132 preventing rapid change)
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300133
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800134 This document is crucial for understanding the Linux development
135 philosophy and is very important for people moving to Linux from
136 development on other Operating Systems.
137
Mauro Carvalho Chehab8c27ceff32016-10-18 10:12:27 -0200138 :ref:`Documentation/admin-guide/security-bugs.rst <securitybugs>`
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800139 If you feel you have found a security problem in the Linux kernel,
140 please follow the steps in this document to help notify the kernel
141 developers, and help solve the issue.
142
Mauro Carvalho Chehab8c27ceff32016-10-18 10:12:27 -0200143 :ref:`Documentation/process/management-style.rst <managementstyle>`
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800144 This document describes how Linux kernel maintainers operate and the
145 shared ethos behind their methodologies. This is important reading
146 for anyone new to kernel development (or anyone simply curious about
147 it), as it resolves a lot of common misconceptions and confusion
148 about the unique behavior of kernel maintainers.
149
Mauro Carvalho Chehab8c27ceff32016-10-18 10:12:27 -0200150 :ref:`Documentation/process/stable-kernel-rules.rst <stable_kernel_rules>`
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800151 This file describes the rules on how the stable kernel releases
152 happen, and what to do if you want to get a change into one of these
153 releases.
154
Mauro Carvalho Chehab8c27ceff32016-10-18 10:12:27 -0200155 :ref:`Documentation/process/kernel-docs.rst <kernel_docs>`
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800156 A list of external documentation that pertains to kernel
157 development. Please consult this list if you do not find what you
158 are looking for within the in-kernel documentation.
159
Mauro Carvalho Chehab8c27ceff32016-10-18 10:12:27 -0200160 :ref:`Documentation/process/applying-patches.rst <applying_patches>`
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800161 A good introduction describing exactly what a patch is and how to
162 apply it to the different development branches of the kernel.
163
164The kernel also has a large number of documents that can be
Mauro Carvalho Chehab43fb67a2016-09-19 08:07:57 -0300165automatically generated from the source code itself or from
166ReStructuredText markups (ReST), like this one. This includes a
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800167full description of the in-kernel API, and rules on how to handle
Mauro Carvalho Chehab43fb67a2016-09-19 08:07:57 -0300168locking properly.
169
Jonathan Corbet1b49ecf2016-09-20 18:46:36 -0600170All such documents can be generated as PDF or HTML by running::
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300171
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800172 make pdfdocs
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800173 make htmldocs
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300174
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800175respectively from the main kernel source directory.
176
Mauro Carvalho Chehab43fb67a2016-09-19 08:07:57 -0300177The documents that uses ReST markup will be generated at Documentation/output.
Jonathan Corbet1b49ecf2016-09-20 18:46:36 -0600178They can also be generated on LaTeX and ePub formats with::
Mauro Carvalho Chehab43fb67a2016-09-19 08:07:57 -0300179
180 make latexdocs
181 make epubdocs
182
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800183Becoming A Kernel Developer
184---------------------------
185
186If you do not know anything about Linux kernel development, you should
187look at the Linux KernelNewbies project:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300188
Mauro Carvalho Chehabf1eebe92016-09-19 08:07:59 -0300189 https://kernelnewbies.org
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300190
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800191It consists of a helpful mailing list where you can ask almost any type
192of basic kernel development question (make sure to search the archives
193first, before asking something that has already been answered in the
194past.) It also has an IRC channel that you can use to ask questions in
195real-time, and a lot of helpful documentation that is useful for
196learning about Linux kernel development.
197
198The website has basic information about code organization, subsystems,
199and current projects (both in-tree and out-of-tree). It also describes
200some basic logistical information, like how to compile a kernel and
201apply a patch.
202
203If you do not know where you want to start, but you want to look for
204some task to start doing to join into the kernel development community,
205go to the Linux Kernel Janitor's project:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300206
Mauro Carvalho Chehabf1eebe92016-09-19 08:07:59 -0300207 https://kernelnewbies.org/KernelJanitors
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300208
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800209It is a great place to start. It describes a list of relatively simple
210problems that need to be cleaned up and fixed within the Linux kernel
211source tree. Working with the developers in charge of this project, you
212will learn the basics of getting your patch into the Linux kernel tree,
213and possibly be pointed in the direction of what to go work on next, if
214you do not already have an idea.
215
216If you already have a chunk of code that you want to put into the kernel
217tree, but need some help getting it in the proper form, the
218kernel-mentors project was created to help you out with this. It is a
219mailing list, and can be found at:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300220
Mauro Carvalho Chehabf1eebe92016-09-19 08:07:59 -0300221 https://selenic.com/mailman/listinfo/kernel-mentors
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800222
223Before making any actual modifications to the Linux kernel code, it is
224imperative to understand how the code in question works. For this
225purpose, nothing is better than reading through it directly (most tricky
226bits are commented well), perhaps even with the help of specialized
227tools. One such tool that is particularly recommended is the Linux
228Cross-Reference project, which is able to present source code in a
229self-referential, indexed webpage format. An excellent up-to-date
230repository of the kernel code may be found at:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300231
Doug Smythies1d125542015-11-22 15:42:48 -0800232 http://lxr.free-electrons.com/
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800233
234
235The development process
236-----------------------
237
238Linux kernel development process currently consists of a few different
239main kernel "branches" and lots of different subsystem-specific kernel
240branches. These different branches are:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300241
Mario Carrilloe4144fe2015-08-24 09:33:09 -0500242 - main 4.x kernel tree
243 - 4.x.y -stable kernel tree
244 - 4.x -git kernel patches
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800245 - subsystem specific kernel trees and patches
Mario Carrilloe4144fe2015-08-24 09:33:09 -0500246 - the 4.x -next kernel tree for integration tests
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800247
Mario Carrilloe4144fe2015-08-24 09:33:09 -05002484.x kernel tree
SeongJae Parkd80b9d22016-11-01 05:27:10 +0900249~~~~~~~~~~~~~~~
250
Mario Carrilloe4144fe2015-08-24 09:33:09 -05002514.x kernels are maintained by Linus Torvalds, and can be found on
Mauro Carvalho Chehabf1eebe92016-09-19 08:07:59 -0300252https://kernel.org in the pub/linux/kernel/v4.x/ directory. Its development
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800253process is as follows:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300254
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800255 - As soon as a new kernel is released a two weeks window is open,
256 during this period of time maintainers can submit big diffs to
257 Linus, usually the patches that have already been included in the
Stefan Richterc607cf02010-01-15 13:23:51 +0100258 -next kernel for a few weeks. The preferred way to submit big changes
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800259 is using git (the kernel's source management tool, more information
Mauro Carvalho Chehabf1eebe92016-09-19 08:07:59 -0300260 can be found at https://git-scm.com/) but plain patches are also just
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800261 fine.
Dan Carpenter388f9b22016-11-28 17:43:15 +0300262 - After two weeks a -rc1 kernel is released and the focus is on making the
263 new kernel as rock solid as possible. Most of the patches at this point
264 should fix a regression. Bugs that have always existed are not
265 regressions, so only push these kinds of fixes if they are important.
266 Please note that a whole new driver (or filesystem) might be accepted
267 after -rc1 because there is no risk of causing regressions with such a
268 change as long as the change is self-contained and does not affect areas
269 outside of the code that is being added. git can be used to send
270 patches to Linus after -rc1 is released, but the patches need to also be
271 sent to a public mailing list for review.
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800272 - A new -rc is released whenever Linus deems the current git tree to
273 be in a reasonably sane state adequate for testing. The goal is to
274 release a new -rc kernel every week.
275 - Process continues until the kernel is considered "ready", the
276 process should last around 6 weeks.
277
278It is worth mentioning what Andrew Morton wrote on the linux-kernel
279mailing list about kernel releases:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300280
Mauro Carvalho Chehab34fed7e2016-09-19 08:07:58 -0300281 *"Nobody knows when a kernel will be released, because it's
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800282 released according to perceived bug status, not according to a
Mauro Carvalho Chehab34fed7e2016-09-19 08:07:58 -0300283 preconceived timeline."*
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800284
Mario Carrilloe4144fe2015-08-24 09:33:09 -05002854.x.y -stable kernel tree
SeongJae Parkd80b9d22016-11-01 05:27:10 +0900286~~~~~~~~~~~~~~~~~~~~~~~~~
287
Kees Cook591bfc62012-04-18 23:16:45 -0700288Kernels with 3-part versions are -stable kernels. They contain
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800289relatively small and critical fixes for security problems or significant
Mario Carrilloe4144fe2015-08-24 09:33:09 -0500290regressions discovered in a given 4.x kernel.
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800291
292This is the recommended branch for users who want the most recent stable
293kernel and are not interested in helping test development/experimental
294versions.
295
Mario Carrilloe4144fe2015-08-24 09:33:09 -0500296If no 4.x.y kernel is available, then the highest numbered 4.x
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800297kernel is the current stable kernel.
298
Mario Carrilloe4144fe2015-08-24 09:33:09 -05002994.x.y are maintained by the "stable" team <stable@vger.kernel.org>, and
Joe Perches2eb7f202011-12-09 14:12:00 -0800300are released as needs dictate. The normal release period is approximately
Jonathan Corbet0fe8a3c2008-04-25 11:23:56 -0600301two weeks, but it can be longer if there are no pressing problems. A
302security-related problem, instead, can cause a release to happen almost
303instantly.
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800304
Mauro Carvalho Chehab8c27ceff32016-10-18 10:12:27 -0200305The file Documentation/process/stable-kernel-rules.rst in the kernel tree
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800306documents what kinds of changes are acceptable for the -stable tree, and
307how the release process works.
308
Mario Carrilloe4144fe2015-08-24 09:33:09 -05003094.x -git patches
SeongJae Parkd80b9d22016-11-01 05:27:10 +0900310~~~~~~~~~~~~~~~~
311
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800312These are daily snapshots of Linus' kernel tree which are managed in a
313git repository (hence the name.) These patches are usually released
314daily and represent the current state of Linus' tree. They are more
315experimental than -rc kernels since they are generated automatically
316without even a cursory glance to see if they are sane.
317
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800318Subsystem Specific kernel trees and patches
SeongJae Parkd80b9d22016-11-01 05:27:10 +0900319~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
320
Stefan Richterc607cf02010-01-15 13:23:51 +0100321The maintainers of the various kernel subsystems --- and also many
322kernel subsystem developers --- expose their current state of
323development in source repositories. That way, others can see what is
324happening in the different areas of the kernel. In areas where
325development is rapid, a developer may be asked to base his submissions
326onto such a subsystem kernel tree so that conflicts between the
327submission and other already ongoing work are avoided.
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800328
Stefan Richterc607cf02010-01-15 13:23:51 +0100329Most of these repositories are git trees, but there are also other SCMs
330in use, or patch queues being published as quilt series. Addresses of
331these subsystem repositories are listed in the MAINTAINERS file. Many
Mauro Carvalho Chehabf1eebe92016-09-19 08:07:59 -0300332of them can be browsed at https://git.kernel.org/.
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800333
Stefan Richterc607cf02010-01-15 13:23:51 +0100334Before a proposed patch is committed to such a subsystem tree, it is
335subject to review which primarily happens on mailing lists (see the
336respective section below). For several kernel subsystems, this review
337process is tracked with the tool patchwork. Patchwork offers a web
338interface which shows patch postings, any comments on a patch or
339revisions to it, and maintainers can mark patches as under review,
340accepted, or rejected. Most of these patchwork sites are listed at
Mauro Carvalho Chehabf1eebe92016-09-19 08:07:59 -0300341https://patchwork.kernel.org/.
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800342
Mario Carrilloe4144fe2015-08-24 09:33:09 -05003434.x -next kernel tree for integration tests
SeongJae Parkd80b9d22016-11-01 05:27:10 +0900344~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
345
Mario Carrilloe4144fe2015-08-24 09:33:09 -0500346Before updates from subsystem trees are merged into the mainline 4.x
Stefan Richterc607cf02010-01-15 13:23:51 +0100347tree, they need to be integration-tested. For this purpose, a special
348testing repository exists into which virtually all subsystem trees are
349pulled on an almost daily basis:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300350
Mauro Carvalho Chehabf1eebe92016-09-19 08:07:59 -0300351 https://git.kernel.org/?p=linux/kernel/git/next/linux-next.git
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800352
Stefan Richterc607cf02010-01-15 13:23:51 +0100353This way, the -next kernel gives a summary outlook onto what will be
354expected to go into the mainline kernel at the next merge period.
355Adventurous testers are very welcome to runtime-test the -next kernel.
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800356
Stefan Richter8a465c32007-06-27 14:10:01 -0700357
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800358Bug Reporting
359-------------
360
Mauro Carvalho Chehabf1eebe92016-09-19 08:07:59 -0300361https://bugzilla.kernel.org is where the Linux kernel developers track kernel
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800362bugs. Users are encouraged to report all bugs that they find in this
363tool. For details on how to use the kernel bugzilla, please see:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300364
Mauro Carvalho Chehabf1eebe92016-09-19 08:07:59 -0300365 https://bugzilla.kernel.org/page.cgi?id=faq.html
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800366
Mauro Carvalho Chehab8c27ceff32016-10-18 10:12:27 -0200367The file admin-guide/reporting-bugs.rst in the main kernel source directory has a good
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800368template for how to report a possible kernel bug, and details what kind
369of information is needed by the kernel developers to help track down the
370problem.
371
372
Diego Calleja3f271002006-09-30 23:27:49 -0700373Managing bug reports
374--------------------
375
376One of the best ways to put into practice your hacking skills is by fixing
377bugs reported by other people. Not only you will help to make the kernel
378more stable, you'll learn to fix real world problems and you will improve
379your skills, and other developers will be aware of your presence. Fixing
Qi Yong4db29c12007-06-12 13:06:49 +0800380bugs is one of the best ways to get merits among other developers, because
Diego Calleja3f271002006-09-30 23:27:49 -0700381not many people like wasting time fixing other people's bugs.
382
Mauro Carvalho Chehabf1eebe92016-09-19 08:07:59 -0300383To work in the already reported bug reports, go to https://bugzilla.kernel.org.
Diego Calleja3f271002006-09-30 23:27:49 -0700384If you want to be advised of the future bug reports, you can subscribe to the
385bugme-new mailing list (only new bug reports are mailed here) or to the
386bugme-janitor mailing list (every change in the bugzilla is mailed here)
387
Mauro Carvalho Chehabf1eebe92016-09-19 08:07:59 -0300388 https://lists.linux-foundation.org/mailman/listinfo/bugme-new
389
390 https://lists.linux-foundation.org/mailman/listinfo/bugme-janitors
Diego Calleja3f271002006-09-30 23:27:49 -0700391
392
393
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800394Mailing lists
395-------------
396
397As some of the above documents describe, the majority of the core kernel
398developers participate on the Linux Kernel Mailing list. Details on how
399to subscribe and unsubscribe from the list can be found at:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300400
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800401 http://vger.kernel.org/vger-lists.html#linux-kernel
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300402
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800403There are archives of the mailing list on the web in many different
404places. Use a search engine to find these archives. For example:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300405
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800406 http://dir.gmane.org/gmane.linux.kernel
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300407
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800408It is highly recommended that you search the archives about the topic
409you want to bring up, before you post it to the list. A lot of things
410already discussed in detail are only recorded at the mailing list
411archives.
412
413Most of the individual kernel subsystems also have their own separate
414mailing list where they do their development efforts. See the
415MAINTAINERS file for a list of what these lists are for the different
416groups.
417
418Many of the lists are hosted on kernel.org. Information on them can be
419found at:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300420
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800421 http://vger.kernel.org/vger-lists.html
422
423Please remember to follow good behavioral habits when using the lists.
424Though a bit cheesy, the following URL has some simple guidelines for
425interacting with the list (or any list):
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300426
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800427 http://www.albion.com/netiquette/
428
429If multiple people respond to your mail, the CC: list of recipients may
430get pretty large. Don't remove anybody from the CC: list without a good
431reason, or don't reply only to the list address. Get used to receiving the
432mail twice, one from the sender and the one from the list, and don't try
433to tune that by adding fancy mail-headers, people will not like it.
434
435Remember to keep the context and the attribution of your replies intact,
436keep the "John Kernelhacker wrote ...:" lines at the top of your reply, and
437add your statements between the individual quoted sections instead of
438writing at the top of the mail.
439
440If you add patches to your mail, make sure they are plain readable text
Mauro Carvalho Chehab8c27ceff32016-10-18 10:12:27 -0200441as stated in Documentation/process/submitting-patches.rst.
Mauro Carvalho Chehab34fed7e2016-09-19 08:07:58 -0300442Kernel developers don't want to deal with
443attachments or compressed patches; they may want to comment on
444individual lines of your patch, which works only that way. Make sure you
445use a mail program that does not mangle spaces and tab characters. A
446good first test is to send the mail to yourself and try to apply your
447own patch by yourself. If that doesn't work, get your mail program fixed
448or change it until it works.
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800449
450Above all, please remember to show respect to other subscribers.
451
452
453Working with the community
454--------------------------
455
456The goal of the kernel community is to provide the best possible kernel
457there is. When you submit a patch for acceptance, it will be reviewed
458on its technical merits and those alone. So, what should you be
459expecting?
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300460
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800461 - criticism
462 - comments
463 - requests for change
464 - requests for justification
465 - silence
466
467Remember, this is part of getting your patch into the kernel. You have
468to be able to take criticism and comments about your patches, evaluate
469them at a technical level and either rework your patches or provide
470clear and concise reasoning as to why those changes should not be made.
471If there are no responses to your posting, wait a few days and try
472again, sometimes things get lost in the huge volume.
473
474What should you not do?
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300475
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800476 - expect your patch to be accepted without question
477 - become defensive
478 - ignore comments
479 - resubmit the patch without making any of the requested changes
480
481In a community that is looking for the best technical solution possible,
482there will always be differing opinions on how beneficial a patch is.
483You have to be cooperative, and willing to adapt your idea to fit within
484the kernel. Or at least be willing to prove your idea is worth it.
485Remember, being wrong is acceptable as long as you are willing to work
486toward a solution that is right.
487
488It is normal that the answers to your first patch might simply be a list
Mauro Carvalho Chehab34fed7e2016-09-19 08:07:58 -0300489of a dozen things you should correct. This does **not** imply that your
490patch will not be accepted, and it is **not** meant against you
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800491personally. Simply correct all issues raised against your patch and
492resend it.
493
494
495Differences between the kernel community and corporate structures
496-----------------------------------------------------------------
497
498The kernel community works differently than most traditional corporate
499development environments. Here are a list of things that you can try to
Cristian Stoica106f9d92012-12-07 16:37:11 +0200500do to avoid problems:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300501
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800502 Good things to say regarding your proposed changes:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300503
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800504 - "This solves multiple problems."
505 - "This deletes 2000 lines of code."
506 - "Here is a patch that explains what I am trying to describe."
507 - "I tested it on 5 different architectures..."
508 - "Here is a series of small patches that..."
509 - "This increases performance on typical machines..."
510
511 Bad things you should avoid saying:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300512
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800513 - "We did it this way in AIX/ptx/Solaris, so therefore it must be
514 good..."
515 - "I've being doing this for 20 years, so..."
516 - "This is required for my company to make money"
517 - "This is for our Enterprise product line."
518 - "Here is my 1000 page design document that describes my idea"
519 - "I've been working on this for 6 months..."
520 - "Here's a 5000 line patch that..."
521 - "I rewrote all of the current mess, and here it is..."
522 - "I have a deadline, and this patch needs to be applied now."
523
524Another way the kernel community is different than most traditional
525software engineering work environments is the faceless nature of
526interaction. One benefit of using email and irc as the primary forms of
527communication is the lack of discrimination based on gender or race.
528The Linux kernel work environment is accepting of women and minorities
529because all you are is an email address. The international aspect also
530helps to level the playing field because you can't guess gender based on
531a person's name. A man may be named Andrea and a woman may be named Pat.
532Most women who have worked in the Linux kernel and have expressed an
533opinion have had positive experiences.
534
535The language barrier can cause problems for some people who are not
536comfortable with English. A good grasp of the language can be needed in
537order to get ideas across properly on mailing lists, so it is
538recommended that you check your emails to make sure they make sense in
539English before sending them.
540
541
542Break up your changes
543---------------------
544
545The Linux kernel community does not gladly accept large chunks of code
546dropped on it all at once. The changes need to be properly introduced,
547discussed, and broken up into tiny, individual portions. This is almost
548the exact opposite of what companies are used to doing. Your proposal
549should also be introduced very early in the development process, so that
550you can receive feedback on what you are doing. It also lets the
551community feel that you are working with them, and not simply using them
552as a dumping ground for your feature. However, don't send 50 emails at
553one time to a mailing list, your patch series should be smaller than
554that almost all of the time.
555
556The reasons for breaking things up are the following:
557
5581) Small patches increase the likelihood that your patches will be
559 applied, since they don't take much time or effort to verify for
560 correctness. A 5 line patch can be applied by a maintainer with
561 barely a second glance. However, a 500 line patch may take hours to
562 review for correctness (the time it takes is exponentially
563 proportional to the size of the patch, or something).
564
565 Small patches also make it very easy to debug when something goes
566 wrong. It's much easier to back out patches one by one than it is
567 to dissect a very large patch after it's been applied (and broken
568 something).
569
5702) It's important not only to send small patches, but also to rewrite
571 and simplify (or simply re-order) patches before submitting them.
572
573Here is an analogy from kernel developer Al Viro:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300574
Mauro Carvalho Chehab34fed7e2016-09-19 08:07:58 -0300575 *"Think of a teacher grading homework from a math student. The
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800576 teacher does not want to see the student's trials and errors
577 before they came up with the solution. They want to see the
578 cleanest, most elegant answer. A good student knows this, and
579 would never submit her intermediate work before the final
Mauro Carvalho Chehab34fed7e2016-09-19 08:07:58 -0300580 solution.*
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800581
Mauro Carvalho Chehab34fed7e2016-09-19 08:07:58 -0300582 *The same is true of kernel development. The maintainers and
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800583 reviewers do not want to see the thought process behind the
584 solution to the problem one is solving. They want to see a
Mauro Carvalho Chehab34fed7e2016-09-19 08:07:58 -0300585 simple and elegant solution."*
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800586
587It may be challenging to keep the balance between presenting an elegant
588solution and working together with the community and discussing your
589unfinished work. Therefore it is good to get early in the process to
590get feedback to improve your work, but also keep your changes in small
591chunks that they may get already accepted, even when your whole task is
592not ready for inclusion now.
593
594Also realize that it is not acceptable to send patches for inclusion
595that are unfinished and will be "fixed up later."
596
597
598Justify your change
599-------------------
600
601Along with breaking up your patches, it is very important for you to let
602the Linux community know why they should add this change. New features
603must be justified as being needed and useful.
604
605
606Document your change
607--------------------
608
609When sending in your patches, pay special attention to what you say in
610the text in your email. This information will become the ChangeLog
611information for the patch, and will be preserved for everyone to see for
612all time. It should describe the patch completely, containing:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300613
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800614 - why the change is necessary
615 - the overall design approach in the patch
616 - implementation details
617 - testing results
618
619For more details on what this should all look like, please see the
620ChangeLog section of the document:
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300621
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800622 "The Perfect Patch"
SeongJae Park937d9f52013-12-06 18:04:42 +0900623 http://www.ozlabs.org/~akpm/stuff/tpp.txt
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800624
625
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800626All of these things are sometimes very hard to do. It can take years to
627perfect these practices (if at all). It's a continuous process of
628improvement that requires a lot of patience and determination. But
629don't give up, it's possible. Many have done it before, and each had to
630start exactly where you are now.
631
632
633
634
635----------
Mauro Carvalho Chehab022e04d2016-09-19 08:07:39 -0300636
Paolo Ciarrocchi5bd982e2006-04-25 22:47:51 +0200637Thanks to Paolo Ciarrocchi who allowed the "Development Process"
Mauro Carvalho Chehabf1eebe92016-09-19 08:07:59 -0300638(https://lwn.net/Articles/94386/) section
Greg Kroah-Hartmand36cc9d2005-11-18 09:31:11 -0800639to be based on text he had written, and to Randy Dunlap and Gerrit
640Huizenga for some of the list of things you should and should not say.
641Also thanks to Pat Mochel, Hanna Linder, Randy Dunlap, Kay Sievers,
642Vojtech Pavlik, Jan Kara, Josh Boyer, Kees Cook, Andrew Morton, Andi
643Kleen, Vadim Lobanov, Jesper Juhl, Adrian Bunk, Keri Harris, Frans Pop,
644David A. Wheeler, Junio Hamano, Michael Kerrisk, and Alex Shepard for
645their review, comments, and contributions. Without their help, this
646document would not have been possible.
647
648
649
650Maintainer: Greg Kroah-Hartman <greg@kroah.com>