Sat, Sep 07, 2019 at 06:17:30AM CEST, jakub.kicin...@netronome.com wrote: >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.
Ok. > >We'd also like to add that to 'fw_load_policy'. WDYT? Ok.