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

From: Parav Pandit
Date: Sun Feb 28 2016 - 04:03:08 EST


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 <haggaie@xxxxxxxxxxxx> wrote:
> On 24/02/2016 17:21, Parav Pandit wrote:
>> On Wed, Feb 24, 2016 at 7:56 PM, Haggai Eran <haggaie@xxxxxxxxxxxx> wrote:
>>> On 20/02/2016 13:00, Parav Pandit wrote:
>>>> Added documentation for v1 and v2 version describing high
>>>> level design and usage examples on using rdma controller.
>>>>
>>>> Signed-off-by: Parav Pandit <pandit.parav@xxxxxxxxx>
>>>
>>> I think you might want to mention that resource limits are reflected
>>> in the results returned from ib_uverbs_query_device/ibv_query_device
>>> or printed from "ibv_devinfo -v".
>>>
>> Its valid point.
>> Since this documentation is for rdma controller, I was wondering
>> should I have it this documentation or should I add the uverbs_cmds.c?
>
> I was thinking it should be in the documentation because an application
> developer might look there first, without reading uverbs_cmd.c.
>
> Haggai
>