On Wed, May 20, 2015 at 11:32:01AM -0600, Hefty, Sean wrote:
> > diff --git a/drivers/infiniband/core/smi.c b/drivers/infiniband/core/smi.c
> > index b09fa1dec865..4969ce3338fb 100644
> > --- a/drivers/infiniband/core/smi.c
> > +++ b/drivers/infiniband/core/smi.c
> > @@ -236,21 +236,20 @@ enum smi_action smi_handle_dr_smp_recv(struct ib_smp
> > *smp, u8 node_type,
> > smp->dr_slid == IB_LID_PERMISSIVE);
> > }
> >
> > -enum smi_forward_action smi_check_forward_dr_smp(struct ib_smp *smp)
> > +static inline
> > +enum smi_forward_action __smi_check_forward_dr_smp(u8 hop_ptr, u8
> > hop_cnt,
> > + u8 direction,
> > + int dr_dlid_is_permissive,
> > + int dr_slid_is_permissive)
>
> snip
>
> > +enum smi_forward_action smi_check_forward_dr_smp(struct ib_smp *smp)
> > +{
> > + return __smi_check_forward_dr_smp(smp->hop_ptr, smp->hop_cnt,
> > + ib_get_smp_direction(smp),
> > + smp->dr_dlid == IB_LID_PERMISSIVE,
> > + smp->dr_slid != IB_LID_PERMISSIVE);
>
> The dr_slid check looks backwards.
Yes. The logic is correct but the variable name is wrong.
These should be bool as well.
Ira
--
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
