From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=40.107.3.77; helo=eur03-am5-obe.outbound.protection.outlook.com; envelope-from=achin.gupta@arm.com; receiver=edk2-devel@lists.01.org Received: from EUR03-AM5-obe.outbound.protection.outlook.com (mail-eopbgr30077.outbound.protection.outlook.com [40.107.3.77]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 126722112DF7D for ; Tue, 19 Jun 2018 01:07:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=armh.onmicrosoft.com; s=selector1-arm-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ypxBwqojc84Rh3fmuGfoK+o9yD2TdtmM8x12TN1+5Wk=; b=I8ByXDHNiqt+mHwGCw5Tx3BQiHHb0cqmLyfJ1ijuEIRXVS+XAWKvHmM/0Rv7JSXY4rsc9qe8dKUQ4t2cB4y7agvKwGejx/PjAXyGQT23PWE8Vp+2AHC2wqPwe7hbsdVOEArs6hXCElilacODQoDgcAc3VCn5Up5Edxb6ckhjgGY= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Achin.Gupta@arm.com; Received: from e102648.cambridge.arm.com (217.140.96.140) by VI1PR08MB2991.eurprd08.prod.outlook.com (2603:10a6:803:44::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.841.14; Tue, 19 Jun 2018 08:07:51 +0000 Date: Tue, 19 Jun 2018 09:23:35 +0100 From: Achin Gupta To: "Yao, Jiewen" Cc: Thomas Abraham , Sughosh Ganu , "edk2-devel@lists.01.org" , "Gao, Liming" , nd@arm.com Message-ID: <20180619082333.GE8875@e102648.cambridge.arm.com> References: <20180604221351.34760-1-supreeth.venkatesh@arm.com> <74D8A39837DF1E4DA445A8C0B3885C503AC4EE35@shsmsx102.ccr.corp.intel.com> MIME-Version: 1.0 In-Reply-To: <74D8A39837DF1E4DA445A8C0B3885C503AC4EE35@shsmsx102.ccr.corp.intel.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-Originating-IP: [217.140.96.140] X-ClientProxiedBy: LO2P265CA0043.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:61::31) To VI1PR08MB2991.eurprd08.prod.outlook.com (2603:10a6:803:44::21) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 7891ca62-d240-4b85-a627-08d5d5bbc127 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(7020095)(4652020)(4534165)(4627221)(201703031133081)(201702281549075)(48565401081)(5600026)(711020)(2017052603328)(7153060)(7193020); SRVR:VI1PR08MB2991; X-Microsoft-Exchange-Diagnostics: 1; VI1PR08MB2991; 3:7SAe83YxikS9EuPhugL/sbzJw3VEbBV+WJ21iCbtRAP5vPscyNRPwEv4bEVQv8wIemzkjlcbNoBn+boxCaJIsioBtupns09uRMWiSkXRv8F1UAd9/QCKtjfmuv7hEE70CfzuOLRzJDMHhA81sNUNzVfSV6ZLR/GIdKfAUDO91gNwB9l+qCr1oPyJBzfisaMI0ztUxqQ/CsNd7rSbjAe64kY9Wn7UsYrGr5B0Ta5RiRsdLQ0X/4SdJP/MAiGRf1le; 25:zG4ITlx5kZAPY0/EIAMGhPz4NlbnjnR5geEnVUvOMmKOBlYefBprNOUzc8at48U43Nzw8z3UkkZRR6SVHWI6bN7TnIZWLBIyLxY0l/GDOh9y9yE0pEb0JOwBzK9HYPecBcp+i/4AxK6VGwFKknSA7Ksal/dRMJ8vSJSKvrQb851yCdY1oQ3vwnHR8MgC+SfILk9jhTqMzQZ/O7Cd8RCRdg+UJ+vZtIVItBWW4ZE3JN/E/9N4vy/9qFxnjaS+3v61TVwmBzMhnoqdFDxZMh+PobxDaxbvS86735oxtYYimBdIrPauJAfalF0HX4RKfvLKJakhuUfwOnjj2RaSS+Maxw==; 31:88HyWAKnAl2sDoX627ByLBc2/UJgDBJ/vt+/DXV0G1J/Km77svWj6NVce6/nuV70JFAj2pbi0/+eR6FC2SptobwcxhiGvveoNjKySLJWaCeSmYPS2u73OVLhAVdk0vcZ4g7ZpeGyU36GOrBrJcs+BMaN5DZn1ZLCXNQVGZW8G5ylUobNEj5OcBhAPROZ2we+heHMTQN4mRurUKo0r7aunHc5+32gg6RgaMtGhQmrYS4= X-MS-TrafficTypeDiagnostic: VI1PR08MB2991: NoDisclaimer: True X-Microsoft-Exchange-Diagnostics: 1; VI1PR08MB2991; 20:0KWjMbEydsn5aCU2k8aQX+6EO3pHPyHZrVB+GIMj6JWlwFIuwTidVWDCVE127JHYla0+/ejiQAlyb8uRDsllPpn4+25ExwnGYSzucObI7IoE6fnqdDJLZTMWtfLmxAO1Ts7pHFxJiR4+2O6GCEF2G0rC8MNgLdD9U8Mlg7knMYG9mCAN/6x1Sm/inLkpUUC68FZEup1WBKwL9oSVo9R+l0jQ6mcj+wBZ3iVbW5tyVbIL12dz+ZhuOTOOkp6zDAIB; 4:M7YZhFg/3aRchnPLlk0WZVTJ2JP/xU+a8haHLO28otFYB/EEb5BUyH+tbwnYXSOLu3v9gaTXOPF4411f4eS7xJn2cPzh0DRpQFNOTdQR3NeCDSZvqakvdJyxXhNqV59DCcRWTnYSKa4Bog36UJDZBPz9uBH0vZ2vOZ2yh0FdD5382oRMzqz3QO+QJnGLQmGCiR+VytP8wZuii5z9wyB+60rZpYhm6ZqxM/RGEsqMpoCWLN2RdrLfS59QzYNcqc/K/eo1SLCSZSGqx9yJ6mAaVUf70SOkeO+Ocko00bECAPMeuZql8TAeTjoQEFO+J7s/+FqwVMp1DzGbIGXT18AI7Ytm5VBwMUL3easz02STA73zj3taeoeCKCMlQBjIIaAs4+VRI9fzBdXTmc8g5+maPtHBUDjUce8kRM4zmjDIgls= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(180628864354917)(85827821059158)(162533806227266)(228905959029699); X-MS-Exchange-SenderADCheck: 1 X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(3231254)(944501410)(52105095)(93006095)(93001095)(3002001)(10201501046)(6055026)(149027)(150027)(6041310)(20161123558120)(20161123562045)(20161123564045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123560045)(6072148)(201708071742011)(7699016); SRVR:VI1PR08MB2991; BCL:0; PCL:0; RULEID:; SRVR:VI1PR08MB2991; X-Forefront-PRVS: 07083FF734 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(346002)(39380400002)(396003)(376002)(366004)(39860400002)(189003)(199004)(13464003)(59450400001)(26005)(2906002)(6306002)(478600001)(53936002)(39060400002)(86362001)(4326008)(386003)(53546011)(106356001)(6246003)(105586002)(68736007)(25786009)(93886005)(8676002)(6116002)(1076002)(81166006)(33656002)(81156014)(16586007)(47776003)(54906003)(52116002)(76176011)(23726003)(966005)(7696005)(3846002)(72206003)(50466002)(316002)(8936002)(6916009)(58126008)(229853002)(186003)(97736004)(6666003)(16526019)(486006)(7736002)(476003)(305945005)(956004)(55016002)(11346002)(44832011)(446003)(66066001)(5660300001)(18370500001); DIR:OUT; SFP:1101; SCL:1; SRVR:VI1PR08MB2991; H:e102648.cambridge.arm.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1; Received-SPF: None (protection.outlook.com: arm.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; VI1PR08MB2991; 23:IXLgtzYUQNN89ESbtTn4Gd6rki7MsHZtjeoEqCECo?= =?us-ascii?Q?uwuf1yhAomAcHikJV9PqhPZSrDv0mJ/oMWDcFiqNf1+PecsVyizVKBZhI8xE?= =?us-ascii?Q?m8xmDmasroKP9Blz0ss9VXSLqWeZOvCf6IHFkYcvZmQQrJ5bXBqqyd7zmxou?= =?us-ascii?Q?87Q/vhtiMfS+2MPv2znSWfS7Yqakl47C7Do5NuIEX1I8BCdlPvTKPvOZXXuJ?= =?us-ascii?Q?m19sS8Nr/Jv85DmUTg0LQBjRA7FsSoR1BdrxczYAk9S0NczYZ0R68zjYpKZB?= =?us-ascii?Q?1+WvzEP69YZBQ+isHemldDuhCdPEWR4epa9yb491tJoNs5VjPDSWB+OBvzkO?= =?us-ascii?Q?s8EgHDoyvxin2gWjNtqPhT3QfFg10AFVI8MtvLSthH6o0+mGTu2FHQLXc3vo?= =?us-ascii?Q?ovQXHeP9435M6KMGRfWDNlDujDZtadvHQF87bWhDzgb7xlSx6cjiNFzh3Q34?= =?us-ascii?Q?G6sZ5lmBj0yeegPIgAERe11JZpZmqbb2HhAeGVbiQcDljpF396ZkPL62DWgO?= =?us-ascii?Q?+/zYGL8gwwPAhEfodqCIU781oIfUgOwEbA57bSuGsQYQjRfn0dH/Yqbz1XDl?= =?us-ascii?Q?1kdHPYU/ejn3KuBcDPXsfrIg1YCLvFhGw0Uhyh70qk31Um5QerCW45JaLJEO?= =?us-ascii?Q?voJmOkWaI+2QkSqtnU9H8ZzUxaIRP31fh9rUU1IJbqSXxnPVijvpDMeSieyI?= =?us-ascii?Q?FirjIYYFLCYfSUSH1bkVH8IKX16DAtg30eqlVcBvfDbWWA6x9qeg1ZCUUN8d?= =?us-ascii?Q?/c3X5E/kg4RxDEPcYP2JiYj0B7WppehYEqqDiZgZjaeRLsvG2cNfhAFCJzNK?= =?us-ascii?Q?seZVtrRy9eDddZVJ5mFGrfhNdzW/bDGwbdBLGvZ1wPGHhT5aceUsoQ+UwJ9Z?= =?us-ascii?Q?MMuy8K2hTfaC1sIyXHdziW/0Ephhz37TFRfinXsYFqYJOSRiFYZOBZE/pav8?= =?us-ascii?Q?xy/F/kgR9xQP62ji77gaZIvfiBONgszfm3hDtETyt6UQANWlwkNF25G5jIuM?= =?us-ascii?Q?VQPX36b1keplzV0w+pxrdsWvl/3sk4izEv0VEGgy2Aj8/nxAmlILTV1aE0FG?= =?us-ascii?Q?Kt5sNGNVEPlN1Qyb6GP0I1yTCBk/mauHjvRfPYa7RH7OHguf42z7dZkTFetN?= =?us-ascii?Q?o3ZWQv/7mxDhj676d9vHebaIs57Z5g2BkoDMgiZvGbdyYSx+Rfx8tkvcvCZa?= =?us-ascii?Q?ibWsdqtVpx0HabvjxkjUFgpk7x38TzX46diRjWAobfSRdx5gsROS3TBFNFXf?= =?us-ascii?Q?uccKIKkjsIhFo6pBqDPHRfmjNoLnaWmd+0LopHM32Wqdte7g0Kd3j3JqhYc6?= =?us-ascii?Q?U/t1hM+fEFNfhxBRjca5yhqcxDLGCfsYiJjeT7wIuhFYRaJR+YGg7vtGrz9G?= =?us-ascii?Q?bLCPgphVhj4sSx/5RZw6/Zn1WF3r/dDOE91HCV0+r1ema2KW5NnTyReJ0lM1?= =?us-ascii?Q?14Sb9CONA=3D=3D?= X-Microsoft-Antispam-Message-Info: aTsRAZRRVIEv4jV/+VLQWm/AX/arLnfNiHG9mlMy51/nz21NxxU5kkPA9eXPlG2c6uxd8O2rfxfdRkQouBmqJ2bJs7J+ZqqcfQItWwyVmC3FctWInJvYxn2vYjbwMvY6aNb6gIyLvbAwUyul6dMXkRPedPVwd5A5ECv0D6gzc7L/ehk2KMNui5A3BBGbeX8JL1iPDbeNtahX/+n/nDag9sly0/wS8W4NdeNoXuCrtx/iVaJR/YkNHyd6ujbiBImW5aHu72Pf3Kv78C+A2LD3teP630XZJ0zkMLQHCNthi0+W1g4K+VlFYqRCTh7+AhYyainiUQMVmyd3nNuo80+AXw== X-Microsoft-Exchange-Diagnostics: 1; VI1PR08MB2991; 6:NLEG8+YStkgdwDfH7ktpTh//2tg/n2tJAlpKI2eU42tKipp7uWTfIUNEnml3AtLj2XjCu00bAZCnQdwNihIErLpmkdHTTi7HC1Qn9JMhZh4CFFT1iVI4+koATQ0cZR33X0fj/WjNxvgUeLggGP6/JMNvSGSN5Lf8EaVGNIrMMH9hCqUkcBConCXsUV8Y6oTYJwNoePaCC9E+xOpZFTOhTtm4PgX32xbp4h6ojkzBT5rdI93SYUIwSIgl0azUO9a5D+HLri+KjTJpREVWoy4/o0JJ/9yeklhURXUz3iqARtpxYMm1OiN01eX8wzfmSvlojv/R8aWQhO7udxWwEp7TxAy3CyIBJlN3DBPf+AJM3ylhCzl8sBlZnfBNFD99QTYgoFuIxeL3BNjSjLl/OzE45af5WS7gdoaUgkGskYezTsRnYF9i7uqAlSrNv2H5wJPkO9eHxV1I+kygUvxEQD7WNA==; 5:zFSiFYjmNShhqL9D1vs10pDH02eWEHqinLviThVttZtuc0D+hoksNtk8CXzZ5OEdNeTsmJ368uom8poGezLlCYgl5SoKfix7aE9ioL3ZpIVrrdrjM0c8/6qL7ylkqceUkWrQZf3hZj0Usz8cCWDClZvOSwzZZ4hbyU8Q+P9BbvQ=; 24:CoECpBrZ2gADIMMbkGUQg8kKXQv/ipL2f4NCJJCFw8KxOTEHvK3x3kGVJs1PQq3rzvtQqL08nf71+I5u8SquXTQTUVQaIvLqyfD4HphMTOQ= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; VI1PR08MB2991; 7:a1dZyiOlHGUjwSek1JpEVsv0EirM+x9h1zmOcIRqr1Bo42fxIebAmx7uv3NCgy5Eb6l+AaPH5ttc9cDgAvgg8q3ThNM6Ozr/6HLvdsIKmoNXMNyTfdFRkrSHSVjD0FY8ASNABQnjg0h4zRO05fK06hYm2YFrZs4w0yI0HhioQeP0A5wM4+rn0F6Lni+wRNZZWn4YEwamZj6jelIis2Y3e5/UlS1ub5ZmSfsJty1A5ybUQZb3wpUCaNp6lgJlqu+e X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Jun 2018 08:07:51.5892 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 7891ca62-d240-4b85-a627-08d5d5bbc127 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR08MB2991 Subject: Re: [PATCH v3 00/17] *** Standalone Management Mode Core Interface for AARCH64 Platforms *** X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 19 Jun 2018 08:07:56 -0000 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Hi Jiewen, On Mon, Jun 18, 2018 at 03:12:46PM +0000, Yao, Jiewen wrote: > Yes. I think so. > > However, I found the V3 just contains partial of the patch. It is hard to find some V2 and some V3. > Also this series includes multiple package. We need different package maintainer to push different patches. > > I recommend we do in this way: > Please split this big one into 3 - BaseTool patch, ArmPkg patch, and StandaloneMmPkg patch. > > Then BaseTool maintainer can push BaseTool patch, ArmPkg maintainer can push ArmPkg patch, and I can help push StandaloneMmPkg patch. Thomas and Sugosh will refactor the patches and repost since Supreeth is on holiday. I started late but would like to finish reviewing the Arm specific bits in the StandaloneMmPkg before they are merged. Will not take more than 2 days. Could you please merge after I ack. Hope that is fine? cheers, Achin > > Thank you > Yao Jiewen > > > -----Original Message----- > > From: Thomas Abraham [mailto:thomas.abraham@arm.com] > > Sent: Monday, June 18, 2018 6:07 AM > > To: Sughosh Ganu > > Cc: Supreeth Venkatesh ; > > edk2-devel@lists.01.org; Yao, Jiewen ; Gao, Liming > > > > Subject: Re: [edk2] [PATCH v3 00/17] *** Standalone Management Mode Core > > Interface for AARCH64 Platforms *** > > > > On Wed, Jun 6, 2018 at 4:50 PM, Sughosh Ganu wrote: > > > On Tue, Jun 5, 2018 at 3:43 AM, Supreeth Venkatesh > > > wrote: > > >> *** > > >> This patchset v3 contains only the patches that got feedback/comments > > frome the previous revision v2. > > >> The patches are > > >> [PATCH v3 06/17] StandaloneMmPkg: Delete StandaloneMmPkg file. > > >> [PATCH v3 13/17] StandaloneMmPkg: Add an AArch64 specific entry point > > library. > > >> [PATCH v3 17/17] BaseTools/AutoGen: Update header file for MM modules. > > >> > > >> Changes Since v2: > > >> (*) Address feedback provided for the commit "BaseTools/AutoGen: Update > > header file for MM modules." > > >> (*) Edit parameters for the StandaloneMmCpu Driver in the commit > > "StandaloneMmPkg: Add an AArch64 specific entry point library." > > >> > > >> Changes Since v1: > > >> (*) Reorder and Reword commits. > > >> (*) Reorganize structure of StandaloneMmPkg and rename libraries. > > >> (*) Address Review Comments from Achin, Jiewen and Daniil. > > >> *** > > >> Supreeth Venkatesh (17): > > >> 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. > > >> StandaloneMmPkg: Delete StandaloneMmPkg file. > > >> StandaloneMmPkg/FvLib: Add a common FV Library for management > > mode. > > >> StandaloneMmPkg/MemLib: Add Standalone MM instance of memory > > check > > >> library. > > >> StandaloneMmPkg/MemoryAllocationLib: Add MM memory allocation > > library. > > >> StandaloneMmPkg/HobLib: Add HOB Library for management mode. > > >> StandaloneMmPkg: MM driver entry point library. > > >> StandaloneMmPkg/Core: Implementation of Standalone MM Core > > Module. > > >> StandaloneMmPkg: Add an AArch64 specific entry point library. > > >> StandaloneMmPkg: Add CPU driver suitable for ARM Platforms. > > >> StandaloneMmPkg: Describe the declaration and definition files. > > >> ArmPkg: Extra action to update permissions for S-ELO MM Image. > > >> BaseTools/AutoGen: Update header file for MM modules. > > > > > > Tested all changes for RAS error injection and error handling on > > > SGI575 platform. > > > > > > Tested-by: Sughosh Ganu > > > > > > -sughosh > > > > There have been no further comments on this series. Can this patch > > series be merged? > > > > Thanks, > > Thomas. > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel