RE: [PATCH 1/2] device: Stop requiring that struct device is embedded in struct pci_dev

2017-03-06 Thread Parav Pandit
Hi Greg, > -Original Message- > From: Greg Kroah-Hartman [mailto:gre...@linuxfoundation.org] > Sent: Monday, March 6, 2017 10:53 PM > To: Bart Van Assche <bart.vanass...@sandisk.com> > Cc: Doug Ledford <dledf...@redhat.com>; Sebastian Ott > <seb...@linu

RE: [PATCH 1/2] device: Stop requiring that struct device is embedded in struct pci_dev

2017-03-06 Thread Parav Pandit
Hi Greg, > -Original Message- > From: Greg Kroah-Hartman [mailto:gre...@linuxfoundation.org] > Sent: Monday, March 6, 2017 10:53 PM > To: Bart Van Assche > Cc: Doug Ledford ; Sebastian Ott > ; Parav Pandit ; linux- > r...@vger.kernel.org; linux-kernel@vger.kern

RE: [PATCH 1/2] device: Stop requiring that struct device is embedded in struct pci_dev

2017-03-06 Thread Parav Pandit
xfoundation.org>; Sebastian Ott > <seb...@linux.vnet.ibm.com>; Parav Pandit <pa...@mellanox.com>; linux- > r...@vger.kernel.org; linux-kernel@vger.kernel.org; Bart Van Assche > <bart.vanass...@sandisk.com>; Bjorn Helgaas <bhelg...@google.com>; > Benjam

RE: [PATCH 1/2] device: Stop requiring that struct device is embedded in struct pci_dev

2017-03-06 Thread Parav Pandit
Hi Bart, > -Original Message- > From: linux-rdma-ow...@vger.kernel.org [mailto:linux-rdma- > ow...@vger.kernel.org] On Behalf Of Bart Van Assche > Sent: Monday, March 6, 2017 6:36 PM > To: Doug Ledford > Cc: Greg Kroah-Hartman ; Sebastian Ott > ; Para

RE: [PATCH 1/2] device: Stop requiring that struct device is embedded in struct pci_dev

2017-03-06 Thread Parav Pandit
xfoundation.org>; Sebastian Ott > <seb...@linux.vnet.ibm.com>; Parav Pandit <pa...@mellanox.com>; linux- > r...@vger.kernel.org; linux-kernel@vger.kernel.org; Bart Van Assche > <bart.vanass...@sandisk.com>; Bjorn Helgaas <bhelg...@google.com>; > Benjam

RE: [PATCH 1/2] device: Stop requiring that struct device is embedded in struct pci_dev

2017-03-06 Thread Parav Pandit
Hi Bart, > -Original Message- > From: linux-rdma-ow...@vger.kernel.org [mailto:linux-rdma- > ow...@vger.kernel.org] On Behalf Of Bart Van Assche > Sent: Monday, March 6, 2017 6:36 PM > To: Doug Ledford > Cc: Greg Kroah-Hartman ; Sebastian Ott > ; Para

RE: IB on s390 broken with commit 99db94940 "IB/core: Remove ib_device.dma_device"

2017-02-28 Thread Parav Pandit
Hi Bart, I am using Linux-block tree testing on x86_64. git://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git Commit ac1820fb286b552b6885d40ab34f1e59b815f1f1 introduced dma_ops related change that you made. With this change I am hitting below error in mlx5_ib driver. "DMAR:

RE: IB on s390 broken with commit 99db94940 "IB/core: Remove ib_device.dma_device"

2017-02-28 Thread Parav Pandit
Hi Bart, I am using Linux-block tree testing on x86_64. git://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git Commit ac1820fb286b552b6885d40ab34f1e59b815f1f1 introduced dma_ops related change that you made. With this change I am hitting below error in mlx5_ib driver. "DMAR:

RE: [PATCH RESEND] nsproxy: Reuse already available put_nsproxy helper function.

2017-02-20 Thread Parav Pandit
Hi Eric, Will you be able to merge this patch if it looks o.k. Parav > -Original Message- > From: Parav Pandit [mailto:pa...@mellanox.com] > Sent: Friday, January 27, 2017 1:39 PM > To: ebied...@xmission.com; adityak...@google.com; t...@kernel.org; linux- > ker...@vger.

