From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steve Wise Subject: Re: [PATCH v2 13/51] IB/qib: Add qib_driver.c Date: Wed, 07 Apr 2010 14:24:06 -0500 Message-ID: <4BBCDBD6.8010809@opengridcomputing.com> References: <20091203190305.29507.58158.stgit@chromite.mv.qlogic.com> <20091203190413.29507.17962.stgit@chromite.mv.qlogic.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Roland Dreier Cc: Ralph Campbell , linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-rdma@vger.kernel.org Roland Dreier wrote: > > +unsigned qib_debug; > > +module_param_named(debug, qib_debug, uint, S_IWUSR | S_IRUGO); > > +MODULE_PARM_DESC(debug, "mask for debug prints"); > > Did you look at using trace events for this stuff? That gives you > extremely low overhead when tracing is turned off (dynamic patching to > NOP out the tracing when it's disabled) and also very fine-grained (per > trace site) control over what gets printed; plus you get dumping of the > trace buffer on crash, etc. > > - R. > Where can I find information on trace events? Something in Documentation/*? Thanks, Steve. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html