pdump: fix log messages
authorReshma Pattan <reshma.pattan@intel.com>
Thu, 10 Nov 2016 16:29:40 +0000 (16:29 +0000)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Sat, 12 Nov 2016 21:27:09 +0000 (22:27 +0100)
The ethdev Rx/Tx remove callback apis doesn't set rte_errno during
failures, instead they just return negative error number, so using
that number in logs instead of rte_errno upon Rx and Tx callback
removal failures.

Fixes: 278f9454 ("pdump: add new library for packet capture")

Signed-off-by: Reshma Pattan <reshma.pattan@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
lib/librte_pdump/rte_pdump.c

index 504a1ce..5968683 100644 (file)
@@ -269,7 +269,7 @@ pdump_regitser_rx_callbacks(uint16_t end_q, uint8_t port, uint16_t queue,
                        if (ret < 0) {
                                RTE_LOG(ERR, PDUMP,
                                        "failed to remove rx callback, errno=%d\n",
-                                       rte_errno);
+                                       -ret);
                                return ret;
                        }
                        cbs->cb = NULL;
@@ -324,7 +324,7 @@ pdump_regitser_tx_callbacks(uint16_t end_q, uint8_t port, uint16_t queue,
                        if (ret < 0) {
                                RTE_LOG(ERR, PDUMP,
                                        "failed to remove tx callback, errno=%d\n",
-                                       rte_errno);
+                                       -ret);
                                return ret;
                        }
                        cbs->cb = NULL;