Re: [PATCHv14 0/3] rdmacg: IB/core: rdma controller support

2017-01-10 Thread Parav Pandit
On Tue, Jan 10, 2017 at 10:15 AM, Tejun Heo <t...@kernel.org> wrote: > On Tue, Jan 10, 2017 at 12:02:12AM +0000, Parav Pandit wrote: >> Patchset is compiled and tested against below Tejun's cgroup tree >> using cgroup v1 and v2 mode. >> URL: git://git.kernel.org/

[PATCHv14 2/3] IB/core: added support to use rdma cgroup controller

2017-01-09 Thread Parav Pandit
functionality. Added support during query_device uverb operation to ensure it returns resource limits by honoring rdma cgroup configured limits. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- drivers/infiniband/core/Makefile | 1 + drivers/infiniband/core/cgroup.c

[PATCHv14 3/3] rdmacg: Added documentation for rdmacg

2017-01-09 Thread Parav Pandit
Added documentation for v1 and v2 version describing high level design and usage examples on using rdma controller. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- Documentation/cgroup-v1/rdma.txt | 109 +++ Documentation/cgroup-v2.txt

[PATCHv14 1/3] rdmacg: Added rdma cgroup controller

2017-01-09 Thread Parav Pandit
, instead of creating them during cgroup creation and device registration time. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- include/linux/cgroup_rdma.h | 53 include/linux/cgroup_subsys.h | 4 + init/Kconfig | 10 + kernel/cgroup/Makefile

[PATCHv14 0/3] rdmacg: IB/core: rdma controller support

2017-01-09 Thread Parav Pandit
drivers). * Created new rdma controller instead of piggyback on device cgroup. * Fixed race conditions for multiple tasks sharing rdma resources. * Removed dependency on the task_struct. Parav Pandit (3): rdmacg: Added rdma cgroup controller IB/core: added support to use rdma cgr

Re: [PATCHv13 0/3] rdmacg: IB/core: rdma controller support

2017-01-09 Thread Parav Pandit
Hi Tejun, On Mon, Jan 9, 2017 at 7:02 AM, Tejun Heo <t...@kernel.org> wrote: > On Fri, Dec 02, 2016 at 07:07:14PM +0000, Parav Pandit wrote: >> Patch is generated and tested against below Doug's linux-rdma >> git tree. >> URL: git://git.kernel.org/pub/scm/linux/k

Re: [PATCHv13 0/3] rdmacg: IB/core: rdma controller support

2016-12-05 Thread Parav Pandit
ok. Thanks Tejun. Parav On Tue, Dec 6, 2016 at 1:05 AM, Tejun Heo wrote: > Parav, it's a bit too late for this cycle. Let's target v4.11. I'll > review the patches after the merge window. Please ping me if I don't. > > Thanks. > > -- > tejun -- To unsubscribe from this list:

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

2016-10-18 Thread Parav Pandit
be simpler if that happens. Please provide the direction. [7] https://lkml.org/lkml/2016/10/5/134 [8] https://lkml.org/lkml/2016/10/5/30 Regards, Parav Pandit On Tue, Oct 4, 2016 at 11:49 PM, Parav Pandit <pandit.pa...@gmail.com> wrote: > Hi Doug, > > I am still waiting for Leon

Re: [PATCHv12 0/3] rdmacg: IB/core: rdma controller support

2016-10-06 Thread Parav Pandit
Hi Leon, On Wed, Oct 5, 2016 at 4:52 PM, Leon Romanovsky <l...@kernel.org> wrote: > On Wed, Aug 31, 2016 at 02:07:24PM +0530, Parav Pandit wrote: >> rdmacg: IB/core: rdma controller support >> >> Patch is generated and tested against below Doug's linux-rdma

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

