diff options
| author | Jonathan Corbet <corbet@lwn.net> | 2011-03-25 14:17:53 -0400 |
|---|---|---|
| committer | Jonathan Corbet <corbet@lwn.net> | 2011-03-25 16:30:31 -0400 |
| commit | 5c050fb96380a87a85aad9084b68fdcd2b84c193 (patch) | |
| tree | b1d0bf29716a4e8a0da6d4b9b96bfe9635b58271 /Documentation/development-process/6.Followthrough | |
| parent | 9cad7962704d617ab1e4ae304baaaa22d727932b (diff) | |
docs: update the development process document
Here's a set of changes updating Documentation/development-process. I have
update kernel releases and relevant statistics, added information for a
couple of tools, zapped some trailing white space, and generally tried to
make it more closely match the current state of affairs.
[Typo fixes from Joe Perches and Nicolas Kaiser incorporated]
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
Acked-by: Greg KH <greg@kroah.com>
Cc: Randy Dunlap <rdunlap@xenotime.net>
Diffstat (limited to 'Documentation/development-process/6.Followthrough')
| -rw-r--r-- | Documentation/development-process/6.Followthrough | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/Documentation/development-process/6.Followthrough b/Documentation/development-process/6.Followthrough index a8fba3d83a85..41d324a9420d 100644 --- a/Documentation/development-process/6.Followthrough +++ b/Documentation/development-process/6.Followthrough | |||
| @@ -66,6 +66,11 @@ be easy to become blinded by your own solution to a problem to the point | |||
| 66 | that you don't realize that something is fundamentally wrong or, perhaps, | 66 | that you don't realize that something is fundamentally wrong or, perhaps, |
| 67 | you're not even solving the right problem. | 67 | you're not even solving the right problem. |
| 68 | 68 | ||
| 69 | Andrew Morton has suggested that every review comment which does not result | ||
| 70 | in a code change should result in an additional code comment instead; that | ||
| 71 | can help future reviewers avoid the questions which came up the first time | ||
| 72 | around. | ||
| 73 | |||
| 69 | One fatal mistake is to ignore review comments in the hope that they will | 74 | One fatal mistake is to ignore review comments in the hope that they will |
| 70 | go away. They will not go away. If you repost code without having | 75 | go away. They will not go away. If you repost code without having |
| 71 | responded to the comments you got the time before, you're likely to find | 76 | responded to the comments you got the time before, you're likely to find |
| @@ -100,7 +105,7 @@ entry into a subsystem maintainer's tree. How that works varies from one | |||
| 100 | subsystem to the next; each maintainer has his or her own way of doing | 105 | subsystem to the next; each maintainer has his or her own way of doing |
| 101 | things. In particular, there may be more than one tree - one, perhaps, | 106 | things. In particular, there may be more than one tree - one, perhaps, |
| 102 | dedicated to patches planned for the next merge window, and another for | 107 | dedicated to patches planned for the next merge window, and another for |
| 103 | longer-term work. | 108 | longer-term work. |
| 104 | 109 | ||
| 105 | For patches applying to areas for which there is no obvious subsystem tree | 110 | For patches applying to areas for which there is no obvious subsystem tree |
| 106 | (memory management patches, for example), the default tree often ends up | 111 | (memory management patches, for example), the default tree often ends up |
| @@ -109,11 +114,10 @@ through the -mm tree. | |||
| 109 | 114 | ||
| 110 | Inclusion into a subsystem tree can bring a higher level of visibility to a | 115 | Inclusion into a subsystem tree can bring a higher level of visibility to a |
| 111 | patch. Now other developers working with that tree will get the patch by | 116 | patch. Now other developers working with that tree will get the patch by |
| 112 | default. Subsystem trees typically feed into -mm and linux-next as well, | 117 | default. Subsystem trees typically feed linux-next as well, making their |
| 113 | making their contents visible to the development community as a whole. At | 118 | contents visible to the development community as a whole. At this point, |
| 114 | this point, there's a good chance that you will get more comments from a | 119 | there's a good chance that you will get more comments from a new set of |
| 115 | new set of reviewers; these comments need to be answered as in the previous | 120 | reviewers; these comments need to be answered as in the previous round. |
| 116 | round. | ||
| 117 | 121 | ||
| 118 | What may also happen at this point, depending on the nature of your patch, | 122 | What may also happen at this point, depending on the nature of your patch, |
| 119 | is that conflicts with work being done by others turn up. In the worst | 123 | is that conflicts with work being done by others turn up. In the worst |
