linux-kernel - Re: [PATCHv6 3/3] rdmacg: Added documentation for rdmacg

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAG53R5XH2SbvNH=z7OL4v9i+r7cj4D8vPNZDSutjay3myoRkVw@mail.gmail.com>
Date:	Sun, 28 Feb 2016 14:32:45 +0530
From:	Parav Pandit <pandit.parav@...il.com>
To:	Haggai Eran <haggaie@...lanox.com>
Cc:	cgroups@...r.kernel.org, linux-doc@...r.kernel.org,
	linux-kernel@...r.kernel.org, linux-rdma@...r.kernel.org,
	Tejun Heo <tj@...nel.org>, lizefan@...wei.com,
	Johannes Weiner <hannes@...xchg.org>,
	Doug Ledford <dledford@...hat.com>,
	Liran Liss <liranl@...lanox.com>,
	"Hefty, Sean" <sean.hefty@...el.com>,
	Jason Gunthorpe <jgunthorpe@...idianresearch.com>,
	Jonathan Corbet <corbet@....net>, james.l.morris@...cle.com,
	serge@...lyn.com, Or Gerlitz <ogerlitz@...lanox.com>,
	Matan Barak <matanb@...lanox.com>, raindel@...lanox.com,
	akpm@...ux-foundation.org, linux-security-module@...r.kernel.org
Subject: Re: [PATCHv6 3/3] rdmacg: Added documentation for rdmacg

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@...lanox.com> wrote:
> On 24/02/2016 17:21, Parav Pandit wrote:
>> On Wed, Feb 24, 2016 at 7:56 PM, Haggai Eran <haggaie@...lanox.com> 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@...il.com>
>>>
>>> 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
>