[openib-general] Re: [PATCH 1 of 2] static rate encoding change (kernel)

[Roland Dreier]

    Michael> ib_max_link_rate then?  The point is that we cant force
    Michael> the actual rate at which the connection works, but we can
    Michael> set the top limit to avoid back-pressure along the path,
    Michael> so its the maximum rate, not the actual rate.

I don't get where the "max" is coming from for the type name.  This
enum represents values of the PathRecord:Rate field, so it's just a
rate.  In fact maybe enum ib_path_rate is the best name for it.

Then a consumer can take the path rate and put it in the static_rate
field of an ah_attr to control the maximum rate that packets are
injected.  If anything, "static_rate" is what would make sense to
rename to "max_rate".

In other words, the enum just holds rates.  It's the
ib_ah_attr.static_rate field where the rate represents a maximum.

 - R.