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

Ard Biesheuvel posted 7 patches 4 years, 10 months ago
Failed in applying to current master (apply log)
.gitmodules                                            |    3 +
ArmPkg/Library/ArmLib/Arm/ArmV7Support.S               |    2 +
ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_cdcmp.asm   |   41 -
ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_cfcmp.asm   |   37 -
ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpeq.c    |   30 -
ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpge.c    |   28 -
ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpgt.c    |   30 -
ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmple.c    |   30 -
ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmplt.c    |   30 -
ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpun.c    |   35 -
ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpeq.c    |   30 -
ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpge.c    |   30 -
ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpgt.c    |   30 -
ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmple.c    |   30 -
ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmplt.c    |   30 -
ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpun.c    |   35 -
ArmPkg/Library/ArmSoftFloatLib/Arm/softfloat.h         |  345 ---
ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c       |  283 +++
ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.inf     |   83 +-
ArmPkg/Library/ArmSoftFloatLib/arm-gcc.h               |  108 -
ArmPkg/Library/ArmSoftFloatLib/berkeley-softfloat-3    |    1 +
ArmPkg/Library/ArmSoftFloatLib/bits32/softfloat-macros |  648 ------
ArmPkg/Library/ArmSoftFloatLib/bits32/softfloat.c      | 2354 --------------------
ArmPkg/Library/ArmSoftFloatLib/milieu.h                |   38 -
ArmPkg/Library/ArmSoftFloatLib/platform.h              |   11 +
ArmPkg/Library/ArmSoftFloatLib/softfloat-for-gcc.h     |  242 --
ArmPkg/Library/ArmSoftFloatLib/softfloat-specialize    |  525 -----
BaseTools/Conf/tools_def.template                      |    8 +-
MdeModulePkg/Include/Protocol/PeCoffImageEmulator.h    |    4 +-
MdePkg/Include/Protocol/HiiPopup.h                     |    4 +-
MdePkg/Include/Protocol/ResetNotification.h            |    4 +-
31 files changed, 374 insertions(+), 4735 deletions(-)
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_cdcmp.asm
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_cfcmp.asm
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpeq.c
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpge.c
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpgt.c
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmple.c
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmplt.c
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpun.c
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpeq.c
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpge.c
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpgt.c
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmple.c
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmplt.c
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpun.c
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/softfloat.h
create mode 100644 ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/arm-gcc.h
create mode 160000 ArmPkg/Library/ArmSoftFloatLib/berkeley-softfloat-3
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/bits32/softfloat-macros
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/bits32/softfloat.c
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/milieu.h
create mode 100644 ArmPkg/Library/ArmSoftFloatLib/platform.h
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/softfloat-for-gcc.h
delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/softfloat-specialize
[edk2-devel] [PATCH v3 0/7] update ArmSoftFloatLib to latest upstream version
Posted by Ard Biesheuvel 4 years, 10 months ago
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)

Note that this drops support for RVCT, but this toolchain is untested
and will likely be removed. [0]

Code can be found here:
https://github.com/ardbiesheuvel/edk2/tree/arm-softfloat-upgrade-v3

Changes since v2:
- dodge the license debate by importing the external project as a Git
  submodule (which turns out to be maintained by the author/owner of
  the project)
- drop file deletion and SPDX header changes that are no longer needed
- drop tested-by tags
- add Laszlo's ack where appropriate
- add new patches to fix some protocol headers that deviate from the
  normal pattern of how we define protocols, which breaks C99 and
  therefore Clang with the atomics workaround in #4 applied

Changes since v1:
- Update the INLINE macro to pass __attribute__((always_inline)), to work
  around a build issue that occurs when optimization is disabled.
- Add a patch to remove the unused files. As it turns out, 385 of the ~450
  files are unused (which is a lot more than Laszlo reported), and so it
  makes sense to merge patch #4 into patch #1
