On 05/15/2014 08:09 AM, chenj wrote: > This will bring at most 50% performance gain on loongson3a. > See > http://dev.lemote.com/files/upload/software/csum-opti/csum-opti-benchmark.html > > The benchmark is done in userspace through > http://dev.lemote.com/files/upload/software/csum-opti/csum-test.tar.gz > --- > arch/mips/lib/csum_partial.S | 38 +++++++++++++++++++------------------- > 1 file changed, 19 insertions(+), 19 deletions(-) Hi chenj, My opinion is that the commit message is not ideal. If the http links ever go away, the commit message will be mostly useless for someone trying to understand the reason for this patch. I would suggest to explain the reason for the optimizations in the commit message and put the http links as a comment to this patch which will still be visible in the mailing list archives. Or you can keep them in the commit message but I think the reason for the patch should be explained as well. -- markos
