Thread (3 messages) flat view 3 messages, 3 authors, 2021-11-18

Re: [PATCH net v1] devlink: Don't throw an error if flash notification sent before devlink visible

From: Jakub Kicinski <kuba@kernel.org>
Date: 2021-11-18 04:29:40
Also in: lkml

On Wed, 17 Nov 2021 16:49:09 +0200 Leon Romanovsky wrote:
From: Leon Romanovsky <leonro@nvidia.com>

The mlxsw driver calls to various devlink flash routines even before
users can get any access to the devlink instance itself. For example,
mlxsw_core_fw_rev_validate() one of such functions.

__mlxsw_core_bus_device_register
 -> mlxsw_core_fw_rev_validate
  -> mlxsw_core_fw_flash
   -> mlxfw_firmware_flash
    -> mlxfw_status_notify
     -> devlink_flash_update_status_notify
      -> __devlink_flash_update_notify
       -> WARN_ON(...)  

It causes to the WARN_ON to trigger warning about devlink not registered.

Fixes: cf530217408e ("devlink: Notify users when objects are accessible")
Reported-by: Danielle Ratson <redacted>
Tested-by: Danielle Ratson <redacted>
Signed-off-by: Leon Romanovsky <leonro@nvidia.com>
Acked-by: Jakub Kicinski <kuba@kernel.org>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help