[REGRESSION] Re: next/master bisection: baseline.bootrr.intel-igb-probed on kontron-pitx-imx8m
Saravana Kannan
saravanak at google.com
Fri Aug 12 16:48:23 PDT 2022
On Fri, Aug 12, 2022 at 12:30 PM Bjorn Helgaas <helgaas at kernel.org> wrote:
>
> [+cc regressions]
>
> Michael says likely the same as:
> https://lore.kernel.org/lkml/Yr3vEDDulZj1Dplv@sirena.org.uk/
> and Saravana suggests the root cause and a patch are at:
> https://lore.kernel.org/lkml/CAGETcx_1qa=gGT4LVkyPpcA1vFM9FzuJE+0DhL_nFyg5cbFjVg@mail.gmail.com/
I sent a series that should fix all the issues reported in that
thread. Can anyone give this a shot?
https://lore.kernel.org/lkml/20220810060040.321697-1-saravanak@google.com/
-Saravana
>
> On Mon, Aug 08, 2022 at 01:22:47PM +0100, Mark Brown wrote:
> > On Sat, Aug 06, 2022 at 01:48:25PM -0700, KernelCI bot wrote:
> >
> > The KernelCI bisection bot found that 5a46079a96451 "PM: domains: Delete
> > usage of driver_deferred_probe_check_state()" triggered a failure to
> > probe the PCI controller or attached ethernet on kontron-pitx-imx8m.
> > There's no obvious errors in the boot log when things fail, we simply
> > don't see any of the announcements from the PCI controller probe like we
> > do on a working boot:
> >
> > <6>[ 2.434563] imx6q-pcie 33800000.pcie: host bridge /soc at 0/pcie at 33800000 ranges:
> > <6>[ 2.436149] imx6q-pcie 33c00000.pcie: host bridge /soc at 0/pcie at 33c00000 ranges:
> > <6>[ 2.436168] imx6q-pcie 33c00000.pcie: No bus range found for /soc at 0/pcie at 33c00000, using [bus 00-ff]
> > <6>[ 2.436208] imx6q-pcie 33c00000.pcie: IO 0x0027f80000..0x0027f8ffff -> 0x0000000000
> > <6>[ 2.436233] imx6q-pcie 33c00000.pcie: MEM 0x0020000000..0x0027efffff -> 0x0020000000
> > <6>[ 2.436985] imx6q-pcie 33c00000.pcie: iATU unroll: enabled
> > <6>[ 2.436993] imx6q-pcie 33c00000.pcie: Detected iATU regions: 4 outbound, 4 inbound
> > <6>[ 2.439329] sdhci-esdhc-imx 30b50000.mmc: Got WP GPIO
> > <6>[ 2.447119] imx6q-pcie 33800000.pcie: IO 0x001ff80000..0x001ff8ffff -> 0x0000000000
> > <4>[ 2.455485] sdhci-esdhc-imx 30b50000.mmc: drop HS400 support since no 8-bit bus
> > <6>[ 2.464884] imx6q-pcie 33800000.pcie: MEM 0x0018000000..0x001fefffff -> 0x0018000000
> > <6>[ 2.464895] mmc0: SDHCI controller on 30b40000.mmc [30b40000.mmc] using ADMA
> > <6>[ 2.504586] mmc1: SDHCI controller on 30b50000.mmc [30b50000.mmc] using ADMA
> > <6>[ 2.537041] imx6q-pcie 33c00000.pcie: Link up
> > <6>[ 2.547713] imx6q-pcie 33c00000.pcie: Link: Gen2 disabled
> > <6>[ 2.553652] imx6q-pcie 33c00000.pcie: Link up, Gen1
> > <6>[ 2.559036] imx6q-pcie 33c00000.pcie: Link up
> > <6>[ 2.563987] imx6q-pcie 33c00000.pcie: PCI host bridge to bus 0001:00
> >
> > https://storage.kernelci.org/mainline/master/v5.19-7280-gb44f2fd87919b/arm64/defconfig/clang-14/lab-kontron/baseline-kontron-pitx-imx8m.txt
> >
> > (interleaved with the SDHCI probe which is in the failing boot) though I
> > am wondering if the "no bus range found..." warning above might have
> > been escallated into a failure to probe.
> >
> > I've included the full bisection report below, including links to more
> > information including full boot failing logs plus a reported-by tag for
> > the bot.
> >
> > > * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
> > > * This automated bisection report was sent to you on the basis *
> > > * that you may be involved with the breaking commit it has *
> > > * found. No manual investigation has been done to verify it, *
> > > * and the root cause of the problem may be somewhere else. *
> > > * *
> > > * If you do send a fix, please include this trailer: *
> > > * Reported-by: "kernelci.org bot" <bot at kernelci.org> *
> > > * *
> > > * Hope this helps! *
> > > * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
> > >
> > > next/master bisection: baseline.bootrr.intel-igb-probed on kontron-pitx-imx8m
> > >
> > > Summary:
> > > Start: 861397378de91 Add linux-next specific files for 20220803
> > > Plain log: https://storage.kernelci.org/next/master/next-20220803/arm64/defconfig+crypto/gcc-10/lab-kontron/baseline-kontron-pitx-imx8m.txt
> > > HTML log: https://storage.kernelci.org/next/master/next-20220803/arm64/defconfig+crypto/gcc-10/lab-kontron/baseline-kontron-pitx-imx8m.html
> > > Result: 5a46079a96451 PM: domains: Delete usage of driver_deferred_probe_check_state()
> > >
> > > Checks:
> > > revert: PASS
> > > verify: PASS
> > >
> > > Parameters:
> > > Tree: next
> > > URL: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
> > > Branch: master
> > > Target: kontron-pitx-imx8m
> > > CPU arch: arm64
> > > Lab: lab-kontron
> > > Compiler: gcc-10
> > > Config: defconfig+crypto
> > > Test case: baseline.bootrr.intel-igb-probed
> > >
> > > Breaking commit found:
> > >
> > > -------------------------------------------------------------------------------
> > > commit 5a46079a96451cfb15e4f5f01f73f7ba24ef851a
> > > Author: Saravana Kannan <saravanak at google.com>
> > > Date: Wed Jun 1 00:06:57 2022 -0700
> > >
> > > PM: domains: Delete usage of driver_deferred_probe_check_state()
> > >
> > > Now that fw_devlink=on by default and fw_devlink supports
> > > "power-domains" property, the execution will never get to the point
> > > where driver_deferred_probe_check_state() is called before the supplier
> > > has probed successfully or before deferred probe timeout has expired.
> > >
> > > So, delete the call and replace it with -ENODEV.
> > >
> > > Tested-by: Geert Uytterhoeven <geert+renesas at glider.be>
> > > Reviewed-by: Ulf Hansson <ulf.hansson at linaro.org>
> > > Signed-off-by: Saravana Kannan <saravanak at google.com>
> > > Link: https://lore.kernel.org/r/20220601070707.3946847-2-saravanak@google.com
> > > Signed-off-by: Greg Kroah-Hartman <gregkh at linuxfoundation.org>
> > >
> > > diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c
> > > index 739e52cd4aba5..3e86772d5fac5 100644
> > > --- a/drivers/base/power/domain.c
> > > +++ b/drivers/base/power/domain.c
> > > @@ -2730,7 +2730,7 @@ static int __genpd_dev_pm_attach(struct device *dev, struct device *base_dev,
> > > mutex_unlock(&gpd_list_lock);
> > > dev_dbg(dev, "%s() failed to find PM domain: %ld\n",
> > > __func__, PTR_ERR(pd));
> > > - return driver_deferred_probe_check_state(base_dev);
> > > + return -ENODEV;
> > > }
> > >
> > > dev_dbg(dev, "adding to PM domain %s\n", pd->name);
> > > -------------------------------------------------------------------------------
> > >
> > >
> > > Git bisection log:
> > >
> > > -------------------------------------------------------------------------------
> > > git bisect start
> > > # good: [7c6327c77d509e78bff76f2a4551fcfee851682e] Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net
> > > git bisect good 7c6327c77d509e78bff76f2a4551fcfee851682e
> > > # bad: [861397378de91c64dec69a160595b891f443294f] Add linux-next specific files for 20220803
> > > git bisect bad 861397378de91c64dec69a160595b891f443294f
> > > # good: [9caf5c540b57105ac31f351518d83e1884c3d64a] Merge branch 'master' of git://linuxtv.org/media_tree.git
> > > git bisect good 9caf5c540b57105ac31f351518d83e1884c3d64a
> > > # good: [5ec97d04fa30ac49b48a141c0ee6bbe3038b0dad] Merge branch 'next' of git://git.kernel.org/pub/scm/virt/kvm/kvm.git
> > > git bisect good 5ec97d04fa30ac49b48a141c0ee6bbe3038b0dad
> > > # bad: [f95826f64a15aefb5e82a2a51341afd30cae191a] Merge branch 'next' of git://github.com/awilliam/linux-vfio.git
> > > git bisect bad f95826f64a15aefb5e82a2a51341afd30cae191a
> > > # good: [b5276c924497705ca927ad85a763c37f2de98349] drivers: lkdtm: fix clang -Wformat warning
> > > git bisect good b5276c924497705ca927ad85a763c37f2de98349
> > > # bad: [8d0a69bdbdaaf73040a80479497f153f43120787] Merge branch 'driver-core-next' of git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git
> > > git bisect bad 8d0a69bdbdaaf73040a80479497f153f43120787
> > > # good: [62776e969e13c7124e614937c0333dc24c0db0db] Merge branch 'for-next' of git://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git
> > > git bisect good 62776e969e13c7124e614937c0333dc24c0db0db
> > > # good: [3413a46f9d313b6870e1c2fd5bae2892b725ef2e] Merge branch 'for-next' of git://git.kernel.org/pub/scm/linux/kernel/git/pavel/linux-leds.git
> > > git bisect good 3413a46f9d313b6870e1c2fd5bae2892b725ef2e
> > > # bad: [2fd26970cf66bd52dc42843c46968040caa8c9a1] Revert "kernfs: Change kernfs_notify_list to llist."
> > > git bisect bad 2fd26970cf66bd52dc42843c46968040caa8c9a1
> > > # bad: [5f8954e099b8ae96e7de1bb95950e00c85bedd40] Revert "mwifiex: fix sleep in atomic context bugs caused by dev_coredumpv"
> > > git bisect bad 5f8954e099b8ae96e7de1bb95950e00c85bedd40
> > > # bad: [71066545b48e4259f89481199a0bbc7c35457738] driver core: Set fw_devlink.strict=1 by default
> > > git bisect bad 71066545b48e4259f89481199a0bbc7c35457738
> > > # bad: [f8217275b57aa48d98cc42051c2aac34152718d6] net: mdio: Delete usage of driver_deferred_probe_check_state()
> > > git bisect bad f8217275b57aa48d98cc42051c2aac34152718d6
> > > # bad: [24a026f85241a01bbcfe1b263caeeaa9a79bab40] pinctrl: devicetree: Delete usage of driver_deferred_probe_check_state()
> > > git bisect bad 24a026f85241a01bbcfe1b263caeeaa9a79bab40
> > > # bad: [5a46079a96451cfb15e4f5f01f73f7ba24ef851a] PM: domains: Delete usage of driver_deferred_probe_check_state()
> > > git bisect bad 5a46079a96451cfb15e4f5f01f73f7ba24ef851a
> > > # first bad commit: [5a46079a96451cfb15e4f5f01f73f7ba24ef851a] PM: domains: Delete usage of driver_deferred_probe_check_state()
> > > -------------------------------------------------------------------------------
> > >
> > >
> > > -=-=-=-=-=-=-=-=-=-=-=-
> > > Groups.io Links: You receive all messages sent to this group.
> > > View/Reply Online (#30209): https://groups.io/g/kernelci-results/message/30209
> > > Mute This Topic: https://groups.io/mt/92825373/1131744
> > > Group Owner: kernelci-results+owner at groups.io
> > > Unsubscribe: https://groups.io/g/kernelci-results/unsub [broonie at kernel.org]
> > > -=-=-=-=-=-=-=-=-=-=-=-
> > >
> > >
>
>
More information about the linux-arm-kernel
mailing list