Commit f13a5ad8 authored by Parav Pandit's avatar Parav Pandit Committed by David S. Miller
Browse files

devlink: Add new "enable_eth" generic device param 



Add new device generic parameter to enable/disable creation of
Ethernet auxiliary device and associated device functionality
in the devlink instance.

User who prefers to disable such functionality can disable it using below
example.

$ devlink dev param set pci/0000:06:00.0 \
              name enable_eth value false cmode driverinit
$ devlink dev reload pci/0000:06:00.0

At this point devlink instance do not create auxiliary device for the
Ethernet functionality.

Signed-off-by: default avatarParav Pandit <parav@nvidia.com>
Reviewed-by: default avatarJiri Pirko <jiri@nvidia.com>
Reviewed-by: default avatarLeon Romanovsky <leonro@nvidia.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent e9c130ad

Documentation/networking/devlink/devlink-params.rst

+4 −0
Original line number Diff line number Diff line
@@ -97,6 +97,10 @@ own name. 
   * - ``enable_roce``

     - Boolean

     - Enable handling of RoCE traffic in the device.

   * - ``enable_eth``

     - Boolean

     - When enabled, the device driver will instantiate Ethernet specific

       auxiliary device of the devlink device.

   * - ``internal_err_reset``

     - Boolean

     - When enabled, the device driver will reset the device on internal

include/net/devlink.h

+4 −0
Original line number Diff line number Diff line
@@ -519,6 +519,7 @@ enum devlink_param_generic_id { 
	DEVLINK_PARAM_GENERIC_ID_RESET_DEV_ON_DRV_PROBE,

	DEVLINK_PARAM_GENERIC_ID_ENABLE_ROCE,

	DEVLINK_PARAM_GENERIC_ID_ENABLE_REMOTE_DEV_RESET,

	DEVLINK_PARAM_GENERIC_ID_ENABLE_ETH,



	/* add new param generic ids above here*/

	__DEVLINK_PARAM_GENERIC_ID_MAX,
@@ -559,6 +560,9 @@ enum devlink_param_generic_id { 
#define DEVLINK_PARAM_GENERIC_ENABLE_REMOTE_DEV_RESET_NAME "enable_remote_dev_reset"

#define DEVLINK_PARAM_GENERIC_ENABLE_REMOTE_DEV_RESET_TYPE DEVLINK_PARAM_TYPE_BOOL



#define DEVLINK_PARAM_GENERIC_ENABLE_ETH_NAME "enable_eth"

#define DEVLINK_PARAM_GENERIC_ENABLE_ETH_TYPE DEVLINK_PARAM_TYPE_BOOL



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

{									\

	.id = DEVLINK_PARAM_GENERIC_ID_##_id,				\

net/core/devlink.c

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

		.type = DEVLINK_PARAM_GENERIC_ENABLE_REMOTE_DEV_RESET_TYPE,

	},

	{

		.id = DEVLINK_PARAM_GENERIC_ID_ENABLE_ETH,

		.name = DEVLINK_PARAM_GENERIC_ENABLE_ETH_NAME,

		.type = DEVLINK_PARAM_GENERIC_ENABLE_ETH_TYPE,

	},

};



static int devlink_param_generic_verify(const struct devlink_param *param)