U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: Pull request for UEFI sub-system for efi-2020-10-rc1 (2)
Date: Sat, 11 Jul 2020 08:16:22 -0400	[thread overview]
Message-ID: <20200711121622.GN6246@bill-the-cat> (raw)
In-Reply-To: <89c4e523-88df-35fd-d446-3bcc2c6a1c63@gmx.de>

On Sat, Jul 11, 2020 at 09:00:16AM +0200, Heinrich Schuchardt wrote:
> On 7/10/20 8:09 PM, Tom Rini wrote:
> > On Thu, Jul 09, 2020 at 06:12:02PM +0200, Heinrich Schuchardt wrote:
> >
> >> The following changes since commit 61608f395e7dcb2be6060407a72a1149b046430a:
> >>
> >>   Merge branch '2020-07-08-misc-features-and-fixes' (2020-07-08 20:20:24
> >> -0400)
> >>
> >> are available in the Git repository at:
> >>
> >>   https://gitlab.denx.de/u-boot/custodians/u-boot-efi.git efi-2020-10-rc1-2
> >>
> >> for you to fetch changes up to f4cef8e7585c268f05a8c39e368ca115c25e40d5:
> >>
> >>   efi_selftest: adjust runtime test for variables (2020-07-09 12:08:41
> >> +0200)
> >>
> >
> > NAK.  This is reliably failing here:
> > https://gitlab.denx.de/u-boot/u-boot/-/jobs/122018
> >
> > I see it passed Azure, and hasn't run through Travis yet.  Maybe it
> > needs to be run repeatedly to fail and we just got "lucky" ?
> >
> 
> Hello Tom,
> 
> you saw unreproducible results with multiple runs failing and one run
> succeeding. The reason is that when signing with sign-efi-sig-list in
> out Python tests without passing a timestamp two signatures may be in
> the same second or not.
> 
> When using the signed files to set UEFI variables a variable can only be
> overwritten by a file with a newer timestamp. But without setting
> timestamps explicitly using parameter -t passed to sign-efi-sig-list we
> have no control.
> 
> I already fixed this for some elder tests but missed to fix this for the
> merged patches from Takahiro.

Ah, thanks for the explanation.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200711/d11c47dc/attachment.sig>

  reply	other threads:[~2020-07-11 12:16 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-09 16:12 Pull request for UEFI sub-system for efi-2020-10-rc1 (2) Heinrich Schuchardt
2020-07-10 18:09 ` Tom Rini
2020-07-11  7:00   ` Heinrich Schuchardt
2020-07-11 12:16     ` Tom Rini [this message]
2020-07-11 22:05       ` Heinrich Schuchardt
2020-07-12  4:32         ` Heinrich Schuchardt
2020-07-12 23:19         ` AKASHI Takahiro

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200711121622.GN6246@bill-the-cat \
    --to=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox