diff options
author | Eran Ben Elisha <eranbe@mellanox.com> | 2019-03-03 10:57:29 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2019-03-04 11:00:43 -0800 |
commit | a0a21adb6a935c24111ae3a68ed406025753ce98 (patch) | |
tree | 73c814f8a69845f475f7a75df251616d5df8b6cc /include | |
parent | 901efe12318b1ea8d3e2c88a7b75ed6e6d5d7245 (diff) |
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 'include')
0 files changed, 0 insertions, 0 deletions