On Tue, Dec 19, 2017 at 03:12:01PM -0600, Steve Wise wrote: > > On Tue, Dec 19, 2017 at 04:14:16PM +0530, Potnuri Bharat Teja wrote: > > > @@ -64,12 +64,6 @@ > > > #define DRV_NAME "iw_cxgb4" > > > #define MOD DRV_NAME ":" > > > > > > -#ifdef pr_fmt > > > -#undef pr_fmt > > > -#endif > > > - > > > -#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt > > > - > > > > Lots of other places in RDMA set a prefix, can we come to some agreement > if > > we > > should drop or keep this unverisally? > > Either way we go, pr_fmt being redefined screws up pr_debug() which > optionally prepends the function, line, module names depending on the > dynamic debug settings. You end up with the module name embedded in the > middle of the log statement, and duplicated with +m is used. That is why > Bharat removed it from iw_cxgb4. > > In general, I like module names in log statements, but we just need to do it > the right way to avoid the pr_debug issue... Seems like a bigger problem than for linux-rdma? Shouldn't many of these prints be using dev_* stuff and be prefixed with their device name? Usually pr should only be used in places where the device does not yet exist. 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
