From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [PATCH -next] drbd: trace depends on TRACING Date: Tue, 29 Sep 2009 11:37:47 -0400 Message-ID: <20090929153747.GA29621@infradead.org> References: <20090925133830.1ba29584.sfr@canb.auug.org.au> <4ABD4C28.9070202@oracle.com> <200909291732.51383.philipp.reisner@linbit.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <200909291732.51383.philipp.reisner-63ez5xqkn6DQT0dZR+AlfA@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: drbd-dev-bounces-cunTk1MwBs8qoQakbn7OcQ@public.gmane.org Errors-To: drbd-dev-bounces-cunTk1MwBs8qoQakbn7OcQ@public.gmane.org To: Philipp Reisner Cc: Randy Dunlap , Stephen Rothwell , LKML , linux-next-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, akpm , drbd-dev-cunTk1MwBs8qoQakbn7OcQ@public.gmane.org List-Id: linux-next.vger.kernel.org On Tue, Sep 29, 2009 at 05:32:50PM +0200, Philipp Reisner wrote: > On Saturday 26 September 2009 01:03:04 Randy Dunlap wrote: > > From: Randy Dunlap > > > > DRBD_TRACE should depend on TRACING. > > It's also possible that TRACEPOINTS should depend on TRACING. > > > > Hi Randy, > > I tried to figure out in which way that should be done. By looking > at "BLK_DEV_IO_TRACE" I got the impression that this should > be done by adding "select GENERIC_TRACER". Neither really. The only way of tracing that should be used is the TRACE_EVENT macros, which compile away to nothing if tracing is disabled, so no dependency is needed at all.