[edk2-devel] [PATCH 1/1] Maintainers.txt: designate Gerd Hoffmann as UefiCpuPkg reviewer

Ni, Ray ray.ni at intel.com
Wed Jan 4 14:06:12 UTC 2023


Reviewed-by: Ray Ni <ray.ni at intel.com>

> -----Original Message-----
> From: Laszlo Ersek <lersek at redhat.com>
> Sent: Wednesday, January 4, 2023 12:06 AM
> To: devel at edk2.groups.io
> Cc: Andrew Fish <afish at apple.com>; Dong, Eric <eric.dong at intel.com>; Gerd Hoffmann <kraxel at redhat.com>; Leif
> Lindholm <quic_llindhol at quicinc.com>; Kinney, Michael D <michael.d.kinney at intel.com>; Kumar, Rahul R
> <rahul.r.kumar at intel.com>; Ni, Ray <ray.ni at intel.com>
> Subject: [PATCH 1/1] Maintainers.txt: designate Gerd Hoffmann as UefiCpuPkg reviewer
> 
> I suggest that Gerd be notified about all UefiCpuPkg patches, so he may
> take a quick look at, or (by his preference) even test, the proposed
> change, in a genuine QEMU/KVM environment.
> 
> Assuming this patch is accepted -- subsequently, please *wait* for Gerd's
> approval on UefiCpuPkg patches, before merging them.
> 
> Notes:
> 
> - It's perfectly fine for a reviewer to give an A-b just so the review
>   process be unblocked, if they don't have anything to add, or don't have
>   time to review or test in detail. The point is that someone outside of
>   Intel should *consistently get a chance* to raise concerns about
>   UefiCpuPkg patches before they are merged.
> 
> - My A-b's and R-b's on UefiCpuPkg patches were never supposed to be
>   "sufficient", only "necessary", for merging. The intent is the same
>   here, with Gerd's designation as a reviewer.
> 
> Cc: Andrew Fish <afish at apple.com>
> Cc: Eric Dong <eric.dong at intel.com>
> Cc: Gerd Hoffmann <kraxel at redhat.com>
> Cc: Leif Lindholm <quic_llindhol at quicinc.com>
> Cc: Michael D Kinney <michael.d.kinney at intel.com>
> Cc: Rahul Kumar <rahul1.kumar at intel.com>
> Cc: Ray Ni <ray.ni at intel.com>
> Signed-off-by: Laszlo Ersek <lersek at redhat.com>
> ---
>  Maintainers.txt | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 13e1335a57dd..3399981bbae7 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -605,6 +605,7 @@ W: https://github.com/tianocore/tianocore.github.io/wiki/UefiCpuPkg
>  M: Eric Dong <eric.dong at intel.com> [ydong10]
>  M: Ray Ni <ray.ni at intel.com> [niruiyu]
>  R: Rahul Kumar <rahul1.kumar at intel.com> [rahul1-kumar]
> +R: Gerd Hoffmann <kraxel at redhat.com> [kraxel]
> 
>  UefiCpuPkg: Sec related modules
>  F: UefiCpuPkg/SecCore/


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#97941): https://edk2.groups.io/g/devel/message/97941
Mute This Topic: https://groups.io/mt/96030658/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-




More information about the edk2-devel-archive mailing list