Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Sun, 28 Oct 2018 05:52:33 -0400
From: Steven Rostedt <rostedt@...dmis.org>
To: Igor Stoppa <igor.stoppa@...il.com>
Cc: Mathieu Desnoyers <mathieu.desnoyers@...icios.com>, Mimi Zohar
 <zohar@...ux.vnet.ibm.com>, Kees Cook <keescook@...omium.org>, Matthew
 Wilcox <willy@...radead.org>, Dave Chinner <david@...morbit.com>, James
 Morris <jmorris@...ei.org>, Michal Hocko <mhocko@...nel.org>,
 kernel-hardening@...ts.openwall.com, linux-integrity@...r.kernel.org,
 linux-security-module@...r.kernel.org, igor stoppa
 <igor.stoppa@...wei.com>, Dave Hansen <dave.hansen@...ux.intel.com>,
 Jonathan Corbet <corbet@....net>, Laura Abbott <labbott@...hat.com>, Thomas
 Gleixner <tglx@...utronix.de>, Kate Stewart <kstewart@...uxfoundation.org>,
 "David S. Miller" <davem@...emloft.net>, Greg Kroah-Hartman
 <gregkh@...uxfoundation.org>, Philippe Ombredanne <pombredanne@...b.com>,
 "Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>, Josh Triplett
 <josh@...htriplett.org>, Lai Jiangshan <jiangshanlai@...il.com>,
 linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 14/17] prmem: llist, hlist, both plain and rcu

On Wed, 24 Oct 2018 17:03:01 +0300
Igor Stoppa <igor.stoppa@...il.com> wrote:

> I was hoping that by CC-ing kernel.org, the explicit recipients would 
> get both the mail directly intended for them (as subsystem 
> maintainers/supporters) and the rest.
> 
> The next time I'll err in the opposite direction.

Please don't.

> 
> In the meanwhile, please find the whole set here:
> 
> https://www.openwall.com/lists/kernel-hardening/2018/10/23/3

Note, it is critical that every change log stands on its own. You do
not need to Cc the entire patch set to everyone. Each patch should have
enough information in it to know exactly what the patch does.

It's OK if each change log has duplicate information from other
patches. The important part is that one should be able to look at the
change log of a specific patch and understand exactly what the patch is
doing.

This is because 5 years from now, if someone does a git blame and comes
up with this commit, they wont have access to the patch series. All
they will have is this single commit log to explain why these changes
were done.

If a change log depends on other commits for context, it is
insufficient.

Thanks,

-- Steve

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.