eal/linux: fix device monitor stop return

The ret value in rte_dev_event_monitor_stop stands for whether the
monitor has been successfully closed, and should not bind with
rte_intr_callback_unregister, so once it goes to the right exit point of
rte_dev_event_monitor, the ret value should be set to 0.

Also, the refmonitor count has been carefully evaluated, the value
change from 1 to 0, so there is no potential memory leak failure.

Fixes: 1fef6ced07f3 ("eal/linux: allow multiple starts of event monitor")
Cc: stable@dpdk.org
Signed-off-by: Wenxuan Wu <wenxuanx.wu@intel.com>

diff --git a/lib/eal/linux/eal_dev.c b/lib/eal/linux/eal_dev.c
index e6f509bcfff58914ceb0d8bb26322c10bfabf7b4..02ae1cde29d0b09a4a5aae3a98bd2af5aa542562 100644 (file)
--- a/lib/eal/linux/eal_dev.c
+++ b/lib/eal/linux/eal_dev.c
@@ -380,6 +380,7 @@ rte_dev_event_monitor_stop(void)
        close(rte_intr_fd_get(intr_handle));
        rte_intr_instance_free(intr_handle);
        intr_handle = NULL;
+       ret = 0;
 
        monitor_refcount--;