[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAFd5g47Xj6pR=CqXMtYxL6iacqxoZ9PvHxsiG8dfUML24yN1gQ@mail.gmail.com>
Date: Wed, 20 May 2020 11:21:49 -0700
From: Brendan Higgins <brendanhiggins@...gle.com>
To: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
Cc: Petr Mladek <pmladek@...e.com>,
Greg KH <gregkh@...uxfoundation.org>,
"Rafael J. Wysocki" <rafael@...nel.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
David Gow <davidgow@...gle.com>,
Heidi Fahim <heidifahim@...gle.com>,
Heikki Krogerus <heikki.krogerus@...ux.intel.com>,
Hans de Goede <hdegoede@...hat.com>,
"Rafael J . Wysocki" <rafael.j.wysocki@...el.com>,
Sakari Ailus <sakari.ailus@...ux.intel.com>,
Rob Herring <robh@...nel.org>
Subject: Re: [PATCH v1] Revert "software node: Simplify software_node_release()
function"
On Wed, May 20, 2020 at 9:42 AM Andy Shevchenko
<andriy.shevchenko@...ux.intel.com> wrote:
>
> On Wed, May 20, 2020 at 05:02:27PM +0200, Petr Mladek wrote:
> > On Thu 2020-02-27 16:00:01, Brendan Higgins wrote:
>
> > I have found similar report from a test robot, see
> > https://lore.kernel.org/lkml/20200303002816.GW6548@shao2-debian/
> >
> >
> > I was staring into it for a while and do not understand it. The revert
> > makes sense. I wonder if it somehow changes the order in which
> > the release methods are called.
> >
> > Anyway, reverting the revert makes test_printf working.
>
> There is a proper fix IIRC from Heikki in driver core (no link at hand, sorry).
The fix for this patch can be found here: https://lkml.org/lkml/2020/5/13/1070
Cheers
Powered by blists - more mailing lists