On Fri, 6 Sep 2019 11:40:54 -0700, Dirk van der Merwe wrote: > >> DEVLINK_PARAM_RESET_DEV_VALUE_UNKNOWN (0) > >> + Unknown or invalid value. > > Why do you need this? Do you have usecase for this value? > > I added this in to avoid having the entire netlink dump fail when there > are invalid values read from hardware. > > This way, it can report an unknown or invalid value instead of failing > the operation.
That's the first reason, the second is that we also want to report the unknown value if it's not recognized by the driver. For u8/enum parameters the value may possibly be set to a value older driver doesn't understand, but users should still be able to set them to one of the known ones. We'd also like to add that to 'fw_load_policy'. WDYT?