[PATCH v6 00/17] efi_loader: add capsule update support

[AKASHI Takahiro]

On Wed, Sep 09, 2020 at 10:58:53PM -0400, Tom Rini wrote:
> On Thu, Sep 10, 2020 at 11:52:37AM +0900, AKASHI Takahiro wrote:
> > On Wed, Sep 09, 2020 at 12:56:28PM -0400, Tom Rini wrote:
> > > On Wed, Sep 09, 2020 at 04:48:30PM +0200, Heinrich Schuchardt wrote:
> > > > On 07.09.20 07:34, AKASHI Takahiro wrote:
> > > [snip]
> > > > > required and partly because there is a problem with virt-make-fs.
> > > > 
> > > > What problem with virt-make-fs exists? How will this be solved?
> > > 
> > > This I suspect is related to the difficulty of getting tests to run in
> > > all of our CI environments, where at least in one case they run and
> > > pass, rather than skip.  This will require that the series get run
> > > through at least Travis and Azure before re-posting, at this point.
> > > Thanks.
> > 
> > I think that we discussed this issue several times before.
> > 
> > https://lists.denx.de/pipermail/u-boot/2020-July/419430.html
> > https://lists.denx.de/pipermail/u-boot/2020-July/420712.html
> > 
> > I have not implemented a fallback handling in case of virt-make-fs failure
> > as Heinrich doesn't like it (even dropped the code from my patches.)
> > 
> > The only solution is Heinrich's patch:
> > https://lists.denx.de/pipermail/u-boot/2020-July/419976.html
> 
> The problem is that Heinrich's patch breaks things.  A complete solution
> is required.

What kind of issue did you see?

> That patch might be part of the solution, along with
> perhaps a rework of the logic to try and gracefully catch and re-try
> when we have a problem with one of the possible tools.

Anyway, I can't do anything here as Heinrich rejects any sudo-based
solution. See:

https://lists.denx.de/pipermail/u-boot/2020-July/419951.html

So he is responsible for fixing it.

-Takahiro Akashi


> -- 
> Tom