On Mon, Apr 06, 2015 at 04:43:01PM -0600, Hefty, Sean wrote:
> > I understand your point and I originally called this "attributes" but it
> > was
> > suggested to call it cached_dev_attrs.
> > 
> > https://www.mail-archive.com/linux-rdma%40vger.kernel.org/msg22486.html
> > 
> > I can change it back if we are all agreed.
> 
> I'll disagree with Or on this.  Unless some of these values can change 
> dynamically, these are the attributes.

Or

Any comment?

Personally, this patch was not originally part of my series.  I had a
ib_query_device call which Or objected to "yet another caching" of a device
attribute.

With the discussions on "management helpers" I probably don't need this at all.
So, if I can remove it I will and then we can debate its merits separate from 
the
OPA changes.

Ira

> 
> This leads me to, is there a reason to keep the per device query_attribute() 
> call?  (This would not be part of this series.)  Are there attributes being 
> returned from the kernel to user space that fall outside of the defined 
> attribute area?
> 
--
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

Reply via email to