On Mon, Jun 05, 2017 at 01:42:37PM +0300, Moni Shoua wrote: > On Mon, Jun 5, 2017 at 12:14 PM, Yuval Shaia <yuval.shaia@xxxxxxxxxx> wrote: > > The function get_link_ksettings might return bad status indicating a > > failure to retrieve interface atttibutes. > > Check return value to cover this case. > > > > While there, change the zero-initialization to "compiler-helper" instead > > of an expensive call to memcpy. > > > > Signed-off-by: Yuval Shaia <yuval.shaia@xxxxxxxxxx> > > --- > > drivers/infiniband/hw/bnxt_re/ib_verbs.c | 16 +++++++++------- > > 1 file changed, 9 insertions(+), 7 deletions(-) > > > > diff --git a/drivers/infiniband/hw/bnxt_re/ib_verbs.c b/drivers/infiniband/hw/bnxt_re/ib_verbs.c > > index 7ba9e69..10c7189 100644 > > --- a/drivers/infiniband/hw/bnxt_re/ib_verbs.c > > +++ b/drivers/infiniband/hw/bnxt_re/ib_verbs.c > > @@ -183,17 +183,19 @@ int bnxt_re_modify_device(struct ib_device *ibdev, > > > > static void __to_ib_speed_width(struct net_device *netdev, u8 *speed, u8 *width) > > { > > - struct ethtool_link_ksettings lksettings; > > - u32 espeed; > > + u32 espeed = SPEED_UNKNOWN; > > > > if (netdev->ethtool_ops && netdev->ethtool_ops->get_link_ksettings) { > > - memset(&lksettings, 0, sizeof(lksettings)); > > + struct ethtool_link_ksettings lksettings = {0}; > > + int rc; > > + > > rtnl_lock(); > > - netdev->ethtool_ops->get_link_ksettings(netdev, &lksettings); > > + rc = netdev->ethtool_ops->get_link_ksettings(netdev, > > + &lksettings); > > rtnl_unlock(); > > - espeed = lksettings.base.speed; > > - } else { > > - espeed = SPEED_UNKNOWN; > > + > > + if (!rc) > > + espeed = lksettings.base.speed; > > } > > switch (espeed) { > > case SPEED_1000: > > -- > > 2.9.4 > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > > the body of a message to majordomo@xxxxxxxxxxxxxxx > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > It looks like that bnxt driver also has similar code (function > __to_ib_speed_width() in drivers/infiniband/hw/bnxt_re/ib_verbs.c) Sorry, i'm not following, this one fixes bnxt_re > Maybe you can move this function to IB/core and use it in both places (or more) > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html