From patchwork Tue Jan 28 07:45:42 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: AMARANATH SOMALAPURAM X-Patchwork-Id: 65199 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 798BBA04B3; Tue, 28 Jan 2020 08:49:11 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 893711C117; Tue, 28 Jan 2020 08:49:10 +0100 (CET) Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2061.outbound.protection.outlook.com [40.107.223.61]) by dpdk.org (Postfix) with ESMTP id 4A6C81BF7D; Tue, 28 Jan 2020 08:49:08 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=mikv6aySSjh+o/7DstPt86Pu9LJnRLgDkGdThL/TmVjbH+PfJz75onlq7rHkl3URWjKqvXv3HmRUUwPW+eQ4yL7kcRx+y0Vk/oEC8f+9gpgzb79XFkcCYyutp0sFa18/tiP6Dgl51RJ/KUmYrPWw1uNpEUqNQ9YXK/ksvxRtWpwRf97+rrpEJMT8m65BODs4N1ATRyVMLr82hCiX6aExCA0/3IoRxpJxricN/pvW8CocYp3HAJ+g4lPuxG84ANrFjhlla0xxbwBdVLBG/tvhufh1r9WVxdloAZ0kT+rP0T1hyxrSlh7bWUtFtZYdhdHRwtDo8Q/prRmw/zhzF8XJmA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=zE7CT6B85ox/t531u+CpiNjPqp7Zr94nKv3HAqZ+Olw=; b=iR7/vVh10harYgkbajlrJAqKliKR8aZnGw7qdCaADSU0ZSCr6ZFkWWLlQ4GkFxZuV3OjznetDnFbFynHZyMoiPR0ZN0bstgb9UhnrZAD5mSV455LNG8Qiojs0cSwGu/S4TyID2UjrIskB292OXnWziGHnsPaqj1Xq6uoSzcPCTtjQF7w1dYw/fOrOwzeBeuOG0RMnuXvJGB8uVzjJkxsbCNJnP1yEQEi0hv9nmNMBwdAebmaT/aygYhaXETYOwYauUtBozKf15tZzE22rzxW2xOILUwUSk0+17kGNye+WGjFrO/rwqURC5bJkeaDaemwF1WSFXFbYoaMHVHpB2WD+Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amdcloud.onmicrosoft.com; s=selector2-amdcloud-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=zE7CT6B85ox/t531u+CpiNjPqp7Zr94nKv3HAqZ+Olw=; b=jHMVZ4gulgDdFsYEZayyiIBkCyxeayk67CuJrww7+WpGA4aSDkMoUzkigNgW+X7ecAjIPPP/ePPtrImGPtKZtTxqm6ZoGn84PDCUVhvDyvB653CP8DnzKLrxObnam0+UD//iKzz5obaXWqgZ4Cb5AFKG6yo2YuHCNZExq5Eb528= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Amaranath.Somalapuram@amd.com; Received: from DM5PR1201MB2474.namprd12.prod.outlook.com (10.172.87.136) by DM5PR1201MB2491.namprd12.prod.outlook.com (10.172.91.151) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2665.20; Tue, 28 Jan 2020 07:49:05 +0000 Received: from DM5PR1201MB2474.namprd12.prod.outlook.com ([fe80::3d24:2943:582c:c7b5]) by DM5PR1201MB2474.namprd12.prod.outlook.com ([fe80::3d24:2943:582c:c7b5%6]) with mapi id 15.20.2665.025; Tue, 28 Jan 2020 07:49:05 +0000 From: asomalap@amd.com To: dev@dpdk.org Cc: stable@dpdk.org Date: Tue, 28 Jan 2020 13:15:42 +0530 Message-Id: <20200128074542.44687-1-asomalap@amd.com> X-Mailer: git-send-email 2.17.1 X-ClientProxiedBy: MAXPR01CA0118.INDPRD01.PROD.OUTLOOK.COM (2603:1096:a00:5d::36) To DM5PR1201MB2474.namprd12.prod.outlook.com (2603:10b6:3:e3::8) MIME-Version: 1.0 Received: from amd.amd.com (165.204.157.251) by MAXPR01CA0118.INDPRD01.PROD.OUTLOOK.COM (2603:1096:a00:5d::36) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2665.20 via Frontend Transport; Tue, 28 Jan 2020 07:49:03 +0000 X-Mailer: git-send-email 2.17.1 X-Originating-IP: [165.204.157.251] X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 95b69a63-473a-4ae8-8631-08d7a3c68c4d X-MS-TrafficTypeDiagnostic: DM5PR1201MB2491: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:989; X-Forefront-PRVS: 029651C7A1 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(4636009)(366004)(199004)(189003)(36756003)(956004)(8936002)(2616005)(81166006)(81156014)(5660300002)(6666004)(66476007)(66556008)(66946007)(8676002)(498600001)(6916009)(1076003)(450100002)(6486002)(186003)(16526019)(26005)(9686003)(52116002)(7696005)(2906002)(4326008); DIR:OUT; SFP:1101; SCL:1; SRVR:DM5PR1201MB2491; H:DM5PR1201MB2474.namprd12.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1; Received-SPF: None (protection.outlook.com: amd.com does not designate permitted sender hosts) X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: SB/4YdnsZK28ji7w+60+H2FneqQB8uy6k6FUApHUbfe92VAlTVzRPQNXBi/JkJkeTbbFz2O2iX7zCCdZFtO9yPs5v2kYixiA/AUDklSZ2aboAfDDF9LmIXnMFa7Bla7OQIO8dd4LHtNf66MWc2PbH0kku1Ltc5yxpjNEotGtBB0JRQBUlUzoEgyNcxZINhC/Wc3sF3/uqRQK45ZjqqaZLqfm9r5jbXBMaOhIat/82Na+BPfX15AZCSx0JBKN5CJKM75M1qvH70IB9zkVZbHQqYcVhgTZjC+UGN2DPFwTnKxH/w8HMGfnyUq311yuf+df6vRJV6YV6DaVke5WJgEWXRFGg7WpH5vEbOOHlAVPJwsRdZ6HDjHuYNijkMM+UyVroOdVh6fX/gtSiWmN/epcRXMfFezqJS4WmiIQoTULD1Zv5H12xx6vLBV7JIM+CGmU X-MS-Exchange-AntiSpam-MessageData: +Vxw9kgapjFMlVAbTxxerqDHsGWAmdp+iZT5obrgF+F2hotatXWXpfaVXxuWz2BaJCBxIp/jTSujKm7gqCFXD6wzM94obJR/82NbbA5DLEA9KzCctnUgOs84RxFo+QnuEJ2E5XCQ2TZXfnGXv3NHhQ== X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: 95b69a63-473a-4ae8-8631-08d7a3c68c4d X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Jan 2020 07:49:05.3565 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: ZnnoZkMmez56JGJNFxwl+Us/234tWKOI9sLDQWg9XGzS05QN5nUyqcYkdooBd2ikh8ISG/H52WrfBN+HBXf3RQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR1201MB2491 Subject: [dpdk-dev] [PATCH v1] net/axgbe: swupport flow control API X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 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" From: Amaranath Somalapuram adding api for flow_ctrl_get and flow_ctrl_set by default axgbe driver flow control is dissabled adding dpdk flow control to set water high and low Signed-off-by: Amaranath Somalapuram --- drivers/net/axgbe/axgbe_ethdev.c | 85 +++++++++++++++++++++++++++++++- drivers/net/axgbe/axgbe_ethdev.h | 10 ++++ 2 files changed, 94 insertions(+), 1 deletion(-) diff --git a/drivers/net/axgbe/axgbe_ethdev.c b/drivers/net/axgbe/axgbe_ethdev.c index d0b6f091f..b88ad55ac 100644 --- a/drivers/net/axgbe/axgbe_ethdev.c +++ b/drivers/net/axgbe/axgbe_ethdev.c @@ -43,7 +43,11 @@ axgbe_dev_xstats_get_names_by_id(struct rte_eth_dev *dev, unsigned int size); static int axgbe_dev_xstats_reset(struct rte_eth_dev *dev); static int axgbe_dev_info_get(struct rte_eth_dev *dev, - struct rte_eth_dev_info *dev_info); + struct rte_eth_dev_info *dev_info); +static int axgbe_flow_ctrl_get(struct rte_eth_dev *dev, + struct rte_eth_fc_conf *fc_conf); +static int axgbe_flow_ctrl_set(struct rte_eth_dev *dev, + struct rte_eth_fc_conf *fc_conf); struct axgbe_xstats { char name[RTE_ETH_XSTATS_NAME_SIZE]; @@ -170,6 +174,8 @@ static const struct eth_dev_ops axgbe_eth_dev_ops = { .rx_queue_release = axgbe_dev_rx_queue_release, .tx_queue_setup = axgbe_dev_tx_queue_setup, .tx_queue_release = axgbe_dev_tx_queue_release, + .flow_ctrl_get = axgbe_flow_ctrl_get, + .flow_ctrl_set = axgbe_flow_ctrl_set, }; static int axgbe_phy_reset(struct axgbe_port *pdata) @@ -815,6 +821,83 @@ axgbe_dev_info_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info) return 0; } +static int +axgbe_flow_ctrl_get(struct rte_eth_dev *dev, struct rte_eth_fc_conf *fc_conf) +{ + struct axgbe_port *pdata = dev->data->dev_private; + struct xgbe_fc_info fc = pdata->fc; + unsigned int reg, reg_val = 0; + + reg = MAC_Q0TFCR; + reg_val = AXGMAC_IOREAD(pdata, reg); + fc.low_water[0] = AXGMAC_MTL_IOREAD_BITS(pdata, 0, MTL_Q_RQFCR, RFA); + fc.high_water[0] = AXGMAC_MTL_IOREAD_BITS(pdata, 0, MTL_Q_RQFCR, RFD); + fc.pause_time[0] = AXGMAC_GET_BITS(reg_val, MAC_Q0TFCR, PT); + fc.autoneg = pdata->pause_autoneg; + + if (pdata->rx_pause && pdata->tx_pause) + fc.mode = RTE_FC_FULL; + else if (pdata->rx_pause) + fc.mode = RTE_FC_RX_PAUSE; + else if (pdata->tx_pause) + fc.mode = RTE_FC_TX_PAUSE; + else + fc.mode = RTE_FC_NONE; + + fc_conf->high_water = (1024 + (fc.low_water[0] << 9)) / 1024; + fc_conf->low_water = (1024 + (fc.high_water[0] << 9)) / 1024; + fc_conf->pause_time = fc.pause_time[0]; + fc_conf->send_xon = fc.send_xon; + fc_conf->mode = fc.mode; + + return 0; +} + +static int +axgbe_flow_ctrl_set(struct rte_eth_dev *dev, struct rte_eth_fc_conf *fc_conf) +{ + struct axgbe_port *pdata = dev->data->dev_private; + struct xgbe_fc_info fc = pdata->fc; + unsigned int reg, reg_val = 0; + reg = MAC_Q0TFCR; + + pdata->pause_autoneg = fc_conf->autoneg; + pdata->phy.pause_autoneg = pdata->pause_autoneg; + fc.send_xon = fc_conf->send_xon; + AXGMAC_MTL_IOWRITE_BITS(pdata, 0, MTL_Q_RQFCR, RFA, + AXGMAC_FLOW_CONTROL_VALUE(1024 * fc_conf->high_water)); + AXGMAC_MTL_IOWRITE_BITS(pdata, 0, MTL_Q_RQFCR, RFD, + AXGMAC_FLOW_CONTROL_VALUE(1024 * fc_conf->low_water)); + AXGMAC_SET_BITS(reg_val, MAC_Q0TFCR, PT, fc_conf->pause_time); + AXGMAC_IOWRITE(pdata, reg, reg_val); + fc.mode = fc_conf->mode; + + if (fc.mode == RTE_FC_FULL) { + pdata->tx_pause = 1; + pdata->rx_pause = 1; + } else if (fc.mode == RTE_FC_RX_PAUSE) { + pdata->tx_pause = 0; + pdata->rx_pause = 1; + } else if (fc.mode == RTE_FC_TX_PAUSE) { + pdata->tx_pause = 1; + pdata->rx_pause = 0; + } else { + pdata->tx_pause = 0; + pdata->rx_pause = 0; + } + + if (pdata->tx_pause != (unsigned int)pdata->phy.tx_pause) + pdata->hw_if.config_tx_flow_control(pdata); + + if (pdata->rx_pause != (unsigned int)pdata->phy.rx_pause) + pdata->hw_if.config_rx_flow_control(pdata); + + pdata->hw_if.config_flow_control(pdata); + pdata->phy.tx_pause = pdata->tx_pause; + pdata->phy.rx_pause = pdata->rx_pause; + + return 0; +} static void axgbe_get_all_hw_features(struct axgbe_port *pdata) { diff --git a/drivers/net/axgbe/axgbe_ethdev.h b/drivers/net/axgbe/axgbe_ethdev.h index a1083b17b..746fb2f15 100644 --- a/drivers/net/axgbe/axgbe_ethdev.h +++ b/drivers/net/axgbe/axgbe_ethdev.h @@ -485,6 +485,15 @@ struct axgbe_mmc_stats { uint64_t rxwatchdogerror; }; +/* Flow control parameters */ +struct xgbe_fc_info { + uint32_t high_water[AXGBE_PRIORITY_QUEUES]; + uint32_t low_water[AXGBE_PRIORITY_QUEUES]; + uint16_t pause_time[AXGBE_PRIORITY_QUEUES]; + uint16_t send_xon; + enum rte_eth_fc_mode mode; + uint8_t autoneg; +}; /* * Structure to store private data for each port. */ @@ -625,6 +634,7 @@ struct axgbe_port { uint32_t rx_csum_enable; struct axgbe_mmc_stats mmc_stats; + struct xgbe_fc_info fc; }; void axgbe_init_function_ptrs_dev(struct axgbe_hw_if *hw_if);