From: "Zurcher, Christopher J" <christopher.j.zurcher@intel.com>
To: "Yao, Jiewen" <jiewen.yao@intel.com>, Laszlo Ersek <lersek@redhat.com>
Cc: "devel@edk2.groups.io" <devel@edk2.groups.io>,
gaoliming <gaoliming@byosoft.com.cn>,
"Wang, Jian J" <jian.j.wang@intel.com>,
"Lu, XiaoyuX" <xiaoyux.lu@intel.com>,
"Kinney, Michael D" <michael.d.kinney@intel.com>,
Ard Biesheuvel <ard.biesheuvel@arm.com>
Subject: Re: [edk2-devel] [PATCH v5 0/2] CryptoPkg/OpensslLib: Add native instruction support for X64
Date: Wed, 11 Nov 2020 01:43:13 +0000 [thread overview]
Message-ID: <MWHPR1101MB2125B99478F32CC699BFB281B3E80@MWHPR1101MB2125.namprd11.prod.outlook.com> (raw)
In-Reply-To: <CY4PR11MB1288533B7D1C420014B1C0628CE90@CY4PR11MB1288.namprd11.prod.outlook.com>
I don't want to speak for Laszlo but I filed an issue against OpenSSL that the NASM build should not assume win64:
https://github.com/openssl/openssl/issues/12712
The issue was triaged as a bug by OpenSSL, so I think the long-term plan would be to fix OpenSSL to not set win64 flag by default on all NASM builds, at which point I think we should be able to use the same NASM files for VS and GCC. I'm not sure if the classification as a bug means the fix could be made in 1.1.1x builds or if it could only go into 3.x.
Thanks,
Christopher Zurcher
> -----Original Message-----
> From: Yao, Jiewen <jiewen.yao@intel.com>
> Sent: Tuesday, November 10, 2020 09:08
> To: Laszlo Ersek <lersek@redhat.com>; Zurcher, Christopher J
> <christopher.j.zurcher@intel.com>
> Cc: devel@edk2.groups.io; gaoliming <gaoliming@byosoft.com.cn>; Wang, Jian J
> <jian.j.wang@intel.com>; Lu, XiaoyuX <xiaoyux.lu@intel.com>; Kinney, Michael
> D <michael.d.kinney@intel.com>; Ard Biesheuvel <ard.biesheuvel@arm.com>
> Subject: RE: [edk2-devel] [PATCH v5 0/2] CryptoPkg/OpensslLib: Add native
> instruction support for X64
>
> Laszlo.
> If you disagree, what is your proposal?
>
>
> > -----Original Message-----
> > From: Laszlo Ersek <lersek@redhat.com>
> > Sent: Tuesday, November 10, 2020 8:31 PM
> > To: Yao, Jiewen <jiewen.yao@intel.com>; Zurcher, Christopher J
> > <christopher.j.zurcher@intel.com>
> > Cc: devel@edk2.groups.io; gaoliming <gaoliming@byosoft.com.cn>; Wang,
> > Jian J <jian.j.wang@intel.com>; Lu, XiaoyuX <xiaoyux.lu@intel.com>; Kinney,
> > Michael D <michael.d.kinney@intel.com>; Ard Biesheuvel
> > <ard.biesheuvel@arm.com>
> > Subject: Re: [edk2-devel] [PATCH v5 0/2] CryptoPkg/OpensslLib: Add native
> > instruction support for X64
> >
> > On 11/07/20 03:24, Yao, Jiewen wrote:
> > > The reason we choose NASM is that we can use same assembly in windows
> > build and Linux build. However if this NASM cannot be used in Linux, then
> > the benefit does not exist any more. You can generate GAS to support GCC
> > build, and check in .S file.
> >
> > I disagree with this idea. To me (as an exclusive GCC user), uniformity
> > of assembly files is *much* more important than getting native
> > instruction support in OpenSSL with all toolchains at the exact same time.
> >
> > If we enable native instruction support for (a) VS and CLANGPDB now, and
> > (b) for GCC later, then that's two steps, with each step being in the
> > forward direction. Performing just (a) for now creates no technical
> > debt. A feature gap is not technical debt; you cannot mistake a missing
> > feature for a working feature.
> >
> > If we re-add .S files now, for whatever purpose, that's a step *back*,
> > however. It creates technical debt. A working feature on an invalid
> > basis *can* be mistaken for a working feature, and we shouldn't do that
> > (unless there are strong business needs for some participants, *AND* we
> > have a *very specific* plan and timeline for backing out the hack). I
> > really don't have any trust in technical debt being "paid" in edk2
> > anytime soon, though.
> >
> > Thanks
> > Laszlo
next prev parent reply other threads:[~2020-11-11 1:43 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-03 21:58 [PATCH v5 0/2] CryptoPkg/OpensslLib: Add native instruction support for X64 Zurcher, Christopher J
2020-11-03 21:58 ` [PATCH v5 1/2] " Zurcher, Christopher J
2020-11-03 21:58 ` [PATCH v5 2/2] CryptoPkg/OpensslLib: Commit the auto-generated assembly files " Zurcher, Christopher J
2020-11-06 5:56 ` [PATCH v5 0/2] CryptoPkg/OpensslLib: Add native instruction support " Yao, Jiewen
[not found] ` <1644D590FF4B7423.25549@groups.io>
2020-11-06 6:13 ` [edk2-devel] " Yao, Jiewen
2020-11-06 9:50 ` Zurcher, Christopher J
2020-11-06 10:22 ` Yao, Jiewen
2020-11-06 19:35 ` Zurcher, Christopher J
2020-11-06 23:06 ` Yao, Jiewen
2020-11-07 2:02 ` Zurcher, Christopher J
2020-11-07 2:24 ` Yao, Jiewen
2020-11-07 7:26 ` Michael D Kinney
2020-11-10 12:31 ` Laszlo Ersek
2020-11-10 14:28 ` 回复: " gaoliming
2020-11-11 19:05 ` Laszlo Ersek
2020-11-10 17:07 ` Yao, Jiewen
2020-11-11 1:43 ` Zurcher, Christopher J [this message]
2020-11-11 2:19 ` Yao, Jiewen
2020-11-11 19:09 ` Laszlo Ersek
2020-11-13 1:22 ` Zurcher, Christopher J
[not found] ` <1646ECAE89844BF6.31886@groups.io>
2020-11-13 2:28 ` Zurcher, Christopher J
2020-11-13 7:35 ` Ard Biesheuvel
2020-11-13 20:02 ` Laszlo Ersek
2020-11-11 19:08 ` Laszlo Ersek
2020-11-11 19:07 ` Laszlo Ersek
2020-11-09 7:03 ` 回复: " gaoliming
2020-11-10 12:24 ` Laszlo Ersek
2020-11-09 6:56 ` 回复: " gaoliming
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-list 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=MWHPR1101MB2125B99478F32CC699BFB281B3E80@MWHPR1101MB2125.namprd11.prod.outlook.com \
--to=devel@edk2.groups.io \
/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