Re: [PATCH 0/6]IA64 ftrace port

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 9 Jan 2009, Shaohua Li wrote:

> This series are the ia64 ftrace porting. The patches hasn't functional
> changes against my repvious post. I just split them for merge. Patches
> are against latest -tip tree.
> 
> Steve,
> the first 4 patches are preparation patch and not in IA64 arch.
> the last two are IA64 specific.

Thanks, Just a quick note on change logs and subject lines.

The following is needed, in the subject have the subsystem name included.

ie.

	Subject: [PATCH] ia64, ftrace: ia64 ftrace port

If it is just one subsystem you only need:

	Subject: [PATCH] ia64: ...

But since this also has to deal with ftrace, do the former instead.

Also, we've been starting to add Impact lines to the change log. But I'm 
not too picky on that.

But, please do not lightly reference another patch in the log of a 
patch. Each change log needs to stand on its own, if someone does a
"git blame" and looks at the log, they will get quite upset if they see 
something like:

	This is needed for the same reason as the previous patch.

Thanks,

-- Steve

--
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [Sparc Linux]     [DCCP]     [Linux ARM]     [Yosemite News]     [Linux SCSI]     [Linux x86_64]     [Linux for Ham Radio]

  Powered by Linux