URL: https://github.com/SSSD/sssd/pull/906
Author: mzidek-rh
Title: #906: memcache: Stop using the word fastcache for memcache
Action: opened
PR body:
"""
This patch removes the word fastche from our codebase.
We do not reffer to the fast in-memory mmaped cache consistantly.
Sometimes we call it fastcache, sometimes memcache and sometimes
mmap cache.
While we can not get rid of memcache, because it is rooted
in some config option names and in the documentation and
mmap cache is used a lot internally in our code, so getting
rid of that would require a lot of code changes, we can
easyly get rid of the 'fastcache' word because it is
only in a handful of places.
I think we should really stick to calling it memcache in
the documentation and debug messages and ideally also in
code comments to avoid confusion.
"""
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/906/head:pr906
git checkout pr906
From a0fd5c6444bb83599128e1e5b23545af2ce8e211 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Michal=20=C5=BDidek?= <mzi...@redhat.com>
Date: Fri, 18 Oct 2019 12:37:33 +0200
Subject: [PATCH] memcache: Stop using the word fastcache for memcache
This patch removes the word fastche from our codebase.
We do not reffer to the fast in-memory mmaped cache consistantly.
Sometimes we call it fastcache, sometimes memcache and sometimes
mmap cache.
While we can not get rid of memcache, because it is rooted
in some config option names and in the documentation and
mmap cache is used a lot internally in our code, so getting
rid of that would require a lot of code changes, we can
easylly get rid of the 'fastcache' word because it is
only in a handful of places.
I think we should really stick to calling it memcache in
the documentation and debug messages and ideally also in
code comments to avoid confusion.
---
src/responder/nss/nsssrv_mmap_cache.c | 8 ++++----
src/tools/tools_mc_util.c | 6 +++---
2 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/src/responder/nss/nsssrv_mmap_cache.c b/src/responder/nss/nsssrv_mmap_cache.c
index 6efb24b34c..42c0f6e002 100644
--- a/src/responder/nss/nsssrv_mmap_cache.c
+++ b/src/responder/nss/nsssrv_mmap_cache.c
@@ -539,7 +539,7 @@ static struct sss_mc_rec *sss_mc_find_record(struct sss_mc_ctx *mcc,
while (slot != MC_INVALID_VAL) {
if (!MC_SLOT_WITHIN_BOUNDS(slot, mcc->dt_size)) {
DEBUG(SSSDBG_FATAL_FAILURE,
- "Corrupted fastcache. Slot number too big.\n");
+ "Corrupted memcache. Slot number too big.\n");
sss_mc_save_corrupted(mcc);
sss_mmap_cache_reset(mcc);
return NULL;
@@ -570,7 +570,7 @@ static struct sss_mc_rec *sss_mc_find_record(struct sss_mc_ctx *mcc,
|| strs_offset > max_addr - (uint8_t *)rec->data
|| strs_len > max_addr - (uint8_t *)rec->data - strs_offset) {
DEBUG(SSSDBG_FATAL_FAILURE,
- "Corrupted fastcache entry at slot %u. "
+ "Corrupted memcache entry at slot %u. "
"name_ptr value is %u.\n", slot, name_ptr);
sss_mc_save_corrupted(mcc);
sss_mmap_cache_reset(mcc);
@@ -813,7 +813,7 @@ errno_t sss_mmap_cache_pw_invalidate_uid(struct sss_mc_ctx *mcc, uid_t uid)
while (slot != MC_INVALID_VAL) {
if (!MC_SLOT_WITHIN_BOUNDS(slot, mcc->dt_size)) {
- DEBUG(SSSDBG_FATAL_FAILURE, "Corrupted fastcache.\n");
+ DEBUG(SSSDBG_FATAL_FAILURE, "Corrupted memcache.\n");
sss_mc_save_corrupted(mcc);
sss_mmap_cache_reset(mcc);
ret = ENOENT;
@@ -952,7 +952,7 @@ errno_t sss_mmap_cache_gr_invalidate_gid(struct sss_mc_ctx *mcc, gid_t gid)
while (slot != MC_INVALID_VAL) {
if (!MC_SLOT_WITHIN_BOUNDS(slot, mcc->dt_size)) {
- DEBUG(SSSDBG_FATAL_FAILURE, "Corrupted fastcache.\n");
+ DEBUG(SSSDBG_FATAL_FAILURE, "Corrupted memcache.\n");
sss_mc_save_corrupted(mcc);
sss_mmap_cache_reset(mcc);
ret = ENOENT;
diff --git a/src/tools/tools_mc_util.c b/src/tools/tools_mc_util.c
index 08503a5647..c65f07237d 100644
--- a/src/tools/tools_mc_util.c
+++ b/src/tools/tools_mc_util.c
@@ -129,7 +129,7 @@ errno_t sss_memcache_invalidate(const char *mc_filename)
return ret;
}
-static int clear_fastcache(bool *sssd_nss_is_off)
+static int clear_memcache(bool *sssd_nss_is_off)
{
int ret;
ret = sss_memcache_invalidate(SSS_NSS_MCACHE_DIR"/passwd");
@@ -199,13 +199,13 @@ errno_t sss_memcache_clear_all(void)
bool sssd_nss_is_off = false;
FILE *clear_mc_flag;
- ret = clear_fastcache(&sssd_nss_is_off);
+ ret = clear_memcache(&sssd_nss_is_off);
if (ret != EOK) {
DEBUG(SSSDBG_CRIT_FAILURE, "Failed to clear caches.\n");
return EIO;
}
if (!sssd_nss_is_off) {
- /* sssd_nss is running -> signal monitor to invalidate fastcache */
+ /* sssd_nss is running -> signal monitor to invalidate memcache */
clear_mc_flag = fopen(SSS_NSS_MCACHE_DIR"/"CLEAR_MC_FLAG, "w");
if (clear_mc_flag == NULL) {
DEBUG(SSSDBG_CRIT_FAILURE,
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct:
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives:
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