RE: [PATCH RESEND] nsproxy: Reuse already available put_nsproxy helper function.

2017-02-20 Thread Parav Pandit
Hi Eric, Will you be able to merge this patch if it looks o.k. Parav > -Original Message- > From: Parav Pandit [mailto:pa...@mellanox.com] > Sent: Friday, January 27, 2017 1:39 PM > To: ebied...@xmission.com; adityak...@google.com; t...@kernel.org; linux- > ker...@vger.

[PATCH RESEND] nsproxy: Reuse already available put_nsproxy helper function.

2017-01-27 Thread Parav Pandit
. Resending it to include Eric Biederman to merge the changes. Signed-off-by: Parav Pandit <pa...@mellanox.com> --- include/linux/nsproxy.h | 10 +- kernel/nsproxy.c| 12 +++- 2 files changed, 12 insertions(+), 10 deletions(-) diff --git a/include/linux/nsproxy.h b/i

[PATCH RESEND] nsproxy: Reuse already available put_nsproxy helper function.

2017-01-27 Thread Parav Pandit
. Resending it to include Eric Biederman to merge the changes. Signed-off-by: Parav Pandit --- include/linux/nsproxy.h | 10 +- kernel/nsproxy.c| 12 +++- 2 files changed, 12 insertions(+), 10 deletions(-) diff --git a/include/linux/nsproxy.h b/include/linux/nsproxy.h

[PATCH] nsproxy: Reuse already available put_nsproxy helper function.

2017-01-24 Thread Parav Pandit
. Signed-off-by: Parav Pandit <pa...@mellanox.com> --- include/linux/nsproxy.h | 10 +- kernel/nsproxy.c| 12 +++- 2 files changed, 12 insertions(+), 10 deletions(-) diff --git a/include/linux/nsproxy.h b/include/linux/nsproxy.h index ac0d65b..24556f4 100644 --- a/i

[PATCH] nsproxy: Reuse already available put_nsproxy helper function.

2017-01-24 Thread Parav Pandit
. Signed-off-by: Parav Pandit --- include/linux/nsproxy.h | 10 +- kernel/nsproxy.c| 12 +++- 2 files changed, 12 insertions(+), 10 deletions(-) diff --git a/include/linux/nsproxy.h b/include/linux/nsproxy.h index ac0d65b..24556f4 100644 --- a/include/linux/nsproxy.h +++ b

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/

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 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/pub/scm/linux/kernel/git/tj

