Remove __need_list_t and __need_res_state.
authorZack Weinberg <zackw@panix.com>
Wed, 22 Mar 2017 13:55:25 +0000 (09:55 -0400)
committerZack Weinberg <zackw@panix.com>
Sat, 20 May 2017 23:01:46 +0000 (19:01 -0400)
These __need macros are only used internally, by nptl/descr.h.
However, including all of resolv.h from descr.h causes build failures
due to resolv.h's dozens of pseudo-struct-field macros, some of which
collide with struct fields in NPTL internal data structures.
Similarly, including all of list.h from descr.h produces an include
cycle, atomic.h -> atomic-machine.h -> tls.h -> descr.h -> list.h ->
atomic.h, and then list.h tries to use atomic.h macros that haven't
been defined yet.  So we do need mini-headers for these.  In the
list.h case I called it include/list_t.h since it isn't going to be
installed.

* resolv/resolv.h: Remove __need_res_state logic.
Move definition of res_state and related constants to ...
* resolv/bits/types/res_state.h: ...this new file.
* resolv/Makefile: Install bits/types/res_state.h.
* include/bits/types/res_state.h: New wrapper.
* include/list.h: Remove __need_list_t logic.
Move definition of list_t to ...
* include/list_t.h: ...this new file.

* nptl/descr.h: Include list_t.h and bits/types/res_state.h
instead of list.h and resolv.h.

ChangeLog
include/bits/types/res_state.h [new file with mode: 0644]
include/list.h
include/list_t.h [new file with mode: 0644]
nptl/descr.h
resolv/Makefile
resolv/bits/types/res_state.h [new file with mode: 0644]
resolv/resolv.h

index 5e7b766..43beed2 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,17 @@
+2017-05-20  Zack Weinberg  <zackw@panix.com>
+
+       * resolv/resolv.h: Remove __need_res_state logic.
+       Move definition of res_state and related constants to ...
+       * resolv/bits/types/res_state.h: ...this new file.
+       * resolv/Makefile: Install bits/types/res_state.h.
+       * include/bits/types/res_state.h: New wrapper.
+       * include/list.h: Remove __need_list_t logic.
+       Move definition of list_t to ...
+       * include/list_t.h: ...this new file.
+
+       * nptl/descr.h: Include list_t.h and bits/types/res_state.h
+       instead of list.h and resolv.h.
+
 2017-05-20  H.J. Lu  <hongjiu.lu@intel.com>
 
        * sysdeps/x86_64/rawmemchr.S (L(return_null)): Removed.
diff --git a/include/bits/types/res_state.h b/include/bits/types/res_state.h
new file mode 100644 (file)
index 0000000..4b99b68
--- /dev/null
@@ -0,0 +1 @@
+#include <resolv/bits/types/res_state.h>
index 6846783..dec5dee 100644 (file)
    <http://www.gnu.org/licenses/>.  */
 
 #ifndef _LIST_H
+#define _LIST_H        1
 
-#ifndef __need_list_t
-# define _LIST_H       1
-#endif
+/* Internal: doubly linked lists.  */
 
 /* The definitions of this file are adopted from those which can be
    found in the Linux kernel headers to enable people familiar with
    the latter find their way in these sources as well.  */
 
-
-#if defined __need_list_t || defined _LIST_H
-# ifndef __list_t_defined
-#  define __list_t_defined
-/* Basic type for the double-link list.  */
-typedef struct list_head
-{
-  struct list_head *next;
-  struct list_head *prev;
-} list_t;
-# endif
-# undef __need_list_t
-#endif
-
-#ifdef _LIST_H
-
-# include <atomic.h>
+#include <list_t.h>
+#include <atomic.h>
 
 /* Define a variable with the head and tail of the list.  */
-# define LIST_HEAD(name) \
+#define LIST_HEAD(name) \
   list_t name = { &(name), &(name) }
 
 /* Initialize a new list head.  */
-# define INIT_LIST_HEAD(ptr) \
+#define INIT_LIST_HEAD(ptr) \
   (ptr)->next = (ptr)->prev = (ptr)
 
 
@@ -90,28 +74,26 @@ list_splice (list_t *add, list_t *head)
 
 
 /* Get typed element from list at a given position.  */
-# define list_entry(ptr, type, member) \
+#define list_entry(ptr, type, member) \
   ((type *) ((char *) (ptr) - (unsigned long) (&((type *) 0)->member)))
 
 
 
 /* Iterate forward over the elements of the list.  */
-# define list_for_each(pos, head) \
+#define list_for_each(pos, head) \
   for (pos = (head)->next; pos != (head); pos = pos->next)
 
 
 /* Iterate forward over the elements of the list.  */
-# define list_for_each_prev(pos, head) \
+#define list_for_each_prev(pos, head) \
   for (pos = (head)->prev; pos != (head); pos = pos->prev)
 
 
 /* Iterate backwards over the elements list.  The list elements can be
    removed from the list while doing this.  */
