From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by spool.mail.gandi.net (Postfix) with ESMTPS id CC6F5941201 for ; Tue, 9 Apr 2024 05:27:19 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=KLf0q9FuWft9ki25AVkpp3hErhlnWWWZZ81L8Bc0/pM=; c=relaxed/simple; d=groups.io; h=From:To:CC:Subject:Thread-Topic:Thread-Index:Date:Message-ID:References:In-Reply-To:Accept-Language:msip_labels:MIME-Version:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Resent-Date:Resent-From:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Language:Content-Type; s=20240206; t=1712640438; v=1; b=Qy3/A/fXuFO2VQtzVoCSDBJOAgF6XGTmt2LVUKhG9nCOWNqk/ETpwblu15jxVdu4fubrxwYU xJXgZu+1f1N1o5GYDcxZ7OjGdSFuxfb1U9PTZ8KS4W/JW0AX9BbW/OqD1OHZrCMkt8tY9TZnxJS PXyERwQi175J5tnahQ7BAuHztE01QNwPyVaHOeco9wDx2z4kS2Hy+PWHRfobrSK6Eg+B0aYifa6 HA0v4NMEsYYrKHEZQFqux8xJHZrIbjbw82suKel7h21EP2x0e3VZ//rbNbjR9Xjgr7DwGJvAPb2 Zju0PbSV77d6axgiX0CU6Cc0Rl43IHXgarskvqxxvPtBg== X-Received: by 127.0.0.2 with SMTP id ktHeYY7687511xtQwSp7OR53; Mon, 08 Apr 2024 22:27:18 -0700 X-Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.8]) by mx.groups.io with SMTP id smtpd.web11.129546.1712640437618264339 for ; Mon, 08 Apr 2024 22:27:17 -0700 X-CSE-ConnectionGUID: 4oC/ZHL0RqGeAaK9WQdfUQ== X-CSE-MsgGUID: JiEQJJxURrCBx4aGiq1lmw== X-IronPort-AV: E=McAfee;i="6600,9927,11038"; a="25442753" X-IronPort-AV: E=Sophos;i="6.07,188,1708416000"; d="scan'208,217";a="25442753" X-Received: from orviesa006.jf.intel.com ([10.64.159.146]) by fmvoesa102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 08 Apr 2024 22:27:16 -0700 X-CSE-ConnectionGUID: mNE21BOTQsinlufw0ROp1w== X-CSE-MsgGUID: cU9+BEhhSdWSyRAAPDKg+g== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.07,188,1708416000"; d="scan'208,217";a="20562137" X-Received: from orsmsx603.amr.corp.intel.com ([10.22.229.16]) by orviesa006.jf.intel.com with ESMTP/TLS/AES256-GCM-SHA384; 08 Apr 2024 22:27:16 -0700 X-Received: from orsmsx612.amr.corp.intel.com (10.22.229.25) by ORSMSX603.amr.corp.intel.com (10.22.229.16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Mon, 8 Apr 2024 22:27:15 -0700 X-Received: from orsmsx603.amr.corp.intel.com (10.22.229.16) by ORSMSX612.amr.corp.intel.com (10.22.229.25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Mon, 8 Apr 2024 22:27:15 -0700 X-Received: from orsedg603.ED.cps.intel.com (10.7.248.4) by orsmsx603.amr.corp.intel.com (10.22.229.16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35 via Frontend Transport; Mon, 8 Apr 2024 22:27:15 -0700 X-Received: from NAM10-MW2-obe.outbound.protection.outlook.com (104.47.55.101) by edgegateway.intel.com (134.134.137.100) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.35; Mon, 8 Apr 2024 22:27:15 -0700 X-Received: from DM4PR11MB8226.namprd11.prod.outlook.com (2603:10b6:8:182::12) by DS7PR11MB6013.namprd11.prod.outlook.com (2603:10b6:8:70::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7452.26; Tue, 9 Apr 2024 05:27:13 +0000 X-Received: from DM4PR11MB8226.namprd11.prod.outlook.com ([fe80::4b2d:823c:e71a:541e]) by DM4PR11MB8226.namprd11.prod.outlook.com ([fe80::4b2d:823c:e71a:541e%6]) with mapi id 15.20.7430.045; Tue, 9 Apr 2024 05:27:07 +0000 From: "Ni, Ray" To: Chao Li , "devel@edk2.groups.io" , Gerd Hoffmann CC: "Kumar, Rahul R" , Sami Mujawar , Sunil V L , Bibo Mao , Dongyan Qian Subject: Re: [edk2-devel] [PATCH v2 00/13] Part 2 patch set to add LoongArch support into UefiCpuPkg Thread-Topic: [edk2-devel] [PATCH v2 00/13] Part 2 patch set to add LoongArch support into UefiCpuPkg Thread-Index: AQHaeqKOwBdqqtDKIUqwU86L2eCtHLFDts8AgAQOACOABjb/hIARUVf0gAAqmACAAA6Jlw== Date: Tue, 9 Apr 2024 05:27:07 +0000 Message-ID: References: <20240320084152.268323-1-lichao@loongson.cn> <17BFE34457098413.21233@groups.io> <17C02C7EE39BF604.20354@groups.io> <9972bb2c-0790-4879-89f3-946574f0d802@loongson.cn> <1ed0abe7-0fd7-400c-81b9-d0f4609901a8@loongson.cn> In-Reply-To: <1ed0abe7-0fd7-400c-81b9-d0f4609901a8@loongson.cn> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: msip_labels: x-ms-publictraffictype: Email x-ms-traffictypediagnostic: DM4PR11MB8226:EE_|DS7PR11MB6013:EE_ x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam-message-info: IivWwZN++qwa1eZxRjwW8qRFh89uwUaG8eeE7KYAicCn6O79ZF2jx05DEOo42TrX154xAPqgv0cp1IHwBnKd0uyt2PPjinvvq2vkC9pMw1w0tIipQ3RfWmFVhV+s/v3iB90W85WPQ9xq1hkyFqQdOqwc5084f6ybXHpXxSTUUmLkEFDgf0++FC5KMHIrKn2H5yYaJzb+U0SqWchmGIGm21LZ21JK9ZjF3CRc6zQ7NN81p7eGX7OoEuS7SkcyJEQvsSlL/5MP7EW0rkT6XL+NTXZSzQvHrxci9+P+GDXh7fZUOoxY8eNfu0QBoww/UXnC2lUs/Xa37xoxkeqQqOrIzMM3EV85dm9SOmFCEDc2FUteq7BvaCqzOiWNC0Ms9qP3YuJPXVVe1GKL1/DzfZQD2UlLZBrxrULrmeOa7z1LGePqNY5zs9vqHQ3POTOMmRuFfla4Yi02QTwINnIhvzTrjpF5w7/sJ+I2bOppg2QqWumORkEX5nMAUzGwwLBw2MR7sWCVVf0L/TUjUPWSMVic5teLOKxe1rroJUJuU2R1UDkOn3fZ5d4FDprBu4LuAyHWCPrumiB0TNlELqmAcf44nYpoPLkaQQ2AUrq+fOb0Q0B64t5cDDBKJhEOmfQR1v4q x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?rZbe741ZvbmQEiI0bVgCIqXLol1Llbw2xfnru1Yfg8GCq8+njz0X8NEjLT?= =?iso-8859-1?Q?0hc5evBuMYXJNrqG2IUcGJRpkf3a4RDjm2sRtXt30ZxalA7anUp9R4KaWZ?= =?iso-8859-1?Q?0rG3wCJLMgZCVrKdTG0jFuhK9xKKcdY1j7o3Qz/hizuzwQbYw0o6t7oGHF?= =?iso-8859-1?Q?GZEHrpxDVjLbHdYhqOX3QSVY9XhSXyXjrzPctweWlNVYorqgTOwn0SvKEK?= =?iso-8859-1?Q?omsxzzIzph67NjlGpty63/BnV3U0aI+jfZBLNn9BFRJHSZ9IGyVHgqFcxN?= =?iso-8859-1?Q?5pbgjTzx9p0mwfETrTXRXC3j6y+AughkUeLhNrUxbMw0HzdkTA0ENsxrRo?= =?iso-8859-1?Q?RXFELqFSu6ftGF+96qZiDiQ52OA+NN6N5baFKCZwmESbkeJybFvdu2NIuT?= =?iso-8859-1?Q?2iYFOqr9xdL5Kz1rqxtJhQ8TFhyHdbVI+rAotFynBR0uEHWpdgF7zu/8TG?= =?iso-8859-1?Q?pCx88A3D0t7JOuy6MHyl3TN0uPFIhGJgHe+zu72kHFZdAORENe0pDwnUL+?= =?iso-8859-1?Q?FDu3LjlfhzC7ZIts2eLiPnzKY1RCx62uBecew9dyk6eELW1maO5HBublv7?= =?iso-8859-1?Q?pUSsDnyWVEPskzV0vYhGrneCnglipUIKG17AW4J8CptcQZM3qH2Rx57gqo?= =?iso-8859-1?Q?mtZ78tALfR2NVTb2wwPVcxZoyqtmnAoTrbRWMhbK0DxmnTYZ+WWVsEOBl9?= =?iso-8859-1?Q?N4NHI/MWyElBC/yQw8eapCICBqKUTtd4QhWhXT4PnCkrQxS9eVGgB7Go4W?= =?iso-8859-1?Q?sJfQH8dZvaEepmNzuWcEQnK5e6sKbgtoGt9sxt8mSE3vYvQlmiqKo1zQcM?= =?iso-8859-1?Q?H5VVOfxaz67CgnXGX3X6DnP/bSUyEoOlXmUivzommfD/vJR9vCU+kyhV7v?= =?iso-8859-1?Q?TlbSTgHeE7xBR9IA/6Enky06pxrbTp9uASLM63FIEUj9L5LUSk67DUa0Az?= =?iso-8859-1?Q?ndDK6TgdUMDoJXx2sjKf6KBKlTiZW/VyxZKxo/bfRBb6kbPdKKM3ZucCGS?= =?iso-8859-1?Q?G8+77b6rQcm7mHTPs0Ngi2lBkcDnWOQhdXm3YAtWL6RVJveR7TJq2RAYAN?= =?iso-8859-1?Q?A+nVUDvjVcT8O7UWmKY5cJc9XRVdQTMfildq45oNwwDCGhmvqk9jLlxXDT?= =?iso-8859-1?Q?BHwhQnpuUJDDhOWrcvL+ok/wUm592B2sjdwSprhHxQiGEJwAAwqPoRdqLm?= =?iso-8859-1?Q?EoJsIMwQCCro0mWl6tPZzP9zFsfMZKMzLnuGN/YSryaq/e//LimvFIRBLS?= =?iso-8859-1?Q?yR2Wxj2+yMz/xSPZqXXnSD8xSJnPw0VV4V04xQga3sw7KD1FOaW1M22+w5?= =?iso-8859-1?Q?3tk1h7TN3XfV4Fr3L9xbjnR+RR3HKoUUdjF/f6gokUsk3XZAW120uKbuLF?= =?iso-8859-1?Q?AamqA4g+48k1xi4T66h5QnGgHUYxHaFTtfsFYcdvGBHkQlgBSL9ninD429?= =?iso-8859-1?Q?I5f601UpDVKGQuy1boMsMXOM0P5TzWiveflRIXtpMsCGaVHgFhQoo3c/GN?= =?iso-8859-1?Q?ZktnYs+aNRag6AXXHYyB9hA/EmyYmbiuNZ91B8geMmpioZZ/B/B3WAyNQV?= =?iso-8859-1?Q?0uR6DszNnoh41ozlQAsmy4hATBeH/b3qq9m2Q6olvzPtHpo2UBwGSo4B69?= =?iso-8859-1?Q?Tu2MqmCKc3sPI=3D?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM4PR11MB8226.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 11db2cfd-316a-4ee6-7813-08dc5855b2d1 X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Apr 2024 05:27:07.5236 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: PG0WFiMil+a3RtfqV6OiGYHsoNHUlm4WlaqRk4lyjMZmL3cSp282HCNd1RRHkbO2k4rvDd6jW7WlVwk3vCMCZQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR11MB6013 X-OriginatorOrg: intel.com Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Resent-Date: Mon, 08 Apr 2024 22:27:17 -0700 Resent-From: ray.ni@intel.com Reply-To: devel@edk2.groups.io,ray.ni@intel.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: tQaef6EFhnpQ4W0MKLhTX6Yix7686176AA= Content-Language: en-US Content-Type: multipart/alternative; boundary="_000_DM4PR11MB82265F4A30E7BD3AC638DFF88C072DM4PR11MB8226namp_" X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20240206 header.b="Qy3/A/fX"; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=intel.com (policy=none); spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce@groups.io --_000_DM4PR11MB82265F4A30E7BD3AC638DFF88C072DM4PR11MB8226namp_ Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Chao, Current patch introduces the CpuMmuInitLib which contains ConfigureMemoryMa= nagementUnit () API. You told me the API will be called by a PEIM. Then, the new proposal is to move the library code into that PEIM. I don't quite understand your needs of the new GUID to store the memory map= resource. How is the GUID used to store the memory map resource? Can the PEIM be placed in edk2-platforms repo? Thanks, Ray ________________________________ From: Chao Li Sent: Tuesday, April 9, 2024 12:29 To: devel@edk2.groups.io ; Ni, Ray = ; Gerd Hoffmann Cc: Kumar, Rahul R ; Sami Mujawar ; Sunil V L ; Bibo Mao ; Dongyan Qian Subject: Re: [edk2-devel] [PATCH v2 00/13] Part 2 patch set to add LoongArc= h support into UefiCpuPkg Hi Ray, I'm willing change it to a PEIM if it doesn't fit as a library. I think if = it is a PEIM, we need a new GUID to sotre the memory map resouce, or use an= already defined GUID. I will put it under the UefiCpuPkg, called CpuMmuInitPei, folder: UefiCpuPk= g/CpuMmuInitPei/LoongArch64/. May I? Thanks, Chao On 2024/4/9 10:06, Ni, Ray wrote: Chao, Sorry I missed your mail. If ConfigureMemoryManagementUnit() is called in PEI, can you move the logic= to a LoongArch specific PEIM? My concern is we may need more review on the= lib API ConfigureMemoryManagementUnit() if we position it as a library. If we move the logic in a PEIM and the implementation becomes a PEIM intern= al logic, we can lower the quality expectation of the function prototype as= no other module is able to call it. Thanks, Ray For patches 10, 11: Can the lib be avoided if the logic is implemented in C= puDxe driver? This library is will be called in the PEI stage, so I can't move it under t= he CpuDxe. This library is the low-level libary of CpuMmuLib, which will consume CpuMm= uLIb to configure the MMU. This way is suggested by Laszlo, who saied if CpuMmuLib can not content the= configure API(high-level libary is the basecal libaray, it should not incl= ude the configure API), we can split it into two, where the hight-livel is = CpuMmuLib, and the low-level is CpuMmuInitLib. For patch 12(UefiCpuPkg: Add multiprocessor library for LoongArch64): Revie= wed-by: Ray Ni For patch 13: Please make accordingly changes when you address comments for= patch 8. OK. Thanks, Ray ________________________________ From: Gerd Hoffmann Sent: Friday, March 22, 2024 20:39 To: Chao Li Cc: devel@edk2.groups.io ; Ni, Ray ; Kumar, Rahul R ; Sami Mujawar ; = Sunil V L ; Bibo= Mao ; Dongyan Qian Subject: Re: [PATCH v2 00/13] Part 2 patch set to add LoongArch support int= o UefiCpuPkg On Wed, Mar 20, 2024 at 04:41:52PM +0800, Chao Li wrote: > This patch set adjusted some order in UefiCpuPig alphabetically, added > LoongArch libraries and drivers into UefiCpuPkg, it is a continuation of > the first patch series v8 submitted at > https://edk2.groups.io/g/devel/message/114526. > > And also separated from https://edk2.groups.io/g/devel/message/116583. > > This series only contents the changes for UefiCpuPkg. > > Patch1-Patch4: Reorder some INF files located in UefiCpuPkg > alphabetically. > > Patch5-Patch13: Added Timer, CpuMmuLib, CpuMmuInitLib, MpInitLib, CpuDxe > for LoongArch, and added some PCD and header files requested by the > above libraries and drivers. > > Modfied modules: UefiCpuPkg > > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3D4726 > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3D4734 > > PR: https://github.com/tianocore/edk2/pull/5483 > > V1 -> V2: > 1. Removed PcdCpuMmuIsEnabled. > 2. Removed API GetMemoryRegionAttributes API as it is no longer needed. > 3. Patch3, added two empty line in DXE and PEI INF files. > 4. Patch5, added the Status check in GetTimeInnanoSecond function. > 5. Separated into two series, this is series one, and the second one is > OvmfPkg. While I can't comment on the loongarch architecture details the code and the integration into build system looks overall sane to me. Series: Acked-by: Gerd Hoffmann take care, Gerd -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#117529): https://edk2.groups.io/g/devel/message/117529 Mute This Topic: https://groups.io/mt/105041080/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- --_000_DM4PR11MB82265F4A30E7BD3AC638DFF88C072DM4PR11MB8226namp_ Content-Type: text/html; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable
Chao, 
Current patch introduces the CpuMmuInitLib which contains ConfigureMemoryManagementUnit () API. You told me the API will be ca= lled by a PEIM.
Then, the new proposal is to move the library code into that PEIM.

