On Thu, 2010-07-29 at 17:56 +0200, Bart Van Assche wrote:
> Reduces the number of source code lines in ib_srp.c by replacing an explicit
> loop by a call to print_hex_dump().

I like the idea, minor nit in that we loose the SCSI prefix that tells
use which host device printed it. Then again it's a debugging aid, so I
don't know that it matters all that much.

Easily solvable with s/""/dev_name(target->scsi_host.shost_gendev)/

Hmm, that may be more ugly than it is worth.

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to