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 52672940D0A for ; Fri, 1 Mar 2024 01:00:30 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=NJ05RLRUI1VC6kqNEORTvGBMMlMHQVqG0K6in6xpCt4=; c=relaxed/simple; d=groups.io; h=ARC-Seal:ARC-Message-Signature:ARC-Authentication-Results:From:To:CC:Subject:Thread-Topic:Thread-Index:Date:Message-ID:References:In-Reply-To:Accept-Language:MIME-Version:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Language:Content-Type:Content-Transfer-Encoding; s=20140610; t=1709254828; v=1; b=fIAHdSLOa4INY5lMOQe0Ijg/ge+JoUW22222xJVEg1NDW7xvfDUNw/+0kljD+Mo/+z5+VhTf 5NeR2Ts6/5o7nuke+e6My3FWX73m92l2AG5tfi3wulFU2GS6qkWmDD/bHFvuD46WT/iQpIVkUOC xRhabjcV4u5dP77foO9nOgGY= X-Received: by 127.0.0.2 with SMTP id NHIpYY7687511xuZsn6sjxgo; Thu, 29 Feb 2024 17:00:28 -0800 X-Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.8]) by mx.groups.io with SMTP id smtpd.web11.11963.1709254827812762534 for ; Thu, 29 Feb 2024 17:00:28 -0800 X-IronPort-AV: E=McAfee;i="6600,9927,10999"; a="21314042" X-IronPort-AV: E=Sophos;i="6.06,194,1705392000"; d="scan'208";a="21314042" X-Received: from fmviesa008.fm.intel.com ([10.60.135.148]) by fmvoesa102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 Feb 2024 17:00:28 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.06,194,1705392000"; d="scan'208";a="8150469" X-Received: from fmsmsx603.amr.corp.intel.com ([10.18.126.83]) by fmviesa008.fm.intel.com with ESMTP/TLS/AES256-GCM-SHA384; 29 Feb 2024 17:00:27 -0800 X-Received: from fmsmsx610.amr.corp.intel.com (10.18.126.90) by fmsmsx603.amr.corp.intel.com (10.18.126.83) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Thu, 29 Feb 2024 17:00:26 -0800 X-Received: from fmsmsx610.amr.corp.intel.com (10.18.126.90) by fmsmsx610.amr.corp.intel.com (10.18.126.90) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Thu, 29 Feb 2024 17:00:26 -0800 X-Received: from fmsedg601.ED.cps.intel.com (10.1.192.135) by fmsmsx610.amr.corp.intel.com (10.18.126.90) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35 via Frontend Transport; Thu, 29 Feb 2024 17:00:26 -0800 X-Received: from NAM12-DM6-obe.outbound.protection.outlook.com (104.47.59.168) by edgegateway.intel.com (192.55.55.70) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.35; Thu, 29 Feb 2024 17:00:26 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=IYRTdsiKwfb1zjdCuwamZGwbwhZRAvX6vGbPi/cqb6FEfMLbZ47qm2mwiQFjnzK3SQkI2zBytKuYb+roiBu8fPZwJd2MY+hbCdQ1+Xl32v1Ey4mWxTmmSjc9TgqxdF0r7pRtsp3UjfXse8n94dRyfmGxOEpqKQ9uKv1hjhuqegBZ3iwfEJsOLbuuZKXgt675cCFd/EilvWIzYcwv9gbcHIXHYazzN4LhWrHlxZTRRGOnOGJdJtkr0WDA5+SN2hfwMGAOBRny1Gc0vrEo2P/vnnouujvZ5Z+z4/hEAUFZgROBy3mk9S503djfcI5saHV0KCdY7djV87AEz86LLejnSg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=0pqwqTONTo+w8m1CIKd7OWxjA8Ltj5f/SZMJAZmPlFE=; b=A5k3AvpDLIGL3Z2X6zow09aNNRAOcjZwVi8tNNvOdXszzHaMGo75RQY/zrxxBJiWZlwfo1r6/swEoegdYUjADD/TH9QB3Uikbu7+mdnWVemYMJ8aaUxqUq3YfhWZAI2rwxfWafWX0d02Ir30+T/8GlWh5A5PZ72VdexpxVHA2nQS+KiCdyT3tInvybd32HnbH2HOvUF3YVSkuzX+yrEuy8HKp7ZOTW+ZO0in465Wt/aq6wJRTXDjUltz9bT/uBL2eYI3f+gOt4yZeP2Sj6t8woFuXtL994UnrcGzvKaMU8PRbaIm+5Fs/1h/UcYCHkop6yMBAJ1KsFlWzV44/utDUQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none X-Received: from MW4PR11MB5872.namprd11.prod.outlook.com (2603:10b6:303:169::14) by PH0PR11MB5015.namprd11.prod.outlook.com (2603:10b6:510:39::5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7339.25; Fri, 1 Mar 2024 01:00:22 +0000 X-Received: from MW4PR11MB5872.namprd11.prod.outlook.com ([fe80::92a7:f6a2:9262:d150]) by MW4PR11MB5872.namprd11.prod.outlook.com ([fe80::92a7:f6a2:9262:d150%3]) with mapi id 15.20.7339.024; Fri, 1 Mar 2024 01:00:18 +0000 From: "Yao, Jiewen" To: Leif Lindholm , Pierre Gondois , "devel@edk2.groups.io" CC: Ard Biesheuvel , Liming Gao , "Kinney, Michael D" , Sami Mujawar , "Liu, Zhiguang" , "Yao, Jiewen" Subject: Re: [edk2-devel] [RFC PATCH 1/1] ArmPkg,MdePkg: move ArmLib.h to MdePkg Thread-Topic: [RFC PATCH 1/1] ArmPkg,MdePkg: move ArmLib.h to MdePkg Thread-Index: AQHaW2zd8/J7V2CILkCmun+Z0kdnLbEG+diAgAADp4CAGzJnEA== Date: Fri, 1 Mar 2024 01:00:18 +0000 Message-ID: References: <20231121164228.766299-1-quic_llindhol@quicinc.com> <4951c86b-ecd8-4253-95e7-ceb00b8192b9@arm.com> <725dcaa0-8365-46d8-a6fe-f95b1d495518@arm.com> <128c59bc-dd86-4b26-81f7-ecc07f4c72ed@quicinc.com> In-Reply-To: <128c59bc-dd86-4b26-81f7-ecc07f4c72ed@quicinc.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-publictraffictype: Email x-ms-traffictypediagnostic: MW4PR11MB5872:EE_|PH0PR11MB5015:EE_ x-ms-office365-filtering-correlation-id: f0464922-7ef9-44f4-eae2-08dc398af695 x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam-message-info: 1PwwRV9wqJ6M2vxb8bkE0bfdj14b+dH078YNxT7ZIYNQePAJOEQVWkeFZqFnKh/+EbF2RKLAPWB4orjJ3QoSUffJ73hDWZ/ee6D7L0sM8eid8H05J96V6O8MpDCCcOEYx/40/6hadrda5p+CB4bXmaNVIXe3za772wbFDtiH+hNLDH0vzdfnTePMC/nptJ0Xjw5XB1mNgoBMJe0nVqN7tSF3+C3mc7muczRHOajJz/cDEMt450AvSrq/1grtTrXiiUnPiHAQB7g7ZJL70AuPtLihYEBjkT+jCBm3vU3/mEAqOAhK+1+kzpxzX8q81WiD58Ftk4aa3V2NnGm/e7MyjESjoSQ1UPpa0Y/LUrue5nL1Z3CPX5x99MuaF5Zivs6t3vjrY9oJKJwMgICqa5D7up48ltZFSfN+gX6i0lLJ2wSfpYoLcUv6SbWz6MY2NnU9dHitFajShSauDTgSPPDyEnHnOXmgdfprfdW8dYyCNayuV9hFrRuErY3nq5eR4jDAtcIlZmgpKk9zCTm+PMY6E+DBM3W6hQygiospJPMvj6LzOhPEJI66A1eVjnbq66UUpQzD4zr+jYdONhZU2QgFfPI7koUEd1WyYBoGJfHmQkAXl1CaDfvKjb7A15FaOej4p/FNAjgvUOoEYvMV/VTmMdRkOO83YINqB4hdgZHlsFL5h3Vht2giICC3/+BYqz6b x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?uYQIeHwgrTOSBFxGkg8PZLoTD+bKKo9dIqaXdgz1bSaK7+KyPfjzHwOgqs?= =?iso-8859-1?Q?jbp7Q75sLrzHU52rYEkPOrOMnsD3mXLwBeEce2kvZFt5F/akUR85EB4IVb?= =?iso-8859-1?Q?3bRxOFq5rhvk0wCjKrKS3RPmhEwIFaz4/QOMmVolykyyf69gEmtWrZMI00?= =?iso-8859-1?Q?8YghVkTeNJy6R2EpUf2S+fVQ4ufbjisJuIknNdFMuhxkpxPep+IOY6Nvtz?= =?iso-8859-1?Q?yqbfZPxHJMPb+x2OHjH2mv4dRrpMj3y/LWQniGKCp8h6/mzm5ruUAg6H4u?= =?iso-8859-1?Q?k8duErOObg+RCdjGpif/J3qsDH8stYNwS5kqiiIR6DsgojeAok4DPvI0e+?= =?iso-8859-1?Q?lErvfBDMQT5/QQatDau3/ddWNqKUgcu7tK5jqwzh8uLzMNnM+zW5s+xMAc?= =?iso-8859-1?Q?saTS59YKPwQ0ETS2O3CTVYeH/Q4tP8e4OQ3RfX/Xz21MB1+9g+d7qnMh6t?= =?iso-8859-1?Q?tJNigHED43nKPLGFOySdvtNmM+L7cWXglODtdQ1xn2zk9JGISwVnr/ZnFA?= =?iso-8859-1?Q?9OxbUI4yF5bfLxeam4b7KsCOc2SSTUC0wIMO12elYWwjxTiMVJCxTQ0md+?= =?iso-8859-1?Q?YMWuyHl4d+961qLJW37NEKJxRaPL9KZRbAKfjiQHhPyQMNq8erl8X6SUMT?= =?iso-8859-1?Q?cRMTYZS25cI1xCWdp/bwuxHUR2Bgei+z4Ut7CCsQR3tFvy+YBNAj6Rp2Xr?= =?iso-8859-1?Q?z1zapc9KfgLET0aDaXHyxi1L7g/OOwwYF6jweEwWoqkvjbztUUrl+0DTtE?= =?iso-8859-1?Q?FNA6u4DTaPKTSjsoTl4h+F3O5S+rSZHl3xsKlOJoeX6qNkjHsh9UhOTrDc?= =?iso-8859-1?Q?5utDbIBNlk4vjFcZpJPfPKGAXKeIdfbD28chcs8haCigoyCJsenL+XjAaj?= =?iso-8859-1?Q?qaIk+MxZdHpHlQLY6hL1bvpD0P0fV7G/9YTO6HcaAAR3GSghXXdgXe36uR?= =?iso-8859-1?Q?XLqxEyAeQ+ahIKRLtvhD8UtJX1Euh395v7xGkWtRwQTnZwROAibmXwqzVd?= =?iso-8859-1?Q?pQyxb5qre/PyMV+Rro8q9etTstFThm3W1BqrPtarrnfKUChUFn+7R9FTl9?= =?iso-8859-1?Q?mhKofdeqaVn89Bkjqooxg0mY1R6efcKM7PeDhyJ9Vi5+QZFOuoegseMdn6?= =?iso-8859-1?Q?zBNxZJhqykFHpMmgMkG9m+GnIfktzD+5o/XhYm81I3BjwuVS0iRdcrFdzE?= =?iso-8859-1?Q?wyHCfbijYoee04IM8fRf+x/I0m7BsRGALO2xJTHIEyQOak1IRg8fK0ymf4?= =?iso-8859-1?Q?vU2QGzXNUgjRuQdNLWR2D09VPJmrtJPdd2i29X7mUSP1M8UmHEdU/iaqj8?= =?iso-8859-1?Q?fdQCyu82V2v08bebNwTEyYs13sowMDhR19F69+UgMb8+4RKIy0DSK3HIrD?= =?iso-8859-1?Q?r/BDWPdZBJwIxMceXMC+S166ILpZ3VVlL9rL2LyBLnnngYi/bDj89KoZ9e?= =?iso-8859-1?Q?Ve7r7eCZf8EBIH5nSd526tqh+KayadccJ9OEg+7QPHxAHXawHsdu6ow+OW?= =?iso-8859-1?Q?8IgvG/qC6WhWoHvCGFTDTcrXCPP36bcOpFzWN/trCtJqZpRVpNQSw1c/bw?= =?iso-8859-1?Q?5G4EZJw35eVkhux9vnYFmUHUl9BSmAv6VTqjv/uuwdHk6nuoO4iIljzZc1?= =?iso-8859-1?Q?Z8MfM7tGVnk70nnqG4JfkNJTB2Q68v3iaF?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: MW4PR11MB5872.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: f0464922-7ef9-44f4-eae2-08dc398af695 X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Mar 2024 01:00:18.4694 (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: IN8gIbckzRHd5XvkZ0Vs/yb455OjXgqnkk/F7vvbhbYM50mC14whMpyYXMdQ9XYP1aw6KBHMBwOpU5sOZL+0rA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB5015 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 Reply-To: devel@edk2.groups.io,jiewen.yao@intel.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: v8gSWcMwZKwUe7OxaX3mvRG1x7686176AA= Content-Language: en-US Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20140610 header.b=fIAHdSLO; 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 Sure. When we say "dependency", what we really mean is the dependency in INF file= , not "dependency" in DSC file. >From package release perspective, only INF is the interface to other packag= e.=20 The DSC is only the package internal stuff, you can create multiple DSCs or= add/remove DSC freely. Having "dependency" in DSC does not matter. Having dependency in INF is something we should care about. Thank you Yao, Jiewen > -----Original Message----- > From: Leif Lindholm > Sent: Tuesday, February 13, 2024 1:38 AM > To: Pierre Gondois ; devel@edk2.groups.io; Yao, > Jiewen > Cc: Ard Biesheuvel ; Liming Gao > ; Kinney, Michael D ; > Sami Mujawar ; Liu, Zhiguang > > Subject: Re: [RFC PATCH 1/1] ArmPkg,MdePkg: move ArmLib.h to MdePkg >=20 > Jiewen, can you clarify what you said back in > https://edk2.groups.io/g/devel/message/111551 > ? >=20 > On 2024-02-12 17:24, Pierre Gondois wrote: > >>> A ArmLibNull.inf library might also need to be created. If the > >>> OpensslLibFullAccel.inf module will depend on the ArmLib library, > >>> a Null implementation will be necessary for non-Arm architectures. > >> > >> Can ArmLib be declared under a [LibraryClasses.AArch64, > >> LibraryClasses.ARM]? Have I forgotten something that we discussed back > >> in ... November? > > > > From [1], it seems the MdePkg/CryptoPkg should build without a depende= ncy > > on the ArmPkg. This is currently not really the case. cf. [2]. > > > > However, having a ArmLibNull implementation in the MdePkg would allow t= o > > avoid going in this direction when providing libraries to CryptoPkg.dsc= . > > > > (Just in case, I think this ArmLibNull is a minor point.) >=20 > Well, sure, it is now. > Until we need a RiscV64LibNull, LoongarchLibNull, ... >=20 > > [1] https://edk2.groups.io/g/devel/message/111545 > > [2] > > > https://github.com/tianocore/edk2/blob/8801c75b4d77c2e6e06b3ddc8560e0db > 590f6342/CryptoPkg/CryptoPkg.dsc#L117 > > > >> > >>> Otherwise I could apply and run the CryptoPkg/Arm native instructions > >>> patchset on top of this patch. > >>> > >>> --- > >>> > >>> As a side note, it also seems that: > >>> - ArmPkg/Include/Chipset/ArmCortexA5x.h > >>> =A0 =A0 isn't used anymore in edk2/edk2-platorms > >>> - ArmPkg/Include/Chipset/ArmCortexA9.h > >>> =A0 =A0 is barely used in edk2-platforms. > >>> Maybe the files should have been removed/simplified as part of > >>> - cffa7925a293 ("ArmPkg: remove ArmCpuLib header and implementations"= ) > >>> - a913ad02479d ("ArmPlatformPkg: remove ArmVExpressPkg") > >> > >> I think you're right. > >> Well, ArmCortexA9.h is still *used*, but I can't imagine the Arm branc= h > >> of ArmVExpressLib has been build by anyone for some time. And surely t= he > >> inclusion of ArmVExpressLibSec in ArmVExpress-FVP-AArch64.dsc is > >> superfluous (such that that .inf can be deleted)? > > > > The file could just be moved in the Library. I assume you/Sami/Ard > > will know more on the usage of the library itself, >=20 > Sami? >=20 > / > Leif >=20 -=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 (#116193): https://edk2.groups.io/g/devel/message/116193 Mute This Topic: https://groups.io/mt/102731845/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-