From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f179.google.com (mail-wr0-f179.google.com [209.85.128.179]) by dpdk.org (Postfix) with ESMTP id 8B3CD2629 for ; Mon, 24 Jul 2017 14:09:18 +0200 (CEST) Received: by mail-wr0-f179.google.com with SMTP id v105so75076718wrb.0 for ; Mon, 24 Jul 2017 05:09:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to :user-agent; bh=VPTWHTZJrfyFclE+SW4Rfb0dIvxPbddMxcEtxbOdTn4=; b=Cm2RKlSNGUD3YER8qb8ZlpgjUFIo6RoBVmLPAOZUBD7lqM/aNkfJlHOQI9SNDsSAnd VhimC17+wIqCztoHhJxmTthg+11epSe67/doNToirT76oQny7OP8CzxBraQSrTBd38JP M3NCVpthvKhcqpF7yj8koi2QvCnNT5GGR3y1O2vDO3FQD1fw5YS42Mj9hiurMgWqH1vU X4htmYt7hKvUFNYczi7V+USsDgJ0I1Mya1WJ7l8/AzQehscYyGhhI9H6WJCzyTbVGphg OU7uuKWvcq150+BfN0/aqxFRdNkCvNaoo+lp0gPWXrtFtNJnIVSaF8VonwuITi7c4xfs pMHg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to:user-agent; bh=VPTWHTZJrfyFclE+SW4Rfb0dIvxPbddMxcEtxbOdTn4=; b=EvCc3PYnUU3eF3Xl2xZMqiyt54wZxLwmNPu1Nt53RgkRmpMEVjkJkJZfugOeUjd2tc 5Pr8I7abp86aSNa5/edq5AtcFrJNGDn3lpKTS69zH4xXMWAoxZyfMGgKR5qsyg/+ADmB XE3DtRsvAh1qQ0mjyKzAhMDVYITYQPl+k0Wmfa3aSMevJITZeCMHw56WCGvvcxoP0YKq t3RgStEXsLEC65Nlprj9wBgMlQsGXLxcDcoE1JG+yLAgmQQS2b5hUibYbUVs+djgAz6Y YktlEioxWRwQqtlK+Ux4LhZfw+zHc+ci14J4SB5tQlKhbEUHEtOl1d75ate2llVJNYYm 9e3g== X-Gm-Message-State: AIVw111OqakzEr2BvN4uF+6gJ+YvotZtqFL/nT1f9RVT8XKkeoUoV4bI 6lWtGB56nSV31Xeh X-Received: by 10.223.160.174 with SMTP id m43mr15014865wrm.194.1500898158117; Mon, 24 Jul 2017 05:09:18 -0700 (PDT) Received: from autoinstall.dev.6wind.com (host.78.145.23.62.rev.coltfrance.com. [62.23.145.78]) by smtp.gmail.com with ESMTPSA id y202sm2199235wme.5.2017.07.24.05.09.17 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 24 Jul 2017 05:09:17 -0700 (PDT) Date: Mon, 24 Jul 2017 14:09:08 +0200 From: =?iso-8859-1?Q?N=E9lio?= Laranjeiro To: Gowrishankar Cc: Adrien Mazarguil , dev@dpdk.org, Thomas Monjalon , Ferruh Yigit Message-ID: <20170724120908.GE2320@autoinstall.dev.6wind.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Subject: Re: [dpdk-dev] [PATCH] mlx5: fix get link settings call for speed and duplex values 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, 24 Jul 2017 12:09:18 -0000 Hi Gowrishankar, Sorry, it took me a little time to test this patch, nice catch. Small format points, the title should be net/mlx5 and there is a complain with checkpatch see below. On Tue, Jul 18, 2017 at 11:58:34AM +0530, Gowrishankar wrote: > From: Gowrishankar Muthukrishnan > > mlx5 pmd does not correctly retrieve link speed and duplex values > in linksetting struct through ETHTOOL_GLINKSETTINGS ioctl call. I would suggest to make it a little shorter by adding the following fact: This returns an inconsistent link status. It does not only impact traffic generator but anything relying on the link status. > Due to which pktgen application could not correctly calculate tx/rx > rate and hence, very few packets (of count 32) only sent all the time. > > Link settings can be derived through ETHTOOL_GLINKSETTINGS ioctl > only after updating ethtool_link_settings struct with appropriate > link mode bitmask words returned by its kernel side implementation. > With this patch, pktgen app is able to tx/rx packets on mlx5 ports. As it is fixing an issue it should have the fixes tag line, in this case add the following one: Fixes: 188408719888 ("net/mlx5: fix support for newer link speeds") Cc: stable@dpdk.org > Signed-off-by: Gowrishankar Muthukrishnan > --- > drivers/net/mlx5/mlx5_ethdev.c | 27 +++++++++++++++++++-------- > 1 file changed, 19 insertions(+), 8 deletions(-) > > diff --git a/drivers/net/mlx5/mlx5_ethdev.c b/drivers/net/mlx5/mlx5_ethdev.c > index b70b7b9..003cef3 100644 > --- a/drivers/net/mlx5/mlx5_ethdev.c > +++ b/drivers/net/mlx5/mlx5_ethdev.c > @@ -119,6 +119,7 @@ struct ethtool_link_settings { > #define ETHTOOL_LINK_MODE_100000baseCR4_Full_BIT 38 > #define ETHTOOL_LINK_MODE_100000baseLR4_ER4_Full_BIT 39 > #endif > +#define ETHTOOL_LINK_MODE_MASK_MAX_KERNEL_NU32 (SCHAR_MAX) > > /** > * Return private structure associated with an Ethernet device. > @@ -807,9 +808,11 @@ struct priv * > mlx5_link_update_unlocked_gs(struct rte_eth_dev *dev, int wait_to_complete) > { > struct priv *priv = mlx5_get_priv(dev); > - struct ethtool_link_settings edata = { > - .cmd = ETHTOOL_GLINKSETTINGS, > - }; > + struct { > + struct ethtool_link_settings edata; > + __u32 link_mode_data[3 * ETHTOOL_LINK_MODE_MASK_MAX_KERNEL_NU32]; Checkpatch complains about the line size here. Thanks, -- Nélio Laranjeiro 6WIND