* [PATCH 0/2] Move sem_open/close/unlink to sysdeps/pthread
@ 2020-12-16 1:51 Samuel Thibault
2020-12-16 1:51 ` [PATCH 1/2] pthread: Move semaphore initialization for open to semaphoreP.h Samuel Thibault
2020-12-16 1:51 ` [PATCH 2/2] htl: Get sem_open/sem_close/sem_unlink support [BZ #25524] Samuel Thibault
0 siblings, 2 replies; 7+ messages in thread
From: Samuel Thibault @ 2020-12-16 1:51 UTC (permalink / raw)
To: libc-alpha; +Cc: Samuel Thibault, commit-hurd, adhemerval.zanella
This first separates out the struct new_sem initialization to
semaphoreP.h (no behavior change), so that sem_open/close/unlink can
then simply be used as it is by various ports, since the rest of them is
completely port-independant.
Tested on i686-gnu, i686-linux-gnu, x86_64-linux-gnu
Samuel Thibault (2):
pthread: Move semaphore initialization for open to semaphoreP.h
htl: Get sem_open/sem_close/sem_unlink support [BZ #25524]
htl/Makefile | 4 +--
htl/semaphoreP.h | 49 ++++++++++++++++++++++++++
nptl/semaphoreP.h | 15 ++++++++
sysdeps/htl/sem-close.c | 31 ----------------
sysdeps/htl/sem-open.c | 31 ----------------
sysdeps/htl/sem-unlink.c | 31 ----------------
sysdeps/mach/hurd/i386/Makefile | 7 ----
{nptl => sysdeps/pthread}/sem_close.c | 0
{nptl => sysdeps/pthread}/sem_open.c | 12 +------
{nptl => sysdeps/pthread}/sem_unlink.c | 0
10 files changed, 67 insertions(+), 113 deletions(-)
create mode 100644 htl/semaphoreP.h
delete mode 100644 sysdeps/htl/sem-close.c
delete mode 100644 sysdeps/htl/sem-open.c
delete mode 100644 sysdeps/htl/sem-unlink.c
rename {nptl => sysdeps/pthread}/sem_close.c (100%)
rename {nptl => sysdeps/pthread}/sem_open.c (96%)
rename {nptl => sysdeps/pthread}/sem_unlink.c (100%)
--
2.29.2
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] pthread: Move semaphore initialization for open to semaphoreP.h
2020-12-16 1:51 [PATCH 0/2] Move sem_open/close/unlink to sysdeps/pthread Samuel Thibault
@ 2020-12-16 1:51 ` Samuel Thibault
2020-12-16 13:25 ` Adhemerval Zanella
2020-12-16 1:51 ` [PATCH 2/2] htl: Get sem_open/sem_close/sem_unlink support [BZ #25524] Samuel Thibault
1 sibling, 1 reply; 7+ messages in thread
From: Samuel Thibault @ 2020-12-16 1:51 UTC (permalink / raw)
To: libc-alpha; +Cc: Samuel Thibault, commit-hurd, adhemerval.zanella
This allows to easily reuse all of the sem_open/sem_close/sem_unlink
implementations in the various ports.
---
nptl/sem_open.c | 12 +-----------
nptl/semaphoreP.h | 15 +++++++++++++++
2 files changed, 16 insertions(+), 11 deletions(-)
diff --git a/nptl/sem_open.c b/nptl/sem_open.c
index 05e286002b..634bdf7425 100644
--- a/nptl/sem_open.c
+++ b/nptl/sem_open.c
@@ -211,17 +211,7 @@ sem_open (const char *name, int oflag, ...)
struct new_sem newsem;
} sem;
-#if __HAVE_64B_ATOMICS
- sem.newsem.data = value;
-#else
- sem.newsem.value = value << SEM_VALUE_SHIFT;
- sem.newsem.nwaiters = 0;
-#endif
- /* pad is used as a mutex on pre-v9 sparc and ignored otherwise. */
- sem.newsem.pad = 0;
-
- /* This always is a shared semaphore. */
- sem.newsem.private = FUTEX_SHARED;
+ __new_sem_open_init (&sem.newsem, value);
/* Initialize the remaining bytes as well. */
memset ((char *) &sem.initsem + sizeof (struct new_sem), '\0',
diff --git a/nptl/semaphoreP.h b/nptl/semaphoreP.h
index f25ba329d7..2f8757e1f0 100644
--- a/nptl/semaphoreP.h
+++ b/nptl/semaphoreP.h
@@ -17,6 +17,7 @@
<https://www.gnu.org/licenses/>. */
#include <semaphore.h>
+#include <futex-internal.h>
#include "pthreadP.h"
#define SEM_SHM_PREFIX "sem."
@@ -42,6 +43,20 @@ extern int __sem_mappings_lock attribute_hidden;
/* Comparison function for search in tree with existing mappings. */
extern int __sem_search (const void *a, const void *b) attribute_hidden;
+static inline void __new_sem_open_init (struct new_sem *sem, unsigned value)
+{
+#if __HAVE_64B_ATOMICS
+ sem->data = value;
+#else
+ sem->value = value << SEM_VALUE_SHIFT;
+ sem->nwaiters = 0;
+#endif
+ /* pad is used as a mutex on pre-v9 sparc and ignored otherwise. */
+ sem->pad = 0;
+
+ /* This always is a shared semaphore. */
+ sem->private = FUTEX_SHARED;
+}
/* Prototypes of functions with multiple interfaces. */
extern int __new_sem_init (sem_t *sem, int pshared, unsigned int value);
--
2.29.2
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 2/2] htl: Get sem_open/sem_close/sem_unlink support [BZ #25524]
2020-12-16 1:51 [PATCH 0/2] Move sem_open/close/unlink to sysdeps/pthread Samuel Thibault
2020-12-16 1:51 ` [PATCH 1/2] pthread: Move semaphore initialization for open to semaphoreP.h Samuel Thibault
@ 2020-12-16 1:51 ` Samuel Thibault
2020-12-16 13:32 ` Adhemerval Zanella
1 sibling, 1 reply; 7+ messages in thread
From: Samuel Thibault @ 2020-12-16 1:51 UTC (permalink / raw)
To: libc-alpha; +Cc: Samuel Thibault, commit-hurd, adhemerval.zanella
This just moves the existing nptl implementation to reuse as it is in
htl.
---
htl/Makefile | 4 +--
htl/semaphoreP.h | 49 ++++++++++++++++++++++++++
sysdeps/htl/sem-close.c | 31 ----------------
sysdeps/htl/sem-open.c | 31 ----------------
sysdeps/htl/sem-unlink.c | 31 ----------------
sysdeps/mach/hurd/i386/Makefile | 7 ----
{nptl => sysdeps/pthread}/sem_close.c | 0
{nptl => sysdeps/pthread}/sem_open.c | 0
{nptl => sysdeps/pthread}/sem_unlink.c | 0
9 files changed, 51 insertions(+), 102 deletions(-)
create mode 100644 htl/semaphoreP.h
delete mode 100644 sysdeps/htl/sem-close.c
delete mode 100644 sysdeps/htl/sem-open.c
delete mode 100644 sysdeps/htl/sem-unlink.c
rename {nptl => sysdeps/pthread}/sem_close.c (100%)
rename {nptl => sysdeps/pthread}/sem_open.c (100%)
rename {nptl => sysdeps/pthread}/sem_unlink.c (100%)
diff --git a/htl/Makefile b/htl/Makefile
index 901deae5f9..7eccd7acf0 100644
--- a/htl/Makefile
+++ b/htl/Makefile
@@ -128,8 +128,8 @@ libpthread-routines := pt-attr pt-attr-destroy pt-attr-getdetachstate \
pt-getschedparam pt-setschedparam pt-setschedprio \
pt-yield \
\
- sem-close sem-destroy sem-getvalue sem-init sem-open \
- sem-post sem-timedwait sem-trywait sem-unlink \
+ sem_close sem-destroy sem-getvalue sem-init sem_open \
+ sem-post sem-timedwait sem-trywait sem_unlink \
sem-wait sem-waitfast \
\
shm-directory \
diff --git a/htl/semaphoreP.h b/htl/semaphoreP.h
new file mode 100644
index 0000000000..22732122b0
--- /dev/null
+++ b/htl/semaphoreP.h
@@ -0,0 +1,49 @@
+/* Copyright (C) 2002-2020 Free Software Foundation, Inc.
+ This file is part of the GNU C Library.
+ Contributed by Ulrich Drepper <drepper@redhat.com>, 2002.
+
+ 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/>. */
+
+#include <semaphore.h>
+#include "pt-internal.h"
+
+#define SEM_SHM_PREFIX "sem."
+
+/* Keeping track of currently used mappings. */
+struct inuse_sem
+{
+ dev_t dev;
+ ino_t ino;
+ int refcnt;
+ sem_t *sem;
+ char name[0];
+};
+
+
+/* The search tree for existing mappings. */
+extern void *__sem_mappings attribute_hidden;
+
+/* Lock to protect the search tree. */
+extern int __sem_mappings_lock attribute_hidden;
+
+
+/* Comparison function for search in tree with existing mappings. */
+extern int __sem_search (const void *a, const void *b) attribute_hidden;
+
+static inline void __new_sem_open_init (struct new_sem *sem, unsigned value)
+{
+ /* This always is a shared semaphore. */
+ *sem = (struct new_sem) __SEMAPHORE_INITIALIZER (1, value);
+}
diff --git a/sysdeps/htl/sem-close.c b/sysdeps/htl/sem-close.c
deleted file mode 100644
index e568758532..0000000000
--- a/sysdeps/htl/sem-close.c
+++ /dev/null
@@ -1,31 +0,0 @@
-/* Close a named semaphore. Generic version.
- Copyright (C) 2005-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/>. */
-
-#include <semaphore.h>
-#include <errno.h>
-
-#include <pt-internal.h>
-
-int
-__sem_close (sem_t *sem)
-{
- errno = EOPNOTSUPP;
- return -1;
-}
-
-strong_alias (__sem_close, sem_close);
diff --git a/sysdeps/htl/sem-open.c b/sysdeps/htl/sem-open.c
deleted file mode 100644
index 682f0e1a88..0000000000
--- a/sysdeps/htl/sem-open.c
+++ /dev/null
@@ -1,31 +0,0 @@
-/* Open a named semaphore. Generic version.
- Copyright (C) 2005-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/>. */
-
-#include <semaphore.h>
-#include <errno.h>
-
-#include <pt-internal.h>
-
-sem_t *
-__sem_open (const char *name, int open_flags, ...)
-{
- errno = ENOSYS;
- return SEM_FAILED;
-}
-
-strong_alias (__sem_open, sem_open);
diff --git a/sysdeps/htl/sem-unlink.c b/sysdeps/htl/sem-unlink.c
deleted file mode 100644
index c92c8317b1..0000000000
--- a/sysdeps/htl/sem-unlink.c
+++ /dev/null
@@ -1,31 +0,0 @@
-/* Unlink a named semaphore. Generic version.
- Copyright (C) 2005-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/>. */
-
-#include <semaphore.h>
-#include <errno.h>
-
-#include <pt-internal.h>
-
-int
-__sem_unlink (const char *name)
-{
- errno = EOPNOTSUPP;
- return -1;
-}
-
-strong_alias (__sem_unlink, sem_unlink);
diff --git a/sysdeps/mach/hurd/i386/Makefile b/sysdeps/mach/hurd/i386/Makefile
index d056e06278..7400ea98cc 100644
--- a/sysdeps/mach/hurd/i386/Makefile
+++ b/sysdeps/mach/hurd/i386/Makefile
@@ -129,13 +129,6 @@ test-xfail-tst-signal2 = yes
test-xfail-tst-cond24 = yes
test-xfail-tst-cond25 = yes
-# For bug 25524
-# (sem_open support)
-test-xfail-tst-sem4 = yes
-test-xfail-tst-sem7 = yes
-test-xfail-tst-sem8 = yes
-test-xfail-tst-sem9 = yes
-
# For bug 25563
# (robust support against dead threads)
test-xfail-tst-robust1 = yes
diff --git a/nptl/sem_close.c b/sysdeps/pthread/sem_close.c
similarity index 100%
rename from nptl/sem_close.c
rename to sysdeps/pthread/sem_close.c
diff --git a/nptl/sem_open.c b/sysdeps/pthread/sem_open.c
similarity index 100%
rename from nptl/sem_open.c
rename to sysdeps/pthread/sem_open.c
diff --git a/nptl/sem_unlink.c b/sysdeps/pthread/sem_unlink.c
similarity index 100%
rename from nptl/sem_unlink.c
rename to sysdeps/pthread/sem_unlink.c
--
2.29.2
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] pthread: Move semaphore initialization for open to semaphoreP.h
2020-12-16 1:51 ` [PATCH 1/2] pthread: Move semaphore initialization for open to semaphoreP.h Samuel Thibault
@ 2020-12-16 13:25 ` Adhemerval Zanella
2020-12-16 13:28 ` Samuel Thibault
0 siblings, 1 reply; 7+ messages in thread
From: Adhemerval Zanella @ 2020-12-16 13:25 UTC (permalink / raw)
To: Samuel Thibault, libc-alpha; +Cc: commit-hurd
On 15/12/2020 22:51, Samuel Thibault wrote:
> This allows to easily reuse all of the sem_open/sem_close/sem_unlink
> implementations in the various ports.
LGTM, thanks.
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
> ---
> nptl/sem_open.c | 12 +-----------
> nptl/semaphoreP.h | 15 +++++++++++++++
> 2 files changed, 16 insertions(+), 11 deletions(-)
>
> diff --git a/nptl/sem_open.c b/nptl/sem_open.c
> index 05e286002b..634bdf7425 100644
> --- a/nptl/sem_open.c
> +++ b/nptl/sem_open.c
> @@ -211,17 +211,7 @@ sem_open (const char *name, int oflag, ...)
> struct new_sem newsem;
> } sem;
>
> -#if __HAVE_64B_ATOMICS
> - sem.newsem.data = value;
> -#else
> - sem.newsem.value = value << SEM_VALUE_SHIFT;
> - sem.newsem.nwaiters = 0;
> -#endif
> - /* pad is used as a mutex on pre-v9 sparc and ignored otherwise. */
> - sem.newsem.pad = 0;
> -
> - /* This always is a shared semaphore. */
> - sem.newsem.private = FUTEX_SHARED;
> + __new_sem_open_init (&sem.newsem, value);
>
> /* Initialize the remaining bytes as well. */
> memset ((char *) &sem.initsem + sizeof (struct new_sem), '\0',
Ok.
> diff --git a/nptl/semaphoreP.h b/nptl/semaphoreP.h
> index f25ba329d7..2f8757e1f0 100644
> --- a/nptl/semaphoreP.h
> +++ b/nptl/semaphoreP.h
> @@ -17,6 +17,7 @@
> <https://www.gnu.org/licenses/>. */
>
> #include <semaphore.h>
> +#include <futex-internal.h>
> #include "pthreadP.h"
>
> #define SEM_SHM_PREFIX "sem."
> @@ -42,6 +43,20 @@ extern int __sem_mappings_lock attribute_hidden;
> /* Comparison function for search in tree with existing mappings. */
> extern int __sem_search (const void *a, const void *b) attribute_hidden;
>
> +static inline void __new_sem_open_init (struct new_sem *sem, unsigned value)
> +{
> +#if __HAVE_64B_ATOMICS
> + sem->data = value;
> +#else
> + sem->value = value << SEM_VALUE_SHIFT;
> + sem->nwaiters = 0;
> +#endif
> + /* pad is used as a mutex on pre-v9 sparc and ignored otherwise. */
> + sem->pad = 0;
> +
> + /* This always is a shared semaphore. */
> + sem->private = FUTEX_SHARED;
> +}
>
> /* Prototypes of functions with multiple interfaces. */
> extern int __new_sem_init (sem_t *sem, int pshared, unsigned int value);
>
Ok.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] pthread: Move semaphore initialization for open to semaphoreP.h
2020-12-16 13:25 ` Adhemerval Zanella
@ 2020-12-16 13:28 ` Samuel Thibault
0 siblings, 0 replies; 7+ messages in thread
From: Samuel Thibault @ 2020-12-16 13:28 UTC (permalink / raw)
To: Adhemerval Zanella; +Cc: libc-alpha, commit-hurd
Adhemerval Zanella, le mer. 16 déc. 2020 10:25:59 -0300, a ecrit:
> On 15/12/2020 22:51, Samuel Thibault wrote:
> > This allows to easily reuse all of the sem_open/sem_close/sem_unlink
> > implementations in the various ports.
>
> LGTM, thanks.
Thanks for the prompt feedback!
Samuel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] htl: Get sem_open/sem_close/sem_unlink support [BZ #25524]
2020-12-16 1:51 ` [PATCH 2/2] htl: Get sem_open/sem_close/sem_unlink support [BZ #25524] Samuel Thibault
@ 2020-12-16 13:32 ` Adhemerval Zanella
2020-12-16 13:53 ` Samuel Thibault
0 siblings, 1 reply; 7+ messages in thread
From: Adhemerval Zanella @ 2020-12-16 13:32 UTC (permalink / raw)
To: Samuel Thibault, libc-alpha; +Cc: commit-hurd
On 15/12/2020 22:51, Samuel Thibault wrote:
> This just moves the existing nptl implementation to reuse as it is in
> htl.
\
> diff --git a/htl/semaphoreP.h b/htl/semaphoreP.h
> new file mode 100644
> index 0000000000..22732122b0
> --- /dev/null
> +++ b/htl/semaphoreP.h
> @@ -0,0 +1,49 @@
> +/* Copyright (C) 2002-2020 Free Software Foundation, Inc.
> + This file is part of the GNU C Library.
> + Contributed by Ulrich Drepper <drepper@redhat.com>, 2002.
Remove the 'Contributed by'.
> +
> + 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/>. */
> +
> +#include <semaphore.h>
> +#include "pt-internal.h"
> +
> +#define SEM_SHM_PREFIX "sem."
> +
> +/* Keeping track of currently used mappings. */
> +struct inuse_sem
> +{
> + dev_t dev;
> + ino_t ino;
> + int refcnt;
> + sem_t *sem;
> + char name[0];
Use a C99 flexible array.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] htl: Get sem_open/sem_close/sem_unlink support [BZ #25524]
2020-12-16 13:32 ` Adhemerval Zanella
@ 2020-12-16 13:53 ` Samuel Thibault
0 siblings, 0 replies; 7+ messages in thread
From: Samuel Thibault @ 2020-12-16 13:53 UTC (permalink / raw)
To: Adhemerval Zanella; +Cc: libc-alpha, commit-hurd
Adhemerval Zanella, le mer. 16 déc. 2020 10:32:33 -0300, a ecrit:
> On 15/12/2020 22:51, Samuel Thibault wrote:
> > diff --git a/htl/semaphoreP.h b/htl/semaphoreP.h
> > new file mode 100644
> > index 0000000000..22732122b0
> > --- /dev/null
> > +++ b/htl/semaphoreP.h
> > @@ -0,0 +1,49 @@
> > +/* Copyright (C) 2002-2020 Free Software Foundation, Inc.
> > + This file is part of the GNU C Library.
> > + Contributed by Ulrich Drepper <drepper@redhat.com>, 2002.
>
> Remove the 'Contributed by'.
>
> > +/* Keeping track of currently used mappings. */
> > +struct inuse_sem
> > +{
> > + dev_t dev;
> > + ino_t ino;
> > + int refcnt;
> > + sem_t *sem;
> > + char name[0];
>
> Use a C99 flexible array.
These were copied over from the existing files, but right, fixed so :)
Samuel
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2020-12-16 13:53 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-16 1:51 [PATCH 0/2] Move sem_open/close/unlink to sysdeps/pthread Samuel Thibault
2020-12-16 1:51 ` [PATCH 1/2] pthread: Move semaphore initialization for open to semaphoreP.h Samuel Thibault
2020-12-16 13:25 ` Adhemerval Zanella
2020-12-16 13:28 ` Samuel Thibault
2020-12-16 1:51 ` [PATCH 2/2] htl: Get sem_open/sem_close/sem_unlink support [BZ #25524] Samuel Thibault
2020-12-16 13:32 ` Adhemerval Zanella
2020-12-16 13:53 ` Samuel Thibault
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).