[Devel] Re: [PATCH 1/1] Memory usage limit notification addition to memcg

[Dan Malek]

On Jul 13, 2009, at 6:43 PM, KOSAKI Motohiro wrote:

> I like multiple threshold and per-thresold file-descriptor.
> it solve multiple waiters issue.
>
> but How about this?
>
>   /cgroup
>     /group1
>       /notifications
>          /threashold-A
>          /threashold-B

Why are you making this so complicated?
As a group, there is a limit and a notification
threshold.  Use the power of the cgroup hierarchy
if you want something with related limits and
different thresholds.  I don't understand the system
design  that would desire group cooperation
but yet different threshold notifications, except in
the case of upper/lower limits.  I'm not arguing
you can't do it, but the value in doing so.

The complexity of the event delivery will cause
extensive discussion as well.  You have great
ideas but scratch just a little below the surface
and there are complex problems to solve.  For
example, if you have a "below limit" notification
I can think of many challenges to solve.  Do you
constantly deliver the event?  Only once on crossing?
At some time interval?  How often? On a new attach
to the event?  Hysteresis? How do you control these?

The purpose of this memory notification was to improve
upon a previous attempt at such a feature.  It's a useful
feature that is today being used in some applications
to successfully manage the constrained resource.

If you look at my presentation from the last ELC, you
will see this patch is one small step of many to improve
resource management.  This event notification discussion
is important, but still just a tiny implementation detail in
a bigger resource management scheme.  We need to
make the small steps to make people aware of new features,
write applications that utilize these features, and to
perhaps discover something even better we aren't
even considering.

Thanks.

	-- Dan

_______________________________________________
Containers mailing list
Containers at lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers