linux-ext4 - Re: [PATCH v5.1 00/14] crc32c: Add faster algorithm and self-test code

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20111130142911.673c2d99.akpm@linux-foundation.org>
Date:	Wed, 30 Nov 2011 14:29:11 -0800
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	"Darrick J. Wong" <djwong@...ibm.com>
Cc:	Herbert Xu <herbert@...dor.hengli.com.au>,
	Theodore Tso <tytso@....edu>,
	Joakim Tjernlund <joakim.tjernlund@...nsmode.se>,
	Bob Pearson <rpearson@...temfabricworks.com>,
	linux-kernel <linux-kernel@...r.kernel.org>,
	Andreas Dilger <adilger.kernel@...ger.ca>,
	linux-crypto <linux-crypto@...r.kernel.org>,
	linux-fsdevel <linux-fsdevel@...r.kernel.org>,
	Mingming Cao <cmm@...ibm.com>, linux-ext4@...r.kernel.org
Subject: Re: [PATCH v5.1 00/14] crc32c: Add faster algorithm and self-test
 code

On Mon, 28 Nov 2011 14:36:59 -0800
"Darrick J. Wong" <djwong@...ibm.com> wrote:

> This patchset (re)uses Bob Pearson's crc32 slice-by-8 code to stamp out a
> software crc32c implementation.

I think the attributions here are all messed up.  As the patches stand,
it appears that you wrote all of them.  But I don't think that is the
case.

If Bob wrote a particular patch then that patch should be sent with a
From: Bob Pearson <rpearson@...temfabricworks.com> right at the start
of the changelog so that he is recorded as the primary author.  If the
email sender (ie: you) was the primary author then this attribution can
be omitted and we fall back to using the From: from the email headers.

Also, every one of these patches should have you own signed-off-by,
regardless of its authorship.  For reasons explained in
Documentation/SubmittingPatches, section 12.

Please fix these things up and resend.

Also, it would be conventional and useful if each patch title was
prefixed by its subsystem identifier.  ie, "removed two instances of
trailing whitespaces" should be titled "crc32: remove two instances of
trailing whitespace" or "lib/crc32.c: remove two instances of trailing
whitespaces".

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