- Add a patch to add SPDX identifiers to the files that we are keeping (#5).
  This patch can be merged into #1 as well.
- Add patches to work around issues that prevent the new code to build with
  Clang. Patch #6 is actually related to the OpenSSL upgrade, while patch #7
  is related to Clang 7 (and has been posted already 6 months ago)

Note that another issue popped up, which affects the OpenSSL 1.1.1b upgrade:

  .../CryptoPkg/Library/OpensslLib/openssl/crypto/conf/conf_sap.c:71:12:
  error: variable 'ret' is uninitialized when used here [-Werror,-Wuninitialized]
    return ret;
           ^~~
  .../CryptoPkg/Library/OpensslLib/openssl/crypto/conf/conf_sap.c:44:12:
  note: initialize the variable 'ret' to silence this warning
    int ret;
           ^
            = 0
  1 error generated.

This appears to be a regression in the OpenSSL upstream code (commit
25eb9299cec), which no longer initializes ret if OPENSSL_SYS_UEFI is #defined.

Cc: Laszlo Ersek <lersek@redhat.com>
Cc: "Gao, Liming" <liming.gao@intel.com>
Cc: "Wang, Jian J" <jian.j.wang@intel.com>
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Michael D Kinney <michael.d.kinney@intel.com>

[0] https://bugzilla.tianocore.org/show_bug.cgi?id=1750

Ard Biesheuvel (7):
  ArmPkg: import Berkeley Softfloat library as git submodule
  ArmPkg/ArmSoftFloatLib: switch to new version of softfloat library
  ArmPkg/ArmSoftFloatLib: remove source files that are no longer used
  BaseTools/tools_def CLANG3x ARM AARCH64: force use of C99 standard
  ArmPkg/ArmLib ARM: set .fpu to let Clang 7 assemble ArmV7Support.S
  MdeModulePkg/PeCoffImageEmulator: avoid redeclaration of typedef
  MdePkg: avoid redeclaration of typedefs

 .gitmodules                                            |    3 +
 ArmPkg/Library/ArmLib/Arm/ArmV7Support.S               |    2 +
 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_cdcmp.asm   |   41 -
 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_cfcmp.asm   |   37 -
 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpeq.c    |   30 -
 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpge.c    |   28 -
 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpgt.c    |   30 -
 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmple.c    |   30 -
 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmplt.c    |   30 -
 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpun.c    |   35 -
 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpeq.c    |   30 -
 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpge.c    |   30 -
 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpgt.c    |   30 -
 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmple.c    |   30 -
 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmplt.c    |   30 -
 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpun.c    |   35 -
 ArmPkg/Library/ArmSoftFloatLib/Arm/softfloat.h         |  345 ---
 ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c       |  283 +++
 ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.inf     |   83 +-
 ArmPkg/Library/ArmSoftFloatLib/arm-gcc.h               |  108 -
 ArmPkg/Library/ArmSoftFloatLib/berkeley-softfloat-3    |    1 +
 ArmPkg/Library/ArmSoftFloatLib/bits32/softfloat-macros |  648 ------
 ArmPkg/Library/ArmSoftFloatLib/bits32/softfloat.c      | 2354 --------------------
 ArmPkg/Library/ArmSoftFloatLib/milieu.h                |   38 -
 ArmPkg/Library/ArmSoftFloatLib/platform.h              |   11 +
 ArmPkg/Library/ArmSoftFloatLib/softfloat-for-gcc.h     |  242 --
 ArmPkg/Library/ArmSoftFloatLib/softfloat-specialize    |  525 -----
 BaseTools/Conf/tools_def.template                      |    8 +-
 MdeModulePkg/Include/Protocol/PeCoffImageEmulator.h    |    4 +-
 MdePkg/Include/Protocol/HiiPopup.h                     |    4 +-
 MdePkg/Include/Protocol/ResetNotification.h            |    4 +-
 31 files changed, 374 insertions(+), 4735 deletions(-)
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_cdcmp.asm
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_cfcmp.asm
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpeq.c
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpge.c
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpgt.c
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmple.c
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmplt.c
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpun.c
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpeq.c
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpge.c
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpgt.c
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmple.c
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmplt.c
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpun.c
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/softfloat.h
 create mode 100644 ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/arm-gcc.h
 create mode 160000 ArmPkg/Library/ArmSoftFloatLib/berkeley-softfloat-3
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/bits32/softfloat-macros
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/bits32/softfloat.c
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/milieu.h
 create mode 100644 ArmPkg/Library/ArmSoftFloatLib/platform.h
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/softfloat-for-gcc.h
 delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/softfloat-specialize

-- 
2.20.1


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#41735): https://edk2.groups.io/g/devel/message/41735
Mute This Topic: https://groups.io/mt/31879986/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v3 0/7] update ArmSoftFloatLib to latest upstream version
Posted by Leif Lindholm 4 years, 10 months ago
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).

/
    Leif

> Note that this drops support for RVCT, but this toolchain is untested
> and will likely be removed. [0]
> 
> Code can be found here:
> https://github.com/ardbiesheuvel/edk2/tree/arm-softfloat-upgrade-v3
> 
> Changes since v2:
> - dodge the license debate by importing the external project as a Git
>   submodule (which turns out to be maintained by the author/owner of
>   the project)
> - drop file deletion and SPDX header changes that are no longer needed
> - drop tested-by tags
> - add Laszlo's ack where appropriate
> - add new patches to fix some protocol headers that deviate from the
>   normal pattern of how we define protocols, which breaks C99 and
>   therefore Clang with the atomics workaround in #4 applied
> 
> Changes since v1:
> - Update the INLINE macro to pass __attribute__((always_inline)), to work
>   around a build issue that occurs when optimization is disabled.
> - Add a patch to remove the unused files. As it turns out, 385 of the ~450
>   files are unused (which is a lot more than Laszlo reported), and so it
>   makes sense to merge patch #4 into patch #1
> - Add a patch to add SPDX identifiers to the files that we are keeping (#5).
>   This patch can be merged into #1 as well.
> - Add patches to work around issues that prevent the new code to build with
>   Clang. Patch #6 is actually related to the OpenSSL upgrade, while patch #7
>   is related to Clang 7 (and has been posted already 6 months ago)
> 
> Note that another issue popped up, which affects the OpenSSL 1.1.1b upgrade:
> 
>   .../CryptoPkg/Library/OpensslLib/openssl/crypto/conf/conf_sap.c:71:12:
>   error: variable 'ret' is uninitialized when used here [-Werror,-Wuninitialized]
>     return ret;
>            ^~~
>   .../CryptoPkg/Library/OpensslLib/openssl/crypto/conf/conf_sap.c:44:12:
>   note: initialize the variable 'ret' to silence this warning
>     int ret;
>            ^
>             = 0
>   1 error generated.
> 
> This appears to be a regression in the OpenSSL upstream code (commit
> 25eb9299cec), which no longer initializes ret if OPENSSL_SYS_UEFI is #defined.
> 
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: "Gao, Liming" <liming.gao@intel.com>
> Cc: "Wang, Jian J" <jian.j.wang@intel.com>
> Cc: Leif Lindholm <leif.lindholm@linaro.org>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> 
> [0] https://bugzilla.tianocore.org/show_bug.cgi?id=1750
> 
> Ard Biesheuvel (7):
>   ArmPkg: import Berkeley Softfloat library as git submodule
>   ArmPkg/ArmSoftFloatLib: switch to new version of softfloat library
>   ArmPkg/ArmSoftFloatLib: remove source files that are no longer used
>   BaseTools/tools_def CLANG3x ARM AARCH64: force use of C99 standard
>   ArmPkg/ArmLib ARM: set .fpu to let Clang 7 assemble ArmV7Support.S
>   MdeModulePkg/PeCoffImageEmulator: avoid redeclaration of typedef
>   MdePkg: avoid redeclaration of typedefs
> 
>  .gitmodules                                            |    3 +
>  ArmPkg/Library/ArmLib/Arm/ArmV7Support.S               |    2 +
>  ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_cdcmp.asm   |   41 -
>  ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_cfcmp.asm   |   37 -
>  ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpeq.c    |   30 -
>  ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpge.c    |   28 -
>  ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpgt.c    |   30 -
>  ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmple.c    |   30 -
>  ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmplt.c    |   30 -
>  ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpun.c    |   35 -
>  ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpeq.c    |   30 -
>  ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpge.c    |   30 -
>  ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpgt.c    |   30 -
>  ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmple.c    |   30 -
>  ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmplt.c    |   30 -
>  ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpun.c    |   35 -
>  ArmPkg/Library/ArmSoftFloatLib/Arm/softfloat.h         |  345 ---
>  ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c       |  283 +++
>  ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.inf     |   83 +-
>  ArmPkg/Library/ArmSoftFloatLib/arm-gcc.h               |  108 -
>  ArmPkg/Library/ArmSoftFloatLib/berkeley-softfloat-3    |    1 +
>  ArmPkg/Library/ArmSoftFloatLib/bits32/softfloat-macros |  648 ------
>  ArmPkg/Library/ArmSoftFloatLib/bits32/softfloat.c      | 2354 --------------------
>  ArmPkg/Library/ArmSoftFloatLib/milieu.h                |   38 -
>  ArmPkg/Library/ArmSoftFloatLib/platform.h              |   11 +
>  ArmPkg/Library/ArmSoftFloatLib/softfloat-for-gcc.h     |  242 --
>  ArmPkg/Library/ArmSoftFloatLib/softfloat-specialize    |  525 -----
>  BaseTools/Conf/tools_def.template                      |    8 +-
>  MdeModulePkg/Include/Protocol/PeCoffImageEmulator.h    |    4 +-
>  MdePkg/Include/Protocol/HiiPopup.h                     |    4 +-
>  MdePkg/Include/Protocol/ResetNotification.h            |    4 +-
>  31 files changed, 374 insertions(+), 4735 deletions(-)
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_cdcmp.asm
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_cfcmp.asm
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpeq.c
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpge.c
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpgt.c
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmple.c
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmplt.c
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_dcmpun.c
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpeq.c
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpge.c
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpgt.c
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmple.c
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmplt.c
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/__aeabi_fcmpun.c
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/Arm/softfloat.h
>  create mode 100644 ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/arm-gcc.h
>  create mode 160000 ArmPkg/Library/ArmSoftFloatLib/berkeley-softfloat-3
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/bits32/softfloat-macros
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/bits32/softfloat.c
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/milieu.h
>  create mode 100644 ArmPkg/Library/ArmSoftFloatLib/platform.h
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/softfloat-for-gcc.h
>  delete mode 100644 ArmPkg/Library/ArmSoftFloatLib/softfloat-specialize
> 
> -- 
> 2.20.1
> 

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#41743): https://edk2.groups.io/g/devel/message/41743
Mute This Topic: https://groups.io/mt/31879986/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v3 0/7] update ArmSoftFloatLib to latest upstream version
Posted by Ard Biesheuvel 4 years, 10 months ago
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.

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#41750): https://edk2.groups.io/g/devel/message/41750
Mute This Topic: https://groups.io/mt/31879986/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v3 0/7] update ArmSoftFloatLib to latest upstream version
Posted by Michael D Kinney 4 years, 10 months ago
Ard,

Thank you for all the hard work on this issue.

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.
> 
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#41752): https://edk2.groups.io/g/devel/message/41752
Mute This Topic: https://groups.io/mt/31879986/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v3 0/7] update ArmSoftFloatLib to latest upstream version
Posted by Leif Lindholm 4 years, 10 months ago
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.
> > 
> > 
> 

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#41759): https://edk2.groups.io/g/devel/message/41759
Mute This Topic: https://groups.io/mt/31879986/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v3 0/7] update ArmSoftFloatLib to latest upstream version
Posted by Michael D Kinney 4 years, 10 months ago
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.
> > >
> > >
> >
> 
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#41760): https://edk2.groups.io/g/devel/message/41760
Mute This Topic: https://groups.io/mt/31879986/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v3 0/7] update ArmSoftFloatLib to latest upstream version
Posted by Liming Gao 4 years, 9 months ago
Leif:
  I see Ard patch ArmPkg/ArmSoftFloatLib GCC4x: fix build failure https://edk2.groups.io/g/devel/topic/patch/31884674?p=,,,20,0,0,0::recentpostdate%2Fsticky,,,20,2,0,31884674

  Does it fix your CI issue?

Thanks
Liming
>-----Original Message-----
>From: Leif Lindholm [mailto:leif.lindholm@linaro.org]
>Sent: Saturday, June 01, 2019 3:29 AM
>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.
>> >
>> > 
>>

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#41785): https://edk2.groups.io/g/devel/message/41785
Mute This Topic: https://groups.io/mt/31879986/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v3 0/7] update ArmSoftFloatLib to latest upstream version
Posted by Ard Biesheuvel 4 years, 9 months ago
On Mon, 3 Jun 2019 at 02:59, Gao, Liming <liming.gao@intel.com> wrote:
>
> Leif:
>   I see Ard patch ArmPkg/ArmSoftFloatLib GCC4x: fix build failure https://edk2.groups.io/g/devel/topic/patch/31884674?p=,,,20,0,0,0::recentpostdate%2Fsticky,,,20,2,0,31884674
>
>   Does it fix your CI issue?
>

Yes, the CI issue is fixed now.

Patches 6/7 and 7/7 are not required for that, since I did not merge
the patch that adds -std=c99 to all Clang invocations. This was
unnecessary since Jian's patch already adds it as well, but only for
building OpensslLib, which is sufficient.

I will merge 6/7 and 7/7 after the stable tag is created.


> >-----Original Message-----
> >From: Leif Lindholm [mailto:leif.lindholm@linaro.org]
> >Sent: Saturday, June 01, 2019 3:29 AM
> >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.
> >> >
> >> > 
> >>

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#41791): https://edk2.groups.io/g/devel/message/41791
Mute This Topic: https://groups.io/mt/31879986/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v3 0/7] update ArmSoftFloatLib to latest upstream version
Posted by Liming Gao 4 years, 9 months ago
Ard:

>-----Original Message-----
>From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Ard
>Biesheuvel
>Sent: Saturday, June 01, 2019 12: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
>
Great thanks for your effort to resolve it. 

By the way, the patch 6/7, 7/7 are still for this stable tag? They are OK to me. 
I am sure whether we still have the similar cases in MdePkg/MdeModulePkg. 
Can you submit BZ to remove those changes all in Mdekg/MdeModulePkg.

>Thanks all,
>Ard.
>
>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#41779): https://edk2.groups.io/g/devel/message/41779
Mute This Topic: https://groups.io/mt/31879986/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-