I don't quite underst= and your needs of the new GUID to store the memory map resource. How is the= GUID used to store the memory map resource?
Can the PEIM be place= d in edk2-platforms repo?

Thanks,
Ray

From: Chao Li <lichao@loongson.cn>
Sent: Tuesday, April 9, 2024 12:29
To: devel@edk2.groups.io <devel@edk2.groups.io>; Ni, Ray = <ray.ni@intel.com>; Gerd Hoffmann <kraxel@redhat.com>
Cc: Kumar, Rahul R <rahul.r.kumar@intel.com>; Sami Mujawa= r <sami.mujawar@arm.com>; Sunil V L <sunilvl@ventanamicro.com>;= Bibo Mao <maobibo@loongson.cn>; Dongyan Qian <qiandongyan@loongso= n.cn>
Subject: Re: [edk2-devel] [PATCH v2 00/13] Part 2 patch set to = add LoongArch support into UefiCpuPkg
 

Hi Ray,

I'm willing change it to a PEIM if it doesn't fit as a library. I t= hink if it is a PEIM, we need a new GUID to sotre the memory map resouce, o= r use an already defined GUID.

I will put it under the UefiCpuPkg, called CpuMmuInitPei, folder: U= efiCpuPkg/CpuMmuInitPei/LoongArch64/. May I?


Thanks,
Chao
On 2024/4/9 10:06, Ni, Ray wrote:
Chao,
Sorry I missed your mail.

If ConfigureMemoryManagementUnit() is ca= lled in PEI, can you move the logic to a LoongArch specific PEIM? My concern is we may need more review on the= lib API ConfigureMemoryManagementUnit() if we position it as a library.

If = we move the logic in a PEIM and the implementation becomes a PEIM internal = logic, we can lower the quality expectation of the function prototype as no other module is able to call it.


Thanks,
Ray


For patches 10, 11: Can the lib be avoided if the logic is implemented in C= puDxe driver?

This library is will be c= alled in the PEI stage, so I can't move it under the CpuDxe.

This library is the low-l= evel libary of CpuMmuLib, which will consume CpuMmuLIb to configure the MMU= .

This way is suggested by = Laszlo, who saied if CpuMmuLib can not content the configure API(high-level= libary is the basecal libaray, it should not include the configure API), w= e can split it into two, where the hight-livel is CpuMmuLib, and the low-level is CpuMmuInitLib.


For patch 12(UefiCpuPkg: Add multiprocessor library for LoongArch64): Revie= wed-by: Ray Ni <ray.ni@intel.com>
For patch 13: Please make accordingly changes when you address comments for= patch 8.
OK.

