On 21.06.2017 21:14, Ola Liljedahl wrote: > > On 20/06/2017, 15:04, "Savolainen, Petri (Nokia - FI/Espoo)" > <[email protected]> wrote: > >> >> >>> +++ b/platform/linux-generic/include/odp_bitset.h >>> @@ -0,0 +1,210 @@ >>> +/* Copyright (c) 2017, ARM Limited >>> + * All rights reserved. >>> + * >>> + * SPDX-License-Identifier: BSD-3-Clause >>> + */ >>> + >>> +#ifndef _ODP_BITSET_H_ >>> +#define _ODP_BITSET_H_ >>> + >>> +#include <odp_cpu.h> >>> + >>> +#include <limits.h> >>> + >>> >>> +/*********************************************************************** >>> * >>> ****** >>> + * bitset abstract data type >>> + >>> >>> ************************************************************************* >>> * >>> ***/ >>> +/* This could be a struct of scalars to support larger bit sets */ >>> + >>> +/* >>> + * Size of atomic bit set. This limits the max number of threads, >>> + * scheduler groups and reorder windows. On ARMv8/64-bit and x86-64, >>> the >>> + * (lock-free) max is 128 >>> + */ >>> + >>> +/* Find a suitable data type that supports lock-free atomic operations >>> */ >>> +#if defined(__ARM_ARCH) && __ARM_ARCH == 8 && __ARM_64BIT_STATE == 1 >>> && >> >> Why ifdef ARM? Why this code is not in arch directory ? > Why is this car red? > Because I like it like that.
I think it was agreed that arch-specific code should go to arch/ dirs, wasn't it? -- With best wishes Dmitry
