summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | fix comment/message on GF_E_SP128_A/GF_E_SP128_SDanny Al-Gaaf2014-04-222-4/+4
| | | | | | | | | | | | | | | | | | Swap comments/messages on GF_E_SP128_A/GF_E_SP128_S. Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
| * | gf_w8.c: add missing breaksDanny Al-Gaaf2014-04-221-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since there is no comment indicating fallthrough on purpose added a break in switch value 5 and 6. Fix for coverity issue from Ceph project: CID 1193084 (#1 of 1): Missing break in switch (MISSING_BREAK) unterminated_case: This case (value 5) is not terminated by a 'break' statement. Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
| * | gf_w4.c: add missing breaksDanny Al-Gaaf2014-04-221-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since there is no comment indicating fallthrough on purpose added a break in switch value 5 and 6. Fix for coverity issue from Ceph project: CID 1193082 (#1 of 1): Missing break in switch (MISSING_BREAK) unterminated_case: This case (value 5) is not terminated by a 'break' statement. CID 1193083 (#1 of 1): Missing break in switch (MISSING_BREAK) unterminated_case: This case (value 6) is not terminated by a 'break' statement. Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
| * | gf_w32.c: add missing breaksDanny Al-Gaaf2014-04-221-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since there is no comment indicating fallthrough on purpose added a break in switch value 3 and 5/before default. Fix for coverity issue from Ceph project: CID 1193080 (#1 of 1): Missing break in switch (MISSING_BREAK) unterminated_case: This case (value 3) is not terminated by a 'break' statement. CID 1193081 (#1 of 1): Missing break in switch (MISSING_BREAK) unterminated_case: This case (value 5) is not terminated by a 'break' statement. Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
| * | gf_w16.c: add missing breakDanny Al-Gaaf2014-04-221-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since there is no comment indicating fallthrough on purpose added a break in switch value 5/before default. Fix for coverity issue from Ceph project: CID 1193079 (#1 of 1): Missing break in switch (MISSING_BREAK) unterminated_case: This case (value 5) is not terminated by a 'break' statement. Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
| * | gf_general.c: fix pointless expressionDanny Al-Gaaf2014-04-221-1/+1
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of checking w128[0] twice check for w128[0] and w128[1]. Fix for coverity issue from Ceph project: CID 1193072 (#1 of 1): Same on both sides (CONSTANT_EXPRESSION_RESULT) pointless_expression: The expression v1->w128[0] == v2->w128[0] && v1->w128[0] == v2->w128[0] does not accomplish anything because it evaluates to either of its identical operands, v1->w128[0] == v2->w128[0]. Did you intend the operands to be different? Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
* | Merged in dachary/gf-complete/wip-uint-8 (pull request #19) Kevin Greenan2014-04-106-29/+29
|\ \ | | | | | | prefer uint8_t to char in pointer arithmetic
| * | prefer uint8_t to char in pointer arithmeticLoic Dachary2014-04-105-28/+28
| | | | | | | | | | | | Signed-off-by: Loic Dachary <loic@dachary.org>
| * | Show pointer arithmetic warnings by defaultLoic Dachary2014-04-101-1/+1
| | | | | | | | | | | | | | | | | | So that a void* being used in pointer arithmetic does not go unnoticed. Signed-off-by: Loic Dachary <loic@dachary.org>
* | | Merged in dachary/gf-complete/wip-hard-coded-note (pull request #18) Kevin Greenan2014-04-101-0/+1
|\ \ \ | |/ / |/| | TODO reminder for KMG/JSP about hardcoded constant
| * | TODO reminder for KMG/JSP about hardcoded constantLoic Dachary2014-04-101-0/+1
|/ / | | | | | | Signed-off-by: Loic Dachary <loic@dachary.org>
* | Ran autogen to pick-up the changes needed to run 'make check'Kevin Greenan2014-04-0220-7655/+12881
| |
* | Merged in dachary/gf-complete/wip-make-check (pull request #11) Kevin Greenan2014-04-023-0/+15
|\ \ | | | | | | add make check target and basic tests
| * | add make check target and basic testsLoic Dachary2014-04-023-0/+15
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To conveniently run tests as $ make check ============================================================================ Testsuite summary for gf-complete 1.0 ============================================================================ # TOTAL: 1 # PASS: 1 # SKIP: 0 # XFAIL: 0 # FAIL: 0 # XPASS: 0 # ERROR: 0 ============================================================================ The run-tests.sh script loops over gf_methods and is introduced because autotools does not allow tests to have parameters in the Makefile.am Signed-off-by: Loic Dachary <loic@dachary.org>
* | Merged in dachary/gf-complete/wip-compilation-warnings (pull request #12) Kevin Greenan2014-03-311-1/+1
|\ \ | | | | | | fix void* arithmetic compilation warning
| * | fix void* arithmetic compilation warningLoic Dachary2014-03-291-1/+1
| | | | | | | | | | | | Signed-off-by: Loic Dachary <loic@dachary.org>
* | | Merged in dachary/gf-complete/wip-sse4 (pull request #13) Kevin Greenan2014-03-311-2/+2
|\ \ \ | | | | | | | | do not compile if used in SSE4 code path only
| * | | do not compile if used in SSE4 code path onlyLoic Dachary2014-03-301-2/+2
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Acknowledge that gf_w128_split_4_128_multiply_region and gf_w128_split_4_128_sse_multiply_region are only used when the INTEL_SSE4 flag is present, even though they only need INTEL_SSSE3 It suppresses a compilation warning complaining about them not being used if INTEL_SSE4 is absent and INTEL_SSSE3 is present. Signed-off-by: Loic Dachary <loic@dachary.org>
* | | Merged in dachary/gf-complete/wip-gitignore (pull request #10) Kevin Greenan2014-03-311-0/+4
|\ \ \ | |/ / |/| | .gitignore: ignore autotools stuff
| * | .gitignore: ignore autotools stuffSage Weil2014-03-281-0/+4
|/ / | | | | | | | | Signed-off-by: Sage Weil <sage@inktank.com> (cherry picked from commit c70286782425b1f792313e4fd1ce1773c72dcd76)
* | Merged in dachary/gf-complete/wip-gitignore (pull request #6) Kevin Greenan2014-03-281-0/+1
|\ \ | | | | | | create a .gitignore
| * | create a .gitignoreLoic Dachary2014-03-191-0/+1
| |/ | | | | | | Signed-off-by: Loic Dachary <loic@dachary.org>
* | Merged in dachary/gf-complete/wip-compilation-warnings (pull request #3) Kevin Greenan2014-03-2824-724/+304
|\ \ | |/ |/| fix compilation warnings
| * remove unused static function gf_w128_group_r_sse_initLoic Dachary2014-03-281-1/+1
| | | | | | | | Signed-off-by: Loic Dachary <loic@dachary.org>
| * remove unused argument from SSE_AB2Loic Dachary2014-03-061-23/+23
| | | | | | | | Signed-off-by: Loic Dachary <loic@dachary.org>
| * main() returns intLoic Dachary2014-03-066-1/+6
| | | | | | | | Signed-off-by: Loic Dachary <loic@dachary.org>
| * remove unused variables from #if SSE blocsLoic Dachary2014-03-0610-141/+76
| | | | | | | | Signed-off-by: Loic Dachary <loic@dachary.org>
| * remove spurious i < in for loopLoic Dachary2014-03-061-1/+1
| | | | | | | | Signed-off-by: Loic Dachary <loic@dachary.org>
| * do not use gf_internal_t when it is nullLoic Dachary2014-03-061-1/+1
| | | | | | | | Signed-off-by: Loic Dachary <loic@dachary.org>
| * silence some warnings for unset variablesLoic Dachary2014-03-065-8/+11
| | | | | | | | | | | | | | By setting them when they are defined and when the compiler fails to see the logic is ok. Signed-off-by: Loic Dachary <loic@dachary.org>
| * remove duplicate headers from srcLoic Dachary2014-03-063-283/+0
| | | | | | | | Signed-off-by: Loic Dachary <loic@dachary.org>
| * move #if to avoid unused warningLoic Dachary2014-03-066-108/+104
| | | | | | | | Signed-off-by: Loic Dachary <loic@dachary.org>
| * initialize pointer early in the functionLoic Dachary2014-03-061-1/+2
| | | | | | | | | | | | otherwise it may be used uninitialized Signed-off-by: Loic Dachary <loic@dachary.org>
| * cast void* to char* for pointer arithmeticLoic Dachary2014-03-066-33/+34
| | | | | | | | Signed-off-by: Loic Dachary <loic@dachary.org>
| * remove unused variablesLoic Dachary2014-03-0610-122/+45
| | | | | | | | | | | | | | In some places move variables in the scope of the CPP define where they are used. Signed-off-by: Loic Dachary <loic@dachary.org>
| * increase compilation warning level to -WallLoic Dachary2014-03-061-2/+1
|/ | | | | | and remove unused CXXFLAGS Signed-off-by: Loic Dachary <loic@dachary.org>
* FormattingJim Plank2014-01-292-2/+18
|
* FormattingJim Plank2014-01-291-0/+1
|
* Updated manual and readme.Jim Plank2014-01-293-9/+12
|
* Changed %d to %u in gf_general.cJim Plank2014-01-251-2/+2
|
* Updated autoconf to put BSD license in COPYING.Kevin Greenan2014-01-072-675/+33
|
* Added more header files to the distribution, which will allowKevin Greenan2014-01-023-5/+2
| | | | clients of the lib to take advantage of even more stuff.
* Revision 1.02 is done. Manual updated.Jim Plank2014-01-013-2/+2
|
* Removed GROUP/128/SSE. It wasn't compiling, and it needed an overhaul.Jim Plank2014-01-015-303/+3
| | | | I'll do it someday when I'm bored.
* Fixed the problem with PCLMUL and gf_complete.h. RemovedJim Plank2013-12-317-48/+45
| | | | | ARCH_64 from everything but 128/GROUP/SSE. Fortunately, no one ever uses that.
* Make INTEL_SSE4_PCLMUL flag consistent (was INTEL_PCLMUL in gf.c)Kevin Greenan2013-12-301-1/+1
|
* Third.1 time's a charm (autoconf non-sense for PCLMUL).Kevin Greenan2013-12-3024-9785/+7587
|
* Revert "Third time's a charm (autoconf non-sense for PCLMUL)."Kevin Greenan2013-12-3021-1156/+1426
| | | | | | The commit was not successfully pushed (not sure what happened). This reverts commit 762926920aa6b0067443e7c085bddf22d03a1d04.
* Third time's a charm (autoconf non-sense for PCLMUL).Kevin Greenan2013-12-3021-1426/+1156
|
* Added entry to configure.ac to avoid running autotools during normal build.Kevin Greenan2013-12-309-16/+99
|