Thanks,
Ray
From: Gerd Hoffmann <kraxel@redhat.com>
Sent: Friday, March 22, 2024 20:39
To: Chao Li <lichao@loongson.cn>
Cc: devel@edk2.groups.io <devel@edk2.groups.io>; Ni, Ray <ray.ni@intel.com>; Kumar, Rahul R <rahul.r.kumar@intel.com>; Sami Mujawar <sami.mujawar@arm.com>; Sunil V L <sunilvl@ventanamicro.com>; Bibo Mao <maobibo@loongson.cn>; Dongyan Qian <qiandongyan@loongson.cn>
Subject: Re: [PATCH v2 00/13] Part 2 patch set to add LoongArch= support into UefiCpuPkg
 
On Wed, Mar 20, 2024 at 04:41:52PM +0800, C= hao Li wrote:
> This patch set adjusted some order in UefiCpuPig alphabetically, added=
> LoongArch libraries and drivers into UefiCpuPkg, it is a continuation = of
> the first patch series v8 submitted at
> https://edk2.groups.io/g/devel/message/114526.
>
> And also separated from https://edk2.groups.io/g/devel/message/116583.
>
> This series only contents the changes for UefiCpuPkg.
>
> Patch1-Patch4: Reorder some INF files located in UefiCpuPkg
> alphabetically.
>
> Patch5-Patch13: Added Timer, CpuMmuLib, CpuMmuInitLib, MpInitLib, CpuD= xe
> for LoongArch, and added some PCD and header files requested by the > above libraries and drivers.
>
> Modfied modules: UefiCpuPkg
>
> BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3D4726
> BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3D4734
>
>
> V1 -> V2:
> 1. Removed PcdCpuMmuIsEnabled.
> 2. Removed API GetMemoryRegionAttributes API as it is no longer needed= .
> 3. Patch3, added two empty line in DXE and PEI INF files.
> 4. Patch5, added the Status check in GetTimeInnanoSecond function.
> 5. Separated into two series, this is series one, and the second one i= s
> OvmfPkg.

While I can't comment on the loongarch architecture details the code
and the integration into build system looks overall sane to me.

Series:
Acked-by: Gerd Hoffmann <kraxel@redhat.com>

take care,
  Gerd

_._,_._,_
Groups.io Links:

=20 You receive all messages sent to this group. =20 =20

View/Reply Online (#117529) | =20 | Mute= This Topic | New Topic
Your Subscriptio= n | Contact Group Owner | Unsubscribe [rebecca@openfw.io]

_._,_._,_
--_000_DM4PR11MB82265F4A30E7BD3AC638DFF88C072DM4PR11MB8226namp_--