On Wed, May 07, 2014 at 02:30:04PM +0000, Weiny, Ira wrote: > > > > On Mon, May 05, 2014 at 09:44:47PM -0400, ira.weiny@xxxxxxxxx wrote: > > > From: Ira Weiny <ira.weiny@xxxxxxxxx> > > > > > > Also combine previously line broken user-visible strings as per > > > Documentation/CodingStyle > > > > These seem like nice cleanups, however it would be much nicer to use the > > dev_* versions when a struct device is available, I think many of these cases > > can reach the /dev/umadX struct device.. > > Ok, this works nice for ib_umad: > > infiniband_mad umad1: ib_umad_reg_agent2 failed: invalid registration flags specified 0x2; supported 0x1 > vs. > user_mad: ib_umad_reg_agent2 failed: invalid registration flags specified 0x2; supported 0x1 Right.. > However in ib_mad I'm not so sure. The error now implies it is the > driver which is flagging the error, _not_ ib_mad. Does it matter? ib_mad is a kernel implementation detail, and a kernel consumer could trigger the debugging print as well. The function name is a big enough clue that the problem is ib_mad related, IMHO. 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