diff options
| author | Hal Rosenstock <halr@voltaire.com> | 2006-04-12 21:29:10 -0400 |
|---|---|---|
| committer | Roland Dreier <rolandd@cisco.com> | 2006-04-19 14:40:11 -0400 |
| commit | 64cb9c6aff273b1cd449e773c937378d68233f8b (patch) | |
| tree | a39978e7cbc58de3edda228c95ca328ced510435 | |
| parent | f80887d0b9e1af481dc4a30fc145dfed24ddfd59 (diff) | |
IB/mad: Fix RMPP version check during agent registration
Only check that RMPP version is not specified when MAD class does not
support RMPP. Just because a class is allowed to use RMPP doesn't
mean that rmpp_version needs to be set for the MAD agent to
register. Checking this was a recent change which was too pedantic.
Signed-off-by: Hal Rosenstock <halr@voltaire.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
| -rw-r--r-- | drivers/infiniband/core/mad.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/infiniband/core/mad.c b/drivers/infiniband/core/mad.c index 3a702da83e41..469b6923a2e2 100644 --- a/drivers/infiniband/core/mad.c +++ b/drivers/infiniband/core/mad.c | |||
| @@ -228,10 +228,7 @@ struct ib_mad_agent *ib_register_mad_agent(struct ib_device *device, | |||
| 228 | goto error1; | 228 | goto error1; |
| 229 | } | 229 | } |
| 230 | /* Make sure class supplied is consistent with RMPP */ | 230 | /* Make sure class supplied is consistent with RMPP */ |
| 231 | if (ib_is_mad_class_rmpp(mad_reg_req->mgmt_class)) { | 231 | if (!ib_is_mad_class_rmpp(mad_reg_req->mgmt_class)) { |
| 232 | if (!rmpp_version) | ||
| 233 | goto error1; | ||
| 234 | } else { | ||
| 235 | if (rmpp_version) | 232 | if (rmpp_version) |
| 236 | goto error1; | 233 | goto error1; |
| 237 | } | 234 | } |