[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 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 --- drivers/infiniband/core/Makefile | 1 + drivers/infiniband/core/cgroup.c | 62 + drivers

[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 --- Documentation/cgroup-v1/rdma.txt | 109 +++ Documentation/cgroup-v2.txt | 38 ++ 2 files

[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 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 --- include/linux/cgroup_rdma.h | 53 include/linux/cgroup_subsys.h | 4 + init/Kconfig | 10 + kernel/cgroup/Makefile| 1 + kernel/cgroup/rdma.c

[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

[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

2017-01-09 Thread Parav Pandit
Hi Tejun, On Mon, Jan 9, 2017 at 7:02 AM, Tejun Heo 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/kernel/git/dledford/rdma.

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

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

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

2016-12-02 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

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

2016-12-02 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 --- drivers/infiniband/core/Makefile | 1 + drivers/infiniband/core/cgroup.c | 62 ++ drivers

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

2016-12-02 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

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

2016-12-02 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 --- Documentation/cgroup-v1/rdma.txt | 109 +++ Documentation/cgroup-v2.txt | 38 ++ 2 files

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

2016-12-02 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 to use r

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

2016-12-02 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/Makefile | 1 +

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

2016-12-02 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 to use r

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

2016-12-02 Thread Parav Pandit
, instead of creating them during cgroup creation and device registration time. Signed-off-by: Parav Pandit --- include/linux/cgroup_rdma.h | 53 include/linux/cgroup_subsys.h | 4 + init/Kconfig | 10 + kernel/Makefile | 1 + kernel/cgroup_rdma.c

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 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 wrote: > Hi Doug, > > I am still waiting for Leon to provide his comments if an

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 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 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 >> git tree. >> >

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

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

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-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 wrote: > Hi Tejun, > > On Wed, Sep 21, 2016 at 7:56 PM, Teju

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-21 Thread Parav Pandit
Hi Tejun, On Wed, Sep 21, 2016 at 7:56 PM, Tejun Heo 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 drivers. >> Matan's

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-20 Thread Parav Pandit
Hi Leon, On Fri, Sep 16, 2016 at 12:26 AM, Leon Romanovsky 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 Matan, Leon, Jason, >> Apar

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

2016-09-19 Thread Parav Pandit
Hi Denny, On Mon, Sep 19, 2016 at 6:40 PM, Dalessandro, Dennis <dennis.dalessan...@intel.com> wrote: > On Wed, 2016-09-14 at 12:36 +0530, Parav Pandit wrote: >> Hi Dennis, >> >> Do you know how would HFI1 driver would work along with rdma cgroup? > > Keep i

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

2016-09-19 Thread Parav Pandit
Hi Denny, On Mon, Sep 19, 2016 at 6:40 PM, Dalessandro, Dennis wrote: > On Wed, 2016-09-14 at 12:36 +0530, Parav Pandit wrote: >> Hi Dennis, >> >> Do you know how would HFI1 driver would work along with rdma cgroup? > > Keep in mind HFI1 driver has two "modes&q

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

2016-09-14 Thread Parav Pandit
Hi Matan, On Wed, Sep 14, 2016 at 1:44 PM, Matan Barak <mat...@mellanox.com> wrote: > On 14/09/2016 10:06, Parav Pandit wrote: >> >> Hi Dennis, >> >> Do you know how would HFI1 driver would work along with rdma cgroup? >> >> Hi Matan, Leon, Jason, &

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

2016-09-14 Thread Parav Pandit
Hi Matan, On Wed, Sep 14, 2016 at 1:44 PM, Matan Barak wrote: > On 14/09/2016 10:06, Parav Pandit wrote: >> >> Hi Dennis, >> >> Do you know how would HFI1 driver would work along with rdma cgroup? >> >> Hi Matan, Leon, Jason, >> Apart from HFI1, is

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-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-08 Thread Parav Pandit
On Thu, Sep 8, 2016 at 11:42 AM, Leon Romanovsky <l...@kernel.org> wrote: > On Wed, Sep 07, 2016 at 08:37:23PM +0530, Parav Pandit wrote: >> Did you get a chance to review the series? > > We need to decide on fundamental question before reviewing it, which is > "how t

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

2016-09-08 Thread Parav Pandit
On Thu, Sep 8, 2016 at 11:42 AM, Leon Romanovsky wrote: > On Wed, Sep 07, 2016 at 08:37:23PM +0530, Parav Pandit wrote: >> Did you get a chance to review the series? > > We need to decide on fundamental question before reviewing it, which is > "how to fit rdmacg to new

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
Hi Leon, >> Signed-off-by: Parav Pandit >> +static struct rdmacg_resource_pool * >> +get_cg_rpool_locked(struct rdma_cgroup *cg, struct rdmacg_device *device) >> +{ >> + struct rdmacg_resource_pool *rpool; >> + >> + rpool = find_cg_rp

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
On Wed, Sep 7, 2016 at 2:21 PM, Matan Barak wrote: > On 07/09/2016 10:55, Parav Pandit wrote: >> >> 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: >>

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 >>

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 >> reason (to

[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

[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 --- drivers/infiniband/core/Makefile| 1 + drivers/infiniband/core/cgroup.c| 93

[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 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 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 --- Documentation/cgroup-v1/rdma.txt | 117 +++ Documentation/cgroup-v2.txt | 45 +++ 2 files

[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 --- include/linux/cgroup_rdma.h | 66 + include/linux/cgroup_subsys.h | 4 + init/Kconfig | 10 + kernel/Makefile | 1 + kernel

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

2016-08-25 Thread Parav Pandit
Hi Christoph, On Thu, Aug 25, 2016 at 1:02 PM, Christoph Hellwig wrote: > On Wed, Aug 24, 2016 at 05:17:47PM -0400, Tejun Heo wrote: >> Looks good to me. I just have a nit in the documentation. Christoph, >> what do you think? > > Looks reasonable from a quick look, but I didn't

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

2016-08-25 Thread Parav Pandit
Hi Christoph, On Thu, Aug 25, 2016 at 1:02 PM, Christoph Hellwig wrote: > On Wed, Aug 24, 2016 at 05:17:47PM -0400, Tejun Heo wrote: >> Looks good to me. I just have a nit in the documentation. Christoph, >> what do you think? > > Looks reasonable from a quick look, but I didn't do a full

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: [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: >> +mlx4_0 uctx=max

[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 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 --- Documentation/cgroup-v1/rdma.txt | 117 +++ Documentation/cgroup-v2.txt | 44 +++ 2 files

[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 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 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 --- include/linux/cgroup_rdma.h | 66 + include/linux/cgroup_subsys.h | 4 + init/Kconfig | 10 + kernel/Makefile | 1 + kernel

[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 --- drivers/infiniband/core/Makefile | 1 + drivers/infiniband/core/cgroup.c | 69 + drivers

[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

[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-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 wrote: > Hi Christoph, > > I was on travel. Sorry for the late inline response and

Re: CQ and RDMA READ/WRITE APIs

2016-05-18 Thread Parav Pandit
On Wed, May 18, 2016 at 7:37 PM, Doug Ledford <dledf...@redhat.com> wrote: > On 05/17/2016 10:48 PM, Parav Pandit wrote: >> Hi Doug, >> >> On Tue, May 17, 2016 at 11:02 PM, Doug Ledford <dledf...@redhat.com> wrote: >>> Nice catch there Bart. That was we

Re: CQ and RDMA READ/WRITE APIs

2016-05-18 Thread Parav Pandit
On Wed, May 18, 2016 at 7:37 PM, Doug Ledford wrote: > On 05/17/2016 10:48 PM, Parav Pandit wrote: >> Hi Doug, >> >> On Tue, May 17, 2016 at 11:02 PM, Doug Ledford wrote: >>> Nice catch there Bart. That was well before my role as maintainer and >>> so set

Re: CQ and RDMA READ/WRITE APIs

2016-05-17 Thread Parav Pandit
Hi Doug, On Tue, May 17, 2016 at 11:02 PM, Doug Ledford wrote: > Nice catch there Bart. That was well before my role as maintainer and > so settles things well enough for me. IOW, I don't feel I need to worry > about trying to maintain the dual license nature of the RDMA

Re: CQ and RDMA READ/WRITE APIs

2016-05-17 Thread Parav Pandit
Hi Doug, On Tue, May 17, 2016 at 11:02 PM, Doug Ledford wrote: > Nice catch there Bart. That was well before my role as maintainer and > so settles things well enough for me. IOW, I don't feel I need to worry > about trying to maintain the dual license nature of the RDMA stack as it > was

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

2016-04-19 Thread Parav Pandit
Hi Christoph, I was on travel. Sorry for the late inline response and question. Parav On Tue, Apr 5, 2016 at 10:57 PM, Christoph Hellwig <h...@infradead.org> wrote: > On Tue, Apr 05, 2016 at 05:55:26AM -0700, Parav Pandit wrote: >> Just because we add one more rdma resource,

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

2016-04-19 Thread Parav Pandit
Hi Christoph, I was on travel. Sorry for the late inline response and question. Parav On Tue, Apr 5, 2016 at 10:57 PM, Christoph Hellwig wrote: > On Tue, Apr 05, 2016 at 05:55:26AM -0700, Parav Pandit wrote: >> Just because we add one more rdma resource, we need to ask someone to &

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

2016-04-05 Thread Parav Pandit
On Tue, Apr 5, 2016 at 7:01 AM, Tejun Heo <t...@kernel.org> wrote: > Hello, Parav. > > On Mon, Apr 04, 2016 at 07:22:38PM -0700, Parav Pandit wrote: >> > Is it actually customary to have rdma core module updated more >> > frequently separate from the kernel? Out-o

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

2016-04-05 Thread Parav Pandit
On Tue, Apr 5, 2016 at 7:01 AM, Tejun Heo wrote: > Hello, Parav. > > On Mon, Apr 04, 2016 at 07:22:38PM -0700, Parav Pandit wrote: >> > Is it actually customary to have rdma core module updated more >> > frequently separate from the kernel? Out-of-tree modules be

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
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 going away, I can

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 5:42 AM, Christoph Hellwig 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 in-kernel module t

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-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 separately happens

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 6:25 PM, Tejun Heo 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. So it is certainl

<    1   2   3   4   5   6   7   >