From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?B?0JzQvtGA0L7QtyDQntC70LXQsw==?= Subject: Re: [PATCH v2 0/3] PCI: Revert "Implement pcibios_alloc_irq() and pcibios_free_irq()" Date: Thu, 18 Feb 2016 08:13:41 +0300 Message-ID: <56C55305.2050404@mcc.vniiem.ru> References: <20160217182718.302.10272.stgit@bhelgaas-glaptop2.roam.corp.google.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <20160217182718.302.10272.stgit@bhelgaas-glaptop2.roam.corp.google.com> Sender: linux-kernel-owner@vger.kernel.org To: Bjorn Helgaas , linux-pci@vger.kernel.org Cc: Joerg Roedel , "Rafael J. Wysocki" , linux-kernel@vger.kernel.org, Sunjin Yang , linux-acpi@vger.kernel.org, Thomas Gleixner , Yinghai Lu , Jiang Liu List-Id: linux-acpi@vger.kernel.org On 17.02.2016 21:36, Bjorn Helgaas wrote: > These are clean reverts of 991de2e59090 ("PCI, x86: Implement > pcibios_alloc_irq() and pcibios_free_irq()") and related changes. > > We have two reports of drivers broken in v4.3 by Jiang's change, > 991de2e59090 ("PCI, x86: Implement pcibios_alloc_irq() and > pcibios_free_irq()"). > > It would be best to fix the problem instead of reverting 991de2e59090= , but > I don't have time to do that myself, and we haven't heard from Jiang,= so > the only choice I have is to revert the commit. > > Note that reverting will likely break IOAPIC hotplug. > > =D0=9E=D0=BB=D0=B5=D0=B3 and Sunjin, can you please test this and see= whether it fixes your > drivers? This patch is based on v4.5-rc1. > > I intend to merge these for v4.5 unless I hear objections. > > They're on my for-linus branch, so they should appear in linux-next > tomorrow. > > Changes from the v1 posting: > - Explicitly revert related patches instead of resolving conflicts > manually > --- > > Bjorn Helgaas (3): > Revert "x86/PCI: Don't alloc pcibios-irq when MSI is enabled" > Revert "PCI: Add helpers to manage pci_dev->irq and pci_dev->i= rq_managed" > Revert "PCI, x86: Implement pcibios_alloc_irq() and pcibios_fr= ee_irq()" > > > arch/x86/include/asm/pci_x86.h | 2 ++ > arch/x86/pci/common.c | 26 ++++++++++---------------- > arch/x86/pci/intel_mid_pci.c | 9 +++------ > arch/x86/pci/irq.c | 23 +++++++++++++++++++---- > drivers/acpi/pci_irq.c | 17 +++++++++++++---- > include/linux/pci.h | 17 ----------------- > 6 files changed, 47 insertions(+), 47 deletions(-) As i undestand this patch will appear in kernel git repo tomorrow? I=20 don't need to apply in manyally to v4.5-rc1?