Re: eata fails to load on post 4.2 kernels
Jiang Liu wrote on 10/09/15 17:43: Hi Authur, Thanks for the updating. Seem Bjorn doesn't like neither of my two patches. So I'm trying to convert eata to formal PCI driver, but the change will be much more bigger and still not sure whether we could achieve that. Will keep you updated. Thanks! Gerry Thanks, I'm a bit concerned since the original commit 991de2e59090e55c65a7f59a049142e3c480f7bd broke things badly for me (requiring irqpoll to avoid a kernel hang) and neither of the patches enabled kexec reboots to work like before the original commit. I just tested a kexec reboot with irqpoll enabled and that continues to fail, so I'm back to running 4.2 kernel until there is a patch that works with kexec reboots. Arthur. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: eata fails to load on post 4.2 kernels
Hi Authur, Thanks for the updating. Seem Bjorn doesn't like neither of my two patches. So I'm trying to convert eata to formal PCI driver, but the change will be much more bigger and still not sure whether we could achieve that. Will keep you updated. Thanks! Gerry On 2015/9/10 15:50, Arthur Marsh wrote: > > > Jiang Liu wrote on 08/09/15 14:49: >> Hi Auhur, >> Could you please help to apply the test patch >> against the latest mainstream linux kernel? >> Thanks! >> Gerry > ... > > git bisect good > 991de2e59090e55c65a7f59a049142e3c480f7bd is the first bad commit > commit 991de2e59090e55c65a7f59a049142e3c480f7bd > Author: Jiang Liu > Date: Wed Jun 10 16:54:59 2015 +0800 > > PCI, x86: Implement pcibios_alloc_irq() and pcibios_free_irq() > > To support IOAPIC hotplug, we need to allocate PCI IRQ > resources on > demand > and free them when not used anymore. > > Implement pcibios_alloc_irq() and pcibios_free_irq() to > dynamically > allocate and free PCI IRQs. > > Remove mp_should_keep_irq(), which is no longer used. > > [bhelgaas: changelog] > Signed-off-by: Jiang Liu > Signed-off-by: Bjorn Helgaas > Acked-by: Thomas Gleixner > > :04 04 765e2d5232d53247ec260b34b51589c3bccb36ae > f680234a27685e94b1a35ae2a7218f8eafa9071a M arch > :04 04 d55a682bcde72682e883365e88ad1df6186fd54d > f82c470a04a6845fcf5e0aa934512c75628f798d M drivers > > I tried to do a kexec shut-down with the first version of your patch: > >>From 3085626fb2e677c1d88f158397948935b73f5239 Mon Sep 17 00:00:00 2001 > From: Jiang Liu > Date: Tue, 8 Sep 2015 10:41:19 +0800 > Subject: [PATCH] > > > Signed-off-by: Jiang Liu > --- > drivers/pci/pci-driver.c |1 + > drivers/scsi/eata.c |2 ++ > 2 files changed, 3 insertions(+) > > diff --git a/drivers/pci/pci-driver.c b/drivers/pci/pci-driver.c > index 52a880ca1768..17d2a0b1de18 100644 > --- a/drivers/pci/pci-driver.c > +++ b/drivers/pci/pci-driver.c > @@ -392,6 +392,7 @@ int __weak pcibios_alloc_irq(struct pci_dev *dev) > { > return 0; > } > +EXPORT_SYMBOL_GPL(pcibios_alloc_irq); > > void __weak pcibios_free_irq(struct pci_dev *dev) > { > diff --git a/drivers/scsi/eata.c b/drivers/scsi/eata.c > index 227dd2c2ec2f..7e6eaf867987 100644 > --- a/drivers/scsi/eata.c > +++ b/drivers/scsi/eata.c > @@ -1061,6 +1061,7 @@ static void enable_pci_ports(void) > driver_name, dev->bus->number, dev->devfn); > #endif > > +pcibios_alloc_irq(dev); > if (pci_enable_device(dev)) > printk > ("%s: warning, pci_enable_device failed, bus %d devfn > 0x%x.\n", > @@ -1520,6 +1521,7 @@ static void add_pci_ports(void) > if (!(dev = pci_get_class(PCI_CLASS_STORAGE_SCSI << 8, dev))) > break; > > +pcibios_alloc_irq(dev); > if (pci_enable_device(dev)) { > #if defined(DEBUG_PCI_DETECT) > printk -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: eata fails to load on post 4.2 kernels
Jiang Liu wrote on 08/09/15 14:49: Hi Auhur, Could you please help to apply the test patch against the latest mainstream linux kernel? Thanks! Gerry ... git bisect good 991de2e59090e55c65a7f59a049142e3c480f7bd is the first bad commit commit 991de2e59090e55c65a7f59a049142e3c480f7bd Author: Jiang Liu Date: Wed Jun 10 16:54:59 2015 +0800 PCI, x86: Implement pcibios_alloc_irq() and pcibios_free_irq() To support IOAPIC hotplug, we need to allocate PCI IRQ resources on demand and free them when not used anymore. Implement pcibios_alloc_irq() and pcibios_free_irq() to dynamically allocate and free PCI IRQs. Remove mp_should_keep_irq(), which is no longer used. [bhelgaas: changelog] Signed-off-by: Jiang Liu Signed-off-by: Bjorn Helgaas Acked-by: Thomas Gleixner :04 04 765e2d5232d53247ec260b34b51589c3bccb36ae f680234a27685e94b1a35ae2a7218f8eafa9071a M arch :04 04 d55a682bcde72682e883365e88ad1df6186fd54d f82c470a04a6845fcf5e0aa934512c75628f798d M drivers I tried to do a kexec shut-down with the first version of your patch: >From 3085626fb2e677c1d88f158397948935b73f5239 Mon Sep 17 00:00:00 2001 From: Jiang Liu Date: Tue, 8 Sep 2015 10:41:19 +0800 Subject: [PATCH] Signed-off-by: Jiang Liu --- drivers/pci/pci-driver.c |1 + drivers/scsi/eata.c |2 ++ 2 files changed, 3 insertions(+) diff --git a/drivers/pci/pci-driver.c b/drivers/pci/pci-driver.c index 52a880ca1768..17d2a0b1de18 100644 --- a/drivers/pci/pci-driver.c +++ b/drivers/pci/pci-driver.c @@ -392,6 +392,7 @@ int __weak pcibios_alloc_irq(struct pci_dev *dev) { return 0; } +EXPORT_SYMBOL_GPL(pcibios_alloc_irq); void __weak pcibios_free_irq(struct pci_dev *dev) { diff --git a/drivers/scsi/eata.c b/drivers/scsi/eata.c index 227dd2c2ec2f..7e6eaf867987 100644 --- a/drivers/scsi/eata.c +++ b/drivers/scsi/eata.c @@ -1061,6 +1061,7 @@ static void enable_pci_ports(void) driver_name, dev->bus->number, dev->devfn); #endif + pcibios_alloc_irq(dev); if (pci_enable_device(dev)) printk ("%s: warning, pci_enable_device failed, bus %d devfn 0x%x.\n", @@ -1520,6 +1521,7 @@ static void add_pci_ports(void) if (!(dev = pci_get_class(PCI_CLASS_STORAGE_SCSI << 8, dev))) break; + pcibios_alloc_irq(dev); if (pci_enable_device(dev)) { #if defined(DEBUG_PCI_DETECT) printk -- 1.7.10.4 but I experience identical kexec shutdown and restart problems as with the second version of your patch, as seen here: http://www.users.on.net/~arthur.marsh/20150910541.jpg the original commit 991de2e59090e55c65a7f59a049142e3c480f7bd quoted above seems to have not only lead to start-up problems unless irqpoll was enabled but also lead to kexec shutdown/restart problems. I'm not sure what the solution is but it is good to continue to allow kexec reboots to work. Arthur. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: eata fails to load on post 4.2 kernels
On 2015/9/8 14:32, Arthur Marsh wrote: > > > Jiang Liu wrote on 08/09/15 14:49: >> Hi Auhur, >> Could you please help to apply the test patch >> against the latest mainstream linux kernel? >> Thanks! >> Gerry > > Done, and it appears to work properly thanks! Hi Authur, We just got the root cause, and still need to work out a formal patch to fix it. So will send new formal patch to you for testing:) Thanks! Gerry > > Arthur. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: eata fails to load on post 4.2 kernels
Jiang Liu wrote on 08/09/15 14:49: Hi Auhur, Could you please help to apply the test patch against the latest mainstream linux kernel? Thanks! Gerry Done, and it appears to work properly thanks! Arthur. [0.00] Initializing cgroup subsys cpuset [0.00] Initializing cgroup subsys cpu [0.00] Initializing cgroup subsys cpuacct [0.00] Linux version 4.2.0+ (root@victoria) (gcc version 5.2.1 20150903 (Debian 5.2.1-16) ) #30 SMP PREEMPT Tue Sep 8 15:10:49 ACST 2015 [0.00] x86/fpu: Legacy x87 FPU detected. [0.00] x86/fpu: Using 'lazy' FPU context switches. [0.00] e820: BIOS-provided physical RAM map: [0.00] BIOS-e820: [mem 0x-0x0009fbff] usable [0.00] BIOS-e820: [mem 0x0009fc00-0x0009] reserved [0.00] BIOS-e820: [mem 0x000e-0x000f] reserved [0.00] BIOS-e820: [mem 0x0010-0x3ffa] usable [0.00] BIOS-e820: [mem 0x3ffb-0x3ffbdfff] ACPI data [0.00] BIOS-e820: [mem 0x3ffbe000-0x3ffd] ACPI NVS [0.00] BIOS-e820: [mem 0x3ffe-0x3fff] reserved [0.00] BIOS-e820: [mem 0xe000-0xefff] reserved [0.00] BIOS-e820: [mem 0xfec0-0xfec00fff] reserved [0.00] BIOS-e820: [mem 0xff78-0x] reserved [0.00] Notice: NX (Execute Disable) protection cannot be enabled: non-PAE kernel! [0.00] SMBIOS 2.3 present. [0.00] DMI: System manufacturer System Product Name/A8V-MX, BIOS 0503 12/06/2005 [0.00] e820: update [mem 0x-0x0fff] usable ==> reserved [0.00] e820: remove [mem 0x000a-0x000f] usable [0.00] e820: last_pfn = 0x3ffb0 max_arch_pfn = 0x10 [0.00] MTRR default type: uncachable [0.00] MTRR fixed ranges enabled: [0.00] 0-9 write-back [0.00] A-E uncachable [0.00] F-F write-protect [0.00] MTRR variable ranges enabled: [0.00] 0 base 00 mask FFC000 write-back [0.00] 1 base 00D000 mask FFF000 write-combining [0.00] 2 disabled [0.00] 3 disabled [0.00] 4 disabled [0.00] 5 disabled [0.00] 6 disabled [0.00] 7 disabled [0.00] x86/PAT: Configuration [0-7]: WB WC UC- UC WB WC UC- WT [0.00] found SMP MP-table at [mem 0x000ff780-0x000ff78f] mapped at [c00ff780] [0.00] initial memory mapped: [mem 0x-0x023f] [0.00] Base memory trampoline at [c009b000] 9b000 size 16384 [0.00] init_memory_mapping: [mem 0x-0x000f] [0.00] [mem 0x-0x000f] page 4k [0.00] init_memory_mapping: [mem 0x35c0-0x35ff] [0.00] [mem 0x35c0-0x35ff] page 4M [0.00] init_memory_mapping: [mem 0x0010-0x35bf] [0.00] [mem 0x0010-0x003f] page 4k [0.00] [mem 0x0040-0x35bf] page 4M [0.00] init_memory_mapping: [mem 0x3600-0x377fdfff] [0.00] [mem 0x3600-0x373f] page 4M [0.00] [mem 0x3740-0x377fdfff] page 4k [0.00] BRK [0x02075000, 0x02075fff] PGTABLE [0.00] RAMDISK: [mem 0x3614c000-0x3709dfff] [0.00] ACPI: Early table checksum verification disabled [0.00] ACPI: RSDP 0x000FAC60 24 (v02 ACPIAM) [0.00] ACPI: XSDT 0x3FFB0100 3C (v01 A M I OEMXSDT 12000506 MSFT 0097) [0.00] ACPI: FACP 0x3FFB0290 F4 (v03 A M I OEMFACP 12000506 MSFT 0097) [0.00] ACPI: DSDT 0x3FFB03F0 0046F0 (v01 A0347 A0347001 0001 INTL 02002026) [0.00] ACPI: FACS 0x3FFBE000 40 [0.00] ACPI: FACS 0x3FFBE000 40 [0.00] ACPI: APIC 0x3FFB0390 5C (v01 A M I OEMAPIC 12000506 MSFT 0097) [0.00] ACPI: OEMB 0x3FFBE040 46 (v01 A M I AMI_OEM 12000506 MSFT 0097) [0.00] ACPI: Local APIC address 0xfee0 [0.00] 135MB HIGHMEM available. [0.00] 887MB LOWMEM available. [0.00] mapped low ram: 0 - 377fe000 [0.00] low ram: 0 - 377fe000 [0.00] BRK [0x02076000, 0x02076fff] PGTABLE [0.00] Zone ranges: [0.00] DMA [mem 0x1000-0x00ff] [0.00] Normal [mem 0x0100-0x377fdfff] [0.00] HighMem [mem 0x377fe000-0x3ffa] [0.00] Movable zone start for each node [0.00] Early memory node ranges [0.00] node 0: [mem 0x1000-0x0009efff] [0.00] node 0: [mem 0x0010-0x3ffa] [0.00] Initmem setup node 0 [mem 0x1000-0x3ffa] [0.00] On node 0 totalpages: 261966 [0.00]
Re: eata fails to load on post 4.2 kernels
Hi Auhur, Could you please help to apply the test patch against the latest mainstream linux kernel? Thanks! Gerry On 2015/9/7 14:26, Arthur Marsh wrote: > > > Jiang Liu wrote on 07/09/15 12:36: >> On 2015/9/7 4:31, Arthur Marsh wrote: >>> Arthur Marsh wrote on 06/09/15 21:07: Arthur Marsh wrote on 06/09/15 18:34: > Arthur Marsh wrote on 06/09/15 15:58: >> Hi, I'm seeing the following on post 4.2 kernels, am currently >> bisecting >> to find where it started: First kernel in the bisection that worked without needing irqpoll: [ 73.751482] EATA0: IRQ 10 mapped to IO-APIC IRQ 17. [ 73.776711] EATA/DMA 2.0x: Copyright (C) 1994-2003 Dario Ballabio. [ 73.802005] EATA config options -> tm:1, lc:y, mq:16, rs:y, et:n, ip:n, ep:n, pp:y. [ 73.829175] EATA0: 2.0C, PCI 0xd890, IRQ 17, BMST, SG 122, MB 64. [ 73.82] EATA0: wide SCSI support enabled, max_id 16, max_lun 8. [ 73.881125] EATA0: SCSI channel 0 enabled, host target ID 7. >>> >>> After a git bisect, I get: >>> >>> git bisect good >>> 991de2e59090e55c65a7f59a049142e3c480f7bd is the first bad commit >>> commit 991de2e59090e55c65a7f59a049142e3c480f7bd >>> Author: Jiang Liu >>> Date: Wed Jun 10 16:54:59 2015 +0800 >>> >>> PCI, x86: Implement pcibios_alloc_irq() and pcibios_free_irq() >>> >>> To support IOAPIC hotplug, we need to allocate PCI IRQ resources on >>> demand >>> and free them when not used anymore. >>> >>> Implement pcibios_alloc_irq() and pcibios_free_irq() to dynamically >>> allocate and free PCI IRQs. >>> >>> Remove mp_should_keep_irq(), which is no longer used. >>> >>> [bhelgaas: changelog] >>> Signed-off-by: Jiang Liu >>> Signed-off-by: Bjorn Helgaas >>> Acked-by: Thomas Gleixner >>> >>> :04 04 765e2d5232d53247ec260b34b51589c3bccb36ae >>> f680234a27685e94b1a35ae2a7218f8eafa9071a M arch >>> :04 04 d55a682bcde72682e883365e88ad1df6186fd54d >>> f82c470a04a6845fcf5e0aa934512c75628f798d M drivers >>> >>> I'm happy to supply more details if needed. >> Hi Arthur, >> Thanks for reporting this. It seems to be an irq misrouting >> issue. Could you please help to provide: >> 1) full dmesg with the latest code >> 2) full dmesg and /proc/interrupts with the latest code and >> kernel parameter "irqpoll" specified >> Thanks! >> Gerry > > The pc locks up when loading the eata module so I've attached a photo of > the monitor screen. > > Arthur. > > >From 3085626fb2e677c1d88f158397948935b73f5239 Mon Sep 17 00:00:00 2001 From: Jiang Liu Date: Tue, 8 Sep 2015 10:41:19 +0800 Subject: [PATCH] Signed-off-by: Jiang Liu --- drivers/pci/pci-driver.c |1 + drivers/scsi/eata.c |2 ++ 2 files changed, 3 insertions(+) diff --git a/drivers/pci/pci-driver.c b/drivers/pci/pci-driver.c index 52a880ca1768..17d2a0b1de18 100644 --- a/drivers/pci/pci-driver.c +++ b/drivers/pci/pci-driver.c @@ -392,6 +392,7 @@ int __weak pcibios_alloc_irq(struct pci_dev *dev) { return 0; } +EXPORT_SYMBOL_GPL(pcibios_alloc_irq); void __weak pcibios_free_irq(struct pci_dev *dev) { diff --git a/drivers/scsi/eata.c b/drivers/scsi/eata.c index 227dd2c2ec2f..7e6eaf867987 100644 --- a/drivers/scsi/eata.c +++ b/drivers/scsi/eata.c @@ -1061,6 +1061,7 @@ static void enable_pci_ports(void) driver_name, dev->bus->number, dev->devfn); #endif + pcibios_alloc_irq(dev); if (pci_enable_device(dev)) printk ("%s: warning, pci_enable_device failed, bus %d devfn 0x%x.\n", @@ -1520,6 +1521,7 @@ static void add_pci_ports(void) if (!(dev = pci_get_class(PCI_CLASS_STORAGE_SCSI << 8, dev))) break; + pcibios_alloc_irq(dev); if (pci_enable_device(dev)) { #if defined(DEBUG_PCI_DETECT) printk -- 1.7.10.4
Fwd: Re: eata fails to load on post 4.2 kernels
Forwarding without image attachment to get below message size limit of the mailing lists. I've uploaded the image to: http://www.users.on.net/~arthur.marsh/20150907539.jpg Forwarded Message Subject: Re: eata fails to load on post 4.2 kernels Date: Mon, 07 Sep 2015 15:56:02 +0930 From: Arthur Marsh To: Jiang Liu CC: Bjorn Helgaas , t...@linutronix.de, linux-s...@vger.kernel.org, linux-kernel@vger.kernel.org Jiang Liu wrote on 07/09/15 12:36: On 2015/9/7 4:31, Arthur Marsh wrote: Arthur Marsh wrote on 06/09/15 21:07: Arthur Marsh wrote on 06/09/15 18:34: Arthur Marsh wrote on 06/09/15 15:58: Hi, I'm seeing the following on post 4.2 kernels, am currently bisecting to find where it started: First kernel in the bisection that worked without needing irqpoll: [ 73.751482] EATA0: IRQ 10 mapped to IO-APIC IRQ 17. [ 73.776711] EATA/DMA 2.0x: Copyright (C) 1994-2003 Dario Ballabio. [ 73.802005] EATA config options -> tm:1, lc:y, mq:16, rs:y, et:n, ip:n, ep:n, pp:y. [ 73.829175] EATA0: 2.0C, PCI 0xd890, IRQ 17, BMST, SG 122, MB 64. [ 73.82] EATA0: wide SCSI support enabled, max_id 16, max_lun 8. [ 73.881125] EATA0: SCSI channel 0 enabled, host target ID 7. After a git bisect, I get: git bisect good 991de2e59090e55c65a7f59a049142e3c480f7bd is the first bad commit commit 991de2e59090e55c65a7f59a049142e3c480f7bd Author: Jiang Liu Date: Wed Jun 10 16:54:59 2015 +0800 PCI, x86: Implement pcibios_alloc_irq() and pcibios_free_irq() To support IOAPIC hotplug, we need to allocate PCI IRQ resources on demand and free them when not used anymore. Implement pcibios_alloc_irq() and pcibios_free_irq() to dynamically allocate and free PCI IRQs. Remove mp_should_keep_irq(), which is no longer used. [bhelgaas: changelog] Signed-off-by: Jiang Liu Signed-off-by: Bjorn Helgaas Acked-by: Thomas Gleixner :04 04 765e2d5232d53247ec260b34b51589c3bccb36ae f680234a27685e94b1a35ae2a7218f8eafa9071a M arch :04 04 d55a682bcde72682e883365e88ad1df6186fd54d f82c470a04a6845fcf5e0aa934512c75628f798d M drivers I'm happy to supply more details if needed. Hi Arthur, Thanks for reporting this. It seems to be an irq misrouting issue. Could you please help to provide: 1) full dmesg with the latest code 2) full dmesg and /proc/interrupts with the latest code and kernel parameter "irqpoll" specified Thanks! Gerry The pc locks up when loading the eata module so I've attached a photo of the monitor screen. Arthur. [0.00] Initializing cgroup subsys cpuset [0.00] Initializing cgroup subsys cpu [0.00] Initializing cgroup subsys cpuacct [0.00] Linux version 4.2.0+ (root@victoria) (gcc version 5.2.1 20150903 (Debian 5.2.1-16) ) #29 SMP PREEMPT Mon Sep 7 07:10:45 ACST 2015 [0.00] x86/fpu: Legacy x87 FPU detected. [0.00] x86/fpu: Using 'lazy' FPU context switches. [0.00] e820: BIOS-provided physical RAM map: [0.00] BIOS-e820: [mem 0x-0x0009fbff] usable [0.00] BIOS-e820: [mem 0x0009fc00-0x0009] reserved [0.00] BIOS-e820: [mem 0x000e-0x000f] reserved [0.00] BIOS-e820: [mem 0x0010-0x3ffa] usable [0.00] BIOS-e820: [mem 0x3ffb-0x3ffbdfff] ACPI data [0.00] BIOS-e820: [mem 0x3ffbe000-0x3ffd] ACPI NVS [0.00] BIOS-e820: [mem 0x3ffe-0x3fff] reserved [0.00] BIOS-e820: [mem 0xe000-0xefff] reserved [0.00] BIOS-e820: [mem 0xfec0-0xfec00fff] reserved [0.00] BIOS-e820: [mem 0xff78-0x] reserved [0.00] Notice: NX (Execute Disable) protection cannot be enabled: non-PAE kernel! [0.00] SMBIOS 2.3 present. [0.00] DMI: System manufacturer System Product Name/A8V-MX, BIOS 0503 12/06/2005 [0.00] e820: update [mem 0x-0x0fff] usable ==> reserved [0.00] e820: remove [mem 0x000a-0x000f] usable [0.00] e820: last_pfn = 0x3ffb0 max_arch_pfn = 0x10 [0.00] MTRR default type: uncachable [0.00] MTRR fixed ranges enabled: [0.00] 0-9 write-back [0.00] A-E uncachable [0.00] F-F write-protect [0.00] MTRR variable ranges enabled: [0.00] 0 base 00 mask FFC000 write-back [0.00] 1 base 00D000 mask FFF000 write-combining [0.00] 2 disabled [0.00] 3 disabled [0.00] 4 disabled [0.00] 5 disabled [0.00] 6 disabled [0.00] 7 disabled [0.00] x86/PAT: Configuration [0-7]: WB WC UC- UC WB WC UC- WT [0.00] found SMP MP-table at [mem 0x000ff780-0x000ff78f] mapped at [c00ff780] [0.
Re: eata fails to load on post 4.2 kernels
On 2015/9/7 4:31, Arthur Marsh wrote: > Arthur Marsh wrote on 06/09/15 21:07: >> Arthur Marsh wrote on 06/09/15 18:34: >>> Arthur Marsh wrote on 06/09/15 15:58: Hi, I'm seeing the following on post 4.2 kernels, am currently bisecting to find where it started: >> >> First kernel in the bisection that worked without needing irqpoll: >> >> [ 73.751482] EATA0: IRQ 10 mapped to IO-APIC IRQ 17. >> [ 73.776711] EATA/DMA 2.0x: Copyright (C) 1994-2003 Dario Ballabio. >> [ 73.802005] EATA config options -> tm:1, lc:y, mq:16, rs:y, et:n, >> ip:n, ep:n, pp:y. >> [ 73.829175] EATA0: 2.0C, PCI 0xd890, IRQ 17, BMST, SG 122, MB 64. >> [ 73.82] EATA0: wide SCSI support enabled, max_id 16, max_lun 8. >> [ 73.881125] EATA0: SCSI channel 0 enabled, host target ID 7. > > After a git bisect, I get: > > git bisect good > 991de2e59090e55c65a7f59a049142e3c480f7bd is the first bad commit > commit 991de2e59090e55c65a7f59a049142e3c480f7bd > Author: Jiang Liu > Date: Wed Jun 10 16:54:59 2015 +0800 > > PCI, x86: Implement pcibios_alloc_irq() and pcibios_free_irq() > > To support IOAPIC hotplug, we need to allocate PCI IRQ resources on > demand > and free them when not used anymore. > > Implement pcibios_alloc_irq() and pcibios_free_irq() to dynamically > allocate and free PCI IRQs. > > Remove mp_should_keep_irq(), which is no longer used. > > [bhelgaas: changelog] > Signed-off-by: Jiang Liu > Signed-off-by: Bjorn Helgaas > Acked-by: Thomas Gleixner > > :04 04 765e2d5232d53247ec260b34b51589c3bccb36ae > f680234a27685e94b1a35ae2a7218f8eafa9071a M arch > :04 04 d55a682bcde72682e883365e88ad1df6186fd54d > f82c470a04a6845fcf5e0aa934512c75628f798d M drivers > > I'm happy to supply more details if needed. Hi Arthur, Thanks for reporting this. It seems to be an irq misrouting issue. Could you please help to provide: 1) full dmesg with the latest code 2) full dmesg and /proc/interrupts with the latest code and kernel parameter "irqpoll" specified Thanks! Gerry > > Regards, > > Arthur. > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/