On Wed, Jan 6, 2016 at 3:23 AM, Tejun Heo <t...@kernel.org> wrote: > Hello, > > On Wed, Jan 06, 2016 at 12:28:06AM +0530, Parav Pandit wrote: >> +5-4-1. RDMA Interface Files >> + >> + rdma.resource.verb.list >> + rdma.resource.verb.limit >> + rdma.resource.verb.usage >> + rdma.resource.verb.failcnt >> + rdma.resource.hw.list >> + rdma.resource.hw.limit >> + rdma.resource.hw.usage >> + rdma.resource.hw.failcnt > > Can you please read the rest of cgroup.txt and put the interface in > line with the common conventions followed by other controllers? >
Yes. I read through. I can see two changes to be made in V2 version of this patch. 1. rdma.resource.verb.usage and rdma.resource.verb.limit to change respectively to, 2. rdma.resource.verb.stat and rdma.resource.verb.max. 3. rdma.resource.verb.failcnt indicate failure events, which I think should go to events. I roll out new patch for events post this patch as additional feature and remove this feature in V2. rdma.resource.verb.list file is unique to rdma cgroup, so I believe this is fine. We will conclude whether to have rdma.resource.hw.<files> or not in other patches. I am in opinion to keep "resource" and "verb" or "hw" tags around to keep it verbose enough to know what are we trying to control. Is that ok? > Thanks. > > -- > tejun -- To unsubscribe from this list: send the line "unsubscribe linux-security-module" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
