Commit 5bbd21df authored by Dirk van der Merwe's avatar Dirk van der Merwe Committed by David S. Miller
Browse files

devlink: add 'reset_dev_on_drv_probe' param 



Add the 'reset_dev_on_drv_probe' devlink parameter, controlling the
device reset policy on driver probe.

This parameter is useful in conjunction with the existing
'fw_load_policy' parameter.

Signed-off-by: default avatarDirk van der Merwe <dirk.vandermerwe@netronome.com>
Signed-off-by: default avatarJakub Kicinski <jakub.kicinski@netronome.com>
Signed-off-by: default avatarSimon Horman <simon.horman@netronome.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent e019a3b1

Documentation/networking/devlink-params.txt

+14 −0
Original line number Diff line number Diff line
@@ -51,3 +51,17 @@ fw_load_policy [DEVICE, GENERIC] 
			* DEVLINK_PARAM_FW_LOAD_POLICY_VALUE_DISK (2)

			  Load firmware currently available on host's disk.

			Type: u8



reset_dev_on_drv_probe	[DEVICE, GENERIC]

			Controls the device's reset policy on driver probe.

			Valid values:

			* DEVLINK_PARAM_RESET_DEV_ON_DRV_PROBE_VALUE_UNKNOWN (0)

			  Unknown or invalid value.

			* DEVLINK_PARAM_RESET_DEV_ON_DRV_PROBE_VALUE_ALWAYS (1)

			  Always reset device on driver probe.

			* DEVLINK_PARAM_RESET_DEV_ON_DRV_PROBE_VALUE_NEVER (2)

			  Never reset device on driver probe.

			* DEVLINK_PARAM_RESET_DEV_ON_DRV_PROBE_VALUE_DISK (3)

			  Reset only if device firmware can be found in the

			  filesystem.

			Type: u8

include/net/devlink.h

+5 −0
Original line number Diff line number Diff line
@@ -398,6 +398,7 @@ enum devlink_param_generic_id { 
	DEVLINK_PARAM_GENERIC_ID_MSIX_VEC_PER_PF_MAX,

	DEVLINK_PARAM_GENERIC_ID_MSIX_VEC_PER_PF_MIN,

	DEVLINK_PARAM_GENERIC_ID_FW_LOAD_POLICY,

	DEVLINK_PARAM_GENERIC_ID_RESET_DEV_ON_DRV_PROBE,



	/* add new param generic ids above here*/

	__DEVLINK_PARAM_GENERIC_ID_MAX,
@@ -428,6 +429,10 @@ enum devlink_param_generic_id { 
#define DEVLINK_PARAM_GENERIC_FW_LOAD_POLICY_NAME "fw_load_policy"

#define DEVLINK_PARAM_GENERIC_FW_LOAD_POLICY_TYPE DEVLINK_PARAM_TYPE_U8



#define DEVLINK_PARAM_GENERIC_RESET_DEV_ON_DRV_PROBE_NAME \

	"reset_dev_on_drv_probe"

#define DEVLINK_PARAM_GENERIC_RESET_DEV_ON_DRV_PROBE_TYPE DEVLINK_PARAM_TYPE_U8



#define DEVLINK_PARAM_GENERIC(_id, _cmodes, _get, _set, _validate)	\

{									\

	.id = DEVLINK_PARAM_GENERIC_ID_##_id,				\

include/uapi/linux/devlink.h

+7 −0
Original line number Diff line number Diff line
@@ -205,6 +205,13 @@ enum devlink_param_fw_load_policy_value { 
	DEVLINK_PARAM_FW_LOAD_POLICY_VALUE_DISK,

};



enum devlink_param_reset_dev_on_drv_probe_value {

	DEVLINK_PARAM_RESET_DEV_ON_DRV_PROBE_VALUE_UNKNOWN,

	DEVLINK_PARAM_RESET_DEV_ON_DRV_PROBE_VALUE_ALWAYS,

	DEVLINK_PARAM_RESET_DEV_ON_DRV_PROBE_VALUE_NEVER,

	DEVLINK_PARAM_RESET_DEV_ON_DRV_PROBE_VALUE_DISK,

};



enum {

	DEVLINK_ATTR_STATS_RX_PACKETS,		/* u64 */

	DEVLINK_ATTR_STATS_RX_BYTES,		/* u64 */

net/core/devlink.c

+5 −0
Original line number Diff line number Diff line
@@ -2852,6 +2852,11 @@ static const struct devlink_param devlink_param_generic[] = { 
		.name = DEVLINK_PARAM_GENERIC_FW_LOAD_POLICY_NAME,

		.type = DEVLINK_PARAM_GENERIC_FW_LOAD_POLICY_TYPE,

	},

	{

		.id = DEVLINK_PARAM_GENERIC_ID_RESET_DEV_ON_DRV_PROBE,

		.name = DEVLINK_PARAM_GENERIC_RESET_DEV_ON_DRV_PROBE_NAME,

		.type = DEVLINK_PARAM_GENERIC_RESET_DEV_ON_DRV_PROBE_TYPE,

	},

};



static int devlink_param_generic_verify(const struct devlink_param *param)

CRA Git | Maintained and supported by SUSTech CRA and CCSE