diff options
author | Roland Dreier <rolandd@cisco.com> | 2007-02-26 15:57:08 -0500 |
---|---|---|
committer | Roland Dreier <rolandd@cisco.com> | 2007-02-26 15:57:08 -0500 |
commit | 843613b04744d5b65c2f37975c5310f366a0d070 (patch) | |
tree | 08b188743c4304cc34964a99fac1595267eb3ab9 | |
parent | 2f236735fd05259a07a28233dcd07a8a6dddee9b (diff) |
IPoIB: Correct debugging output when path record lookup fails
If path_rec_completion() is passed a non-NULL path record pointer
along with an unsuccessful status value, the tracing code incorrectly
prints the (invalid) DLID from the path record rather than the more
interesting status code. The actual logic of the function correctly
uses the path record only if the status indicates a successful lookup.
Signed-off-by: Roland Dreier <rolandd@cisco.com>
-rw-r--r-- | drivers/infiniband/ulp/ipoib/ipoib_main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c index 18d27fd352ad..f9dbc6f68145 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_main.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c | |||
@@ -385,7 +385,7 @@ static void path_rec_completion(int status, | |||
385 | struct sk_buff *skb; | 385 | struct sk_buff *skb; |
386 | unsigned long flags; | 386 | unsigned long flags; |
387 | 387 | ||
388 | if (pathrec) | 388 | if (!status) |
389 | ipoib_dbg(priv, "PathRec LID 0x%04x for GID " IPOIB_GID_FMT "\n", | 389 | ipoib_dbg(priv, "PathRec LID 0x%04x for GID " IPOIB_GID_FMT "\n", |
390 | be16_to_cpu(pathrec->dlid), IPOIB_GID_ARG(pathrec->dgid)); | 390 | be16_to_cpu(pathrec->dlid), IPOIB_GID_ARG(pathrec->dgid)); |
391 | else | 391 | else |