IB/mad: Remove redundant check from smi_check_local_dr_smp()
authorRalph Campbell <ralphc@pathscale.com>
Fri, 3 Feb 2006 22:30:24 +0000 (14:30 -0800)
committerRoland Dreier <rolandd@cisco.com>
Mon, 20 Mar 2006 18:08:11 +0000 (10:08 -0800)
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>
drivers/infiniband/core/agent.c

index 34b724afd28daf15ca52e14270a796fc9cfcede3..1a8f94588364a15a8cc861981728ddba82e8d680 100644 (file)
@@ -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 "