From: Michal Swiatkowski <michal.swiatkowski@linux.intel.com>
To: netdev@vger.kernel.org, davem@davemloft.net, kuba@kernel.org,
pabeni@redhat.com, edumazet@google.com
Cc: intel-wired-lan@lists.osuosl.org, jiri@nvidia.com,
anthony.l.nguyen@intel.com, alexandr.lobakin@intel.com,
sridhar.samudrala@intel.com, wojciech.drewek@intel.com,
lukasz.czapnik@intel.com, shiraz.saleem@intel.com,
jesse.brandeburg@intel.com, mustafa.ismail@intel.com,
przemyslaw.kitszel@intel.com, piotr.raczynski@intel.com,
jacob.e.keller@intel.com, david.m.ertman@intel.com,
leszek.kaliszczuk@intel.com,
Michal Kubiak <michal.kubiak@intel.com>,
Michal Swiatkowski <michal.swiatkowski@linux.intel.com>
Subject: [PATCH net-next 13/13] devlink, ice: add MSIX vectors as devlink resource
Date: Mon, 14 Nov 2022 13:57:55 +0100 [thread overview]
Message-ID: <20221114125755.13659-14-michal.swiatkowski@linux.intel.com> (raw)
In-Reply-To: <20221114125755.13659-1-michal.swiatkowski@linux.intel.com>
From: Michal Kubiak <michal.kubiak@intel.com>
Implement devlink resource to control how many MSI-X vectors are
used for eth, VF and RDMA. Show misc MSI-X as read only.
This is first approach to control the mix of resources managed
by ice driver. This commit registers number of available MSI-X
as devlink resource and also add specific resources for eth, vf and RDMA.
Also, make those resources generic.
$ devlink resource show pci/0000:31:00.0
name msix size 1024 occ 172 unit entry dpipe_tables none
resources:
name msix_misc size 4 unit entry dpipe_tables none
name msix_eth size 92 occ 92 unit entry size_min 1 size_max
128 size_gran 1 dpipe_tables none
name msix_vf size 128 occ 0 unit entry size_min 0 size_max
1020 size_gran 1 dpipe_tables none
name msix_rdma size 76 occ 76 unit entry size_min 0 size_max
132 size_gran 1 dpipe_tables none
example commands:
$ devlink resource set pci/0000:31:00.0 path msix/msix_eth size 16
$ devlink resource set pci/0000:31:00.0 path msix/msix_vf size 512
$ devlink dev reload pci/0000:31:00.0
Signed-off-by: Michal Kubiak <michal.kubiak@intel.com>
Co-developed-by: Piotr Raczynski <piotr.raczynski@intel.com>
Signed-off-by: Piotr Raczynski <piotr.raczynski@intel.com>
Co-developed-by: Michal Swiatkowski <michal.swiatkowski@linux.intel.com>
Signed-off-by: Michal Swiatkowski <michal.swiatkowski@linux.intel.com>
---
.../networking/devlink/devlink-resource.rst | 10 ++
drivers/net/ethernet/intel/ice/ice_devlink.c | 160 ++++++++++++++++++
drivers/net/ethernet/intel/ice/ice_devlink.h | 2 +
drivers/net/ethernet/intel/ice/ice_main.c | 5 +
drivers/net/ethernet/intel/ice/ice_sriov.c | 3 +-
include/net/devlink.h | 14 ++
6 files changed, 192 insertions(+), 2 deletions(-)
diff --git a/Documentation/networking/devlink/devlink-resource.rst b/Documentation/networking/devlink/devlink-resource.rst
index 3d5ae51e65a2..dcd3d124ebfb 100644
--- a/Documentation/networking/devlink/devlink-resource.rst
+++ b/Documentation/networking/devlink/devlink-resource.rst
@@ -36,6 +36,16 @@ device drivers and their description must be added to the following table:
- Description
* - ``physical_ports``
- A limited capacity of physical ports that the switch ASIC can support
+ * - ``msix``
+ - Amount of MSIX interrupt vectors that the device can support
+ * - ``msix_misc``
+ - Amount of MSIX interrupt vectors that can be used for miscellaneous purposes
+ * - ``msix_eth``
+ - Amount of MSIX interrupt vectors that can be used by ethernet
+ * - ``msix_vf``
+ - Amount of MSIX interrupt vectors that can be used by virtual functions
+ * - ``msix_rdma``
+ - Amount of MSIX interrupt vectors that can be used by RDMA
example usage
-------------
diff --git a/drivers/net/ethernet/intel/ice/ice_devlink.c b/drivers/net/ethernet/intel/ice/ice_devlink.c
index 8f73c4008a56..00f01cf84812 100644
--- a/drivers/net/ethernet/intel/ice/ice_devlink.c
+++ b/drivers/net/ethernet/intel/ice/ice_devlink.c
@@ -496,6 +496,27 @@ ice_devlink_reload_empr_finish(struct ice_pf *pf,
return 0;
}
+static void ice_devlink_read_resources_size(struct ice_pf *pf)
+{
+ struct devlink *devlink = priv_to_devlink(pf);
+ u64 size_new;
+
+ devl_resource_size_get(devlink,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX_ETH,
+ &size_new);
+ pf->req_msix.eth = size_new;
+
+ devl_resource_size_get(devlink,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX_VF,
+ &size_new);
+ pf->msix.vf = size_new;
+
+ devl_resource_size_get(devlink,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX_RDMA,
+ &size_new);
+ pf->req_msix.rdma = size_new;
+}
+
/**
* ice_devlink_port_opt_speed_str - convert speed to a string
* @speed: speed value
@@ -761,6 +782,7 @@ ice_devlink_reload_up(struct devlink *devlink,
switch (action) {
case DEVLINK_RELOAD_ACTION_DRIVER_REINIT:
*actions_performed = BIT(DEVLINK_RELOAD_ACTION_DRIVER_REINIT);
+ ice_devlink_read_resources_size(pf);
return ice_load(pf);
case DEVLINK_RELOAD_ACTION_FW_ACTIVATE:
*actions_performed = BIT(DEVLINK_RELOAD_ACTION_FW_ACTIVATE);
@@ -955,6 +977,144 @@ void ice_devlink_unregister(struct ice_pf *pf)
devlink_unregister(priv_to_devlink(pf));
}
+static u64 ice_devlink_res_msix_pf_occ_get(void *priv)
+{
+ struct ice_pf *pf = priv;
+
+ return pf->msix.eth;
+}
+
+static u64 ice_devlink_res_msix_vf_occ_get(void *priv)
+{
+ struct ice_pf *pf = priv;
+
+ return pf->vfs.num_msix_per * ice_get_num_vfs(pf);
+}
+
+static u64 ice_devlink_res_msix_rdma_occ_get(void *priv)
+{
+ struct ice_pf *pf = priv;
+
+ return pf->msix.rdma;
+}
+
+static u64 ice_devlink_res_msix_occ_get(void *priv)
+{
+ struct ice_pf *pf = priv;
+
+ return ice_devlink_res_msix_pf_occ_get(priv) +
+ ice_devlink_res_msix_vf_occ_get(priv) +
+ ice_devlink_res_msix_rdma_occ_get(priv) +
+ pf->msix.misc;
+}
+
+int ice_devlink_register_resources(struct ice_pf *pf)
+{
+ struct devlink *devlink = priv_to_devlink(pf);
+ struct devlink_resource_size_params params;
+ struct device *dev = ice_pf_to_dev(pf);
+ struct ice_msix *req = &pf->req_msix;
+ int err, max_pf_msix, max_rdma_msix;
+ const char *res_name;
+
+ max_pf_msix = min_t(int, num_online_cpus(), ICE_MAX_LG_RSS_QS);
+ max_rdma_msix = max_pf_msix + ICE_RDMA_NUM_AEQ_MSIX;
+
+ devlink_resource_size_params_init(¶ms, req->all, req->all, 1,
+ DEVLINK_RESOURCE_UNIT_ENTRY);
+ res_name = DEVLINK_RESOURCE_GENERIC_NAME_MSIX;
+ err = devlink_resource_register(devlink, res_name, req->all,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX,
+ DEVLINK_RESOURCE_ID_PARENT_TOP,
+ ¶ms);
+ if (err)
+ goto res_create_err;
+
+ devlink_resource_size_params_init(¶ms, req->misc, req->misc, 1,
+ DEVLINK_RESOURCE_UNIT_ENTRY);
+ res_name = DEVLINK_RESOURCE_GENERIC_NAME_MSIX_MISC;
+ err = devlink_resource_register(devlink, res_name, req->misc,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX_MISC,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX,
+ ¶ms);
+ if (err)
+ goto res_create_err;
+
+ devlink_resource_size_params_init(¶ms, ICE_MIN_LAN_TXRX_MSIX,
+ max_pf_msix, 1,
+ DEVLINK_RESOURCE_UNIT_ENTRY);
+ res_name = DEVLINK_RESOURCE_GENERIC_NAME_MSIX_ETH;
+ err = devlink_resource_register(devlink, res_name, req->eth,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX_ETH,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX,
+ ¶ms);
+ if (err)
+ goto res_create_err;
+
+ devlink_resource_size_params_init(¶ms, 0, req->all - req->misc, 1,
+ DEVLINK_RESOURCE_UNIT_ENTRY);
+
+ res_name = DEVLINK_RESOURCE_GENERIC_NAME_MSIX_VF;
+ err = devlink_resource_register(devlink, res_name, req->vf,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX_VF,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX,
+ ¶ms);
+ if (err)
+ goto res_create_err;
+
+ devlink_resource_size_params_init(¶ms, ICE_MIN_RDMA_MSIX,
+ max_rdma_msix, 1,
+ DEVLINK_RESOURCE_UNIT_ENTRY);
+
+ res_name = DEVLINK_RESOURCE_GENERIC_NAME_MSIX_RDMA;
+ err = devlink_resource_register(devlink, res_name, req->rdma,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX_RDMA,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX,
+ ¶ms);
+ if (err)
+ goto res_create_err;
+
+ devlink_resource_occ_get_register(devlink,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX,
+ ice_devlink_res_msix_occ_get, pf);
+
+ devlink_resource_occ_get_register(devlink,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX_ETH,
+ ice_devlink_res_msix_pf_occ_get, pf);
+
+ devlink_resource_occ_get_register(devlink,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX_VF,
+ ice_devlink_res_msix_vf_occ_get, pf);
+
+ devlink_resource_occ_get_register(devlink,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX_RDMA,
+ ice_devlink_res_msix_rdma_occ_get,
+ pf);
+ return 0;
+
+res_create_err:
+ dev_err(dev, "Failed to register devlink resource: %s error: %pe\n",
+ res_name, ERR_PTR(err));
+ devlink_resources_unregister(devlink);
+
+ return err;
+}
+
+void ice_devlink_unregister_resources(struct ice_pf *pf)
+{
+ struct devlink *devlink = priv_to_devlink(pf);
+
+ devlink_resource_occ_get_unregister(devlink,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX);
+ devlink_resource_occ_get_unregister(devlink,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX_ETH);
+ devlink_resource_occ_get_unregister(devlink,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX_VF);
+ devlink_resource_occ_get_unregister(devlink,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX_RDMA);
+ devlink_resources_unregister(devlink);
+}
+
/**
* ice_devlink_set_switch_id - Set unique switch id based on pci dsn
* @pf: the PF to create a devlink port for
diff --git a/drivers/net/ethernet/intel/ice/ice_devlink.h b/drivers/net/ethernet/intel/ice/ice_devlink.h
index fe006d9946f8..35233ea087fc 100644
--- a/drivers/net/ethernet/intel/ice/ice_devlink.h
+++ b/drivers/net/ethernet/intel/ice/ice_devlink.h
@@ -10,6 +10,8 @@ void ice_devlink_register(struct ice_pf *pf);
void ice_devlink_unregister(struct ice_pf *pf);
int ice_devlink_register_params(struct ice_pf *pf);
void ice_devlink_unregister_params(struct ice_pf *pf);
+int ice_devlink_register_resources(struct ice_pf *pf);
+void ice_devlink_unregister_resources(struct ice_pf *pf);
int ice_devlink_create_pf_port(struct ice_pf *pf);
void ice_devlink_destroy_pf_port(struct ice_pf *pf);
int ice_devlink_create_vf_port(struct ice_vf *vf);
diff --git a/drivers/net/ethernet/intel/ice/ice_main.c b/drivers/net/ethernet/intel/ice/ice_main.c
index a4c283bc8da0..ba6f93e7e640 100644
--- a/drivers/net/ethernet/intel/ice/ice_main.c
+++ b/drivers/net/ethernet/intel/ice/ice_main.c
@@ -4970,6 +4970,10 @@ static int ice_init_devlink(struct ice_pf *pf)
if (err)
return err;
+ err = ice_devlink_register_resources(pf);
+ if (err)
+ return err;
+
ice_devlink_init_regions(pf);
ice_devlink_register(pf);
@@ -4980,6 +4984,7 @@ static void ice_deinit_devlink(struct ice_pf *pf)
{
ice_devlink_unregister(pf);
ice_devlink_destroy_regions(pf);
+ ice_devlink_unregister_resources(pf);
ice_devlink_unregister_params(pf);
}
diff --git a/drivers/net/ethernet/intel/ice/ice_sriov.c b/drivers/net/ethernet/intel/ice/ice_sriov.c
index 3ba1408c56a9..d42df5983494 100644
--- a/drivers/net/ethernet/intel/ice/ice_sriov.c
+++ b/drivers/net/ethernet/intel/ice/ice_sriov.c
@@ -518,8 +518,7 @@ static int ice_set_per_vf_res(struct ice_pf *pf, u16 num_vfs)
return -ENOSPC;
/* determine MSI-X resources per VF */
- msix_avail_for_sriov = pf->hw.func_caps.common_cap.num_msix_vectors -
- pf->irq_tracker->num_entries;
+ msix_avail_for_sriov = pf->msix.vf;
msix_avail_per_vf = msix_avail_for_sriov / num_vfs;
if (msix_avail_per_vf >= ICE_NUM_VF_MSIX_MED) {
num_msix_per_vf = ICE_NUM_VF_MSIX_MED;
diff --git a/include/net/devlink.h b/include/net/devlink.h
index 611a23a3deb2..a041c50f72c6 100644
--- a/include/net/devlink.h
+++ b/include/net/devlink.h
@@ -409,7 +409,21 @@ typedef u64 devlink_resource_occ_get_t(void *priv);
#define DEVLINK_RESOURCE_ID_PARENT_TOP 0
+enum devlink_msix_resource_id {
+ /* generic resource for MSIX */
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX = 1,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX_MISC,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX_ETH,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX_VF,
+ DEVLINK_RESOURCE_GENERIC_ID_MSIX_RDMA,
+};
+
#define DEVLINK_RESOURCE_GENERIC_NAME_PORTS "physical_ports"
+#define DEVLINK_RESOURCE_GENERIC_NAME_MSIX "msix"
+#define DEVLINK_RESOURCE_GENERIC_NAME_MSIX_MISC "msix_misc"
+#define DEVLINK_RESOURCE_GENERIC_NAME_MSIX_ETH "msix_eth"
+#define DEVLINK_RESOURCE_GENERIC_NAME_MSIX_VF "msix_vf"
+#define DEVLINK_RESOURCE_GENERIC_NAME_MSIX_RDMA "msix_rdma"
#define __DEVLINK_PARAM_MAX_STRING_VALUE 32
enum devlink_param_type {
--
2.36.1
next prev parent reply other threads:[~2022-11-14 13:13 UTC|newest]
Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-11-14 12:57 [PATCH net-next 00/13] resource management using devlink reload Michal Swiatkowski
2022-11-14 12:57 ` [PATCH net-next 01/13] ice: move RDMA init to ice_idc.c Michal Swiatkowski
2022-11-14 12:57 ` [PATCH net-next 02/13] ice: alloc id for RDMA using xa_array Michal Swiatkowski
2022-11-14 12:57 ` [PATCH net-next 03/13] ice: cleanup in VSI config/deconfig code Michal Swiatkowski
2022-11-14 12:57 ` [PATCH net-next 04/13] ice: split ice_vsi_setup into smaller functions Michal Swiatkowski
2022-11-15 5:08 ` Jakub Kicinski
2022-11-15 6:49 ` Michal Swiatkowski
2022-11-14 12:57 ` [PATCH net-next 05/13] ice: stop hard coding the ICE_VSI_CTRL location Michal Swiatkowski
2022-11-14 12:57 ` [PATCH net-next 06/13] ice: split probe into smaller functions Michal Swiatkowski
2022-11-14 12:57 ` [PATCH net-next 07/13] ice: sync netdev filters after clearing VSI Michal Swiatkowski
2022-11-14 12:57 ` [PATCH net-next 08/13] ice: move VSI delete outside deconfig Michal Swiatkowski
2022-11-14 12:57 ` [PATCH net-next 09/13] ice: update VSI instead of init in some case Michal Swiatkowski
2022-11-14 12:57 ` [PATCH net-next 10/13] ice: implement devlink reinit action Michal Swiatkowski
2022-11-14 12:57 ` [PATCH net-next 11/13] ice: introduce eswitch capable flag Michal Swiatkowski
2022-11-14 12:57 ` [PATCH net-next 12/13] ice, irdma: prepare reservation of MSI-X to reload Michal Swiatkowski
2022-11-15 5:08 ` Jakub Kicinski
2022-11-15 6:49 ` Michal Swiatkowski
2022-11-14 12:57 ` Michal Swiatkowski [this message]
2022-11-14 15:28 ` [PATCH net-next 13/13] devlink, ice: add MSIX vectors as devlink resource Jiri Pirko
2022-11-14 16:03 ` Piotr Raczynski
2022-11-15 6:56 ` Michal Swiatkowski
2022-11-15 12:08 ` Jiri Pirko
2022-11-14 13:23 ` [PATCH net-next 00/13] resource management using devlink reload Leon Romanovsky
2022-11-14 15:31 ` Samudrala, Sridhar
2022-11-14 16:58 ` Keller, Jacob E
2022-11-14 17:09 ` Leon Romanovsky
2022-11-15 7:00 ` Michal Swiatkowski
2022-11-14 17:07 ` Leon Romanovsky
2022-11-15 7:12 ` Michal Swiatkowski
2022-11-15 8:11 ` Leon Romanovsky
2022-11-15 9:04 ` Michal Swiatkowski
2022-11-15 9:32 ` Leon Romanovsky
2022-11-15 10:16 ` Michal Swiatkowski
2022-11-15 12:12 ` Leon Romanovsky
2022-11-15 14:02 ` Michal Swiatkowski
2022-11-15 17:57 ` Leon Romanovsky
2022-11-16 1:59 ` Samudrala, Sridhar
2022-11-16 6:04 ` Leon Romanovsky
2022-11-16 12:04 ` Michal Swiatkowski
2022-11-16 17:59 ` Leon Romanovsky
2022-11-17 11:10 ` Michal Swiatkowski
2022-11-17 11:45 ` Leon Romanovsky
2022-11-17 13:39 ` Michal Swiatkowski
2022-11-17 17:38 ` Leon Romanovsky
2022-11-18 3:36 ` Jakub Kicinski
2022-11-18 6:20 ` Leon Romanovsky
2022-11-18 14:23 ` Saleem, Shiraz
2022-11-18 17:31 ` Leon Romanovsky
2022-11-20 22:24 ` Samudrala, Sridhar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20221114125755.13659-14-michal.swiatkowski@linux.intel.com \
--to=michal.swiatkowski@linux.intel.com \
--cc=alexandr.lobakin@intel.com \
--cc=anthony.l.nguyen@intel.com \
--cc=davem@davemloft.net \
--cc=david.m.ertman@intel.com \
--cc=edumazet@google.com \
--cc=intel-wired-lan@lists.osuosl.org \
--cc=jacob.e.keller@intel.com \
--cc=jesse.brandeburg@intel.com \
--cc=jiri@nvidia.com \
--cc=kuba@kernel.org \
--cc=leszek.kaliszczuk@intel.com \
--cc=lukasz.czapnik@intel.com \
--cc=michal.kubiak@intel.com \
--cc=mustafa.ismail@intel.com \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=piotr.raczynski@intel.com \
--cc=przemyslaw.kitszel@intel.com \
--cc=shiraz.saleem@intel.com \
--cc=sridhar.samudrala@intel.com \
--cc=wojciech.drewek@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).