From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934125AbeEWVqy (ORCPT ); Wed, 23 May 2018 17:46:54 -0400 Received: from mail.kernel.org ([198.145.29.99]:58586 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933936AbeEWVqw (ORCPT ); Wed, 23 May 2018 17:46:52 -0400 Date: Wed, 23 May 2018 16:46:51 -0500 From: Bjorn Helgaas To: Jeff Kirsher , Ganesh Goudar , Michael Chan , Ariel Elior , "David S. Miller" Cc: linux-pci@vger.kernel.org, everest-linux-l2@cavium.com, intel-wired-lan@lists.osuosl.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Tal Gilboa , Tariq Toukan , Jacob Keller , Jakub Kicinski Subject: Re: [PATCH v6 0/5] PCI: Improve PCIe link status reporting Message-ID: <20180523214651.GE150632@bhelgaas-glaptop.roam.corp.google.com> References: <152537719056.62474.2571390812509425478.stgit@bhelgaas-glaptop.roam.corp.google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <152537719056.62474.2571390812509425478.stgit@bhelgaas-glaptop.roam.corp.google.com> User-Agent: Mutt/1.9.2 (2017-12-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org [+to Davem] On Thu, May 03, 2018 at 03:00:07PM -0500, Bjorn Helgaas wrote: > This is based on Tal's recent work to unify the approach for reporting PCIe > link speed/width and whether the device is being limited by a slower > upstream link. > > The new pcie_print_link_status() interface appeared in v4.17-rc1; see > 9e506a7b5147 ("PCI: Add pcie_print_link_status() to log link speed and > whether it's limited"). > > That's a good way to replace use of pcie_get_minimum_link(), which gives > misleading results when a path contains both a fast, narrow link and a > slow, wide link: it reports the equivalent of a slow, narrow link. > > This series removes the remaining uses of pcie_get_minimum_link() and then > removes the interface itself. I'd like to merge them all through the PCI > tree to make the removal easy. > > This does change the dmesg reporting of link speeds, and in the ixgbe case, > it changes the reporting from KERN_WARN level to KERN_INFO. If that's an > issue, let's talk about it. I'm hoping the reduce code size, improved > functionality, and consistency across drivers is enough to make this > worthwhile. > > --- > > Bjorn Helgaas (5): > bnx2x: Report PCIe link properties with pcie_print_link_status() > bnxt_en: Report PCIe link properties with pcie_print_link_status() > cxgb4: Report PCIe link properties with pcie_print_link_status() > ixgbe: Report PCIe link properties with pcie_print_link_status() > PCI: Remove unused pcie_get_minimum_link() > > > drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c | 23 ++----- > drivers/net/ethernet/broadcom/bnxt/bnxt.c | 19 ------ > drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c | 75 ---------------------- > drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 47 -------------- > drivers/pci/pci.c | 43 ------------- > include/linux/pci.h | 2 - > 6 files changed, 9 insertions(+), 200 deletions(-) I applied all of these on pci/enumeration for v4.18. If you'd rather take them, Dave, let me know and I'll drop them. I solicited more acks, but only heard from Jeff.