public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Rohit Mathew" <rohit.mathew@arm.com>
To: Sami Mujawar <Sami.Mujawar@arm.com>,
	"devel@edk2.groups.io" <devel@edk2.groups.io>
Cc: Michael D Kinney <michael.d.kinney@intel.com>,
	Liming Gao <gaoliming@byosoft.com.cn>,
	Thomas Abraham <thomas.abraham@arm.com>,
	James Morse <James.Morse@arm.com>, Ray Ni <ray.ni@intel.com>,
	Zhichao Gao <zhichao.gao@intel.com>, nd <nd@arm.com>
Subject: Re: [edk2-devel] [PATCH V6 0/6] MPAM ACPI definitions and parser
Date: Mon, 15 Jul 2024 07:56:43 +0000	[thread overview]
Message-ID: <AM6PR08MB37833AA8E699B6FAF641DEB98FA12@AM6PR08MB3783.eurprd08.prod.outlook.com> (raw)
In-Reply-To: <34908D55-A3BE-405F-8472-9393E9DB2EA9@arm.com>

Hi Sami,

I have added these as well.

Regards,
Rohit

> -----Original Message-----
> From: Sami Mujawar <Sami.Mujawar@arm.com>
> Sent: Tuesday, July 9, 2024 9:11 AM
> To: Rohit Mathew <Rohit.Mathew@arm.com>; devel@edk2.groups.io
> Cc: Michael D Kinney <michael.d.kinney@intel.com>; Liming Gao
> <gaoliming@byosoft.com.cn>; Thomas Abraham
> <thomas.abraham@arm.com>; James Morse <James.Morse@arm.com>; Ray
> Ni <ray.ni@intel.com>; Zhichao Gao <zhichao.gao@intel.com>; nd
> <nd@arm.com>
> Subject: Re: [PATCH V6 0/6] MPAM ACPI definitions and parser
> 
> Hi Rohit,
> 
> I think the pull request template must be used. In addition to the patch series
> description you have added, it has questions like if the change is a breaking
> change, how was the patch tested etc.
> It would be good if you can update the pull request to use the PR template.
> 
> Regards,
> 
> Sami Mujawar
> 
> On 09/07/2024, 09:03, "Rohit Mathew" <Rohit.Mathew@arm.com
> <mailto:Rohit.Mathew@arm.com>> wrote:
> 
> 
> Thank you, Sami, for letting me know. I had created a PR as well for making
> sure the CI goes through here - https://github.com/tianocore/edk2/pull/5888
> <https://github.com/tianocore/edk2/pull/5888>
> 
> 
> Could you please let me know if this would be sufficient?
> 
> 
> Regards,
> Rohit
> 
> 
> > -----Original Message-----
> > From: Sami Mujawar <Sami.Mujawar@arm.com
> > <mailto:Sami.Mujawar@arm.com>>
> > Sent: Tuesday, July 9, 2024 8:16 AM
> > To: Rohit Mathew <Rohit.Mathew@arm.com
> <mailto:Rohit.Mathew@arm.com>>;
> > devel@edk2.groups.io <mailto:devel@edk2.groups.io>
> > Cc: Michael D Kinney <michael.d.kinney@intel.com
> > <mailto:michael.d.kinney@intel.com>>; Liming Gao
> > <gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn>>; Thomas
> > Abraham <thomas.abraham@arm.com
> <mailto:thomas.abraham@arm.com>>;
> > James Morse <James.Morse@arm.com <mailto:James.Morse@arm.com>>;
> Ray Ni
> > <ray.ni@intel.com <mailto:ray.ni@intel.com>>; Zhichao Gao
> > <zhichao.gao@intel.com <mailto:zhichao.gao@intel.com>>; nd
> <nd@arm.com
> > <mailto:nd@arm.com>>
> > Subject: Re: [PATCH V6 0/6] MPAM ACPI definitions and parser
> >
> > Hi Rohit,
> >
> > Edk2 has moved to a pull request model for code review and merge.
> > See the latest guidelines at
> > https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Developme
> > nt-
> > <https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Developm
> > ent->
> > Process
> > I replied to your previous patch series on the mailing list as it was
> > sent before the new process was introduced.
> > You will need to create a PR for the new series to progress.
> >
> > Regards,
> >
> > Sami Mujawar
> >
> > On 08/07/2024, 22:35, "Rohit Mathew" <rohit.mathew@arm.com
> > <mailto:rohit.mathew@arm.com> <mailto:rohit.mathew@arm.com
> <mailto:rohit.mathew@arm.com>>> wrote:
> >
> >
> > This series adds the following
> >
> >
> > * definitions corresponding to MPAM ACPI 2.0 specification.
> > * MPAM parser An MPAM
> >
> >
> > ACPI table formulated using the newly added MPAM ACPI definitions were
> > validated on the linux kernel tree at [1]. The same table was parsed
> > via acpiview using the newly added parser. Certain aspects of the MPAM
> > ACPI specification are still not implemented by the kernel tree. These
> > aspects were verified only using acpiview.
> >
> >
> > Changes since V1:
> > Addressed comments on MPAM ACPI definitions from Sami.
> > V1 did not incorporate the parser. V2 has this implemented.
> >
> >
> > Changes since V2:
> > Addressed comments related to Interface/Link type defintions from Sami.
> >
> >
> > Changes since V3:
> > Addressed comments from Pierre
> >
> >
> > Changes since V4:
> > Addressed comments from Sami regarding Parser.
> >
> >
> > Changes since V5:
> > Addressed comments from Sami regarding Parser.
> >
> >
> > Changes can be found at:
> > https://github.com/rohit-arm/edk2/tree/mpam_acpi_v6
> > <https://github.com/rohit-arm/edk2/tree/mpam_acpi_v6>
> > <https://github.com/rohit-arm/edk2/tree/mpam_acpi_v6>
> > <https://github.com/rohit-arm/edk2/tree/mpam_acpi_v6&gt;>
> >
> >
> > CI results/PR can be found at:
> > https://github.com/tianocore/edk2/pull/5888
> > <https://github.com/tianocore/edk2/pull/5888>
> > <https://github.com/tianocore/edk2/pull/5888>
> > <https://github.com/tianocore/edk2/pull/5888&gt;>
> >
> >
> > [1]
> > https://git.kernel.org/pub/scm/linux/kernel/git/morse/linux.git/log/?h
> > =mpam
> > <https://git.kernel.org/pub/scm/linux/kernel/git/morse/linux.git/log/?
> > h=mpam>
> > /snapshot/v6.7-rc2
> > <https://git.kernel.org/pub/scm/linux/kernel/git/morse/linux.git/log/?
> > h=mpa
> > <https://git.kernel.org/pub/scm/linux/kernel/git/morse/linux.git/log/?
> > h=mpa>
> > m/snapshot/v6.7-rc2>
> >
> >
> > Rohit Mathew (6):
> > MdePkg/IndustryStandard: Add definitions for MPAM ACPI specification
> > ShellPkg/AcpiView: Update field-validator prototype
> > ShellPkg/AcpiView: Update print-formatter prototype
> > ShellPkg: acpiview: Add routine to print 16 chars
> > ShellPkg: acpiview: Add routines to print reserved fields
> > ShellPkg/AcpiView: Add MPAM Parser
> >
> >
> > MdePkg/Include/IndustryStandard/Acpi65.h | 7 +-
> > MdePkg/Include/IndustryStandard/Mpam.h | 246 ++++
> > ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.c | 207 +++-
> > ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.h | 113 +-
> > ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Aest/AestParser.c
> > |
> > 36 +-
> > ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Dbg2/Dbg2Parser.c
> > |
> > 8 +-
> > ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Erst/ErstParser.c
> > | 28 +-
> > ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Fadt/FadtParser.c
> > |
> > 24 +-
> > ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Gtdt/GtdtParser.c
> > |
> > 14 +-
> >
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Hmat/HmatParser.c
> > |
> > 12 +-
> > ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Hpet/HpetParser.c
> > |
> > 19 +-
> > ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Iort/IortParser.c
> > | 32
> > +-
> > ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Madt/MadtParser.c
> > |
> > 24 +-
> >
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Mpam/MpamParser.
> > c | 1241 ++++++++++++++++++++
> > ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Pcct/PcctParser.c
> > |
> > 56 +-
> > ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Pptt/PpttParser.c
> > |
> > 38 +-
> > ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Rsdp/RsdpParser.c
> > |
> > 14 +-
> > ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Spcr/SpcrParser.c
> > |
> > 14 +-
> > ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Srat/SratParser.c
> > | 26
> > +-
> >
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Wsmt/WsmtParser.c
> > | 20 +-
> >
> ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommand
> > Lib.c | 3 +-
> >
> ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommand
> > Lib.inf | 3 +-
> >
> ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommand
> > Lib.uni | 3 +-
> > 23 files changed, 2040 insertions(+), 148 deletions(-) create mode
> > 100644 MdePkg/Include/IndustryStandard/Mpam.h
> > create mode 100644
> >
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Mpam/MpamParser.
> > c
> >
> >
> > --
> > 2.34.1
> >
> >
> >
> >
> >
> 
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#119932): https://edk2.groups.io/g/devel/message/119932
Mute This Topic: https://groups.io/mt/107112126/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



  reply	other threads:[~2024-07-15  7:57 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-07-08 21:35 [edk2-devel] [PATCH V6 0/6] MPAM ACPI definitions and parser Rohit Mathew
2024-07-09  7:15 ` Sami Mujawar
2024-07-09  8:03   ` Rohit Mathew
2024-07-09  8:10     ` Sami Mujawar
2024-07-15  7:56       ` Rohit Mathew [this message]
     [not found]       ` <17E253DECB9ED3EF.4882@groups.io>
2024-07-29 19:50         ` Rohit Mathew

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=AM6PR08MB37833AA8E699B6FAF641DEB98FA12@AM6PR08MB3783.eurprd08.prod.outlook.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox