On Thu, Jun 09, 2016 at 04:40:38PM -0600, Jason Gunthorpe wrote:
> On Thu, Jun 09, 2016 at 05:54:07PM -0400, ira.weiny@xxxxxxxxx wrote:
> >
> > +static ssize_t show_fw_ver(struct device *device, struct device_attribute *attr,
> > + char *buf)
> > +{
> > + char tmp[ETHTOOL_FWVERS_LEN];
> > + struct ib_device *dev = container_of(device, struct ib_device, dev);
> > +
> > + ib_get_device_fw_str(dev, tmp, sizeof(tmp));
> > +
> > + return scnprintf(buf, PAGE_SIZE, "%s\n", tmp);
>
> No need to use the stack:
>
> ib_get_device_fw_str(dev, buf, PAGE_SIZE);
> strlcat(buf, "", PAGE_SIZE);
> return strlen(buf);
That works. Fixes Leon's comment as well.
Done.
>
> > &dev_attr_node_guid,
> > - &dev_attr_node_desc
> > + &dev_attr_node_desc,
> > + &dev_attr_fw_ver
>
> Should be
> + &dev_attr_fw_ver,
>
> Note trailing comma. Avoids git churn like the above on append.
Done.
Ira
>
> Jason
--
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
