Re: [PATCHv13 1/3] rdmacg: Added rdma cgroup controller | cgroups | Ratatoskr

[PATCHv13 0/3] rdmacg: IB/core: rdma controller support · Parav Pandit <hidden> · 2016-12-02
[PATCHv13 1/3] rdmacg: Added rdma cgroup controller · Parav Pandit <hidden> · 2016-12-02
Re: [PATCHv13 1/3] rdmacg: Added rdma cgroup controller · Tejun Heo <tj@kernel.org> · 2017-01-09
[PATCHv13 3/3] rdmacg: Added documentation for rdmacg · Parav Pandit <hidden> · 2016-12-02
[PATCHv13 2/3] IB/core: added support to use rdma cgroup controller · Parav Pandit <hidden> · 2016-12-02
Re: [PATCHv13 2/3] IB/core: added support to use rdma cgroup controller · Tejun Heo <tj@kernel.org> · 2017-01-09
Re: [PATCHv13 0/3] rdmacg: IB/core: rdma controller support · Tejun Heo <tj@kernel.org> · 2016-12-05
Re: [PATCHv13 0/3] rdmacg: IB/core: rdma controller support · Parav Pandit <hidden> · 2016-12-06
Re: [PATCHv13 0/3] rdmacg: IB/core: rdma controller support · Tejun Heo <tj@kernel.org> · 2017-01-09
Re: [PATCHv13 0/3] rdmacg: IB/core: rdma controller support · Parav Pandit <hidden> · 2017-01-09
Re: [PATCHv13 0/3] rdmacg: IB/core: rdma controller support · Tejun Heo <tj@kernel.org> · 2017-01-09

Re: [PATCHv13 1/3] rdmacg: Added rdma cgroup controller

From: Tejun Heo <tj@kernel.org>
Date: 2017-01-09 12:59:23
Also in: linux-rdma, lkml

Hello, Parav.

Looks good to me.  Just one minor nit below.

+static void rdmacg_uncharge_hirerchy(struct rdma_cgroup *cg,

 hierarchy

Other than that, looks good to me.

Thanks.

-- 
tejun