lkml.org 
[lkml]   [2014]   [Jan]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] cpusets: Allocate heap only when required
On Fri, 24 Jan 2014, Tejun Heo wrote:

> > Nobody is depending on shit, the patch is removing a completely pointless
> > memory allocation in braindead cpuset code. What you think is "harmful"
> > or "more harmful" is irrelevant, but nobody said anything about depending
> > on that behavior to do anything.
>
> Weren't you talking something of that effect in memcg? Or was it
> Michal?

In a completely different thread, I was talking about how we'd like to
provide a small amount of memory in oom conditions so that you could do
things like read the cgroup tasks file, but you'd also need the same thing
to do just about anything, ls, ps, read /proc/pid/status, etc with true
slab accounting. Forget about this unnecessary heap allocation, you
couldn't even do the open() in an oom condition.

That functionality would be provided by the memory reserves set aside for
userspace oom handlers as part of that feature, cgroups wouldn't be
different than anything else in that regard, it's a memcg and page
allocator issue only.

> At any rate, I think you're missing the point why Li replied
> that it's harmless. He, I think, meant that it doesn't make any
> semantical difference to userland, so your reply saying that it's not
> harmless listing the failure mode under memory pressure seemed
> misleading, so I thought clarification was necessary.

I would consider any memory allocation that is completely unnecessary to
cause anything to fail unnecessarily to be harmful, nothing specific here
about update_flag(), cpusets, or cgroups. Saying something is "harmless"
makes it sound like there's no downside to doing it, and that's obviously
not the case.

We agree and I think the only outcome of this discussion is that we both
wasted time :)


\
 
 \ /
  Last update: 2014-01-24 12:41    [W:0.037 / U:0.160 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site