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

Jason Gunthorpe jgunthorpe at obsidianresearch.com
Tue Jul 7 09:48:03 PDT 2009


On Tue, Jul 07, 2009 at 01:12:54PM +0300, Or Gerlitz wrote:
> Steve Wise wrote:
> >I agree that this is useful for debugging.
> Roland, Sean, I agree with Steve and Moni. Today there's no way to know 
> what rdma-cm connections/sessions are open now and with patch there is a 
> way, so debugfs support is very useful for debugging rdma-cm ULPs, 
> simple as that (e.g in the same manner ipoib has debugfs support), lets 
> get this in for 2.6.32
> 
> Next, for more production ready monitoring support, netlink solutions 
> could be great, Jason, thoughts?

Well, thats the rub isn't it? debugfs is not 'production ready' (by
definition) so why spend time on it?

I honestly don't understand the reluctance to do a good job here. We
all hear the same things from users - more visibility is needed - this
debufs patch kinda addresses a very tiny portion of that
visbility. Why the resistance to doing a proper job and solving the
actual problem?

debufs is entirely the wrong answer - this is not kernel debugging,
this is required user space diagnostic information.

Jason



More information about the general mailing list