From: "Renyong Wan" <wanry@yunsilicon.com>
To: <dev@dpdk.org>
Cc: <thomas@monjalon.net>, <stephen@networkplumber.org>,
<qianr@yunsilicon.com>, <nana@yunsilicon.com>,
<zhangxx@yunsilicon.com>, <xudw@yunsilicon.com>,
<jacky@yunsilicon.com>, <weihg@yunsilicon.com>,
<zhenghy@yunsilicon.com>
Subject: [PATCH 05/14] net/xsc: add link status support
Date: Fri, 29 Aug 2025 16:24:17 +0800 [thread overview]
Message-ID: <20250829082415.24369-6-wanry@yunsilicon.com> (raw)
In-Reply-To: <20250829082406.24369-1-wanry@yunsilicon.com>
Add support for getting and setting the link state.
Signed-off-by: Rong Qian <qianr@yunsilicon.com>
Signed-off-by: Renyong Wan <wanry@yunsilicon.com>
---
doc/guides/nics/features/xsc.ini | 1 +
drivers/net/xsc/xsc_cmd.h | 73 +++++++++++++++++++-----
drivers/net/xsc/xsc_dev.c | 21 ++-----
drivers/net/xsc/xsc_dev.h | 11 ++--
drivers/net/xsc/xsc_ethdev.c | 34 +++++++----
drivers/net/xsc/xsc_vfio.c | 96 ++++++++++++++++++++++++++++++++
6 files changed, 189 insertions(+), 47 deletions(-)
diff --git a/doc/guides/nics/features/xsc.ini b/doc/guides/nics/features/xsc.ini
index dfbfc88dbc..0d98296973 100644
--- a/doc/guides/nics/features/xsc.ini
+++ b/doc/guides/nics/features/xsc.ini
@@ -4,6 +4,7 @@
; Refer to default.ini for the full list of available PMD features.
;
[Features]
+Link status = Y
MTU update = Y
TSO = Y
Promiscuous mode = Y
diff --git a/drivers/net/xsc/xsc_cmd.h b/drivers/net/xsc/xsc_cmd.h
index 82244d4011..207f4456d5 100644
--- a/drivers/net/xsc/xsc_cmd.h
+++ b/drivers/net/xsc/xsc_cmd.h
@@ -15,20 +15,22 @@
#define XSC_CMD_QUERY_HCA_CAP_V1 1
enum xsc_cmd_opcode {
- XSC_CMD_OP_QUERY_HCA_CAP = 0x100,
- XSC_CMD_OP_CREATE_CQ = 0x400,
- XSC_CMD_OP_DESTROY_CQ = 0x401,
- XSC_CMD_OP_CREATE_QP = 0x500,
- XSC_CMD_OP_DESTROY_QP = 0x501,
- XSC_CMD_OP_RTR2RTS_QP = 0x504,
- XSC_CMD_OP_QP_2RST = 0x50A,
- XSC_CMD_OP_CREATE_MULTI_QP = 0x515,
- XSC_CMD_OP_ACCESS_REG = 0x805,
- XSC_CMD_OP_MODIFY_NIC_HCA = 0x812,
- XSC_CMD_OP_MODIFY_RAW_QP = 0x81f,
- XSC_CMD_OP_EXEC_NP = 0x900,
- XSC_CMD_OP_SET_MTU = 0x1100,
- XSC_CMD_OP_QUERY_ETH_MAC = 0X1101,
+ XSC_CMD_OP_QUERY_HCA_CAP = 0x100,
+ XSC_CMD_OP_CREATE_CQ = 0x400,
+ XSC_CMD_OP_DESTROY_CQ = 0x401,
+ XSC_CMD_OP_CREATE_QP = 0x500,
+ XSC_CMD_OP_DESTROY_QP = 0x501,
+ XSC_CMD_OP_RTR2RTS_QP = 0x504,
+ XSC_CMD_OP_QP_2RST = 0x50A,
+ XSC_CMD_OP_CREATE_MULTI_QP = 0x515,
+ XSC_CMD_OP_ACCESS_REG = 0x805,
+ XSC_CMD_OP_MODIFY_NIC_HCA = 0x812,
+ XSC_CMD_OP_MODIFY_RAW_QP = 0x81f,
+ XSC_CMD_OP_QUERY_LINK_INFO = 0x832,
+ XSC_CMD_OP_EXEC_NP = 0x900,
+ XSC_CMD_OP_SET_MTU = 0x1100,
+ XSC_CMD_OP_QUERY_ETH_MAC = 0X1101,
+ XSC_CMD_OP_SET_PORT_ADMIN_STATUS = 0x1801,
XSC_CMD_OP_MAX
};
@@ -399,4 +401,47 @@ struct xsc_cmd_access_reg_mbox_out {
rte_be32_t data[];
};
+struct xsc_cmd_set_port_admin_status_mbox_in {
+ struct xsc_cmd_inbox_hdr hdr;
+ uint16_t admin_status;
+};
+
+struct xsc_cmd_set_port_admin_status_mbox_out {
+ struct xsc_cmd_outbox_hdr hdr;
+ uint32_t status;
+};
+
+struct xsc_cmd_query_linkinfo_mbox_in {
+ struct xsc_cmd_inbox_hdr hdr;
+};
+
+struct xsc_cmd_linkinfo {
+ uint8_t status; /*link status: 0-down, 1-up */
+ uint8_t port;
+ uint8_t duplex;
+ uint8_t autoneg;
+ rte_be32_t linkspeed;
+ rte_be64_t supported;
+ rte_be64_t advertising;
+ rte_be64_t supported_fec;
+ rte_be64_t advertised_fec;
+ rte_be64_t supported_speed[2];
+ rte_be64_t advertising_speed[2];
+};
+
+struct xsc_cmd_query_linkinfo_mbox_out {
+ struct xsc_cmd_outbox_hdr hdr;
+ struct xsc_cmd_linkinfo ctx;
+};
+
+struct xsc_cmd_modify_linkinfo_mbox_in {
+ struct xsc_cmd_inbox_hdr hdr;
+ struct xsc_cmd_linkinfo ctx;
+};
+
+struct xsc_cmd_modify_linkinfo_mbox_out {
+ struct xsc_cmd_outbox_hdr hdr;
+ uint32_t status;
+};
+
#endif /* _XSC_CMD_H_ */
diff --git a/drivers/net/xsc/xsc_dev.c b/drivers/net/xsc/xsc_dev.c
index 2c55306d35..28581459b9 100644
--- a/drivers/net/xsc/xsc_dev.c
+++ b/drivers/net/xsc/xsc_dev.c
@@ -64,30 +64,21 @@ xsc_dev_mailbox_exec(struct xsc_dev *xdev, void *data_in,
}
int
-xsc_dev_set_link_up(struct xsc_dev *xdev)
+xsc_dev_link_status_set(struct xsc_dev *xdev, uint16_t status)
{
- if (xdev->dev_ops->set_link_up == NULL)
+ if (xdev->dev_ops->link_status_set == NULL)
return -ENOTSUP;
- return xdev->dev_ops->set_link_up(xdev);
+ return xdev->dev_ops->link_status_set(xdev, status);
}
int
-xsc_dev_set_link_down(struct xsc_dev *xdev)
+xsc_dev_link_get(struct xsc_dev *xdev, struct rte_eth_link *link)
{
- if (xdev->dev_ops->set_link_down == NULL)
+ if (xdev->dev_ops->link_get == NULL)
return -ENOTSUP;
- return xdev->dev_ops->set_link_down(xdev);
-}
-
-int
-xsc_dev_link_update(struct xsc_dev *xdev, uint8_t funcid_type, int wait_to_complete)
-{
- if (xdev->dev_ops->link_update == NULL)
- return -ENOTSUP;
-
- return xdev->dev_ops->link_update(xdev, funcid_type, wait_to_complete);
+ return xdev->dev_ops->link_get(xdev, link);
}
int
diff --git a/drivers/net/xsc/xsc_dev.h b/drivers/net/xsc/xsc_dev.h
index 5cf19aaf3d..e543b6892d 100644
--- a/drivers/net/xsc/xsc_dev.h
+++ b/drivers/net/xsc/xsc_dev.h
@@ -144,7 +144,6 @@ struct xsc_dev {
const struct xsc_dev_ops *dev_ops;
struct xsc_devargs devargs;
struct xsc_hwinfo hwinfo;
- struct rte_eth_link pf_dev_link;
uint32_t link_speed_capa;
int vfos_logical_in_port;
int vfrep_offset;
@@ -199,9 +198,8 @@ struct xsc_dev_ops {
int (*dev_init)(struct xsc_dev *xdev);
int (*dev_close)(struct xsc_dev *xdev);
int (*get_mac)(struct xsc_dev *xdev, uint8_t *mac);
- int (*set_link_up)(struct xsc_dev *xdev);
- int (*set_link_down)(struct xsc_dev *xdev);
- int (*link_update)(struct xsc_dev *xdev, uint8_t funcid_type, int wait_to_complete);
+ int (*link_status_set)(struct xsc_dev *xdev, uint16_t status);
+ int (*link_get)(struct xsc_dev *xdev, struct rte_eth_link *link);
int (*set_mtu)(struct xsc_dev *xdev, uint16_t mtu);
int (*destroy_qp)(void *qp);
int (*destroy_cq)(void *cq);
@@ -222,9 +220,8 @@ struct xsc_dev_ops {
int xsc_dev_mailbox_exec(struct xsc_dev *xdev, void *data_in,
int in_len, void *data_out, int out_len);
void xsc_dev_ops_register(struct xsc_dev_ops *new_ops);
-int xsc_dev_set_link_up(struct xsc_dev *xdev);
-int xsc_dev_set_link_down(struct xsc_dev *xde);
-int xsc_dev_link_update(struct xsc_dev *xdev, uint8_t funcid_type, int wait_to_complete);
+int xsc_dev_link_status_set(struct xsc_dev *xdev, uint16_t status);
+int xsc_dev_link_get(struct xsc_dev *xdev, struct rte_eth_link *link);
int xsc_dev_destroy_qp(struct xsc_dev *xdev, void *qp);
int xsc_dev_destroy_cq(struct xsc_dev *xdev, void *cq);
int xsc_dev_modify_qp_status(struct xsc_dev *xdev, uint32_t qpn, int num, int opcode);
diff --git a/drivers/net/xsc/xsc_ethdev.c b/drivers/net/xsc/xsc_ethdev.c
index efe1ad72f6..3574da91eb 100644
--- a/drivers/net/xsc/xsc_ethdev.c
+++ b/drivers/net/xsc/xsc_ethdev.c
@@ -389,7 +389,9 @@ xsc_ethdev_set_link_up(struct rte_eth_dev *dev)
struct xsc_ethdev_priv *priv = TO_XSC_ETHDEV_PRIV(dev);
struct xsc_dev *xdev = priv->xdev;
- return xsc_dev_set_link_up(xdev);
+ if (xsc_dev_is_vf(xdev))
+ return -ENOTSUP;
+ return xsc_dev_link_status_set(xdev, RTE_ETH_LINK_UP);
}
static int
@@ -398,24 +400,26 @@ xsc_ethdev_set_link_down(struct rte_eth_dev *dev)
struct xsc_ethdev_priv *priv = TO_XSC_ETHDEV_PRIV(dev);
struct xsc_dev *xdev = priv->xdev;
- return xsc_dev_set_link_down(xdev);
+ if (xsc_dev_is_vf(xdev))
+ return -ENOTSUP;
+ return xsc_dev_link_status_set(xdev, RTE_ETH_LINK_DOWN);
}
static int
-xsc_ethdev_link_update(struct rte_eth_dev *dev,
- int wait_to_complete)
+xsc_ethdev_link_update(struct rte_eth_dev *dev, int wait_to_complete)
{
struct xsc_ethdev_priv *priv = TO_XSC_ETHDEV_PRIV(dev);
struct xsc_dev *xdev = priv->xdev;
+ struct rte_eth_link link = { };
int ret = 0;
- ret = xsc_dev_link_update(xdev, priv->funcid_type, wait_to_complete);
- if (ret == 0) {
- dev->data->dev_link = xdev->pf_dev_link;
- dev->data->dev_link.link_autoneg = !(dev->data->dev_conf.link_speeds &
- RTE_ETH_LINK_SPEED_FIXED);
- }
- return ret;
+ RTE_SET_USED(wait_to_complete);
+ ret = xsc_dev_link_get(xdev, &link);
+ if (ret != 0)
+ return ret;
+ rte_eth_linkstatus_set(dev, &link);
+
+ return 0;
}
static uint64_t
@@ -980,6 +984,14 @@ xsc_ethdev_init_representors(struct rte_eth_dev *eth_dev)
}
}
+ if (!xsc_dev_is_vf(priv->xdev)) {
+ ret = xsc_ethdev_set_link_up(eth_dev);
+ if (ret != 0) {
+ PMD_DRV_LOG(ERR, "Failed to set port %u link up", eth_dev->data->port_id);
+ goto destroy_reprs;
+ }
+ }
+
return 0;
destroy_reprs:
diff --git a/drivers/net/xsc/xsc_vfio.c b/drivers/net/xsc/xsc_vfio.c
index f61cf92569..cc3660e259 100644
--- a/drivers/net/xsc/xsc_vfio.c
+++ b/drivers/net/xsc/xsc_vfio.c
@@ -31,6 +31,21 @@ enum xsc_cq_type {
XSC_CQ_TYPE_VIRTIO = 1,
};
+enum xsc_speed_mode {
+ XSC_MODULE_SPEED_UNKNOWN,
+ XSC_MODULE_SPEED_10G,
+ XSC_MODULE_SPEED_25G,
+ XSC_MODULE_SPEED_40G_R4,
+ XSC_MODULE_SPEED_50G_R,
+ XSC_MODULE_SPEED_50G_R2,
+ XSC_MODULE_SPEED_100G_R2,
+ XSC_MODULE_SPEED_100G_R4,
+ XSC_MODULE_SPEED_200G_R4,
+ XSC_MODULE_SPEED_200G_R8,
+ XSC_MODULE_SPEED_400G_R8,
+ XSC_MODULE_SPEED_MAX,
+};
+
struct xsc_vfio_cq {
const struct rte_memzone *mz;
struct xsc_dev *xdev;
@@ -43,6 +58,20 @@ struct xsc_vfio_qp {
uint32_t qpn;
};
+static const uint32_t xsc_link_speed[] = {
+ [XSC_MODULE_SPEED_UNKNOWN] = RTE_ETH_SPEED_NUM_UNKNOWN,
+ [XSC_MODULE_SPEED_10G] = RTE_ETH_SPEED_NUM_10G,
+ [XSC_MODULE_SPEED_25G] = RTE_ETH_SPEED_NUM_25G,
+ [XSC_MODULE_SPEED_40G_R4] = RTE_ETH_SPEED_NUM_40G,
+ [XSC_MODULE_SPEED_50G_R] = RTE_ETH_SPEED_NUM_50G,
+ [XSC_MODULE_SPEED_50G_R2] = RTE_ETH_SPEED_NUM_50G,
+ [XSC_MODULE_SPEED_100G_R2] = RTE_ETH_SPEED_NUM_100G,
+ [XSC_MODULE_SPEED_100G_R4] = RTE_ETH_SPEED_NUM_100G,
+ [XSC_MODULE_SPEED_200G_R4] = RTE_ETH_SPEED_NUM_200G,
+ [XSC_MODULE_SPEED_200G_R8] = RTE_ETH_SPEED_NUM_200G,
+ [XSC_MODULE_SPEED_400G_R8] = RTE_ETH_SPEED_NUM_400G,
+};
+
static void
xsc_vfio_pcie_no_init(struct xsc_hwinfo *hwinfo)
{
@@ -333,6 +362,71 @@ xsc_vfio_get_mac(struct xsc_dev *xdev, uint8_t *mac)
return 0;
}
+static int
+xsc_vfio_modify_link_status(struct xsc_dev *xdev, uint16_t status)
+{
+ struct xsc_cmd_set_port_admin_status_mbox_in in = { };
+ struct xsc_cmd_set_port_admin_status_mbox_out out = { };
+ int ret = 0;
+
+ in.hdr.opcode = rte_cpu_to_be_16(XSC_CMD_OP_SET_PORT_ADMIN_STATUS);
+ in.admin_status = rte_cpu_to_be_16(status);
+
+ ret = xsc_vfio_mbox_exec(xdev, &in, sizeof(in), &out, sizeof(out));
+ if (ret != 0 || out.hdr.status != 0) {
+ PMD_DRV_LOG(ERR, "Failed to set link status, ret=%d, status=%d",
+ ret, out.hdr.status);
+ return -ENOEXEC;
+ }
+
+ return ret;
+}
+
+static int
+xsc_vfio_link_status_set(struct xsc_dev *xdev, uint16_t status)
+{
+ if (status != RTE_ETH_LINK_UP && status != RTE_ETH_LINK_DOWN)
+ return -EINVAL;
+
+ return xsc_vfio_modify_link_status(xdev, status);
+}
+
+static uint32_t
+xsc_vfio_link_speed_translate(uint32_t mode)
+{
+ if (mode >= XSC_MODULE_SPEED_MAX)
+ return RTE_ETH_SPEED_NUM_NONE;
+
+ return xsc_link_speed[mode];
+}
+
+static int
+xsc_vfio_link_get(struct xsc_dev *xdev, struct rte_eth_link *link)
+{
+ struct xsc_cmd_query_linkinfo_mbox_in in = { };
+ struct xsc_cmd_query_linkinfo_mbox_out out = { };
+ struct xsc_cmd_linkinfo linkinfo;
+ int ret;
+ uint32_t speed_mode;
+
+ in.hdr.opcode = rte_cpu_to_be_16(XSC_CMD_OP_QUERY_LINK_INFO);
+ ret = xsc_vfio_mbox_exec(xdev, &in, sizeof(in), &out, sizeof(out));
+ if (ret != 0 || out.hdr.status != 0) {
+ PMD_DRV_LOG(ERR, "Failed to get link info, ret=%d, status=%d",
+ ret, out.hdr.status);
+ return -ENOEXEC;
+ }
+
+ linkinfo = out.ctx;
+ link->link_status = linkinfo.status ? RTE_ETH_LINK_UP : RTE_ETH_LINK_DOWN;
+ speed_mode = rte_be_to_cpu_32(linkinfo.linkspeed);
+ link->link_speed = xsc_vfio_link_speed_translate(speed_mode);
+ link->link_duplex = linkinfo.duplex;
+ link->link_autoneg = linkinfo.autoneg;
+
+ return 0;
+};
+
static int
xsc_vfio_modify_qp_status(struct xsc_dev *xdev, uint32_t qpn, int num, int opcode)
{
@@ -770,6 +864,8 @@ static struct xsc_dev_ops *xsc_vfio_ops = &(struct xsc_dev_ops) {
.dev_close = xsc_vfio_dev_close,
.set_mtu = xsc_vfio_set_mtu,
.get_mac = xsc_vfio_get_mac,
+ .link_status_set = xsc_vfio_link_status_set,
+ .link_get = xsc_vfio_link_get,
.destroy_qp = xsc_vfio_destroy_qp,
.destroy_cq = xsc_vfio_destroy_cq,
.modify_qp_status = xsc_vfio_modify_qp_status,
--
2.25.1
next prev parent reply other threads:[~2025-08-29 8:24 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-29 8:24 [PATCH 00/14] net/xsc: PMD updates Renyong Wan
2025-08-29 8:24 ` [PATCH 01/14] net/xsc: add FW version get support Renyong Wan
2025-08-29 8:24 ` [PATCH 02/14] net/xsc: add TSO support Renyong Wan
2025-08-29 8:24 ` [PATCH 03/14] net/xsc: support module EEPROM dump Renyong Wan
2025-08-29 8:24 ` [PATCH 04/14] net/xsc: support promiscuous mode Renyong Wan
2025-08-29 8:24 ` Renyong Wan [this message]
2025-08-29 8:24 ` [PATCH 06/14] net/xsc: add link status event support Renyong Wan
2025-08-29 8:24 ` [PATCH 07/14] net/xsc: add FEC get and set support Renyong Wan
2025-08-29 8:24 ` [PATCH 08/14] net/xsc: optimize RSS queue creation Renyong Wan
2025-08-29 8:24 ` [PATCH 09/14] net/xsc: optimize QP and CQ memory allocation Renyong Wan
2025-08-29 8:24 ` [PATCH 10/14] net/xsc: optimize Rx path Renyong Wan
2025-08-29 8:24 ` [PATCH 11/14] net/xsc: optimize stop and close Renyong Wan
2025-08-29 8:24 ` [PATCH 12/14] net/xsc: support per port for multi-process Renyong Wan
2025-08-29 8:24 ` [PATCH 13/14] net/xsc: fix uninitialized value Renyong Wan
2025-08-29 8:24 ` [PATCH 14/14] net/xsc: update release notes for xsc PMD Renyong Wan
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=20250829082415.24369-6-wanry@yunsilicon.com \
--to=wanry@yunsilicon.com \
--cc=dev@dpdk.org \
--cc=jacky@yunsilicon.com \
--cc=nana@yunsilicon.com \
--cc=qianr@yunsilicon.com \
--cc=stephen@networkplumber.org \
--cc=thomas@monjalon.net \
--cc=weihg@yunsilicon.com \
--cc=xudw@yunsilicon.com \
--cc=zhangxx@yunsilicon.com \
--cc=zhenghy@yunsilicon.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).