diff options
author | Eran Ben Elisha <eranbe@mellanox.com> | 2019-03-03 09:57:29 +0100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2019-03-04 20:00:43 +0100 |
commit | a0a21adb6a935c24111ae3a68ed406025753ce98 (patch) | |
tree | 73c814f8a69845f475f7a75df251616d5df8b6cc /Documentation/DMA-API.txt | |
parent | sctp: call iov_iter_revert() after sending ABORT (diff) | |
download | linux-a0a21adb6a935c24111ae3a68ed406025753ce98.tar.xz linux-a0a21adb6a935c24111ae3a68ed406025753ce98.zip |
devlink: Update reporter state to error even if recover aborted
If devlink_health_report() aborted the recover flow due to grace period checker,
it left the reporter status as DEVLINK_HEALTH_REPORTER_STATE_HEALTHY, which is
a bug. Fix that by always setting the reporter state to
DEVLINK_HEALTH_REPORTER_STATE_ERROR prior to running the checker mentioned above.
In addition, save the previous health_state in a temporary variable, then use
it in the abort check comparison instead of using reporter->health_state which
might be already changed.
Fixes: c8e1da0bf923 ("devlink: Add health report functionality")
Signed-off-by: Eran Ben Elisha <eranbe@mellanox.com>
Acked-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'Documentation/DMA-API.txt')
0 files changed, 0 insertions, 0 deletions