On Thu, Dec 08, 2016 at 12:16:39PM -0600, Tatyana Nikolova wrote: > +extern unsigned int i40iw_dbg; > +#define i40iw_debug(fmt, args...) \ > + if (i40iw_dbg) \ > + fprintf(stderr, PFX "%s: " fmt, __FUNCTION__, ##args) > + No, this is an unsafe way to use defines, wrap it in do / while (0) I would also welcome providing general infrastructure for this - every drvier seems to have this same basic approach. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html