Messages in this thread Patch in this message | | | Date | Thu, 29 Mar 2012 15:31:34 +0900 | From | KAMEZAWA Hiroyuki <> | Subject | [BUGFIX][PATCH 3/3] memcg/tcp: ignore tcp usage before accounting started. |
| |
tcp memcontrol starts accouting after res->limit is set. So, if a sockets starts before setting res->limit, there are already used resource. After setting res->limit, the resource will be uncharged and make res_counter below 0. This causes warning.
This patch fixes that by adding res_counter_uncharge_nowarn() and ignore the usage.
Signed-off-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com> --- include/linux/res_counter.h | 2 ++ include/net/sock.h | 3 ++- kernel/res_counter.c | 18 ++++++++++++++++++ 3 files changed, 22 insertions(+), 1 deletions(-)
diff --git a/include/linux/res_counter.h b/include/linux/res_counter.h index da81af0..e081948 100644 --- a/include/linux/res_counter.h +++ b/include/linux/res_counter.h @@ -134,6 +134,8 @@ int __must_check res_counter_charge_nofail(struct res_counter *counter, void res_counter_uncharge_locked(struct res_counter *counter, unsigned long val); void res_counter_uncharge(struct res_counter *counter, unsigned long val); +void res_counter_uncharge_nowarn(struct res_counter *counter, + unsigned long val); /** * res_counter_margin - calculate chargeable space of a counter diff --git a/include/net/sock.h b/include/net/sock.h index a6ba1f8..a1b3f4802 100644 --- a/include/net/sock.h +++ b/include/net/sock.h @@ -1048,7 +1048,8 @@ static inline void memcg_memory_allocated_add(struct cg_proto *prot, static inline void memcg_memory_allocated_sub(struct cg_proto *prot, unsigned long amt) { - res_counter_uncharge(prot->memory_allocated, amt << PAGE_SHIFT); + res_counter_uncharge_nowarn(prot->memory_allocated, + amt << PAGE_SHIFT); } static inline u64 memcg_memory_allocated_read(struct cg_proto *prot) diff --git a/kernel/res_counter.c b/kernel/res_counter.c index d508363..2bb01ac 100644 --- a/kernel/res_counter.c +++ b/kernel/res_counter.c @@ -113,6 +113,24 @@ void res_counter_uncharge(struct res_counter *counter, unsigned long val) local_irq_restore(flags); } +void res_counter_uncharge_nowarn(struct res_counter *counter, + unsigned long val) +{ + struct res_counter *c; + unsigned long flags; + + local_irq_save(flags); + + for (c = counter; c != NULL; c = c->parent) { + spin_lock(&c->lock); + if (c->usage < val) + val = c->usage; + res_counter_uncharge_locked(c, val); + spin_unlock(&c->lock); + } + local_irq_restore(flags); +} + static inline unsigned long long * res_counter_member(struct res_counter *counter, int member) -- 1.7.4.1
| |