Re: [ovs-dev] [PATCHv3 1/5] lib: Add CRC32C Implementation

2013-06-12 Thread Ben Pfaff
Nope. On Thu, Jun 13, 2013 at 08:58:31AM +0900, Joe Stringer wrote: > No reason. I wasn't sure whether there were Debian-specific policies on > this or something. > > > On Thu, Jun 13, 2013 at 7:22 AM, Ben Pfaff wrote: > > > On Wed, Jun 12, 2013 at 02:35:54PM +0900, Joe Stringer wrote: > > > T

Re: [ovs-dev] [PATCHv3 1/5] lib: Add CRC32C Implementation

2013-06-12 Thread Joe Stringer
No reason. I wasn't sure whether there were Debian-specific policies on this or something. On Thu, Jun 13, 2013 at 7:22 AM, Ben Pfaff wrote: > On Wed, Jun 12, 2013 at 02:35:54PM +0900, Joe Stringer wrote: > > This implementation was derived from FreeBSD: > > http://code.google.com/p/freebsd-hea

Re: [ovs-dev] [PATCHv3 1/5] lib: Add CRC32C Implementation

2013-06-12 Thread Ben Pfaff
On Wed, Jun 12, 2013 at 02:35:54PM +0900, Joe Stringer wrote: > This implementation was derived from FreeBSD: > http://code.google.com/p/freebsd-head/source/browse/sys/libkern/crc32.c > > Reviewed-by: Simon Horman > Signed-off-by: Joe Stringer Looks good, but is there any reason not to fold pat

[ovs-dev] [PATCHv3 1/5] lib: Add CRC32C Implementation

2013-06-11 Thread Joe Stringer
This implementation was derived from FreeBSD: http://code.google.com/p/freebsd-head/source/browse/sys/libkern/crc32.c Reviewed-by: Simon Horman Signed-off-by: Joe Stringer --- v3: Rebase Return crc32c checksum in network byte-order Add checksum tests based on Intel IPPs documentation v2: