[PATCH 3/6] net/nfp: remove the flower service dead loop

[Chaoyong He]

From: Long Wu <long...@corigine.com>

The flower service does not require high real-time performance,
so remove the dead loop to share service core with other services.

Signed-off-by: Long Wu <long...@corigine.com>
Reviewed-by: Peng Zhang <peng.zh...@corigine.com>
Reviewed-by: Chaoyong He <chaoyong...@corigine.com>
---
 drivers/net/nfp/flower/nfp_flower_service.c | 29 +++++++++------------
 1 file changed, 13 insertions(+), 16 deletions(-)

diff --git a/drivers/net/nfp/flower/nfp_flower_service.c 
b/drivers/net/nfp/flower/nfp_flower_service.c
index 515b4abef0..64753f5bf6 100644
--- a/drivers/net/nfp/flower/nfp_flower_service.c
+++ b/drivers/net/nfp/flower/nfp_flower_service.c
@@ -34,28 +34,25 @@ nfp_flower_service_handle_get(struct nfp_net_hw_priv 
*hw_priv)
 }
 
 static int
-nfp_flower_service_loop(void *arg)
+nfp_flower_service_func(void *arg)
 {
        uint16_t slot;
        struct nfp_app_fw_flower *app;
        struct nfp_flower_service *service_handle;
 
        service_handle = arg;
-       /* Waiting for enabling service */
-       while (!service_handle->enabled)
-               rte_delay_ms(1);
-
-       while (rte_service_runstate_get(service_handle->info.id) != 0) {
-               rte_spinlock_lock(&service_handle->spinlock);
-               for (slot = 0; slot < MAX_FLOWER_SERVICE_SLOT; slot++) {
-                       app = service_handle->slots[slot];
-                       if (app == NULL)
-                               continue;
-
-                       nfp_flower_ctrl_vnic_process(app);
-               }
-               rte_spinlock_unlock(&service_handle->spinlock);
+       if (!service_handle->enabled)
+               return 0;
+
+       rte_spinlock_lock(&service_handle->spinlock);
+       for (slot = 0; slot < MAX_FLOWER_SERVICE_SLOT; slot++) {
+               app = service_handle->slots[slot];
+               if (app == NULL)
+                       continue;
+
+               nfp_flower_ctrl_vnic_process(app);
        }
+       rte_spinlock_unlock(&service_handle->spinlock);
 
        return 0;
 }
@@ -67,7 +64,7 @@ nfp_flower_service_enable(struct nfp_flower_service 
*service_handle)
 
        const struct rte_service_spec flower_service = {
                .name              = "flower_ctrl_vnic_service",
-               .callback          = nfp_flower_service_loop,
+               .callback          = nfp_flower_service_func,
                .callback_userdata = (void *)service_handle,
        };
 
-- 
2.39.1