From patchwork Fri Jul 30 16:08:29 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Harman Kalra X-Patchwork-Id: 96472 X-Patchwork-Delegate: jerinj@marvell.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 03FCAA0C40; Fri, 30 Jul 2021 18:08:56 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id B54E2410DA; Fri, 30 Jul 2021 18:08:54 +0200 (CEST) Received: from mx0b-0016f401.pphosted.com (mx0a-0016f401.pphosted.com [67.231.148.174]) by mails.dpdk.org (Postfix) with ESMTP id 16BEF410DA for ; Fri, 30 Jul 2021 18:08:52 +0200 (CEST) Received: from pps.filterd (m0045849.ppops.net [127.0.0.1]) by mx0a-0016f401.pphosted.com (8.16.0.43/8.16.0.43) with SMTP id 16UG6GLO002356 for ; Fri, 30 Jul 2021 09:08:52 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h=from : to : cc : subject : date : message-id : in-reply-to : references : mime-version : content-type; s=pfpt0220; bh=Xeh8kHEoErUNt0hen2pNPnAx/nE1yJzxRaW2bhHa+DM=; b=gGo7pYZEQVTQfolG8ZDl9jL+xA61RJHis6X1seqEQt1QkFXydBb8y3InHVOMt7X9+Sle 9DtdFmPIbW+xk03pHAXENkwoHH8Nlgn7YehvyRieM3uS44FcBzSKehYV1ZnXNtlcJFOv AYJu5ymEOI2mIlOIlVDHvSLETWTLWu8+QcyjhVQ1/+c5+TPX/yNt8nUoVUCgr4VCxFK5 39gIOcTNCb24GisgkcYBRsAo2lzFit8IZ/M/ikSVvn8zX+r9sgxWnjqZ6mPtIt6wpp7m +oOAzXqkljuZIAZzEHu2/22o11Sh9pFi1xlYYCCC4lrDYVsFuGbp1dq3YA2j48bZOs5b Dw== Received: from dc5-exch02.marvell.com ([199.233.59.182]) by mx0a-0016f401.pphosted.com with ESMTP id 3a4866try5-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT) for ; Fri, 30 Jul 2021 09:08:52 -0700 Received: from DC5-EXCH01.marvell.com (10.69.176.38) by DC5-EXCH02.marvell.com (10.69.176.39) with Microsoft SMTP Server (TLS) id 15.0.1497.18; Fri, 30 Jul 2021 09:08:50 -0700 Received: from maili.marvell.com (10.69.176.80) by DC5-EXCH01.marvell.com (10.69.176.38) with Microsoft SMTP Server id 15.0.1497.18 via Frontend Transport; Fri, 30 Jul 2021 09:08:50 -0700 Received: from localhost.localdomain (unknown [10.29.52.211]) by maili.marvell.com (Postfix) with ESMTP id EAF843F705F; Fri, 30 Jul 2021 09:08:48 -0700 (PDT) From: Harman Kalra To: Nithin Dabilpuram , Kiran Kumar K , Sunil Kumar Kori , Satha Rao CC: , Harman Kalra Date: Fri, 30 Jul 2021 21:38:29 +0530 Message-ID: <20210730160829.13597-2-hkalra@marvell.com> X-Mailer: git-send-email 2.18.0 In-Reply-To: <20210730160829.13597-1-hkalra@marvell.com> References: <20210730160829.13597-1-hkalra@marvell.com> MIME-Version: 1.0 X-Proofpoint-GUID: n1zHwxChvAA0gauPDgTsWFS-QNJ2G-FR X-Proofpoint-ORIG-GUID: n1zHwxChvAA0gauPDgTsWFS-QNJ2G-FR X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.391, 18.0.790 definitions=2021-07-30_11:2021-07-30, 2021-07-30 signatures=0 Subject: [dpdk-dev] [PATCH 2/2] net/cnxk: callback for getting link status X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Adding a new callback for reading the link status. PF can read it's link status and can forward the same to VF once it comes up. Signed-off-by: Harman Kalra Acked-by: Nithin Dabilpuram --- drivers/net/cnxk/cnxk_ethdev.c | 9 +++++++++ drivers/net/cnxk/cnxk_ethdev.h | 2 ++ drivers/net/cnxk/cnxk_link.c | 23 +++++++++++++++++++++++ 3 files changed, 34 insertions(+) diff --git a/drivers/net/cnxk/cnxk_ethdev.c b/drivers/net/cnxk/cnxk_ethdev.c index 0e3652ed51..7152dcd002 100644 --- a/drivers/net/cnxk/cnxk_ethdev.c +++ b/drivers/net/cnxk/cnxk_ethdev.c @@ -1314,6 +1314,10 @@ cnxk_eth_dev_init(struct rte_eth_dev *eth_dev) /* Register up msg callbacks */ roc_nix_mac_link_cb_register(nix, cnxk_eth_dev_link_status_cb); + /* Register up msg callbacks */ + roc_nix_mac_link_info_get_cb_register(nix, + cnxk_eth_dev_link_status_get_cb); + dev->eth_dev = eth_dev; dev->configured = 0; dev->ptype_disable = 0; @@ -1415,6 +1419,11 @@ cnxk_eth_dev_uninit(struct rte_eth_dev *eth_dev, bool reset) /* Disable link status events */ roc_nix_mac_link_event_start_stop(nix, false); + /* Unregister the link update op, this is required to stop VFs from + * receiving link status updates on exit path. + */ + roc_nix_mac_link_cb_unregister(nix); + /* Free up SQs */ for (i = 0; i < eth_dev->data->nb_tx_queues; i++) { dev_ops->tx_queue_release(eth_dev->data->tx_queues[i]); diff --git a/drivers/net/cnxk/cnxk_ethdev.h b/drivers/net/cnxk/cnxk_ethdev.h index 4eead03905..4caf26303f 100644 --- a/drivers/net/cnxk/cnxk_ethdev.h +++ b/drivers/net/cnxk/cnxk_ethdev.h @@ -349,6 +349,8 @@ int cnxk_nix_rss_hash_conf_get(struct rte_eth_dev *eth_dev, void cnxk_nix_toggle_flag_link_cfg(struct cnxk_eth_dev *dev, bool set); void cnxk_eth_dev_link_status_cb(struct roc_nix *nix, struct roc_nix_link_info *link); +void cnxk_eth_dev_link_status_get_cb(struct roc_nix *nix, + struct roc_nix_link_info *link); int cnxk_nix_link_update(struct rte_eth_dev *eth_dev, int wait_to_complete); int cnxk_nix_queue_stats_mapping(struct rte_eth_dev *dev, uint16_t queue_id, uint8_t stat_idx, uint8_t is_rx); diff --git a/drivers/net/cnxk/cnxk_link.c b/drivers/net/cnxk/cnxk_link.c index 3fdbdba495..6a70801675 100644 --- a/drivers/net/cnxk/cnxk_link.c +++ b/drivers/net/cnxk/cnxk_link.c @@ -45,6 +45,29 @@ nix_link_status_print(struct rte_eth_dev *eth_dev, struct rte_eth_link *link) plt_info("Port %d: Link Down", (int)(eth_dev->data->port_id)); } +void +cnxk_eth_dev_link_status_get_cb(struct roc_nix *nix, + struct roc_nix_link_info *link) +{ + struct cnxk_eth_dev *dev = (struct cnxk_eth_dev *)nix; + struct rte_eth_link eth_link; + struct rte_eth_dev *eth_dev; + + if (!link || !nix) + return; + + eth_dev = dev->eth_dev; + if (!eth_dev) + return; + + rte_eth_linkstatus_get(eth_dev, ð_link); + + link->status = eth_link.link_status; + link->speed = eth_link.link_speed; + link->autoneg = eth_link.link_autoneg; + link->full_duplex = eth_link.link_duplex; +} + void cnxk_eth_dev_link_status_cb(struct roc_nix *nix, struct roc_nix_link_info *link) {