On Thu, Jun 30, 2022 at 11:44 AM Mark Brown <broonie@xxxxxxxxxx> wrote: > > On Thu, Jun 30, 2022 at 11:32:06AM -0700, KernelCI bot wrote: > > The KernelCI bisection bot identified a failure to probe the PCI bus on > kontron-pitx-imx8m in -next resulting from commit (5a46079a96451 PM: > domains: Delete usage of driver_deferred_probe_check_state()) with at > least an arm64 defconfig+64K_PAGES. > > The only logging I see from PCI in the failing boot is: > > <6>[ 0.580973] PCI: CLS 0 bytes, default 64 > > there's none of the host bridge enumeration starting with > > <6>[ 2.394399] imx6q-pcie 33800000.pcie: host bridge /soc@0/pcie@33800000 ranges: > <6>[ 2.396012] imx6q-pcie 33c00000.pcie: host bridge /soc@0/pcie@33c00000 ranges: > > that is seen with working boots. > > I've left the full bot report below, it's got a Reported-by tag, links > to more details including full boot logs and more. The bot checked that > reverting the patch seems to fix the problem. Thanks for the report. I'll look into this alongside the issue Tony reported. -Saravana > > * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * > > * 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@xxxxxxxxxxxx> * > > * * > > * Hope this helps! * > > * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * > > > > next/master bisection: baseline.bootrr.imx6q-pcie-pcie0-probed on kontron-pitx-imx8m > > > > Summary: > > Start: 6cc11d2a17592 Add linux-next specific files for 20220630 > > Plain log: https://storage.kernelci.org/next/master/next-20220630/arm64/defconfig+CONFIG_ARM64_64K_PAGES=y/gcc-10/lab-kontron/baseline-kontron-pitx-imx8m.txt > > HTML log: https://storage.kernelci.org/next/master/next-20220630/arm64/defconfig+CONFIG_ARM64_64K_PAGES=y/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+CONFIG_ARM64_64K_PAGES=y > > Test case: baseline.bootrr.imx6q-pcie-pcie0-probed > > > > Breaking commit found: > > > > ------------------------------------------------------------------------------- > > commit 5a46079a96451cfb15e4f5f01f73f7ba24ef851a > > Author: Saravana Kannan <saravanak@xxxxxxxxxx> > > 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@xxxxxxxxx> > > Reviewed-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx> > > Signed-off-by: Saravana Kannan <saravanak@xxxxxxxxxx> > > Link: https://lore.kernel.org/r/20220601070707.3946847-2-saravanak@xxxxxxxxxx > > Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > > > > 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: [d9b2ba67917c18822c6a09af41c32fa161f1606b] Merge tag 'platform-drivers-x86-v5.19-3' of git://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86 > > git bisect good d9b2ba67917c18822c6a09af41c32fa161f1606b > > # bad: [6cc11d2a1759275b856e464265823d94aabd5eaf] Add linux-next specific files for 20220630 > > git bisect bad 6cc11d2a1759275b856e464265823d94aabd5eaf > > # good: [7391068f14aafb8c5bb9d5aeb07ecfa55c89be42] Merge branch 'drm-next' of https://gitlab.freedesktop.org/agd5f/linux > > git bisect good 7391068f14aafb8c5bb9d5aeb07ecfa55c89be42 > > # good: [17daf6a2ab5178cf52a20d1c85470ea4638d4310] Merge branch 'next' of git://git.kernel.org/pub/scm/virt/kvm/kvm.git > > git bisect good 17daf6a2ab5178cf52a20d1c85470ea4638d4310 > > # bad: [a143ea0f3ce59385089e6e7b71b04fd0b5621bd8] Merge branch 'for-next' of git://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup.git > > git bisect bad a143ea0f3ce59385089e6e7b71b04fd0b5621bd8 > > # bad: [57b6609eb7251280cf9f34fdebf1244f10673749] Merge branch 'icc-next' of git://git.kernel.org/pub/scm/linux/kernel/git/djakov/icc.git > > git bisect bad 57b6609eb7251280cf9f34fdebf1244f10673749 > > # bad: [eb3fd63a935b759df99bfe4a6b13c820204f81d3] Merge branch 'usb-next' of git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git > > git bisect bad eb3fd63a935b759df99bfe4a6b13c820204f81d3 > > # good: [048914d1bed271f04f726b7f78d0bef8cd1809f5] Merge branch 'for-next' of git://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-hsi.git > > git bisect good 048914d1bed271f04f726b7f78d0bef8cd1809f5 > > # good: [40a959d7042bb7711e404ad2318b30e9f92c6b9b] usb: host: ohci-ppc-of: Fix refcount leak bug > > git bisect good 40a959d7042bb7711e404ad2318b30e9f92c6b9b > > # good: [849f35422319a46c2a52289e2d5c85eb3346a921] Merge tag 'thunderbolt-for-v5.20-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/westeri/thunderbolt into usb-next > > git bisect good 849f35422319a46c2a52289e2d5c85eb3346a921 > > # bad: [a52ed4866d2b90dd5e4ae9dabd453f3ed8fa3cbc] mwifiex: fix sleep in atomic context bugs caused by dev_coredumpv > > git bisect bad a52ed4866d2b90dd5e4ae9dabd453f3ed8fa3cbc > > # bad: [f516d01b9df2782b9399c44fa1d21c3d09211f8a] Revert "driver core: Set default deferred_probe_timeout back to 0." > > git bisect bad f516d01b9df2782b9399c44fa1d21c3d09211f8a > > # 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 (#28727): https://groups.io/g/kernelci-results/message/28727 > > Mute This Topic: https://groups.io/mt/92093224/1131744 > > Group Owner: kernelci-results+owner@xxxxxxxxx > > Unsubscribe: https://groups.io/g/kernelci-results/unsub [broonie@xxxxxxxxxx] > > -=-=-=-=-=-=-=-=-=-=-=- > > > >
