From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from NAM04-CO1-obe.outbound.protection.outlook.com (NAM04-CO1-obe.outbound.protection.outlook.com [40.107.69.65]) by mx.groups.io with SMTP id smtpd.web11.23.1588771184700676584 for ; Wed, 06 May 2020 06:19:45 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@amdcloud.onmicrosoft.com header.s=selector2-amdcloud-onmicrosoft-com header.b=15LJ8nvB; spf=none, err=SPF record not found (domain: amd.com, ip: 40.107.69.65, mailfrom: thomas.lendacky@amd.com) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=jXNJp1ZHpC57GU59Pgv3DxerXMeSCGqzawFu3ESG+iYZuz1+RPLu29AfKXXzc4+E650ie57bQyZQ6gheusdH8XC8I+lFcV0tSxyQIGOmXAZeWQyTKxdREVLnFvbvTt2OwvJ8ieIVfeuhi11n6BKUOaBTAmWSmISbdoUu0ZV96cIubwvhqWJcYWURh9zmz6asbxY6GrgpZzUTyfO/JWXsk3QvcyPbL9QpvYaNFfh/PlSupFXv85b01d1Ak6vEa2i++DXD+jbDhP7vuG6HgfJuby2FNtazygcHlms0ydCoT/PVfvoRdTyf0SGIHg37d+b7LQhwJIghM9jSFbdnMN/oOg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=2Y8ftKEswxRghWvje4KiGTCgaRitbfMmGYyDEAUMUcU=; b=F5YzrqSzneBFiGT4qAVtwldKDWaVEj8zeVhIwDld/z/TyRrCumQRng2lscImKL0Y4ecKNPBXsWDgjVZq7i46qNOrVlJsZB0zpz3xPdL/gCx+4YYWcZsQM5bbeyxgtvZTAHsP4JeHEIVrEDh+UOips2BtXvLWVySB5PRsE70PaYFL5FC6380PX5U6rCL/Ag22n+Ig7eeCa1vrOiCUnz594WE+9wiA/BiViaTT1AFDVwq7GEJzES+0BoHBGd7MdSmTliGM5q+SmS04CPwsSUIXQrs8Zfy7gz8ePg2sI9krQibhYJn98XZjXQKhaO+AAMBtlWPLy6ErJzeoGHeruDsqxQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amdcloud.onmicrosoft.com; s=selector2-amdcloud-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=2Y8ftKEswxRghWvje4KiGTCgaRitbfMmGYyDEAUMUcU=; b=15LJ8nvBPDTNPFp9sjr6FM6YCVc9wFanx4rgCih0DcR8XY9e60mNM5SEpds/uIZqVnlnMpEFjIFBFzI6xtJA3sAuR48Uxu6ARIvTzQVgB3FGhApADm6PqR77ByKEn0ksNiQIiTL08612j8v8H4/wDx+w4Me+ualqXEqLcPfi9oI= Authentication-Results: intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=none action=none header.from=amd.com; Received: from DM5PR12MB1355.namprd12.prod.outlook.com (2603:10b6:3:6e::7) by DM5PR12MB2421.namprd12.prod.outlook.com (2603:10b6:4:b4::26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2979.29; Wed, 6 May 2020 13:19:40 +0000 Received: from DM5PR12MB1355.namprd12.prod.outlook.com ([fe80::4ce1:9947:9681:c8b1]) by DM5PR12MB1355.namprd12.prod.outlook.com ([fe80::4ce1:9947:9681:c8b1%10]) with mapi id 15.20.2958.030; Wed, 6 May 2020 13:19:40 +0000 Subject: Re: [edk2-devel] [PATCH v7 01/43] MdeModulePkg: Create PCDs to be used in support of SEV-ES To: "Dong, Eric" , "devel@edk2.groups.io" , "lersek@redhat.com" Cc: "Justen, Jordan L" , Ard Biesheuvel , "Kinney, Michael D" , "Gao, Liming" , "Ni, Ray" , Brijesh Singh , "Wang, Jian J" , "Wu, Hao A" References: <3af2e4a8-fa4d-bb29-a282-c406ada7cf06@amd.com> <66c3ab32-0b1c-676b-2924-f14214be5de6@redhat.com> From: "Lendacky, Thomas" Message-ID: <226c8566-6550-613c-e07c-636686a851f9@amd.com> Date: Wed, 6 May 2020 08:19:38 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 In-Reply-To: X-ClientProxiedBy: SN6PR04CA0074.namprd04.prod.outlook.com (2603:10b6:805:f2::15) To DM5PR12MB1355.namprd12.prod.outlook.com (2603:10b6:3:6e::7) Return-Path: thomas.lendacky@amd.com MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from office-linux.texastahm.com (67.79.209.213) by SN6PR04CA0074.namprd04.prod.outlook.com (2603:10b6:805:f2::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2979.26 via Frontend Transport; Wed, 6 May 2020 13:19:39 +0000 X-Originating-IP: [67.79.209.213] X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 1b1a804f-c989-48be-2dd5-08d7f1c02205 X-MS-TrafficTypeDiagnostic: DM5PR12MB2421:|DM5PR12MB2421: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:5236; X-Forefront-PRVS: 03950F25EC X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: wgXKRgCRRSBpfuuuD5Y8RM7Dn309WTU+mvdARCJue6fZM/ojSw3oKyZD4dLmaheNkoeKxkL2h5bIUN6loR7arq0T2WNrUBWgPWTE24jLhBtq457R9oGrFTZmRsTZ6Md6hRDocrxBfRXxqZMSb33MqYrmdhqQkVWnsPksWv3sk95OdehdIA3FA621sbattDNrcl9NZiExFiyo+b5Binh308oL7+BygVfxAb6+cSXW3ChIZH1q8Q5xr4NediM5e6ydw2xL4xLDlAZtH76RbeG/t9RrH5dzSCszl+kkRiENXxRg0HSdbLpMn777JsZnSr4AOEZaUKQ098C/AyCI7xNJJmlXtHmIL8GG/dZb/BhkeT8BOs4PBChPw+/ftlL/vB3tcq0HS4kIx3c2//6bFnYzxjKUpLVLJ5xl1/3wHt25ut3hEf+66Qei7udoxntl/0x8mDLFQMNxwV02+Z/CnxjJlYD6Yc9JPtAbajvpYic6s706DbixJOuOv/rl4GFPls2lOMxqfJ1l3CrZP3AFB3F+vVbcuziLW/Iw5gXf+jNuSJ1vCqfwU9/NQpJ1WgPdyg+/+5Xzr229YnOmruZYZWjnFQRnGv+jlWE6IKiXreoO1K7tw47URHMHZoP17hQ1oJN7BY35YIWWxpwE8wWf+IeYIA== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DM5PR12MB1355.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(4636009)(39860400002)(396003)(366004)(136003)(376002)(346002)(33430700001)(31686004)(478600001)(956004)(2616005)(6506007)(53546011)(16526019)(54906003)(186003)(36756003)(316002)(2906002)(86362001)(4326008)(33440700001)(7416002)(6486002)(31696002)(966005)(45080400002)(26005)(110136005)(8676002)(8936002)(66556008)(66476007)(5660300002)(52116002)(66946007)(6512007)(43740500002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData: GSjjHYquC/RYczzTpXlTIZIvhtOEzOq64s5bGBO5B5HgUt1cGnbqsVy6u/b1Ybeh/D3n4auKRn7x/xvECZ3Hz1M1XbVW5ffsvitNx+Jjwor6wsgnmlq26UiFKx+9W2OpenkmxSguuyfG3LRF07WSPrxSpTL3sTeF3Blzmj3DL7PF2fdbCteIZuBe2RCFAtOc/7u9Phee0IgK+dXBR43cO7qXsERHu5GXUuBzr/pTlLLi5QZEb8A6jkJl1x5J6OVcuP0Ziebv8R2pYJND2kzoCZElGisUK5j1eVaYuhnTv2eTACGeooK/KGohyiOKq3SXamkj09O1KS0yC8xiYnSOGyiqCA4n+vMqCq1/2QcaRNuE+kLfyOxJPirQfRfyzCbhZZTR2OjSqO5dAqGMP+blmUX6F52YcvStg+N/yRFF6jXBsKCe8p2AyoTfECKQyZ623GmAB8jTq/7QXVZWOK8GjBJNvCyklOkLmbih20f6z0y1ebRtC158ybtc7ZOAtvvBOBlzddOV/+CG9uWCuna5mQ39slx1qqvtCGUPJEsa6PefIbjYWNd1jG0lJvyNx8KzDeGu0quUfvacyAbgXrM2ZT/AlC5GQfRx/GfVLhPzEbOUKBfYAIAwpyma7MK6vJ/IsjDxT2VeHBDCB79OqhyEebElJqZvP3J89GJseIoVKFb4ylaYUcCTAMHEbutAhCIf3mWNSb6zqfVPaP7w3d44udQ8NdYIA27kxFAw/HbKBsRgDvrbcAfuye8v2E5M6Sk9unkjDxN0D5F5RQ5on9ESL/OnVxcbBsjwhe1R+lkd7ZY= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: 1b1a804f-c989-48be-2dd5-08d7f1c02205 X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 May 2020 13:19:40.5922 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: Oykf7uJuxQTOnnKKn/gDiny0R1sfiwrTb2K92PZ7XNJ4EDrIxUc9dzJ6ObGMGrJcpUWQk8TWnlcneUT1Vy3E/Q== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR12MB2421 Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit On 5/5/20 8:53 PM, Dong, Eric wrote: > > >> -----Original Message----- >> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of >> Laszlo Ersek >> Sent: Tuesday, May 5, 2020 11:30 PM >> To: Tom Lendacky ; Dong, Eric >> ; devel@edk2.groups.io >> Cc: Justen, Jordan L ; Ard Biesheuvel >> ; Kinney, Michael D >> ; Gao, Liming ; Ni, Ray >> ; Brijesh Singh ; Wang, Jian J >> ; Wu, Hao A >> Subject: Re: [edk2-devel] [PATCH v7 01/43] MdeModulePkg: Create PCDs to >> be used in support of SEV-ES >> >> On 05/04/20 18:41, Tom Lendacky wrote: >> >>> Is there an easy way to run everything that this link points, too? Is >>> it just creating a pull request that does this? I don't want to take >>> up a lot of your time, so if there's some documentation on how to run >>> an integration test to find and fix issues like this, just point me to it. >> >> Just create a pull request; it will set off CI, and you can review VS build errors >> there (if any). >> >> Your PR will automatically be closed (rejected) regardless of whether CI >> succeeds or not. PRs are merged -- in fact, *auto*-merged, by the "mergify >> bot" -- if and only if (a) the CI run succeeds, and (b) the PR has the "push" >> label set. >> >> And only edk2 maintainers have permission to set the "push" label. Any PR >> without the "push" label qualifies as a "personal test build". So you can freely >> experiment with PRs, because you can't (even unwittingly) satisfy condition >> (b). >> >> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftianocore%2Ftianocore.github.io%2Fwiki%2FEDK-II-Development-&data=02%7C01%7Cthomas.lendacky%40amd.com%7C9cff3475aff84a95728508d7f1604c99%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637243268217382019&sdata=3%2FIKB174QaVLaqO0u1gdrL0izXmhEZ%2Byvj3iC13UYBc%3D&reserved=0 >> Process >> > > Thanks Laszlo for your explanation. > > I found this patch serial is incompatible for the existed platforms. Can you help to fix the build failure for these platforms in https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftianocore%2Fedk2-platforms&data=02%7C01%7Cthomas.lendacky%40amd.com%7C9cff3475aff84a95728508d7f1604c99%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637243268217382019&sdata=jU0qrB%2BV6ZvFmPzjcxGo9o2Pu1%2FrhRW0gUZTMv%2BiXDQ%3D&reserved=0 > I have fixed all of the build issues associated with the VS compiler using the pull request method that Laszlo mentioned. I then successfully built the RPi4 platform under GCC (build -n 32 -a AARCH64 -t GCC5 -p Platform/RaspberryPi/RPi4/RPi4.dsc) using the AARCH64 cross compiler. Is there a particular platform that experiences an issue or are the failures related to the VS compiler errors that my next series will have fixed? > I think you also needs to add an wiki page to explain what need to do if an platform needs to integrate your changes, also it's better to explain this feature in the page. > https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftianocore%2Ftianocore.github.io%2Fwiki&data=02%7C01%7Cthomas.lendacky%40amd.com%7C9cff3475aff84a95728508d7f1604c99%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637243268217382019&sdata=xLkoV4zWhxtsbqszqPc0lEAl%2BYLL%2B2wg1nIXql8a64E%3D&reserved=0 I don't see any platform other than OVMF using this feature as it is a virtualization feature. Having said that I can add an explanation of what is needed should another virtualization platform be created under EDK2 that wants to support SEV-ES. And, as you said, I can also explain the feature overall on the page. > > > If you want to include this change in the next edk2 release, you need to add one item for it in the release plan page, sample can be found in below pages: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftianocore%2Ftianocore.github.io%2Fwiki%2FEDK-II-Release-Planning&data=02%7C01%7Cthomas.lendacky%40amd.com%7C9cff3475aff84a95728508d7f1604c99%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637243268217382019&sdata=kcDVjYHMS9bRRZOlKEk5ynFNT39AnxchJAMak%2Bn870I%3D&reserved=0 Thanks. Is there anyone in particular that I need to request this feature be added? Thanks, Tom > > > Thanks, > Eric > >> Thanks, >> Laszlo >> >> >>