From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=217.140.101.70; helo=foss.arm.com; envelope-from=thomas.abraham@arm.com; receiver=edk2-devel@lists.01.org Received: from foss.arm.com (usa-sjc-mx-foss1.foss.arm.com [217.140.101.70]) by ml01.01.org (Postfix) with ESMTP id 2F22C21B02822 for ; Wed, 21 Nov 2018 08:01:43 -0800 (PST) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 9FF811D70 for ; Wed, 21 Nov 2018 08:01:43 -0800 (PST) Received: from mail-it1-f176.google.com (usa-sjc-mx-foss1.foss.arm.com [217.140.101.70]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 7F1FA3F71A for ; Wed, 21 Nov 2018 08:01:43 -0800 (PST) Received: by mail-it1-f176.google.com with SMTP id a205-v6so9550626itd.4 for ; Wed, 21 Nov 2018 08:01:43 -0800 (PST) X-Gm-Message-State: AGRZ1gLfqfYdy+rtXatFgMZb/HPeaTINp+F1Qks8TzfbNKOz5iO/RlSP U/j38+l3aq2BFVro3z8itUqUBpcpQ2+56BB34yU= X-Google-Smtp-Source: AJdET5c3e1pITBa/88RKFpmaMq74T6YwyYe7JOxen4ORimKI7ifTTrU1NxzjEiX/572KyFRKYBuMylAUC9T7QKzC6Fo= X-Received: by 2002:a24:28d2:: with SMTP id h201mr6215276ith.136.1542816102770; Wed, 21 Nov 2018 08:01:42 -0800 (PST) MIME-Version: 1.0 References: <1540452759-4875-1-git-send-email-sughosh.ganu@arm.com> <20181119050045.GA4937@arm.com> In-Reply-To: <20181119050045.GA4937@arm.com> From: Thomas Abraham Date: Wed, 21 Nov 2018 21:31:31 +0530 X-Gmail-Original-Message-ID: Message-ID: To: sughosh.ganu@arm.com Cc: Ard Biesheuvel , Leif Lindholm , edk2-devel@lists.01.org Subject: Re: [PATCH v3 0/6] ArmPkg related changes for StandaloneMM package X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 21 Nov 2018 16:01:44 -0000 Content-Type: text/plain; charset="UTF-8" Hi Ard, Leif, On Mon, Nov 19, 2018 at 10:39 AM Sughosh Ganu wrote: > > hi Ard, > > On Thu Oct 25, 2018 at 01:02:33PM +0530, Sughosh Ganu wrote: > > The following patch series adds support for Management Mode related > > changes for aarch64 based platforms. > > > > Changes since v2: > > Based on review comments from Ard, moved the memory attribute updation > > changes out of DebugPeCoffExtraActionLib into an extra action library > > added in StandaloneMM package. The patch for setting the memory > > attributes, now under StandaloneMmPkg directory, will be submitted > > separately from this series. > > Can you please review the V3 of this series. Based on your review > comments, i have moved the memory attribute updation into an extra > action lib added under StandaloneMmPkg. Can you please check if these > updated patches for ArmPkg[1] and the one under StandaloneMmPkg[2] are > fine. > > I had put you and Leif on Cc while sending out the patches, but i > think that the mailserver deleted those, which is why you may have > missed the patches. > > -sughosh > > [1] - https://lists.01.org/pipermail/edk2-devel/2018-October/031377.html > [2] - https://lists.01.org/pipermail/edk2-devel/2018-October/031384.html > > > > > > > Achin Gupta (5): > > ArmPkg: Add PCDs needed for MM communication driver. > > ArmPkg/Drivers: Add EFI_MM_COMMUNICATION_PROTOCOL DXE driver. > > ArmPkg/Include: Add MM interface SVC return codes. > > ArmPkg/ArmMmuLib: Add MMU Library suitable for use in S-EL0. > > ArmPkg/ArmMmuLib: Add MMU library inf file suitable for use in S-EL0. > > > > Sughosh Ganu (1): > > ArmPkg/Include: Fix the SPM version SVC ID > > > > ArmPkg/ArmPkg.dec | 3 + > > ArmPkg/Drivers/MmCommunicationDxe/MmCommunication.inf | 56 +++ > > ArmPkg/Library/ArmMmuLib/{ArmMmuPeiLib.inf => ArmMmuStandaloneMmCoreLib.inf} | 23 +- > > ArmPkg/Drivers/MmCommunicationDxe/MmCommunicate.h | 28 ++ > > ArmPkg/Include/IndustryStandard/ArmMmSvc.h | 9 +- > > ArmPkg/Drivers/MmCommunicationDxe/MmCommunication.c | 395 ++++++++++++++++++++ > > ArmPkg/Library/ArmMmuLib/AArch64/ArmMmuStandaloneMmCoreLib.c | 204 ++++++++++ > > 7 files changed, 704 insertions(+), 14 deletions(-) > > create mode 100644 ArmPkg/Drivers/MmCommunicationDxe/MmCommunication.inf > > copy ArmPkg/Library/ArmMmuLib/{ArmMmuPeiLib.inf => ArmMmuStandaloneMmCoreLib.inf} (51%) > > create mode 100644 ArmPkg/Drivers/MmCommunicationDxe/MmCommunicate.h > > create mode 100644 ArmPkg/Drivers/MmCommunicationDxe/MmCommunication.c > > create mode 100644 ArmPkg/Library/ArmMmuLib/AArch64/ArmMmuStandaloneMmCoreLib.c Any feedback on this patch series. This series is important in getting the MM Standalone mode feature supported for Arm platforms. There are patches being worked on to enable secure variable access feature for Arm platforms and it depends on this series. So if there any comments on these patches, please let us know. Thanks, Thomas. > > > > -- > > 2.7.4 > > > > _______________________________________________ > > edk2-devel mailing list > > edk2-devel@lists.01.org > > https://lists.01.org/mailman/listinfo/edk2-devel > > -- > -sughosh > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel