Re: next/master bisection: baseline.bootrr.imx6q-pcie-pcie0-probed on kontron-pitx-imx8m

Re: next/master bisection: baseline.bootrr.imx6q-pcie-pcie0-probed on kontron-pitx-imx8m

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 8003 bytes --]

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.

> * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
> * 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@kernelci.org>          *
> *                                                               *
> * 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@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@glider.be>
>     Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>
>     Signed-off-by: Saravana Kannan <saravanak@google.com>
>     Link: https://lore.kernel.org/r/20220601070707.3946847-2-saravanak@google.com
>     Signed-off-by: Greg Kroah-Hartman <gregkh@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: [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@groups.io
> Unsubscribe: https://groups.io/g/kernelci-results/unsub [broonie@kernel.org]
> -=-=-=-=-=-=-=-=-=-=-=-
> 
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: next/master bisection: baseline.bootrr.imx6q-pcie-pcie0-probed on kontron-pitx-imx8m

[Saravana Kannan]

On Thu, Jun 30, 2022 at 11:44 AM Mark Brown <broonie@kernel.org> 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@kernelci.org>          *
> > *                                                               *
> > * 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@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@glider.be>
> >     Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>
> >     Signed-off-by: Saravana Kannan <saravanak@google.com>
> >     Link: https://lore.kernel.org/r/20220601070707.3946847-2-saravanak@google.com
> >     Signed-off-by: Greg Kroah-Hartman <gregkh@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: [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@groups.io
> > Unsubscribe: https://groups.io/g/kernelci-results/unsub [broonie@kernel.org]
> > -=-=-=-=-=-=-=-=-=-=-=-
> >
> >

Re: next/master bisection: baseline.bootrr.imx6q-pcie-pcie0-probed on kontron-pitx-imx8m

[Saravana Kannan]

On Thu, Jun 30, 2022 at 2:48 PM Saravana Kannan <saravanak@google.com> wrote:
>
> On Thu, Jun 30, 2022 at 11:44 AM Mark Brown <broonie@kernel.org> 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.

Hi Mark,

I think the root cause is the same as the issue I debugged here:
https://lore.kernel.org/lkml/CAGETcx_1qa=gGT4LVkyPpcA1vFM9FzuJE+0DhL_nFyg5cbFjVg@mail.gmail.com/

The patch attached to that email will probably fix this issue. I
haven't dealt with Kernel CI bot before. Is there a way to get it to
test a patch?

Thanks,
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@kernelci.org>          *
> > > *                                                               *
> > > * 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@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@glider.be>
> > >     Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>
> > >     Signed-off-by: Saravana Kannan <saravanak@google.com>
> > >     Link: https://lore.kernel.org/r/20220601070707.3946847-2-saravanak@google.com
> > >     Signed-off-by: Greg Kroah-Hartman <gregkh@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: [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@groups.io
> > > Unsubscribe: https://groups.io/g/kernelci-results/unsub [broonie@kernel.org]
> > > -=-=-=-=-=-=-=-=-=-=-=-
> > >
> > >

Re: next/master bisection: baseline.bootrr.imx6q-pcie-pcie0-probed on kontron-pitx-imx8m

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 361 bytes --]

On Thu, Jun 30, 2022 at 06:02:04PM -0700, Saravana Kannan wrote:

> The patch attached to that email will probably fix this issue. I
> haven't dealt with Kernel CI bot before. Is there a way to get it to
> test a patch?

It is but I don't have access to do that - either the lab owner
(Michael, CCed here) or Guillaume (also CCed) should be able to help
there.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: next/master bisection: baseline.bootrr.imx6q-pcie-pcie0-probed on kontron-pitx-imx8m

[Saravana Kannan]

On Fri, Jul 1, 2022 at 2:16 AM Mark Brown <broonie@kernel.org> wrote:
>
> On Thu, Jun 30, 2022 at 06:02:04PM -0700, Saravana Kannan wrote:
>
> > The patch attached to that email will probably fix this issue. I
> > haven't dealt with Kernel CI bot before. Is there a way to get it to
> > test a patch?
>
> It is but I don't have access to do that - either the lab owner
> (Michael, CCed here) or Guillaume (also CCed) should be able to help
> there.

I found a much simpler solution that I think should work. Snippet at
the end of this email.
https://lore.kernel.org/lkml/CAGETcx-fLAXnG+1S4MHJwg9t7O6jj6Mp+q25bh==C_Z1CLs-mg@mail.gmail.com/

I'm waiting for Alexander in that thread to give it a shot.

Michael/Guillaume, if you want to give it a shot too, that'll be nice :)

Thanks,
Saravana

Re: next/master bisection: baseline.bootrr.imx6q-pcie-pcie0-probed on kontron-pitx-imx8m

[Michael Walle]

Am 2. Juli 2022 00:28:58 MESZ schrieb Saravana Kannan <saravanak@google.com>:
>On Fri, Jul 1, 2022 at 2:16 AM Mark Brown <broonie@kernel.org> wrote:
>>
>> On Thu, Jun 30, 2022 at 06:02:04PM -0700, Saravana Kannan wrote:
>>
>> > The patch attached to that email will probably fix this issue. I
>> > haven't dealt with Kernel CI bot before. Is there a way to get it to
>> > test a patch?
>>
>> It is but I don't have access to do that - either the lab owner
>> (Michael, CCed here) or Guillaume (also CCed) should be able to help
>> there.
>
>I found a much simpler solution that I think should work. Snippet at
>the end of this email.
>https://lore.kernel.org/lkml/CAGETcx-fLAXnG+1S4MHJwg9t7O6jj6Mp+q25bh==C_Z1CLs-mg@mail.gmail.com/
>
>I'm waiting for Alexander in that thread to give it a shot.
>
>Michael/Guillaume, if you want to give it a shot too, that'll be nice :)
>
>Thanks,
>Saravana

Hi,

can do on Monday as I'm currently on the road without any real internet access nor laptop.

-michael

Re: next/master bisection: baseline.bootrr.imx6q-pcie-pcie0-probed on kontron-pitx-imx8m

[Michael Walle]

[+ Heiko]

Am 2022-07-02 00:28, schrieb Saravana Kannan:
> On Fri, Jul 1, 2022 at 2:16 AM Mark Brown <broonie@kernel.org> wrote:
>> 
>> On Thu, Jun 30, 2022 at 06:02:04PM -0700, Saravana Kannan wrote:
>> 
>> > The patch attached to that email will probably fix this issue. I
>> > haven't dealt with Kernel CI bot before. Is there a way to get it to
>> > test a patch?
>> 
>> It is but I don't have access to do that - either the lab owner
>> (Michael, CCed here) or Guillaume (also CCed) should be able to help
>> there.
> 
> I found a much simpler solution that I think should work. Snippet at
> the end of this email.
> https://lore.kernel.org/lkml/CAGETcx-fLAXnG+1S4MHJwg9t7O6jj6Mp+q25bh==C_Z1CLs-mg@mail.gmail.com/
> 
> I'm waiting for Alexander in that thread to give it a shot.
> 
> Michael/Guillaume, if you want to give it a shot too, that'll be nice 
> :)

Heiko will give it a try on his kontron-pitx-imx8m board.

-michael