Re: [PATCH v1] driver core: Update device link status correctly for SYNC_STATE_ONLY links

From: Saravana Kannan <saravanak@google.com>
To: Michael Walle <michael@walle.cc>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	"Rafael J. Wysocki" <rafael@kernel.org>,
	Android Kernel Team <kernel-team@android.com>,
	"Rafael J. Wysocki" <rafael.j.wysocki@intel.com>,
	LKML <linux-kernel@vger.kernel.org>,
	stable <stable@vger.kernel.org>
Subject: Re: [PATCH v1] driver core: Update device link status correctly for SYNC_STATE_ONLY links
Date: Tue, 26 May 2020 11:08:12 -0700	[thread overview]
Message-ID: <CAGETcx_iHs5xsXq81k4YRzrFxs2j01u3HzfnSNU9Vxs8ZTGGuA@mail.gmail.com> (raw)
In-Reply-To: <dac3da8b373f131e53e18083f6fac5b0@walle.cc>

On Tue, May 26, 2020 at 4:04 AM Michael Walle <michael@walle.cc> wrote:
>
> Hi Saravana,
>
> Am 2020-05-26 09:07, schrieb Saravana Kannan:
> >> Greg,
> >>
> >> I think this is the issue Michael ran into. I'd like him to test the
> >> fix
> >> before it's pulled in.
> >>
> >> Michael,
> >>
> >> If you can test this on the branch you saw the issue in and give a
> >> Tested-by if it works, that'd be great.
>
> Unfortunately, now I'm triggering another WARN_ON() in
> device_links_driver_bound():
>    WARN_ON(link->status != DL_STATE_DORMANT);
>
> I've attached two bootlog one based on linux-next-20200525 with this
> patch applied and another one where your previous debug printfs are
> applied, too.
>
> Let me know, if you need any other debug outputs.

Can you make the same debug changes I did for the other WARN and get
me the logs?

Thanks,
Saravana