[ofa-general] Re: [PATCH v2] rdma_cm: Add debugfs entries to monitor rdma_cm connections

[Jason Gunthorpe]

On Mon, Apr 27, 2009 at 04:15:22PM +0300, Or Gerlitz wrote:
> Moni Shoua wrote:
>> A listener that listens an all will appear on all devices
> This sounds like something that can cause confusion when someone is looking 
> on multiple devices... but maybe we can live with that?
>
>>> I'd like to see how we come up with a solution with somehow less 
>>> space-ing between columns, maybe take a look at the netperf code, e.g 
>>> that generates output as below. 
>> I reserved enough space for IPV6 addresses. The output below is good for 
>> IPV4 addresses  only. 
> I understand that, but lets try to think if/how this can be done better, 
> specifically if we want to add more params such as IB L4/L3/L2 (QPN/PKEY, 
> GUID, LID/SL) info to the output.

Sounds like you should just bite the bullet and implement this with
netlink. All of Or's concerns are easially addressed that
way. Including all the IB path information, and APM information is
definately worthwhile.

BTW, including a PID is not best, you should include enough
information to figure out the pid(s) from proc/xx/fd, and vice versa.

Jason