2016-10-06 Thread Parav Pandit
Hi Christoph, On Wed, Oct 5, 2016 at 12:07 PM, Christoph Hellwig wrote: > FYI, the patches look fine to me: > > Acked-by: Christoph Hellwig > Thanks a lot for review. Parav -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a

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

2016-10-04 Thread Parav Pandit
[5] https://lkml.org/lkml/2016/9/19/571 [6] http://www.spinics.net/lists/linux-rdma/msg40337.html [7] email subject: [RFC ABI V4 0/7] SG-based RDMA ABI Proposal Regards, Parav Pandit On Wed, Sep 21, 2016 at 9:32 PM, Parav Pandit <pandit.pa...@gmail.com> wrote: > Hi Tejun, > > On Wed,

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

2016-09-21 Thread Parav Pandit
Hi Tejun, On Wed, Sep 21, 2016 at 7:56 PM, Tejun Heo <t...@kernel.org> wrote: > Hello, Parav. > > On Wed, Sep 21, 2016 at 10:13:38AM +0530, Parav Pandit wrote: >> We have completed review from Tejun, Christoph. >> HFI driver folks also provided feedback for Intel drive

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

2016-09-20 Thread Parav Pandit
Hi Leon, On Fri, Sep 16, 2016 at 12:26 AM, Leon Romanovsky <l...@kernel.org> wrote: > On Wed, Sep 14, 2016 at 12:36:19PM +0530, Parav Pandit wrote: >> Hi Dennis, >> >> Do you know how would HFI1 driver would work along with rdma cgroup? >> >> Hi

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

2016-09-14 Thread Parav Pandit
Hi Dennis, Do you know how would HFI1 driver would work along with rdma cgroup? Hi Matan, Leon, Jason, Apart from HFI1, is there any other concern? Or Patch is good to go? 4.8 dates are close by (2 weeks) and there are two git trees involved (that might cause merge error to Linus) so if there

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

