Re: [PATCH v6 04/13] ARC: Atomics and Locking primitives

[Adhemerval Zanella <adhemerval.zanella@linaro.org>]

On 22/04/2020 22:41, Vineet Gupta via Libc-alpha wrote:
> Signed-off-by: Vineet Gupta <vgupta@synopsys.com>

glibc uses copyright assignment to the FSF for contributions, and not
the DCO.

> ---
>  sysdeps/arc/atomic-machine.h      | 69 +++++++++++++++++++++++++++++++
>  sysdeps/arc/nptl/bits/semaphore.h | 32 ++++++++++++++
>  2 files changed, 101 insertions(+)
>  create mode 100644 sysdeps/arc/atomic-machine.h
>  create mode 100644 sysdeps/arc/nptl/bits/semaphore.h
> 

> diff --git a/sysdeps/arc/nptl/bits/semaphore.h b/sysdeps/arc/nptl/bits/semaphore.h
> new file mode 100644
> index 000000000000..772dc4cb9b01
> --- /dev/null
> +++ b/sysdeps/arc/nptl/bits/semaphore.h

This might a good candidate to a refactor to add a Linux default one,
thus avoiding any new architecture to copy/paste it.

> @@ -0,0 +1,32 @@
> +/* Machine-specific POSIX semaphore type layouts.  ARC version.
> +   Copyright (C) 2002-2020 Free Software Foundation, Inc.
> +   This file is part of the GNU C Library.
> +
> +   The GNU C Library is free software; you can redistribute it and/or
> +   modify it under the terms of the GNU Lesser General Public
> +   License as published by the Free Software Foundation; either
> +   version 2.1 of the License, or (at your option) any later version.
> +
> +   The GNU C Library is distributed in the hope that it will be useful,
> +   but WITHOUT ANY WARRANTY; without even the implied warranty of
> +   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
> +   Lesser General Public License for more details.
> +
> +   You should have received a copy of the GNU Lesser General Public
> +   License along with the GNU C Library; if not, see
> +   <https://www.gnu.org/licenses/>.  */
> +
> +#ifndef _SEMAPHORE_H
> +# error "Never use <bits/semaphore.h> directly; include <semaphore.h> instead."
> +#endif
> +
> +#define __SIZEOF_SEM_T	16
> +
> +/* Value returned if `sem_open' failed.  */
> +#define SEM_FAILED      ((sem_t *) 0)
> +
> +typedef union
> +{
> +  char __size[__SIZEOF_SEM_T];
> +  long int __align;
> +} sem_t;
>