All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eddie Kovsky <ekovsky@redhat.com>
To: Tom Rini <trini@konsulko.com>
Cc: Eddie Kovsky <ekovsky@redhat.com>,
	Mattijs Korpershoek <mkorpershoek@kernel.org>,
	Tobias Olausson <tobias@eub.se>,
	Paul HENRYS <paul.henrys_ext@softathome.com>,
	Simon Glass <sjg@chromium.org>, Jan Stancek <jstancek@redhat.com>,
	Enric Balletbo i Serra <eballetb@redhat.com>,
	a.fatoum@pengutronix.de, mark.kettenis@xs4all.nl,
	u-boot@lists.denx.de
Subject: Re: [PATCH v3] Add support for OpenSSL Provider API
Date: Fri, 27 Feb 2026 10:36:53 -0700	[thread overview]
Message-ID: <aaHWNU0LQqJ0zNDD@daedalus> (raw)
In-Reply-To: <20260219172836.GN3233182@bill-the-cat>

On 02/19/26, Tom Rini wrote:
> On Thu, Feb 19, 2026 at 09:51:05AM -0700, Eddie Kovsky wrote:
> 
> > On 01/29/26, Mattijs Korpershoek wrote:
> > > Hi Eddie,
> > > 
> > > Thank you for the patch.
> > > 
> > 
> > Hi Mattijs
> > 
> > Thanks for the review.
> > 
> > > On Tue, Jan 20, 2026 at 09:45, Eddie Kovsky <ekovsky@redhat.com> wrote:
> > > 
> > > > The Engine API has been deprecated since the release of OpenSSL 3.0. End
> > > > users have been advised to migrate to the new Provider interface.
> > > > Several distributions have already removed support for engines, which is
> > > > preventing U-Boot from being compiled in those environments.
> > > >
> > > > Add support for the Provider API while continuing to support the existing
> > > > Engine API on distros shipping older releases of OpenSSL.
> > > >
> > > > This is based on similar work contributed by Jan Stancek updating Linux
> > > > to use the Provider interface.
> > > >
> > > >     commit 558bdc45dfb2669e1741384a0c80be9c82fa052c
> > > >     Author: Jan Stancek <jstancek@redhat.com>
> > > >     Date:   Fri Sep 20 19:52:48 2024 +0300
> > > >
> > > >         sign-file,extract-cert: use pkcs11 provider for OPENSSL MAJOR >= 3
> > > >
> > > > The changes have been tested with the FIT signature verification vboot
> > > > tests on Fedora 42 and Debian 13. All 30 tests pass with both the legacy
> > > > Engine library installed and with the Provider API.
> > > >
> > > > Signed-off-by: Eddie Kovsky <ekovsky@redhat.com>
> [snip]
> > Sure, I can update the comment for v4.

Hi Tom

> 
> Since we're talking about v4, can you please make sure that for v4 it:
> - Passes CI https://docs.u-boot.org/en/latest/develop/ci_testing.html as
>   that will cover some non-Linux host builds.

I don't have resources available to set up a Gitlab runner. Based on the
documentation you provided it seems like this wouldn't be effective for
me as a non-custodian.

I did use GitHub to trigger an Azure pipeline. There was one failure and
several errors in the binman Command Line test.

    https://github.com/u-boot/u-boot/pull/875/checks?check_run_id=65015204887

These are PKCS11 errors, so of course I thought my patch was to blame.
But I'm seeing the same errors on Debian 13 running 'binman test'
manually on the master branch.

> - See if you can get access to a FreeBSD or OpenBSD host and make sure
>   the tools build still works there too? I was hoping Mark would have
>   commented / tested-by v3 because I do want to make sure the libressl
>   case still builds. At worst case, I have a freebie Oracle VM that's
>   FreeBSD based, so you can maybe spin one of those up as well?
> 

I spent some time again setting up OpenBSD and FreeBSD virtual machines, but I was
unable to reproduce the build environment for U-Boot. But thanks to
Enric and Mark's work it looks like we have the LibreSSL use case
covered now.


Eddie


  parent reply	other threads:[~2026-02-27 18:13 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-20 16:45 [PATCH v3] Add support for OpenSSL Provider API Eddie Kovsky
2026-01-29 20:08 ` Mattijs Korpershoek
2026-02-19 16:51   ` Eddie Kovsky
2026-02-19 17:28     ` Tom Rini
2026-02-24 12:08       ` Enric Balletbo i Serra
2026-02-24 15:48         ` Tom Rini
2026-02-24 22:23         ` Mark Kettenis
2026-02-27 17:36       ` Eddie Kovsky [this message]
2026-02-27 17:47         ` Tom Rini
2026-04-01 22:05           ` Eddie Kovsky
2026-04-02 16:27             ` Tom Rini
2026-04-11  1:02               ` Eddie Kovsky
2026-04-13 18:12                 ` Tom Rini
2026-04-27 20:43                   ` Eddie Kovsky
2026-02-25 16:16     ` Mattijs Korpershoek

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=aaHWNU0LQqJ0zNDD@daedalus \
    --to=ekovsky@redhat.com \
    --cc=a.fatoum@pengutronix.de \
    --cc=eballetb@redhat.com \
    --cc=jstancek@redhat.com \
    --cc=mark.kettenis@xs4all.nl \
    --cc=mkorpershoek@kernel.org \
    --cc=paul.henrys_ext@softathome.com \
    --cc=sjg@chromium.org \
    --cc=tobias@eub.se \
    --cc=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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.