From 780424639afe3e9a91e85bf3f88142606833f310 Mon Sep 17 00:00:00 2001 From: Alasdair G Kergon Date: Fri, 27 May 2016 14:35:11 +0100 Subject: [PATCH] Revert "libdm: trace missing settings" This reverts commit 8fd886f735125de4b3e32fc0a188af0192a0d173. This was a deliberate omission because logging token-by-token metadata parsing greatly increases the amount of logging for hardly any benefit. In general, only LVM config file settings need to be logged, and in places where it's considered important to log particular elements of metadata that should be done using specific log_* lines. This area can be revisited. --- WHATS_NEW_DM | 1 - libdm/libdm-config.c | 15 +++------------ 2 files changed, 3 insertions(+), 13 deletions(-) diff --git a/WHATS_NEW_DM b/WHATS_NEW_DM index 165cb90ad..d4469970c 100644 --- a/WHATS_NEW_DM +++ b/WHATS_NEW_DM @@ -12,7 +12,6 @@ Version 1.02.124 - 30th April 2016 Version 1.02.123 - 23rd April 2016 ================================== - Trace more settings in very verbose mode for libdm-config. Do not strip LVM- when debug reporting not found uuid. Version 1.02.122 - 9th April 2016 diff --git a/libdm/libdm-config.c b/libdm/libdm-config.c index 86955c51d..dcb7c3f81 100644 --- a/libdm/libdm-config.c +++ b/libdm/libdm-config.c @@ -1154,11 +1154,8 @@ int dm_config_get_uint32(const struct dm_config_node *cn, const char *path, if (!n || !n->v || n->v->type != DM_CFG_INT) return 0; - if (result) { - log_very_verbose("Setting %s to %u", path, (uint32_t) n->v->v.i); + if (result) *result = n->v->v.i; - } - return 1; } @@ -1172,11 +1169,8 @@ int dm_config_get_uint64(const struct dm_config_node *cn, const char *path, if (!n || !n->v || n->v->type != DM_CFG_INT) return 0; - if (result) { - log_very_verbose("Setting %s to " FMTu64 , path, (uint64_t) n->v->v.i); + if (result) *result = (uint64_t) n->v->v.i; - } - return 1; } @@ -1190,11 +1184,8 @@ int dm_config_get_str(const struct dm_config_node *cn, const char *path, if (!n || !n->v || n->v->type != DM_CFG_STRING) return 0; - if (result) { - log_very_verbose("Setting %s to %s", path, n->v->v.str); + if (result) *result = n->v->v.str; - } - return 1; }