From 32abe7841e4514e8a1c4925e18d7ac242ed00d98 Mon Sep 17 00:00:00 2001
From: Glauber Costa <glommer@cloudius-systems.com>
Date: Thu, 16 Jan 2014 20:00:17 +0400
Subject: [PATCH] balloon: remove leftover code
This code should not be here. I am 100 % positive that I removed it in my
testings, but I must have forgotten to git add the remove before I sent out the
patch and it ended up in tree.
This is simply a test leftover, it will have the effect of having threads to
loop forever and never waiting because the initial value won't be 0.
Signed-off-by: Glauber Costa <glommer@cloudius-systems.com>
Signed-off-by: Pekka Enberg <penberg@cloudius-systems.com>
---
core/mempool.cc | 6 ------
1 file changed, 6 deletions(-)
diff --git a/core/mempool.cc b/core/mempool.cc
index 9e5d47353..3b9faa73b 100644
--- a/core/mempool.cc
+++ b/core/mempool.cc
@@ -604,12 +604,6 @@ void reclaimer_waiters::post(unsigned units)
reclaimer::reclaimer()
: _oom_blocked(), _thread(NULL)
{
- // Set the semaphore the the current amount of free memory. We don't do it
- // in the constructor list so we can hold the lock and guarantee free
- // memory is not wildly changing.
- WITH_LOCK(free_page_ranges_lock) {
- _oom_blocked.post(stats::free());
- }
// This cannot be a sched::thread because it may call into JNI functions,
// if the JVM balloon is registered as a shrinker. It expects the full
// pthread API to be functional, and for sched::threads it is not.
--
GitLab