[dpdk-test-report] |WARNING| pw65559 [RFC] Accelerator API to chain packet processing functions

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Feb 4 15:46:41 CET 2020


Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/65559

_coding style issues_


WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#57: 
The main aim of this library is to provide a flexible and extensible way of combining one or more packet-processing functions into a single operation, thereby allowing these to be performed in parallel in optimized software libraries or in a hardware accelerator. These functions can include cryptography, compression and CRC/checksum calculation, while others can potentially be added in the future. Performing these functions in parallel as a single operation can enable a significant performance improvement.

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#563: FILE: lib/librte_accelerator/rte_accelerator.h:198:
+ * ^I^I^Ichain$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#598: FILE: lib/librte_accelerator/rte_accelerator.h:233:
+ * ^I^I^Ibitmask allows this function determine the maximum size$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#599: FILE: lib/librte_accelerator/rte_accelerator.h:234:
+ * ^I^I^Iof operation which must be accommodated in the mempool$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#600: FILE: lib/librte_accelerator/rte_accelerator.h:235:
+ * ^I^I^Ielements. See ::rte_accelerator_op_type for possible$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#601: FILE: lib/librte_accelerator/rte_accelerator.h:236:
+ * ^I^I^Ibitmask values$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#604: FILE: lib/librte_accelerator/rte_accelerator.h:239:
+ * ^I^I^Irte_mempool_create() for further details about cache$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#605: FILE: lib/librte_accelerator/rte_accelerator.h:240:
+ * ^I^I^Isize$

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#617: FILE: lib/librte_accelerator/rte_accelerator.h:252:
+			       unsigned nb_elts,

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#618: FILE: lib/librte_accelerator/rte_accelerator.h:253:
+			       unsigned cache_size,

ERROR:CODE_INDENT: code indent should use tabs where possible
#1004: FILE: lib/librte_compressdev/rte_compressdev_internal.h:83:
+        /**< Context for accelerator ops */$

ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)

total: 2 errors, 10 warnings, 650 lines checked


More information about the test-report mailing list