ipc: fix timeout handling in async
authorJianfeng Tan <jianfeng.tan@intel.com>
Fri, 20 Apr 2018 15:20:12 +0000 (15:20 +0000)
committerThomas Monjalon <thomas@monjalon.net>
Mon, 23 Apr 2018 20:45:05 +0000 (22:45 +0200)
In original implementation, timeout event for an async request
will be ignored. As a result, an async request will never
trigger the action if it cannot receive any reply any more.

We fix this by counting timeout as a processed reply.

Fixes: f05e26051c15 ("eal: add IPC asynchronous request")

Signed-off-by: Jianfeng Tan <jianfeng.tan@intel.com>
Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
Acked-by: Anatoly Burakov <anatoly.burakov@intel.com>
lib/librte_eal/common/eal_common_proc.c

index f8cb0e8..9136fb0 100644 (file)
@@ -419,7 +419,13 @@ process_async_request(struct pending_request *sr, const struct timespec *now)
        } else if (sr->reply_received == -1) {
                /* we were asked to ignore this process */
                reply->nb_sent--;
+       } else if (timeout) {
+               /* count it as processed response, but don't increment
+                * nb_received.
+                */
+               param->n_responses_processed++;
        }
+
        free(sr->reply);
 
        last_msg = param->n_responses_processed == reply->nb_sent;