From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ua0-f194.google.com (mail-ua0-f194.google.com [209.85.217.194]) by dpdk.org (Postfix) with ESMTP id D21631E3E9 for ; Mon, 11 Jun 2018 10:01:13 +0200 (CEST) Received: by mail-ua0-f194.google.com with SMTP id s13-v6so9472338uad.2 for ; Mon, 11 Jun 2018 01:01:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=semihalf-com.20150623.gappssmtp.com; s=20150623; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=fugDpi+HPkPcTZjnyhNO7F8PB2Mg5jxGdrGCqTtWqrA=; b=I4TvyzGvSsVvWqQ7GoSvhriwwXaAZHZTuGvo/XTbbxevlcU2qb64u04/pKiBRIqFgH y5yWuMuLAk2T69j3fNgi/INyS3vNbvS39h7ZLPfg5wxybaI8ZwRX7FhK0Rrk/8x6hJZP CiIzDmYFtwdPOFUol+sH+JHktRgigTBu7eBTjJDl1y5Wv+zhjUE3npHuUErAtIoq6fCb 6+Z5IQV+LPNfE5nXuHcbu3/mAozEJUc2hWYhL5qllcqdqhrFTt5d8cAyYiuNX4o961ow fN8WU3KXB/QxE2BB1Z610cfh0/eb1VZVawcHiZOCYFjofy+qfSTljR8CSBVPk2jrtLBx MIww== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=fugDpi+HPkPcTZjnyhNO7F8PB2Mg5jxGdrGCqTtWqrA=; b=EnlSJUhLSeNWC/q7BVR8UWkb0/h+SejVbqWBaDJ40ttErR0w9wNyDcEfIRinORs6zO QSmZtXaR3shsl1OZ+pZEepf2uxcb7+GLIWOQ4NDdZWKoEYQnUdaEKo+oTRsuMOVEvFFa XLW9iPqADFtZI2fYitD5Qytk+CVmgz7wLrj//j+eWdKqjL+BTgYbrGYjeCD9WwaRD65E u55ymXv8U9LLNhqTlbnOPSmRucgoq6SdJpNfCTw3GGpemoT2DhzU3NBjQHep6a1VLaAv SXkQXSCuFy4WmbpVNX03BFDJbVRv3/bJBpSZdM7mC6vFNbtQtrXvEAzT99g3Ryaa1EUZ V1ww== X-Gm-Message-State: APt69E0MbBvBsZ7nsDafHOSOUpJlu1UVhvK9ETjR262PXqHv3mRDgxiJ ntdscYq7lsn2ijfLs+5w1FYNJU3DhHiB3wulZS/+EA== X-Google-Smtp-Source: ADUXVKLBo9aHdgYk+ArJO/Dykc5HA+rHPwU0Rx0SPJR3A97i/T12CNlynJ9rFHyetLYINCzgC2S+Sgcd0lbnBGQRkhU= X-Received: by 2002:ab0:596f:: with SMTP id o44-v6mr10611145uad.29.1528704073101; Mon, 11 Jun 2018 01:01:13 -0700 (PDT) MIME-Version: 1.0 Received: by 2002:a67:62c7:0:0:0:0:0 with HTTP; Mon, 11 Jun 2018 01:01:12 -0700 (PDT) In-Reply-To: References: <20180607094322.14312-1-mk@semihalf.com> <20180607094322.14312-27-mk@semihalf.com> From: =?UTF-8?Q?Micha=C5=82_Krawczyk?= Date: Mon, 11 Jun 2018 10:01:12 +0200 Message-ID: To: Chas Williams <3chas3@gmail.com> Cc: Ferruh Yigit , Marcin Wojtas , "Tzalik, Guy" , "Schmeilin, Evgeny" , dev@dpdk.org, "Matushevsky, Alexander" , =?UTF-8?B?UmFmYcWCIEtvemlr?= Content-Type: text/plain; charset="UTF-8" Subject: Re: [dpdk-dev] [PATCH v3 27/27] net/ena: set link speed as none 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: , X-List-Received-Date: Mon, 11 Jun 2018 08:01:14 -0000 2018-06-10 3:35 GMT+02:00 Chas Williams <3chas3@gmail.com>: > > > On Fri, Jun 8, 2018 at 3:37 PM Ferruh Yigit wrote: >> >> On 6/7/2018 10:43 AM, Michal Krawczyk wrote: >> > From: Rafal Kozik >> > >> > Link speed should is not limited to 10Gb/s and it shouldn't be >> > hardcoded. >> > >> > They link speed is set to none instead and the applications shouldn't >> > rely on this value when using ENA PMD. >> >> Why not able to set link speed? >> >> And what is the link_autoneg, ETH_LINK_FIXED? What is the point of setting >> link >> speed FIXED and later speed value NONE? > > > The link speed is part of the spanning tree path computation. I don't think > picking 0 (or undefined) > is the appropriate choice. For virtual interfaces link speed is a > troublesome quantity but advertising > undefined seems worse then just use some constant value. > Setting constant value is not making much sense in the virtualized environment (ENA can be only used in the cloud). Link speed can be very flexible in that case and that's why ENA isn't providing it. If we will use constant value, what it should be? It may lead to confusion or can result in the performance issues because some applications are relying on this number to send max traffic. Applications should not make assumption that the link speed is always giving valid value and if it is not, they should try to configure rate dynamically, instead of statically. >> >> >> > >> > Fixes: 1173fca ("ena: add polling-mode driver") >> >> For next time can you please use git alias to have consistent Fixes format >> >> > >> > Signed-off-by: Rafal Kozik >> > Acked-by: Michal Krawczyk >> > --- >> > drivers/net/ena/ena_ethdev.c | 2 +- >> > 1 file changed, 1 insertion(+), 1 deletion(-) >> > >> > diff --git a/drivers/net/ena/ena_ethdev.c b/drivers/net/ena/ena_ethdev.c >> > index 5c3b6494f..9ae73e331 100644 >> > --- a/drivers/net/ena/ena_ethdev.c >> > +++ b/drivers/net/ena/ena_ethdev.c >> > @@ -848,7 +848,7 @@ static int ena_link_update(struct rte_eth_dev *dev, >> > adapter = (struct ena_adapter *)(dev->data->dev_private); >> > >> > link->link_status = adapter->link_status ? ETH_LINK_UP : >> > ETH_LINK_DOWN; >> > - link->link_speed = ETH_SPEED_NUM_10G; >> > + link->link_speed = ETH_SPEED_NUM_NONE; >> > link->link_duplex = ETH_LINK_FULL_DUPLEX; >> > >> > return 0; >> > >> >