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 DE029740051 for ; Tue, 21 Nov 2023 16:03:14 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=+/nxmDBVeTJ/SJKlZP80NItCLf0SFS3uaHV7bTqhmGE=; c=relaxed/simple; d=groups.io; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject:To:Cc:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Type:Content-Transfer-Encoding; s=20140610; t=1700582593; v=1; b=ePkFublarH5AcIUqaJONnf3U2Db0ChjhHZ1/ZpwIbDClfl8c0odUYxFQx73XbCvLjWHgONpg wLnENooBMMxgKasse6AVCqX93Bq/u2qD1OqqMrjkL72QaJYw8X4RrfwJ1XyZAX0Sro4u1LfZLAr 3Cb6L+PYktEYtHq9ugD4ibTM= X-Received: by 127.0.0.2 with SMTP id YTx6YY7687511xecn7oys1lI; Tue, 21 Nov 2023 08:03:13 -0800 X-Received: from ams.source.kernel.org (ams.source.kernel.org [145.40.68.75]) by mx.groups.io with SMTP id smtpd.web11.45169.1700582592717744973 for ; Tue, 21 Nov 2023 08:03:13 -0800 X-Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by ams.source.kernel.org (Postfix) with ESMTP id A20C6B822B8 for ; Tue, 21 Nov 2023 16:03:10 +0000 (UTC) X-Received: by smtp.kernel.org (Postfix) with ESMTPSA id A6436C433CA for ; Tue, 21 Nov 2023 16:03:09 +0000 (UTC) X-Received: by mail-lj1-f180.google.com with SMTP id 38308e7fff4ca-2c8879a1570so22081011fa.1 for ; Tue, 21 Nov 2023 08:03:09 -0800 (PST) X-Gm-Message-State: LqwjZdkDGvvUvaA9MSR8cLF7x7686176AA= X-Google-Smtp-Source: AGHT+IEW76fvXjImw/CWduwsq+geDHeBGJ9ua/BuQXHS51KONGw6+OtwMO2mENtZUzZFupFwNMcjTg0ZKRpxeE8p3R0= X-Received: by 2002:a2e:9117:0:b0:2bf:e596:36f9 with SMTP id m23-20020a2e9117000000b002bfe59636f9mr7257385ljg.13.1700582587790; Tue, 21 Nov 2023 08:03:07 -0800 (PST) MIME-Version: 1.0 References: <20231121084715.1393672-1-pierre.gondois@arm.com> <20231121084715.1393672-3-pierre.gondois@arm.com> In-Reply-To: From: "Ard Biesheuvel" Date: Tue, 21 Nov 2023 11:02:56 -0500 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [edk2-devel] [PATCH v5 2/6] CryptoPkg/CryptoPkg.ci.yaml: Allow dependency upon ArmPkg To: devel@edk2.groups.io, quic_llindhol@quicinc.com Cc: "Yao, Jiewen" , Pierre Gondois , "Li, Yi1" , "Lu, Xiaoyu1" , "Jiang, Guomin" , Ard Biesheuvel , Sami Mujawar , Gerd Hoffmann 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,ardb@kernel.org List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: Content-Type: text/plain; charset="UTF-8" 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=ePkFubla; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=kernel.org (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 On Tue, 21 Nov 2023 at 10:55, Leif Lindholm wro= te: > > On Tue, Nov 21, 2023 at 14:46:05 +0000, Yao, Jiewen wrote: > > This Bugzilla is filed in 2022-10-26. Now it is 2023-11-21. > > Oh, I'm sure I voiced the same opinion for many years before someone > (rightly) told me to go gile that bugzilla. > > > I agree with you that it is a big task. May I know what is the plan? > > E.g. who is working on that? When do you expect it will be done? > > On my list of "big items" to deal with, this comes after github PR > migration and line-ending conversion. > > > According to the dependency rule, what we need is only *interface* > > definition, but not *implementation*. > > That means the really requirement here is to move *interface* from > > ArmPkg to MdePkg, you can still keep the library implementation in > > ArmPkg. (It is just a subset of this Bugzilla) > > That ... is an option I had not previously considered. > Long-term we would still like to smash ArmLib into BaseLib, but if > MdePkg maintainers would be OK with moving ArmLib.h into MdePkg... > > > Also, I don=E2=80=99t think CPUID check really matters here - since it = is only implementation. > > As long as, you have interface in MdePkg, then your INF can declare tha= t interface. > > You can still put real implementation in ArmPkg - no requirement to mov= e. > > That benefit is that you don=E2=80=99t need to add ArmPkg dependency in= yaml. > > I can spin up a patch for that to get merged shortly after stable tag > to give plenty of time to catch any issues that may arise from moving > such a fundamental file. (These would likely be bugs, but > nevertheless...) > This sounds like a reasonable solution to me for the short term. -=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 (#111556): https://edk2.groups.io/g/devel/message/111556 Mute This Topic: https://groups.io/mt/102725178/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-