From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f52.google.com (mail-pj1-f52.google.com [209.85.216.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3CEE518A92D for ; Tue, 7 Jan 2025 22:14:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736288086; cv=none; b=c44W5AFF2GSAeQS/wISGfBcihp9E7Vw1ilVqhGb5dQuwo/rUXYE62P3rgRclO/9gpJB+nR9mOjbI0yKrnoJ0Yv55jrViE1PfzmwS9MCJdCpNHb0wX3eEgySrr+64iFYnIEN1wk9stRddMNs3qZnQi1augxdW9/KYnRqxfiDhaHw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736288086; c=relaxed/simple; bh=Q7gsPmAwcKW+iCD5IVNi31ZUXWQBjg1I4E0jayTr9Dk=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=hpdU82aiMNqyPWqcgdNIWaqcbHMKvp20Vd8oc4sHIMp7sZfWXebyKGLEUvLotLcJVO6p1+sUDd7/+90ICRf+QTeQ4JcFSkQ9nv6WzpkB/icWmhq0fRcvsXsk6qh6nj+lHp1itNAtw0jNPpDrWz//fS2H9JJZEzd2awoQLFQ4Vi0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org; spf=pass smtp.mailfrom=chromium.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b=lFOYiXlE; arc=none smtp.client-ip=209.85.216.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=chromium.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b="lFOYiXlE" Received: by mail-pj1-f52.google.com with SMTP id 98e67ed59e1d1-2ee989553c1so23264605a91.3 for ; Tue, 07 Jan 2025 14:14:45 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; t=1736288084; x=1736892884; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=RV+4e9KhhM+feBvIE88GxANGaUR7qFryxHJqEAou5R8=; b=lFOYiXlEDbzSieWXPm/49QCQUL0R+ksdaM+8gQl1rc8uKy6gjUakUiagavB+G3l9le /VzJC4ECyl2HpJacfH2ZzrtesaZBbdUSv9Dcqdcjg95TqECYwrxPcnvHDer1wVujXcyy b9cydzzwJvKwz6gyWhT4NhdwMKojwajedu0Q4= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1736288084; x=1736892884; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=RV+4e9KhhM+feBvIE88GxANGaUR7qFryxHJqEAou5R8=; b=HVr50MXt9CdXNL0chKZaEDIZ5mltdRj8i9X761AUTssJUUR+6kEZFMtArguPHn1h9I BPAIZlrK5cvRsPGAWJCOrcCW++Hs2OHTofSLQqqMNnolIwp6BOUMrHtXS6daCaPQtUqa hQNs8VBriBIPuHXFRrVr41xpitMMzrhF73xWfYRkHXTERbN2NhSoCDWfSK2+LDy/p2LS GLWUZtfHBwRdy9GgZRxteGBaBCcVKs4/AEpzhaQt00TAiABg62h5XUHY5pLuYMGbhatr RKTq8IAV3UGkutExtB/d7wopqhBatWpQPKwcOF20u3a6WLY4HE4qZJBEkZ8cJr0qifnU E0WQ== X-Forwarded-Encrypted: i=1; AJvYcCWRr4K2iufIfLIEP1rP3prPrBxqejr7XM8KaZh0k49HMiuohyIishCnPg5vs45OkAQ6hCHdQYqkYe4=@vger.kernel.org X-Gm-Message-State: AOJu0YxHkcqyEBlbqHsJMvwXyOFuKjDaaVNU+Wl3YwNPYcY8A5lzS2qo FuVNDL+m0cdwcIo8199Ivx8Y3OB/tUVAk/D4CGWneMzr735rSJqS+NTNSEjkbA== X-Gm-Gg: ASbGncuORokVUxt2bammvqvKsCTO1Zrx8lsmP0hviLVzQh9CYo1s3XsfcFLGv3Chyf3 axGKhbjIrGPiwxlQBvWOE5SU/QPZGl/U0/trsTcPYkg63PzBdO1IXjMat00wSWYg8MgMnmfkD5I 6Fr2Xhvui8vo8HXw1lQ3kAt0h1TY0JkgBwfztmh7RpUGVP83v5q/yPGoi6PubbtqCt9qmqsXbOQ r9bY6W2i/1Ro4U+iI4FPmsfryL+jN/eDnCMDtTR00VXpjrNfgMrIssY/ntTJCL7VvcarXp1CSQS +gz8en83GGWfJogkYc8= X-Google-Smtp-Source: AGHT+IEkAisXGBz/e/a8gkUWJb8Iue3ft+92hn87eXDD4Dkl25YTjIs6OX+OKDfqwZwQcXi/uW2Pcg== X-Received: by 2002:a05:6a00:ad4:b0:725:ebab:bb32 with SMTP id d2e1a72fcca58-72d2201be2amr783291b3a.26.1736288084620; Tue, 07 Jan 2025 14:14:44 -0800 (PST) Received: from localhost ([2a00:79e0:2e14:7:183c:e247:20b9:87a9]) by smtp.gmail.com with UTF8SMTPSA id d2e1a72fcca58-72aad8fd7b0sm23040661b3a.139.2025.01.07.14.14.43 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 07 Jan 2025 14:14:44 -0800 (PST) Date: Tue, 7 Jan 2025 14:14:42 -0800 From: Brian Norris To: Bjorn Helgaas , "Rafael J. Wysocki" , Len Brown Cc: Hsin-Yi Wang , linux-kernel@vger.kernel.org, mika.westerberg@linux.intel.com, linux-acpi@vger.kernel.org, linux-pci@vger.kernel.org, lukas@wunner.de, Manivannan Sadhasivam Subject: Re: [PATCH v5] PCI: Allow PCI bridges to go to D3Hot on all Devicetree based platforms Message-ID: References: <20241126151711.v5.1.Id0a0e78ab0421b6bce51c4b0b87e6aebdfc69ec7@changeid> Precedence: bulk X-Mailing-List: linux-pci@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20241126151711.v5.1.Id0a0e78ab0421b6bce51c4b0b87e6aebdfc69ec7@changeid> Hi Bjorn, On Tue, Nov 26, 2024 at 03:17:11PM -0800, Brian Norris wrote: > From: Manivannan Sadhasivam > > Unlike ACPI based platforms, there are no known issues with D3Hot for > the PCI bridges in Device Tree based platforms. Past discussions (Link > [1]) determined the restrictions around D3 should be relaxed for all > Device Tree systems. So let's allow the PCI bridges to go to D3Hot > during runtime. > > To match devm_pci_alloc_host_bridge() -> devm_of_pci_bridge_init(), we > look at the host bridge's parent when determining whether this is a > Device Tree based platform. Not all bridges have their own node, but the > parent (controller) should. > > Link: https://lore.kernel.org/linux-pci/20240227225442.GA249898@bhelgaas/ [1] > Link: https://lore.kernel.org/linux-pci/20240828210705.GA37859@bhelgaas/ [2] > Signed-off-by: Manivannan Sadhasivam > [Brian: look at host bridge's parent, not bridge node; rewrite > description] > Signed-off-by: Brian Norris > --- > Based on prior work by Manivannan Sadhasivam that was part of a bigger > series that stalled: > > [PATCH v5 4/4] PCI: Allow PCI bridges to go to D3Hot on all Devicetree based platforms > https://lore.kernel.org/linux-pci/20240802-pci-bridge-d3-v5-4-2426dd9e8e27@linaro.org/ > > I'm resubmitting this single patch, since it's useful and seemingly had > agreement. I massaged it a bit to relax some restrictions on how the > Device Tree should look. > > Changes in v5: > - Pulled out of the larger series, as there were more controversial > changes in there, while this one had agreement (Link [2]). > - Rewritten with a relaxed set of rules, because the above patch > required us to modify many device trees to add bridge nodes. I'm wondering if you have any thoughts on this. Manivannan seemed happy with this in his reply. I'd like to see this land in mainline, so I can avoid the hacks that everyone seems to be picking up (such as adding "pcie_port_pm=force" to their command lines). (While I'm at it ... apologies for the poor versioning. The subject here should probably be "v6", since I'm clearly quoting above that the prior version from Manivannan was v5.) Thanks, Brian