summaryrefslogtreecommitdiff
path: root/src/lazyfree.c
diff options
context:
space:
mode:
Diffstat (limited to 'src/lazyfree.c')
-rw-r--r--src/lazyfree.c11
1 files changed, 8 insertions, 3 deletions
diff --git a/src/lazyfree.c b/src/lazyfree.c
index cfab6a31a..f1de0c898 100644
--- a/src/lazyfree.c
+++ b/src/lazyfree.c
@@ -64,9 +64,14 @@ int dbAsyncDelete(redisDb *db, robj *key) {
robj *val = dictGetVal(de);
size_t free_effort = lazyfreeGetFreeEffort(val);
- /* If releasing the object is too much work and the refcount
- * is 1, that means the object really needs to be freed,
- * let's put it into the lazy free list. */
+ /* If releasing the object is too much work, do it in the background
+ * by adding the object to the lazy free list.
+ * Note that if the object is shared, to reclaim it now it is not
+ * possible. This rarely happens, however sometimes the implementation
+ * of parts of the Redis core may call incrRefCount() to protect
+ * objects, and then call dbDelete(). In this case we'll fall
+ * through and reach the dictFreeUnlinkedEntry() call, that will be
+ * equivalent to just calling decrRefCount(). */
if (free_effort > LAZYFREE_THRESHOLD && val->refcount == 1) {
atomicIncr(lazyfree_objects,1);
bioCreateBackgroundJob(BIO_LAZY_FREE,val,NULL,NULL);