Re: [patch 4/6] fs: d_delete change

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Linus Torvalds
Date: Thursday, November 11, 2010 - 3:07 pm

On Wed, Nov 10, 2010 at 4:27 PM, Nick Piggin <npiggin@kernel.dk> wrote:

The standard way is to just put it after the "---" thing. So you end
up having the actual kernel changelog at the top (which is, after all,
the summary and should be readable on its own, so making people read
that first makes perfect sense), and then put the extended explanation
that doesn't make sense for the changelog after the "---".

Of course, quite often you would want the commit message to be
exhaustive, so maybe the extended explanation ends up making sense for
the changelog too. But if it's more of a "discussion material" thing
(eg things like "Hi guys, this is _why_ I think this is the right
approach: ...") that way it's clear both to tools and by now to most
developers that the stuff that comes after the "---" is really
extended material.

                      Linus
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[patch 1/6] fs: icache RCU free inodes, Nick Piggin, (Tue Nov 9, 5:46 am)
[patch 3/6] fs: dcache documentation cleanup, Nick Piggin, (Tue Nov 9, 5:58 am)
[patch 4/6] fs: d_delete change, Nick Piggin, (Tue Nov 9, 6:01 am)
[patch 5/6] fs: d_compare change for rcu-walk, Nick Piggin, (Tue Nov 9, 6:02 am)
[patch 6/6] fs: d_hash change for rcu-walk, Nick Piggin, (Tue Nov 9, 6:03 am)
Re: [patch 1/6] fs: icache RCU free inodes, Andi Kleen, (Tue Nov 9, 7:19 am)
Re: [patch 1/6] fs: icache RCU free inodes, Linus Torvalds, (Tue Nov 9, 9:02 am)
Re: [patch 1/6] fs: icache RCU free inodes, Christoph Hellwig, (Tue Nov 9, 9:21 am)
Re: [patch 1/6] fs: icache RCU free inodes, Eric Dumazet, (Tue Nov 9, 9:21 am)
Re: [patch 3/6] fs: dcache documentation cleanup, Christoph Hellwig, (Tue Nov 9, 9:24 am)
Re: [patch 4/6] fs: d_delete change, Christoph Hellwig, (Tue Nov 9, 9:25 am)
Re: [patch 5/6] fs: d_compare change for rcu-walk, Christoph Hellwig, (Tue Nov 9, 9:25 am)
Re: [patch 1/6] fs: icache RCU free inodes, Linus Torvalds, (Tue Nov 9, 10:08 am)
Re: [patch 1/6] fs: icache RCU free inodes, Nick Piggin, (Tue Nov 9, 2:36 pm)
Re: [patch 1/6] fs: icache RCU free inodes, Nick Piggin, (Tue Nov 9, 2:44 pm)
Re: [patch 1/6] fs: icache RCU free inodes, Nick Piggin, (Tue Nov 9, 2:48 pm)
Re: [patch 1/6] fs: icache RCU free inodes, Nick Piggin, (Tue Nov 9, 3:05 pm)
Re: [patch 3/6] fs: dcache documentation cleanup, Nick Piggin, (Tue Nov 9, 3:06 pm)
Re: [patch 4/6] fs: d_delete change, Nick Piggin, (Tue Nov 9, 3:08 pm)
Re: [patch 5/6] fs: d_compare change for rcu-walk, Nick Piggin, (Tue Nov 9, 6:48 pm)
Re: [patch 1/6] fs: icache RCU free inodes, Andi Kleen, (Wed Nov 10, 7:47 am)
Re: [patch 3/6] fs: dcache documentation cleanup, Christoph Hellwig, (Wed Nov 10, 9:27 am)
Re: [patch 4/6] fs: d_delete change, Christoph Hellwig, (Wed Nov 10, 9:32 am)
Re: [patch 4/6] fs: d_delete change, Nick Piggin, (Wed Nov 10, 5:27 pm)
Re: [patch 1/6] fs: icache RCU free inodes, Nick Piggin, (Wed Nov 10, 9:27 pm)
Re: [patch 4/6] fs: d_delete change, Linus Torvalds, (Thu Nov 11, 3:07 pm)
Re: [patch 1/6] fs: icache RCU free inodes, Nick Piggin, (Thu Nov 11, 6:24 pm)
Re: [patch 1/6] fs: icache RCU free inodes, Linus Torvalds, (Thu Nov 11, 9:48 pm)
Re: [patch 1/6] fs: icache RCU free inodes, Nick Piggin, (Thu Nov 11, 11:02 pm)
Re: [patch 1/6] fs: icache RCU free inodes, Nick Piggin, (Thu Nov 11, 11:49 pm)
Re: [patch 1/6] fs: icache RCU free inodes, Linus Torvalds, (Fri Nov 12, 10:33 am)
Re: [patch 1/6] fs: icache RCU free inodes, Nick Piggin, (Fri Nov 12, 4:17 pm)
Re: [patch 1/6] fs: icache RCU free inodes, Dave Chinner, (Sun Nov 14, 6:00 pm)
Re: [patch 1/6] fs: icache RCU free inodes, Nick Piggin, (Sun Nov 14, 9:21 pm)
Re: [patch 1/6] fs: icache RCU free inodes, Dave Chinner, (Mon Nov 15, 8:02 pm)
Re: [patch 1/6] fs: icache RCU free inodes, Nick Piggin, (Mon Nov 15, 8:49 pm)
Re: [patch 1/6] fs: icache RCU free inodes, Dave Chinner, (Tue Nov 16, 6:12 pm)
Re: [patch 1/6] fs: icache RCU free inodes, Nick Piggin, (Tue Nov 16, 10:56 pm)
Re: [patch 1/6] fs: icache RCU free inodes, Nick Piggin, (Tue Nov 16, 11:04 pm)