Ensure posix_spawnattr_{get,set}sched{policy,param} are defined.

diff --git a/ChangeLog b/ChangeLog
index 0db6644e50c34276ed68ca8eaa30eae26a33a253..e91b6cc3f65b547bea1ad0ec000bbbea233699be 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,33 @@
+2010-03-07  Bruno Haible  <bruno@clisp.org>
+
+       Ensure posix_spawnattr_{get,set}sched{policy,param} are defined.
+       * lib/spawn.in.h (POSIX_SPAWN_SETSCHEDPARAM): Define fallback.
+       (POSIX_SPAWN_SETSCHEDULER): Likewise.
+       (POSIX_SPAWN_USEVFORK): Define in a way that works when
+       POSIX_SPAWN_SETSCHEDPARAM and POSIX_SPAWN_SETSCHEDULER are zero.
+       (posix_spawnattr_getschedpolicy, posix_spawnattr_setschedpolicy): Also
+       declare when POSIX_SPAWN_SETSCHEDULER is zero.
+       (posix_spawnattr_getschedparam, posix_spawnattr_setschedparam): Also
+       declare when POSIX_SPAWN_SETSCHEDPARAM is zero.
+       * m4/posix_spawn.m4 (gl_POSIX_SPAWN_BODY): Test whether
+       POSIX_SPAWN_SETSCHEDULER or POSIX_SPAWN_SETSCHEDPARAM are zero.
+       * modules/posix_spawnattr_getschedparam (configure.ac): Enable the
+       replacement also when POSIX_SPAWN_SETSCHEDPARAM is zero.
+       * modules/posix_spawnattr_setschedparam (configure.ac): Likewise.
+       * modules/posix_spawnattr_getschedpolicy (configure.ac): Enable the
+       replacement also when POSIX_SPAWN_SETSCHEDULER is zero.
+       * modules/posix_spawnattr_setschedpolicy (configure.ac): Likewise.
+       * lib/spawnattr_getschedparam.c (posix_spawnattr_getschedparam): Do
+       nothing if POSIX_SPAWN_SETSCHEDPARAM is zero.
+       * lib/spawnattr_setschedparam.c (posix_spawnattr_setschedparam):
+       Likewise.
+       * lib/spawnattr_getschedpolicy.c (posix_spawnattr_getschedpolicy): Do
+       nothing if POSIX_SPAWN_SETSCHEDULER is zero.
+       * lib/spawnattr_setschedpolicy.c (posix_spawnattr_setschedpolicy):
+       Likewise.
+       * tests/test-spawn.c (main): Make it work when
+       POSIX_SPAWN_SETSCHEDPARAM and POSIX_SPAWN_SETSCHEDULER are zero.
+
 2010-03-07  Bruno Haible  <bruno@clisp.org>
 
        Fix incorrect Makefile.am generation in German locale.

diff --git a/lib/spawn.in.h b/lib/spawn.in.h
index 1e8c936bd7621db6635fae1fc199900baebe0415..d049c040a9fb7bba06864bb1c0767c627404e8a1 100644 (file)
--- a/lib/spawn.in.h
+++ b/lib/spawn.in.h
@@ -110,7 +110,15 @@ typedef struct
 
 
 /* Flags to be set in the `posix_spawnattr_t'.  */
-#if !@HAVE_POSIX_SPAWN@
+#if @HAVE_POSIX_SPAWN@
+/* Use the values from the system, but provide the missing ones.  */
+# ifndef POSIX_SPAWN_SETSCHEDPARAM
+#  define POSIX_SPAWN_SETSCHEDPARAM 0
+# endif
+# ifndef POSIX_SPAWN_SETSCHEDULER
+#  define POSIX_SPAWN_SETSCHEDULER 0
+# endif
+#else
 # if @REPLACE_POSIX_SPAWN@
 /* Use the values from the system, for better compatibility.  */
 /* But this implementation does not support AIX extensions.  */
