On Mon, Dec 11, 2017 at 09:57:23AM -0500, Hal Rosenstock wrote: > > Signed-off-by: Hal Rosenstock <hal@xxxxxxxxxxxx> > --- > src/smpdump.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/src/smpdump.c b/src/smpdump.c > index bc8559a..bb304d4 100644 > --- a/src/smpdump.c > +++ b/src/smpdump.c > @@ -281,6 +281,8 @@ int main(int argc, char *argv[]) > if (umad_recv(portid, umad, &length, -1) != mad_agent) > IBPANIC("recv error: %s", strerror(errno)); > > + printf("%d bytes received\n", length); This seems like it is more for debugging. How will this affect any scripts which include this command? Should we make this debug output? Ira > + > if (!dump_char) { > xdump(stdout, 0, smp->data, 64); > if (smp->status) > -- > 2.8.4 > -- 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