From patchwork Wed Apr 3 13:59:53 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Marek Pazdan X-Patchwork-Id: 139082 X-Patchwork-Delegate: ferruh.yigit@amd.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 4776F43DEE; Wed, 3 Apr 2024 18:09:44 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 26320402E8; Wed, 3 Apr 2024 18:09:39 +0200 (CEST) Received: from mail-oo1-f97.google.com (mail-oo1-f97.google.com [209.85.161.97]) by mails.dpdk.org (Postfix) with ESMTP id 8A28E4025C for ; Wed, 3 Apr 2024 15:59:55 +0200 (CEST) Received: by mail-oo1-f97.google.com with SMTP id 006d021491bc7-5a4f7a648dbso4003494eaf.3 for ; Wed, 03 Apr 2024 06:59:55 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1712152795; x=1712757595; h=content-transfer-encoding:references:in-reply-to:message-id:date :subject:cc:to:from:dkim-signature:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=7ZBh5weVpChsH75Wef9UINqrt1A9b85T/5MeNtS11UA=; b=WnsN0CkwAN308FMmIrJX9albay/IbWMwJxxOVDgxX6iHLEn0inNezzaD5J/cL81+Pi id23YPc/D4tzsy0AQdAkHpa1/KfHR5/HyhtCJZG/Uvg03a8DGowXKNN97cTgn8Im+9JC 4STPy+BQ7ZHpDo1ON+BH4eDoE7Go4P3YcOT0W7ILO1yjKevzK5jwDrB04gjgAxv+6spM KiwL4vT+Yxoq8XewMotf+c1QIbQ254VPHb07l1f74Yvikfe6Evq4lUVbKisY+1MzACR+ TBhywF0+xeRaAI/mVDP8zwwbvak6u6fkvc82e1v//8UalqP8IUwCWIqxlNLxBbTBU1jK /AMw== X-Gm-Message-State: AOJu0YxdpUIvbSC7huDefx0Dnarz2WVHOhkz+pbjtQIShWoQGJwpPIoi 2zy3R/kpTOL9gWGdKJ22V4GC7KFW6sobp9rTT4n/5CaPg1+l2kOPZiS7MBYqhLg3LqFo7eUVLIR 3uMguzwyGx6n24fmOdPcanIGzjSBXmg== X-Google-Smtp-Source: AGHT+IEEsdfZmlNjarIYopxwGTp9QfrP9etrzY8hjjVNFxRgCRAwMnuXMCcWZVvO+PAUSs7R16uGYVAEHrN1 X-Received: by 2002:a05:6820:2715:b0:5a5:21df:7eef with SMTP id db21-20020a056820271500b005a521df7eefmr14968141oob.2.1712152794816; Wed, 03 Apr 2024 06:59:54 -0700 (PDT) Received: from smtp.aristanetworks.com ([74.123.28.25]) by smtp-relay.gmail.com with ESMTPS id er10-20020a05622a5e8a00b004311ec03154sm1334233qtb.9.2024.04.03.06.59.54 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Wed, 03 Apr 2024 06:59:54 -0700 (PDT) X-Relaying-Domain: arista.com Received: from mpazdan-autoneg-dpdk-0.sjc.aristanetworks.com (mpazdan-autoneg-dpdk-0-us184.sjc.aristanetworks.com [10.243.132.215]) by smtp.aristanetworks.com (Postfix) with ESMTP id 9CDC1402054; Wed, 3 Apr 2024 06:59:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arista.com; s=Arista-A; t=1712152793; bh=7ZBh5weVpChsH75Wef9UINqrt1A9b85T/5MeNtS11UA=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=Fj7t0B1M73uAj43FioVzhQINLcmK8gWkRqCBZAjRSB1IRsatpsKeM4Df5iVxZT1WP o3Doz91KGci04+2UqyYy8okwoa31Q4RXp6tbPB0hj25tm5fXFB2L+i4i244ZPAa8ZR 39mOIgMyxCEfNVMLgnqW0X5GvvdTYo6ksYpfCEm7x5g9rj9UMG+9aVfs57Ju+kqxdY Xf83wHqhdJp4NUG6c2fYgpgoPd5SDq4Z7q7FMnSGYY461QhafHLtkWoM8KoepEVZds z/+a183DeX0/Pn3kdZ488OqSgmcC4VoKCM7ZiTXQVV+CjuVxiqZq/3mmgZKgqmHUi/ 1hC/SN0qY2BwA== X-SMTP-Authentication: Allow-List-permitted X-SMTP-Authentication: Allow-List-permitted From: Marek Pazdan To: Thomas Monjalon , Ferruh Yigit , Andrew Rybchenko Cc: dev@dpdk.org, Marek Pazdan Subject: [PATCH] lib: add get/set link settings interface Date: Wed, 3 Apr 2024 06:59:53 -0700 Message-ID: <20240403135953.7209-1-mpazdan@arista.com> In-Reply-To: <20240326235909.25276-1-mpazdan@arista.com> References: <20240326235909.25276-1-mpazdan@arista.com> X-Mailman-Approved-At: Wed, 03 Apr 2024 18:09:36 +0200 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 There are link settings parameters available from PMD drivers level which are currently not exposed to the user via consistent interface. When interface is available for system level those information can be acquired with 'ethtool DEVNAME' (ioctl: ETHTOOL_SLINKSETTINGS/ ETHTOOL_GLINKSETTINGS). There are use cases where physical interface is passthrough to dpdk driver and is not available from system level. Information provided by ioctl carries information useful for link auto negotiation settings among others. Signed-off-by: Marek Pazdan --- .mailmap | 1 + lib/ethdev/ethdev_driver.h | 33 ++++++ lib/ethdev/rte_ethdev.c | 26 +++++ lib/ethdev/rte_ethdev.h | 215 +++++++++++++++++++++++++++++++++++-- lib/ethdev/version.map | 3 + 5 files changed, 270 insertions(+), 8 deletions(-) diff --git a/.mailmap b/.mailmap index 3843868716..145ce726ba 100644 --- a/.mailmap +++ b/.mailmap @@ -885,6 +885,7 @@ Marcin Wojtas Marcin Zapolski Marco Varlese Marc Sune +Marek Pazdan Maria Lingemark Mario Carrillo Mário Kuka diff --git a/lib/ethdev/ethdev_driver.h b/lib/ethdev/ethdev_driver.h index 0dbf2dd6a2..ee8f786b99 100644 --- a/lib/ethdev/ethdev_driver.h +++ b/lib/ethdev/ethdev_driver.h @@ -1119,6 +1119,34 @@ typedef const uint32_t *(*eth_buffer_split_supported_hdr_ptypes_get_t)(struct rt */ typedef int (*eth_dev_priv_dump_t)(struct rte_eth_dev *dev, FILE *file); +/** @internal Retrieve physical link settings of a port. + * + * @param dev + * Port (ethdev) handle + * + * @param[out] settings + * Physical port link settings. + * + * @return + * Negative errno value on error, zero otherwise + */ +typedef int (*eth_get_link_settings_t)(struct rte_eth_dev *dev, + struct rte_link_settings *settings); + +/** @internal Configure physical link settings of a port. + * + * @param dev + * Port (ethdev) handle + * + * @param settings + * Physical port link settings. + * + * @return + * Negative errno value on error, zero otherwise + */ +typedef int (*eth_set_link_settings_t)(struct rte_eth_dev *dev, + const struct rte_link_settings *settings); + /** * @internal Set Rx queue available descriptors threshold. * @see rte_eth_rx_avail_thresh_set() @@ -1474,6 +1502,11 @@ struct eth_dev_ops { eth_count_aggr_ports_t count_aggr_ports; /** Map a Tx queue with an aggregated port of the DPDK port */ eth_map_aggr_tx_affinity_t map_aggr_tx_affinity; + + /** Retrieve physical link settings */ + eth_get_link_settings_t get_link_settings; + /** Configure physical link settings */ + eth_set_link_settings_t set_link_settings; }; /** diff --git a/lib/ethdev/rte_ethdev.c b/lib/ethdev/rte_ethdev.c index f1c658f49e..d2c52921ef 100644 --- a/lib/ethdev/rte_ethdev.c +++ b/lib/ethdev/rte_ethdev.c @@ -6844,6 +6844,32 @@ rte_eth_dev_priv_dump(uint16_t port_id, FILE *file) return eth_err(port_id, (*dev->dev_ops->eth_dev_priv_dump)(dev, file)); } +int rte_eth_dev_get_link_settings(uint16_t port_id, + struct rte_link_settings *settings) +{ + struct rte_eth_dev *dev; + + RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV); + dev = &rte_eth_devices[port_id]; + + if (*dev->dev_ops->get_link_settings == NULL) + return -ENOTSUP; + return eth_err(port_id, (*dev->dev_ops->get_link_settings)(dev, settings)); +} + +int rte_eth_dev_set_link_settings(uint16_t port_id, + const struct rte_link_settings *settings) +{ + struct rte_eth_dev *dev; + + RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV); + dev = &rte_eth_devices[port_id]; + + if (*dev->dev_ops->set_link_settings == NULL) + return -ENOTSUP; + return eth_err(port_id, (*dev->dev_ops->set_link_settings)(dev, settings)); +} + int rte_eth_rx_descriptor_dump(uint16_t port_id, uint16_t queue_id, uint16_t offset, uint16_t num, FILE *file) diff --git a/lib/ethdev/rte_ethdev.h b/lib/ethdev/rte_ethdev.h index 147257d6a2..66aad925d0 100644 --- a/lib/ethdev/rte_ethdev.h +++ b/lib/ethdev/rte_ethdev.h @@ -335,7 +335,7 @@ struct rte_eth_stats { __extension__ struct __rte_aligned(8) rte_eth_link { /**< aligned for atomic64 read/write */ uint32_t link_speed; /**< RTE_ETH_SPEED_NUM_ */ - uint16_t link_duplex : 1; /**< RTE_ETH_LINK_[HALF/FULL]_DUPLEX */ + uint16_t link_duplex : 2; /**< RTE_ETH_LINK_[HALF/FULL/UNKNOWN]_DUPLEX */ uint16_t link_autoneg : 1; /**< RTE_ETH_LINK_[AUTONEG/FIXED] */ uint16_t link_status : 1; /**< RTE_ETH_LINK_[DOWN/UP] */ }; @@ -343,13 +343,27 @@ struct __rte_aligned(8) rte_eth_link { /**< aligned for atomic64 read/write */ /**@{@name Link negotiation * Constants used in link management. */ -#define RTE_ETH_LINK_HALF_DUPLEX 0 /**< Half-duplex connection (see link_duplex). */ -#define RTE_ETH_LINK_FULL_DUPLEX 1 /**< Full-duplex connection (see link_duplex). */ -#define RTE_ETH_LINK_DOWN 0 /**< Link is down (see link_status). */ -#define RTE_ETH_LINK_UP 1 /**< Link is up (see link_status). */ -#define RTE_ETH_LINK_FIXED 0 /**< No autonegotiation (see link_autoneg). */ -#define RTE_ETH_LINK_AUTONEG 1 /**< Autonegotiated (see link_autoneg). */ -#define RTE_ETH_LINK_MAX_STR_LEN 40 /**< Max length of default link string. */ +#define RTE_ETH_LINK_HALF_DUPLEX 0 /**< Half-duplex connection (see link_duplex). */ +#define RTE_ETH_LINK_FULL_DUPLEX 1 /**< Full-duplex connection (see link_duplex). */ +#define RTE_ETH_LINK_UNKNOWN_DUPLEX 2 /**