CVS commit: pkgsrc/net/bind916

To: pkgsrc-changes%NetBSD.org@localhost
Subject: CVS commit: pkgsrc/net/bind916
From: "Rin Okuyama" <rin%netbsd.org@localhost>
Date: Tue, 20 Jul 2021 06:39:45 +0000

Module Name:    pkgsrc
Committed By:   rin
Date:           Tue Jul 20 06:39:45 UTC 2021

Modified Files:
        pkgsrc/net/bind916: Makefile distinfo
        pkgsrc/net/bind916/patches: patch-lib_dns_include_dns_zone.h
            patch-lib_dns_zone.c patch-lib_isc_stats.c
            patch-lib_ns_include_ns_client.h
Added Files:
        pkgsrc/net/bind916/patches: patch-lib_isc_include_isc_types.h
            patch-lib_ns_client.c

Log Message:
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.


To generate a diff of this commit:
cvs rdiff -u -r1.19 -r1.20 pkgsrc/net/bind916/Makefile
cvs rdiff -u -r1.16 -r1.17 pkgsrc/net/bind916/distinfo
cvs rdiff -u -r1.1 -r1.2 \
    pkgsrc/net/bind916/patches/patch-lib_dns_include_dns_zone.h \
    pkgsrc/net/bind916/patches/patch-lib_isc_stats.c \
    pkgsrc/net/bind916/patches/patch-lib_ns_include_ns_client.h
cvs rdiff -u -r1.2 -r1.3 pkgsrc/net/bind916/patches/patch-lib_dns_zone.c
cvs rdiff -u -r0 -r1.3 \
    pkgsrc/net/bind916/patches/patch-lib_isc_include_isc_types.h
cvs rdiff -u -r0 -r1.4 pkgsrc/net/bind916/patches/patch-lib_ns_client.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: pkgsrc/net/bind916/Makefile
diff -u pkgsrc/net/bind916/Makefile:1.19 pkgsrc/net/bind916/Makefile:1.20
--- pkgsrc/net/bind916/Makefile:1.19    Wed Jun 23 20:33:14 2021
+++ pkgsrc/net/bind916/Makefile Tue Jul 20 06:39:45 2021
@@ -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

Index: pkgsrc/net/bind916/distinfo
diff -u pkgsrc/net/bind916/distinfo:1.16 pkgsrc/net/bind916/distinfo:1.17
--- pkgsrc/net/bind916/distinfo:1.16    Sat Jun 19 13:18:21 2021
+++ pkgsrc/net/bind916/distinfo Tue Jul 20 06:39:45 2021
@@ -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-contrib_dlz_modules_wildcard
 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_sdb.c) = ed447ec7a13
 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

Index: pkgsrc/net/bind916/patches/patch-lib_dns_include_dns_zone.h
diff -u pkgsrc/net/bind916/patches/patch-lib_dns_include_dns_zone.h:1.1 pkgsrc/net/bind916/patches/patch-lib_dns_include_dns_zone.h:1.2
--- pkgsrc/net/bind916/patches/patch-lib_dns_include_dns_zone.h:1.1     Sun Aug  9 15:20:22 2020
+++ pkgsrc/net/bind916/patches/patch-lib_dns_include_dns_zone.h Tue Jul 20 06:39:45 2021
@@ -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 @@ $NetBSD: patch-lib_dns_include_dns_zone.
        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 @@ $NetBSD: patch-lib_dns_include_dns_zone.
        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;
Index: pkgsrc/net/bind916/patches/patch-lib_isc_stats.c
diff -u pkgsrc/net/bind916/patches/patch-lib_isc_stats.c:1.1 pkgsrc/net/bind916/patches/patch-lib_isc_stats.c:1.2
--- pkgsrc/net/bind916/patches/patch-lib_isc_stats.c:1.1        Sun Aug  9 15:20:22 2020
+++ pkgsrc/net/bind916/patches/patch-lib_isc_stats.c    Tue Jul 20 06:39:45 2021
@@ -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
Index: pkgsrc/net/bind916/patches/patch-lib_ns_include_ns_client.h
diff -u pkgsrc/net/bind916/patches/patch-lib_ns_include_ns_client.h:1.1 pkgsrc/net/bind916/patches/patch-lib_ns_include_ns_client.h:1.2
--- pkgsrc/net/bind916/patches/patch-lib_ns_include_ns_client.h:1.1     Sun Aug  9 15:20:22 2020
+++ pkgsrc/net/bind916/patches/patch-lib_ns_include_ns_client.h Tue Jul 20 06:39:45 2021
@@ -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

Index: pkgsrc/net/bind916/patches/patch-lib_dns_zone.c
diff -u pkgsrc/net/bind916/patches/patch-lib_dns_zone.c:1.2 pkgsrc/net/bind916/patches/patch-lib_dns_zone.c:1.3
--- pkgsrc/net/bind916/patches/patch-lib_dns_zone.c:1.2 Sun Mar 21 04:16:17 2021
+++ pkgsrc/net/bind916/patches/patch-lib_dns_zone.c     Tue Jul 20 06:39:45 2021
@@ -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 @@ $NetBSD: patch-lib_dns_zone.c,v 1.2 2021
        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 @@ $NetBSD: patch-lib_dns_zone.c,v 1.2 2021
  
        /*%
         * 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)

Added files:

Index: pkgsrc/net/bind916/patches/patch-lib_isc_include_isc_types.h
diff -u /dev/null pkgsrc/net/bind916/patches/patch-lib_isc_include_isc_types.h:1.3
--- /dev/null   Tue Jul 20 06:39:45 2021
+++ pkgsrc/net/bind916/patches/patch-lib_isc_include_isc_types.h        Tue Jul 20 06:39:45 2021
@@ -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;

Index: pkgsrc/net/bind916/patches/patch-lib_ns_client.c
diff -u /dev/null pkgsrc/net/bind916/patches/patch-lib_ns_client.c:1.4
--- /dev/null   Tue Jul 20 06:39:45 2021
+++ pkgsrc/net/bind916/patches/patch-lib_ns_client.c    Tue Jul 20 06:39:45 2021
@@ -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) */

Prev by Date: CVS commit: pkgsrc/doc
Next by Date: CVS commit: pkgsrc/net/bind916
Previous by Thread: CVS commit: pkgsrc/doc
Next by Thread: CVS commit: pkgsrc/net/bind916
Indexes:

Home | Main Index | Thread Index | Old Index