Messages in this thread | | | Date | Thu, 1 Dec 2011 12:12:37 -0800 | From | "Darrick J. Wong" <> | Subject | Re: [PATCH v5.1 00/14] crc32c: Add faster algorithm and self-test code |
| |
On Wed, Nov 30, 2011 at 02:29:11PM -0800, Andrew Morton wrote: > On Mon, 28 Nov 2011 14:36:59 -0800 > "Darrick J. Wong" <djwong@us.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@systemfabricworks.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".
Okay, I'll massage the changelogs to give them more descriptive subjects, and fix the attribution chain. Thank you for the feedback.
--D > > -- > To unsubscribe from this list: send the line "unsubscribe linux-ext4" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >
| |