[pkgsrc/trunk]: pkgsrc/net/bind916 net/bind916 Use atomic 32-bit integers whe...

To: pkgsrc-changes-hg%NetBSD.org@localhost
Subject: [pkgsrc/trunk]: pkgsrc/net/bind916 net/bind916 Use atomic 32-bit integers whe...
From: rin <rin%pkgsrc.org@localhost>
Date: Tue, 20 Jul 2021 07:44:44 +0000
details:   https://anonhg.NetBSD.org/pkgsrc/rev/7f8b1dd3a801
branches:  trunk
changeset: 456116:7f8b1dd3a801
user:      rin <rin%pkgsrc.org@localhost>
date:      Tue Jul 20 06:39:45 2021 +0000

description:
net/bind916 Use atomic 32-bit integers where appropriate, revision++

Mostly taken from NetBSD base:

- For counters, make sure to use 32-bit integers for !_LP64 platforms.
  In the previous revisions, this is partially done, i.e., incomplete.

- For flags fit within 32-bit width, use 32-bit integers for everyone.
  In the previous, this is incomplete, and restricted for __NetBSD__.
  Fix and generalize to everyone.

- Make comments in patches more helpful.

Fix build for ILP32 platforms as reported in PR pkg/56315.

Thanks jklos@ for testing.

diffstat:

 net/bind916/Makefile                                  |   4 +-
 net/bind916/distinfo                                  |  16 +++++------
 net/bind916/patches/patch-lib_dns_include_dns_zone.h  |   8 ++--
 net/bind916/patches/patch-lib_dns_zone.c              |  26 +++++++++++++-----
 net/bind916/patches/patch-lib_isc_include_isc_types.h |  15 ++++++++++
 net/bind916/patches/patch-lib_isc_stats.c             |   4 +-
 net/bind916/patches/patch-lib_ns_client.c             |  15 ++++++++++
 net/bind916/patches/patch-lib_ns_include_ns_client.h  |   4 +-
 8 files changed, 65 insertions(+), 27 deletions(-)

diffs (205 lines):

diff -r f0667a611b22 -r 7f8b1dd3a801 net/bind916/Makefile
--- a/net/bind916/Makefile      Tue Jul 20 06:37:49 2021 +0000
+++ b/net/bind916/Makefile      Tue Jul 20 06:39:45 2021 +0000
@@ -1,8 +1,8 @@
-# $NetBSD: Makefile,v 1.19 2021/06/23 20:33:14 nia Exp $
+# $NetBSD: Makefile,v 1.20 2021/07/20 06:39:45 rin Exp $
 
 DISTNAME=      bind-${BIND_VERSION}
 PKGNAME=       ${DISTNAME:S/-P/pl/}
-PKGREVISION=   1
+PKGREVISION=   2
 CATEGORIES=    net
 MASTER_SITES=  ftp://ftp.isc.org/isc/bind9/${BIND_VERSION}/
 EXTRACT_SUFX=  .tar.xz
diff -r f0667a611b22 -r 7f8b1dd3a801 net/bind916/distinfo
--- a/net/bind916/distinfo      Tue Jul 20 06:37:49 2021 +0000
+++ b/net/bind916/distinfo      Tue Jul 20 06:39:45 2021 +0000
@@ -1,4 +1,4 @@
-$NetBSD: distinfo,v 1.16 2021/06/19 13:18:21 otis Exp $
+$NetBSD: distinfo,v 1.17 2021/07/20 06:39:45 rin Exp $
 
 SHA1 (bind-9.16.18.tar.xz) = d4d5dc9097647a75732ff3fdd9ed370bb3bca7d4
 RMD160 (bind-9.16.18.tar.xz) = c26ba6a638f4dafe766fff7c45ee22d3fe56c049
@@ -23,7 +23,7 @@
 SHA1 (patch-lib_dns_byaddr.c) = 647ddaaaf040233e18d1a87d83bc2bd63d2a20e3
 SHA1 (patch-lib_dns_dnsrps.c) = b4c66a808eeb1e859801028afacd7237cd89903a
 SHA1 (patch-lib_dns_gssapi__link.c) = acd5f3c975d4edf391e77fd39cfa91810ad17ba2
-SHA1 (patch-lib_dns_include_dns_zone.h) = 8abf4c18935629624e68a14361dbeaf00a440c90
+SHA1 (patch-lib_dns_include_dns_zone.h) = aafd99bf0b0c84aa69b444106608c6cff61938f1
 SHA1 (patch-lib_dns_lookup.c) = 6c7463aca16abf6bd578aba1733a3217608a39d3
 SHA1 (patch-lib_dns_peer.c) = 16cc26fd4e792a23caef6e091f94f974041179e7
 SHA1 (patch-lib_dns_pkcs11ecdsa__link.c) = 99f386b056c5a6471f69841c41a2698d36b6b275
@@ -35,23 +35,21 @@
 SHA1 (patch-lib_dns_sdlz.c) = 4fc15a577c64501c10c144eab147e54686e80309
 SHA1 (patch-lib_dns_validator.c) = 03dd60d4c38be7248d1e07f2c29ddd543b5f7454
 SHA1 (patch-lib_dns_view.c) = 5b092f0344b92d003f1fe1f28e4cc4c76b2505cf
-SHA1 (patch-lib_dns_zone.c) = cad37c0fc4a05a2adb6159d4e1453fdbd3f08b41
+SHA1 (patch-lib_dns_zone.c) = e8f7062204818df36de80c4856821b05d7f61968
 SHA1 (patch-lib_isc_backtrace.c) = 1b6bca9b3de7f7298882c6c88274b0baf1dad507
 SHA1 (patch-lib_isc_include_isc_netmgr.h) = 48ac44c6a9b81e6b442deba6c075653d3691464b
 SHA1 (patch-lib_isc_include_isc_socket.h) = dc6376cd9e8391fa96efd805faee1a5d0647a142
-SHA1 (patch-lib_isc_include_isc_types.h) = bc0cdeb762d974b7be6e602b7dc2aac17fa7fe2f
-SHA1 (patch-lib_isc_include_isc_util.h) = 4a3998c2f0c95bffedf2e00b16a9c80475159775
+SHA1 (patch-lib_isc_include_isc_types.h) = e5fcf15ee03d1d032a67f2e6d23b2e409042ab64
 SHA1 (patch-lib_isc_netmgr_netmgr.c) = 63168c8901646e7b2079e1dc3324087508bb1e62
 SHA1 (patch-lib_isc_rwlock.c) = 1d114248ddee20db7a7429afab446f8b2f0dca82
-SHA1 (patch-lib_isc_stats.c) = 024a372d9919fa751b894f8fc8267691297f00f4
-SHA1 (patch-lib_isc_task.c) = 1947084d986f8e7fda5babd308171c498e20b7ec
+SHA1 (patch-lib_isc_stats.c) = 8d962fa360740770588fccf1d303d7fe22ae724b
 SHA1 (patch-lib_isc_unix_include_isc_align.h) = 1bbd78f1617a40079d1044175cfe037bbd1d95b2
 SHA1 (patch-lib_isc_unix_net.c) = c654f60a1feefdba9bf980dcfa46ce37f46918aa
 SHA1 (patch-lib_isc_unix_socket.c) = ed1816b50698107cb2034e0e3c61406ee458e697
 SHA1 (patch-lib_isc_unix_time.c) = fac9e66754e099f53ba8c499f5f179825a0b0fbd
 SHA1 (patch-lib_ns_Makefile.in) = 5d3ecf4d6673cf227635c03779e9969e6eeb8227
-SHA1 (patch-lib_ns_client.c) = c5d9e41f58027bdfa6a1cb43cd2cd35b44e85227
-SHA1 (patch-lib_ns_include_ns_client.h) = 56009c601d54491ffd16f6438d55206705a5aa23
+SHA1 (patch-lib_ns_client.c) = db2ee0cc915f20ae09beb2c8a917e0b3e60dfedc
+SHA1 (patch-lib_ns_include_ns_client.h) = 3ccd630aee77ea638fb3b954b85bad9c74b0c86e
 SHA1 (patch-lib_ns_include_ns_pfilter.h) = cc86752971b4f9f7492283c4ad3ff29bc1bae237
 SHA1 (patch-lib_ns_interfacemgr.c) = fc31720734b02155acd7fa9b370a6ebb82022532
 SHA1 (patch-lib_ns_pfilter.c) = 8f4a3b3a729360a131eb1962c42a9f9f985c7e7b
diff -r f0667a611b22 -r 7f8b1dd3a801 net/bind916/patches/patch-lib_dns_include_dns_zone.h
--- a/net/bind916/patches/patch-lib_dns_include_dns_zone.h      Tue Jul 20 06:37:49 2021 +0000
+++ b/net/bind916/patches/patch-lib_dns_include_dns_zone.h      Tue Jul 20 06:39:45 2021 +0000
@@ -1,6 +1,6 @@
-$NetBSD: patch-lib_dns_include_dns_zone.h,v 1.1 2020/08/09 15:20:22 taca Exp $
+$NetBSD: patch-lib_dns_include_dns_zone.h,v 1.2 2021/07/20 06:39:45 rin Exp $
 
