Thread (8 messages) 8 messages, 2 authors, 2022-02-19

Re: [PATCH net-next 1/4] ionic: catch transition back to RUNNING with fw_generation 0

From: Jakub Kicinski <kuba@kernel.org>
Date: 2022-02-18 04:12:19

On Thu, 17 Feb 2022 14:02:49 -0800 Shannon Nelson wrote:
In some graceful updates that get initially triggered by the
RESET event, especially with older firmware, the fw_generation
bits don't change but the fw_status is seen to go to 0 then back
to 1.  However, the driver didn't perform the restart, remained
waiting for fw_generation to change, and got left in limbo.

This is because the clearing of idev->fw_status_ready to 0
didn't happen correctly as it was buried in the transition
trigger: since the transition down was triggered not here
but in the RESET event handler, the clear to 0 didn't happen,
so the transition back to 1 wasn't detected.

Fix this particular case by bringing the setting of
idev->fw_status_ready back out to where it was before.

Fixes: 398d1e37f960 ("ionic: add FW_STOPPING state")
Signed-off-by: Shannon Nelson <redacted>
This looks like a fix, and should go separately to net.
Is there a reason behind posting together? The other patches
don't even depend on this one.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help