From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Hunt Subject: [PATCH] test: fix incorrect cast in distributor unit test Date: Thu, 10 May 2018 02:25:38 +0100 Message-ID: <20180510012538.176757-1-david.hunt@intel.com> Cc: bruce.richardson@intel.com, david.hunt@intel.com To: dev@dpdk.org Return-path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id D202D1B971 for ; Thu, 10 May 2018 10:26:47 +0200 (CEST) List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" New warning with gcc 8.1: When casting the function pointer passed to rte_eal_remote_launch() as an lcore_function_t *, it expects the cast function to return an int, but instead it was returning void. Fixed to return an int. Fixes: c3eabff124e6 ("distributor: add unit tests") Signed-off-by: David Hunt --- test/test/test_distributor_perf.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/test/test/test_distributor_perf.c b/test/test/test_distributor_perf.c index 557715e13..edf1998ab 100644 --- a/test/test/test_distributor_perf.c +++ b/test/test/test_distributor_perf.c @@ -31,7 +31,7 @@ struct worker_stats worker_stats[RTE_MAX_LCORE]; * worker thread used for testing the time to do a round-trip of a cache * line between two cores and back again */ -static void +static int flip_bit(volatile uint64_t *arg) { uint64_t old_val = 0; @@ -41,6 +41,7 @@ flip_bit(volatile uint64_t *arg) old_val = *arg; *arg = 0; } + return 0; } /* -- 2.17.0