URL:
https://github.com/SSSD/sssd/pull/999
Title: #999: Mem-cache fixes/improvements
sumit-bose commented:
"""
Hi,
thanks for the latest version. I agree with the changes. But while testing I realized that
I see the mem cache full log message not the at the first time the cache was full. I think
this is caused by an older issue where `mcc->next-slot` is not updated while the cache
is filled for the first time. I'd like to suggest to include a patch similar to the
following in your patch set:
```
diff --git a/src/responder/nss/nsssrv_mmap_cache.c
b/src/responder/nss/nsssrv_mmap_cache.c
index 18f0c6c..d639cd0 100644
--- a/src/responder/nss/nsssrv_mmap_cache.c
+++ b/src/responder/nss/nsssrv_mmap_cache.c
@@ -441,6 +441,7 @@ static errno_t sss_mc_find_free_slots(struct sss_mc_ctx *mcc,
}
if (cur == t) {
/* ok found num_slots consecutive free bits */
+ mcc->next_slot = cur;
*free_slot = cur - num_slots;
return EOK;
}
```
bye,
Sumit
"""
See the full comment at
https://github.com/SSSD/sssd/pull/999#issuecomment-603881327