[dpdk-dev] test: fix incorrect cast in distributor unit test
Checks
Commit Message
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 <david.hunt@intel.com>
---
test/test/test_distributor_perf.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
Comments
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of David Hunt
> Sent: Thursday, May 10, 2018 2:26 AM
> To: dev@dpdk.org
> Cc: Richardson, Bruce <bruce.richardson@intel.com>; Hunt, David
> <david.hunt@intel.com>
> Subject: [dpdk-dev] [PATCH] test: fix incorrect cast in distributor unit test
>
> 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 <david.hunt@intel.com>
Cc: stable@dpdk.org
Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
> > 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 <david.hunt@intel.com>
>
> Cc: stable@dpdk.org
>
> Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
Applied, thanks
@@ -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;
}
/*