Date: Tue, 27 Jun 2017 20:03:41 +0200
From: "Luis R. Rodriguez" <mcgrof@kernel.org>
To: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Jakub Kicinski <jakub.kicinski@netronome.com>,
        "Luis R. Rodriguez" <mcgrof@kernel.org>,
        Ming Lei <ming.lei@redhat.com>, yi1.li@linux.intel.com,
        "AKASHI, Takahiro" <takahiro.akashi@linaro.org>, nbroeking@me.com,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>, mfuzzey@parkeon.com,
        ebiederm@xmission.com, Dmitry Torokhov <dmitry.torokhov@gmail.com>,
        Daniel Wagner <wagi@monom.org>, David Woodhouse <dwmw2@infradead.org>,
        jewalt@lgsinnovations.com, rafal@milecki.pl,
        Arend van Spriel <arend.vanspriel@broadcom.com>,
        "Rafael J. Wysocki" <rjw@rjwysocki.net>, atull@kernel.org,
        Moritz Fischer <moritz.fischer@ettus.com>, pmladek@suse.com,
        Johannes Berg <johannes.berg@intel.com>, emmanuel.grumbach@intel.com,
        Luca Coelho <luciano.coelho@intel.com>, luto@kernel.org,
        Linus Torvalds <torvalds@linux-foundation.org>,
        Kees Cook <keescook@chromium.org>, David Howells <dhowells@redhat.com>,
        pjones@redhat.com, Hans de Goede <hdegoede@redhat.com>,
        alan@linux.intel.com, "Theodore Ts'o" <tytso@mit.edu>,
        paul.gortmaker@windriver.com, mtosatti@redhat.com,
        mawilcox@microsoft.com, Stephen Boyd <stephen.boyd@linaro.org>,
        Vikram Mulukutla <markivx@codeaurora.org>,
        lkml <linux-kernel@vger.kernel.org>, oss-drivers@netronome.com
Subject: Re: [PATCH] firmware: wake all waiters
Message-ID: <20170627180341.GT21846@wotan.suse.de>
References: <20170623233702.20564-1-jakub.kicinski@netronome.com>
 <20170626212036.GE21846@wotan.suse.de>
 <20170626191009.0c11eed0@cakuba.netronome.com>
 <CAOCOHw4GtC-7OM0g4wLAeOqYsJgQd-mTnBjqLMP9n_x06a8LqQ@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAOCOHw4GtC-7OM0g4wLAeOqYsJgQd-mTnBjqLMP9n_x06a8LqQ@mail.gmail.com>
User-Agent: Mutt/1.6.0 (2016-04-01)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1756
Lines: 37

On Tue, Jun 27, 2017 at 10:48:25AM -0700, Bjorn Andersson wrote:
> On Mon, Jun 26, 2017 at 7:10 PM, Jakub Kicinski
> <jakub.kicinski@netronome.com> wrote:
> > On Mon, 26 Jun 2017 23:20:36 +0200, Luis R. Rodriguez wrote:
> [..]
> >  - how to stay bound and retry the direct default FW load until rootfs
> >    is mounted (equivalent to when -EPROBE_DEFER would give up);
> 
> If you constrain this problem to only await the mounting of a root
> file system you miss the various cases where rootfs is later pivoted

I do believe the firmwared case can work with pivot root, I can't see
why not. In fact it was a case considered from what I can tell

> or the firmware isn't stored in the root file system (e.g. every
> Android device out there).

This was also considered as part of the design. I had particular mentioned
to Tom and Daniel the case of an NVRAM sitting somewhere custom and this
needed to be tossed laster somehow thorugh some custom mechanism.

Let's consider a crazy case where the uevent gets triggered, and userspace goes
and signals Elon Musk somehow to transmit the needed firmware from Mars through
a serial satellite link to earth, and somehow someday the device is finally
ready to upload firmware from userspace. Once Elon's firmware lands home, we
know all needed firmware has arrived so anything missing we can acknowledge now
as missing, so we upload what we can and kick firmward into final-mode to tell
the kernel we know we're really ready and any pending things will have to be
given up.

This would prove the custom fallback crap was also never needed.

I think perhaps one enhancement consideration here may be having the option
for best-effort mode and final-mode be per file, but that's all daemon
specific code.

  Luis