Re: [edk2-devel] [PATCH 1/2] Mde Pkg: Support for MPAM ACPI Table

2022-10-03 Thread Sami Mujawar
Hi Swatisri, Apologies for the delay in feedback. There is a new version of "ACPI for Memory System Resource Partitioning and Monitoring 2.0, Platform Design Document" published on 2022/Sep/30 at https://developer.arm.com/documentation/den0065/latest. The version 2.0 of ACPI MPAM table has a

Re: [edk2-devel] [PATCH 1/2] Mde Pkg: Support for MPAM ACPI Table

2022-08-25 Thread Hesham Almatary via groups.io
lf Of hesham.almatary via groups.io Sent: 24 August 2022 15:39 To: Andrew Fish ; devel@edk2.groups.io Subject: Re: [edk2-devel] [PATCH 1/2] Mde Pkg: Support for MPAM ACPI Table Hello Rohit and Swatisri, On Tue, Aug 23, 2022 at 09:11 PM, Andrew Fish wrote: [Rohit ] Shouldn't " Locator " field

Re: [edk2-devel] [PATCH 1/2] Mde Pkg: Support for MPAM ACPI Table

2022-08-24 Thread Rohit Mathew
; Let me know your thoughts on the approach. Regards, Rohit From: devel@edk2.groups.io On Behalf Of hesham.almatary via groups.io Sent: 24 August 2022 15:39 To: Andrew Fish ; devel@edk2.groups.io Subject: Re: [edk2-devel] [PATCH 1/2] Mde Pkg: Support for MPAM ACPI Table Hello Rohit and

Re: [edk2-devel] [PATCH 1/2] Mde Pkg: Support for MPAM ACPI Table

2022-08-24 Thread hesham.almatary via groups.io
Hello Rohit and Swatisri, On Tue, Aug 23, 2022 at 09:11 PM, Andrew Fish wrote: > > [Rohit ] Shouldn't " Locator " field be 12 bytes in size, possibly a > separate type? (MPAM ACPI 1.0, section 2.2, table 7 - Resource node and > section 2.3.2 table 10 - locator descriptor) I'd just go for UINT64

Re: [edk2-devel] [PATCH 1/2] Mde Pkg: Support for MPAM ACPI Table

2022-08-24 Thread Rohit Mathew
k2.groups.io On Behalf Of Name > > > > > via groups.io > > > > > Sent: 16 August 2022 21:18 > > > > > To: devel@edk2.groups.io; Sami Mujawar ; > > > > > Alexei Fedorov ; > > > > > michael.d.kin...@inte

Re: [edk2-devel] [PATCH 1/2] Mde Pkg: Support for MPAM ACPI Table

2022-08-23 Thread Andrew Fish via groups.io
.cn <mailto:gaolim...@byosoft.com.cn>; > zhiguang@intel.com <mailto:zhiguang@intel.com>; Swatisri Kantamsetti > mailto:swatis...@nvidia.com>>; Thomas Abraham > mailto:thomas.abra...@arm.com>>; Thanu Rangarajan > mailto:thanu.rangara...@arm.com>>;

Re: [edk2-devel] [PATCH 1/2] Mde Pkg: Support for MPAM ACPI Table

2022-08-23 Thread Rohit Mathew
: [edk2-devel] [PATCH 1/2] Mde Pkg: Support for MPAM ACPI Table On Aug 19, 2022, at 1:26 AM, Rohit Mathew mailto:rohit.mat...@arm.com>> wrote: Hi Swatisri, Thanks for the patch. Please find my comments inline marked [Rohit] - -Original Message- From: devel@edk2.groups.io<mai

Re: [edk2-devel] [PATCH 1/2] Mde Pkg: Support for MPAM ACPI Table

2022-08-23 Thread Andrew Fish via groups.io
fedo...@arm.com>>; >> michael.d.kin...@intel.com <mailto:michael.d.kin...@intel.com>; >> gaolim...@byosoft.com.cn <mailto:gaolim...@byosoft.com.cn>; >> zhiguang@intel.com <mailto:zhiguang....@intel.com> >> Cc: Swatisri Kantamsetti mailto:swatis...@n

Re: [edk2-devel] [PATCH 1/2] Mde Pkg: Support for MPAM ACPI Table

2022-08-23 Thread Swatisri Kantamsetti via groups.io
Just a reminder to provide feedback on this patch. Thanks, Swati swatis...@nvidia.com From: Name Sent: Tuesday, August 16, 2022 2:18 PM To: devel@edk2.groups.io ; sami.muja...@arm.com ; alexei.fedo...@arm.com ; michael.d.kin...@intel.com ; gaolim...@byosoft.co

Re: [edk2-devel] [PATCH 1/2] Mde Pkg: Support for MPAM ACPI Table

2022-08-19 Thread Rohit Mathew
d.kin...@intel.com; > gaolim...@byosoft.com.cn; zhiguang@intel.com > Cc: Swatisri Kantamsetti > Subject: [edk2-devel] [PATCH 1/2] Mde Pkg: Support for MPAM ACPI Table > > From: Swatisri Kantamsetti > > Added MPAM table header, MSC and Resource Node info structures &

[edk2-devel] [PATCH 1/2] Mde Pkg: Support for MPAM ACPI Table

2022-08-16 Thread Name via groups.io
From: Swatisri Kantamsetti Added MPAM table header, MSC and Resource Node info structures Signed-off-by: Swatisri Kantamsetti --- MdePkg/Include/IndustryStandard/Acpi64.h | 5 ++ MdePkg/Include/IndustryStandard/Mpam.h | 69 2 files changed, 74 insertions(+) create