From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiang Liu Subject: Re: [PATCH v3, part1 04/10] PCI: mark pci_scan_bus_parented() as __deprecated Date: Fri, 07 Jun 2013 23:30:11 +0800 Message-ID: <51B1FC83.7090402@gmail.com> References: <1369489718-25869-1-git-send-email-jiang.liu@huawei.com> <1369489718-25869-5-git-send-email-jiang.liu@huawei.com> <51B0B99C.4020009@gmail.com> <20130607143758.GA13427@phenom.dumpdata.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <20130607143758.GA13427@phenom.dumpdata.com> Sender: linux-kernel-owner@vger.kernel.org To: Konrad Rzeszutek Wilk Cc: Bjorn Helgaas , Yinghai Lu , Jiang Liu , "Rafael J . Wysocki" , Greg Kroah-Hartman , Gu Zheng , Toshi Kani , Myron Stowe , Yijing Wang , "linux-pci@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Chris Metcalf , Thierry Reding , Jeremy Fitzhardinge , xen-devel List-Id: xen-devel@lists.xenproject.org On 06/07/2013 10:37 PM, Konrad Rzeszutek Wilk wrote: > On Fri, Jun 07, 2013 at 12:32:28AM +0800, Jiang Liu wrote: >> On Thu 06 Jun 2013 04:04:36 AM CST, Bjorn Helgaas wrote: >>> [+cc Konrad, Jeremy, xen-devel (users of interface you're deprecati= ng)] >>> >>> On Sat, May 25, 2013 at 7:48 AM, Jiang Liu wrote= : >>>> Mark pci_scan_bus_parented() as __deprecated and clean up outdated >>>> comments. >>>> >>>> Signed-off-by: Jiang Liu >>>> Cc: Chris Metcalf >>>> Cc: Greg Kroah-Hartman >>>> Cc: Thierry Reding >>>> Cc: linux-kernel@vger.kernel.org >>>> Cc: linux-pci@vger.kernel.org >>>> --- >>>> arch/tile/kernel/pci.c | 3 --- >>>> include/linux/pci.h | 4 ++-- >>>> 2 files changed, 2 insertions(+), 5 deletions(-) >>>> >>>> diff --git a/arch/tile/kernel/pci.c b/arch/tile/kernel/pci.c >>>> index 67237d3..936e087 100644 >>>> --- a/arch/tile/kernel/pci.c >>>> +++ b/arch/tile/kernel/pci.c >>>> @@ -309,9 +309,6 @@ int __init pcibios_init(void) >>>> * >>>> * It reads the PCI tree for this bus into= the Linux >>>> * data structures. >>>> - * >>>> - * This is inlined in linux/pci.h and call= s into >>>> - * pci_scan_bus_parented() in probe.c. >>>> */ >>>> pci_add_resource(&resources, &ioport_resou= rce); >>>> pci_add_resource(&resources, &iomem_resour= ce); >>>> diff --git a/include/linux/pci.h b/include/linux/pci.h >>>> index b0f4a82..7b23fa0 100644 >>>> --- a/include/linux/pci.h >>>> +++ b/include/linux/pci.h >>>> @@ -720,8 +720,8 @@ void pcibios_bus_to_resource(struct pci_dev *d= ev, struct resource *res, >>>> void pcibios_scan_specific_bus(int busn); >>>> struct pci_bus *pci_find_bus(int domain, int busnr); >>>> void pci_bus_add_devices(const struct pci_bus *bus); >>>> -struct pci_bus *pci_scan_bus_parented(struct device *parent, int = bus, >>>> - struct pci_ops *ops, void *s= ysdata); >>>> +struct pci_bus * __deprecated pci_scan_bus_parented(struct device= *parent, >>>> + int bus, struct pci_ops *ops, void *sysdat= a); >>>> struct pci_bus *pci_scan_bus(int bus, struct pci_ops *ops, void *= sysdata); >>>> struct pci_bus *pci_create_root_bus(struct device *parent, int bu= s, >>>> struct pci_ops *ops, void *sys= data, >>> >>> I think this patch is a good idea, but I think we need to change th= e >>> only existing user of pci_scan_bus_parented() (pcifront_scan_root()= in >>> drivers/pci/xen-pcifront.c) at the same time, so xen doesn't start >>> getting build warnings. The deprecation warnings are really intend= ed >>> for out-of-tree users that we aren't able to fix ourselves. >>> >>> I'm getting this series queued up in >>> http://git.kernel.org/cgit/linux/kernel/git/helgaas/pci.git/log/?h=3D= pci/jiang-bus-lock-v3, >>> so if you want to send a patch just for xen, I can fold that in. >>> >>> Bjorn >> Hi Bjorn, >> I have posted a big patch for that, but still need help from= =20 >> Xen experts to review it. >> Please refer to https://patchwork.kernel.org/patch/2578551/ >=20 > Hm, I seem to get: >=20 > /home/konrad/linux/drivers/pci/xen-pcifront.c: In function =E2=80=98p= cifront_free_roots=E2=80=99: > /home/konrad/linux/drivers/pci/xen-pcifront.c:559: error: implicit de= claration of function =E2=80=98for_each_pci_root_bus=E2=80=99 > /home/konrad/linux/drivers/pci/xen-pcifront.c:559: error: expected =E2= =80=98;=E2=80=99 before =E2=80=98{=E2=80=99 token > /home/konrad/linux/drivers/pci/xen-pcifront.c:554: warning: unused va= riable =E2=80=98sd=E2=80=99 >=20 > with it? Is there an up-to-date patch? > I have these in my tree: >=20 > 311db40 PCI, xen-pcifront: use new PCI interfaces to simplify impleme= ntation > 4079fee Merge remote-tracking branch 'bjorn/pci/jiang-bus-lock-v3' in= to testing > ddb7c6b PCI: Hide remove and rescan sysfs interfaces for SR-IOV virtu= al functions > f5cfa3a PCI: Simplify IOV implementation and fix reference count race= s > 28b4f07 PCI: Drop redundant setting of bus->is_added in virtfn_add_bu= s() > 442655a unicore32/PCI: Remove redundant call of pci_bus_add_devices() > defd601 m68k/PCI: Remove redundant call of pci_bus_add_devices() > 340e3fb PCI: Make PCI host bridge/bus creating and destroying logic s= ymmetric > c7025f7 ia64/PCI: Clean up pci_scan_root_bus() usage > c1f41ce PCI: Mark pci_scan_bus_parented() as __deprecated > 13e5057 PCI: Convert alloc_pci_dev(void) to pci_alloc_dev(bus) > 007042a PCI: Introduce pci_alloc_dev(struct pci_bus*) to replace allo= c_pci_dev() > fe830ef PCI: Introduce pci_bus_{get|put}() to manage PCI bus referenc= e count Hi Konrad, Could you please help to apply this simple patch onto to your tree? It should fix the build failure issue. diff --git a/drivers/pci/xen-pcifront.c b/drivers/pci/xen-pcifront.c index 79ad229..b5e0e66 100644 --- a/drivers/pci/xen-pcifront.c +++ b/drivers/pci/xen-pcifront.c @@ -556,7 +556,7 @@ static void pcifront_free_roots(struct pcifront_device *pdev) dev_dbg(&pdev->xdev->dev, "cleaning up root buses\n"); - for_each_pci_root_bus(bus) { + list_for_each_entry(bus, &pci_root_buses, node) { sd =3D bus->sysdata; if (sd->pdev =3D=3D pdev) { pci_stop_root_bus(bus); >=20 > Also oddly enough your SoB does not match your Author. Here is > what git shows: >=20 > =3D=3D=3D > Author: Jiang Liu 2013-05-25 09:48:38 > =20 > =3D=3D=3D > Signed-off-by: Jiang Liu >=20 > You can add in the gmail an new 'email alias' so that the emails will > look as they come from your huawei address. >=20