From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750879AbeEJWvf (ORCPT ); Thu, 10 May 2018 18:51:35 -0400 Received: from mail.kernel.org ([198.145.29.99]:55136 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750711AbeEJWve (ORCPT ); Thu, 10 May 2018 18:51:34 -0400 Date: Thu, 10 May 2018 17:51:32 -0500 From: Bjorn Helgaas To: Jan Kiszka Cc: Bjorn Helgaas , Linux Kernel Mailing List , linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Jingoo Han , Joao Pinto , Lorenzo Pieralisi , Will Deacon Subject: Re: [PATCH v2 00/10] PCI: leak fixes, removable generic PCI host, assorted stuff Message-ID: <20180510225132.GD190385@bhelgaas-glaptop.roam.corp.google.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: 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 On Mon, Apr 30, 2018 at 07:48:34AM +0200, Jan Kiszka wrote: > Changes in v2: > - patch 1: commit message reworking as suggested by Lorenzo > - patch 3-6: split-up as suggested by Bjorn > - patch 8: new > - patch 10: select PCI_DOMAINS from PCI_HOST_GENERIC, rather than > allowing manual choice, as suggested by Lorenzo > > This primarily enables to unbind the generic PCI host controller without > leaving lots of memory leaks behind. A previous proposal patch 5 was > rejected because of those issues [1]. > > The fixes have been validated in the Jailhouse setup, where we add and > remove a virtual PCI host controller on hypervisor activation/ > deactivation, with the help of kmemleak. > > Besides that, there is tiny PCI API cleanup at the beginning and > support for manually enabled PCI domains at the end that enables the > Jailhouse scenario. > > Jan > > [1] http://lkml.iu.edu/hypermail/linux/kernel/1606.3/00072.html > > > CC: Jingoo Han > CC: Joao Pinto > CC: Lorenzo Pieralisi > CC: Will Deacon > > Jan Kiszka (10): > PCI: Make pci_get_new_domain_nr() static > PCI: Fix memory leak of devm_pci_alloc_host_bridge() > PCI: Factor out __of_pci_get_host_bridge_resources() > PCI: Add dev parameter to __of_pci_get_host_bridge_resources() > PCI: Replace pr_*() with dev_*() in > __of_pci_get_host_bridge_resources() > PCI: Introduce devm_of_pci_get_host_bridge_resources() > PCI: Convert of_pci_get_host_bridge_resources() users to devm variant > PCI: Deprecate of_pci_get_host_bridge_resources() > PCI: Add support for unbinding the generic PCI host controller > PCI: Enable PCI_DOMAINS along with generic PCI host controller Applied to pci/resource for v4.18, thanks! I added the reviewed-by tags from Vladimir (thank you!) and replaced the "Deprecate" patch by the v3 "Remove" patch. > drivers/pci/dwc/pcie-designware-host.c | 2 +- > drivers/pci/host/Kconfig | 1 + > drivers/pci/host/pci-aardvark.c | 5 +- > drivers/pci/host/pci-ftpci100.c | 4 +- > drivers/pci/host/pci-host-common.c | 13 ++++ > drivers/pci/host/pci-host-generic.c | 1 + > drivers/pci/host/pci-v3-semi.c | 3 +- > drivers/pci/host/pci-versatile.c | 3 +- > drivers/pci/host/pci-xgene.c | 3 +- > drivers/pci/host/pcie-altera.c | 5 +- > drivers/pci/host/pcie-iproc-platform.c | 4 +- > drivers/pci/host/pcie-rcar.c | 5 +- > drivers/pci/host/pcie-rockchip.c | 4 +- > drivers/pci/host/pcie-xilinx-nwl.c | 4 +- > drivers/pci/host/pcie-xilinx.c | 4 +- > drivers/pci/of.c | 105 +++++++++++++++++++++------------ > drivers/pci/pci.c | 6 +- > drivers/pci/probe.c | 4 +- > include/linux/of_pci.h | 42 ++++++++++++- > include/linux/pci-ecam.h | 1 + > include/linux/pci.h | 3 - > 21 files changed, 149 insertions(+), 73 deletions(-) > > -- > 2.13.6 >