linux-kernel - Re: [PATCH v1] dd: Invoke one probe retry cycle after some initcall levels

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAJZ5v0hRw-bkOZPLtKohW6aPYnFFsVGNQbYo9yvZPcUFzioOGQ@mail.gmail.com>
Date:   Sun, 12 Aug 2018 10:26:02 +0200
From:   "Rafael J. Wysocki" <rafael@...nel.org>
To:     Rishabh Bhatnagar <rishabhb@...eaurora.org>
Cc:     "Rafael J. Wysocki" <rafael@...nel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Sodagudi Prasad <psodagud@...eaurora.org>,
        tsoni@...eaurora.org, ckadabi@...eaurora.org,
        Vikram Mulukutla <markivx@...eaurora.org>
Subject: Re: [PATCH v1] dd: Invoke one probe retry cycle after some initcall levels

On Fri, Aug 10, 2018 at 11:52 PM Rishabh Bhatnagar
<rishabhb@...eaurora.org> wrote:
>
> Drivers that are registered at an initcall level may have to
> wait until late_init before the probe deferral mechanism can
> retry their probe functions. It is possible that their
> dependencies were resolved much earlier, in some cases even
> before the next initcall level. Invoke one probe retry cycle
> at every _sync initcall level after subsys initcall, allowing
> these drivers to be probed earlier.
>
> Signed-off-by: Vikram Mulukutla <markivx@...eaurora.org>
> Signed-off-by: Rishabh Bhatnagar <rishabhb@...eaurora.org>
> ---
>
> To give an example many Qualcomm drivers are dependent on the regulator and
> bus driver. Both the regulator and bus driver are probed in the
> subsys_initcall level. Now the probe of bus driver requires regulator to be
> working. If the probe of bus driver happens before regulator, then bus
> driver's probe will be deferred and all other device's probes which depend
> on bus driver will also be deferred.
> The impact of this problem is reduced if we have this patch.

Please move the above to the changelog proper (ie. above the tags) and
resend the patch with CCs to linux-pm@...r.kernel.org and LKML.

Thanks,
Rafael