2005-04-17 06:20:36 +08:00
|
|
|
/*
|
|
|
|
|
* There are multiple 16-bit CRC polynomials in common use, but this is
|
|
|
|
|
* *the* standard CRC-32 polynomial, first popularized by Ethernet.
|
|
|
|
|
* x^32+x^26+x^23+x^22+x^16+x^12+x^11+x^10+x^8+x^7+x^5+x^4+x^2+x^1+x^0
|
|
|
|
|
*/
|
|
|
|
|
#define CRCPOLY_LE 0xedb88320
|
|
|
|
|
#define CRCPOLY_BE 0x04c11db7
|
|
|
|
|
|
2012-03-24 06:02:23 +08:00
|
|
|
/* How many bits at a time to use. Valid values are 1, 2, 4, 8, and 32. */
|
|
|
|
|
/* For less performance-sensitive, use 4 or 8 */
|
crc32: remove two instances of trailing whitespaces
This patchset (re)uses Bob Pearson's crc32 slice-by-8 code to stamp out
a software crc32c implementation. It removes the crc32c implementation
in crypto/ in favor of using the stamped-out one in lib/. There is also
a change to Kconfig so that the kernel builder can pick an
implementation best suited for the hardware.
The motivation for this patchset is that I am working on adding full
metadata checksumming to ext4. As far as performance impact of adding
checksumming goes, I see nearly no change with a standard mail server
ffsb simulation. On a test that involves only file creation and
deletion and extent tree writes, I see a drop of about 50 pcercent with
the current kernel crc32c implementation; this improves to a drop of
about 20 percent with the enclosed crc32c code.
When metadata is usually a small fraction of total IO, this new
implementation doesn't help much because metadata is usually a small
fraction of total IO. However, when we are doing IO that is almost all
metadata (such as rm -rf'ing a tree), then this patch speeds up the
operation substantially.
Incidentally, given that iscsi, sctp, and btrfs also use crc32c, this
patchset should improve their speed as well. I have not yet quantified
that, however. This latest submission combines Bob's patches from late
August 2011 with mine so that they can be one coherent patch set.
Please excuse my inability to combine some of the patches; I've been
advised to leave Bob's patches alone and build atop them instead. :/
Since the last posting, I've also collected some crc32c test results on
a bunch of different x86/powerpc/sparc platforms. The results can be
viewed here: http://goo.gl/sgt3i ; the "crc32-kern-le" and "crc32c"
columns describe the performance of the kernel's current crc32 and
crc32c software implementations. The "crc32c-by8-le" column shows
crc32c performance with this patchset applied. I expect crc32
performance to be roughly the same.
The two _boost columns at the right side of the spreadsheet shows how much
faster the new implementation is over the old one. As you can see, crc32
rises substantially, and crc32c experiences a huge increase.
This patch:
- remove trailing whitespace from lib/crc32.c
- remove trailing whitespace from lib/crc32defs.h
[djwong@us.ibm.com: changelog tweaks]
Signed-off-by: Bob Pearson <rpearson@systemfabricworks.com>
Signed-off-by: Darrick J. Wong <djwong@us.ibm.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2012-03-24 06:02:21 +08:00
|
|
|
#ifndef CRC_LE_BITS
|
2012-03-24 06:02:23 +08:00
|
|
|
# define CRC_LE_BITS 32
|
2005-04-17 06:20:36 +08:00
|
|
|
#endif
|
|
|
|
|
#ifndef CRC_BE_BITS
|
2012-03-24 06:02:23 +08:00
|
|
|
# define CRC_BE_BITS 32
|
2005-04-17 06:20:36 +08:00
|
|
|
#endif
|
|
|
|
|
|
|
|
|
|
/*
|
|
|
|
|
* Little-endian CRC computation. Used with serial bit streams sent
|
|
|
|
|
* lsbit-first. Be sure to use cpu_to_le32() to append the computed CRC.
|
|
|
|
|
*/
|
2012-03-24 06:02:23 +08:00
|
|
|
#if CRC_LE_BITS > 32 || CRC_LE_BITS < 1 || CRC_LE_BITS == 16 || \
|
|
|
|
|
CRC_LE_BITS & CRC_LE_BITS-1
|
|
|
|
|
# error "CRC_LE_BITS must be one of {1, 2, 4, 8, 32}"
|
2005-04-17 06:20:36 +08:00
|
|
|
#endif
|
|
|
|
|
|
|
|
|
|
/*
|
|
|
|
|
* Big-endian CRC computation. Used with serial bit streams sent
|
|
|
|
|
* msbit-first. Be sure to use cpu_to_be32() to append the computed CRC.
|
|
|
|
|
*/
|
2012-03-24 06:02:23 +08:00
|
|
|
#if CRC_BE_BITS > 32 || CRC_BE_BITS < 1 || CRC_BE_BITS == 16 || \
|
|
|
|
|
CRC_BE_BITS & CRC_BE_BITS-1
|
|
|
|
|
# error "CRC_BE_BITS must be one of {1, 2, 4, 8, 32}"
|
2005-04-17 06:20:36 +08:00
|
|
|
#endif
|