* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 7:45 ` Ming Lei 0 siblings, 0 replies; 34+ messages in thread From: Ming Lei @ 2015-07-10 7:45 UTC (permalink / raw) To: Bob Moore, Lv Zheng, Rafael J. Wysocki Cc: Linux Kernel Mailing List, linux-arm-kernel, Thomas Gleixner, Jason Cooper, Hanjun Guo Hi, Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) causes the following failure on APM mustang board(arm64) when booting via UEFI and ACPI: No valid GICC entries exist ACPI: Failed to initialize GIC IRQ controller Kernel panic - not syncing: No interrupt controller found. CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 Hardware name: APM X-Gene Mustang board (DT) Call trace: [<ffffffc000089b94>] dump_backtrace+0x0/0x12c [<ffffffc000089cd0>] show_stack+0x10/0x1c [<ffffffc0005fac18>] dump_stack+0x8c/0xdc [<ffffffc0005f7218>] panic+0xe4/0x220 [<ffffffc00082631c>] init_IRQ+0x24/0x30 [<ffffffc00082486c>] start_kernel+0x274/0x3d8 ---[ end Kernel panic - not syncing: No interrupt controller found. Thanks, Ming ^ permalink raw reply [flat|nested] 34+ messages in thread
* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 7:45 ` Ming Lei 0 siblings, 0 replies; 34+ messages in thread From: Ming Lei @ 2015-07-10 7:45 UTC (permalink / raw) To: linux-arm-kernel Hi, Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) causes the following failure on APM mustang board(arm64) when booting via UEFI and ACPI: No valid GICC entries exist ACPI: Failed to initialize GIC IRQ controller Kernel panic - not syncing: No interrupt controller found. CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 Hardware name: APM X-Gene Mustang board (DT) Call trace: [<ffffffc000089b94>] dump_backtrace+0x0/0x12c [<ffffffc000089cd0>] show_stack+0x10/0x1c [<ffffffc0005fac18>] dump_stack+0x8c/0xdc [<ffffffc0005f7218>] panic+0xe4/0x220 [<ffffffc00082631c>] init_IRQ+0x24/0x30 [<ffffffc00082486c>] start_kernel+0x274/0x3d8 ---[ end Kernel panic - not syncing: No interrupt controller found. Thanks, Ming ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: ACPI: regression: Failed to initialize GIC IRQ controller 2015-07-10 7:45 ` Ming Lei @ 2015-07-10 7:58 ` Marc Zyngier -1 siblings, 0 replies; 34+ messages in thread From: Marc Zyngier @ 2015-07-10 7:58 UTC (permalink / raw) To: Ming Lei, Bob Moore, Lv Zheng, Rafael J. Wysocki Cc: Hanjun Guo, Thomas Gleixner, Linux Kernel Mailing List, linux-arm-kernel, Jason Cooper, Catalin Marinas On 10/07/15 08:45, Ming Lei wrote: > Hi, > > Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) > causes the following failure on APM mustang board(arm64) when > booting via UEFI and ACPI: > > No valid GICC entries exist > ACPI: Failed to initialize GIC IRQ controller > Kernel panic - not syncing: No interrupt controller found. > CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 > Hardware name: APM X-Gene Mustang board (DT) > Call trace: > [<ffffffc000089b94>] dump_backtrace+0x0/0x12c > [<ffffffc000089cd0>] show_stack+0x10/0x1c > [<ffffffc0005fac18>] dump_stack+0x8c/0xdc > [<ffffffc0005f7218>] panic+0xe4/0x220 > [<ffffffc00082631c>] init_IRQ+0x24/0x30 > [<ffffffc00082486c>] start_kernel+0x274/0x3d8 > ---[ end Kernel panic - not syncing: No interrupt controller found. Isn't that addressed by [1] which Catalin has queued for -rc2? Thanks, M. [1] https://lkml.org/lkml/2015/7/6/876 -- Jazz is not dead. It just smells funny... ^ permalink raw reply [flat|nested] 34+ messages in thread
* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 7:58 ` Marc Zyngier 0 siblings, 0 replies; 34+ messages in thread From: Marc Zyngier @ 2015-07-10 7:58 UTC (permalink / raw) To: linux-arm-kernel On 10/07/15 08:45, Ming Lei wrote: > Hi, > > Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) > causes the following failure on APM mustang board(arm64) when > booting via UEFI and ACPI: > > No valid GICC entries exist > ACPI: Failed to initialize GIC IRQ controller > Kernel panic - not syncing: No interrupt controller found. > CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 > Hardware name: APM X-Gene Mustang board (DT) > Call trace: > [<ffffffc000089b94>] dump_backtrace+0x0/0x12c > [<ffffffc000089cd0>] show_stack+0x10/0x1c > [<ffffffc0005fac18>] dump_stack+0x8c/0xdc > [<ffffffc0005f7218>] panic+0xe4/0x220 > [<ffffffc00082631c>] init_IRQ+0x24/0x30 > [<ffffffc00082486c>] start_kernel+0x274/0x3d8 > ---[ end Kernel panic - not syncing: No interrupt controller found. Isn't that addressed by [1] which Catalin has queued for -rc2? Thanks, M. [1] https://lkml.org/lkml/2015/7/6/876 -- Jazz is not dead. It just smells funny... ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: ACPI: regression: Failed to initialize GIC IRQ controller 2015-07-10 7:58 ` Marc Zyngier @ 2015-07-10 8:17 ` Suman Tripathi -1 siblings, 0 replies; 34+ messages in thread From: Suman Tripathi @ 2015-07-10 8:17 UTC (permalink / raw) To: Marc Zyngier Cc: Ming Lei, Bob Moore, Lv Zheng, Rafael J. Wysocki, Jason Cooper, Catalin Marinas, Linux Kernel Mailing List, Hanjun Guo, Thomas Gleixner, linux-arm-kernel On Fri, Jul 10, 2015 at 1:28 PM, Marc Zyngier <marc.zyngier@arm.com> wrote: > > On 10/07/15 08:45, Ming Lei wrote: > > Hi, > > > > Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) > > causes the following failure on APM mustang board(arm64) when > > booting via UEFI and ACPI: > > > > No valid GICC entries exist > > ACPI: Failed to initialize GIC IRQ controller > > Kernel panic - not syncing: No interrupt controller found. > > CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 > > Hardware name: APM X-Gene Mustang board (DT) > > Call trace: > > [<ffffffc000089b94>] dump_backtrace+0x0/0x12c > > [<ffffffc000089cd0>] show_stack+0x10/0x1c > > [<ffffffc0005fac18>] dump_stack+0x8c/0xdc > > [<ffffffc0005f7218>] panic+0xe4/0x220 > > [<ffffffc00082631c>] init_IRQ+0x24/0x30 > > [<ffffffc00082486c>] start_kernel+0x274/0x3d8 > > ---[ end Kernel panic - not syncing: No interrupt controller found. > > Isn't that addressed by [1] which Catalin has queued for -rc2? I am also seeing the same . But it fixes after I apply the parking protocol patch but that is not upstreamed. > > > Thanks, > > M. > > [1] https://lkml.org/lkml/2015/7/6/876 > > -- > Jazz is not dead. It just smells funny... > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Thanks, with regards, Suman Tripathi ^ permalink raw reply [flat|nested] 34+ messages in thread
* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 8:17 ` Suman Tripathi 0 siblings, 0 replies; 34+ messages in thread From: Suman Tripathi @ 2015-07-10 8:17 UTC (permalink / raw) To: linux-arm-kernel On Fri, Jul 10, 2015 at 1:28 PM, Marc Zyngier <marc.zyngier@arm.com> wrote: > > On 10/07/15 08:45, Ming Lei wrote: > > Hi, > > > > Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) > > causes the following failure on APM mustang board(arm64) when > > booting via UEFI and ACPI: > > > > No valid GICC entries exist > > ACPI: Failed to initialize GIC IRQ controller > > Kernel panic - not syncing: No interrupt controller found. > > CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 > > Hardware name: APM X-Gene Mustang board (DT) > > Call trace: > > [<ffffffc000089b94>] dump_backtrace+0x0/0x12c > > [<ffffffc000089cd0>] show_stack+0x10/0x1c > > [<ffffffc0005fac18>] dump_stack+0x8c/0xdc > > [<ffffffc0005f7218>] panic+0xe4/0x220 > > [<ffffffc00082631c>] init_IRQ+0x24/0x30 > > [<ffffffc00082486c>] start_kernel+0x274/0x3d8 > > ---[ end Kernel panic - not syncing: No interrupt controller found. > > Isn't that addressed by [1] which Catalin has queued for -rc2? I am also seeing the same . But it fixes after I apply the parking protocol patch but that is not upstreamed. > > > Thanks, > > M. > > [1] https://lkml.org/lkml/2015/7/6/876 > > -- > Jazz is not dead. It just smells funny... > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel at lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Thanks, with regards, Suman Tripathi ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: ACPI: regression: Failed to initialize GIC IRQ controller 2015-07-10 8:17 ` Suman Tripathi @ 2015-07-10 8:23 ` Marc Zyngier -1 siblings, 0 replies; 34+ messages in thread From: Marc Zyngier @ 2015-07-10 8:23 UTC (permalink / raw) To: Suman Tripathi Cc: Ming Lei, Bob Moore, Lv Zheng, Rafael J. Wysocki, Jason Cooper, Catalin Marinas, Linux Kernel Mailing List, hanjun.guo, Thomas Gleixner, linux-arm-kernel On 10/07/15 09:17, Suman Tripathi wrote: > On Fri, Jul 10, 2015 at 1:28 PM, Marc Zyngier <marc.zyngier@arm.com> wrote: >> >> On 10/07/15 08:45, Ming Lei wrote: >>> Hi, >>> >>> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) >>> causes the following failure on APM mustang board(arm64) when >>> booting via UEFI and ACPI: >>> >>> No valid GICC entries exist >>> ACPI: Failed to initialize GIC IRQ controller >>> Kernel panic - not syncing: No interrupt controller found. >>> CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 >>> Hardware name: APM X-Gene Mustang board (DT) >>> Call trace: >>> [<ffffffc000089b94>] dump_backtrace+0x0/0x12c >>> [<ffffffc000089cd0>] show_stack+0x10/0x1c >>> [<ffffffc0005fac18>] dump_stack+0x8c/0xdc >>> [<ffffffc0005f7218>] panic+0xe4/0x220 >>> [<ffffffc00082631c>] init_IRQ+0x24/0x30 >>> [<ffffffc00082486c>] start_kernel+0x274/0x3d8 >>> ---[ end Kernel panic - not syncing: No interrupt controller found. >> >> Isn't that addressed by [1] which Catalin has queued for -rc2? > > > I am also seeing the same . But it fixes after I apply the parking > protocol patch but that is not upstreamed. I'm more interested to find out if the patch I mentioned in my original email fixes it or not. An additional dependency on something that is not aimed for mainline yet doesn't really help. Thanks, M. -- Jazz is not dead. It just smells funny... ^ permalink raw reply [flat|nested] 34+ messages in thread
* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 8:23 ` Marc Zyngier 0 siblings, 0 replies; 34+ messages in thread From: Marc Zyngier @ 2015-07-10 8:23 UTC (permalink / raw) To: linux-arm-kernel On 10/07/15 09:17, Suman Tripathi wrote: > On Fri, Jul 10, 2015 at 1:28 PM, Marc Zyngier <marc.zyngier@arm.com> wrote: >> >> On 10/07/15 08:45, Ming Lei wrote: >>> Hi, >>> >>> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) >>> causes the following failure on APM mustang board(arm64) when >>> booting via UEFI and ACPI: >>> >>> No valid GICC entries exist >>> ACPI: Failed to initialize GIC IRQ controller >>> Kernel panic - not syncing: No interrupt controller found. >>> CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 >>> Hardware name: APM X-Gene Mustang board (DT) >>> Call trace: >>> [<ffffffc000089b94>] dump_backtrace+0x0/0x12c >>> [<ffffffc000089cd0>] show_stack+0x10/0x1c >>> [<ffffffc0005fac18>] dump_stack+0x8c/0xdc >>> [<ffffffc0005f7218>] panic+0xe4/0x220 >>> [<ffffffc00082631c>] init_IRQ+0x24/0x30 >>> [<ffffffc00082486c>] start_kernel+0x274/0x3d8 >>> ---[ end Kernel panic - not syncing: No interrupt controller found. >> >> Isn't that addressed by [1] which Catalin has queued for -rc2? > > > I am also seeing the same . But it fixes after I apply the parking > protocol patch but that is not upstreamed. I'm more interested to find out if the patch I mentioned in my original email fixes it or not. An additional dependency on something that is not aimed for mainline yet doesn't really help. Thanks, M. -- Jazz is not dead. It just smells funny... ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: ACPI: regression: Failed to initialize GIC IRQ controller 2015-07-10 8:23 ` Marc Zyngier @ 2015-07-10 9:37 ` Suman Tripathi -1 siblings, 0 replies; 34+ messages in thread From: Suman Tripathi @ 2015-07-10 9:37 UTC (permalink / raw) To: Marc Zyngier Cc: Ming Lei, Bob Moore, Lv Zheng, Rafael J. Wysocki, Jason Cooper, Catalin Marinas, Linux Kernel Mailing List, hanjun.guo, Thomas Gleixner, linux-arm-kernel On Fri, Jul 10, 2015 at 1:53 PM, Marc Zyngier <marc.zyngier@arm.com> wrote: > On 10/07/15 09:17, Suman Tripathi wrote: >> On Fri, Jul 10, 2015 at 1:28 PM, Marc Zyngier <marc.zyngier@arm.com> wrote: >>> >>> On 10/07/15 08:45, Ming Lei wrote: >>>> Hi, >>>> >>>> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) >>>> causes the following failure on APM mustang board(arm64) when >>>> booting via UEFI and ACPI: >>>> >>>> No valid GICC entries exist >>>> ACPI: Failed to initialize GIC IRQ controller >>>> Kernel panic - not syncing: No interrupt controller found. >>>> CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 >>>> Hardware name: APM X-Gene Mustang board (DT) >>>> Call trace: >>>> [<ffffffc000089b94>] dump_backtrace+0x0/0x12c >>>> [<ffffffc000089cd0>] show_stack+0x10/0x1c >>>> [<ffffffc0005fac18>] dump_stack+0x8c/0xdc >>>> [<ffffffc0005f7218>] panic+0xe4/0x220 >>>> [<ffffffc00082631c>] init_IRQ+0x24/0x30 >>>> [<ffffffc00082486c>] start_kernel+0x274/0x3d8 >>>> ---[ end Kernel panic - not syncing: No interrupt controller found. >>> >>> Isn't that addressed by [1] which Catalin has queued for -rc2? >> >> >> I am also seeing the same . But it fixes after I apply the parking >> protocol patch but that is not upstreamed. > > I'm more interested to find out if the patch I mentioned in my original > email fixes it or not. An additional dependency on something that is not > aimed for mainline yet doesn't really help. Yes the patch mentioned by you fixes the issue. [1] https://lkml.org/lkml/2015/7/6/876 > > Thanks, > > M. > -- > Jazz is not dead. It just smells funny... -- Thanks, with regards, Suman Tripathi ^ permalink raw reply [flat|nested] 34+ messages in thread
* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 9:37 ` Suman Tripathi 0 siblings, 0 replies; 34+ messages in thread From: Suman Tripathi @ 2015-07-10 9:37 UTC (permalink / raw) To: linux-arm-kernel On Fri, Jul 10, 2015 at 1:53 PM, Marc Zyngier <marc.zyngier@arm.com> wrote: > On 10/07/15 09:17, Suman Tripathi wrote: >> On Fri, Jul 10, 2015 at 1:28 PM, Marc Zyngier <marc.zyngier@arm.com> wrote: >>> >>> On 10/07/15 08:45, Ming Lei wrote: >>>> Hi, >>>> >>>> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) >>>> causes the following failure on APM mustang board(arm64) when >>>> booting via UEFI and ACPI: >>>> >>>> No valid GICC entries exist >>>> ACPI: Failed to initialize GIC IRQ controller >>>> Kernel panic - not syncing: No interrupt controller found. >>>> CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 >>>> Hardware name: APM X-Gene Mustang board (DT) >>>> Call trace: >>>> [<ffffffc000089b94>] dump_backtrace+0x0/0x12c >>>> [<ffffffc000089cd0>] show_stack+0x10/0x1c >>>> [<ffffffc0005fac18>] dump_stack+0x8c/0xdc >>>> [<ffffffc0005f7218>] panic+0xe4/0x220 >>>> [<ffffffc00082631c>] init_IRQ+0x24/0x30 >>>> [<ffffffc00082486c>] start_kernel+0x274/0x3d8 >>>> ---[ end Kernel panic - not syncing: No interrupt controller found. >>> >>> Isn't that addressed by [1] which Catalin has queued for -rc2? >> >> >> I am also seeing the same . But it fixes after I apply the parking >> protocol patch but that is not upstreamed. > > I'm more interested to find out if the patch I mentioned in my original > email fixes it or not. An additional dependency on something that is not > aimed for mainline yet doesn't really help. Yes the patch mentioned by you fixes the issue. [1] https://lkml.org/lkml/2015/7/6/876 > > Thanks, > > M. > -- > Jazz is not dead. It just smells funny... -- Thanks, with regards, Suman Tripathi ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: ACPI: regression: Failed to initialize GIC IRQ controller 2015-07-10 8:23 ` Marc Zyngier @ 2015-07-10 10:11 ` Ming Lei -1 siblings, 0 replies; 34+ messages in thread From: Ming Lei @ 2015-07-10 10:11 UTC (permalink / raw) To: Marc Zyngier Cc: Suman Tripathi, Bob Moore, Lv Zheng, Rafael J. Wysocki, Jason Cooper, Catalin Marinas, Linux Kernel Mailing List, hanjun.guo, Thomas Gleixner, linux-arm-kernel On Fri, Jul 10, 2015 at 4:23 PM, Marc Zyngier <marc.zyngier@arm.com> wrote: > On 10/07/15 09:17, Suman Tripathi wrote: >> On Fri, Jul 10, 2015 at 1:28 PM, Marc Zyngier <marc.zyngier@arm.com> wrote: >>> >>> On 10/07/15 08:45, Ming Lei wrote: >>>> Hi, >>>> >>>> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) >>>> causes the following failure on APM mustang board(arm64) when >>>> booting via UEFI and ACPI: >>>> >>>> No valid GICC entries exist >>>> ACPI: Failed to initialize GIC IRQ controller >>>> Kernel panic - not syncing: No interrupt controller found. >>>> CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 >>>> Hardware name: APM X-Gene Mustang board (DT) >>>> Call trace: >>>> [<ffffffc000089b94>] dump_backtrace+0x0/0x12c >>>> [<ffffffc000089cd0>] show_stack+0x10/0x1c >>>> [<ffffffc0005fac18>] dump_stack+0x8c/0xdc >>>> [<ffffffc0005f7218>] panic+0xe4/0x220 >>>> [<ffffffc00082631c>] init_IRQ+0x24/0x30 >>>> [<ffffffc00082486c>] start_kernel+0x274/0x3d8 >>>> ---[ end Kernel panic - not syncing: No interrupt controller found. >>> >>> Isn't that addressed by [1] which Catalin has queued for -rc2? >> >> >> I am also seeing the same . But it fixes after I apply the parking >> protocol patch but that is not upstreamed. > > I'm more interested to find out if the patch I mentioned in my original > email fixes it or not. An additional dependency on something that is not > aimed for mainline yet doesn't really help. Yes, Al's patches do fix the issue. Thanks, Ming ^ permalink raw reply [flat|nested] 34+ messages in thread
* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 10:11 ` Ming Lei 0 siblings, 0 replies; 34+ messages in thread From: Ming Lei @ 2015-07-10 10:11 UTC (permalink / raw) To: linux-arm-kernel On Fri, Jul 10, 2015 at 4:23 PM, Marc Zyngier <marc.zyngier@arm.com> wrote: > On 10/07/15 09:17, Suman Tripathi wrote: >> On Fri, Jul 10, 2015 at 1:28 PM, Marc Zyngier <marc.zyngier@arm.com> wrote: >>> >>> On 10/07/15 08:45, Ming Lei wrote: >>>> Hi, >>>> >>>> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) >>>> causes the following failure on APM mustang board(arm64) when >>>> booting via UEFI and ACPI: >>>> >>>> No valid GICC entries exist >>>> ACPI: Failed to initialize GIC IRQ controller >>>> Kernel panic - not syncing: No interrupt controller found. >>>> CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 >>>> Hardware name: APM X-Gene Mustang board (DT) >>>> Call trace: >>>> [<ffffffc000089b94>] dump_backtrace+0x0/0x12c >>>> [<ffffffc000089cd0>] show_stack+0x10/0x1c >>>> [<ffffffc0005fac18>] dump_stack+0x8c/0xdc >>>> [<ffffffc0005f7218>] panic+0xe4/0x220 >>>> [<ffffffc00082631c>] init_IRQ+0x24/0x30 >>>> [<ffffffc00082486c>] start_kernel+0x274/0x3d8 >>>> ---[ end Kernel panic - not syncing: No interrupt controller found. >>> >>> Isn't that addressed by [1] which Catalin has queued for -rc2? >> >> >> I am also seeing the same . But it fixes after I apply the parking >> protocol patch but that is not upstreamed. > > I'm more interested to find out if the patch I mentioned in my original > email fixes it or not. An additional dependency on something that is not > aimed for mainline yet doesn't really help. Yes, Al's patches do fix the issue. Thanks, Ming ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: ACPI: regression: Failed to initialize GIC IRQ controller 2015-07-10 10:11 ` Ming Lei @ 2015-07-10 11:10 ` Hanjun Guo -1 siblings, 0 replies; 34+ messages in thread From: Hanjun Guo @ 2015-07-10 11:10 UTC (permalink / raw) To: Ming Lei, Marc Zyngier Cc: Suman Tripathi, Bob Moore, Lv Zheng, Rafael J. Wysocki, Jason Cooper, Catalin Marinas, Linux Kernel Mailing List, Thomas Gleixner, linux-arm-kernel On 07/10/2015 06:11 PM, Ming Lei wrote: > On Fri, Jul 10, 2015 at 4:23 PM, Marc Zyngier <marc.zyngier@arm.com> wrote: >> On 10/07/15 09:17, Suman Tripathi wrote: >>> On Fri, Jul 10, 2015 at 1:28 PM, Marc Zyngier <marc.zyngier@arm.com> wrote: >>>> >>>> On 10/07/15 08:45, Ming Lei wrote: >>>>> Hi, >>>>> >>>>> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) >>>>> causes the following failure on APM mustang board(arm64) when >>>>> booting via UEFI and ACPI: >>>>> >>>>> No valid GICC entries exist >>>>> ACPI: Failed to initialize GIC IRQ controller >>>>> Kernel panic - not syncing: No interrupt controller found. >>>>> CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 >>>>> Hardware name: APM X-Gene Mustang board (DT) >>>>> Call trace: >>>>> [<ffffffc000089b94>] dump_backtrace+0x0/0x12c >>>>> [<ffffffc000089cd0>] show_stack+0x10/0x1c >>>>> [<ffffffc0005fac18>] dump_stack+0x8c/0xdc >>>>> [<ffffffc0005f7218>] panic+0xe4/0x220 >>>>> [<ffffffc00082631c>] init_IRQ+0x24/0x30 >>>>> [<ffffffc00082486c>] start_kernel+0x274/0x3d8 >>>>> ---[ end Kernel panic - not syncing: No interrupt controller found. >>>> >>>> Isn't that addressed by [1] which Catalin has queued for -rc2? >>> >>> >>> I am also seeing the same . But it fixes after I apply the parking >>> protocol patch but that is not upstreamed. >> >> I'm more interested to find out if the patch I mentioned in my original >> email fixes it or not. An additional dependency on something that is not >> aimed for mainline yet doesn't really help. > > Yes, Al's patches do fix the issue. I should have read ahead :) Thanks Hanjun ^ permalink raw reply [flat|nested] 34+ messages in thread
* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 11:10 ` Hanjun Guo 0 siblings, 0 replies; 34+ messages in thread From: Hanjun Guo @ 2015-07-10 11:10 UTC (permalink / raw) To: linux-arm-kernel On 07/10/2015 06:11 PM, Ming Lei wrote: > On Fri, Jul 10, 2015 at 4:23 PM, Marc Zyngier <marc.zyngier@arm.com> wrote: >> On 10/07/15 09:17, Suman Tripathi wrote: >>> On Fri, Jul 10, 2015 at 1:28 PM, Marc Zyngier <marc.zyngier@arm.com> wrote: >>>> >>>> On 10/07/15 08:45, Ming Lei wrote: >>>>> Hi, >>>>> >>>>> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) >>>>> causes the following failure on APM mustang board(arm64) when >>>>> booting via UEFI and ACPI: >>>>> >>>>> No valid GICC entries exist >>>>> ACPI: Failed to initialize GIC IRQ controller >>>>> Kernel panic - not syncing: No interrupt controller found. >>>>> CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 >>>>> Hardware name: APM X-Gene Mustang board (DT) >>>>> Call trace: >>>>> [<ffffffc000089b94>] dump_backtrace+0x0/0x12c >>>>> [<ffffffc000089cd0>] show_stack+0x10/0x1c >>>>> [<ffffffc0005fac18>] dump_stack+0x8c/0xdc >>>>> [<ffffffc0005f7218>] panic+0xe4/0x220 >>>>> [<ffffffc00082631c>] init_IRQ+0x24/0x30 >>>>> [<ffffffc00082486c>] start_kernel+0x274/0x3d8 >>>>> ---[ end Kernel panic - not syncing: No interrupt controller found. >>>> >>>> Isn't that addressed by [1] which Catalin has queued for -rc2? >>> >>> >>> I am also seeing the same . But it fixes after I apply the parking >>> protocol patch but that is not upstreamed. >> >> I'm more interested to find out if the patch I mentioned in my original >> email fixes it or not. An additional dependency on something that is not >> aimed for mainline yet doesn't really help. > > Yes, Al's patches do fix the issue. I should have read ahead :) Thanks Hanjun ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: ACPI: regression: Failed to initialize GIC IRQ controller 2015-07-10 7:58 ` Marc Zyngier @ 2015-07-10 10:49 ` Hanjun Guo -1 siblings, 0 replies; 34+ messages in thread From: Hanjun Guo @ 2015-07-10 10:49 UTC (permalink / raw) To: Marc Zyngier, Ming Lei, Bob Moore, Lv Zheng, Rafael J. Wysocki Cc: Thomas Gleixner, Linux Kernel Mailing List, linux-arm-kernel, Jason Cooper, Catalin Marinas On 07/10/2015 03:58 PM, Marc Zyngier wrote: > On 10/07/15 08:45, Ming Lei wrote: >> Hi, >> >> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) >> causes the following failure on APM mustang board(arm64) when >> booting via UEFI and ACPI: >> >> No valid GICC entries exist >> ACPI: Failed to initialize GIC IRQ controller >> Kernel panic - not syncing: No interrupt controller found. >> CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 >> Hardware name: APM X-Gene Mustang board (DT) >> Call trace: >> [<ffffffc000089b94>] dump_backtrace+0x0/0x12c >> [<ffffffc000089cd0>] show_stack+0x10/0x1c >> [<ffffffc0005fac18>] dump_stack+0x8c/0xdc >> [<ffffffc0005f7218>] panic+0xe4/0x220 >> [<ffffffc00082631c>] init_IRQ+0x24/0x30 >> [<ffffffc00082486c>] start_kernel+0x274/0x3d8 >> ---[ end Kernel panic - not syncing: No interrupt controller found. > > Isn't that addressed by [1] which Catalin has queued for -rc2? > > Thanks, > > M. > > [1] https://lkml.org/lkml/2015/7/6/876 Yes, already fixed by Al and Catalin already queued for 4.2-rc2. Thanks Hanjun ^ permalink raw reply [flat|nested] 34+ messages in thread
* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 10:49 ` Hanjun Guo 0 siblings, 0 replies; 34+ messages in thread From: Hanjun Guo @ 2015-07-10 10:49 UTC (permalink / raw) To: linux-arm-kernel On 07/10/2015 03:58 PM, Marc Zyngier wrote: > On 10/07/15 08:45, Ming Lei wrote: >> Hi, >> >> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) >> causes the following failure on APM mustang board(arm64) when >> booting via UEFI and ACPI: >> >> No valid GICC entries exist >> ACPI: Failed to initialize GIC IRQ controller >> Kernel panic - not syncing: No interrupt controller found. >> CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 >> Hardware name: APM X-Gene Mustang board (DT) >> Call trace: >> [<ffffffc000089b94>] dump_backtrace+0x0/0x12c >> [<ffffffc000089cd0>] show_stack+0x10/0x1c >> [<ffffffc0005fac18>] dump_stack+0x8c/0xdc >> [<ffffffc0005f7218>] panic+0xe4/0x220 >> [<ffffffc00082631c>] init_IRQ+0x24/0x30 >> [<ffffffc00082486c>] start_kernel+0x274/0x3d8 >> ---[ end Kernel panic - not syncing: No interrupt controller found. > > Isn't that addressed by [1] which Catalin has queued for -rc2? > > Thanks, > > M. > > [1] https://lkml.org/lkml/2015/7/6/876 Yes, already fixed by Al and Catalin already queued for 4.2-rc2. Thanks Hanjun ^ permalink raw reply [flat|nested] 34+ messages in thread
* RE: ACPI: regression: Failed to initialize GIC IRQ controller 2015-07-10 7:45 ` Ming Lei @ 2015-07-10 14:28 ` Moore, Robert -1 siblings, 0 replies; 34+ messages in thread From: Moore, Robert @ 2015-07-10 14:28 UTC (permalink / raw) To: Ming Lei, Zheng, Lv, Wysocki, Rafael J Cc: Linux Kernel Mailing List, linux-arm-kernel, Thomas Gleixner, Jason Cooper, Hanjun Guo [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #1: Type: text/plain; charset="utf-8", Size: 1375 bytes --] > -----Original Message----- > From: Ming Lei [mailto:ming.lei@canonical.com] > Sent: Friday, July 10, 2015 12:46 AM > To: Moore, Robert; Zheng, Lv; Wysocki, Rafael J > Cc: Linux Kernel Mailing List; linux-arm-kernel; Thomas Gleixner; Jason > Cooper; Hanjun Guo > Subject: ACPI: regression: Failed to initialize GIC IRQ controller > > Hi, > > Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) > causes the following failure on APM mustang board(arm64) when booting via > UEFI and ACPI: I would be interested to know just what exactly about this change broke things. Bob > > No valid GICC entries exist > ACPI: Failed to initialize GIC IRQ controller Kernel panic - not syncing: > No interrupt controller found. > CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 Hardware name: > APM X-Gene Mustang board (DT) Call trace: > [<ffffffc000089b94>] dump_backtrace+0x0/0x12c [<ffffffc000089cd0>] > show_stack+0x10/0x1c [<ffffffc0005fac18>] dump_stack+0x8c/0xdc > [<ffffffc0005f7218>] panic+0xe4/0x220 [<ffffffc00082631c>] > init_IRQ+0x24/0x30 [<ffffffc00082486c>] start_kernel+0x274/0x3d8 ---[ end > Kernel panic - not syncing: No interrupt controller found. > > > Thanks, > Ming ÿôèº{.nÇ+·®+%Ëÿ±éݶ\x17¥wÿº{.nÇ+·¥{±þG«éÿ{ayº\x1dÊÚë,j\a¢f£¢·hïêÿêçz_è®\x03(éÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?¨èÚ&£ø§~á¶iOæ¬z·vØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?I¥ ^ permalink raw reply [flat|nested] 34+ messages in thread
* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 14:28 ` Moore, Robert 0 siblings, 0 replies; 34+ messages in thread From: Moore, Robert @ 2015-07-10 14:28 UTC (permalink / raw) To: linux-arm-kernel > -----Original Message----- > From: Ming Lei [mailto:ming.lei at canonical.com] > Sent: Friday, July 10, 2015 12:46 AM > To: Moore, Robert; Zheng, Lv; Wysocki, Rafael J > Cc: Linux Kernel Mailing List; linux-arm-kernel; Thomas Gleixner; Jason > Cooper; Hanjun Guo > Subject: ACPI: regression: Failed to initialize GIC IRQ controller > > Hi, > > Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) > causes the following failure on APM mustang board(arm64) when booting via > UEFI and ACPI: I would be interested to know just what exactly about this change broke things. Bob > > No valid GICC entries exist > ACPI: Failed to initialize GIC IRQ controller Kernel panic - not syncing: > No interrupt controller found. > CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 Hardware name: > APM X-Gene Mustang board (DT) Call trace: > [<ffffffc000089b94>] dump_backtrace+0x0/0x12c [<ffffffc000089cd0>] > show_stack+0x10/0x1c [<ffffffc0005fac18>] dump_stack+0x8c/0xdc > [<ffffffc0005f7218>] panic+0xe4/0x220 [<ffffffc00082631c>] > init_IRQ+0x24/0x30 [<ffffffc00082486c>] start_kernel+0x274/0x3d8 ---[ end > Kernel panic - not syncing: No interrupt controller found. > > > Thanks, > Ming ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: ACPI: regression: Failed to initialize GIC IRQ controller 2015-07-10 14:28 ` Moore, Robert @ 2015-07-10 14:43 ` Ming Lei -1 siblings, 0 replies; 34+ messages in thread From: Ming Lei @ 2015-07-10 14:43 UTC (permalink / raw) To: Moore, Robert Cc: Zheng, Lv, Wysocki, Rafael J, Linux Kernel Mailing List, linux-arm-kernel, Thomas Gleixner, Jason Cooper, Hanjun Guo On Fri, Jul 10, 2015 at 10:28 PM, Moore, Robert <robert.moore@intel.com> wrote: > > >> -----Original Message----- >> From: Ming Lei [mailto:ming.lei@canonical.com] >> Sent: Friday, July 10, 2015 12:46 AM >> To: Moore, Robert; Zheng, Lv; Wysocki, Rafael J >> Cc: Linux Kernel Mailing List; linux-arm-kernel; Thomas Gleixner; Jason >> Cooper; Hanjun Guo >> Subject: ACPI: regression: Failed to initialize GIC IRQ controller >> >> Hi, >> >> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) >> causes the following failure on APM mustang board(arm64) when booting via >> UEFI and ACPI: > > > I would be interested to know just what exactly about this change broke things. sizeof(struct acpi_madt_generic_interrupt) ^ permalink raw reply [flat|nested] 34+ messages in thread
* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 14:43 ` Ming Lei 0 siblings, 0 replies; 34+ messages in thread From: Ming Lei @ 2015-07-10 14:43 UTC (permalink / raw) To: linux-arm-kernel On Fri, Jul 10, 2015 at 10:28 PM, Moore, Robert <robert.moore@intel.com> wrote: > > >> -----Original Message----- >> From: Ming Lei [mailto:ming.lei at canonical.com] >> Sent: Friday, July 10, 2015 12:46 AM >> To: Moore, Robert; Zheng, Lv; Wysocki, Rafael J >> Cc: Linux Kernel Mailing List; linux-arm-kernel; Thomas Gleixner; Jason >> Cooper; Hanjun Guo >> Subject: ACPI: regression: Failed to initialize GIC IRQ controller >> >> Hi, >> >> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) >> causes the following failure on APM mustang board(arm64) when booting via >> UEFI and ACPI: > > > I would be interested to know just what exactly about this change broke things. sizeof(struct acpi_madt_generic_interrupt) ^ permalink raw reply [flat|nested] 34+ messages in thread
* RE: ACPI: regression: Failed to initialize GIC IRQ controller 2015-07-10 14:43 ` Ming Lei @ 2015-07-10 14:45 ` Moore, Robert -1 siblings, 0 replies; 34+ messages in thread From: Moore, Robert @ 2015-07-10 14:45 UTC (permalink / raw) To: Ming Lei Cc: Zheng, Lv, Wysocki, Rafael J, Linux Kernel Mailing List, linux-arm-kernel, Thomas Gleixner, Jason Cooper, Hanjun Guo [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #1: Type: text/plain; charset="utf-8", Size: 1413 bytes --] It's nice that someone took a sizeof() on the struct -- so, one would hope that no code actually depended on a particular value, no? > -----Original Message----- > From: Ming Lei [mailto:ming.lei@canonical.com] > Sent: Friday, July 10, 2015 7:43 AM > To: Moore, Robert > Cc: Zheng, Lv; Wysocki, Rafael J; Linux Kernel Mailing List; linux-arm- > kernel; Thomas Gleixner; Jason Cooper; Hanjun Guo > Subject: Re: ACPI: regression: Failed to initialize GIC IRQ controller > > On Fri, Jul 10, 2015 at 10:28 PM, Moore, Robert <robert.moore@intel.com> > wrote: > > > > > >> -----Original Message----- > >> From: Ming Lei [mailto:ming.lei@canonical.com] > >> Sent: Friday, July 10, 2015 12:46 AM > >> To: Moore, Robert; Zheng, Lv; Wysocki, Rafael J > >> Cc: Linux Kernel Mailing List; linux-arm-kernel; Thomas Gleixner; > >> Jason Cooper; Hanjun Guo > >> Subject: ACPI: regression: Failed to initialize GIC IRQ controller > >> > >> Hi, > >> > >> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT > >> table.) causes the following failure on APM mustang board(arm64) when > >> booting via UEFI and ACPI: > > > > > > I would be interested to know just what exactly about this change broke > things. > > sizeof(struct acpi_madt_generic_interrupt) ÿôèº{.nÇ+·®+%Ëÿ±éݶ\x17¥wÿº{.nÇ+·¥{±þG«éÿ{ayº\x1dÊÚë,j\a¢f£¢·hïêÿêçz_è®\x03(éÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?¨èÚ&£ø§~á¶iOæ¬z·vØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?I¥ ^ permalink raw reply [flat|nested] 34+ messages in thread
* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 14:45 ` Moore, Robert 0 siblings, 0 replies; 34+ messages in thread From: Moore, Robert @ 2015-07-10 14:45 UTC (permalink / raw) To: linux-arm-kernel It's nice that someone took a sizeof() on the struct -- so, one would hope that no code actually depended on a particular value, no? > -----Original Message----- > From: Ming Lei [mailto:ming.lei at canonical.com] > Sent: Friday, July 10, 2015 7:43 AM > To: Moore, Robert > Cc: Zheng, Lv; Wysocki, Rafael J; Linux Kernel Mailing List; linux-arm- > kernel; Thomas Gleixner; Jason Cooper; Hanjun Guo > Subject: Re: ACPI: regression: Failed to initialize GIC IRQ controller > > On Fri, Jul 10, 2015 at 10:28 PM, Moore, Robert <robert.moore@intel.com> > wrote: > > > > > >> -----Original Message----- > >> From: Ming Lei [mailto:ming.lei at canonical.com] > >> Sent: Friday, July 10, 2015 12:46 AM > >> To: Moore, Robert; Zheng, Lv; Wysocki, Rafael J > >> Cc: Linux Kernel Mailing List; linux-arm-kernel; Thomas Gleixner; > >> Jason Cooper; Hanjun Guo > >> Subject: ACPI: regression: Failed to initialize GIC IRQ controller > >> > >> Hi, > >> > >> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT > >> table.) causes the following failure on APM mustang board(arm64) when > >> booting via UEFI and ACPI: > > > > > > I would be interested to know just what exactly about this change broke > things. > > sizeof(struct acpi_madt_generic_interrupt) ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: ACPI: regression: Failed to initialize GIC IRQ controller 2015-07-10 14:45 ` Moore, Robert @ 2015-07-10 15:17 ` Lorenzo Pieralisi -1 siblings, 0 replies; 34+ messages in thread From: Lorenzo Pieralisi @ 2015-07-10 15:17 UTC (permalink / raw) To: Moore, Robert Cc: Ming Lei, Zheng, Lv, Wysocki, Rafael J, Linux Kernel Mailing List, linux-arm-kernel, Thomas Gleixner, Jason Cooper, hanjun.guo On Fri, Jul 10, 2015 at 03:45:32PM +0100, Moore, Robert wrote: > It's nice that someone took a sizeof() on the struct -- so, one would hope that no code actually depended on a particular value, no? Unfortunately that sizeof has been there forever (x86/ia64), ia64 code ran into a similar issue, so the check was removed to cope with lsapic MADT updates, see: arch/ia64/kernel/acpi.c line 204 /*Skip BAD_MADT_ENTRY check, as lsapic size could vary */ Is checking the subtable length field against a static value really worthwhile/suitable ? Thanks, Lorenzo > > -----Original Message----- > > From: Ming Lei [mailto:ming.lei@canonical.com] > > Sent: Friday, July 10, 2015 7:43 AM > > To: Moore, Robert > > Cc: Zheng, Lv; Wysocki, Rafael J; Linux Kernel Mailing List; linux-arm- > > kernel; Thomas Gleixner; Jason Cooper; Hanjun Guo > > Subject: Re: ACPI: regression: Failed to initialize GIC IRQ controller > > > > On Fri, Jul 10, 2015 at 10:28 PM, Moore, Robert <robert.moore@intel.com> > > wrote: > > > > > > > > >> -----Original Message----- > > >> From: Ming Lei [mailto:ming.lei@canonical.com] > > >> Sent: Friday, July 10, 2015 12:46 AM > > >> To: Moore, Robert; Zheng, Lv; Wysocki, Rafael J > > >> Cc: Linux Kernel Mailing List; linux-arm-kernel; Thomas Gleixner; > > >> Jason Cooper; Hanjun Guo > > >> Subject: ACPI: regression: Failed to initialize GIC IRQ controller > > >> > > >> Hi, > > >> > > >> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT > > >> table.) causes the following failure on APM mustang board(arm64) when > > >> booting via UEFI and ACPI: > > > > > > > > > I would be interested to know just what exactly about this change broke > > things. > > > > sizeof(struct acpi_madt_generic_interrupt) > N???????????????r??????y?????????b???X????????v???^???)??{.n???+????????????{????????????zX??????\x17????????}???????????z???&j:+v?????????\a????????????zZ+??????+zf?????????h?????????~????????????i?????????z???\x1e???w??????????????????????&???)??^[f??????^j??y???m??????@A???a??????\x7f???\f0??????h???\x0f???i\x7f ^ permalink raw reply [flat|nested] 34+ messages in thread
* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 15:17 ` Lorenzo Pieralisi 0 siblings, 0 replies; 34+ messages in thread From: Lorenzo Pieralisi @ 2015-07-10 15:17 UTC (permalink / raw) To: linux-arm-kernel On Fri, Jul 10, 2015 at 03:45:32PM +0100, Moore, Robert wrote: > It's nice that someone took a sizeof() on the struct -- so, one would hope that no code actually depended on a particular value, no? Unfortunately that sizeof has been there forever (x86/ia64), ia64 code ran into a similar issue, so the check was removed to cope with lsapic MADT updates, see: arch/ia64/kernel/acpi.c line 204 /*Skip BAD_MADT_ENTRY check, as lsapic size could vary */ Is checking the subtable length field against a static value really worthwhile/suitable ? Thanks, Lorenzo > > -----Original Message----- > > From: Ming Lei [mailto:ming.lei at canonical.com] > > Sent: Friday, July 10, 2015 7:43 AM > > To: Moore, Robert > > Cc: Zheng, Lv; Wysocki, Rafael J; Linux Kernel Mailing List; linux-arm- > > kernel; Thomas Gleixner; Jason Cooper; Hanjun Guo > > Subject: Re: ACPI: regression: Failed to initialize GIC IRQ controller > > > > On Fri, Jul 10, 2015 at 10:28 PM, Moore, Robert <robert.moore@intel.com> > > wrote: > > > > > > > > >> -----Original Message----- > > >> From: Ming Lei [mailto:ming.lei at canonical.com] > > >> Sent: Friday, July 10, 2015 12:46 AM > > >> To: Moore, Robert; Zheng, Lv; Wysocki, Rafael J > > >> Cc: Linux Kernel Mailing List; linux-arm-kernel; Thomas Gleixner; > > >> Jason Cooper; Hanjun Guo > > >> Subject: ACPI: regression: Failed to initialize GIC IRQ controller > > >> > > >> Hi, > > >> > > >> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT > > >> table.) causes the following failure on APM mustang board(arm64) when > > >> booting via UEFI and ACPI: > > > > > > > > > I would be interested to know just what exactly about this change broke > > things. > > > > sizeof(struct acpi_madt_generic_interrupt) > N???????????????r??????y?????????b???X????????v???^???)??{.n???+????????????{????????????zX??????\x17????????}???????????z???&j:+v?????????\a????????????zZ+??????+zf?????????h?????????~????????????i?????????z???\x1e ???w??????????????????????&???)??^[f??????^j??y???m??????@A???a??????\x7f???\f 0??????h???\x0f???i\x7f ^ permalink raw reply [flat|nested] 34+ messages in thread
* RE: ACPI: regression: Failed to initialize GIC IRQ controller 2015-07-10 15:17 ` Lorenzo Pieralisi @ 2015-07-10 15:47 ` Moore, Robert -1 siblings, 0 replies; 34+ messages in thread From: Moore, Robert @ 2015-07-10 15:47 UTC (permalink / raw) To: Lorenzo Pieralisi Cc: Ming Lei, Zheng, Lv, Wysocki, Rafael J, Linux Kernel Mailing List, linux-arm-kernel, Thomas Gleixner, Jason Cooper, hanjun.guo > -----Original Message----- > From: Lorenzo Pieralisi [mailto:lorenzo.pieralisi@arm.com] > Sent: Friday, July 10, 2015 8:18 AM > To: Moore, Robert > Cc: Ming Lei; Zheng, Lv; Wysocki, Rafael J; Linux Kernel Mailing List; > linux-arm-kernel; Thomas Gleixner; Jason Cooper; hanjun.guo@linaro.org > Subject: Re: ACPI: regression: Failed to initialize GIC IRQ controller > > On Fri, Jul 10, 2015 at 03:45:32PM +0100, Moore, Robert wrote: > > It's nice that someone took a sizeof() on the struct -- so, one would > hope that no code actually depended on a particular value, no? > > Unfortunately that sizeof has been there forever (x86/ia64), > ia64 code ran into a similar issue, so the check was removed to cope with > lsapic MADT updates, see: > > arch/ia64/kernel/acpi.c line 204 > > /*Skip BAD_MADT_ENTRY check, as lsapic size could vary */ > > Is checking the subtable length field against a static value really > worthwhile/suitable ? > I would at least traverse the subtables via the subtable length given in the table, and not use a sizeof() for each subtable. Then, multiple table/subtable versions are handled automatically; you don't have to use any new fields until necessary. > Thanks, > Lorenzo > > > > -----Original Message----- > > > From: Ming Lei [mailto:ming.lei@canonical.com] > > > Sent: Friday, July 10, 2015 7:43 AM > > > To: Moore, Robert > > > Cc: Zheng, Lv; Wysocki, Rafael J; Linux Kernel Mailing List; > > > linux-arm- kernel; Thomas Gleixner; Jason Cooper; Hanjun Guo > > > Subject: Re: ACPI: regression: Failed to initialize GIC IRQ > > > controller > > > > > > On Fri, Jul 10, 2015 at 10:28 PM, Moore, Robert > > > <robert.moore@intel.com> > > > wrote: > > > > > > > > > > > >> -----Original Message----- > > > >> From: Ming Lei [mailto:ming.lei@canonical.com] > > > >> Sent: Friday, July 10, 2015 12:46 AM > > > >> To: Moore, Robert; Zheng, Lv; Wysocki, Rafael J > > > >> Cc: Linux Kernel Mailing List; linux-arm-kernel; Thomas Gleixner; > > > >> Jason Cooper; Hanjun Guo > > > >> Subject: ACPI: regression: Failed to initialize GIC IRQ > > > >> controller > > > >> > > > >> Hi, > > > >> > > > >> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT > > > >> table.) causes the following failure on APM mustang board(arm64) > > > >> when booting via UEFI and ACPI: > > > > > > > > > > > > I would be interested to know just what exactly about this change > > > > broke > > > things. > > > > > > sizeof(struct acpi_madt_generic_interrupt) > > > N???????????????r??????y?????????b???X????????v???^???)??{.n???+?????????? > ??{????????????zX??????\x17????????}???????????z???&j:+v????????? > ????????????zZ+??????+zf?????????h?????????~????????????i?????????z???\x1e??? > w??????????????????????&???)??^[f??????^j??y???m??????@A???a??????\x7f??? > > 0??????h???\x0f???i\x7f ^ permalink raw reply [flat|nested] 34+ messages in thread
* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 15:47 ` Moore, Robert 0 siblings, 0 replies; 34+ messages in thread From: Moore, Robert @ 2015-07-10 15:47 UTC (permalink / raw) To: linux-arm-kernel > -----Original Message----- > From: Lorenzo Pieralisi [mailto:lorenzo.pieralisi at arm.com] > Sent: Friday, July 10, 2015 8:18 AM > To: Moore, Robert > Cc: Ming Lei; Zheng, Lv; Wysocki, Rafael J; Linux Kernel Mailing List; > linux-arm-kernel; Thomas Gleixner; Jason Cooper; hanjun.guo at linaro.org > Subject: Re: ACPI: regression: Failed to initialize GIC IRQ controller > > On Fri, Jul 10, 2015 at 03:45:32PM +0100, Moore, Robert wrote: > > It's nice that someone took a sizeof() on the struct -- so, one would > hope that no code actually depended on a particular value, no? > > Unfortunately that sizeof has been there forever (x86/ia64), > ia64 code ran into a similar issue, so the check was removed to cope with > lsapic MADT updates, see: > > arch/ia64/kernel/acpi.c line 204 > > /*Skip BAD_MADT_ENTRY check, as lsapic size could vary */ > > Is checking the subtable length field against a static value really > worthwhile/suitable ? > I would at least traverse the subtables via the subtable length given in the table, and not use a sizeof() for each subtable. Then, multiple table/subtable versions are handled automatically; you don't have to use any new fields until necessary. > Thanks, > Lorenzo > > > > -----Original Message----- > > > From: Ming Lei [mailto:ming.lei at canonical.com] > > > Sent: Friday, July 10, 2015 7:43 AM > > > To: Moore, Robert > > > Cc: Zheng, Lv; Wysocki, Rafael J; Linux Kernel Mailing List; > > > linux-arm- kernel; Thomas Gleixner; Jason Cooper; Hanjun Guo > > > Subject: Re: ACPI: regression: Failed to initialize GIC IRQ > > > controller > > > > > > On Fri, Jul 10, 2015 at 10:28 PM, Moore, Robert > > > <robert.moore@intel.com> > > > wrote: > > > > > > > > > > > >> -----Original Message----- > > > >> From: Ming Lei [mailto:ming.lei at canonical.com] > > > >> Sent: Friday, July 10, 2015 12:46 AM > > > >> To: Moore, Robert; Zheng, Lv; Wysocki, Rafael J > > > >> Cc: Linux Kernel Mailing List; linux-arm-kernel; Thomas Gleixner; > > > >> Jason Cooper; Hanjun Guo > > > >> Subject: ACPI: regression: Failed to initialize GIC IRQ > > > >> controller > > > >> > > > >> Hi, > > > >> > > > >> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT > > > >> table.) causes the following failure on APM mustang board(arm64) > > > >> when booting via UEFI and ACPI: > > > > > > > > > > > > I would be interested to know just what exactly about this change > > > > broke > > > things. > > > > > > sizeof(struct acpi_madt_generic_interrupt) > > > N???????????????r??????y?????????b???X????????v???^???)??{.n???+?????????? > ??{????????????zX??????\x17????????}???????????z???&j:+v????????? > ????????????zZ+??????+zf?????????h?????????~????????????i?????????z???\x1e ??? > w??????????????????????&???)??^[f??????^j??y???m??????@A???a??????\x7f??? > > 0??????h???\x0f???i\x7f ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: ACPI: regression: Failed to initialize GIC IRQ controller 2015-07-10 15:47 ` Moore, Robert @ 2015-07-10 17:02 ` Lorenzo Pieralisi -1 siblings, 0 replies; 34+ messages in thread From: Lorenzo Pieralisi @ 2015-07-10 17:02 UTC (permalink / raw) To: Moore, Robert Cc: Ming Lei, Zheng, Lv, Wysocki, Rafael J, Linux Kernel Mailing List, linux-arm-kernel, Thomas Gleixner, Jason Cooper, hanjun.guo On Fri, Jul 10, 2015 at 04:47:34PM +0100, Moore, Robert wrote: > > > > -----Original Message----- > > From: Lorenzo Pieralisi [mailto:lorenzo.pieralisi@arm.com] > > Sent: Friday, July 10, 2015 8:18 AM > > To: Moore, Robert > > Cc: Ming Lei; Zheng, Lv; Wysocki, Rafael J; Linux Kernel Mailing List; > > linux-arm-kernel; Thomas Gleixner; Jason Cooper; hanjun.guo@linaro.org > > Subject: Re: ACPI: regression: Failed to initialize GIC IRQ controller > > > > On Fri, Jul 10, 2015 at 03:45:32PM +0100, Moore, Robert wrote: > > > It's nice that someone took a sizeof() on the struct -- so, one would > > hope that no code actually depended on a particular value, no? > > > > Unfortunately that sizeof has been there forever (x86/ia64), > > ia64 code ran into a similar issue, so the check was removed to cope with > > lsapic MADT updates, see: > > > > arch/ia64/kernel/acpi.c line 204 > > > > /*Skip BAD_MADT_ENTRY check, as lsapic size could vary */ > > > > Is checking the subtable length field against a static value really > > worthwhile/suitable ? > > > > I would at least traverse the subtables via the subtable length given in the table, and not use a sizeof() for each subtable. Then, multiple table/subtable versions are handled automatically; you don't have to use any new fields until necessary. I lost you here, sorry. You are describing how the subtable entries are parsed in acpi_parse_entries, but that's not what we are debating here. BAD_MADT_ENTRY checks the subtable length against the ACPICA MADT structs sized through sizeof to determine if the length field is "correct", I do not see how you can do it by traversing the tables (how can you determine where a subtable _really_ ends or to put it differently how to check that a subtable length is _really_ right ?). Thanks, Lorenzo ^ permalink raw reply [flat|nested] 34+ messages in thread
* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 17:02 ` Lorenzo Pieralisi 0 siblings, 0 replies; 34+ messages in thread From: Lorenzo Pieralisi @ 2015-07-10 17:02 UTC (permalink / raw) To: linux-arm-kernel On Fri, Jul 10, 2015 at 04:47:34PM +0100, Moore, Robert wrote: > > > > -----Original Message----- > > From: Lorenzo Pieralisi [mailto:lorenzo.pieralisi at arm.com] > > Sent: Friday, July 10, 2015 8:18 AM > > To: Moore, Robert > > Cc: Ming Lei; Zheng, Lv; Wysocki, Rafael J; Linux Kernel Mailing List; > > linux-arm-kernel; Thomas Gleixner; Jason Cooper; hanjun.guo at linaro.org > > Subject: Re: ACPI: regression: Failed to initialize GIC IRQ controller > > > > On Fri, Jul 10, 2015 at 03:45:32PM +0100, Moore, Robert wrote: > > > It's nice that someone took a sizeof() on the struct -- so, one would > > hope that no code actually depended on a particular value, no? > > > > Unfortunately that sizeof has been there forever (x86/ia64), > > ia64 code ran into a similar issue, so the check was removed to cope with > > lsapic MADT updates, see: > > > > arch/ia64/kernel/acpi.c line 204 > > > > /*Skip BAD_MADT_ENTRY check, as lsapic size could vary */ > > > > Is checking the subtable length field against a static value really > > worthwhile/suitable ? > > > > I would at least traverse the subtables via the subtable length given in the table, and not use a sizeof() for each subtable. Then, multiple table/subtable versions are handled automatically; you don't have to use any new fields until necessary. I lost you here, sorry. You are describing how the subtable entries are parsed in acpi_parse_entries, but that's not what we are debating here. BAD_MADT_ENTRY checks the subtable length against the ACPICA MADT structs sized through sizeof to determine if the length field is "correct", I do not see how you can do it by traversing the tables (how can you determine where a subtable _really_ ends or to put it differently how to check that a subtable length is _really_ right ?). Thanks, Lorenzo ^ permalink raw reply [flat|nested] 34+ messages in thread
* RE: ACPI: regression: Failed to initialize GIC IRQ controller 2015-07-10 17:02 ` Lorenzo Pieralisi @ 2015-07-10 17:07 ` Moore, Robert -1 siblings, 0 replies; 34+ messages in thread From: Moore, Robert @ 2015-07-10 17:07 UTC (permalink / raw) To: Lorenzo Pieralisi Cc: Ming Lei, Zheng, Lv, Wysocki, Rafael J, Linux Kernel Mailing List, linux-arm-kernel, Thomas Gleixner, Jason Cooper, hanjun.guo > -----Original Message----- > From: Lorenzo Pieralisi [mailto:lorenzo.pieralisi@arm.com] > Sent: Friday, July 10, 2015 10:03 AM > To: Moore, Robert > Cc: Ming Lei; Zheng, Lv; Wysocki, Rafael J; Linux Kernel Mailing List; > linux-arm-kernel; Thomas Gleixner; Jason Cooper; hanjun.guo@linaro.org > Subject: Re: ACPI: regression: Failed to initialize GIC IRQ controller > > On Fri, Jul 10, 2015 at 04:47:34PM +0100, Moore, Robert wrote: > > > > > > > -----Original Message----- > > > From: Lorenzo Pieralisi [mailto:lorenzo.pieralisi@arm.com] > > > Sent: Friday, July 10, 2015 8:18 AM > > > To: Moore, Robert > > > Cc: Ming Lei; Zheng, Lv; Wysocki, Rafael J; Linux Kernel Mailing > > > List; linux-arm-kernel; Thomas Gleixner; Jason Cooper; > > > hanjun.guo@linaro.org > > > Subject: Re: ACPI: regression: Failed to initialize GIC IRQ > > > controller > > > > > > On Fri, Jul 10, 2015 at 03:45:32PM +0100, Moore, Robert wrote: > > > > It's nice that someone took a sizeof() on the struct -- so, one > > > > would > > > hope that no code actually depended on a particular value, no? > > > > > > Unfortunately that sizeof has been there forever (x86/ia64), > > > ia64 code ran into a similar issue, so the check was removed to cope > > > with lsapic MADT updates, see: > > > > > > arch/ia64/kernel/acpi.c line 204 > > > > > > /*Skip BAD_MADT_ENTRY check, as lsapic size could vary */ > > > > > > Is checking the subtable length field against a static value really > > > worthwhile/suitable ? > > > > > > > I would at least traverse the subtables via the subtable length given in > the table, and not use a sizeof() for each subtable. Then, multiple > table/subtable versions are handled automatically; you don't have to use > any new fields until necessary. > > I lost you here, sorry. You are describing how the subtable entries are > parsed in acpi_parse_entries, but that's not what we are debating here. > BAD_MADT_ENTRY checks the subtable length against the ACPICA MADT structs > sized through sizeof to determine if the length field is "correct", I do > not see how you can do it by traversing the tables (how can you determine > where a subtable _really_ ends or to put it differently how to check that > a subtable length is _really_ right ?). Ok, this is not the actual traversal. However, the traversal probably uses the subtable length entries already, or it least it should. I think that the subtable lengths are one field that absolutely *must* be trusted, else lots of things would break. Perhaps the MADT does not have variable-length subtable fields (I forget), but many ACPI tables do. Once you've got a subtable, you want to probable check it it is at least as long as you expect. Any longer should be treated as OK, since it probably contains fields that you don't know how to decipher anyway. > > Thanks, > Lorenzo ^ permalink raw reply [flat|nested] 34+ messages in thread
* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 17:07 ` Moore, Robert 0 siblings, 0 replies; 34+ messages in thread From: Moore, Robert @ 2015-07-10 17:07 UTC (permalink / raw) To: linux-arm-kernel > -----Original Message----- > From: Lorenzo Pieralisi [mailto:lorenzo.pieralisi at arm.com] > Sent: Friday, July 10, 2015 10:03 AM > To: Moore, Robert > Cc: Ming Lei; Zheng, Lv; Wysocki, Rafael J; Linux Kernel Mailing List; > linux-arm-kernel; Thomas Gleixner; Jason Cooper; hanjun.guo at linaro.org > Subject: Re: ACPI: regression: Failed to initialize GIC IRQ controller > > On Fri, Jul 10, 2015 at 04:47:34PM +0100, Moore, Robert wrote: > > > > > > > -----Original Message----- > > > From: Lorenzo Pieralisi [mailto:lorenzo.pieralisi at arm.com] > > > Sent: Friday, July 10, 2015 8:18 AM > > > To: Moore, Robert > > > Cc: Ming Lei; Zheng, Lv; Wysocki, Rafael J; Linux Kernel Mailing > > > List; linux-arm-kernel; Thomas Gleixner; Jason Cooper; > > > hanjun.guo at linaro.org > > > Subject: Re: ACPI: regression: Failed to initialize GIC IRQ > > > controller > > > > > > On Fri, Jul 10, 2015 at 03:45:32PM +0100, Moore, Robert wrote: > > > > It's nice that someone took a sizeof() on the struct -- so, one > > > > would > > > hope that no code actually depended on a particular value, no? > > > > > > Unfortunately that sizeof has been there forever (x86/ia64), > > > ia64 code ran into a similar issue, so the check was removed to cope > > > with lsapic MADT updates, see: > > > > > > arch/ia64/kernel/acpi.c line 204 > > > > > > /*Skip BAD_MADT_ENTRY check, as lsapic size could vary */ > > > > > > Is checking the subtable length field against a static value really > > > worthwhile/suitable ? > > > > > > > I would at least traverse the subtables via the subtable length given in > the table, and not use a sizeof() for each subtable. Then, multiple > table/subtable versions are handled automatically; you don't have to use > any new fields until necessary. > > I lost you here, sorry. You are describing how the subtable entries are > parsed in acpi_parse_entries, but that's not what we are debating here. > BAD_MADT_ENTRY checks the subtable length against the ACPICA MADT structs > sized through sizeof to determine if the length field is "correct", I do > not see how you can do it by traversing the tables (how can you determine > where a subtable _really_ ends or to put it differently how to check that > a subtable length is _really_ right ?). Ok, this is not the actual traversal. However, the traversal probably uses the subtable length entries already, or it least it should. I think that the subtable lengths are one field that absolutely *must* be trusted, else lots of things would break. Perhaps the MADT does not have variable-length subtable fields (I forget), but many ACPI tables do. Once you've got a subtable, you want to probable check it it is at least as long as you expect. Any longer should be treated as OK, since it probably contains fields that you don't know how to decipher anyway. > > Thanks, > Lorenzo ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: ACPI: regression: Failed to initialize GIC IRQ controller 2015-07-10 14:28 ` Moore, Robert @ 2015-07-10 14:47 ` Marc Zyngier -1 siblings, 0 replies; 34+ messages in thread From: Marc Zyngier @ 2015-07-10 14:47 UTC (permalink / raw) To: Moore, Robert, Ming Lei, Zheng, Lv, Wysocki, Rafael J Cc: Hanjun Guo, Thomas Gleixner, Linux Kernel Mailing List, linux-arm-kernel, Jason Cooper On 10/07/15 15:28, Moore, Robert wrote: > > >> -----Original Message----- >> From: Ming Lei [mailto:ming.lei@canonical.com] >> Sent: Friday, July 10, 2015 12:46 AM >> To: Moore, Robert; Zheng, Lv; Wysocki, Rafael J >> Cc: Linux Kernel Mailing List; linux-arm-kernel; Thomas Gleixner; Jason >> Cooper; Hanjun Guo >> Subject: ACPI: regression: Failed to initialize GIC IRQ controller >> >> Hi, >> >> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) >> causes the following failure on APM mustang board(arm64) when booting via >> UEFI and ACPI: > > > I would be interested to know just what exactly about this change broke things. The gory details are there: https://lkml.org/lkml/2015/7/6/876 Basically, some data structure (acpi_madt_generic_interrupt) grew by 4 bytes from ACPI 5.1 to 6.0, but BAD_MADT_ENTRY only knows about the new size, and not the old one. When booting on an old(er) version of ACPI, the interrupt controller tables are ignore (not the right size), and the machine stops booting very early. Thanks, M. > Bob > > > >> >> No valid GICC entries exist >> ACPI: Failed to initialize GIC IRQ controller Kernel panic - not syncing: >> No interrupt controller found. >> CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 Hardware name: >> APM X-Gene Mustang board (DT) Call trace: >> [<ffffffc000089b94>] dump_backtrace+0x0/0x12c [<ffffffc000089cd0>] >> show_stack+0x10/0x1c [<ffffffc0005fac18>] dump_stack+0x8c/0xdc >> [<ffffffc0005f7218>] panic+0xe4/0x220 [<ffffffc00082631c>] >> init_IRQ+0x24/0x30 [<ffffffc00082486c>] start_kernel+0x274/0x3d8 ---[ end >> Kernel panic - not syncing: No interrupt controller found. >> >> >> Thanks, >> Ming > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > -- Jazz is not dead. It just smells funny... ^ permalink raw reply [flat|nested] 34+ messages in thread
* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 14:47 ` Marc Zyngier 0 siblings, 0 replies; 34+ messages in thread From: Marc Zyngier @ 2015-07-10 14:47 UTC (permalink / raw) To: linux-arm-kernel On 10/07/15 15:28, Moore, Robert wrote: > > >> -----Original Message----- >> From: Ming Lei [mailto:ming.lei at canonical.com] >> Sent: Friday, July 10, 2015 12:46 AM >> To: Moore, Robert; Zheng, Lv; Wysocki, Rafael J >> Cc: Linux Kernel Mailing List; linux-arm-kernel; Thomas Gleixner; Jason >> Cooper; Hanjun Guo >> Subject: ACPI: regression: Failed to initialize GIC IRQ controller >> >> Hi, >> >> Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) >> causes the following failure on APM mustang board(arm64) when booting via >> UEFI and ACPI: > > > I would be interested to know just what exactly about this change broke things. The gory details are there: https://lkml.org/lkml/2015/7/6/876 Basically, some data structure (acpi_madt_generic_interrupt) grew by 4 bytes from ACPI 5.1 to 6.0, but BAD_MADT_ENTRY only knows about the new size, and not the old one. When booting on an old(er) version of ACPI, the interrupt controller tables are ignore (not the right size), and the machine stops booting very early. Thanks, M. > Bob > > > >> >> No valid GICC entries exist >> ACPI: Failed to initialize GIC IRQ controller Kernel panic - not syncing: >> No interrupt controller found. >> CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 Hardware name: >> APM X-Gene Mustang board (DT) Call trace: >> [<ffffffc000089b94>] dump_backtrace+0x0/0x12c [<ffffffc000089cd0>] >> show_stack+0x10/0x1c [<ffffffc0005fac18>] dump_stack+0x8c/0xdc >> [<ffffffc0005f7218>] panic+0xe4/0x220 [<ffffffc00082631c>] >> init_IRQ+0x24/0x30 [<ffffffc00082486c>] start_kernel+0x274/0x3d8 ---[ end >> Kernel panic - not syncing: No interrupt controller found. >> >> >> Thanks, >> Ming > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel at lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > -- Jazz is not dead. It just smells funny... ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: ACPI: regression: Failed to initialize GIC IRQ controller 2015-07-10 14:28 ` Moore, Robert @ 2015-07-10 14:48 ` Lorenzo Pieralisi -1 siblings, 0 replies; 34+ messages in thread From: Lorenzo Pieralisi @ 2015-07-10 14:48 UTC (permalink / raw) To: Moore, Robert Cc: Ming Lei, Zheng, Lv, Wysocki, Rafael J, Linux Kernel Mailing List, linux-arm-kernel, Thomas Gleixner, Jason Cooper, hanjun.guo On Fri, Jul 10, 2015 at 03:28:36PM +0100, Moore, Robert wrote: > > > > -----Original Message----- > > From: Ming Lei [mailto:ming.lei@canonical.com] > > Sent: Friday, July 10, 2015 12:46 AM > > To: Moore, Robert; Zheng, Lv; Wysocki, Rafael J > > Cc: Linux Kernel Mailing List; linux-arm-kernel; Thomas Gleixner; Jason > > Cooper; Hanjun Guo > > Subject: ACPI: regression: Failed to initialize GIC IRQ controller > > > > Hi, > > > > Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) > > causes the following failure on APM mustang board(arm64) when booting via > > UEFI and ACPI: > > > I would be interested to know just what exactly about this change broke things. struct acpi_madt_generic_interrupt gained 4 bytes, 4 too many for ACPI on ARM64 to boot with 5.1 tables. >From now onwards we have to test acpica changes against the arm64 kernel tree, this must not happen again. See below: https://lkml.org/lkml/2015/7/6/876 Thanks, Lorenzo > Bob > > > > > > > No valid GICC entries exist > > ACPI: Failed to initialize GIC IRQ controller Kernel panic - not syncing: > > No interrupt controller found. > > CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 Hardware name: > > APM X-Gene Mustang board (DT) Call trace: > > [<ffffffc000089b94>] dump_backtrace+0x0/0x12c [<ffffffc000089cd0>] > > show_stack+0x10/0x1c [<ffffffc0005fac18>] dump_stack+0x8c/0xdc > > [<ffffffc0005f7218>] panic+0xe4/0x220 [<ffffffc00082631c>] > > init_IRQ+0x24/0x30 [<ffffffc00082486c>] start_kernel+0x274/0x3d8 ---[ end > > Kernel panic - not syncing: No interrupt controller found. > > > > > > Thanks, > > Ming > N???????????????r??????y?????????b???X????????v???^???)??{.n???+????????????{????????????zX??????\x17????????}???????????z???&j:+v?????????\a????????????zZ+??????+zf?????????h?????????~????????????i?????????z???\x1e???w??????????????????????&???)??^[f??????^j??y???m??????@A???a??????\x7f???\f0??????h???\x0f???i\x7f ^ permalink raw reply [flat|nested] 34+ messages in thread
* ACPI: regression: Failed to initialize GIC IRQ controller @ 2015-07-10 14:48 ` Lorenzo Pieralisi 0 siblings, 0 replies; 34+ messages in thread From: Lorenzo Pieralisi @ 2015-07-10 14:48 UTC (permalink / raw) To: linux-arm-kernel On Fri, Jul 10, 2015 at 03:28:36PM +0100, Moore, Robert wrote: > > > > -----Original Message----- > > From: Ming Lei [mailto:ming.lei at canonical.com] > > Sent: Friday, July 10, 2015 12:46 AM > > To: Moore, Robert; Zheng, Lv; Wysocki, Rafael J > > Cc: Linux Kernel Mailing List; linux-arm-kernel; Thomas Gleixner; Jason > > Cooper; Hanjun Guo > > Subject: ACPI: regression: Failed to initialize GIC IRQ controller > > > > Hi, > > > > Commit 0cff8dc0099f6d4f(ACPICA: ACPI 6.0: Add changes for MADT table.) > > causes the following failure on APM mustang board(arm64) when booting via > > UEFI and ACPI: > > > I would be interested to know just what exactly about this change broke things. struct acpi_madt_generic_interrupt gained 4 bytes, 4 too many for ACPI on ARM64 to boot with 5.1 tables. >From now onwards we have to test acpica changes against the arm64 kernel tree, this must not happen again. See below: https://lkml.org/lkml/2015/7/6/876 Thanks, Lorenzo > Bob > > > > > > > No valid GICC entries exist > > ACPI: Failed to initialize GIC IRQ controller Kernel panic - not syncing: > > No interrupt controller found. > > CPU: 0 PID: 0 Comm: swapper/0 Not tainted 4.2.0-rc1+ #45 Hardware name: > > APM X-Gene Mustang board (DT) Call trace: > > [<ffffffc000089b94>] dump_backtrace+0x0/0x12c [<ffffffc000089cd0>] > > show_stack+0x10/0x1c [<ffffffc0005fac18>] dump_stack+0x8c/0xdc > > [<ffffffc0005f7218>] panic+0xe4/0x220 [<ffffffc00082631c>] > > init_IRQ+0x24/0x30 [<ffffffc00082486c>] start_kernel+0x274/0x3d8 ---[ end > > Kernel panic - not syncing: No interrupt controller found. > > > > > > Thanks, > > Ming > N???????????????r??????y?????????b???X????????v???^???)??{.n???+????????????{????????????zX??????\x17????????}???????????z???&j:+v?????????\a????????????zZ+??????+zf?????????h?????????~????????????i?????????z???\x1e ???w??????????????????????&???)??^[f??????^j??y???m??????@A???a??????\x7f???\f 0??????h???\x0f???i\x7f ^ permalink raw reply [flat|nested] 34+ messages in thread
end of thread, other threads:[~2015-07-10 17:07 UTC | newest] Thread overview: 34+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2015-07-10 7:45 ACPI: regression: Failed to initialize GIC IRQ controller Ming Lei 2015-07-10 7:45 ` Ming Lei 2015-07-10 7:58 ` Marc Zyngier 2015-07-10 7:58 ` Marc Zyngier 2015-07-10 8:17 ` Suman Tripathi 2015-07-10 8:17 ` Suman Tripathi 2015-07-10 8:23 ` Marc Zyngier 2015-07-10 8:23 ` Marc Zyngier 2015-07-10 9:37 ` Suman Tripathi 2015-07-10 9:37 ` Suman Tripathi 2015-07-10 10:11 ` Ming Lei 2015-07-10 10:11 ` Ming Lei 2015-07-10 11:10 ` Hanjun Guo 2015-07-10 11:10 ` Hanjun Guo 2015-07-10 10:49 ` Hanjun Guo 2015-07-10 10:49 ` Hanjun Guo 2015-07-10 14:28 ` Moore, Robert 2015-07-10 14:28 ` Moore, Robert 2015-07-10 14:43 ` Ming Lei 2015-07-10 14:43 ` Ming Lei 2015-07-10 14:45 ` Moore, Robert 2015-07-10 14:45 ` Moore, Robert 2015-07-10 15:17 ` Lorenzo Pieralisi 2015-07-10 15:17 ` Lorenzo Pieralisi 2015-07-10 15:47 ` Moore, Robert 2015-07-10 15:47 ` Moore, Robert 2015-07-10 17:02 ` Lorenzo Pieralisi 2015-07-10 17:02 ` Lorenzo Pieralisi 2015-07-10 17:07 ` Moore, Robert 2015-07-10 17:07 ` Moore, Robert 2015-07-10 14:47 ` Marc Zyngier 2015-07-10 14:47 ` Marc Zyngier 2015-07-10 14:48 ` Lorenzo Pieralisi 2015-07-10 14:48 ` Lorenzo Pieralisi
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.