From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail05.groups.io (mail05.groups.io [45.79.224.7]) by spool.mail.gandi.net (Postfix) with ESMTPS id 12E55D801B4 for ; Thu, 1 Aug 2024 22:39:29 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=K7TKX17CJJd82oUgnwSJUrYsOdyyzyRwzAnkgWt6Ym8=; c=relaxed/simple; d=groups.io; h=DKIM-Filter:Message-ID:Date:MIME-Version:User-Agent:To:From:Subject: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:Content-Transfer-Encoding; s=20240206; t=1722551969; v=1; b=PwB10abqNOQbVSmQj5r1xZnENGva9Ka9CMej55b8SIeH2b62/Lon+9zshJBW4AvqNkX4gofk 8YQye35u/YZDvPEJBg1IYattdD7w/4rBovjJKSgmXGLDQjEHsojXdAAsgn+JDdcyHL4ExFQnsSA lcTpeDt/UBuG6BYI9e4uiTGz1x89Rh3K7i30xtrRZjx6fBR5TwnPTOei0qFEeOT3/22PzxrFocF 5RmEuM9c+3cT1m5e3LVJiRhEtJ7urPIfYl6GRjO/bs4wqT9+m4mmVZlsmceWEieQHYehFz05DuN p4t9dbZQbOHm6oXKfDGvvVjXj6yOFVOJBg4Xsv63C5dUg== X-Received: by 127.0.0.2 with SMTP id nVMkYY7687511xZfYcX7l5cN; Thu, 01 Aug 2024 15:39:28 -0700 X-Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by mx.groups.io with SMTP id smtpd.web11.80814.1722551967883388778 for ; Thu, 01 Aug 2024 15:39:27 -0700 X-Received: from [10.137.194.171] (unknown [131.107.159.43]) by linux.microsoft.com (Postfix) with ESMTPSA id 3462720B7165; Thu, 1 Aug 2024 15:39:27 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 3462720B7165 Message-ID: Date: Thu, 1 Aug 2024 15:39:26 -0700 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird To: "devel@edk2.groups.io" , Ard Biesheuvel , Leif Lindholm , Sami Mujawar , Michael Kinney , Liming Gao , Zhiguang Liu From: "Oliver Smith-Denny" Subject: [edk2-devel] [RFC] Move CompilerIntrinsicsLib and ArmSoftFloatLib to MdePkg 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: Thu, 01 Aug 2024 15:39:28 -0700 Resent-From: osde@linux.microsoft.com Reply-To: devel@edk2.groups.io,osde@linux.microsoft.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: Fip76WmLirf4j1qD08LDQMwex7686176AA= Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: quoted-printable X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20240206 header.b=PwB10abq; spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 45.79.224.7 as permitted sender) smtp.mailfrom=bounce@groups.io; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=linux.microsoft.com (policy=none) CompilerIntrinsicsLib and ArmSoftFloatLib add ARM/AARCH64 compiler intrinsics and floating point functions required by OpenSSL, respectively. CompilerIntrinsicsLib is used almost in every DSC that builds ARM/AARCH64 and ArmSoftFloatLib is used by every DSC that builds logic from OpenSSL. Together these make almost every DSC have a dependency on ArmPkg, which is odd and for a handful, MdeModulePkg, EmbeddedPkg, and ShellPkg, namely, it is a circular dependency. There have been previous mailing list suggestions to move CompilerIntrinsicsLib to MdePkg (possibly combining with other arch intrinsics). I am not sure the end status of those conversations. By moving these two libraries to MdePkg package, we accomplish a few things: - Removing the circular dependency from MdeModulePkg and ShellPkg (EmbeddedPkg has other ArmPkg dependencies) - Aligning MdePkg as the base package where baseline build and spec dependencies are found with other industry standard behavior - Detangling ArmPkg and making ARM/AARCH64 more of a first class citizen in edk2 instead of bolted onto the side There is no functional change here and the amount of work is light, but I think it moves edk2 in the direction it wants to go, so I'm happy to put up a PR for this, but I wanted to get feedback before I did so. This aligns with similar efforts, such as moving more ARM/AARCH64 chipset definitions to MdePkg from ArmPkg [1][2][3]. This also aligns to the overall goal of deleting ArmPkg: https://bugzilla.tianocore.org/show_bug.cgi?id=3D4121 Thanks, Oliver [1]=20 https://github.com/tianocore/edk2/commit/f2b9d5417dccf763bcbb68cd0effed0e25= 890aab [2]=20 https://github.com/tianocore/edk2/commit/cf323e2839ce260fde43487baae205527d= ee1b2f [3]=20 https://github.com/tianocore/edk2/commit/c68fb69dfefa7a76ebad33674a49632c4f= 8c6926 -=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 (#120197): https://edk2.groups.io/g/devel/message/120197 Mute This Topic: https://groups.io/mt/107675828/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-