From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by mx.groups.io with SMTP id smtpd.web08.435.1612996312828150895 for ; Wed, 10 Feb 2021 14:31:52 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=MNIy8nKR; spf=pass (domain: kernel.org, ip: 198.145.29.99, mailfrom: ardb@kernel.org) Received: by mail.kernel.org (Postfix) with ESMTPSA id 477EC64EAC for ; Wed, 10 Feb 2021 22:31:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1612996312; bh=mwWttJo6QMMFNM4xkCL9UC3NwrA5aagpjczoo6brevM=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=MNIy8nKR9Lp4ZNywCzUOCyaXh2zVN5TPbrnSP3VUrji9e+nPSlis57a0szH/OwRma TfitlPuuO2cGIKEuolOQDjzNqsssJ/TTey8YsWYkNsiKe3dI/az/bJvyZl3N9Y88HQ MwQSAcummvdSh+llwxIMFvZNScatHEBocXvbiQ8oyCsf511QGAIRfsWqZ2SNm1aC1n tVRlN+UvP4ilAx7S4LAeogVng5WFmBcBAURa/T/pSY//nx1EYdD/q+SSstdQDXK0t9 RIIrJE9g8/8jppksdd+tkLVySwdxg7QqbNxdeX43yikjcNHDbP4e9iUQz7MzeMepPc 2d7PwroVjl7Sw== Received: by mail-oi1-f182.google.com with SMTP id h6so3942938oie.5 for ; Wed, 10 Feb 2021 14:31:52 -0800 (PST) X-Gm-Message-State: AOAM533cweU7dpuT62xtmM+aTwEwQ720q0tS+yKNsTSmY9Cktx7XWB6n 9s70lwwcGkq7hCqVK8/gARFu+xZL8F5i5pRny1Q= X-Google-Smtp-Source: ABdhPJwoTEHH0CGzfXZSfnBrlwBXCwXykukOBZmcpHHMd1WvKCLmxKaEfvXZw7Y8rA7FIGY6gmWpunYwKx/QO2Wi9Zg= X-Received: by 2002:aca:b6c1:: with SMTP id g184mr856686oif.47.1612996311715; Wed, 10 Feb 2021 14:31:51 -0800 (PST) MIME-Version: 1.0 References: <20210203035415.514-1-mikuback@linux.microsoft.com> In-Reply-To: <20210203035415.514-1-mikuback@linux.microsoft.com> From: "Ard Biesheuvel" Date: Wed, 10 Feb 2021 23:31:40 +0100 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [edk2-devel] [PATCH v1 1/1] StandaloneMmPkg: Add X64 build support To: devel@edk2.groups.io, Michael Kubacki Cc: Ard Biesheuvel , Sami Mujawar , Jiewen Yao , Supreeth Venkatesh Content-Type: text/plain; charset="UTF-8" On Wed, 3 Feb 2021 at 04:54, wrote: > > From: Michael Kubacki > > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3202 > > Several libraries have been added to the package to support X64 but > the package itself does not support X64. This modifies the DSC to > enable X64 build. > > Cc: Ard Biesheuvel > Cc: Sami Mujawar > Cc: Jiewen Yao > Cc: Supreeth Venkatesh > Signed-off-by: Michael Kubacki > --- > StandaloneMmPkg/StandaloneMmPkg.dsc | 13 ++++++++++--- > 1 file changed, 10 insertions(+), 3 deletions(-) > > diff --git a/StandaloneMmPkg/StandaloneMmPkg.dsc b/StandaloneMmPkg/StandaloneMmPkg.dsc > index a566724c88ab..73f3f0f6b1cd 100644 > --- a/StandaloneMmPkg/StandaloneMmPkg.dsc > +++ b/StandaloneMmPkg/StandaloneMmPkg.dsc > @@ -3,6 +3,7 @@ > # > # Copyright (c) 2015, Intel Corporation. All rights reserved.
> # Copyright (c) 2016 - 2021, Arm Limited. All rights reserved.
> +# Copyright (C) Microsoft Corporation
> # > # SPDX-License-Identifier: BSD-2-Clause-Patent > # > @@ -19,7 +20,7 @@ [Defines] > PLATFORM_VERSION = 1.0 > DSC_SPECIFICATION = 0x00010011 > OUTPUT_DIRECTORY = Build/StandaloneMm > - SUPPORTED_ARCHITECTURES = AARCH64 > + SUPPORTED_ARCHITECTURES = AARCH64|X64 > BUILD_TARGETS = DEBUG|RELEASE > SKUID_IDENTIFIER = DEFAULT > > @@ -37,6 +38,7 @@ [LibraryClasses] > # > BaseLib|MdePkg/Library/BaseLib/BaseLib.inf > BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLib.inf > + CacheMaintenanceLib|MdePkg/Library/BaseCacheMaintenanceLib/BaseCacheMaintenanceLib.inf > DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf > DebugPrintErrorLevelLib|MdePkg/Library/BaseDebugPrintErrorLevelLib/BaseDebugPrintErrorLevelLib.inf > ExtractGuidedSectionLib|EmbeddedPkg/Library/PrePiExtractGuidedSectionLib/PrePiExtractGuidedSectionLib.inf > @@ -47,7 +49,7 @@ [LibraryClasses] > MemoryAllocationLib|StandaloneMmPkg/Library/StandaloneMmCoreMemoryAllocationLib/StandaloneMmCoreMemoryAllocationLib.inf > MmServicesTableLib|MdePkg/Library/StandaloneMmServicesTableLib/StandaloneMmServicesTableLib.inf > PcdLib|MdePkg/Library/BasePcdLibNull/BasePcdLibNull.inf > - PeCoffExtraActionLib|StandaloneMmPkg/Library/StandaloneMmPeCoffExtraActionLib/StandaloneMmPeCoffExtraActionLib.inf > + PeCoffExtraActionLib|MdePkg/Library/BasePeCoffExtraActionLibNull/BasePeCoffExtraActionLibNull.inf > PeCoffLib|MdePkg/Library/BasePeCoffLib/BasePeCoffLib.inf > PrintLib|MdePkg/Library/BasePrintLib/BasePrintLib.inf > ReportStatusCodeLib|MdePkg/Library/BaseReportStatusCodeLibNull/BaseReportStatusCodeLibNull.inf > @@ -60,6 +62,7 @@ [LibraryClasses.AARCH64] > StandaloneMmMmuLib|ArmPkg/Library/StandaloneMmMmuLib/ArmMmuStandaloneMmLib.inf > ArmSvcLib|ArmPkg/Library/ArmSvcLib/ArmSvcLib.inf > CacheMaintenanceLib|ArmPkg/Library/ArmCacheMaintenanceLib/ArmCacheMaintenanceLib.inf > + PeCoffExtraActionLib|StandaloneMmPkg/Library/StandaloneMmPeCoffExtraActionLib/StandaloneMmPeCoffExtraActionLib.inf > > NULL|ArmPkg/Library/CompilerIntrinsicsLib/CompilerIntrinsicsLib.inf > NULL|MdePkg/Library/BaseStackCheckLib/BaseStackCheckLib.inf > @@ -110,11 +113,11 @@ [Components.common] > StandaloneMmPkg/Library/StandaloneMmHobLib/StandaloneMmHobLib.inf > StandaloneMmPkg/Library/StandaloneMmMemLib/StandaloneMmMemLib.inf > StandaloneMmPkg/Library/StandaloneMmMemoryAllocationLib/StandaloneMmMemoryAllocationLib.inf > - StandaloneMmPkg/Library/StandaloneMmPeCoffExtraActionLib/StandaloneMmPeCoffExtraActionLib.inf > StandaloneMmPkg/Library/VariableMmDependency/VariableMmDependency.inf > > [Components.AARCH64] > StandaloneMmPkg/Drivers/StandaloneMmCpu/AArch64/StandaloneMmCpu.inf > + StandaloneMmPkg/Library/StandaloneMmPeCoffExtraActionLib/StandaloneMmPeCoffExtraActionLib.inf > > ################################################################################################### > # > @@ -128,3 +131,7 @@ [Components.AARCH64] > [BuildOptions.AARCH64] > GCC:*_*_*_DLINK_FLAGS = -z common-page-size=0x1000 -march=armv8-a+nofp -mstrict-align > GCC:*_*_*_CC_FLAGS = -mstrict-align > + > +[BuildOptions.X64] > + MSFT:*_*_*_DLINK_FLAGS = /ALIGN:4096 > + GCC:*_GCC*_*_DLINK_FLAGS = -z common-page-size=0x1000 > -- > 2.28.0.windows.1 > > This one does not apply either. Perhaps it got mangled because I only received it via the mailing list rather than directly via CC?