Skip to content

Commit

Permalink
IB/mad: Remove redundant check from smi_check_local_dr_smp()
Browse files Browse the repository at this point in the history
smi_check_local_dr_smp() is called only from two places in core/mad.c
It returns 0 or 1.  In smi_check_local_dr_smp(), it checks for
a directed route SMP but this function is only called when the SMP
is a directed route so this is a NOP.

Signed-off-by: Hal Rosenstock <halr@voltaire.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
  • Loading branch information
Ralph Campbell authored and Roland Dreier committed Mar 20, 2006
1 parent d36f34a commit 5f0b67e
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions drivers/infiniband/core/agent.c
Original file line number Diff line number Diff line change
Expand Up @@ -84,9 +84,6 @@ int smi_check_local_dr_smp(struct ib_smp *smp,
{
struct ib_agent_port_private *port_priv;

if (smp->mgmt_class != IB_MGMT_CLASS_SUBN_DIRECTED_ROUTE)
return 1;

port_priv = ib_get_agent_port(device, port_num);
if (!port_priv) {
printk(KERN_DEBUG SPFX "smi_check_local_dr_smp %s port %d "
Expand Down

0 comments on commit 5f0b67e

Please sign in to comment.