public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* 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&amp;data=04%7C01%7Cbret.barkelew%40microsoft.com%7C676a9101f67845dbdc8908d908fa4cd1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637550693569385394%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=Q8ka83ReO2aG8yTVrgpTAVxczJVjl2JBH3ksHo2%2BSHk%3D&amp;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&amp;data=04%7C01%7Cbret.barkelew%40microsoft.com%7C676a9101f67845dbdc8908d908fa4cd1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637550693569385394%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=Q8ka83ReO2aG8yTVrgpTAVxczJVjl2JBH3ksHo2%2BSHk%3D&amp;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&amp;data=04%7C01%7Cbret.barkelew%40microsoft.com%7C676a9101f67845dbdc8908d908fa4cd1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637550693569385394%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=Q8ka83ReO2aG8yTVrgpTAVxczJVjl2JBH3ksHo2%2BSHk%3D&amp;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&amp;data=04%7C01%7Cbret.barkelew%40microsoft.com%7C676a9101f67845dbdc8908d908fa4cd1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637550693569385394%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=Q8ka83ReO2aG8yTVrgpTAVxczJVjl2JBH3ksHo2%2BSHk%3D&amp;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&amp;data=04%7C01%7Cbret.barkelew%40microsoft.com%7C676a9101f67845dbdc8908d908fa4cd1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637550693569385394%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=Q8ka83ReO2aG8yTVrgpTAVxczJVjl2JBH3ksHo2%2BSHk%3D&amp;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&amp;data=04%7C01%7Cbret.barkelew%40microsoft.com%7C676a9101f67845dbdc8908d908fa4cd1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637550693569385394%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=Q8ka83ReO2aG8yTVrgpTAVxczJVjl2JBH3ksHo2%2BSHk%3D&amp;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&amp;data=04%7C01%7Cbret.barkelew%40microsoft.com%7C676a9101f67845dbdc8908d908fa4cd1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637550693569385394%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=Q8ka83ReO2aG8yTVrgpTAVxczJVjl2JBH3ksHo2%2BSHk%3D&amp;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&amp;data=04%7C01%7Cbret.barkelew%40microsoft.com%7C676a9101f67845dbdc8908d908fa4cd1%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637550693569385394%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=Q8ka83ReO2aG8yTVrgpTAVxczJVjl2JBH3ksHo2%2BSHk%3D&amp;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