-* Take from NetBSD base.
+No need to use atomic 64-bit integers for flags fit within 32-bit width.
 
 --- lib/dns/include/dns/zone.h.orig    2020-05-06 09:59:35.000000000 +0000
 +++ lib/dns/include/dns/zone.h
@@ -8,7 +8,7 @@
        DNS_ZONEOPT_CHECKSPF = 1 << 27,         /*%< check SPF records */
        DNS_ZONEOPT_CHECKTTL = 1 << 28,         /*%< check max-zone-ttl */
        DNS_ZONEOPT_AUTOEMPTY = 1 << 29,        /*%< automatic empty zone */
-+#ifndef __NetBSD__
++#if 0
        DNS_ZONEOPT___MAX = UINT64_MAX, /* trick to make the ENUM 64-bit wide */
 +#endif
  } dns_zoneopt_t;
@@ -18,7 +18,7 @@
        DNS_ZONEKEY_CREATE = 0x00000004U,   /*%< make keys when needed */
        DNS_ZONEKEY_FULLSIGN = 0x00000008U, /*%< roll to new keys immediately */
        DNS_ZONEKEY_NORESIGN = 0x00000010U, /*%< no automatic resigning */
-+#ifndef __NetBSD__
++#if 0
        DNS_ZONEKEY___MAX = UINT64_MAX, /* trick to make the ENUM 64-bit wide */
 +#endif
  } dns_zonekey_t;
diff -r f0667a611b22 -r 7f8b1dd3a801 net/bind916/patches/patch-lib_dns_zone.c
--- a/net/bind916/patches/patch-lib_dns_zone.c  Tue Jul 20 06:37:49 2021 +0000
+++ b/net/bind916/patches/patch-lib_dns_zone.c  Tue Jul 20 06:39:45 2021 +0000
@@ -1,14 +1,14 @@
-$NetBSD: patch-lib_dns_zone.c,v 1.2 2021/03/21 04:16:17 taca Exp $
+$NetBSD: patch-lib_dns_zone.c,v 1.3 2021/07/20 06:39:45 rin Exp $
 
-* Take from NetBSD base.
+No need to use atomic 64-bit integers for flags fit within 32-bit width.
 
