vm_blocked_clients count fixed in INFO, thanks to Pietern Noordhuis

This commit is contained in:
antirez 2010-07-21 13:16:26 +02:00
parent 2cffe2993b
commit 1a71fb9669

View File

@ -235,20 +235,25 @@ void freeClient(redisClient *c) {
ln = listSearchKey(server.clients,c); ln = listSearchKey(server.clients,c);
redisAssert(ln != NULL); redisAssert(ln != NULL);
listDelNode(server.clients,ln); listDelNode(server.clients,ln);
/* Remove from the list of clients that are now ready to be restarted /* Remove from the list of clients waiting for swapped keys, or ready
* after waiting for swapped keys */ * to be restarted, but not yet woken up again. */
if (c->flags & REDIS_IO_WAIT && listLength(c->io_keys) == 0) { if (c->flags & REDIS_IO_WAIT) {
redisAssert(server.vm_enabled);
if (listLength(c->io_keys) == 0) {
ln = listSearchKey(server.io_ready_clients,c); ln = listSearchKey(server.io_ready_clients,c);
if (ln) {
/* When this client is waiting to be woken up (REDIS_IO_WAIT),
* it should be present in the list io_ready_clients */
redisAssert(ln != NULL);
listDelNode(server.io_ready_clients,ln); listDelNode(server.io_ready_clients,ln);
server.vm_blocked_clients--; } else {
} while (listLength(c->io_keys)) {
}
/* Remove from the list of clients waiting for swapped keys */
while (server.vm_enabled && listLength(c->io_keys)) {
ln = listFirst(c->io_keys); ln = listFirst(c->io_keys);
dontWaitForSwappedKey(c,ln->value); dontWaitForSwappedKey(c,ln->value);
} }
}
server.vm_blocked_clients--;
}
listRelease(c->io_keys); listRelease(c->io_keys);
/* Master/slave cleanup */ /* Master/slave cleanup */
if (c->flags & REDIS_SLAVE) { if (c->flags & REDIS_SLAVE) {