[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <35df30c5-5066-4933-b414-1f6c918a20e3@paulmck-laptop>
Date: Thu, 8 May 2025 08:14:46 -0700
From: "Paul E. McKenney" <paulmck@...nel.org>
To: Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>
Cc: Bjorn Helgaas <bhelgaas@...gle.com>, Lukas Wunner <lukas@...ner.de>,
linux-pci@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/1] PCI/bwctrl: Remove also pcie_bwctrl_lbms_rwsem
On Thu, May 08, 2025 at 12:00:36PM +0300, Ilpo Järvinen wrote:
> The commit 0238f352a63a ("PCI/bwctrl: Replace lbms_count with
> PCI_LINK_LBMS_SEEN flag") remove all code related to
> pcie_bwctrl_lbms_rwsem but forgot to remove the rwsem itself.
> Remove it and the associated info from the comment now.
>
> Fixes: 0238f352a63a ("PCI/bwctrl: Replace lbms_count with PCI_LINK_LBMS_SEEN flag")
> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>
Tested-by: Paul E. McKenney <paulmck@...nel.org>
> ---
>
> Bjorn, this should be folded into the original commit I think.
>
> drivers/pci/pcie/bwctrl.c | 10 +---------
> 1 file changed, 1 insertion(+), 9 deletions(-)
>
> diff --git a/drivers/pci/pcie/bwctrl.c b/drivers/pci/pcie/bwctrl.c
> index fdafa20e4587..f31fbbd51490 100644
> --- a/drivers/pci/pcie/bwctrl.c
> +++ b/drivers/pci/pcie/bwctrl.c
> @@ -45,15 +45,7 @@ struct pcie_bwctrl_data {
> struct thermal_cooling_device *cdev;
> };
>
> -/*
> - * Prevent port removal during LBMS count accessors and Link Speed changes.
> - *
> - * These have to be differentiated because pcie_bwctrl_change_speed() calls
> - * pcie_retrain_link() which uses LBMS count reset accessor on success
> - * (using just one rwsem triggers "possible recursive locking detected"
> - * warning).
> - */
> -static DECLARE_RWSEM(pcie_bwctrl_lbms_rwsem);
> +/* Prevent port removal during Link Speed changes. */
> static DECLARE_RWSEM(pcie_bwctrl_setspeed_rwsem);
>
> static bool pcie_valid_speed(enum pci_bus_speed speed)
>
> base-commit: 0238f352a63a075ac1f35ea565b5bec3057ec8bd
> --
> 2.39.5
>
Powered by blists - more mailing lists