Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 4 Oct 2020 19:37:16 +0000 (UTC)
From:      Ryan Moeller <freqlabs@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r366433 - in head/sys: contrib/ipfilter/netinet x86/bios
Message-ID:  <202010041937.094JbGZl055585@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: freqlabs
Date: Sun Oct  4 19:37:15 2020
New Revision: 366433
URL: https://svnweb.freebsd.org/changeset/base/366433

Log:
  Explicit CTLFLAG_DYN not needed
  
  Dynamically created OIDs automatically get this flag set.
  
  Reviewed by:	jhb
  MFC after:	1 week
  Sponsored by:	iXsystems, Inc.
  Differential Revision:	https://reviews.freebsd.org/D26561

Modified:
  head/sys/contrib/ipfilter/netinet/mlfk_ipl.c
  head/sys/x86/bios/vpd.c

Modified: head/sys/contrib/ipfilter/netinet/mlfk_ipl.c
==============================================================================
--- head/sys/contrib/ipfilter/netinet/mlfk_ipl.c	Sun Oct  4 17:23:39 2020	(r366432)
+++ head/sys/contrib/ipfilter/netinet/mlfk_ipl.c	Sun Oct  4 19:37:15 2020	(r366433)
@@ -88,19 +88,19 @@ SYSCTL_DECL(_net_inet);
         ptr, val, sysctl_ipf_int, "I", descr)
 #define SYSCTL_DYN_IPF_NAT(parent, nbr, name, access,ptr, val, descr) \
     SYSCTL_ADD_OID(&ipf_clist, SYSCTL_STATIC_CHILDREN(parent), nbr, name, \
-        CTLFLAG_DYN | CTLTYPE_INT | CTLFLAG_VNET | CTLFLAG_MPSAFE |access, \
+        CTLTYPE_INT | CTLFLAG_VNET | CTLFLAG_MPSAFE |access, \
         ptr, val, sysctl_ipf_int_nat, "I", descr)
 #define SYSCTL_DYN_IPF_STATE(parent, nbr, name, access,ptr, val, descr) \
     SYSCTL_ADD_OID(&ipf_clist, SYSCTL_STATIC_CHILDREN(parent), nbr, name, \
-        CTLFLAG_DYN | CTLTYPE_INT | CTLFLAG_VNET | CTLFLAG_MPSAFE | access, \
+        CTLTYPE_INT | CTLFLAG_VNET | CTLFLAG_MPSAFE | access, \
         ptr, val, sysctl_ipf_int_state, "I", descr)
 #define SYSCTL_DYN_IPF_FRAG(parent, nbr, name, access,ptr, val, descr) \
     SYSCTL_ADD_OID(&ipf_clist, SYSCTL_STATIC_CHILDREN(parent), nbr, name, \
-        CTLFLAG_DYN | CTLTYPE_INT | CTLFLAG_VNET | CTLFLAG_MPSAFE | access, \
+        CTLTYPE_INT | CTLFLAG_VNET | CTLFLAG_MPSAFE | access, \
 	ptr, val, sysctl_ipf_int_frag, "I", descr)
 #define SYSCTL_DYN_IPF_AUTH(parent, nbr, name, access,ptr, val, descr) \
     SYSCTL_ADD_OID(&ipf_clist, SYSCTL_STATIC_CHILDREN(parent), nbr, name, \
-        CTLFLAG_DYN | CTLTYPE_INT | CTLFLAG_VNET | CTLFLAG_MPSAFE | access, \
+        CTLTYPE_INT | CTLFLAG_VNET | CTLFLAG_MPSAFE | access, \
 	ptr, val, sysctl_ipf_int_auth, "I", descr)
 static struct sysctl_ctx_list ipf_clist;
 #define	CTLFLAG_OFF	0x00800000	/* IPFilter must be disabled */

Modified: head/sys/x86/bios/vpd.c
==============================================================================
--- head/sys/x86/bios/vpd.c	Sun Oct  4 17:23:39 2020	(r366432)
+++ head/sys/x86/bios/vpd.c	Sun Oct  4 19:37:15 2020	(r366433)
@@ -210,19 +210,19 @@ vpd_attach (device_t dev)
 	sysctl_ctx_init(&sc->ctx);
 	SYSCTL_ADD_STRING(&sc->ctx,
 		SYSCTL_STATIC_CHILDREN(_hw_vpd_machine_type), OID_AUTO,
-		unit, CTLFLAG_RD|CTLFLAG_DYN, sc->MachineType, 0, NULL);
+		unit, CTLFLAG_RD, sc->MachineType, 0, NULL);
 	SYSCTL_ADD_STRING(&sc->ctx,
 		SYSCTL_STATIC_CHILDREN(_hw_vpd_machine_model), OID_AUTO,
-		unit, CTLFLAG_RD|CTLFLAG_DYN, sc->MachineModel, 0, NULL);
+		unit, CTLFLAG_RD, sc->MachineModel, 0, NULL);
 	SYSCTL_ADD_STRING(&sc->ctx,
 		SYSCTL_STATIC_CHILDREN(_hw_vpd_build_id), OID_AUTO,
-		unit, CTLFLAG_RD|CTLFLAG_DYN, sc->BuildID, 0, NULL);
+		unit, CTLFLAG_RD, sc->BuildID, 0, NULL);
 	SYSCTL_ADD_STRING(&sc->ctx,
 		SYSCTL_STATIC_CHILDREN(_hw_vpd_serial_box), OID_AUTO,
-		unit, CTLFLAG_RD|CTLFLAG_DYN, sc->BoxSerial, 0, NULL);
+		unit, CTLFLAG_RD, sc->BoxSerial, 0, NULL);
 	SYSCTL_ADD_STRING(&sc->ctx,
 		SYSCTL_STATIC_CHILDREN(_hw_vpd_serial_planar), OID_AUTO,
-		unit, CTLFLAG_RD|CTLFLAG_DYN, sc->PlanarSerial, 0, NULL);
+		unit, CTLFLAG_RD, sc->PlanarSerial, 0, NULL);
 
 	device_printf(dev, "Machine Type: %.4s, Model: %.3s, Build ID: %.9s\n",
 		sc->MachineType, sc->MachineModel, sc->BuildID);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202010041937.094JbGZl055585>