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

From	Parav Pandit <pandit.parav@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [PATCHv11 0/3] rdmacg: IB/core: rdma controller support
Date	2016-08-25 10:10 +0200
Message-ID	<s9XTz-6QF-1@gated-at.bofh.it> (permalink)
References	<s8WGd-6h8-9@gated-at.bofh.it> <s9NKx-7XX-5@gated-at.bofh.it> <s9Xqx-6r7-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi Christoph,

On Thu, Aug 25, 2016 at 1:02 PM, Christoph Hellwig <hch@lst.de> 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 review yet.

Thanks for the quick review.
I need to generate new patch for 4 typo errors that Rami pointed out
and documentation addition that Tejun asked.
I will wait for your full review, in case if there are any comments,
so I can cover them in one patch.

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCHv11 0/3] rdmacg: IB/core: rdma controller support Parav Pandit <pandit.parav@gmail.com> - 2016-08-22 14:40 +0200
  [PATCHv11 2/3] IB/core: added support to use rdma cgroup controller Parav Pandit <pandit.parav@gmail.com> - 2016-08-22 14:40 +0200
  [PATCHv11 1/3] rdmacg: Added rdma cgroup controller Parav Pandit <pandit.parav@gmail.com> - 2016-08-22 14:40 +0200
  Re: [PATCHv11 0/3] rdmacg: IB/core: rdma controller support Tejun Heo <tj@kernel.org> - 2016-08-24 23:20 +0200
    Re: [PATCHv11 0/3] rdmacg: IB/core: rdma controller support Christoph Hellwig <hch@lst.de> - 2016-08-25 09:40 +0200
      Re: [PATCHv11 0/3] rdmacg: IB/core: rdma controller support Parav Pandit <pandit.parav@gmail.com> - 2016-08-25 10:10 +0200

csiph-web