|
 |
61f723 |
From e0cb3e9ff5337cfc4ecaa6fa5efa189b7bc16246 Mon Sep 17 00:00:00 2001
|
|
|
61f723 |
From: Mark Reynolds <mreynolds@redhat.com>
|
|
|
61f723 |
Date: Mon, 15 May 2017 11:14:43 -0400
|
|
|
61f723 |
Subject: [PATCH 05/10] Ticket 49258 - Allow nsslapd-cache-autosize to be
|
|
|
61f723 |
modified while the server is running
|
|
|
61f723 |
|
|
|
61f723 |
Bug Description: Previously you're not allowed to set nsslapd-cache-autosize, and
|
|
|
61f723 |
nsslapd-cache-autosize-set while the server was running. The only
|
|
|
61f723 |
way to set it was to edit the dse.ldif
|
|
|
61f723 |
|
|
|
61f723 |
Fix Description: Allow it to be set while the server is running. Also added value
|
|
|
61f723 |
validation for these settigs
|
|
|
61f723 |
|
|
|
61f723 |
https://pagure.io/389-ds-base/issue/49258
|
|
|
61f723 |
|
|
|
61f723 |
Reviewed by: tbordaz(Thanks!)
|
|
|
61f723 |
|
|
|
61f723 |
(cherry picked from commit 2d07ca48f9c1232fc544361b5103d353e4791a72)
|
|
|
61f723 |
---
|
|
|
61f723 |
ldap/servers/slapd/back-ldbm/ldbm_config.c | 34 ++++++++++++++++++++++++------
|
|
|
61f723 |
1 file changed, 28 insertions(+), 6 deletions(-)
|
|
|
61f723 |
|
|
|
61f723 |
diff --git a/ldap/servers/slapd/back-ldbm/ldbm_config.c b/ldap/servers/slapd/back-ldbm/ldbm_config.c
|
|
|
61f723 |
index 401cd60..f7edd9e 100644
|
|
|
61f723 |
--- a/ldap/servers/slapd/back-ldbm/ldbm_config.c
|
|
|
61f723 |
+++ b/ldap/servers/slapd/back-ldbm/ldbm_config.c
|
|
|
61f723 |
@@ -1197,8 +1197,19 @@ static int ldbm_config_cache_autosize_set(void *arg, void *value, char *errorbuf
|
|
|
61f723 |
{
|
|
|
61f723 |
struct ldbminfo *li = (struct ldbminfo *)arg;
|
|
|
61f723 |
|
|
|
61f723 |
- if (apply)
|
|
|
61f723 |
- li->li_cache_autosize = (int)((uintptr_t)value);
|
|
|
61f723 |
+ if (apply) {
|
|
|
61f723 |
+ int val = (int)((uintptr_t)value);
|
|
|
61f723 |
+ if (val < 0 || val > 100) {
|
|
|
61f723 |
+ slapi_create_errormsg(errorbuf, SLAPI_DSE_RETURNTEXT_SIZE,
|
|
|
61f723 |
+ "Error: Invalid value for %s (%d). The value must be between \"0\" and \"100\"\n",
|
|
|
61f723 |
+ CONFIG_CACHE_AUTOSIZE, val);
|
|
|
61f723 |
+ slapi_log_err(SLAPI_LOG_ERR, "ldbm_config_cache_autosize_set",
|
|
|
61f723 |
+ "Invalid value for %s (%d). The value must be between \"0\" and \"100\"\n",
|
|
|
61f723 |
+ CONFIG_CACHE_AUTOSIZE, val);
|
|
|
61f723 |
+ return LDAP_UNWILLING_TO_PERFORM;
|
|
|
61f723 |
+ }
|
|
|
61f723 |
+ li->li_cache_autosize = val;
|
|
|
61f723 |
+ }
|
|
|
61f723 |
return LDAP_SUCCESS;
|
|
|
61f723 |
}
|
|
|
61f723 |
|
|
|
61f723 |
@@ -1214,8 +1225,19 @@ static int ldbm_config_cache_autosize_split_set(void *arg, void *value, char *er
|
|
|
61f723 |
{
|
|
|
61f723 |
struct ldbminfo *li = (struct ldbminfo *)arg;
|
|
|
61f723 |
|
|
|
61f723 |
- if (apply)
|
|
|
61f723 |
- li->li_cache_autosize_split = (int)((uintptr_t)value);
|
|
|
61f723 |
+ if (apply) {
|
|
|
61f723 |
+ int val = (int)((uintptr_t)value);
|
|
|
61f723 |
+ if (val < 0 || val > 100) {
|
|
|
61f723 |
+ slapi_create_errormsg(errorbuf, SLAPI_DSE_RETURNTEXT_SIZE,
|
|
|
61f723 |
+ "Error: Invalid value for %s (%d). The value must be between \"0\" and \"100\"\n",
|
|
|
61f723 |
+ CONFIG_CACHE_AUTOSIZE_SPLIT, val);
|
|
|
61f723 |
+ slapi_log_err(SLAPI_LOG_ERR, "ldbm_config_cache_autosize_split_set",
|
|
|
61f723 |
+ "Invalid value for %s (%d). The value must be between \"0\" and \"100\"\n",
|
|
|
61f723 |
+ CONFIG_CACHE_AUTOSIZE_SPLIT, val);
|
|
|
61f723 |
+ return LDAP_UNWILLING_TO_PERFORM;
|
|
|
61f723 |
+ }
|
|
|
61f723 |
+ li->li_cache_autosize_split = val;
|
|
|
61f723 |
+ }
|
|
|
61f723 |
return LDAP_SUCCESS;
|
|
|
61f723 |
}
|
|
|
61f723 |
|
|
|
61f723 |
@@ -1582,8 +1604,8 @@ static config_info ldbm_config[] = {
|
|
|
61f723 |
{CONFIG_DB_DEBUG_CHECKPOINTING, CONFIG_TYPE_ONOFF, "off", &ldbm_config_db_debug_checkpointing_get, &ldbm_config_db_debug_checkpointing_set, 0},
|
|
|
61f723 |
{CONFIG_DB_HOME_DIRECTORY, CONFIG_TYPE_STRING, "", &ldbm_config_db_home_directory_get, &ldbm_config_db_home_directory_set, 0},
|
|
|
61f723 |
{CONFIG_IMPORT_CACHE_AUTOSIZE, CONFIG_TYPE_INT, "-1", &ldbm_config_import_cache_autosize_get, &ldbm_config_import_cache_autosize_set, CONFIG_FLAG_ALWAYS_SHOW|CONFIG_FLAG_ALLOW_RUNNING_CHANGE},
|
|
|
61f723 |
- {CONFIG_CACHE_AUTOSIZE, CONFIG_TYPE_INT, "10", &ldbm_config_cache_autosize_get, &ldbm_config_cache_autosize_set, 0},
|
|
|
61f723 |
- {CONFIG_CACHE_AUTOSIZE_SPLIT, CONFIG_TYPE_INT, "40", &ldbm_config_cache_autosize_split_get, &ldbm_config_cache_autosize_split_set, 0},
|
|
|
61f723 |
+ {CONFIG_CACHE_AUTOSIZE, CONFIG_TYPE_INT, "10", &ldbm_config_cache_autosize_get, &ldbm_config_cache_autosize_set, CONFIG_FLAG_ALWAYS_SHOW|CONFIG_FLAG_ALLOW_RUNNING_CHANGE},
|
|
|
61f723 |
+ {CONFIG_CACHE_AUTOSIZE_SPLIT, CONFIG_TYPE_INT, "40", &ldbm_config_cache_autosize_split_get, &ldbm_config_cache_autosize_split_set, CONFIG_FLAG_ALWAYS_SHOW|CONFIG_FLAG_ALLOW_RUNNING_CHANGE},
|
|
|
61f723 |
{CONFIG_IMPORT_CACHESIZE, CONFIG_TYPE_SIZE_T, "16777216", &ldbm_config_import_cachesize_get, &ldbm_config_import_cachesize_set, CONFIG_FLAG_ALWAYS_SHOW|CONFIG_FLAG_ALLOW_RUNNING_CHANGE},
|
|
|
61f723 |
{CONFIG_IDL_SWITCH, CONFIG_TYPE_STRING, "new", &ldbm_config_idl_get_idl_new, &ldbm_config_idl_set_tune, CONFIG_FLAG_ALWAYS_SHOW},
|
|
|
61f723 |
{CONFIG_IDL_UPDATE, CONFIG_TYPE_ONOFF, "on", &ldbm_config_idl_get_update, &ldbm_config_idl_set_update, 0},
|
|
|
61f723 |
--
|
|
|
61f723 |
2.9.4
|
|
|
61f723 |
|