Re: [PATCH v5 2/5] driver core: Functional dependencies tracking support

Path	csiph.com!1.us.feeder.erje.net!feeder.erje.net!1.eu.feeder.erje.net!newsfeed.fsmpi.rwth-aachen.de!newsfeed.straub-nv.de!news.mixmin.net!aioe.org!bofh.it!news.nic.it!robomod
From	"Luis R. Rodriguez" <mcgrof@kernel.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH v5 2/5] driver core: Functional dependencies tracking support
Date	Mon, 07 Nov 2016 22:30:02 +0100
Message-ID	<sAZEm-eQ-25@gated-at.bofh.it> (permalink)
References	<sff3r-2Cw-3@gated-at.bofh.it> <sqIlr-1ni-3@gated-at.bofh.it> <sqIv8-1FC-29@gated-at.bofh.it> <swupr-2Sj-11@gated-at.bofh.it> <swUWB-3Ik-19@gated-at.bofh.it>
MIME-Version	1.0
Content-Type	text/plain; charset=us-ascii
Content-Disposition	inline
User-Agent	Mutt/1.6.0 (2016-04-01)
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	28
Organization	linux.* mail to news gateway
X-Original-Cc	Lukas Wunner <lukas@wunner.de>, "Rafael J. Wysocki" <rjw@rjwysocki.net>, Linux PM list <linux-pm@vger.kernel.org>, Alan Stern <stern@rowland.harvard.edu>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Tomeu Vizoso <tomeu.vizoso@collabora.com>, Mark Brown <broonie@kernel.org>, Marek Szyprowski <m.szyprowski@samsung.com>, Kevin Hilman <khilman@kernel.org>, Ulf Hansson <ulf.hansson@linaro.org>
X-Original-Date	Mon, 7 Nov 2016 22:22:50 +0100
X-Original-Message-ID	<20161107212250.GH1764@wotan.suse.de>
X-Original-References	<27296716.H9VWo8ShOm@vostro.rjw.lan> <13957403.ZrB4mMbICz@vostro.rjw.lan> <2715729.9U1nlcpFb3@vostro.rjw.lan> <20161026111902.GA6447@wunner.de> <20161027152551.GA15718@kroah.com>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1516599

Show key headers only | View raw

On Thu, Oct 27, 2016 at 05:25:51PM +0200, Greg Kroah-Hartman wrote:
> On Wed, Oct 26, 2016 at 01:19:02PM +0200, Lukas Wunner wrote:
> > Hi Rafael,
> > 
> > sorry for not responding to v5 of your series earlier, just sending
> > this out now in the hope that it reaches you before your travels.
> > 
> > On Mon, Oct 10, 2016 at 02:51:04PM +0200, Rafael J. Wysocki wrote:
> > > - Modify device_links_check_suppliers(), device_links_driver_bound(),
> > >   device_links_no_driver(), device_links_driver_cleanup(), device_links_busy(),
> > >   and device_links_unbind_consumers() to walk link lists under device_links_lock
> > >   (to make the new "driver presence tracking" mechanism work reliably).
> > 
> > This change might increase boot time if drivers return -EPROBE_DEFER.
> 
> "might"?  Please verify this before guessing....
> 
> And don't make this more complex than needed before actually determining
> a real issue.

As clarified by Rafael at Plumbers, this functional dependencies
framework assumes your driver / subsystem supports deferred probe,
if it does not support its not clear what will happen....

We have no explicit semantics to check if a driver / subsystem
supports deferred probe.

  Luis

Thread

Re: [PATCH v5 2/5] driver core: Functional dependencies tracking  support Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-10-27 17:40 +0200
  Re: [PATCH v5 2/5] driver core: Functional dependencies tracking  support Lukas Wunner <lukas@wunner.de> - 2016-10-28 12:00 +0200
  Re: [PATCH v5 2/5] driver core: Functional dependencies tracking  support "Luis R. Rodriguez" <mcgrof@kernel.org> - 2016-11-07 22:30 +0100
    Re: [PATCH v5 2/5] driver core: Functional dependencies tracking  support Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-11-08 07:50 +0100
      Re: [PATCH v5 2/5] driver core: Functional dependencies tracking  support "Luis R. Rodriguez" <mcgrof@kernel.org> - 2016-11-08 20:30 +0100
        Re: [PATCH v5 2/5] driver core: Functional dependencies tracking  support Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-11-08 20:50 +0100
          Re: [PATCH v5 2/5] driver core: Functional dependencies tracking  support "Luis R. Rodriguez" <mcgrof@kernel.org> - 2016-11-08 22:00 +0100
            Re: [PATCH v5 2/5] driver core: Functional dependencies tracking  support Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-11-09 07:50 +0100
              Re: [PATCH v5 2/5] driver core: Functional dependencies tracking  support Andrzej Hajda <a.hajda@samsung.com> - 2016-11-09 10:40 +0100
                Re: [PATCH v5 2/5] driver core: Functional dependencies tracking  support Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-11-09 10:50 +0100
    Re: [PATCH v5 2/5] driver core: Functional dependencies tracking support "Rafael J. Wysocki" <rafael@kernel.org> - 2016-11-10 01:50 +0100
      Re: [PATCH v5 2/5] driver core: Functional dependencies tracking support "Luis R. Rodriguez" <mcgrof@kernel.org> - 2016-11-10 02:10 +0100
        Re: [PATCH v5 2/5] driver core: Functional dependencies tracking support Laurent Pinchart <laurent.pinchart@ideasonboard.com> - 2016-11-10 08:20 +0100
          Re: [PATCH v5 2/5] driver core: Functional dependencies tracking  support "Luis R. Rodriguez" <mcgrof@kernel.org> - 2016-11-10 23:10 +0100
            Re: [PATCH v5 2/5] driver core: Functional dependencies tracking  support Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-11-10 23:50 +0100
              Re: [PATCH v5 2/5] driver core: Functional dependencies tracking support Laurent Pinchart <laurent.pinchart@ideasonboard.com> - 2016-11-11 01:10 +0100
        Re: [PATCH v5 2/5] driver core: Functional dependencies tracking support Geert Uytterhoeven <geert@linux-m68k.org> - 2016-11-10 09:50 +0100
          Re: [PATCH v5 2/5] driver core: Functional dependencies tracking  support "Luis R. Rodriguez" <mcgrof@kernel.org> - 2016-11-10 23:20 +0100

csiph-web