From: Marc Zyngier <maz@kernel.org> To: Christian Zigotzky <chzigotzky@xenosoft.de> Cc: Bjorn Helgaas <helgaas@kernel.org>, "bhelgaas@google.com >> Bjorn Helgaas" <bhelgaas@google.com>, Alyssa Rosenzweig <alyssa@rosenzweig.io>, lorenzo.pieralisi@arm.com, Rob Herring <robh@kernel.org>, Matthew Leaman <matthew@a-eon.biz>, Darren Stevens <darren@stevens-zone.net>, mad skateman <madskateman@gmail.com>, "R.T.Dickinson" <rtd2@xtra.co.nz>, Christian Zigotzky <info@xenosoft.de>, axboe@kernel.dk, damien.lemoal@opensource.wdc.com, kw@linux.com, Arnd Bergmann <arnd@arndb.de>, robert@swiecki.net, Olof Johansson <olof@lixom.net>, linuxppc-dev <linuxppc-dev@lists.ozlabs.org>, "linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org> Subject: Re: [PASEMI] Nemo board doesn't recognize any ATA disks with the pci-v5.16 updates Date: Thu, 11 Nov 2021 07:13:26 +0000 [thread overview] Message-ID: <87r1bn88rt.wl-maz@kernel.org> (raw) In-Reply-To: <8cc64c3b-b0c0-fb41-9836-2e5e6a4459d1@xenosoft.de> On Thu, 11 Nov 2021 05:24:52 +0000, Christian Zigotzky <chzigotzky@xenosoft.de> wrote: > > On 10 November 2021 at 08:09 pm, Marc Zyngier wrote: > > HI all, > > > > On Wed, 10 Nov 2021 18:41:06 +0000, > > Bjorn Helgaas <helgaas@kernel.org> wrote: > >> On Wed, Nov 10, 2021 at 07:07:24PM +0100, Christian Zigotzky wrote: > >>> On 09 November 2021 at 03:45 pm, Christian Zigotzky wrote: > >>>> Hello, > >>>> > >>>> The Nemo board [1] doesn't recognize any ATA disks with the pci-v5.16 > >>> updates [2]. > >>>> Error messages: > >>>> > >>>> ata4.00: gc timeout cmd 0xec > >>>> ata4.00: failed to IDENTIFY (I/O error, error_mask=0x4) > >>>> ata1.00: gc timeout cmd 0xec > >>>> ata1.00: failed to IDENTIFY (I/O error, error_mask=0x4) > >>>> ata3.00: gc timeout cmd 0xec > >>>> ata3.00: failed to IDENTIFY (I/O error, error_mask=0x4) > >>>> > >>>> I was able to revert the new pci-v5.16 updates [2]. After a new > >>> compiling, the kernel recognize all ATA disks correctly. > >>>> Could you please check the pci-v5.16 updates [2]? > >>>> > >>>> Please find attached the kernel config. > >>>> > >>>> Thanks, > >>>> Christian > >>>> > >>>> [1] https://en.wikipedia.org/wiki/AmigaOne_X1000 > >>>> [2] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=0c5c62ddf88c34bc83b66e4ac9beb2bb0e1887d4 > >>> Hi All, > >>> > >>> Many thanks for your nice responses. > >>> > >>> I bisected today [1]. 0412841812265734c306ba5ef8088bcb64d5d3bd (of/irq: > >>> Allow matching of an interrupt-map local to an interrupt controller) [2] is > >>> the first bad commit. > >>> > >>> I was able to revert the first bad commit [1]. After a new compiling, the > >>> kernel detects all ATA disks without any problems. > >>> > >>> I created a patch for an easy reverting the bad commit [1]. With this patch > >>> we can do further our kernel tests. > >>> > >>> Could you please check the first bad commit [2]? > >>> > >>> Thanks, > >>> Christian > >>> > >>> [1] https://forum.hyperion-entertainment.com/viewtopic.php?p=54398#p54398 > >>> [2] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=0412841812265734c306ba5ef8088bcb64d5d3bd > >>> > >>> [+ Marc Zyngier, Alyssa Rosenzweig, Lorenzo Pieralisi, and Rob Herring > >>> because of the first bad commit] > >> Thank you very much for the bisection and for also testing the revert! > >> > >> It's easy enough to revert 041284181226 ("of/irq: Allow matching of an > >> interrupt-map local to an interrupt controller"), and it seems like > >> that's what we need to do. I have it tentatively queued up. > >> > >> That commit was part of the new support for the Apple M1 PCIe > >> interface, and I don't know what effect a revert will have on that > >> support. Marc, Alyssa? > > It is going to badly break the M1 support, as we won't be able to take > > interrupts to detect that the PCIe link is up. > > > > Before we apply a full blown revert and decide that this isn't > > workable (and revert the whole M1 PCIe series, because they are > > otherwise somewhat pointless), I'd like to understand *what* breaks > > exactly. > > > > Christian, could you point me to the full DT that this machine uses? > > This would help understanding what goes wrong, and cook something for > > you to test. > > > > Thanks, > > > > M. > > > Hello Marc, > > Here you are: > https://forum.hyperion-entertainment.com/viewtopic.php?p=54406#p54406 This is not what I asked. I need the actual source file, or at the very least the compiled object (the /sys/firmware/fdt file, for example). Not an interpretation that I can't feed to the kernel. Without this, I can't debug your problem. > We are very happy to have the patch for reverting the bad commit > because we want to test the new PASEMI i2c driver with support for the > Apple M1 [1] on our Nemo boards. You can revert the patch on your own. At this stage, we're not blindly reverting things in the tree, but instead trying to understand what is happening on your particular system. Thanks, M. -- Without deviation from the norm, progress is not possible.
WARNING: multiple messages have this Message-ID (diff)
From: Marc Zyngier <maz@kernel.org> To: Christian Zigotzky <chzigotzky@xenosoft.de> Cc: axboe@kernel.dk, Rob Herring <robh@kernel.org>, lorenzo.pieralisi@arm.com, "R.T.Dickinson" <rtd2@xtra.co.nz>, Arnd Bergmann <arnd@arndb.de>, kw@linux.com, "linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>, damien.lemoal@opensource.wdc.com, Olof Johansson <olof@lixom.net>, Darren Stevens <darren@stevens-zone.net>, Bjorn Helgaas <helgaas@kernel.org>, mad skateman <madskateman@gmail.com>, "bhelgaas@google.com >> Bjorn Helgaas" <bhelgaas@google.com>, robert@swiecki.net, Matthew Leaman <matthew@a-eon.biz>, linuxppc-dev <linuxppc-dev@lists.ozlabs.org>, Alyssa Rosenzweig <alyssa@rosenzweig.io>, Christian Zigotzky <info@xenosoft.de> Subject: Re: [PASEMI] Nemo board doesn't recognize any ATA disks with the pci-v5.16 updates Date: Thu, 11 Nov 2021 07:13:26 +0000 [thread overview] Message-ID: <87r1bn88rt.wl-maz@kernel.org> (raw) In-Reply-To: <8cc64c3b-b0c0-fb41-9836-2e5e6a4459d1@xenosoft.de> On Thu, 11 Nov 2021 05:24:52 +0000, Christian Zigotzky <chzigotzky@xenosoft.de> wrote: > > On 10 November 2021 at 08:09 pm, Marc Zyngier wrote: > > HI all, > > > > On Wed, 10 Nov 2021 18:41:06 +0000, > > Bjorn Helgaas <helgaas@kernel.org> wrote: > >> On Wed, Nov 10, 2021 at 07:07:24PM +0100, Christian Zigotzky wrote: > >>> On 09 November 2021 at 03:45 pm, Christian Zigotzky wrote: > >>>> Hello, > >>>> > >>>> The Nemo board [1] doesn't recognize any ATA disks with the pci-v5.16 > >>> updates [2]. > >>>> Error messages: > >>>> > >>>> ata4.00: gc timeout cmd 0xec > >>>> ata4.00: failed to IDENTIFY (I/O error, error_mask=0x4) > >>>> ata1.00: gc timeout cmd 0xec > >>>> ata1.00: failed to IDENTIFY (I/O error, error_mask=0x4) > >>>> ata3.00: gc timeout cmd 0xec > >>>> ata3.00: failed to IDENTIFY (I/O error, error_mask=0x4) > >>>> > >>>> I was able to revert the new pci-v5.16 updates [2]. After a new > >>> compiling, the kernel recognize all ATA disks correctly. > >>>> Could you please check the pci-v5.16 updates [2]? > >>>> > >>>> Please find attached the kernel config. > >>>> > >>>> Thanks, > >>>> Christian > >>>> > >>>> [1] https://en.wikipedia.org/wiki/AmigaOne_X1000 > >>>> [2] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=0c5c62ddf88c34bc83b66e4ac9beb2bb0e1887d4 > >>> Hi All, > >>> > >>> Many thanks for your nice responses. > >>> > >>> I bisected today [1]. 0412841812265734c306ba5ef8088bcb64d5d3bd (of/irq: > >>> Allow matching of an interrupt-map local to an interrupt controller) [2] is > >>> the first bad commit. > >>> > >>> I was able to revert the first bad commit [1]. After a new compiling, the > >>> kernel detects all ATA disks without any problems. > >>> > >>> I created a patch for an easy reverting the bad commit [1]. With this patch > >>> we can do further our kernel tests. > >>> > >>> Could you please check the first bad commit [2]? > >>> > >>> Thanks, > >>> Christian > >>> > >>> [1] https://forum.hyperion-entertainment.com/viewtopic.php?p=54398#p54398 > >>> [2] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=0412841812265734c306ba5ef8088bcb64d5d3bd > >>> > >>> [+ Marc Zyngier, Alyssa Rosenzweig, Lorenzo Pieralisi, and Rob Herring > >>> because of the first bad commit] > >> Thank you very much for the bisection and for also testing the revert! > >> > >> It's easy enough to revert 041284181226 ("of/irq: Allow matching of an > >> interrupt-map local to an interrupt controller"), and it seems like > >> that's what we need to do. I have it tentatively queued up. > >> > >> That commit was part of the new support for the Apple M1 PCIe > >> interface, and I don't know what effect a revert will have on that > >> support. Marc, Alyssa? > > It is going to badly break the M1 support, as we won't be able to take > > interrupts to detect that the PCIe link is up. > > > > Before we apply a full blown revert and decide that this isn't > > workable (and revert the whole M1 PCIe series, because they are > > otherwise somewhat pointless), I'd like to understand *what* breaks > > exactly. > > > > Christian, could you point me to the full DT that this machine uses? > > This would help understanding what goes wrong, and cook something for > > you to test. > > > > Thanks, > > > > M. > > > Hello Marc, > > Here you are: > https://forum.hyperion-entertainment.com/viewtopic.php?p=54406#p54406 This is not what I asked. I need the actual source file, or at the very least the compiled object (the /sys/firmware/fdt file, for example). Not an interpretation that I can't feed to the kernel. Without this, I can't debug your problem. > We are very happy to have the patch for reverting the bad commit > because we want to test the new PASEMI i2c driver with support for the > Apple M1 [1] on our Nemo boards. You can revert the patch on your own. At this stage, we're not blindly reverting things in the tree, but instead trying to understand what is happening on your particular system. Thanks, M. -- Without deviation from the norm, progress is not possible.
next prev parent reply other threads:[~2021-11-11 7:13 UTC|newest] Thread overview: 137+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-04-30 11:59 Radeon NI: GIT kernel with the nislands_smc commit doesn't boot on a Freescale P5040 board and P.A.Semi Nemo board Christian Zigotzky 2021-04-30 11:59 ` Christian Zigotzky 2021-04-30 15:26 ` Deucher, Alexander 2021-04-30 15:26 ` Deucher, Alexander 2021-04-30 15:26 ` Deucher, Alexander 2021-04-30 15:56 ` Gustavo A. R. Silva 2021-04-30 15:56 ` Gustavo A. R. Silva 2021-04-30 15:56 ` Gustavo A. R. Silva 2021-05-03 22:25 ` [FSL P50x0] Xorg always restarts again and again after the the PowerPC updates 5.13-1 Christian Zigotzky 2021-05-04 4:56 ` Christophe Leroy 2021-05-04 7:21 ` Christian Zigotzky 2021-05-04 7:47 ` Christophe Leroy 2021-05-04 8:29 ` Christian Zigotzky 2021-05-04 8:58 ` Christophe Leroy 2021-05-04 9:09 ` Christian Zigotzky 2021-05-04 9:11 ` Christophe Leroy 2021-05-04 9:46 ` Christian Zigotzky 2021-05-04 9:49 ` Christophe Leroy 2021-05-04 10:07 ` Christian Zigotzky 2021-05-04 11:02 ` Christian Zigotzky 2021-05-04 13:48 ` Christian Zigotzky 2021-05-04 14:48 ` Christophe Leroy 2021-05-04 14:52 ` Christian Zigotzky 2021-05-04 14:41 ` Christophe Leroy 2021-05-04 14:59 ` Christian Zigotzky 2021-05-04 15:17 ` Christophe Leroy 2021-05-05 12:43 ` Christian Zigotzky 2021-05-06 6:12 ` Christophe Leroy 2021-05-06 7:56 ` Christian Zigotzky 2021-05-06 8:09 ` Christophe Leroy 2021-05-06 13:58 ` Christian Zigotzky 2021-05-08 16:39 ` Christian Zigotzky 2021-05-09 12:27 ` Christian Zigotzky 2021-05-09 17:43 ` Christophe Leroy 2021-05-09 21:46 ` Christian Zigotzky 2021-05-09 17:36 ` Christophe Leroy 2021-05-13 6:47 ` Christian Zigotzky 2021-05-13 10:01 ` Christophe Leroy 2021-05-13 15:19 ` Christian Zigotzky 2021-05-13 15:51 ` Christophe Leroy 2021-05-13 16:35 ` Christian Zigotzky 2021-05-13 17:00 ` Christophe Leroy 2021-05-13 20:20 ` Christian Zigotzky 2021-05-13 22:58 ` Nicholas Piggin 2021-05-14 0:19 ` Christian Zigotzky 2021-05-04 14:44 ` Christophe Leroy 2021-05-06 23:55 ` Radeon NI: GIT kernel with the nislands_smc commit doesn't boot on a Freescale P5040 board and P.A.Semi Nemo board Gustavo A. R. Silva 2021-05-06 23:55 ` Gustavo A. R. Silva 2021-05-07 6:43 ` Christian Zigotzky 2021-05-07 6:43 ` Christian Zigotzky 2021-05-08 11:33 ` Christian Zigotzky 2021-05-08 11:33 ` Christian Zigotzky 2021-05-09 23:04 ` Gustavo A. R. Silva 2021-05-09 23:04 ` Gustavo A. R. Silva 2023-11-02 14:45 ` Fbdev issue after the drm updates 'drm-next-2023-10-31-1' Christian Zigotzky 2023-11-07 8:36 ` Christian Zigotzky 2023-11-12 14:23 ` Christian Zigotzky 2023-11-13 12:48 ` Geert Uytterhoeven 2023-11-13 12:48 ` Geert Uytterhoeven 2023-11-13 13:12 ` Christian Zigotzky 2023-11-13 13:12 ` Christian Zigotzky 2023-11-14 9:45 ` Christian Zigotzky 2023-11-14 9:45 ` Christian Zigotzky 2023-11-15 8:33 ` Geert Uytterhoeven 2023-11-15 8:33 ` Geert Uytterhoeven 2023-11-15 8:33 ` Geert Uytterhoeven 2023-11-15 9:22 ` Gerd Hoffmann 2023-11-15 9:22 ` Gerd Hoffmann 2023-11-15 9:22 ` Gerd Hoffmann 2023-11-16 13:13 ` Geert Uytterhoeven 2023-11-16 13:13 ` Geert Uytterhoeven 2023-11-16 13:13 ` Geert Uytterhoeven 2021-11-04 10:45 ` [VirtIO GPU] Xorg doesn't start with the DRM updates 'drm-next-2021-11-03' in a virtual e5500 QEMU KVM-HV machine on a Freescale P5040 board Christian Zigotzky 2021-11-04 21:42 ` [PATCH] drm/virtio: Fix NULL dereference error in virtio_gpu_poll Vivek Kasireddy 2021-11-05 18:05 ` Christian Zigotzky 2021-11-05 18:05 ` Christian Zigotzky 2021-11-08 12:56 ` Gerd Hoffmann 2021-11-08 14:48 ` [FSL P50x0] Cyrus+ board doesn't boot with the PowerPC updates 5.16-1 Christian Zigotzky 2021-11-09 14:45 ` [PASEMI] Nemo board doesn't recognize any ATA disks with the pci-v5.16 updates Christian Zigotzky 2021-11-09 15:10 ` Christian Zigotzky 2021-11-09 15:10 ` Christian Zigotzky 2021-11-09 16:58 ` Bjorn Helgaas 2021-11-09 16:58 ` Bjorn Helgaas 2021-11-09 22:40 ` Krzysztof Wilczyński 2021-11-09 22:40 ` Krzysztof Wilczyński 2021-11-09 23:05 ` Arnd Bergmann 2021-11-09 23:05 ` Arnd Bergmann 2021-11-09 23:18 ` Krzysztof Wilczyński 2021-11-09 23:18 ` Krzysztof Wilczyński 2021-11-10 16:42 ` Robert Święcki 2021-11-10 16:42 ` Robert Święcki 2021-11-10 3:52 ` Damien Le Moal 2021-11-10 3:52 ` Damien Le Moal 2021-11-10 18:07 ` Christian Zigotzky 2021-11-10 18:07 ` Christian Zigotzky 2021-11-10 18:41 ` Bjorn Helgaas 2021-11-10 18:41 ` Bjorn Helgaas 2021-11-10 19:09 ` Marc Zyngier 2021-11-10 19:09 ` Marc Zyngier 2021-11-11 5:24 ` Christian Zigotzky 2021-11-11 5:24 ` Christian Zigotzky 2021-11-11 7:13 ` Marc Zyngier [this message] 2021-11-11 7:13 ` Marc Zyngier 2021-11-11 7:47 ` Christian Zigotzky 2021-11-11 7:47 ` Christian Zigotzky 2021-11-11 10:20 ` Marc Zyngier 2021-11-11 10:20 ` Marc Zyngier 2021-11-11 10:44 ` Christian Zigotzky 2021-11-11 10:44 ` Christian Zigotzky 2021-11-11 11:24 ` Marc Zyngier 2021-11-11 11:24 ` Marc Zyngier 2021-11-11 11:54 ` Christian Zigotzky 2021-11-11 11:54 ` Christian Zigotzky 2021-11-11 22:21 ` Olof Johansson 2021-11-11 22:21 ` Olof Johansson 2021-11-12 11:49 ` Segher Boessenkool 2021-11-12 11:49 ` Segher Boessenkool 2021-11-11 17:39 ` Marc Zyngier 2021-11-11 17:39 ` Marc Zyngier 2021-11-12 9:40 ` Christian Zigotzky 2021-11-12 9:40 ` Christian Zigotzky 2021-11-12 10:11 ` Christian Zigotzky 2021-11-12 10:11 ` Christian Zigotzky 2021-11-12 11:00 ` Christian Zigotzky 2021-11-12 11:00 ` Christian Zigotzky 2021-11-12 13:41 ` Marc Zyngier 2021-11-12 13:41 ` Marc Zyngier 2021-11-12 14:15 ` Christian Zigotzky 2021-11-12 14:15 ` Christian Zigotzky 2021-11-12 14:46 ` Marc Zyngier 2021-11-12 14:46 ` Marc Zyngier 2021-11-12 15:01 ` Christian Zigotzky 2021-11-12 15:01 ` Christian Zigotzky 2021-11-12 15:05 ` Christian Zigotzky 2021-11-12 15:05 ` Christian Zigotzky 2021-11-12 16:14 ` [irqchip: irq/irqchip-fixes] of/irq: Don't ignore interrupt-controller when interrupt-map failed irqchip-bot for Marc Zyngier 2021-11-09 16:13 ` [FSL P50x0] Cyrus+ board doesn't boot with the PowerPC updates 5.16-1 Christophe Leroy
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=87r1bn88rt.wl-maz@kernel.org \ --to=maz@kernel.org \ --cc=alyssa@rosenzweig.io \ --cc=arnd@arndb.de \ --cc=axboe@kernel.dk \ --cc=bhelgaas@google.com \ --cc=chzigotzky@xenosoft.de \ --cc=damien.lemoal@opensource.wdc.com \ --cc=darren@stevens-zone.net \ --cc=helgaas@kernel.org \ --cc=info@xenosoft.de \ --cc=kw@linux.com \ --cc=linux-pci@vger.kernel.org \ --cc=linuxppc-dev@lists.ozlabs.org \ --cc=lorenzo.pieralisi@arm.com \ --cc=madskateman@gmail.com \ --cc=matthew@a-eon.biz \ --cc=olof@lixom.net \ --cc=robert@swiecki.net \ --cc=robh@kernel.org \ --cc=rtd2@xtra.co.nz \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.