On 7/7/2016 1:36 PM, Christoph Lameter wrote: > > V1->V2 > - Tested. Found that we missed modifying one table. > > > Add the missing port_xmit_wait counter. This counter is displayed through > some tools like perfquery but is not available via sysfs. > > For the PORT_PMA_ATTR macro the _counter field is set to zero > allowing us to specify the offset directly like with PORT_PMA_ATTR_EXT I'm not following the relationship of _counter field to the offset other than they are combined in the index and then I don't see counter part of index ever being used. So it looks to me like _counter can be deprecated/removed which is what Roland wrote in the thread you referenced below. > > See also the earlier work in 2008 by Vladimir Skolovsky > > https://www.mail-archive.com/general@xxxxxxxxxxxxxxxxxxxxx/msg20313.html > > Tested-by: Christoph Lameter <cl@xxxxxxxxx> > Signed-off-by: Christoph Lameter <cl@xxxxxxxxx> Reviewed-by: Hal Rosenstock <hal@xxxxxxxxxxxx> > > Index: rh73alpha/drivers/infiniband/core/sysfs.c > =================================================================== > --- rh73alpha.orig/drivers/infiniband/core/sysfs.c > +++ rh73alpha/drivers/infiniband/core/sysfs.c > @@ -530,6 +530,7 @@ static PORT_PMA_ATTR(port_xmit_data > static PORT_PMA_ATTR(port_rcv_data , 13, 32, 224); > static PORT_PMA_ATTR(port_xmit_packets , 14, 32, 256); > static PORT_PMA_ATTR(port_rcv_packets , 15, 32, 288); > +static PORT_PMA_ATTR(port_xmit_wait , 0, 32, 320); > > /* > * Counters added by extended set > @@ -560,6 +561,7 @@ static struct attribute *pma_attrs[] = { > &port_pma_attr_port_rcv_data.attr.attr, > &port_pma_attr_port_xmit_packets.attr.attr, > &port_pma_attr_port_rcv_packets.attr.attr, > + &port_pma_attr_port_xmit_wait.attr.attr, > NULL > }; > > @@ -579,6 +581,7 @@ static struct attribute *pma_attrs_ext[] > &port_pma_attr_ext_port_xmit_data.attr.attr, > &port_pma_attr_ext_port_rcv_data.attr.attr, > &port_pma_attr_ext_port_xmit_packets.attr.attr, > + &port_pma_attr_port_xmit_wait.attr.attr, > &port_pma_attr_ext_port_rcv_packets.attr.attr, > &port_pma_attr_ext_unicast_rcv_packets.attr.attr, > &port_pma_attr_ext_unicast_xmit_packets.attr.attr, > @@ -604,6 +607,7 @@ static struct attribute *pma_attrs_noiet > &port_pma_attr_ext_port_rcv_data.attr.attr, > &port_pma_attr_ext_port_xmit_packets.attr.attr, > &port_pma_attr_ext_port_rcv_packets.attr.attr, > + &port_pma_attr_port_xmit_wait.attr.attr, > NULL > }; > > -- 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