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.

Reply via email to