2016-09-07 Thread Parav Pandit
Hi Leon, >> Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> >> +static struct rdmacg_resource_pool * >> +get_cg_rpool_locked(struct rdma_cgroup *cg, struct rdmacg_device *device) >> +{ >> + struct rdmacg_resource_pool *rpool; >> + >&g

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

2016-09-07 Thread Parav Pandit
On Wed, Sep 7, 2016 at 2:21 PM, Matan Barak <mat...@mellanox.com> wrote: > On 07/09/2016 10:55, Parav Pandit wrote: >> >> Hi Matan, >> >> On Thu, Sep 1, 2016 at 2:14 PM, Christoph Hellwig <h...@lst.de> wrote: >>> >>> On Thu, Sep 01, 2016 at

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

2016-09-07 Thread Parav Pandit
Hi Matan, On Thu, Sep 1, 2016 at 2:14 PM, Christoph Hellwig wrote: > On Thu, Sep 01, 2016 at 10:25:40AM +0300, Matan Barak wrote: >> Well, if I recall, the reason doing so last time was in order to allow >> flexible updating of ib_core independently, which is obviously not a good >>

[PATCHv12 3/3] rdmacg: Added documentation for rdmacg

2016-08-31 Thread Parav Pandit
Added documentation for v1 and v2 version describing high level design and usage examples on using rdma controller. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- Documentation/cgroup-v1/rdma.txt | 117 +++ Documentation/cgroup-v2.txt

[PATCHv12 1/3] rdmacg: Added rdma cgroup controller

2016-08-31 Thread Parav Pandit
necessary, instead of creating them during cgroup creation and device registration time. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- include/linux/cgroup_rdma.h | 66 + include/linux/cgroup_subsys.h | 4 + init/Kconfig | 10 + kernel/Ma

[PATCHv12 0/3] rdmacg: IB/core: rdma controller support

2016-08-31 Thread Parav Pandit
n device cgroup. * Fixed race conditions for multiple tasks sharing rdma resources. * Removed dependency on the task_struct. Parav Pandit (3): rdmacg: Added rdma cgroup controller IB/core: added support to use rdma cgroup controller rdmacg: Added documentation for rdmacg Documentati

[PATCHv12 2/3] IB/core: added support to use rdma cgroup controller

2016-08-31 Thread Parav Pandit
functionality. Added support during query_device uverb operation to ensure it returns resource limits by honoring rdma cgroup configured limits. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- drivers/infiniband/core/Makefile| 1 + drivers/infiniband/core/cgroup.c

Re: [PATCHv11 3/3] rdmacg: Added documentation for rdmacg

2016-08-25 Thread Parav Pandit
Hi Rami Rosen, On Thu, Aug 25, 2016 at 4:25 AM, Rami Rosen wrote: > Hi, > >> +Whenever RDMA resource charing occurs, owner rdma cgroup is returned to > Should be: charging instead of charing > >> +(b) Query resource limit: >> +cat /sys/fs/cgroup/rdma/2/rdma.max >> +#Output:

Re: [PATCH 0/5] Networking cgroup controller

2016-08-23 Thread Parav Pandit
Hi Anoop, Regardless of usecase, I think this functionality is best handled as LSM functionality instead of cgroup. Tasks which are proposed in this patch are related to access control checks. LSM already has required hooks for socket operations such as bind(), listen() as few small examples.

Re: [PATCH 0/5] Networking cgroup controller

2016-08-23 Thread Parav Pandit
Hi Anoop, On Thu, Aug 11, 2016 at 6:23 AM, Anoop Naravaram wrote: > This patchset introduces a cgroup controller for the networking subsystem as a > whole. As of now, this controller will be used for: > > * Limiting the specific ports that a process in a cgroup is allowed

[PATCHv11 2/3] IB/core: added support to use rdma cgroup controller

2016-08-22 Thread Parav Pandit
functionality. Added support during query_device uverb operation to ensure it returns resource limits by honoring rdma cgroup configured limits. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- drivers/infiniband/core/Makefile | 1 + drivers/infiniband/core/cgroup.c

[PATCHv11 3/3] rdmacg: Added documentation for rdmacg

2016-08-22 Thread Parav Pandit
Added documentation for v1 and v2 version describing high level design and usage examples on using rdma controller. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- Documentation/cgroup-v1/rdma.txt | 117 +++ Documentation/cgroup-v2.txt

[PATCHv11 1/3] rdmacg: Added rdma cgroup controller

2016-08-22 Thread Parav Pandit
necessary, instead of creating them during cgroup creation and device registration time. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- include/linux/cgroup_rdma.h | 66 + include/linux/cgroup_subsys.h | 4 + init/Kconfig | 10 + kernel/Ma

[PATCHv11 0/3] rdmacg: IB/core: rdma controller support

2016-08-22 Thread Parav Pandit
ing rdma resources. * Removed dependency on the task_struct. Parav Pandit (3): rdmacg: Added rdma cgroup controller IB/core: added support to use rdma cgroup controller rdmacg: Added documentation for rdmacg Documentation/cgroup-v1/rdma.txt | 117 ++ Documentation/cgroup-v2.tx

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

2016-06-03 Thread Parav Pandit
faster pace and avoids complexity around back-porting on various OS and their kernel flavors. Please let me know your views. Regards, Parav Pandit On Tue, Apr 19, 2016 at 2:26 PM, Parav Pandit <pandit.pa...@gmail.com> wrote: > Hi Christoph, > > I was on travel. Sorry for

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

2016-04-05 Thread Parav Pandit
On Tue, Apr 5, 2016 at 7:07 AM, Tejun Heo wrote: > Just one more thing. > > On Tue, Apr 05, 2016 at 10:01:07AM -0400, Tejun Heo wrote: > ... >> > pool_info in spin lock context, made me allocate memory to get all >> > values upfront through allocation. >> > Now that the lock is

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

2016-04-05 Thread Parav Pandit
Hi Christoph, On Tue, Apr 5, 2016 at 5:42 AM, Christoph Hellwig <h...@infradead.org> wrote: > On Tue, Apr 05, 2016 at 05:39:21AM -0700, Parav Pandit wrote: >> I am not really trying to address OFED issues here. I am sure you >> understand that if ib_core.ko kernel module is

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

2016-04-05 Thread Parav Pandit
Hi Christoph, On Tue, Apr 5, 2016 at 2:06 AM, Christoph Hellwig wrote: > On Mon, Apr 04, 2016 at 09:25:04PM -0400, Tejun Heo wrote: >> Is it actually customary to have rdma core module updated more >> frequently separate from the kernel? Out-of-tree modules being >> updated

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

2016-04-04 Thread Parav Pandit
Hi Tejun, On Mon, Apr 4, 2016 at 6:25 PM, Tejun Heo <t...@kernel.org> wrote: > Hello, > > On Mon, Apr 04, 2016 at 03:50:54PM -0700, Parav Pandit wrote: >> 2nd advantage is, it allows avoiding lot of rework, in bundling kernel >> modules with different kernel versions.

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

2016-04-04 Thread Parav Pandit
Hi Tejun, On Mon, Apr 4, 2016 at 12:36 PM, Tejun Heo wrote: > Hello, Parav. > > Sorry about the delay. > Thanks for the review. Comments and answers inline. >> +struct rdma_cgroup { >> + struct cgroup_subsys_state css; >> + >> + /* protects resource pool list */ >>

[PATCHv10 1/3] rdmacg: Added rdma cgroup controller

2016-03-24 Thread Parav Pandit
is done. Its a tradeoff of memory vs little more code space that creates resource pool whenever necessary, instead of creating them during cgroup creation and device registration time. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- include/linux/cgroup_rdma.h | 52 +++ include

[PATCHv10 0/3] rdmacg: IB/core: rdma controller support

2016-03-24 Thread Parav Pandit
instead of piggyback on device cgroup. * Fixed race conditions for multiple tasks sharing rdma resources. * Removed dependency on the task_struct. Parav Pandit (3): rdmacg: Added rdma cgroup controller IB/core: added support to use rdma cgroup controller rdmacg: Added documentation for rd

[PATCHv10 2/3] IB/core: added support to use rdma cgroup controller

2016-03-24 Thread Parav Pandit
uverbs layer to perform resource charge/uncharge functionality. Added support during query_device uverb operation to ensure it returns resource limits by honoring rdma cgroup configured limits. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- drivers/infiniband/core/Makefile

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

2016-03-19 Thread Parav Pandit
Hi Tejun, On Thu, Mar 17, 2016 at 2:10 AM, Tejun Heo wrote: > >> If this is ok. I will keep the code as it is, because it uses common >> helper functions for max and current files. > > Hmmm... can you please try to refactor the common part to helpers? > It's not a big thing but

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

2016-03-11 Thread Parav Pandit
Hi Tejun, On Sat, Mar 5, 2016 at 10:50 PM, Parav Pandit <pandit.pa...@gmail.com> wrote: > Hi Tejun, > > On Sat, Mar 5, 2016 at 6:22 PM, Tejun Heo <t...@kernel.org> wrote: >> Hello, Parav. >> >> On Sat, Mar 05, 2016 at 04:45:09PM +0530, Parav Pandit wrot

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

2016-03-05 Thread Parav Pandit
Hi Tejun, On Sat, Mar 5, 2016 at 6:22 PM, Tejun Heo <t...@kernel.org> wrote: > Hello, Parav. > > On Sat, Mar 05, 2016 at 04:45:09PM +0530, Parav Pandit wrote: >> Design that remains same from v6 to v10. >> * spin lock is still fine grained at cgroup level instead o

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

2016-03-05 Thread Parav Pandit
cases (as explained in documentation), and its not hot path. Parav On Thu, Mar 3, 2016 at 8:19 AM, Parav Pandit <pandit.pa...@gmail.com> wrote: > Hi Tejun, Haggai, > > On Thu, Mar 3, 2016 at 1:28 AM, Parav Pandit <pandit.pa...@gmail.com> wrote: >>>> + rpoo

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

2016-03-03 Thread Parav Pandit
On Thu, Mar 3, 2016 at 1:44 PM, Haggai Eran <hagg...@mellanox.com> wrote: > On 03/03/2016 05:18, Parav Pandit wrote: >> On Thu, Mar 3, 2016 at 1:28 AM, Parav Pandit <pandit.pa...@gmail.com> wrote: >>> On Wed, Mar 2, 2016 at 11:09 PM, Tejun Heo <t...@kernel.org>

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

2016-03-02 Thread Parav Pandit
On Thu, Mar 3, 2016 at 1:28 AM, Parav Pandit <pandit.pa...@gmail.com> wrote: > On Wed, Mar 2, 2016 at 11:09 PM, Tejun Heo <t...@kernel.org> wrote: >> Nothing seems to prevent @cg from going away if this races with >> @current being migrated to a different cgroup. Have y

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

2016-03-02 Thread Parav Pandit
On Wed, Mar 2, 2016 at 11:09 PM, Tejun Heo wrote: > Hello, > >> +struct rdma_cgroup { >> + struct cgroup_subsys_state css; >> + >> + spinlock_t rpool_list_lock; /* protects resource pool list */ >> + struct list_head rpool_head;/* head to keep track of

Re: [PATCHv9 0/3] rdmacg: IB/core: rdma controller support

2016-03-02 Thread Parav Pandit
Hi Tejun, On Wed, Mar 2, 2016 at 11:10 PM, Tejun Heo wrote: > Hello, Parav. > > It doesn't look like my reviews are getting through. For now, > I have addressed all the review comments that you provided in v5 patch. I admit that few comments have not followed CodingStyle and I

[PATCHv9 1/3] rdmacg: Added rdma cgroup controller

2016-03-01 Thread Parav Pandit
is done. Its a tradeoff of memory vs little more code space that creates resource pool whenever necessary, instead of creating them during cgroup creation and device registration time. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> Reviewed-by: Haggai Eran <hagg...@mellanox.com>

[PATCHv9 2/3] IB/core: added support to use rdma cgroup controller

2016-03-01 Thread Parav Pandit
uverbs layer to perform resource charge/uncharge functionality. Added support during query_device uverb operation to ensure it returns resource limits by honoring rdma cgroup configured limits. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- drivers/infiniband/core/Makefile

[PATCHv9 0/3] rdmacg: IB/core: rdma controller support

2016-03-01 Thread Parav Pandit
etting (optional to drivers). * Created new rdma controller instead of piggyback on device cgroup. * Fixed race conditions for multiple tasks sharing rdma resources. * Removed dependency on the task_struct. Parav Pandit (3): rdmacg: Added rdma cgroup controller IB/core: added support

[PATCHv9 3/3] rdmacg: Added documentation for rdmacg

2016-03-01 Thread Parav Pandit
Added documentation for v1 and v2 version describing high level design and usage examples on using rdma controller. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> Reviewed-by: Haggai Eran <hagg...@mellanox.com> --- Documentation/cgroup-v1/rd

Re: [PATCHv8 2/3] IB/core: added support to use rdma cgroup controller

2016-03-01 Thread Parav Pandit
-rc6 next-20160301] > [if your patch is applied to the wrong git tree, please drop us a note to > help improving the system] > > url: > https://github.com/0day-ci/linux/commits/Parav-Pandit/rdmacg-IB-core-rdma-controller-support/20160301-223149 > base: https://git.kernel.org/pu

[PATCHv8 0/3] rdmacg: IB/core: rdma controller support

2016-03-01 Thread Parav Pandit
roup. * Fixed race conditions for multiple tasks sharing rdma resources. * Removed dependency on the task_struct. Parav Pandit (3): rdmacg: Added rdma cgroup controller IB/core: added support to use rdma cgroup controller rdmacg: Added documentation for rdmacg Documentation

[PATCHv8 2/3] IB/core: added support to use rdma cgroup controller

2016-03-01 Thread Parav Pandit
uverbs layer to perform resource charge/uncharge functionality. Added support during query_device uverb operation to ensure it returns resource limits by honoring rdma cgroup configured limits. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- drivers/infiniband/core/Makefile

[PATCHv8 1/3] rdmacg: Added rdma cgroup controller

2016-03-01 Thread Parav Pandit
is done. Its a tradeoff of memory vs little more code space that creates resource pool whenever necessary, instead of creating them during cgroup creation and device registration time. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> Reviewed-by: Haggai Eran <hagg...@mellanox.com>

[PATCHv8 3/3] rdmacg: Added documentation for rdmacg

2016-03-01 Thread Parav Pandit
Added documentation for v1 and v2 version describing high level design and usage examples on using rdma controller. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> Reviewed-by: Haggai Eran <hagg...@mellanox.com> --- Documentation/cgroup-v1/rd

Re: [PATCHv7 2/3] IB/core: added support to use rdma cgroup controller

2016-03-01 Thread Parav Pandit
On Tue, Mar 1, 2016 at 4:36 PM, Haggai Eran <hagg...@mellanox.com> wrote: > On 01/03/2016 11:22, Parav Pandit wrote: >> On Tue, Mar 1, 2016 at 2:42 PM, Haggai Eran <hagg...@mellanox.com> wrote: >>> On 28/02/2016 16:13, Parav Pandit wrote: >>>> diff --git

Re: [PATCHv7 2/3] IB/core: added support to use rdma cgroup controller

2016-03-01 Thread Parav Pandit
On Tue, Mar 1, 2016 at 2:38 PM, Haggai Eran <hagg...@mellanox.com> wrote: > On 28/02/2016 16:13, Parav Pandit wrote: >> +void ib_rdmacg_query_limit(struct ib_device *device, int *limits, int >> max_count) >> +{ >> + rdmacg_query_limit(>cg_devi

[PATCHv7 3/3] rdmacg: Added documentation for rdmacg

2016-02-28 Thread Parav Pandit
Added documentation for v1 and v2 version describing high level design and usage examples on using rdma controller. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- Documentation/cgroup-v1/rdma.txt | 111 +++ Documentation/cgroup-v2.txt

[PATCHv7 2/3] IB/core: added support to use rdma cgroup controller

2016-02-28 Thread Parav Pandit
uverbs layer to perform resource charge/uncharge functionality. Added support during query_device uverb operation to ensure it returns resource limits by honoring rdma cgroup configured limits. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- drivers/infiniband/core/Makefile

[PATCHv7 1/3] rdmacg: Added rdma cgroup controller

2016-02-28 Thread Parav Pandit
is done. Its a tradeoff of memory vs little more code space that creates resource pool whenever necessary, instead of creating them during cgroup creation and device registration time. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- include/linux/cgroup_rdma.h | 51 +++ include

[PATCHv7 0/3] rdmacg: IB/core: rdma controller support

2016-02-28 Thread Parav Pandit
* Created new rdma controller instead of piggyback on device cgroup. * Fixed race conditions for multiple tasks sharing rdma resources. * Removed dependency on the task_struct. Parav Pandit (3): rdmacg: Added rdma cgroup controller IB/core: added support to use rdma cgroup controller rdmacg:

Re: [PATCHv6 3/3] rdmacg: Added documentation for rdmacg

2016-02-28 Thread Parav Pandit
o.k. I will add a note there that IB stack would honor the limits given by the rdma cgroup. On Sun, Feb 28, 2016 at 2:25 PM, Haggai Eran <hagg...@mellanox.com> wrote: > On 24/02/2016 17:21, Parav Pandit wrote: >> On Wed, Feb 24, 2016 at 7:56 PM, Haggai Eran <hagg...@mellanox.com

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

2016-02-25 Thread Parav Pandit
> Can we call kfree() with spin_lock held? All these years I tend to > avoid doing so. Also it doesn't look correct to hold the lock while freeing the memory which is totally unrelated to the lock. With that I think current code appears ok with exception that its duplicated at two place for code

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

2016-02-25 Thread Parav Pandit
On Thu, Feb 25, 2016 at 5:33 PM, Haggai Eran wrote: +retry: + spin_lock(>rpool_list_lock); + rpool = find_cg_rpool_locked(cg, device); + if (!rpool) { + spin_unlock(>rpool_list_lock); + ret =

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

2016-02-21 Thread Parav Pandit
On Sun, Feb 21, 2016 at 8:39 PM, Leon Romanovsky <l...@leon.nu> wrote: > On Sun, Feb 21, 2016 at 07:41:08PM +0530, Parav Pandit wrote: >> CONFIG_CGROUP_RDMA >> >> On Sun, Feb 21, 2016 at 7:15 PM, Leon Romanovsky <l...@leon.nu> wrote: >> > On Sun, Feb

[PATCHv6 2/3] IB/core: added support to use rdma cgroup controller

2016-02-20 Thread Parav Pandit
uverbs layer to perform resource charge/uncharge functionality. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- drivers/infiniband/core/Makefile | 1 + drivers/infiniband/core/cgroup.c | 114 +++ drivers/infiniband/core/core_priv.h | 36 d

[PATCHv6 1/3] rdmacg: Added rdma cgroup controller

2016-02-20 Thread Parav Pandit
is done. Its a tradeoff of memory vs little more code space that creates resource pool whenever necessary, instead of creating them during cgroup creation and device registration time. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- include/linux/cgroup_rdma.h | 53 +++ include

[PATCHv6 0/3] rdmacg: IB/core: rdma controller support

2016-02-20 Thread Parav Pandit
ails to merge with Doug's tree as it has different code. It requires manual merge. I will generate separate patch for Doug's tree for linux-rdma for just patch 2/3 to avoid merge conflict once we complete overall review. Parav Pandit (3): rdmacg: Added rdma cgroup controller IB/core: added support

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

2016-01-31 Thread Parav Pandit
On Sun, Jan 31, 2016 at 3:32 PM, Tejun Heo <t...@kernel.org> wrote: > Hello, Parav. > > On Sun, Jan 31, 2016 at 02:14:13AM +0530, Parav Pandit wrote: > ... >> V1 patch had IB resources defined in the header file of rdmacg, which >> I believe is very restrictive

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

2016-01-30 Thread Parav Pandit
Hi Tejun, On Sun, Jan 31, 2016 at 12:00 AM, Tejun Heo <t...@kernel.org> wrote: > Hello, > > On Sat, Jan 30, 2016 at 08:53:05PM +0530, Parav Pandit wrote: >> +#ifdef CONFIG_CGROUP_RDMA >> +#define RDMACG_MAX_RESOURCE_INDEX (64) > > The list of resource

[PATCHv2 1/3] rdmacg: Added rdma cgroup controller.

2016-01-30 Thread Parav Pandit
changing kernel, which is frequent as IB stack is going through changes and enhancements. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- include/linux/cgroup_rdma.h | 78 include/linux/cgroup_subsys.h |4 + init/Kconfig | 12 + kernel/Ma

[PATCHv2 2/3] IB/core: added support to use rdma cgroup controller

2016-01-30 Thread Parav Pandit
uverbs layer to perform resource charge/uncharge functionality. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- drivers/infiniband/core/Makefile | 1 + drivers/infiniband/core/cgroup.c | 108 ++ drivers/infiniband/core/core_priv.h | 45 d

[PATCHv2 3/3] rdmacg: Added documentation for rdma controller

2016-01-30 Thread Parav Pandit
Added documentation for rdma controller to use in legacy mode and using new unified hirerchy. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- Documentation/cgroup-v1/rdma.txt | 122 +++ Documentation/cgroup-v2.txt | 43 ++ 2

[PATCHv2 0/3] rdma controller support

2016-01-30 Thread Parav Pandit
/476 [3] https://lkml.org/lkml/2015/10/28/144 This patchset is for Tejun's for-4.5 branch. It is not attempted on Doug's rdma tree yet, which I will do once I receive comments for this pathset. Parav Pandit (3): rdmacg: Added rdma cgroup controller. IB/core: added support to use rdma cg

[PATCHv3 1/3] rdmacg: Added rdma cgroup controller.

2016-01-30 Thread Parav Pandit
pool whenever necessary, instead of creating them during cgroup creation and device registration time. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- include/linux/cgroup_rdma.h | 80 include/linux/cgroup_subsys.h |4 + init/Kconfig | 12 +

[PATCHv3 2/3] IB/core: added support to use rdma cgroup controller

2016-01-30 Thread Parav Pandit
. - Added uverbs layer to perform resource charge/uncharge functionality. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- drivers/infiniband/core/Makefile | 1 + drivers/infiniband/core/cgroup.c | 108 ++ drivers/infiniband/core/core_priv.h

[PATCHv3 0/3] rdmacg: IB/core: rdma controller support

2016-01-30 Thread Parav Pandit
ttps://lkml.org/lkml/2015/9/7/476 [3] https://lkml.org/lkml/2015/10/28/144 This patchset is for Tejun's for-4.5 branch. It is not attempted on Doug's rdma tree yet, which I will do once I receive comments for this pathset. Parav Pandit (3): rdmacg: Added rdma cgroup controller. IB/core: added sup

Re: [PATCHv2 2/3] IB/core: added support to use rdma cgroup controller

2016-01-30 Thread Parav Pandit
your patch is applied to the wrong git tree, please drop us a note to > help improving the system] > > url: > https://github.com/0day-ci/linux/commits/Parav-Pandit/rdma-controller-support/20160130-194534 > base: https://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup.git fo

[PATCHv3 3/3] rdmacg: Added documentation for rdma controller

2016-01-30 Thread Parav Pandit
Added documentation for rdma controller to use in v1 mode and using new unified hirerchy mode v2. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- Documentation/cgroup-v1/rdma.txt | 122 +++ Documentation/cgroup-v2.txt | 43 +++

[PATCHv4 2/3] IB/core: added support to use rdma cgroup controller

2016-01-30 Thread Parav Pandit
uverbs layer to perform resource charge/uncharge functionality. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- drivers/infiniband/core/Makefile | 1 + drivers/infiniband/core/cgroup.c | 137 ++ drivers/infiniband/core/core_priv.h | 45 d

[PATCHv4 0/3] rdmacg: IB/core: rdma controller support

2016-01-30 Thread Parav Pandit
.org/lkml/2015/10/28/144 This patchset is for Tejun's for-4.5 branch. It is not attempted on Doug's rdma tree yet, which I will do once I receive comments for this pathset. Parav Pandit (3): rdmacg: Added rdma cgroup controller. IB/core: added support to use rdma cgroup controller rdmacg: A

[PATCHv5 1/3] rdmacg: Added rdma cgroup controller.

2016-01-30 Thread Parav Pandit
pool whenever necessary, instead of creating them during cgroup creation and device registration time. Signed-off-by: Parav Pandit <pandit.pa...@gmail.com> --- include/linux/cgroup_rdma.h | 78 include/linux/cgroup_subsys.h | 4 + init/Kconfig | 11 + kernel/Ma