Re: [edk2-devel] [PATCH v3 0/7] update ArmSoftFloatLib to latest upstream version

["Michael D Kinney" <michael.d.kinney@intel.com>]

Hi Leif,

As Liming outlined last night, we were waiting for this
fix to start the hard freeze period.  Please keep us
informed of your progress on your CI issue and it can
be fixed during the hard freeze.

Thanks,

Mike

> -----Original Message-----
> From: devel@edk2.groups.io
> [mailto:devel@edk2.groups.io] On Behalf Of Leif
> Lindholm
> Sent: Friday, May 31, 2019 12:29 PM
> To: Kinney, Michael D <michael.d.kinney@intel.com>
> Cc: devel@edk2.groups.io; ard.biesheuvel@linaro.org;
> Laszlo Ersek <lersek@redhat.com>; Gao, Liming
> <liming.gao@intel.com>; Wang, Jian J
> <jian.j.wang@intel.com>
> Subject: Re: [edk2-devel] [PATCH v3 0/7] update
> ArmSoftFloatLib to latest upstream version
> 
> On Fri, May 31, 2019 at 04:21:19PM +0000, Kinney,
> Michael D wrote:
> > Ard,
> >
> > Thank you for all the hard work on this issue.
> 
> I concur.
> *However*, please hold back on tagging the stable
> release. Something
> that looks related has broken our CI.
> 
> Regards,
> 
> Leif
> 
> > I will do one additional patch for the top level
> > Readme.md to list this additional submodule.
> >
> > Thanks,
> >
> > Mike
> >
> > > -----Original Message-----
> > > From: devel@edk2.groups.io
> > > [mailto:devel@edk2.groups.io] On Behalf Of Ard
> > > Biesheuvel
> > > Sent: Friday, May 31, 2019 9:00 AM
> > > To: edk2-devel-groups-io <devel@edk2.groups.io>;
> Leif
> > > Lindholm <leif.lindholm@linaro.org>
> > > Cc: Laszlo Ersek <lersek@redhat.com>; Gao, Liming
> > > <liming.gao@intel.com>; Wang, Jian J
> > > <jian.j.wang@intel.com>; Kinney, Michael D
> > > <michael.d.kinney@intel.com>
> > > Subject: Re: [edk2-devel] [PATCH v3 0/7] update
> > > ArmSoftFloatLib to latest upstream version
> > >
> > > On Fri, 31 May 2019 at 15:09, Leif Lindholm
> > > <leif.lindholm@linaro.org> wrote:
> > > >
> > > > On Fri, May 31, 2019 at 02:49:51PM +0200, Ard
> > > Biesheuvel wrote:
> > > > > Currently, our move to OpenSSL 1.1.1b is being
> > > blocked by an issue in
> > > > > the ARM software floating point library, which
> > > lacks some intrinsics
> > > > > that the ARM EABI spec defines.
> > > > >
> > > > > Since the code was in pretty sorry state, let's
> fix
> > > this by upgrading
> > > > > to the very latest version of the core library
> this
> > > code is based on,
> > > > > dated January 2018 (whereas the NetBSD fork of
> the
> > > old code dates back
> > > > > to 2002)
> > > >
> > > > It is worth mentioning that there is exactly one
> > > patch on top of 3e
> > > > release, and that resolves a syntax error in RICV
> > > code (currently
> > > > unused).
> > > >
> > > > I have a couple of really tiny comments on later
> > > patches, but many
> > > > thanks for unbundling this conundrum (and
> preventing
> > > yet anoher fork).
> > > >
> > >
> > > OK I have merged the following patches of this
> series
> > >
> > > 85af0b8116f3 ArmPkg: import Berkeley Softfloat
> library
> > > as git submodule
> > > 3cc57695df5a ArmPkg/ArmSoftFloatLib: switch to new
> > > version of softfloat library
> > > 99243102b467 ArmPkg/ArmSoftFloatLib: remove source
> > > files that are no longer used
> > > 8594c2073cdb ArmPkg/ArmLib ARM: set .fpu to let
> Clang 7
> > > assemble ArmV7Support.S
> > >
> > > which are the only ones that turn out to be
> required to
> > > build
> > > OpensslLib for ARM when using Jian latest v5 of the
> > > OpenSSL 1.1.1b
> > > upgrade
> > >
> > > Thanks all,
> > > Ard.
> > >
> > >
> >
> 
>