# HG changeset patch # User Daiki Ueno # Date 1416312669 -32400 # Node ID c76a0687b1a70f3801840ba80260852ceeade26c # Parent 629de2a70c2be05bcb8f171fa14503c70332e38a pipe-filter-gi, pipe-filter-ii: port to AIX On AIX 7.1, 'select' is defined as static and cannot be referred to from inline function. * lib/pipe-filter-aux.h (nointr_select): Remove, manually expand the definition... * lib/pipe-filter-gi.c (filter_loop): ...here, and... * lib/pipe-filter-ii.c (pipe_filter_ii_execute): ...here. diff --git a/ChangeLog b/ChangeLog --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,13 @@ +2014-11-22 Daiki Ueno + + pipe-filter-gi, pipe-filter-ii: port to AIX + On AIX 7.1, 'select' is defined as static and cannot be referred + to from inline function. + * lib/pipe-filter-aux.h (nointr_select): Remove, manually expand + the definition... + * lib/pipe-filter-gi.c (filter_loop): ...here, and... + * lib/pipe-filter-ii.c (pipe_filter_ii_execute): ...here. + 2014-11-20 Paul Eggert gitlog-to-changelog: add --until diff --git a/lib/pipe-filter-aux.h b/lib/pipe-filter-aux.h --- a/lib/pipe-filter-aux.h +++ b/lib/pipe-filter-aux.h @@ -87,25 +87,6 @@ #undef write /* avoid warning on VMS */ #define write nonintr_write -# if HAVE_SELECT - -PIPE_FILTER_AUX_INLINE int -nonintr_select (int n, fd_set *readfds, fd_set *writefds, fd_set *exceptfds, - struct timeval *timeout) -{ - int retval; - - do - retval = select (n, readfds, writefds, exceptfds, timeout); - while (retval < 0 && errno == EINTR); - - return retval; -} -# undef select /* avoid warning on VMS */ -# define select nonintr_select - -# endif - #endif /* Non-blocking I/O. */ diff --git a/lib/pipe-filter-gi.c b/lib/pipe-filter-gi.c --- a/lib/pipe-filter-gi.c +++ b/lib/pipe-filter-gi.c @@ -302,7 +302,7 @@ /* Here, if done_writing, filter->reader_terminated is false. When filter->reader_terminated becomes true, this loop is terminated. */ # if HAVE_SELECT - int n; + int n, retval; /* See whether reading or writing is possible. */ n = 1; @@ -317,10 +317,16 @@ if (n <= filter->fd[1]) n = filter->fd[1] + 1; } - n = select (n, - (!filter->reader_terminated ? &filter->readfds : NULL), - (!done_writing ? &filter->writefds : NULL), - NULL, NULL); + /* Do EINTR handling here instead of in pipe-filter-aux.h, + because select() cannot be referred to from an inline + function on AIX 7.1. */ + do + retval = select (n, + (!filter->reader_terminated ? &filter->readfds : NULL), + (!done_writing ? &filter->writefds : NULL), + NULL, NULL); + while (retval < 0 && errno == EINTR); + n = retval; if (n < 0) { diff --git a/lib/pipe-filter-ii.c b/lib/pipe-filter-ii.c --- a/lib/pipe-filter-ii.c +++ b/lib/pipe-filter-ii.c @@ -309,7 +309,7 @@ for (;;) { # if HAVE_SELECT - int n; + int n, retval; FD_SET (fd[0], &readfds); n = fd[0] + 1; @@ -320,8 +320,15 @@ n = fd[1] + 1; } - n = select (n, &readfds, (!done_writing ? &writefds : NULL), NULL, - NULL); + /* Do EINTR handling here instead of in pipe-filter-aux.h, + because select() cannot be referred to from an inline + function on AIX 7.1. */ + do + retval = select (n, &readfds, (!done_writing ? &writefds : NULL), + NULL, NULL); + while (retval < 0 && errno == EINTR); + n = retval; + if (n < 0) { if (exit_on_error)