|WARNING| pw135322 [PATCH] dpdk: Enable GQ stats reporting

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Dec 19 03:05:50 CET 2023


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

_coding style issues_


ERROR:GERRIT_CHANGE_ID: Remove Gerrit Change-Id's before submitting upstream
#78: 
Change-Id: I8123d70ec5019b12f368e67b858c382bb7812e5c

ERROR:CODE_INDENT: code indent should use tabs where possible
#116: FILE: drivers/net/gve/gve_ethdev.c:129:
+               uint16_t nb_tx_queues, uint16_t nb_rx_queues)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#116: FILE: drivers/net/gve/gve_ethdev.c:129:
+               uint16_t nb_tx_queues, uint16_t nb_rx_queues)$

ERROR:CODE_INDENT: code indent should use tabs where possible
#147: FILE: drivers/net/gve/gve_ethdev.c:160:
+        struct gve_priv *priv = dev->data->dev_private;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#147: FILE: drivers/net/gve/gve_ethdev.c:160:
+        struct gve_priv *priv = dev->data->dev_private;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#148: FILE: drivers/net/gve/gve_ethdev.c:161:
+        rte_memzone_free(priv->stats_report_mem);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#148: FILE: drivers/net/gve/gve_ethdev.c:161:
+        rte_memzone_free(priv->stats_report_mem);$

ERROR:OPEN_BRACE: that open brace { should be on the previous line
#196: FILE: drivers/net/gve/gve_ethdev.c:257:
+	if (gve_is_gqi(priv))
+	{

ERROR:CODE_INDENT: code indent should use tabs where possible
#199: FILE: drivers/net/gve/gve_ethdev.c:260:
+                       dev->data->nb_tx_queues,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#199: FILE: drivers/net/gve/gve_ethdev.c:260:
+                       dev->data->nb_tx_queues,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#200: FILE: drivers/net/gve/gve_ethdev.c:261:
+                       dev->data->nb_rx_queues);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#200: FILE: drivers/net/gve/gve_ethdev.c:261:
+                       dev->data->nb_rx_queues);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#202: FILE: drivers/net/gve/gve_ethdev.c:263:
+                                      priv->stats_report_mem->iova,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#202: FILE: drivers/net/gve/gve_ethdev.c:263:
+                                      priv->stats_report_mem->iova,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#203: FILE: drivers/net/gve/gve_ethdev.c:264:
+                                      GVE_STATS_REPORT_TIMER_PERIOD);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#203: FILE: drivers/net/gve/gve_ethdev.c:264:
+                                      GVE_STATS_REPORT_TIMER_PERIOD);$

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

total: 10 errors, 7 warnings, 0 checks, 161 lines checked


More information about the test-report mailing list