[dpdk-test-report] |WARNING| pw29964 [PATCH v3 09/30] net/mlx5: add reference counter on memory region
checkpatch at dpdk.org
checkpatch at dpdk.org
Mon Oct 9 16:46:59 CEST 2017
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/29964
_coding style issues_
WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct priv *' should also have an identifier name
#65: FILE: drivers/net/mlx5/mlx5.h:305:
+struct mlx5_mr *priv_mr_new(struct priv *, struct rte_mempool *);
WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct rte_mempool *' should also have an identifier name
#65: FILE: drivers/net/mlx5/mlx5.h:305:
+struct mlx5_mr *priv_mr_new(struct priv *, struct rte_mempool *);
WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct priv *' should also have an identifier name
#66: FILE: drivers/net/mlx5/mlx5.h:306:
+struct mlx5_mr *priv_mr_get(struct priv *, struct rte_mempool *);
WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct rte_mempool *' should also have an identifier name
#66: FILE: drivers/net/mlx5/mlx5.h:306:
+struct mlx5_mr *priv_mr_get(struct priv *, struct rte_mempool *);
WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct priv *' should also have an identifier name
#67: FILE: drivers/net/mlx5/mlx5.h:307:
+int priv_mr_release(struct priv *, struct mlx5_mr *);
WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct mlx5_mr *' should also have an identifier name
#67: FILE: drivers/net/mlx5/mlx5.h:307:
+int priv_mr_release(struct priv *, struct mlx5_mr *);
WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct priv *' should also have an identifier name
#68: FILE: drivers/net/mlx5/mlx5.h:308:
+int priv_mr_verify(struct priv *);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'start > addr'
#256: FILE: drivers/net/mlx5/mlx5_mr.c:279:
+ if ((start > addr) && (start < addr + len))
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'end > addr'
#258: FILE: drivers/net/mlx5/mlx5_mr.c:281:
+ if ((end > addr) && (end < addr + len))
WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EBUSY)
#328: FILE: drivers/net/mlx5/mlx5_mr.c:351:
+ return EBUSY;
WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct mlx5_txq_data *' should also have an identifier name
#471: FILE: drivers/net/mlx5/mlx5_rxtx.h:352:
+struct mlx5_mr *mlx5_txq_mp2mr_reg(struct mlx5_txq_data *, struct rte_mempool *,
WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct rte_mempool *' should also have an identifier name
#471: FILE: drivers/net/mlx5/mlx5_rxtx.h:352:
+struct mlx5_mr *mlx5_txq_mp2mr_reg(struct mlx5_txq_data *, struct rte_mempool *,
WARNING:FUNCTION_ARGUMENTS: function definition argument 'unsigned int' should also have an identifier name
#471: FILE: drivers/net/mlx5/mlx5_rxtx.h:352:
+struct mlx5_mr *mlx5_txq_mp2mr_reg(struct mlx5_txq_data *, struct rte_mempool *,
total: 0 errors, 11 warnings, 2 checks, 466 lines checked
More information about the test-report
mailing list