mirror of
https://git.savannah.gnu.org/git/guile.git
synced 2025-05-01 20:30:28 +02:00
0ebbcf43c4
As the Autoconf documentation says, "These days portable programs [...] should not rely on `HAVE_RESTARTABLE_SYSCALLS', since nowadays whether a system call is restartable is a dynamic issue, not a configuration-time issue." In other words, if we ever rely on HAVE_RESTARTABLE_SYSCALLS, we are at the mercy of any code that Guile happens to be linked with, because that code could install a signal handler without the SA_RESTART flag, and then a Guile system call could unexpectedly return EINTR. The readline part of this goes back to this problem report: http://sources.redhat.com/ml/guile/2000-05/msg00177.html; and is an excellent example of the above paragraph. It was noted during the discussion that undefining HAVE_RESTARTABLE_SYSCALLS would fix the problem, but that solution wasn't adopted - I guess because Guile was still using cooperative threads then (not pthreads) and so there was a significant concern (whether founded or not) that not using restartable syscalls (where available) could lead to a loss of performance. Now Guile's default mode of operation is with pthreads, where we already don't assume that HAVE_RESTARTABLE_SYSCALLS is reliable, so there is no possible further performance loss. And in any case we really have no choice, if we want correct operation. Thanks to Sylvain Beucler for reporting this and suggesting the fix. * configure.in (AC_SYS_RESTARTABLE_SYSCALLS): Removed. * doc/ref/posix.texi (Signals): Remove statement that Guile always sets SA_RESTART flag. * guile-readline/configure.in (GUILE_SIGWINCH_SA_RESTART_CLEARED): Remove this setting, together with its test code. (HAVE_RL_PRE_INPUT_HOOK): Remove this setting and its code, as no longer needed. * guile-readline/readline.c (sigwinch_enable_restart): Removed. (scm_init_readline): Remove setting of rl_pre_input_hook. * libguile/_scm.h (SCM_SYSCALL): Remove the definition that relies on HAVE_RESTARTABLE_SYSCALLS. * libguile/scmsigs.c (scm_sigaction_for_thread): Don't always set the SA_RESTART flag if available. Update docstring accordingly. (scm_init_scmsigs): Remove code that sets SA_RESTART flag for all signals. * THANKS: Add Sylvain.
165 lines
5.6 KiB
C
165 lines
5.6 KiB
C
/* classes: h_files */
|
||
|
||
#ifndef SCM__SCM_H
|
||
#define SCM__SCM_H
|
||
|
||
/* Copyright (C) 1995,1996,2000,2001, 2002, 2006, 2008 Free Software Foundation, Inc.
|
||
*
|
||
* This 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 3 of
|
||
* the License, or (at your option) any later version.
|
||
*
|
||
* This 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 this library; if not, write to the Free Software
|
||
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
|
||
* 02110-1301 USA
|
||
*/
|
||
|
||
|
||
|
||
/**********************************************************************
|
||
This file is Guile's central private header.
|
||
|
||
When included by other files, this file should preceed any include
|
||
other than __scm.h. See __scm.h for details regarding the purpose of
|
||
and differences between _scm.h and __scm.h.
|
||
**********************************************************************/
|
||
|
||
#if defined(__ia64) && !defined(__ia64__)
|
||
# define __ia64__
|
||
#endif
|
||
|
||
#if HAVE_CONFIG_H
|
||
# include <config.h>
|
||
#endif
|
||
|
||
/* Undefine HAVE_STRUCT_TIMESPEC, because the libguile C code doesn't
|
||
need it anymore, and because on MinGW:
|
||
|
||
- the definition of struct timespec is provided (if at all) by
|
||
pthread.h
|
||
|
||
- pthread.h will _not_ define struct timespec if
|
||
HAVE_STRUCT_TIMESPEC is 1, because then it thinks that it doesn't
|
||
need to.
|
||
|
||
The libguile C code doesn't need HAVE_STRUCT_TIMESPEC anymore,
|
||
because the value of HAVE_STRUCT_TIMESPEC has already been
|
||
incorporated in how scm_t_timespec is defined (in scmconfig.h), and
|
||
the rest of the libguile C code now just uses scm_t_timespec.
|
||
*/
|
||
#ifdef HAVE_STRUCT_TIMESPEC
|
||
#undef HAVE_STRUCT_TIMESPEC
|
||
#endif
|
||
|
||
#include <errno.h>
|
||
#include "libguile/__scm.h"
|
||
|
||
/* Include headers for those files central to the implementation. The
|
||
rest should be explicitly #included in the C files themselves. */
|
||
#include "libguile/error.h" /* Everyone signals errors. */
|
||
#include "libguile/print.h" /* Everyone needs to print. */
|
||
#include "libguile/pairs.h" /* Everyone conses. */
|
||
#include "libguile/list.h" /* Everyone makes lists. */
|
||
#include "libguile/gc.h" /* Everyone allocates. */
|
||
#include "libguile/gsubr.h" /* Everyone defines global functions. */
|
||
#include "libguile/procs.h" /* Same. */
|
||
#include "libguile/numbers.h" /* Everyone deals with fixnums. */
|
||
#include "libguile/symbols.h" /* For length, chars, values, miscellany. */
|
||
#include "libguile/boolean.h" /* Everyone wonders about the truth. */
|
||
#include "libguile/threads.h" /* You are not alone. */
|
||
#include "libguile/snarf.h" /* Everyone snarfs. */
|
||
#include "libguile/variable.h"
|
||
#include "libguile/modules.h"
|
||
#include "libguile/inline.h"
|
||
|
||
#ifndef SCM_SYSCALL
|
||
#ifdef vms
|
||
# ifndef __GNUC__
|
||
# include <ssdef.h>
|
||
# define SCM_SYSCALL(line) do{errno = 0;line;} \
|
||
while(EVMSERR==errno && (vaxc$errno>>3)==(SS$_CONTROLC>>3))
|
||
# endif /* ndef __GNUC__ */
|
||
#endif /* def vms */
|
||
#endif /* ndef SCM_SYSCALL */
|
||
|
||
#ifndef SCM_SYSCALL
|
||
# ifdef EINTR
|
||
# if (EINTR > 0)
|
||
# define SCM_SYSCALL(line) do{errno = 0;line;}while(EINTR==errno)
|
||
# endif /* (EINTR > 0) */
|
||
# endif /* def EINTR */
|
||
#endif /* ndef SCM_SYSCALL */
|
||
|
||
#ifndef SCM_SYSCALL
|
||
# define SCM_SYSCALL(line) line;
|
||
#endif /* ndef SCM_SYSCALL */
|
||
|
||
|
||
|
||
#ifndef min
|
||
#define min(A, B) ((A) <= (B) ? (A) : (B))
|
||
#endif
|
||
#ifndef max
|
||
#define max(A, B) ((A) >= (B) ? (A) : (B))
|
||
#endif
|
||
|
||
|
||
|
||
#if GUILE_USE_64_CALLS && HAVE_STAT64
|
||
#define CHOOSE_LARGEFILE(foo,foo64) foo64
|
||
#else
|
||
#define CHOOSE_LARGEFILE(foo,foo64) foo
|
||
#endif
|
||
|
||
/* These names are a bit long, but they make it clear what they represent. */
|
||
#if SCM_HAVE_STRUCT_DIRENT64 == 1
|
||
# define dirent_or_dirent64 CHOOSE_LARGEFILE(dirent,dirent64)
|
||
#else
|
||
# define dirent_or_dirent64 dirent
|
||
#endif
|
||
#define fstat_or_fstat64 CHOOSE_LARGEFILE(fstat,fstat64)
|
||
#define ftruncate_or_ftruncate64 CHOOSE_LARGEFILE(ftruncate,ftruncate64)
|
||
#define lseek_or_lseek64 CHOOSE_LARGEFILE(lseek,lseek64)
|
||
#define lstat_or_lstat64 CHOOSE_LARGEFILE(lstat,lstat64)
|
||
#define off_t_or_off64_t CHOOSE_LARGEFILE(off_t,off64_t)
|
||
#define open_or_open64 CHOOSE_LARGEFILE(open,open64)
|
||
#define readdir_or_readdir64 CHOOSE_LARGEFILE(readdir,readdir64)
|
||
#if SCM_HAVE_READDIR64_R == 1
|
||
# define readdir_r_or_readdir64_r CHOOSE_LARGEFILE(readdir_r,readdir64_r)
|
||
#else
|
||
# define readdir_r_or_readdir64_r readdir_r
|
||
#endif
|
||
#define stat_or_stat64 CHOOSE_LARGEFILE(stat,stat64)
|
||
#define truncate_or_truncate64 CHOOSE_LARGEFILE(truncate,truncate64)
|
||
#define scm_from_off_t_or_off64_t CHOOSE_LARGEFILE(scm_from_off_t,scm_from_int64)
|
||
#define scm_from_ino_t_or_ino64_t CHOOSE_LARGEFILE(scm_from_ulong,scm_from_uint64)
|
||
#define scm_from_blkcnt_t_or_blkcnt64_t CHOOSE_LARGEFILE(scm_from_ulong,scm_from_uint64)
|
||
#define scm_to_off_t_or_off64_t CHOOSE_LARGEFILE(scm_to_off_t,scm_to_int64)
|
||
|
||
#if SIZEOF_OFF_T == 4
|
||
# define scm_to_off_t scm_to_int32
|
||
# define scm_from_off_t scm_from_int32
|
||
#elif SIZEOF_OFF_T == 8
|
||
# define scm_to_off_t scm_to_int64
|
||
# define scm_from_off_t scm_from_int64
|
||
#else
|
||
# error sizeof(off_t) is not 4 or 8.
|
||
#endif
|
||
#define scm_to_off64_t scm_to_int64
|
||
#define scm_from_off64_t scm_from_int64
|
||
|
||
|
||
#endif /* SCM__SCM_H */
|
||
|
||
/*
|
||
Local Variables:
|
||
c-file-style: "gnu"
|
||
End:
|
||
*/
|