---- lib/dns/zone.c.orig        2021-03-11 13:20:59.000000000 +0000
-+++ lib/dns/zone.c
-@@ -224,8 +224,13 @@ struct dns_zone {
+--- lib/dns/zone.c.orig        2021-06-18 19:08:07.000000000 +0900
++++ lib/dns/zone.c     2021-07-19 08:46:21.613595923 +0900
+@@ -241,8 +241,13 @@ struct dns_zone {
        int32_t journalsize;
        dns_rdataclass_t rdclass;
        dns_zonetype_t type;
-+#ifdef __NetBSD__
++#if 1
 +      atomic_uint_fast32_t flags;
 +      atomic_uint_fast32_t options;
 +#else
@@ -18,11 +18,11 @@
        unsigned int db_argc;
        char **db_argv;
        isc_time_t expiretime;
-@@ -370,7 +375,11 @@ struct dns_zone {
+@@ -387,7 +392,11 @@ struct dns_zone {
        /*%
         * Autosigning/key-maintenance options
         */
-+#ifdef __NetBSD__
++#if 1
 +      atomic_uint_fast32_t keyopts;
 +#else
        atomic_uint_fast64_t keyopts;
@@ -30,3 +30,13 @@
  
        /*%
         * True if added by "rndc addzone"
+@@ -524,7 +533,9 @@ typedef enum {
+        * support 64-bit enum values. Currently they fail on
+        * Windows.
+        */
++#if 1
+       DNS_ZONEFLG___MAX = UINT64_MAX, /* trick to make the ENUM 64-bit wide */
++#endif
+ } dns_zoneflg_t;
+ 
+ #define DNS_ZONE_OPTION(z, o)  ((atomic_load_relaxed(&(z)->options) & (o)) != 0)
diff -r f0667a611b22 -r 7f8b1dd3a801 net/bind916/patches/patch-lib_isc_include_isc_types.h
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/net/bind916/patches/patch-lib_isc_include_isc_types.h     Tue Jul 20 06:39:45 2021 +0000
@@ -0,0 +1,15 @@
+$NetBSD: patch-lib_isc_include_isc_types.h,v 1.3 2021/07/20 06:39:45 rin Exp $
+
+Use 32-bit atomic integers for !_LP64 platforms.
+
+--- lib/isc/include/isc/types.h.orig   2021-07-19 01:22:42.547182690 +0900
++++ lib/isc/include/isc/types.h        2021-07-19 01:22:47.029721976 +0900
+@@ -83,7 +83,7 @@ typedef struct isc_socket      isc_socke
+ typedef struct isc_socketevent isc_socketevent_t;     /*%< Socket Event */
+ typedef struct isc_socketmgr   isc_socketmgr_t;             /*%< Socket Manager */
+ typedef struct isc_stats       isc_stats_t;         /*%< Statistics */
+-#if defined(_WIN32) && !defined(_WIN64)
++#if (defined(_WIN32) && !defined(_WIN64)) || !defined(_LP64)
+ typedef int_fast32_t isc_statscounter_t; /*%< Statistics Counter */
+ #else  /* if defined(_WIN32) && !defined(_WIN64) */
+ typedef int_fast64_t isc_statscounter_t;
diff -r f0667a611b22 -r 7f8b1dd3a801 net/bind916/patches/patch-lib_isc_stats.c
--- a/net/bind916/patches/patch-lib_isc_stats.c Tue Jul 20 06:37:49 2021 +0000
+++ b/net/bind916/patches/patch-lib_isc_stats.c Tue Jul 20 06:39:45 2021 +0000
@@ -1,6 +1,6 @@
-$NetBSD: patch-lib_isc_stats.c,v 1.1 2020/08/09 15:20:22 taca Exp $
+$NetBSD: patch-lib_isc_stats.c,v 1.2 2021/07/20 06:39:45 rin Exp $
 
-* Platform fixes from NetBSD base system.
+Use 32-bit atomic integers for !_LP64 platforms.
 
 --- lib/isc/stats.c.orig       2020-05-06 09:59:35.000000000 +0000
 +++ lib/isc/stats.c
diff -r f0667a611b22 -r 7f8b1dd3a801 net/bind916/patches/patch-lib_ns_client.c
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/net/bind916/patches/patch-lib_ns_client.c Tue Jul 20 06:39:45 2021 +0000
@@ -0,0 +1,15 @@
+$NetBSD: patch-lib_ns_client.c,v 1.4 2021/07/20 06:39:45 rin Exp $
+
+Use 32-bit atomic integers for !_LP64 platforms.
+
+--- lib/ns/client.c.orig       2021-07-19 08:32:57.384417149 +0900
++++ lib/ns/client.c    2021-07-19 08:33:25.780329755 +0900
+@@ -126,7 +126,7 @@
+  * Number of tasks to be used by clients - those are used only when recursing
+  */
+ 
+-#if defined(_WIN32) && !defined(_WIN64)
++#if (defined(_WIN32) && !defined(_WIN64)) || !defined(_LP64)
+ LIBNS_EXTERNAL_DATA atomic_uint_fast32_t ns_client_requests =
+       ATOMIC_VAR_INIT(0);
+ #else  /* if defined(_WIN32) && !defined(_WIN64) */
diff -r f0667a611b22 -r 7f8b1dd3a801 net/bind916/patches/patch-lib_ns_include_ns_client.h
--- a/net/bind916/patches/patch-lib_ns_include_ns_client.h      Tue Jul 20 06:37:49 2021 +0000
+++ b/net/bind916/patches/patch-lib_ns_include_ns_client.h      Tue Jul 20 06:39:45 2021 +0000
@@ -1,6 +1,6 @@
-$NetBSD: patch-lib_ns_include_ns_client.h,v 1.1 2020/08/09 15:20:22 taca Exp $
+$NetBSD: patch-lib_ns_include_ns_client.h,v 1.2 2021/07/20 06:39:45 rin Exp $
 
-* Take from NetBSD base.
+Use 32-bit atomic integers for !_LP64 platforms.
 
 --- lib/ns/include/ns/client.h.orig    2020-05-06 09:59:35.000000000 +0000
 +++ lib/ns/include/ns/client.h
Prev by Date: [pkgsrc/trunk]: pkgsrc/net/bind916 net/bind916 --disable-atomic is no longer ...
Next by Date: [pkgsrc/trunk]: pkgsrc/doc doc: Updated net/bind916 to 9.16.18nb2
Previous by Thread: [pkgsrc/trunk]: pkgsrc/net/bind916 net/bind916 --disable-atomic is no longer ...
Next by Thread: [pkgsrc/trunk]: pkgsrc/doc doc: Updated net/bind916 to 9.16.18nb2
Indexes:
Home | Main Index | Thread Index | Old Index