Skip to content

Commit

Permalink
[9.18] rem: usr: Remove --with-tuning=small/large configuration option
Browse files Browse the repository at this point in the history
The configuration option --with-tuning has been removed as it is no longer required or desired.

Merge branch 'ondrej/remove-tuning-large-9.18' into 'bind-9.18'

See merge request isc-projects/bind9!9959
  • Loading branch information
oerdnj committed Jan 22, 2025
2 parents 008e520 + 1b9d949 commit 327b666
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 35 deletions.
14 changes: 5 additions & 9 deletions bin/named/server.c
Original file line number Diff line number Diff line change
Expand Up @@ -155,11 +155,7 @@
#define SIZE_AS_PERCENT ((size_t)-2)
#endif /* ifndef SIZE_AS_PERCENT */

#ifdef TUNE_LARGE
#define RESOLVER_NTASKS_PERCPU 32
#else
#define RESOLVER_NTASKS_PERCPU 8
#endif /* TUNE_LARGE */
#define RESOLVER_NTASKS 523

/* RFC7828 defines timeout as 16-bit value specified in units of 100
* milliseconds, so the maximum and minimum advertised and keepalive
Expand Down Expand Up @@ -4885,10 +4881,10 @@ configure_view(dns_view_t *view, dns_viewlist_t *viewlist, cfg_obj_t *config,
dns_view_setresquerystats(view, resquerystats);

ndisp = 4 * ISC_MIN(named_g_udpdisp, MAX_UDP_DISPATCH);
CHECK(dns_view_createresolver(
view, named_g_taskmgr, RESOLVER_NTASKS_PERCPU * named_g_cpus,
ndisp, named_g_netmgr, named_g_timermgr, resopts,
named_g_dispatchmgr, dispatch4, dispatch6));
CHECK(dns_view_createresolver(view, named_g_taskmgr, RESOLVER_NTASKS,
ndisp, named_g_netmgr, named_g_timermgr,
resopts, named_g_dispatchmgr, dispatch4,
dispatch6));

/*
* Set the ADB cache size to 1/8th of the max-cache-size or
Expand Down
16 changes: 0 additions & 16 deletions configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -1313,20 +1313,6 @@ AS_IF([test "$with_jemalloc" = "no"],

AM_CONDITIONAL([HAVE_JEMALLOC], [test "$with_jemalloc" = "yes"])

#
# was --with-tuning specified?
#
# [pairwise: --with-tuning=small, --without-tuning]
AC_ARG_WITH([tuning],
AS_HELP_STRING([--with-tuning=ARG],
[Specify server tuning (default or small)]),
[],[with_tuning=no])

AS_CASE([$with_tuning],
[small],[AC_MSG_NOTICE(using small system tuning)],
[AC_DEFINE(TUNE_LARGE, 1, [Define to use default system tuning.])
AC_MSG_NOTICE(using default system tuning)])

#
# was --enable-querytrace or --enable-singletrace specified?
#
Expand Down Expand Up @@ -1576,7 +1562,6 @@ report() {
if test "yes" = "$enable_full_report" -o "standard" = "$with_locktype"; then
echo " Mutex lock type: $with_locktype"
fi
test "small" = "$with_tuning" && echo " Small-system tuning (--with-tuning)"
test "no" = "$enable_dnstap" || \
echo " Allow 'dnstap' packet logging (--enable-dnstap)"
test -z "$MAXMINDDB_LIBS" || echo " GeoIP2 access control (--enable-geoip)"
Expand Down Expand Up @@ -1621,7 +1606,6 @@ report() {
echo " +------------------------------------------+"
echo
fi
test "small" = "$with_tuning" || echo " Small-system tuning (--with-tuning)"

test "no" = "$enable_dnstap" && \
echo " Allow 'dnstap' packet logging (--enable-dnstap)"
Expand Down
4 changes: 0 additions & 4 deletions lib/dns/client.c
Original file line number Diff line number Diff line change
Expand Up @@ -60,11 +60,7 @@
#define UCTX_MAGIC ISC_MAGIC('U', 'c', 't', 'x')
#define UCTX_VALID(c) ISC_MAGIC_VALID(c, UCTX_MAGIC)

#ifdef TUNE_LARGE
#define RESOLVER_NTASKS 523
#else /* ifdef TUNE_LARGE */
#define RESOLVER_NTASKS 31
#endif /* TUNE_LARGE */

#define CHECK(r) \
do { \
Expand Down
6 changes: 0 additions & 6 deletions lib/ns/interfacemgr.c
Original file line number Diff line number Diff line change
Expand Up @@ -54,12 +54,6 @@

#define LISTENING(ifp) (((ifp)->flags & NS_INTERFACEFLAG_LISTENING) != 0)

#ifdef TUNE_LARGE
#define UDPBUFFERS 32768
#else /* ifdef TUNE_LARGE */
#define UDPBUFFERS 1000
#endif /* TUNE_LARGE */

#define IFMGR_MAGIC ISC_MAGIC('I', 'F', 'M', 'G')
#define NS_INTERFACEMGR_VALID(t) ISC_MAGIC_VALID(t, IFMGR_MAGIC)

Expand Down

0 comments on commit 327b666

Please sign in to comment.