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

[Brijesh Singh <brijesh.singh@amd.com>]

On 3/10/21 9:09 AM, Tom Lendacky wrote:
> 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.


Works for me as well.