On Thu, Aug 15, 2013 at 12:02:18PM -0400, Tejun Heo wrote: > Hello, > > Changes from the last take[L] are > > * Event handling updated such that it doesn't require meddling with > internals not normally exposed outside cgroup core proper. dentry > reference counting is replaced with css one and cftype handling is > completely gone. Hopefully, this addresses Michal's complaints. > > * Further simplifications. > > * Rebased on top of the current cgroup/for-3.12 and pending patches. > > Like many other things in cgroup, cgroup_event is way too flexible and > complex - it strives to provide completely flexible event monitoring > facility in cgroup proper which allows any number of users to monitor > custom events. This essentially is a layering violation and leads to > weird issues like worrying about event API mis/abuse from userland in > cgroup controller event source implementation. Blame me. When I added the interface I hoped events will be much more useful. Like for delivering 'last task in the cgroup leaves' signals. But you are right it's too easy to abuse the interface. Few questions before I'll ack the patchset: What's the plan if we will need a similar capability in the future? How would you implement memory thresholds today? -- Kirill A. Shutemov -- To unsubscribe from this list: send the line "unsubscribe cgroups" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