-# define list_for_each_prev_safe(pos, p, head) \
+#define list_for_each_prev_safe(pos, p, head) \
   for (pos = (head)->prev, p = pos->prev; \
        pos != (head); \
        pos = p, p = pos->prev)
 
-#endif /* _LIST_H */
-
 #endif /* list.h */
diff --git a/include/list_t.h b/include/list_t.h
new file mode 100644 (file)
index 0000000..d737e2e
--- /dev/null
@@ -0,0 +1,31 @@
+/* Copyright (C) 2002-2017 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
+   <http://www.gnu.org/licenses/>.  */
+
+#ifndef _LIST_T_H
+#define _LIST_T_H 1
+
+/* Internal: doubly linked lists.  */
+
+/* Basic type for the double-link list.  */
+typedef struct list_head
+{
+  struct list_head *next;
+  struct list_head *prev;
+} list_t;
+
+#endif /* list_t.h */
index a145860..c5ad0c8 100644 (file)
 #include <stdbool.h>
 #include <sys/types.h>
 #include <hp-timing.h>
-#define __need_list_t
-#include <list.h>
+#include <list_t.h>
 #include <lowlevellock.h>
 #include <pthreaddef.h>
 #include <dl-sysdep.h>
 #include "../nptl_db/thread_db.h"
 #include <tls.h>
 #include <unwind.h>
-#define __need_res_state
-#include <resolv.h>
+#include <bits/types/res_state.h>
 #include <kernel-features.h>
 
 #ifndef TCB_ALIGNMENT
index d41fd46..70b6f4f 100644 (file)
@@ -22,7 +22,7 @@ subdir        := resolv
 
 include ../Makeconfig
 
-headers        := resolv.h \
+headers        := resolv.h bits/types/res_state.h \
           netdb.h bits/netdb.h \
           arpa/nameser.h arpa/nameser_compat.h \
           sys/bitypes.h
diff --git a/resolv/bits/types/res_state.h b/resolv/bits/types/res_state.h
new file mode 100644 (file)
index 0000000..cee4b6d
--- /dev/null
@@ -0,0 +1,61 @@
+#ifndef __res_state_defined
+#define __res_state_defined 1
+
+#include <sys/types.h>
+#include <netinet/in.h>
+
+/* res_state: the global state used by the resolver stub.  */
+#define MAXNS                  3       /* max # name servers we'll track */
+#define MAXDFLSRCH             3       /* # default domain levels to try */
+#define MAXDNSRCH              6       /* max # domains in search path */
+#define MAXRESOLVSORT          10      /* number of net to sort on */
+
+struct __res_state {
+       int     retrans;                /* retransmition time interval */
+       int     retry;                  /* number of times to retransmit */
+       unsigned long options;          /* option flags - see below. */
+       int     nscount;                /* number of name servers */
+       struct sockaddr_in
+               nsaddr_list[MAXNS];     /* address of name server */
+       unsigned short id;              /* current message id */
+       /* 2 byte hole here.  */
+       char    *dnsrch[MAXDNSRCH+1];   /* components of domain to search */
+       char    defdname[256];          /* default domain (deprecated) */
+       unsigned long pfcode;           /* RES_PRF_ flags - see below. */
+       unsigned ndots:4;               /* threshold for initial abs. query */
+       unsigned nsort:4;               /* number of elements in sort_list[] */
+       unsigned ipv6_unavail:1;        /* connecting to IPv6 server failed */
+       unsigned unused:23;
+       struct {
+               struct in_addr  addr;
+               uint32_t        mask;
+       } sort_list[MAXRESOLVSORT];
+       /* 4 byte hole here on 64-bit architectures.  */
+       void * __glibc_unused_qhook;
+       void * __glibc_unused_rhook;
+       int     res_h_errno;            /* last one set for this context */
+       int     _vcsock;                /* PRIVATE: for res_send VC i/o */
+       unsigned int _flags;            /* PRIVATE: see below */
+       /* 4 byte hole here on 64-bit architectures.  */
+       union {
+               char    pad[52];        /* On an i386 this means 512b total. */
+               struct {
+                       uint16_t                nscount;
+                       uint16_t                nsmap[MAXNS];
+                       int                     nssocks[MAXNS];
+                       uint16_t                nscount6;
+                       uint16_t                nsinit;
+                       struct sockaddr_in6     *nsaddrs[MAXNS];
+#ifdef _LIBC
+                       unsigned long long int  initstamp
+                         __attribute__((packed));
+#else
+                       unsigned int            _initstamp[2];
+#endif
+               } _ext;
+       } _u;
+};
+
+typedef struct __res_state *res_state;
+
+#endif /* __res_state_defined */
index 9fef8e9..1fb0ad4 100644 (file)
  */
 
 #ifndef _RESOLV_H_
+#define _RESOLV_H_
 
-/* These headers are needed for types used in the `struct res_state'
-   declaration.  */
+#include <sys/cdefs.h>
+#include <sys/param.h>
 #include <sys/types.h>
+#include <stdio.h>
 #include <netinet/in.h>
-
-#ifndef __need_res_state
-# define _RESOLV_H_
-
-# include <sys/param.h>
-# include <sys/cdefs.h>
-# include <stdio.h>
-# include <arpa/nameser.h>
-#endif
-
-#ifndef __res_state_defined
-# define __res_state_defined
+#include <arpa/nameser.h>
+#include <bits/types/res_state.h>
 
 /*
  * Global defines and variables for resolver stub.
  */
-# define MAXNS                 3       /* max # name servers we'll track */
-# define MAXDFLSRCH            3       /* # default domain levels to try */
-# define MAXDNSRCH             6       /* max # domains in search path */
-# define LOCALDOMAINPARTS      2       /* min levels in name that is "local" */
-
-# define RES_TIMEOUT           5       /* min. seconds between retries */
-# define MAXRESOLVSORT         10      /* number of net to sort on */
-# define RES_MAXNDOTS          15      /* should reflect bit field size */
-# define RES_MAXRETRANS                30      /* only for resolv.conf/RES_OPTIONS */
-# define RES_MAXRETRY          5       /* only for resolv.conf/RES_OPTIONS */
-# define RES_DFLRETRY          2       /* Default #/tries. */
-# define RES_MAXTIME           65535   /* Infinity, in milliseconds. */
+#define LOCALDOMAINPARTS       2       /* min levels in name that is "local" */
 
-struct __res_state {
-       int     retrans;                /* retransmition time interval */
-       int     retry;                  /* number of times to retransmit */
-       unsigned long options;          /* option flags - see below. */
-       int     nscount;                /* number of name servers */
-       struct sockaddr_in
-               nsaddr_list[MAXNS];     /* address of name server */
-# define nsaddr        nsaddr_list[0]          /* for backward compatibility */
-       unsigned short id;              /* current message id */
-       /* 2 byte hole here.  */
-       char    *dnsrch[MAXDNSRCH+1];   /* components of domain to search */
-       char    defdname[256];          /* default domain (deprecated) */
-       unsigned long pfcode;           /* RES_PRF_ flags - see below. */
-       unsigned ndots:4;               /* threshold for initial abs. query */
-       unsigned nsort:4;               /* number of elements in sort_list[] */
-       unsigned ipv6_unavail:1;        /* connecting to IPv6 server failed */
-       unsigned unused:23;
-       struct {
-               struct in_addr  addr;
-               uint32_t        mask;
-       } sort_list[MAXRESOLVSORT];
-       /* 4 byte hole here on 64-bit architectures.  */
-       void * __glibc_unused_qhook;
-       void * __glibc_unused_rhook;
-       int     res_h_errno;            /* last one set for this context */
-       int     _vcsock;                /* PRIVATE: for res_send VC i/o */
-       unsigned int _flags;            /* PRIVATE: see below */
-       /* 4 byte hole here on 64-bit architectures.  */
-       union {
-               char    pad[52];        /* On an i386 this means 512b total. */
-               struct {
-                       uint16_t                nscount;
-                       uint16_t                nsmap[MAXNS];
-                       int                     nssocks[MAXNS];
-                       uint16_t                nscount6;
-                       uint16_t                nsinit;
-                       struct sockaddr_in6     *nsaddrs[MAXNS];
-#ifdef _LIBC
-                       unsigned long long int  initstamp
-                         __attribute__((packed));
-#else
-                       unsigned int            _initstamp[2];
-#endif
-               } _ext;
-       } _u;
-};
+#define RES_TIMEOUT            5       /* min. seconds between retries */
+#define RES_MAXNDOTS           15      /* should reflect bit field size */
+#define RES_MAXRETRANS         30      /* only for resolv.conf/RES_OPTIONS */
+#define RES_MAXRETRY           5       /* only for resolv.conf/RES_OPTIONS */
+#define RES_DFLRETRY           2       /* Default #/tries. */
+#define RES_MAXTIME            65535   /* Infinity, in milliseconds. */
 
-typedef struct __res_state *res_state;
-# undef __need_res_state
-#endif
+#define nsaddr nsaddr_list[0]          /* for backward compatibility */
 
-#ifdef _RESOLV_H_
 /*
  * Revision information.  This is the release date in YYYYMMDD format.
  * It can change every day so the right thing to do with it is use it
@@ -361,7 +299,7 @@ int         res_nmkquery (res_state, int, const char *, int, int,
 int            res_nsend (res_state, const unsigned char *, int,
                           unsigned char *, int) __THROW;
 void           res_nclose (res_state) __THROW;
+
 __END_DECLS
-#endif
 
 #endif /* !_RESOLV_H_ */