On Mon, Jun 25, 2012 at 10:33:31PM -0400, Theodore Ts'o wrote:

On Tue, Jun 26, 2012 at 12:10:47PM +1000, Tony Breeds wrote:

quoted

I was thinking that this was the very sinmple matter of making
BMAP_STATS conditional rather that have it always #defined.  Is that
what you were thinking?

Also what's the correct way to get BMAP_STATS_OPS defined?

Yeah, I was thinking to simply rename BMAP_STATS and BMAP_STATS_OPS to
ENABLE_BMAP_STATS and ENABLE_BMAP_STATS_OPS, and then handle them via
an AC_ARG_ENABLE much like --enable-compression is handled (with the
first enabled by default, so --disable-bmap-stats to the configure
script would disable it, while the much more heavier weight from a
performance standpoint BMAP_STATS_OPS would be disabled by default, so
the user would have to specify --enable-bmap-stats-ops to the
configure script to enable it.)

Thanks for the clarification.  Should be doable today.

Yours Tony