On 06/21/17 23:00, Dmitry Eremin-Solenikov wrote: > 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? > > yes, it has to be there.
Maxim.