@@ -126,12 +134,14 @@ typedef struct
 #endif
 /* A GNU extension.  Use the next free bit position.  */
 #define POSIX_SPAWN_USEVFORK \
-  ((POSIX_SPAWN_RESETIDS | (POSIX_SPAWN_RESETIDS - 1)                   \
-    | POSIX_SPAWN_SETPGROUP | (POSIX_SPAWN_SETPGROUP - 1)               \
-    | POSIX_SPAWN_SETSIGDEF | (POSIX_SPAWN_SETSIGDEF - 1)               \
-    | POSIX_SPAWN_SETSIGMASK | (POSIX_SPAWN_SETSIGMASK - 1)             \
-    | POSIX_SPAWN_SETSCHEDPARAM | (POSIX_SPAWN_SETSCHEDPARAM - 1)       \
-    | POSIX_SPAWN_SETSCHEDULER | (POSIX_SPAWN_SETSCHEDULER - 1))        \
+  ((POSIX_SPAWN_RESETIDS | (POSIX_SPAWN_RESETIDS - 1)                     \
+    | POSIX_SPAWN_SETPGROUP | (POSIX_SPAWN_SETPGROUP - 1)                 \
+    | POSIX_SPAWN_SETSIGDEF | (POSIX_SPAWN_SETSIGDEF - 1)                 \
+    | POSIX_SPAWN_SETSIGMASK | (POSIX_SPAWN_SETSIGMASK - 1)               \
+    | POSIX_SPAWN_SETSCHEDPARAM                                           \
+    | (POSIX_SPAWN_SETSCHEDPARAM > 0 ? POSIX_SPAWN_SETSCHEDPARAM - 1 : 0) \
+    | POSIX_SPAWN_SETSCHEDULER                                            \
+    | (POSIX_SPAWN_SETSCHEDULER > 0 ? POSIX_SPAWN_SETSCHEDULER - 1 : 0))  \
    + 1)
 typedef int verify_POSIX_SPAWN_USEVFORK_no_overlap
             [2 * (((POSIX_SPAWN_RESETIDS | POSIX_SPAWN_SETPGROUP
@@ -379,7 +389,7 @@ _GL_WARN_ON_USE (posix_spawnattr_setpgroup, "posix_spawnattr_setpgroup is unport
 # if @REPLACE_POSIX_SPAWN@
 #  define posix_spawnattr_getschedpolicy rpl_posix_spawnattr_getschedpolicy
 # endif
-# if !@HAVE_POSIX_SPAWN@ || @REPLACE_POSIX_SPAWN@
+# if !@HAVE_POSIX_SPAWN@ || @REPLACE_POSIX_SPAWN@ || POSIX_SPAWN_SETSCHEDULER == 0
 extern int posix_spawnattr_getschedpolicy (const posix_spawnattr_t *_Restrict_ __attr,
                                            int *_Restrict_ __schedpolicy)
      __THROW _GL_ARG_NONNULL ((1, 2));
@@ -397,7 +407,7 @@ _GL_WARN_ON_USE (posix_spawnattr_getschedpolicy, "posix_spawnattr_getschedpolicy
 # if @REPLACE_POSIX_SPAWN@
 #  define posix_spawnattr_setschedpolicy rpl_posix_spawnattr_setschedpolicy
 # endif
-# if !@HAVE_POSIX_SPAWN@ || @REPLACE_POSIX_SPAWN@
+# if !@HAVE_POSIX_SPAWN@ || @REPLACE_POSIX_SPAWN@ || POSIX_SPAWN_SETSCHEDULER == 0
 extern int posix_spawnattr_setschedpolicy (posix_spawnattr_t *__attr,
                                            int __schedpolicy)
      __THROW _GL_ARG_NONNULL ((1));
@@ -415,7 +425,7 @@ _GL_WARN_ON_USE (posix_spawnattr_setschedpolicy, "posix_spawnattr_setschedpolicy
 # if @REPLACE_POSIX_SPAWN@
 #  define posix_spawnattr_getschedparam rpl_posix_spawnattr_getschedparam
 # endif
-# if !@HAVE_POSIX_SPAWN@ || @REPLACE_POSIX_SPAWN@
+# if !@HAVE_POSIX_SPAWN@ || @REPLACE_POSIX_SPAWN@ || POSIX_SPAWN_SETSCHEDPARAM == 0
 extern int posix_spawnattr_getschedparam (const posix_spawnattr_t *_Restrict_ __attr,
                                           struct sched_param *_Restrict_ __schedparam)
      __THROW _GL_ARG_NONNULL ((1, 2));
@@ -433,7 +443,7 @@ _GL_WARN_ON_USE (posix_spawnattr_getschedparam, "posix_spawnattr_getschedparam i
 # if @REPLACE_POSIX_SPAWN@
 #  define posix_spawnattr_setschedparam rpl_posix_spawnattr_setschedparam
 # endif
-# if !@HAVE_POSIX_SPAWN@ || @REPLACE_POSIX_SPAWN@
+# if !@HAVE_POSIX_SPAWN@ || @REPLACE_POSIX_SPAWN@ || POSIX_SPAWN_SETSCHEDPARAM == 0
 extern int posix_spawnattr_setschedparam (posix_spawnattr_t *_Restrict_ __attr,
                                           const struct sched_param *_Restrict_ __schedparam)
      __THROW _GL_ARG_NONNULL ((1, 2));

diff --git a/lib/spawnattr_getschedparam.c b/lib/spawnattr_getschedparam.c
index 4ac9c458793a82b57e128633b3bae0fc904b31e8..631ec0c19086f9d31deb5e0dae49e11bf3b845e9 100644 (file)
--- a/lib/spawnattr_getschedparam.c
+++ b/lib/spawnattr_getschedparam.c
@@ -1,4 +1,4 @@
-/* Copyright (C) 2000, 2008, 2009, 2010 Free Software Foundation, Inc.
+/* Copyright (C) 2000, 2008-2010 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
 
    This program is free software: you can redistribute it and/or modify
@@ -24,8 +24,11 @@ int
 posix_spawnattr_getschedparam (const posix_spawnattr_t *attr,
                                struct sched_param *schedparam)
 {
+  /* Do nothing if POSIX_SPAWN_SETSCHEDPARAM is unsupported.  */
+#if POSIX_SPAWN_SETSCHEDPARAM != 0
   /* Copy the scheduling parameters.  */
   *schedparam = attr->_sp;
+#endif
 
   return 0;
 }

diff --git a/lib/spawnattr_getschedpolicy.c b/lib/spawnattr_getschedpolicy.c
index c94a93203f36b5ab99c4740c8db8e291966b3436..248e05a86445545c6dd0a47c688def1406a30678 100644 (file)
--- a/lib/spawnattr_getschedpolicy.c
+++ b/lib/spawnattr_getschedpolicy.c
@@ -1,4 +1,4 @@
-/* Copyright (C) 2000, 2009, 2010 Free Software Foundation, Inc.
+/* Copyright (C) 2000, 2009-2010 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
 
    This program is free software: you can redistribute it and/or modify
@@ -26,8 +26,11 @@ int
 posix_spawnattr_getschedpolicy (const posix_spawnattr_t *attr,
                                 int *schedpolicy)
 {
+  /* Do nothing if POSIX_SPAWN_SETSCHEDULER is unsupported.  */
+#if POSIX_SPAWN_SETSCHEDULER != 0
   /* Copy the scheduling policy.  */
   *schedpolicy = attr->_policy;
+#endif
 
   return 0;
 }

diff --git a/lib/spawnattr_setschedparam.c b/lib/spawnattr_setschedparam.c
index 4e832c7e14fcb5dd431a3e7a1cf6bb4398bfe027..e01e6fcca4b7a2ce949571f45630782a160dd5b0 100644 (file)
--- a/lib/spawnattr_setschedparam.c
+++ b/lib/spawnattr_setschedparam.c
@@ -1,4 +1,4 @@
-/* Copyright (C) 2000, 2009, 2010 Free Software Foundation, Inc.
+/* Copyright (C) 2000, 2009-2010 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
 
    This program is free software: you can redistribute it and/or modify
@@ -24,8 +24,11 @@ int
 posix_spawnattr_setschedparam (posix_spawnattr_t *attr,
                                const struct sched_param *schedparam)
 {
+  /* Do nothing if POSIX_SPAWN_SETSCHEDPARAM is unsupported.  */
+#if POSIX_SPAWN_SETSCHEDPARAM != 0
   /* Store the scheduling parameters.  */
   attr->_sp = *schedparam;
+#endif
 
   return 0;
 }

diff --git a/lib/spawnattr_setschedpolicy.c b/lib/spawnattr_setschedpolicy.c
index b12da4b5d25e69ce860a4fc9a30993eeae9070d3..b166e7278f58611338dc32a1856dfe00748126ca 100644 (file)
--- a/lib/spawnattr_setschedpolicy.c
+++ b/lib/spawnattr_setschedpolicy.c
@@ -1,4 +1,4 @@
-/* Copyright (C) 2000, 2009, 2010 Free Software Foundation, Inc.
+/* Copyright (C) 2000, 2009-2010 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
 
    This program is free software: you can redistribute it and/or modify
@@ -25,12 +25,15 @@
 int
 posix_spawnattr_setschedpolicy (posix_spawnattr_t *attr, int schedpolicy)
 {
+  /* Do nothing if POSIX_SPAWN_SETSCHEDULER is unsupported.  */
+#if POSIX_SPAWN_SETSCHEDULER != 0
   if (schedpolicy != SCHED_OTHER && schedpolicy != SCHED_FIFO
       && schedpolicy != SCHED_RR)
     return EINVAL;
 
   /* Store the policy.  */
   attr->_policy = schedpolicy;
+#endif
 
   return 0;
 }

diff --git a/m4/posix_spawn.m4 b/m4/posix_spawn.m4
index e8f12c40c700cc54af092a200dd76eb3bc0aacef..89e2f7f06477daa9993a4c9a0eeeab1ef05a9a0d 100644 (file)
--- a/m4/posix_spawn.m4
+++ b/m4/posix_spawn.m4
@@ -1,5 +1,5 @@
-# posix_spawn.m4 serial 4
-dnl Copyright (C) 2008, 2009, 2010 Free Software Foundation, Inc.
+# posix_spawn.m4 serial 5
+dnl Copyright (C) 2008-2010 Free Software Foundation, Inc.
 dnl This file is free software; the Free Software Foundation
 dnl gives unlimited permission to copy and/or distribute it,
 dnl with or without modifications, as long as this notice is preserved.
@@ -14,8 +14,8 @@ AC_DEFUN([gl_POSIX_SPAWN_BODY],
 [
   AC_REQUIRE([gl_SPAWN_H_DEFAULTS])
   AC_CHECK_FUNCS_ONCE([posix_spawn])
-  dnl Assume that when the main function exists, all the others are
-  dnl available as well.
+  dnl Assume that when the main function exists, all the others,
+  dnl except posix_spawnattr_{get,set}sched*, are available as well.
   dnl AC_CHECK_FUNCS_ONCE([posix_spawnp])
   dnl AC_CHECK_FUNCS_ONCE([posix_spawn_file_actions_init])
   dnl AC_CHECK_FUNCS_ONCE([posix_spawn_file_actions_addclose])
@@ -27,10 +27,6 @@ AC_DEFUN([gl_POSIX_SPAWN_BODY],
   dnl AC_CHECK_FUNCS_ONCE([posix_spawnattr_setflags])
   dnl AC_CHECK_FUNCS_ONCE([posix_spawnattr_getpgroup])
   dnl AC_CHECK_FUNCS_ONCE([posix_spawnattr_setpgroup])
-  dnl AC_CHECK_FUNCS_ONCE([posix_spawnattr_getschedparam])
-  dnl AC_CHECK_FUNCS_ONCE([posix_spawnattr_setschedparam])
-  dnl AC_CHECK_FUNCS_ONCE([posix_spawnattr_getschedpolicy])
-  dnl AC_CHECK_FUNCS_ONCE([posix_spawnattr_setschedpolicy])
   dnl AC_CHECK_FUNCS_ONCE([posix_spawnattr_getsigdefault])
   dnl AC_CHECK_FUNCS_ONCE([posix_spawnattr_setsigdefault])
   dnl AC_CHECK_FUNCS_ONCE([posix_spawnattr_getsigmask])
@@ -39,7 +35,38 @@ AC_DEFUN([gl_POSIX_SPAWN_BODY],
   if test $ac_cv_func_posix_spawn = yes; then
     gl_POSIX_SPAWN_WORKS
     case "$gl_cv_func_posix_spawn_works" in
-      *yes) ;;
+      *yes)
+        dnl Assume that these functions are available if POSIX_SPAWN_SETSCHEDULER
+        dnl evaluates to nonzero.
+        dnl AC_CHECK_FUNCS_ONCE([posix_spawnattr_getschedpolicy])
+        dnl AC_CHECK_FUNCS_ONCE([posix_spawnattr_setschedpolicy])
+        AC_CACHE_CHECK([whether posix_spawnattr_setschedpolicy is supported],
+          [gl_cv_func_spawnattr_setschedpolicy],
+          [AC_EGREP_CPP([POSIX scheduling supported], [
+#include <spawn.h>
+#if POSIX_SPAWN_SETSCHEDULER
+ POSIX scheduling supported
+#endif
+],
+             [gl_cv_func_spawnattr_setschedpolicy=yes],
+             [gl_cv_func_spawnattr_setschedpolicy=no])
+          ])
+        dnl Assume that these functions are available if POSIX_SPAWN_SETSCHEDPARAM
+        dnl evaluates to nonzero.
+        dnl AC_CHECK_FUNCS_ONCE([posix_spawnattr_getschedparam])
+        dnl AC_CHECK_FUNCS_ONCE([posix_spawnattr_setschedparam])
+        AC_CACHE_CHECK([whether posix_spawnattr_setschedparam is supported],
+          [gl_cv_func_spawnattr_setschedparam],
+          [AC_EGREP_CPP([POSIX scheduling supported], [
+#include <spawn.h>
+#if POSIX_SPAWN_SETSCHEDPARAM
+ POSIX scheduling supported
+#endif
+],
+             [gl_cv_func_spawnattr_setschedparam=yes],
+             [gl_cv_func_spawnattr_setschedparam=no])
+          ])
+        ;;
       *) REPLACE_POSIX_SPAWN=1 ;;
     esac
   else

diff --git a/modules/posix_spawnattr_getschedparam b/modules/posix_spawnattr_getschedparam
index 4691523503782d7b7b1fb30cda259602c6a46eca..e93479a50b71b9e34794f13ead2a7993beaa7e18 100644 (file)
--- a/modules/posix_spawnattr_getschedparam
+++ b/modules/posix_spawnattr_getschedparam
@@ -11,7 +11,9 @@ spawn
 
 configure.ac:
 gl_POSIX_SPAWN
-if test $HAVE_POSIX_SPAWN = 0 || test $REPLACE_POSIX_SPAWN = 1; then
+if test $HAVE_POSIX_SPAWN = 0 \
+   || test $REPLACE_POSIX_SPAWN = 1 \
+   || test $gl_cv_func_spawnattr_setschedparam = no; then
   gl_REPLACE_SPAWN_H
   AC_LIBOBJ([spawnattr_getschedparam])
 fi

diff --git a/modules/posix_spawnattr_getschedpolicy b/modules/posix_spawnattr_getschedpolicy
index 9663e37a361e10d22eba60b81d6b4bc1b228e557..47f15f99e6b846cb7d75a9290bf9913a69d1a9fc 100644 (file)
--- a/modules/posix_spawnattr_getschedpolicy
+++ b/modules/posix_spawnattr_getschedpolicy
@@ -11,7 +11,9 @@ spawn
 
 configure.ac:
 gl_POSIX_SPAWN
-if test $HAVE_POSIX_SPAWN = 0 || test $REPLACE_POSIX_SPAWN = 1; then
+if test $HAVE_POSIX_SPAWN = 0 \
+   || test $REPLACE_POSIX_SPAWN = 1 \
+   || test $gl_cv_func_spawnattr_setschedpolicy = no; then
   gl_REPLACE_SPAWN_H
   AC_LIBOBJ([spawnattr_getschedpolicy])
 fi

diff --git a/modules/posix_spawnattr_setschedparam b/modules/posix_spawnattr_setschedparam
index abef5f28dd4d2f206b1d045640d24eee613f9b5a..8330c87d3013d8afe93996df1d1818bf9e212e4e 100644 (file)
--- a/modules/posix_spawnattr_setschedparam
+++ b/modules/posix_spawnattr_setschedparam
@@ -11,7 +11,9 @@ spawn
 
 configure.ac:
 gl_POSIX_SPAWN
-if test $HAVE_POSIX_SPAWN = 0 || test $REPLACE_POSIX_SPAWN = 1; then
+if test $HAVE_POSIX_SPAWN = 0 \
+   || test $REPLACE_POSIX_SPAWN = 1 \
+   || test $gl_cv_func_spawnattr_setschedparam = no; then
   gl_REPLACE_SPAWN_H
   AC_LIBOBJ([spawnattr_setschedparam])
 fi

diff --git a/modules/posix_spawnattr_setschedpolicy b/modules/posix_spawnattr_setschedpolicy
index 2ee661ae63228f8776df1e24f20a5f60deab1250..fd2046307b831e12b718c73d6bd6f1ed323566db 100644 (file)
--- a/modules/posix_spawnattr_setschedpolicy
+++ b/modules/posix_spawnattr_setschedpolicy
@@ -11,7 +11,9 @@ spawn
 
 configure.ac:
 gl_POSIX_SPAWN
-if test $HAVE_POSIX_SPAWN = 0 || test $REPLACE_POSIX_SPAWN = 1; then
+if test $HAVE_POSIX_SPAWN = 0 \
+   || test $REPLACE_POSIX_SPAWN = 1 \
+   || test $gl_cv_func_spawnattr_setschedpolicy = no; then
   gl_REPLACE_SPAWN_H
   AC_LIBOBJ([spawnattr_setschedpolicy])
 fi

diff --git a/tests/test-spawn.c b/tests/test-spawn.c
index dc6b0a8e79056c36c1c53842bf045edbc8fbb891..3caf8ee6b8380642e6fe17bf2f3e7e2f6bf52c34 100644 (file)
--- a/tests/test-spawn.c
+++ b/tests/test-spawn.c
@@ -1,5 +1,5 @@
 /* Test of <spawn.h> substitute.
-   Copyright (C) 2009, 2010 Free Software Foundation, Inc.
+   Copyright (C) 2009-2010 Free Software Foundation, Inc.
 
    This program is free software: you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
@@ -46,8 +46,8 @@ main (void)
     case POSIX_SPAWN_SETPGROUP:
     case POSIX_SPAWN_SETSIGDEF:
     case POSIX_SPAWN_SETSIGMASK:
-    case POSIX_SPAWN_SETSCHEDPARAM:
-    case POSIX_SPAWN_SETSCHEDULER:
+    case (POSIX_SPAWN_SETSCHEDPARAM != 0 ? POSIX_SPAWN_SETSCHEDPARAM : -1):
+    case (POSIX_SPAWN_SETSCHEDULER != 0 ? POSIX_SPAWN_SETSCHEDULER : -2):
       ;
     }
   return s.c + s.d;