Re: Re: [PATCH v2 3/3] efi: Capsule update with user helper interface

From: Matt Fleming <matt@console-pimps.org>
To: Peter Jones <pjones@redhat.com>
Cc: "Kweh, Hock Leong" <hock.leong.kweh@intel.com>,
	Andy Lutomirski <luto@amacapital.net>,
	Sam Protsenko <semen.protsenko@linaro.org>,
	Ming Lei <ming.lei@canonical.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	"Ong, Boon Leong" <boon.leong.ong@intel.com>,
	LKML <linux-kernel@vger.kernel.org>,
	"linux-efi@vger.kernel.org" <linux-efi@vger.kernel.org>
Subject: Re: Re: [PATCH v2 3/3] efi: Capsule update with user helper interface
Date: Tue, 10 Mar 2015 12:26:52 +0000	[thread overview]
Message-ID: <20150310122652.GA17573@codeblueprint.co.uk> (raw)
In-Reply-To: <20150306213912.GA8020@fenchurch.internal.datastacks.com>

On Fri, 06 Mar, at 04:39:12PM, Peter Jones wrote:
> 
> So again: do we really need or want to do this?

One thing that we totally lose the ability to do is use the capsule
interface for things *other* than firmware updates, e.g.

 https://lkml.org/lkml/2013/10/16/327 

Also, requiring embedded or custom OS to include fwupdate into their
existing boot solutions is a bit heavy handed when literally all they
want to do is cat a binary blob to a sysfs file.

I don't see why we can't have both solutions.

Another thing is that ESRT isn't going to be supported by every
platform.

So, for the sysfs interface, let's not allow loading from /lib. Let's
not require a userland tool. Let's just do,

  # echo /path/to/my/awesome/capsule.bin > /sys/../capsule

and be done with it.

Hmmm?

-- 
Matt Fleming, Intel Open Source Technology Center