From: Kyle Xu <zhenbing...@corigine.com> Add new device generic parameter 'enable_vnet' to enable/disable the vDPA VF instantiation for NFP devices.
This configuration is added to configure how to instantiate the NFP vf devices, if the 'enable_vnet' is enabled, the VFs will be initialized as vDPA networking auxiliary devices, otherwise, they will be initialized as PCI VF devices. This configuration must be done before VFs are created. It can be enabled/disabled and checked as following example. devlink dev param set pci/0000:01:00.0 \ name enable_vnet value true cmode runtime devlink dev param set pci/0000:01:00.0 \ name enable_vnet value false cmode runtime devlink dev param show pci/0000:01:00.0 name enable_vnet Signed-off-by: Kyle Xu <zhenbing...@corigine.com> Signed-off-by: Louis Peens <louis.pe...@corigine.com> --- .../ethernet/netronome/nfp/devlink_param.c | 49 +++++++++++++++++++ drivers/net/ethernet/netronome/nfp/nfp_main.h | 3 ++ .../net/ethernet/netronome/nfp/nfp_net_ctrl.h | 3 ++ .../net/ethernet/netronome/nfp/nfp_net_main.c | 10 ++++ 4 files changed, 65 insertions(+) diff --git a/drivers/net/ethernet/netronome/nfp/devlink_param.c b/drivers/net/ethernet/netronome/nfp/devlink_param.c index 0e1a3800f371..0683e2512b63 100644 --- a/drivers/net/ethernet/netronome/nfp/devlink_param.c +++ b/drivers/net/ethernet/netronome/nfp/devlink_param.c @@ -6,6 +6,8 @@ #include "nfpcore/nfp.h" #include "nfpcore/nfp_nsp.h" #include "nfp_main.h" +#include "nfp_net.h" +#include "nfp_net_ctrl.h" /** * struct nfp_devlink_param_u8_arg - Devlink u8 parameter get/set arguments @@ -192,6 +194,48 @@ nfp_devlink_param_u8_validate(struct devlink *devlink, u32 id, return 0; } +static int +nfp_devlink_param_enable_vnet_get(struct devlink *devlink, u32 id, + struct devlink_param_gset_ctx *ctx) +{ + struct nfp_pf *pf = devlink_priv(devlink); + + ctx->val.vbool = pf->enable_vnet; + + return 0; +} + +static int +nfp_devlink_param_enable_vnet_set(struct devlink *devlink, u32 id, + struct devlink_param_gset_ctx *ctx, + struct netlink_ext_ack *extack) +{ + struct nfp_pf *pf = devlink_priv(devlink); + struct nfp_net *nn; + int err; + + nn = list_first_entry(&pf->vnics, struct nfp_net, vnic_list); + if (!(nn->cap_w1 & NFP_NET_CFG_CTRL_ENABLE_VNET)) { + NL_SET_ERR_MSG_MOD(extack, "NFP_NET_CFG_CTRL_ENABLE_VNET not enabled"); + return -EOPNOTSUPP; + } + + if (pf->num_vfs > 0) { + NL_SET_ERR_MSG_FMT_MOD(extack, "%d VFs already created. Please remove them first", + pf->num_vfs); + return -EBUSY; + } + + err = nfp_net_update_enable_vnet(pf, ctx->val.vbool); + if (err < 0) { + NL_SET_ERR_MSG_MOD(extack, "Write ENABLE_VNET to firmware failed"); + return err; + } + + pf->enable_vnet = ctx->val.vbool; + return 0; +} + static const struct devlink_param nfp_devlink_params[] = { DEVLINK_PARAM_GENERIC(FW_LOAD_POLICY, BIT(DEVLINK_PARAM_CMODE_PERMANENT), @@ -203,6 +247,11 @@ static const struct devlink_param nfp_devlink_params[] = { nfp_devlink_param_u8_get, nfp_devlink_param_u8_set, nfp_devlink_param_u8_validate), + DEVLINK_PARAM_GENERIC(ENABLE_VNET, + BIT(DEVLINK_PARAM_CMODE_RUNTIME), + nfp_devlink_param_enable_vnet_get, + nfp_devlink_param_enable_vnet_set, + NULL), }; static int nfp_devlink_supports_params(struct nfp_pf *pf) diff --git a/drivers/net/ethernet/netronome/nfp/nfp_main.h b/drivers/net/ethernet/netronome/nfp/nfp_main.h index 14a751bfe1fe..9e782fe44c39 100644 --- a/drivers/net/ethernet/netronome/nfp/nfp_main.h +++ b/drivers/net/ethernet/netronome/nfp/nfp_main.h @@ -65,6 +65,7 @@ struct nfp_dumpspec { * @num_vfs: Number of SR-IOV VFs enabled * @fw_loaded: Is the firmware loaded? * @unload_fw_on_remove:Do we need to unload firmware on driver removal? + * @enable_vnet: Devlink parameter config, instantiate VDPA VF when enabled * @ctrl_vnic: Pointer to the control vNIC if available * @mip: MIP handle * @rtbl: RTsym table @@ -114,6 +115,7 @@ struct nfp_pf { bool fw_loaded; bool unload_fw_on_remove; + bool enable_vnet; struct nfp_net *ctrl_vnic; @@ -194,4 +196,5 @@ void nfp_devlink_params_unregister(struct nfp_pf *pf); unsigned int nfp_net_lr2speed(unsigned int linkrate); unsigned int nfp_net_speed2lr(unsigned int speed); +int nfp_net_update_enable_vnet(struct nfp_pf *pf, bool enable_vnet); #endif /* NFP_MAIN_H */ diff --git a/drivers/net/ethernet/netronome/nfp/nfp_net_ctrl.h b/drivers/net/ethernet/netronome/nfp/nfp_net_ctrl.h index 634c63c7f7eb..f0751015a8cb 100644 --- a/drivers/net/ethernet/netronome/nfp/nfp_net_ctrl.h +++ b/drivers/net/ethernet/netronome/nfp/nfp_net_ctrl.h @@ -150,6 +150,7 @@ #define NFP_NET_CFG_UPDATE_MBOX (0x1 << 12) /* Mailbox update */ #define NFP_NET_CFG_UPDATE_VF (0x1 << 13) /* VF settings change */ #define NFP_NET_CFG_UPDATE_CRYPTO (0x1 << 14) /* Crypto on/off */ +#define NFP_NET_CFG_UPDATE_VF_VIRTIO (0x1 << 16) /* VF virtio enable change */ #define NFP_NET_CFG_UPDATE_ERR (0x1 << 31) /* A error occurred */ #define NFP_NET_CFG_TXRS_ENABLE 0x0008 #define NFP_NET_CFG_RXRS_ENABLE 0x0010 @@ -270,7 +271,9 @@ #define NFP_NET_CFG_CTRL_MCAST_FILTER (0x1 << 2) /* Multicast Filter */ #define NFP_NET_CFG_CTRL_FREELIST_EN (0x1 << 6) /* Freelist enable flag bit */ #define NFP_NET_CFG_CTRL_FLOW_STEER (0x1 << 8) /* Flow steering */ +#define NFP_NET_CFG_CTRL_VIRTIO (0x1 << 10) /* Virtio service flag */ #define NFP_NET_CFG_CTRL_USO (0x1 << 16) /* UDP segmentation offload */ +#define NFP_NET_CFG_CTRL_ENABLE_VNET (0x1 << 18) /* vDPA networking device enable */ #define NFP_NET_CFG_CAP_WORD1 0x00a4 diff --git a/drivers/net/ethernet/netronome/nfp/nfp_net_main.c b/drivers/net/ethernet/netronome/nfp/nfp_net_main.c index cbe4972ba104..43bc9d40f755 100644 --- a/drivers/net/ethernet/netronome/nfp/nfp_net_main.c +++ b/drivers/net/ethernet/netronome/nfp/nfp_net_main.c @@ -684,6 +684,16 @@ int nfp_net_refresh_eth_port(struct nfp_port *port) return ret; } +int nfp_net_update_enable_vnet(struct nfp_pf *pf, bool enable_vnet) +{ + struct nfp_net *nn; + + nn = list_first_entry(&pf->vnics, struct nfp_net, vnic_list); + nn_writel(nn, NFP_NET_CFG_CTRL_WORD1, enable_vnet ? NFP_NET_CFG_CTRL_ENABLE_VNET : 0); + + return nfp_net_reconfig(nn, NFP_NET_CFG_UPDATE_VF_VIRTIO); +} + /* * PCI device functions */ -- 2.34.1