[dpdk-test-report] |WARNING| pw39411 [PATCH v6 07/11] app/testpmd: fix RSS flow action configuration

checkpatch at dpdk.org checkpatch at dpdk.org
Mon May 7 07:24:17 CEST 2018


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

_coding style issues_


WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#20: 
There is a core dump when I set i40e fdir flexbytes rule, I bisect the commit version,

ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 0123456789ab ("commit description")'
#21: 
d0ad8648b1c57c0e311ab7a3192bc3b507de5bf6 is the first bad commit

ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 0123456789ab ("commit description")'
#22: 
commit d0ad8648b1c57c0e311ab7a3192bc3b507de5bf6

WARNING:BAD_SIGN_OFF: Do not use whitespace before Cc:
#43: 
    Cc: stable at dpdk.org

WARNING:BAD_SIGN_OFF: Do not use whitespace before Signed-off-by:
#45: 
    Signed-off-by: Adrien Mazarguil <adrien.mazarguil at 6wind.com>

WARNING:BAD_SIGN_OFF: Do not use whitespace before Acked-by:
#46: 
    Acked-by: Nelio Laranjeiro <nelio.laranjeiro at 6wind.com>

WARNING:BAD_SIGN_OFF: Do not use whitespace before Acked-by:
#47: 
    Acked-by: Ferruh Yigit <ferruh.yigit at intel.com>

WARNING:BAD_SIGN_OFF: Duplicate signature
#79: 
Cc: stable at dpdk.org

WARNING:BAD_SIGN_OFF: Duplicate signature
#81: 
Signed-off-by: Adrien Mazarguil <adrien.mazarguil at 6wind.com>

WARNING:BAD_SIGN_OFF: Duplicate signature
#82: 
Acked-by: Nelio Laranjeiro <nelio.laranjeiro at 6wind.com>

ERROR:TRAILING_WHITESPACE: trailing whitespace
#260: FILE: app/test-pmd/cmdline_flow.c:187:
+/** Maximum number of queue indices in struct rte_flow_action_rss. */ $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#263: FILE: app/test-pmd/cmdline_flow.c:190:
+/** Storage for struct rte_flow_action_rss including external data. */ $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#418: FILE: app/test-pmd/config.c:1006:
+/** Compute storage space needed by item specification and copy it. */ $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#419: FILE: app/test-pmd/config.c:1007:
+static size_t flow_item_spec_copy(void *buf, const struct rte_flow_item $

total: 6 errors, 8 warnings, 238 lines checked


More information about the test-report mailing list