Mon, May 18, 2020 at 10:27:15AM CEST, vasundhara-v.vo...@broadcom.com wrote:
>This patchset adds support for a "enable_hot_fw_reset" generic devlink
>parameter and use it in bnxt_en driver.
>
>Also, firmware spec. is updated to 1.10.1.40.
Hi.
We've been discussing this internally for some time.
I don't like to use params for this purpose.
We already have "devlink dev flash" and "devlink dev reload" commands.
Combination of these two with appropriate attributes should provide what
you want. The "param" you are introducing is related to either "flash"
or "reload", so I don't think it is good to have separate param, when we
can extend the command attributes.
How does flash&reload work for mlxsw now:
# devlink flash
Now new version is pending, old FW is running
# devlink reload
Driver resets the device, new FW is loaded
I propose to extend reload like this:
devlink dev reload DEV [ level { driver-default | fw-reset | driver-only |
fw-live-patch } ]
driver-default - means one of following to, according to what is
default for the driver
fw-reset - does FW reset and driver entities re-instantiation
driver-only - does driver entities re-instantiation only
fw-live-patch - does only FW live patching - no effect on kernel
Could be an enum or bitfield. Does not matter. The point is to use
reload with attribute to achieve what user wants. In your usecase, user
would do:
# devlink flash
# devlink reload level fw-live-patch
