On Wed, Jul 28, 2021 at 8:52 PM Shijith Thotton <sthot...@marvell.com> wrote:
>
> Added a macro to swap two numbers and updated common autotest for the
> same.
>
> Signed-off-by: Shijith Thotton <sthot...@marvell.com>
Acked-by: Jerin Jacob <jer...@marvell.com>
> ---
> Needed-for: drivers: add external clock support for cnxk timer
@Thomas Monjalon Could you merge this to the main tree if the patch
looks OK to you.
>
> app/test/test_common.c | 4 ++++
> lib/eal/include/rte_common.h | 8 ++++++++
> 2 files changed, 12 insertions(+)
>
> diff --git a/app/test/test_common.c b/app/test/test_common.c
> index 12bd1cad90..ef177cecb1 100644
> --- a/app/test/test_common.c
> +++ b/app/test/test_common.c
> @@ -30,9 +30,13 @@ test_macros(int __rte_unused unused_parm)
> return -1;}
>
> uintptr_t unused = 0;
> + unsigned int smaller = SMALLER, bigger = BIGGER;
>
> RTE_SET_USED(unused);
>
> + RTE_SWAP(smaller, bigger);
> + if (smaller != BIGGER && bigger != SMALLER)
> + FAIL_MACRO(RTE_SWAP);
> if ((uintptr_t)RTE_PTR_ADD(SMALLER, PTR_DIFF) != BIGGER)
> FAIL_MACRO(RTE_PTR_ADD);
> if ((uintptr_t)RTE_PTR_SUB(BIGGER, PTR_DIFF) != SMALLER)
> diff --git a/lib/eal/include/rte_common.h b/lib/eal/include/rte_common.h
> index d5a32c66a5..09661a5469 100644
> --- a/lib/eal/include/rte_common.h
> +++ b/lib/eal/include/rte_common.h
> @@ -810,6 +810,14 @@ rte_log2_u64(uint64_t v)
> /** Number of elements in the array. */
> #define RTE_DIM(a) (sizeof (a) / sizeof ((a)[0]))
>
> +/** Macro to swap two numbers. */
> +#define RTE_SWAP(a, b) \
> + __extension__ ({ \
> + typeof (a) _a = a; \
> + a = b; \
> + b = _a; \
> + })
> +
> /**
> * Converts a numeric string to the equivalent uint64_t value.
> * As well as straight number conversion, also recognises the suffixes
> --
> 2.25.1
>
