Commit 815bc3ac authored by Moshe Shemesh's avatar Moshe Shemesh Committed by Jakub Kicinski
Browse files

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: default avatarMoshe Shemesh <moshe@nvidia.com>
Reviewed-by: default avatarEran Ben Elisha <eranbe@nvidia.com>
Reviewed-by: default avatarLeon Romanovsky <leonro@nvidia.com>
Link: https://lore.kernel.org/r/1668933412-5498-1-git-send-email-moshe@nvidia.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent c90b6b10

net/core/devlink.c

+0 −2
Original line number Diff line number Diff line
@@ -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 */