diff options
| author | Moshe Shemesh <moshe@nvidia.com> | 2022-11-20 09:36:52 +0100 |
|---|---|---|
| committer | Jakub Kicinski <kuba@kernel.org> | 2022-11-23 05:41:38 +0100 |
| commit | 815bc3ac75e9e34727f8ca78380266f34a3b6c66 (patch) | |
| tree | e39df1fcf17ded96078c45704c3f3165f32e4350 /net/core | |
| parent | tcp: Fix build break when CONFIG_IPV6=n (diff) | |
| download | linux-815bc3ac75e9e34727f8ca78380266f34a3b6c66.tar.xz linux-815bc3ac75e9e34727f8ca78380266f34a3b6c66.zip | |
devlink: remove redundant health state set to error
Reporter health_state is set twice to error in devlink_health_report().
Remove second time as it is redundant.
Signed-off-by: Moshe Shemesh <moshe@nvidia.com>
Reviewed-by: Eran Ben Elisha <eranbe@nvidia.com>
Reviewed-by: Leon Romanovsky <leonro@nvidia.com>
Link: https://lore.kernel.org/r/1668933412-5498-1-git-send-email-moshe@nvidia.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'net/core')
| -rw-r--r-- | net/core/devlink.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/net/core/devlink.c b/net/core/devlink.c index d93bc95cd7cb..cea154ddce7a 100644 --- a/net/core/devlink.c +++ b/net/core/devlink.c @@ -7846,8 +7846,6 @@ int devlink_health_report(struct devlink_health_reporter *reporter, return -ECANCELED; } - reporter->health_state = DEVLINK_HEALTH_REPORTER_STATE_ERROR; - if (reporter->auto_dump) { mutex_lock(&reporter->dump_lock); /* store current dump of current error, for later analysis */ |