Re: [PATCH V1 7/9] ib/pma: add include file for IBA performance counters definitions

[Hal Rosenstock <hal-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>]

On 7/6/2011 2:45 PM, Jason Gunthorpe wrote:
> On Wed, Jul 06, 2011 at 02:24:59PM -0400, Hal Rosenstock wrote:
> 
>>> Vendor specific stuff must be placed into a vendor specific class and
>>> not put into the classes controlled by IBA.
>>
>> Either that or modify that compliance to allow for (optional) vendor
>> specific attributes in the PerfMgt class as is already done for the SM
>> class.
> 
> Just adding vendor usable attributes doesn't really address the
> problem of discoverability. With the SM class it is at least
> imaginable there is some kind of out of band discovery before the
> attributes are used, can't see that for PM. 

There are other ways to deal with this issue IMO. Moving it to a vendor
class doesn't make the issue necessarily go away either.

> Plus, there is no defined SMP vendor specific MAD, only GMP, 

True.

> so it makes a little more sense..

I think it could be done either way.

-- Hal

> Jason
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html