Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 6f9d5613 authored by Eran Ben Elisha's avatar Eran Ben Elisha Committed by David S. Miller
Browse files

devlink: Add health set command 



Add devlink health set command, in order to set configuration parameters
for a specific reporter.
Supported parameters are:
- graceful_period: Time interval between auto recoveries (in msec)
- auto_recover: Determines if the devlink shall execute recover upon
		receiving error for the reporter

Signed-off-by: default avatarEran Ben Elisha <eranbe@mellanox.com>
Reviewed-by: default avatarMoshe Shemesh <moshe@mellanox.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent ff253fed

include/uapi/linux/devlink.h

+1 −0
Original line number Diff line number Diff line
@@ -90,6 +90,7 @@ enum devlink_command { 
	DEVLINK_CMD_REGION_READ,



	DEVLINK_CMD_HEALTH_REPORTER_GET,

	DEVLINK_CMD_HEALTH_REPORTER_SET,



	/* add new commands above here */

	__DEVLINK_CMD_MAX,

net/core/devlink.c

+36 −0
Original line number Diff line number Diff line
@@ -4460,6 +4460,33 @@ devlink_nl_cmd_health_reporter_get_dumpit(struct sk_buff *msg, 
	return msg->len;

}



static int

devlink_nl_cmd_health_reporter_set_doit(struct sk_buff *skb,

					struct genl_info *info)

{

	struct devlink *devlink = info->user_ptr[0];

	struct devlink_health_reporter *reporter;



	reporter = devlink_health_reporter_get_from_info(devlink, info);

	if (!reporter)

		return -EINVAL;



	if (!reporter->ops->recover &&

	    (info->attrs[DEVLINK_ATTR_HEALTH_REPORTER_GRACEFUL_PERIOD] ||

	     info->attrs[DEVLINK_ATTR_HEALTH_REPORTER_AUTO_RECOVER]))

		return -EINVAL;



	if (info->attrs[DEVLINK_ATTR_HEALTH_REPORTER_GRACEFUL_PERIOD])

		reporter->graceful_period =

			nla_get_u64(info->attrs[DEVLINK_ATTR_HEALTH_REPORTER_GRACEFUL_PERIOD]);



	if (info->attrs[DEVLINK_ATTR_HEALTH_REPORTER_AUTO_RECOVER])

		reporter->auto_recover =

			nla_get_u8(info->attrs[DEVLINK_ATTR_HEALTH_REPORTER_AUTO_RECOVER]);



	return 0;

}



static const struct nla_policy devlink_nl_policy[DEVLINK_ATTR_MAX + 1] = {

	[DEVLINK_ATTR_BUS_NAME] = { .type = NLA_NUL_STRING },

	[DEVLINK_ATTR_DEV_NAME] = { .type = NLA_NUL_STRING },
@@ -4486,6 +4513,8 @@ static const struct nla_policy devlink_nl_policy[DEVLINK_ATTR_MAX + 1] = { 
	[DEVLINK_ATTR_REGION_NAME] = { .type = NLA_NUL_STRING },

	[DEVLINK_ATTR_REGION_SNAPSHOT_ID] = { .type = NLA_U32 },

	[DEVLINK_ATTR_HEALTH_REPORTER_NAME] = { .type = NLA_NUL_STRING },

	[DEVLINK_ATTR_HEALTH_REPORTER_GRACEFUL_PERIOD] = { .type = NLA_U64 },

	[DEVLINK_ATTR_HEALTH_REPORTER_AUTO_RECOVER] = { .type = NLA_U8 },

};



static const struct genl_ops devlink_nl_ops[] = {
@@ -4714,6 +4743,13 @@ static const struct genl_ops devlink_nl_ops[] = { 
		.internal_flags = DEVLINK_NL_FLAG_NEED_DEVLINK,

		/* can be retrieved by unprivileged users */

	},

	{

		.cmd = DEVLINK_CMD_HEALTH_REPORTER_SET,

		.doit = devlink_nl_cmd_health_reporter_set_doit,

		.policy = devlink_nl_policy,

		.flags = GENL_ADMIN_PERM,

		.internal_flags = DEVLINK_NL_FLAG_NEED_DEVLINK,

	},

};



static struct genl_family devlink_nl_family __ro_after_init = {