Re: [PATCH 2/2] Maintainers.txt: Add reviewers for Confidential Computing related modules

["Lendacky, Thomas" <thomas.lendacky@amd.com>]

On 3/10/21 8:20 AM, Laszlo Ersek wrote:
> Adding James, Brijesh, Tom; comments below:
> 
> On 03/10/21 03:55, Min Xu wrote:
>> Register reviewers for the Confidential Computing related modules in
>> OvmfPkg.
>>
>> Cc: Andrew Fish <afish@apple.com>
>> Cc: Laszlo Ersek <lersek@redhat.com>
>> Cc: Leif Lindholm <leif@nuviainc.com>
>> Cc: Michael D Kinney <michael.d.kinney@intel.com>
>> Cc: Jiewen Yao <jiewen.yao@intel.com>
>> Signed-off-by: Min Xu <min.m.xu@intel.com>
>> ---

...

> 
> So... can we simply rename the current SEV subsystem to "Confidential
> Computing", and keep both TDX and SEV modules under it? We could place a
> unified email address list there, with Brijesh, James, Jiewen, Min, Tom.
> 
> I don't think this should cause any confusion, because:
> 
> - @intel.com emails are clearly closely associated with TDX, and
> @amd.com emails are clearly closely associated with SEV,
> 
> - most filenames will (or do already) include "AmdSev" or "Tdx",
> 
> - future patches should clearly label themselves as "SEV only", "TDX
> only", or "confidential computing in general" -- this should be clear
> from the patch subjects.
> 
> IOW, there should be no confusion as to who's required to review what,
> but at the same time we'd have a simple solution for cross-posting all
> interested parties.
> 
> Thoughts?

Works for me.

Thanks,
Tom

> 
> Thanks,
> Laszlo
>