* RFC: Adding support for ARM (RNDR etc.) to RngDxe @ 2020-12-09 4:32 Rebecca Cran 2020-12-09 4:47 ` [edk2-devel] " Samer El-Haj-Mahmoud 0 siblings, 1 reply; 15+ messages in thread From: Rebecca Cran @ 2020-12-09 4:32 UTC (permalink / raw) To: devel@edk2.groups.io; +Cc: rfc, Jiewen Yao, Rahul Kumar Currently, RngDxe in SecurityPkg only supports Intel, with RdRand support. This RFC is to start a discussion about adding support for ARM. I have a Git branch with support for the optional ARMv8.5 RNDR instruction at https://github.com/bcran/edk2/commits/bcran-rndr which moves the existing Intel support into a Rand directory, and adds code to support RNDR in a new AArch64 directory. There are other RNG implementations available for ARM, including platform-specific approaches on Graviton (https://lwn.net/Articles/790304/) and other platforms, so a more thorough rearchitecting/redesign may be desired. -- Rebecca Cran ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe 2020-12-09 4:32 RFC: Adding support for ARM (RNDR etc.) to RngDxe Rebecca Cran @ 2020-12-09 4:47 ` Samer El-Haj-Mahmoud 2020-12-10 11:26 ` Sami Mujawar 0 siblings, 1 reply; 15+ messages in thread From: Samer El-Haj-Mahmoud @ 2020-12-09 4:47 UTC (permalink / raw) To: devel@edk2.groups.io; +Cc: rfc@edk2.groups.io, Jiewen Yao, Rahul Kumar [-- Attachment #1: Type: text/plain, Size: 1483 bytes --] There is also the TRNG FW API, which is an architected SMC firmware interface: https://developer.arm.com/documentation/den0098/latest/ ________________________________ From: devel@edk2.groups.io <devel@edk2.groups.io> on behalf of Rebecca Cran via groups.io <rebecca=nuviainc.com@groups.io> Sent: Tuesday, December 8, 2020, 11:33 PM To: devel@edk2.groups.io Cc: rfc@edk2.groups.io; Jiewen Yao; Rahul Kumar Subject: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe Currently, RngDxe in SecurityPkg only supports Intel, with RdRand support. This RFC is to start a discussion about adding support for ARM. I have a Git branch with support for the optional ARMv8.5 RNDR instruction at https://github.com/bcran/edk2/commits/bcran-rndr which moves the existing Intel support into a Rand directory, and adds code to support RNDR in a new AArch64 directory. There are other RNG implementations available for ARM, including platform-specific approaches on Graviton (https://lwn.net/Articles/790304/) and other platforms, so a more thorough rearchitecting/redesign may be desired. -- Rebecca Cran IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. [-- Attachment #2: Type: text/html, Size: 2999 bytes --] ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe 2020-12-09 4:47 ` [edk2-devel] " Samer El-Haj-Mahmoud @ 2020-12-10 11:26 ` Sami Mujawar 2020-12-10 12:52 ` Leif Lindholm 2021-01-14 21:05 ` Rebecca Cran 0 siblings, 2 replies; 15+ messages in thread From: Sami Mujawar @ 2020-12-10 11:26 UTC (permalink / raw) To: devel@edk2.groups.io, Samer El-Haj-Mahmoud, Ard Biesheuvel, rebecca@nuviainc.com, leif@nuviainc.com Cc: rfc@edk2.groups.io, Jiewen Yao, Rahul Kumar, nd [-- Attachment #1: Type: text/plain, Size: 2443 bytes --] Hi All, I am working on the TRNG FW API interface and will share more details for the discussion soon. We had some thoughts about streamlining the RngDxe implementations and would like to share some diagrams for the discussion. My diagrams are in Visio that I can export as JPG images. However, I am open to switching to any other suggested tool. Hi Leif, Can you suggest on how we can collaborate to share diagrams/documents, please? Regards, Sami Mujawar From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Samer El-Haj-Mahmoud via groups.io Sent: 09 December 2020 04:48 AM To: devel@edk2.groups.io Cc: rfc@edk2.groups.io; Jiewen Yao <jiewen.yao@intel.com>; Rahul Kumar <rahul1.kumar@intel.com> Subject: Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe There is also the TRNG FW API, which is an architected SMC firmware interface: https://developer.arm.com/documentation/den0098/latest/ ________________________________ From: devel@edk2.groups.io<mailto:devel@edk2.groups.io> <devel@edk2.groups.io<mailto:devel@edk2.groups.io>> on behalf of Rebecca Cran via groups.io <rebecca=nuviainc.com@groups.io<mailto:rebecca=nuviainc.com@groups.io>> Sent: Tuesday, December 8, 2020, 11:33 PM To: devel@edk2.groups.io<mailto:devel@edk2.groups.io> Cc: rfc@edk2.groups.io<mailto:rfc@edk2.groups.io>; Jiewen Yao; Rahul Kumar Subject: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe Currently, RngDxe in SecurityPkg only supports Intel, with RdRand support. This RFC is to start a discussion about adding support for ARM. I have a Git branch with support for the optional ARMv8.5 RNDR instruction at https://github.com/bcran/edk2/commits/bcran-rndr which moves the existing Intel support into a Rand directory, and adds code to support RNDR in a new AArch64 directory. There are other RNG implementations available for ARM, including platform-specific approaches on Graviton (https://lwn.net/Articles/790304/) and other platforms, so a more thorough rearchitecting/redesign may be desired. -- Rebecca Cran IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. [-- Attachment #2: Type: text/html, Size: 7480 bytes --] ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe 2020-12-10 11:26 ` Sami Mujawar @ 2020-12-10 12:52 ` Leif Lindholm 2020-12-10 13:43 ` [edk2-rfc] " Yao, Jiewen 2021-01-14 21:05 ` Rebecca Cran 1 sibling, 1 reply; 15+ messages in thread From: Leif Lindholm @ 2020-12-10 12:52 UTC (permalink / raw) To: Sami Mujawar Cc: devel@edk2.groups.io, Samer El-Haj-Mahmoud, Ard Biesheuvel, rebecca@nuviainc.com, rfc@edk2.groups.io, Jiewen Yao, Rahul Kumar, nd Hi Sami, JPGs work, but preferably published in a location where they're unlikely to be deleted, and posted as URLs. https://app.diagrams.net/ doesn't require a licensed application to edit, and can be "saved to github" for example. Please make sure to use the diagrams to support/clarify the mailing list conversation rather than replacing it. Best Regards, Leif On Thu, Dec 10, 2020 at 11:26:26 +0000, Sami Mujawar wrote: > Hi All, > > I am working on the TRNG FW API interface and will share more details for the discussion soon. > We had some thoughts about streamlining the RngDxe implementations > and would like to share some diagrams for the discussion. > My diagrams are in Visio that I can export as JPG images. However, I > am open to switching to any other suggested tool. > > Hi Leif, > > Can you suggest on how we can collaborate to share diagrams/documents, please? > > Regards, > > Sami Mujawar > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Samer El-Haj-Mahmoud via groups.io > Sent: 09 December 2020 04:48 AM > To: devel@edk2.groups.io > Cc: rfc@edk2.groups.io; Jiewen Yao <jiewen.yao@intel.com>; Rahul Kumar <rahul1.kumar@intel.com> > Subject: Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe > > There is also the TRNG FW API, which is an architected SMC firmware interface: > > https://developer.arm.com/documentation/den0098/latest/ > > ________________________________ > From: devel@edk2.groups.io<mailto:devel@edk2.groups.io> <devel@edk2.groups.io<mailto:devel@edk2.groups.io>> on behalf of Rebecca Cran via groups.io <rebecca=nuviainc.com@groups.io<mailto:rebecca=nuviainc.com@groups.io>> > Sent: Tuesday, December 8, 2020, 11:33 PM > To: devel@edk2.groups.io<mailto:devel@edk2.groups.io> > Cc: rfc@edk2.groups.io<mailto:rfc@edk2.groups.io>; Jiewen Yao; Rahul Kumar > Subject: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe > > Currently, RngDxe in SecurityPkg only supports Intel, with RdRand support. > > > This RFC is to start a discussion about adding support for ARM. > > > I have a Git branch with support for the optional ARMv8.5 RNDR > instruction at https://github.com/bcran/edk2/commits/bcran-rndr which > moves the existing Intel support into a Rand directory, and adds code to > support RNDR in a new AArch64 directory. > > There are other RNG implementations available for ARM, including > platform-specific approaches on Graviton > (https://lwn.net/Articles/790304/) and other platforms, so a more > thorough rearchitecting/redesign may be desired. > > > -- > Rebecca Cran > > > > > > > IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [edk2-rfc] [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe 2020-12-10 12:52 ` Leif Lindholm @ 2020-12-10 13:43 ` Yao, Jiewen 0 siblings, 0 replies; 15+ messages in thread From: Yao, Jiewen @ 2020-12-10 13:43 UTC (permalink / raw) To: rfc@edk2.groups.io, leif@nuviainc.com, Sami Mujawar Cc: devel@edk2.groups.io, Samer El-Haj-Mahmoud, Ard Biesheuvel, rebecca@nuviainc.com, Kumar, Rahul1, nd Maybe you can upload the content to https://edk2.groups.io/g/devel/files/Designs, where we hold the design review ppt, etc. I assume we want to discuss below two APIs implementation, right? 1) RngLib 2) EFI_RNG_PROTOCOL Thank you Yao Jiewen > -----Original Message----- > From: rfc@edk2.groups.io <rfc@edk2.groups.io> On Behalf Of Leif Lindholm > Sent: Thursday, December 10, 2020 8:52 PM > To: Sami Mujawar <Sami.Mujawar@arm.com> > Cc: devel@edk2.groups.io; Samer El-Haj-Mahmoud <Samer.El-Haj- > Mahmoud@arm.com>; Ard Biesheuvel <Ard.Biesheuvel@arm.com>; > rebecca@nuviainc.com; rfc@edk2.groups.io; Yao, Jiewen > <jiewen.yao@intel.com>; Kumar, Rahul1 <rahul1.kumar@intel.com>; nd > <nd@arm.com> > Subject: Re: [edk2-rfc] [edk2-devel] RFC: Adding support for ARM (RNDR etc.) > to RngDxe > > Hi Sami, > > JPGs work, but preferably published in a location where they're > unlikely to be deleted, and posted as URLs. > > https://app.diagrams.net/ doesn't require a licensed application to > edit, and can be "saved to github" for example. > > Please make sure to use the diagrams to support/clarify the mailing > list conversation rather than replacing it. > > Best Regards, > > Leif > > On Thu, Dec 10, 2020 at 11:26:26 +0000, Sami Mujawar wrote: > > Hi All, > > > > I am working on the TRNG FW API interface and will share more details for > the discussion soon. > > We had some thoughts about streamlining the RngDxe implementations > > and would like to share some diagrams for the discussion. > > My diagrams are in Visio that I can export as JPG images. However, I > > am open to switching to any other suggested tool. > > > > Hi Leif, > > > > Can you suggest on how we can collaborate to share diagrams/documents, > please? > > > > Regards, > > > > Sami Mujawar > > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Samer > El-Haj-Mahmoud via groups.io > > Sent: 09 December 2020 04:48 AM > > To: devel@edk2.groups.io > > Cc: rfc@edk2.groups.io; Jiewen Yao <jiewen.yao@intel.com>; Rahul Kumar > <rahul1.kumar@intel.com> > > Subject: Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to > RngDxe > > > > There is also the TRNG FW API, which is an architected SMC firmware > interface: > > > > https://developer.arm.com/documentation/den0098/latest/ > > > > ________________________________ > > From: devel@edk2.groups.io<mailto:devel@edk2.groups.io> > <devel@edk2.groups.io<mailto:devel@edk2.groups.io>> on behalf of > Rebecca Cran via groups.io > <rebecca=nuviainc.com@groups.io<mailto:rebecca=nuviainc.com@groups.io > >> > > Sent: Tuesday, December 8, 2020, 11:33 PM > > To: devel@edk2.groups.io<mailto:devel@edk2.groups.io> > > Cc: rfc@edk2.groups.io<mailto:rfc@edk2.groups.io>; Jiewen Yao; Rahul > Kumar > > Subject: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe > > > > Currently, RngDxe in SecurityPkg only supports Intel, with RdRand support. > > > > > > This RFC is to start a discussion about adding support for ARM. > > > > > > I have a Git branch with support for the optional ARMv8.5 RNDR > > instruction at https://github.com/bcran/edk2/commits/bcran-rndr which > > moves the existing Intel support into a Rand directory, and adds code to > > support RNDR in a new AArch64 directory. > > > > There are other RNG implementations available for ARM, including > > platform-specific approaches on Graviton > > (https://lwn.net/Articles/790304/) and other platforms, so a more > > thorough rearchitecting/redesign may be desired. > > > > > > -- > > Rebecca Cran > > > > > > > > > > > > > > IMPORTANT NOTICE: The contents of this email and any attachments are > confidential and may also be privileged. If you are not the intended recipient, > please notify the sender immediately and do not disclose the contents to > any other person, use it for any purpose, or store or copy the information in > any medium. Thank you. > > > > > > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe 2020-12-10 11:26 ` Sami Mujawar 2020-12-10 12:52 ` Leif Lindholm @ 2021-01-14 21:05 ` Rebecca Cran 2021-01-16 2:51 ` Sami Mujawar 1 sibling, 1 reply; 15+ messages in thread From: Rebecca Cran @ 2021-01-14 21:05 UTC (permalink / raw) To: Sami Mujawar, devel@edk2.groups.io, Samer El-Haj-Mahmoud, Ard Biesheuvel, leif@nuviainc.com Cc: rfc@edk2.groups.io, Jiewen Yao, Rahul Kumar, nd On 12/10/20 4:26 AM, Sami Mujawar wrote: > I am working on the TRNG FW API interface and will share more details > for the discussion soon. > > We had some thoughts about streamlining the RngDxe implementations and > would like to share some diagrams for the discussion. > > My diagrams are in Visio that I can export as JPG images. However, I am > open to switching to any other suggested tool. Hi Sami, I don't see any further discussions on this. Have you made any progress with sharing the design documents or scheduling a review? -- Rebecca Cran ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe 2021-01-14 21:05 ` Rebecca Cran @ 2021-01-16 2:51 ` Sami Mujawar 2021-02-10 22:49 ` Rebecca Cran 2021-04-20 20:04 ` Rebecca Cran 0 siblings, 2 replies; 15+ messages in thread From: Sami Mujawar @ 2021-01-16 2:51 UTC (permalink / raw) To: devel@edk2.groups.io, rebecca@nuviainc.com, Samer El-Haj-Mahmoud, Ard Biesheuvel, leif@nuviainc.com Cc: rfc@edk2.groups.io, Jiewen Yao, Rahul Kumar, nd, Jose Marinho Hi All, I have shared some initial thoughts on the RNG implementation updates at https://edk2.groups.io/g/devel/files/Designs/2021/0116/EDKII%20-%20Proposed%20update%20to%20RNG%20implementation.pdf Kindly let me know your feedback or if you have any queries. Regards, Sami Mujawar -----Original Message----- From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Rebecca Cran via groups.io Sent: 14 January 2021 09:05 PM To: Sami Mujawar <Sami.Mujawar@arm.com>; devel@edk2.groups.io; Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>; Ard Biesheuvel <Ard.Biesheuvel@arm.com>; leif@nuviainc.com Cc: rfc@edk2.groups.io; Jiewen Yao <jiewen.yao@intel.com>; Rahul Kumar <rahul1.kumar@intel.com>; nd <nd@arm.com> Subject: Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe On 12/10/20 4:26 AM, Sami Mujawar wrote: > I am working on the TRNG FW API interface and will share more details > for the discussion soon. > > We had some thoughts about streamlining the RngDxe implementations and > would like to share some diagrams for the discussion. > > My diagrams are in Visio that I can export as JPG images. However, I am > open to switching to any other suggested tool. Hi Sami, I don't see any further discussions on this. Have you made any progress with sharing the design documents or scheduling a review? -- Rebecca Cran ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe 2021-01-16 2:51 ` Sami Mujawar @ 2021-02-10 22:49 ` Rebecca Cran 2021-02-11 18:26 ` Ard Biesheuvel 2021-04-20 20:04 ` Rebecca Cran 1 sibling, 1 reply; 15+ messages in thread From: Rebecca Cran @ 2021-02-10 22:49 UTC (permalink / raw) To: Sami Mujawar, devel@edk2.groups.io, Samer El-Haj-Mahmoud, Ard Biesheuvel, leif@nuviainc.com Cc: rfc@edk2.groups.io, Jiewen Yao, Rahul Kumar, nd, Jose Marinho On 1/15/21 7:51 PM, Sami Mujawar wrote: > I have shared some initial thoughts on the RNG implementation updates at https://edk2.groups.io/g/devel/files/Designs/2021/0116/EDKII%20-%20Proposed%20update%20to%20RNG%20implementation.pdf > > Kindly let me know your feedback or if you have any queries. The ARMv8.5 RNDRRS instruction appears to be missing from the diagram on page 11 - it has RngLib|RNDR, which is listed under PRNG, but RNDRRS returns a true random number. From the Arm ARM: "Returns a 64-bit random number which is reseeded from the True Random Number source immediately before the read of the random number." -- Rebecca Cran ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe 2021-02-10 22:49 ` Rebecca Cran @ 2021-02-11 18:26 ` Ard Biesheuvel 0 siblings, 0 replies; 15+ messages in thread From: Ard Biesheuvel @ 2021-02-11 18:26 UTC (permalink / raw) To: devel, Rebecca Cran Cc: Sami Mujawar, Samer El-Haj-Mahmoud, Ard Biesheuvel, leif@nuviainc.com, rfc@edk2.groups.io, Jiewen Yao, Rahul Kumar, nd, Jose Marinho On Wed, 10 Feb 2021 at 23:49, Rebecca Cran <rebecca@nuviainc.com> wrote: > > On 1/15/21 7:51 PM, Sami Mujawar wrote: > > > I have shared some initial thoughts on the RNG implementation updates at https://edk2.groups.io/g/devel/files/Designs/2021/0116/EDKII%20-%20Proposed%20update%20to%20RNG%20implementation.pdf > > > > Kindly let me know your feedback or if you have any queries. > The ARMv8.5 RNDRRS instruction appears to be missing from the diagram on > page 11 - it has RngLib|RNDR, which is listed under PRNG, but RNDRRS > returns a true random number. From the Arm ARM: > > "Returns a 64-bit random number which is reseeded from the True > Random Number source immediately before the read of the random number." > This is an unfortunate oversight in the architecture, but RNDRRS most certainly does not return a true random number. RNDR and RNDRRS both return the output of a DRBG (pseudo RNG), and the only difference is the reseed rate: RNDRRS triggers a reseed on every invocation, whereas RNDR triggers a reseed at an IMPDEF rate. ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe 2021-01-16 2:51 ` Sami Mujawar 2021-02-10 22:49 ` Rebecca Cran @ 2021-04-20 20:04 ` Rebecca Cran 2021-04-22 9:30 ` Sami Mujawar 1 sibling, 1 reply; 15+ messages in thread From: Rebecca Cran @ 2021-04-20 20:04 UTC (permalink / raw) To: Sami Mujawar, devel@edk2.groups.io, Samer El-Haj-Mahmoud, Ard Biesheuvel, leif@nuviainc.com Cc: rfc@edk2.groups.io, Jiewen Yao, Rahul Kumar, nd, Jose Marinho Hi Sami, I was wondering if you're still collecting feedback on the design, or if you have a plan and schedule for the implementation? -- Rebecca Cran On 1/15/21 7:51 PM, Sami Mujawar wrote: > Hi All, > > I have shared some initial thoughts on the RNG implementation updates at https://edk2.groups.io/g/devel/files/Designs/2021/0116/EDKII%20-%20Proposed%20update%20to%20RNG%20implementation.pdf > > Kindly let me know your feedback or if you have any queries. > > Regards, > > Sami Mujawar > > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Rebecca Cran via groups.io > Sent: 14 January 2021 09:05 PM > To: Sami Mujawar <Sami.Mujawar@arm.com>; devel@edk2.groups.io; Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>; Ard Biesheuvel <Ard.Biesheuvel@arm.com>; leif@nuviainc.com > Cc: rfc@edk2.groups.io; Jiewen Yao <jiewen.yao@intel.com>; Rahul Kumar <rahul1.kumar@intel.com>; nd <nd@arm.com> > Subject: Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe > > On 12/10/20 4:26 AM, Sami Mujawar wrote: > >> I am working on the TRNG FW API interface and will share more details >> for the discussion soon. >> >> We had some thoughts about streamlining the RngDxe implementations and >> would like to share some diagrams for the discussion. >> >> My diagrams are in Visio that I can export as JPG images. However, I am >> open to switching to any other suggested tool. > > Hi Sami, > > I don't see any further discussions on this. Have you made any progress > with sharing the design documents or scheduling a review? > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe 2021-04-20 20:04 ` Rebecca Cran @ 2021-04-22 9:30 ` Sami Mujawar 2021-04-26 14:45 ` Rebecca Cran 2021-04-26 21:28 ` Rebecca Cran 0 siblings, 2 replies; 15+ messages in thread From: Sami Mujawar @ 2021-04-22 9:30 UTC (permalink / raw) To: Rebecca Cran, devel@edk2.groups.io, Samer El-Haj-Mahmoud, Ard Biesheuvel, leif@nuviainc.com Cc: rfc@edk2.groups.io, Jiewen Yao, Rahul Kumar, nd, Jose Marinho [-- Attachment #1: Type: text/plain, Size: 3223 bytes --] Hi Rebecca, I have been working on the following modules (See slide 11 in “EDKII - Proposed update to RNG implementation.pdf<https://edk2.groups.io/g/devel/files/Designs/2021/0116/EDKII%20-%20Proposed%20update%20to%20RNG%20implementation.pdf>”): 1. TrngLib|FwTrnglib (Arm Firmware TRNG) 2. DrbgLib stack – with support for DrbgAlgorithmLib|CRT_DRBG & AesLib|ArmAesInstructionLib. I plan to post patches for (a) in the next fortnight. Following this I plan to update the proposal with the interface definitions for the various library interfaces in the DrbgLib Stack. I have not looked at RngLib|RNDR as I believe you were interested in implementing the part. Kindly let me know if you plan to implement this and the platform you would be using for testing. It looks like the FVP_Base_AEMv8A-AEMv8A and the FVP-RevC models support RNDR, so these could be used for testing as well. Please feel free to get in touch should you need any help with the model parameters or if you face any issues. Regards, Sami Mujawar From: Rebecca Cran <rebecca@nuviainc.com> Date: Tuesday, 20 April 2021 at 21:04 To: Sami Mujawar <Sami.Mujawar@arm.com>, devel@edk2.groups.io <devel@edk2.groups.io>, Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>, Ard Biesheuvel <Ard.Biesheuvel@arm.com>, leif@nuviainc.com <leif@nuviainc.com> Cc: rfc@edk2.groups.io <rfc@edk2.groups.io>, Jiewen Yao <jiewen.yao@intel.com>, Rahul Kumar <rahul1.kumar@intel.com>, nd <nd@arm.com>, Jose Marinho <Jose.Marinho@arm.com> Subject: Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe Hi Sami, I was wondering if you're still collecting feedback on the design, or if you have a plan and schedule for the implementation? -- Rebecca Cran On 1/15/21 7:51 PM, Sami Mujawar wrote: > Hi All, > > I have shared some initial thoughts on the RNG implementation updates at https://edk2.groups.io/g/devel/files/Designs/2021/0116/EDKII%20-%20Proposed%20update%20to%20RNG%20implementation.pdf > > Kindly let me know your feedback or if you have any queries. > > Regards, > > Sami Mujawar > > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Rebecca Cran via groups.io > Sent: 14 January 2021 09:05 PM > To: Sami Mujawar <Sami.Mujawar@arm.com>; devel@edk2.groups.io; Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>; Ard Biesheuvel <Ard.Biesheuvel@arm.com>; leif@nuviainc.com > Cc: rfc@edk2.groups.io; Jiewen Yao <jiewen.yao@intel.com>; Rahul Kumar <rahul1.kumar@intel.com>; nd <nd@arm.com> > Subject: Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe > > On 12/10/20 4:26 AM, Sami Mujawar wrote: > >> I am working on the TRNG FW API interface and will share more details >> for the discussion soon. >> >> We had some thoughts about streamlining the RngDxe implementations and >> would like to share some diagrams for the discussion. >> >> My diagrams are in Visio that I can export as JPG images. However, I am >> open to switching to any other suggested tool. > > Hi Sami, > > I don't see any further discussions on this. Have you made any progress > with sharing the design documents or scheduling a review? > [-- Attachment #2: Type: text/html, Size: 11357 bytes --] ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe 2021-04-22 9:30 ` Sami Mujawar @ 2021-04-26 14:45 ` Rebecca Cran 2021-04-26 21:28 ` Rebecca Cran 1 sibling, 0 replies; 15+ messages in thread From: Rebecca Cran @ 2021-04-26 14:45 UTC (permalink / raw) To: Sami Mujawar, devel@edk2.groups.io, Samer El-Haj-Mahmoud, Ard Biesheuvel, leif@nuviainc.com Cc: rfc@edk2.groups.io, Jiewen Yao, Rahul Kumar, nd, Jose Marinho Thanks. Yes, I'll work on implementing the RngLib|RNDR part. I'll be using Qemu's sbsa-ref platform for testing. I'll also look into using the FVP_Base_AEMv8A-AEMv8A too. -- Rebecca Cran On 4/22/21 3:30 AM, Sami Mujawar wrote: > Hi Rebecca, > > I have been working on the following modules (See slide 11 in “EDKII - > Proposed update to RNG implementation.pdf > <https://edk2.groups.io/g/devel/files/Designs/2021/0116/EDKII%20-%20Proposed%20update%20to%20RNG%20implementation.pdf>”): > > 1. TrngLib|FwTrnglib (Arm Firmware TRNG) > 2. DrbgLib stack – with support for DrbgAlgorithmLib|CRT_DRBG & > AesLib|ArmAesInstructionLib. > > I plan to post patches for (a) in the next fortnight. Following this I > plan to update the proposal with the interface definitions for the > various library interfaces in the DrbgLib Stack. > > I have not looked at RngLib|RNDR as I believe you were interested in > implementing the part. Kindly let me know if you plan to implement this > and the platform you would be using for testing. It looks like the > FVP_Base_AEMv8A-AEMv8A and the FVP-RevC models support RNDR, so these > could be used for testing as well. Please feel free to get in touch > should you need any help with the model parameters or if you face any > issues. > > Regards, > > Sami Mujawar > > *From: *Rebecca Cran <rebecca@nuviainc.com> > *Date: *Tuesday, 20 April 2021 at 21:04 > *To: *Sami Mujawar <Sami.Mujawar@arm.com>, devel@edk2.groups.io > <devel@edk2.groups.io>, Samer El-Haj-Mahmoud > <Samer.El-Haj-Mahmoud@arm.com>, Ard Biesheuvel <Ard.Biesheuvel@arm.com>, > leif@nuviainc.com <leif@nuviainc.com> > *Cc: *rfc@edk2.groups.io <rfc@edk2.groups.io>, Jiewen Yao > <jiewen.yao@intel.com>, Rahul Kumar <rahul1.kumar@intel.com>, nd > <nd@arm.com>, Jose Marinho <Jose.Marinho@arm.com> > *Subject: *Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to > RngDxe > > Hi Sami, > > I was wondering if you're still collecting feedback on the design, or if > you have a plan and schedule for the implementation? > > -- > Rebecca Cran > > On 1/15/21 7:51 PM, Sami Mujawar wrote: > > Hi All, > > > > I have shared some initial thoughts on the RNG implementation updates > at > https://edk2.groups.io/g/devel/files/Designs/2021/0116/EDKII%20-%20Proposed%20update%20to%20RNG%20implementation.pdf > <https://edk2.groups.io/g/devel/files/Designs/2021/0116/EDKII%20-%20Proposed%20update%20to%20RNG%20implementation.pdf> > > > > Kindly let me know your feedback or if you have any queries. > > > > Regards, > > > > Sami Mujawar > > > > -----Original Message----- > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of > Rebecca Cran via groups.io > > Sent: 14 January 2021 09:05 PM > > To: Sami Mujawar <Sami.Mujawar@arm.com>; devel@edk2.groups.io; Samer > El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>; Ard Biesheuvel > <Ard.Biesheuvel@arm.com>; leif@nuviainc.com > > Cc: rfc@edk2.groups.io; Jiewen Yao <jiewen.yao@intel.com>; Rahul > Kumar <rahul1.kumar@intel.com>; nd <nd@arm.com> > > Subject: Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to > RngDxe > > > > On 12/10/20 4:26 AM, Sami Mujawar wrote: > > > >> I am working on the TRNG FW API interface and will share more details > >> for the discussion soon. > >> > >> We had some thoughts about streamlining the RngDxe implementations and > >> would like to share some diagrams for the discussion. > >> > >> My diagrams are in Visio that I can export as JPG images. However, I am > >> open to switching to any other suggested tool. > > > > Hi Sami, > > > > I don't see any further discussions on this. Have you made any progress > > with sharing the design documents or scheduling a review? > > > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe 2021-04-22 9:30 ` Sami Mujawar 2021-04-26 14:45 ` Rebecca Cran @ 2021-04-26 21:28 ` Rebecca Cran 2021-04-26 21:45 ` [EXTERNAL] " Bret Barkelew 1 sibling, 1 reply; 15+ messages in thread From: Rebecca Cran @ 2021-04-26 21:28 UTC (permalink / raw) To: Sami Mujawar, devel@edk2.groups.io, Samer El-Haj-Mahmoud, Ard Biesheuvel, leif@nuviainc.com Cc: rfc@edk2.groups.io, Jiewen Yao, Rahul Kumar, nd, Jose Marinho Hi Sami, I've been looking through the design document again, and was wondering if the work I previously did will just slot in? Were you thinking the "RngLib|RNDR" would go into ArmPkg (since it's not labeled as being in BaseRngLib)? Or would it still make sense to refactor MdePkg/Library/BaseRngLib to support both x86 (using RDRAND) and aarch64 (using RNDR)? -- Rebecca Cran On 4/22/21 3:30 AM, Sami Mujawar wrote: > Hi Rebecca, > > I have been working on the following modules (See slide 11 in “EDKII - > Proposed update to RNG implementation.pdf > <https://edk2.groups.io/g/devel/files/Designs/2021/0116/EDKII%20-%20Proposed%20update%20to%20RNG%20implementation.pdf>”): > > 1. TrngLib|FwTrnglib (Arm Firmware TRNG) > 2. DrbgLib stack – with support for DrbgAlgorithmLib|CRT_DRBG & > AesLib|ArmAesInstructionLib. > > I plan to post patches for (a) in the next fortnight. Following this I > plan to update the proposal with the interface definitions for the > various library interfaces in the DrbgLib Stack. > > I have not looked at RngLib|RNDR as I believe you were interested in > implementing the part. Kindly let me know if you plan to implement this > and the platform you would be using for testing. It looks like the > FVP_Base_AEMv8A-AEMv8A and the FVP-RevC models support RNDR, so these > could be used for testing as well. Please feel free to get in touch > should you need any help with the model parameters or if you face any > issues. > > Regards, > > Sami Mujawar > > *From: *Rebecca Cran <rebecca@nuviainc.com> > *Date: *Tuesday, 20 April 2021 at 21:04 > *To: *Sami Mujawar <Sami.Mujawar@arm.com>, devel@edk2.groups.io > <devel@edk2.groups.io>, Samer El-Haj-Mahmoud > <Samer.El-Haj-Mahmoud@arm.com>, Ard Biesheuvel <Ard.Biesheuvel@arm.com>, > leif@nuviainc.com <leif@nuviainc.com> > *Cc: *rfc@edk2.groups.io <rfc@edk2.groups.io>, Jiewen Yao > <jiewen.yao@intel.com>, Rahul Kumar <rahul1.kumar@intel.com>, nd > <nd@arm.com>, Jose Marinho <Jose.Marinho@arm.com> > *Subject: *Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to > RngDxe > > Hi Sami, > > I was wondering if you're still collecting feedback on the design, or if > you have a plan and schedule for the implementation? > > -- > Rebecca Cran > > On 1/15/21 7:51 PM, Sami Mujawar wrote: > > Hi All, > > > > I have shared some initial thoughts on the RNG implementation updates > at > https://edk2.groups.io/g/devel/files/Designs/2021/0116/EDKII%20-%20Proposed%20update%20to%20RNG%20implementation.pdf > <https://edk2.groups.io/g/devel/files/Designs/2021/0116/EDKII%20-%20Proposed%20update%20to%20RNG%20implementation.pdf> > > > > Kindly let me know your feedback or if you have any queries. > > > > Regards, > > > > Sami Mujawar > > > > -----Original Message----- > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of > Rebecca Cran via groups.io > > Sent: 14 January 2021 09:05 PM > > To: Sami Mujawar <Sami.Mujawar@arm.com>; devel@edk2.groups.io; Samer > El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>; Ard Biesheuvel > <Ard.Biesheuvel@arm.com>; leif@nuviainc.com > > Cc: rfc@edk2.groups.io; Jiewen Yao <jiewen.yao@intel.com>; Rahul > Kumar <rahul1.kumar@intel.com>; nd <nd@arm.com> > > Subject: Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to > RngDxe > > > > On 12/10/20 4:26 AM, Sami Mujawar wrote: > > > >> I am working on the TRNG FW API interface and will share more details > >> for the discussion soon. > >> > >> We had some thoughts about streamlining the RngDxe implementations and > >> would like to share some diagrams for the discussion. > >> > >> My diagrams are in Visio that I can export as JPG images. However, I am > >> open to switching to any other suggested tool. > > > > Hi Sami, > > > > I don't see any further discussions on this. Have you made any progress > > with sharing the design documents or scheduling a review? > > > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [EXTERNAL] Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe 2021-04-26 21:28 ` Rebecca Cran @ 2021-04-26 21:45 ` Bret Barkelew 2021-04-27 13:22 ` Sami Mujawar 0 siblings, 1 reply; 15+ messages in thread From: Bret Barkelew @ 2021-04-26 21:45 UTC (permalink / raw) To: devel@edk2.groups.io, rebecca@nuviainc.com, Sami Mujawar, Samer El-Haj-Mahmoud, Ard Biesheuvel, leif@nuviainc.com Cc: rfc@edk2.groups.io, Yao, Jiewen, Rahul Kumar, nd, Jose Marinho [-- Attachment #1: Type: text/plain, Size: 6446 bytes --] I vote the latter. - Bret From: Rebecca Cran via groups.io<mailto:rebecca=nuviainc.com@groups.io> Sent: Monday, April 26, 2021 2:29 PM To: Sami Mujawar<mailto:Sami.Mujawar@arm.com>; devel@edk2.groups.io<mailto:devel@edk2.groups.io>; Samer El-Haj-Mahmoud<mailto:Samer.El-Haj-Mahmoud@arm.com>; Ard Biesheuvel<mailto:Ard.Biesheuvel@arm.com>; leif@nuviainc.com<mailto:leif@nuviainc.com> Cc: rfc@edk2.groups.io<mailto:rfc@edk2.groups.io>; Yao, Jiewen<mailto:jiewen.yao@intel.com>; Rahul Kumar<mailto:rahul1.kumar@intel.com>; nd<mailto:nd@arm.com>; Jose Marinho<mailto:Jose.Marinho@arm.com> Subject: [EXTERNAL] Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe Hi Sami, I've been looking through the design document again, and was wondering if the work I previously did will just slot in? Were you thinking the "RngLib|RNDR" would go into ArmPkg (since it's not labeled as being in BaseRngLib)? Or would it still make sense to refactor MdePkg/Library/BaseRngLib to support both x86 (using RDRAND) and aarch64 (using RNDR)? -- Rebecca Cran On 4/22/21 3:30 AM, Sami Mujawar wrote: > Hi Rebecca, > > I have been working on the following modules (See slide 11 in “EDKII - > Proposed update to RNG implementation.pdf > <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fedk2.groups.io%2Fg%2Fdevel%2Ffiles%2FDesigns%2F2021%2F0116%2FEDKII%2520-%2520Proposed%2520update%2520to%2520RNG%2520implementation.pdf&data=04%7C01%7Cbret.barkelew%40microsoft.com%7C676a9101f67845dbdc8908d908fa4cd1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637550693569385394%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=Q8ka83ReO2aG8yTVrgpTAVxczJVjl2JBH3ksHo2%2BSHk%3D&reserved=0>”<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fedk2.groups.io%2Fg%2Fdevel%2Ffiles%2FDesigns%2F2021%2F0116%2FEDKII%2520-%2520Proposed%2520update%2520to%2520RNG%2520implementation.pdf&data=04%7C01%7Cbret.barkelew%40microsoft.com%7C676a9101f67845dbdc8908d908fa4cd1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637550693569385394%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=Q8ka83ReO2aG8yTVrgpTAVxczJVjl2JBH3ksHo2%2BSHk%3D&reserved=0%3e”>): > > 1. TrngLib|FwTrnglib (Arm Firmware TRNG) > 2. DrbgLib stack – with support for DrbgAlgorithmLib|CRT_DRBG & > AesLib|ArmAesInstructionLib. > > I plan to post patches for (a) in the next fortnight. Following this I > plan to update the proposal with the interface definitions for the > various library interfaces in the DrbgLib Stack. > > I have not looked at RngLib|RNDR as I believe you were interested in > implementing the part. Kindly let me know if you plan to implement this > and the platform you would be using for testing. It looks like the > FVP_Base_AEMv8A-AEMv8A and the FVP-RevC models support RNDR, so these > could be used for testing as well. Please feel free to get in touch > should you need any help with the model parameters or if you face any > issues. > > Regards, > > Sami Mujawar > > *From: *Rebecca Cran <rebecca@nuviainc.com> > *Date: *Tuesday, 20 April 2021 at 21:04 > *To: *Sami Mujawar <Sami.Mujawar@arm.com>, devel@edk2.groups.io > <devel@edk2.groups.io>, Samer El-Haj-Mahmoud > <Samer.El-Haj-Mahmoud@arm.com>, Ard Biesheuvel <Ard.Biesheuvel@arm.com>, > leif@nuviainc.com <leif@nuviainc.com> > *Cc: *rfc@edk2.groups.io <rfc@edk2.groups.io>, Jiewen Yao > <jiewen.yao@intel.com>, Rahul Kumar <rahul1.kumar@intel.com>, nd > <nd@arm.com>, Jose Marinho <Jose.Marinho@arm.com> > *Subject: *Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to > RngDxe > > Hi Sami, > > I was wondering if you're still collecting feedback on the design, or if > you have a plan and schedule for the implementation? > > -- > Rebecca Cran > > On 1/15/21 7:51 PM, Sami Mujawar wrote: > > Hi All, > > > > I have shared some initial thoughts on the RNG implementation updates > at > https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fedk2.groups.io%2Fg%2Fdevel%2Ffiles%2FDesigns%2F2021%2F0116%2FEDKII%2520-%2520Proposed%2520update%2520to%2520RNG%2520implementation.pdf&data=04%7C01%7Cbret.barkelew%40microsoft.com%7C676a9101f67845dbdc8908d908fa4cd1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637550693569385394%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=Q8ka83ReO2aG8yTVrgpTAVxczJVjl2JBH3ksHo2%2BSHk%3D&reserved=0 > <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fedk2.groups.io%2Fg%2Fdevel%2Ffiles%2FDesigns%2F2021%2F0116%2FEDKII%2520-%2520Proposed%2520update%2520to%2520RNG%2520implementation.pdf&data=04%7C01%7Cbret.barkelew%40microsoft.com%7C676a9101f67845dbdc8908d908fa4cd1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637550693569385394%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=Q8ka83ReO2aG8yTVrgpTAVxczJVjl2JBH3ksHo2%2BSHk%3D&reserved=0> > > > > Kindly let me know your feedback or if you have any queries. > > > > Regards, > > > > Sami Mujawar > > > > -----Original Message----- > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of > Rebecca Cran via groups.io > > Sent: 14 January 2021 09:05 PM > > To: Sami Mujawar <Sami.Mujawar@arm.com>; devel@edk2.groups.io; Samer > El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>; Ard Biesheuvel > <Ard.Biesheuvel@arm.com>; leif@nuviainc.com > > Cc: rfc@edk2.groups.io; Jiewen Yao <jiewen.yao@intel.com>; Rahul > Kumar <rahul1.kumar@intel.com>; nd <nd@arm.com> > > Subject: Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to > RngDxe > > > > On 12/10/20 4:26 AM, Sami Mujawar wrote: > > > >> I am working on the TRNG FW API interface and will share more details > >> for the discussion soon. > >> > >> We had some thoughts about streamlining the RngDxe implementations and > >> would like to share some diagrams for the discussion. > >> > >> My diagrams are in Visio that I can export as JPG images. However, I am > >> open to switching to any other suggested tool. > > > > Hi Sami, > > > > I don't see any further discussions on this. Have you made any progress > > with sharing the design documents or scheduling a review? > > > [-- Attachment #2: Type: text/html, Size: 10737 bytes --] ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [EXTERNAL] Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe 2021-04-26 21:45 ` [EXTERNAL] " Bret Barkelew @ 2021-04-27 13:22 ` Sami Mujawar 0 siblings, 0 replies; 15+ messages in thread From: Sami Mujawar @ 2021-04-27 13:22 UTC (permalink / raw) To: Bret Barkelew, devel@edk2.groups.io, rebecca@nuviainc.com, Samer El-Haj-Mahmoud, Ard Biesheuvel, leif@nuviainc.com Cc: rfc@edk2.groups.io, Yao, Jiewen, Rahul Kumar, nd, Jose Marinho [-- Attachment #1: Type: text/plain, Size: 7295 bytes --] Hi Rebecca, I agree MdePkg/Library/BaseRngLib can be refactored to support both x86 and AArch64. BaseRngLib would then be a RngLib instance that uses CPU instructions to provide random numbers. Regards, Sami Mujawar From: Bret Barkelew <Bret.Barkelew@microsoft.com> Date: Monday, 26 April 2021 at 22:45 To: devel@edk2.groups.io <devel@edk2.groups.io>, rebecca@nuviainc.com <rebecca@nuviainc.com>, Sami Mujawar <Sami.Mujawar@arm.com>, Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>, Ard Biesheuvel <Ard.Biesheuvel@arm.com>, leif@nuviainc.com <leif@nuviainc.com> Cc: rfc@edk2.groups.io <rfc@edk2.groups.io>, Yao, Jiewen <jiewen.yao@intel.com>, Rahul Kumar <rahul1.kumar@intel.com>, nd <nd@arm.com>, Jose Marinho <Jose.Marinho@arm.com> Subject: RE: [EXTERNAL] Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe I vote the latter. - Bret From: Rebecca Cran via groups.io<mailto:rebecca=nuviainc.com@groups.io> Sent: Monday, April 26, 2021 2:29 PM To: Sami Mujawar<mailto:Sami.Mujawar@arm.com>; devel@edk2.groups.io<mailto:devel@edk2.groups.io>; Samer El-Haj-Mahmoud<mailto:Samer.El-Haj-Mahmoud@arm.com>; Ard Biesheuvel<mailto:Ard.Biesheuvel@arm.com>; leif@nuviainc.com<mailto:leif@nuviainc.com> Cc: rfc@edk2.groups.io<mailto:rfc@edk2.groups.io>; Yao, Jiewen<mailto:jiewen.yao@intel.com>; Rahul Kumar<mailto:rahul1.kumar@intel.com>; nd<mailto:nd@arm.com>; Jose Marinho<mailto:Jose.Marinho@arm.com> Subject: [EXTERNAL] Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to RngDxe Hi Sami, I've been looking through the design document again, and was wondering if the work I previously did will just slot in? Were you thinking the "RngLib|RNDR" would go into ArmPkg (since it's not labeled as being in BaseRngLib)? Or would it still make sense to refactor MdePkg/Library/BaseRngLib to support both x86 (using RDRAND) and aarch64 (using RNDR)? -- Rebecca Cran On 4/22/21 3:30 AM, Sami Mujawar wrote: > Hi Rebecca, > > I have been working on the following modules (See slide 11 in “EDKII - > Proposed update to RNG implementation.pdf > <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fedk2.groups.io%2Fg%2Fdevel%2Ffiles%2FDesigns%2F2021%2F0116%2FEDKII%2520-%2520Proposed%2520update%2520to%2520RNG%2520implementation.pdf&data=04%7C01%7Cbret.barkelew%40microsoft.com%7C676a9101f67845dbdc8908d908fa4cd1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637550693569385394%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=Q8ka83ReO2aG8yTVrgpTAVxczJVjl2JBH3ksHo2%2BSHk%3D&reserved=0>”<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fedk2.groups.io%2Fg%2Fdevel%2Ffiles%2FDesigns%2F2021%2F0116%2FEDKII%2520-%2520Proposed%2520update%2520to%2520RNG%2520implementation.pdf&data=04%7C01%7Cbret.barkelew%40microsoft.com%7C676a9101f67845dbdc8908d908fa4cd1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637550693569385394%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=Q8ka83ReO2aG8yTVrgpTAVxczJVjl2JBH3ksHo2%2BSHk%3D&reserved=0%3e”>): > > 1. TrngLib|FwTrnglib (Arm Firmware TRNG) > 2. DrbgLib stack – with support for DrbgAlgorithmLib|CRT_DRBG & > AesLib|ArmAesInstructionLib. > > I plan to post patches for (a) in the next fortnight. Following this I > plan to update the proposal with the interface definitions for the > various library interfaces in the DrbgLib Stack. > > I have not looked at RngLib|RNDR as I believe you were interested in > implementing the part. Kindly let me know if you plan to implement this > and the platform you would be using for testing. It looks like the > FVP_Base_AEMv8A-AEMv8A and the FVP-RevC models support RNDR, so these > could be used for testing as well. Please feel free to get in touch > should you need any help with the model parameters or if you face any > issues. > > Regards, > > Sami Mujawar > > *From: *Rebecca Cran <rebecca@nuviainc.com> > *Date: *Tuesday, 20 April 2021 at 21:04 > *To: *Sami Mujawar <Sami.Mujawar@arm.com>, devel@edk2.groups.io > <devel@edk2.groups.io>, Samer El-Haj-Mahmoud > <Samer.El-Haj-Mahmoud@arm.com>, Ard Biesheuvel <Ard.Biesheuvel@arm.com>, > leif@nuviainc.com <leif@nuviainc.com> > *Cc: *rfc@edk2.groups.io <rfc@edk2.groups.io>, Jiewen Yao > <jiewen.yao@intel.com>, Rahul Kumar <rahul1.kumar@intel.com>, nd > <nd@arm.com>, Jose Marinho <Jose.Marinho@arm.com> > *Subject: *Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to > RngDxe > > Hi Sami, > > I was wondering if you're still collecting feedback on the design, or if > you have a plan and schedule for the implementation? > > -- > Rebecca Cran > > On 1/15/21 7:51 PM, Sami Mujawar wrote: > > Hi All, > > > > I have shared some initial thoughts on the RNG implementation updates > at > https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fedk2.groups.io%2Fg%2Fdevel%2Ffiles%2FDesigns%2F2021%2F0116%2FEDKII%2520-%2520Proposed%2520update%2520to%2520RNG%2520implementation.pdf&data=04%7C01%7Cbret.barkelew%40microsoft.com%7C676a9101f67845dbdc8908d908fa4cd1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637550693569385394%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=Q8ka83ReO2aG8yTVrgpTAVxczJVjl2JBH3ksHo2%2BSHk%3D&reserved=0 > <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fedk2.groups.io%2Fg%2Fdevel%2Ffiles%2FDesigns%2F2021%2F0116%2FEDKII%2520-%2520Proposed%2520update%2520to%2520RNG%2520implementation.pdf&data=04%7C01%7Cbret.barkelew%40microsoft.com%7C676a9101f67845dbdc8908d908fa4cd1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637550693569385394%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=Q8ka83ReO2aG8yTVrgpTAVxczJVjl2JBH3ksHo2%2BSHk%3D&reserved=0> > > > > Kindly let me know your feedback or if you have any queries. > > > > Regards, > > > > Sami Mujawar > > > > -----Original Message----- > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of > Rebecca Cran via groups.io > > Sent: 14 January 2021 09:05 PM > > To: Sami Mujawar <Sami.Mujawar@arm.com>; devel@edk2.groups.io; Samer > El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>; Ard Biesheuvel > <Ard.Biesheuvel@arm.com>; leif@nuviainc.com > > Cc: rfc@edk2.groups.io; Jiewen Yao <jiewen.yao@intel.com>; Rahul > Kumar <rahul1.kumar@intel.com>; nd <nd@arm.com> > > Subject: Re: [edk2-devel] RFC: Adding support for ARM (RNDR etc.) to > RngDxe > > > > On 12/10/20 4:26 AM, Sami Mujawar wrote: > > > >> I am working on the TRNG FW API interface and will share more details > >> for the discussion soon. > >> > >> We had some thoughts about streamlining the RngDxe implementations and > >> would like to share some diagrams for the discussion. > >> > >> My diagrams are in Visio that I can export as JPG images. However, I am > >> open to switching to any other suggested tool. > > > > Hi Sami, > > > > I don't see any further discussions on this. Have you made any progress > > with sharing the design documents or scheduling a review? > > > [-- Attachment #2: Type: text/html, Size: 12284 bytes --] ^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2021-04-27 13:22 UTC | newest] Thread overview: 15+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2020-12-09 4:32 RFC: Adding support for ARM (RNDR etc.) to RngDxe Rebecca Cran 2020-12-09 4:47 ` [edk2-devel] " Samer El-Haj-Mahmoud 2020-12-10 11:26 ` Sami Mujawar 2020-12-10 12:52 ` Leif Lindholm 2020-12-10 13:43 ` [edk2-rfc] " Yao, Jiewen 2021-01-14 21:05 ` Rebecca Cran 2021-01-16 2:51 ` Sami Mujawar 2021-02-10 22:49 ` Rebecca Cran 2021-02-11 18:26 ` Ard Biesheuvel 2021-04-20 20:04 ` Rebecca Cran 2021-04-22 9:30 ` Sami Mujawar 2021-04-26 14:45 ` Rebecca Cran 2021-04-26 21:28 ` Rebecca Cran 2021-04-26 21:45 ` [EXTERNAL] " Bret Barkelew 2021-04-27 13:22 ` Sami Mujawar
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox