Wed, Oct 18, 2017 at 02:39:42PM CEST, steven.lin1@broadcom.com wrote:

On Wed, Oct 18, 2017 at 3:31 AM, Jiri Pirko [off-list ref] wrote:

quoted

You need to split the config option to those that are per-port and to
those that are per-asic. For each family, you have to use ither
devlink_port of devlink handle. Also, you need to split into those that are
permanent and to those who are teporary (until reset). I think you might
need some flags for that.

All these are permanent; none are temporary - that's (partially) why
we consider these to be devlink/device parameters rather than a
netdev/ethtool thing, since they take effect after the next reset and
before any drivers are loaded (i.e. the card uses these parameters for
its default/startup configuration).

Understood. But I think that this iface should be capable to serve the
options of non-permanent as well. Or this could be 2 separate interfaces
with 2 separate cmd pair. Thoughts?

I will take a closer look at splitting these between per-port and per-asic.

Thanks,
Steve