From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from NAM12-BN8-obe.outbound.protection.outlook.com (NAM12-BN8-obe.outbound.protection.outlook.com [40.107.237.128]) by mx.groups.io with SMTP id smtpd.web10.5657.1586319112474342435 for ; Tue, 07 Apr 2020 21:11:53 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@microsoft.com header.s=selector2 header.b=GXAkbUfJ; spf=pass (domain: microsoft.com, ip: 40.107.237.128, mailfrom: bret.barkelew@microsoft.com) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=BkBG8F+N9NuImcM2KdTQJedjZq/tHWuugMn9srj1eiJm7NbTQud9HlqHGYRmb1OlIJEgPUVsNYbLjyCeI3V38u/Bxku1gy6zQYsQxDbnHWUU6lG/qjqCg9JZdprqt3B1h0lrdxpcw8CO7St1NBVjVfhjo87UHgv1BuX28vow41KN6i/nx9mG1xoISZvJmTzYtmeAtDcz7CWskacxsB7sVK3i0etEt4t+Fu4XMQW3DpOnfrsLanECJ3slX/cZw7yuCDjZfJTmpZP39WJe6AZlP/VtOpJI4BvZ9gEsrHZG48pEmttyR2dE2WL0TUGVVCbhFAJi/eV2nNX3JWyMfDMqOQ== 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=Gs8H7pLR1w0hs7KQmAg9sFL6Ec4AnF/7Dlj31LTGtS4=; b=NMv9CD1n7Ec04j0gghzBa9KbkbxacMBcjrTiN8KQlFoWWe+u12ubW7mw0OugWdDq6m1N86AJenYhYF1Yi4F4th3ORPhR78w8+BbCeTwp9EDkkKPl5UMmSnpbuqW3p7zrjzUhvDYbgprUHg41crgv46KMnmcoqWg3jhJBH41bOaJooijkWw2i8sFLH6iA+wWLroc4pZ90xnXC4Vm26lLcWe9+WvZIEqyTMcMhF7GIDkoGc1mfEJXSHU3RYlw3pGSCzq7Rj0DUDSHz67+jyeTjRqMtj+DYlyiWRJtTlvPpVM8VOdq2ZyZh7yldnwlcLxxEEwk3u8tcowqBxEO1zO5RgA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microsoft.com; dmarc=pass action=none header.from=microsoft.com; dkim=pass header.d=microsoft.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Gs8H7pLR1w0hs7KQmAg9sFL6Ec4AnF/7Dlj31LTGtS4=; b=GXAkbUfJtvD3j4PLvPs/EUshAvmRKkCnCkheAMUubMsgIBAj39kMRR/8Zhqk1S/g3XGmQqiMl2cW0L9ii7DqFm+7k8a/IW1vVlUrPB6bI4M4QEWReVe3ok71OQxsbda1RhrrYT71w6P/yr/3AlpMaD6QZsWAcd/taddZ8yZknSg= Received: from CY4PR21MB0743.namprd21.prod.outlook.com (2603:10b6:903:b2::9) by CY4PR21MB0773.namprd21.prod.outlook.com (2603:10b6:903:b8::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2900.2; Wed, 8 Apr 2020 04:11:50 +0000 Received: from CY4PR21MB0743.namprd21.prod.outlook.com ([fe80::90d:10d9:c5bc:5318]) by CY4PR21MB0743.namprd21.prod.outlook.com ([fe80::90d:10d9:c5bc:5318%12]) with mapi id 15.20.2921.009; Wed, 8 Apr 2020 04:11:50 +0000 From: "Bret Barkelew" To: "Lin, Derek (HPS SW)" , "devel@edk2.groups.io" Subject: Re: [edk2-devel] Basetools fail without error message due to PCD type differs in architecture. Thread-Topic: [edk2-devel] Basetools fail without error message due to PCD type differs in architecture. Thread-Index: AQHWDU8iUFiR9f4mh0qhmFhpdRb6XahunNa+ Date: Wed, 8 Apr 2020 04:11:50 +0000 Message-ID: References: , In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: yes X-MS-TNEF-Correlator: msip_labels: MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Enabled=True;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SiteId=72f988bf-86f1-41af-91ab-2d7cd011db47;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SetDate=2020-04-08T04:11:07.2644587Z;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_ContentBits=0;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Method=Privileged authentication-results: spf=none (sender IP is ) smtp.mailfrom=Bret.Barkelew@microsoft.com; x-originating-ip: [71.212.145.195] x-ms-publictraffictype: Email x-ms-office365-filtering-ht: Tenant x-ms-office365-filtering-correlation-id: dd256cad-2c71-45aa-714c-08d7db72f6c3 x-ms-traffictypediagnostic: CY4PR21MB0773: x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:8273; x-forefront-prvs: 0367A50BB1 x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:CY4PR21MB0743.namprd21.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(10019020)(4636009)(366004)(396003)(346002)(376002)(136003)(39860400002)(33656002)(9686003)(26005)(5660300002)(81156014)(71200400001)(110136005)(8936002)(76236002)(82950400001)(81166007)(82960400001)(316002)(296002)(8676002)(99936003)(55016002)(966005)(2906002)(15650500001)(52536014)(64756008)(53546011)(10290500003)(6506007)(86362001)(76116006)(7696005)(186003)(66446008)(66556008)(66616009)(66946007)(8990500004)(478600001)(66476007);DIR:OUT;SFP:1102; received-spf: None (protection.outlook.com: microsoft.com does not designate permitted sender hosts) x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: y0oVlQEindypM96NaM/CaYJhnPkDQ1FbYzRd9xSf2YxluiNzRar+IxGDGKhIovnljMVnu9h+GCBGOaA8YcdxlP/JVJLRYPFPTOalBO1DOc+0BVQ4wygWQu9NgqwrAucb0eWEfPZR/CrI9KmxvrD2MbCFVmlimJMqToYmkGcyeWFqmgpvRxTeBWma+9+nCl8rqAaCGdwWF6yL9fePdTgcXJRj5ZGRt7tB8EQAdVYfkXmCSw5vW+h522ho0WeBf4ktMvRM8pERz+CffNJhOMJEwPG2El2zE/s8GHg/s4RA6hcgQ4fF4U4SdppOIQOjg1PvMqCjaqP/gOKG3ClFWylN4AsOhaW3OkDB2/4QD2xrvUbXkAtoZ3r5BGh9baYmoHt7N2QWgFwXRQJvUiFqz+zI92R3Sv2izgW1e4AeeHOdlCYdltGKRQXD5Px1/OM3URYNop6A7pKr3BUBfVa5Z6AMxCmzSDjCNXh75qQGiHTbjCw9eYl5SgkE1c2ZZio2/ESgMR9dMAGNx3IcdrkgoDa3TA== x-ms-exchange-antispam-messagedata: g1I0xMak/8+s55wR+SUMgE3A4noUFX3U9AGBDhw3EPybAQCnNMBvIaD/LThd/eSUFtoTSjaj0vmGmNPFDHP9Mt4YfkEsvCYJ9ZXvNC9LcTrsoBcDUVKaYYlPYSDHuyGMAEMElSoREK69zjz5CJhoAQ== x-ms-exchange-transport-forked: True MIME-Version: 1.0 X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-Network-Message-Id: dd256cad-2c71-45aa-714c-08d7db72f6c3 X-MS-Exchange-CrossTenant-originalarrivaltime: 08 Apr 2020 04:11:50.8173 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: eRlTXZvS2LVTjNKixeReMalaIVYV9EF01dBD7g7pxYeuln1mrWUyK5NygvmDEPuH3iaoXAqTI0Gj8IVDMAFdZQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR21MB0773 X-Groupsio-MsgNum: 57047 Content-Language: en-US Content-Type: multipart/related; boundary="_004_CY4PR21MB0743989EE27B2EA630EE4187EFC00CY4PR21MB0743namp_"; type="multipart/alternative" --_004_CY4PR21MB0743989EE27B2EA630EE4187EFC00CY4PR21MB0743namp_ Content-Type: multipart/alternative; boundary="_000_CY4PR21MB0743989EE27B2EA630EE4187EFC00CY4PR21MB0743namp_" --_000_CY4PR21MB0743989EE27B2EA630EE4187EFC00CY4PR21MB0743namp_ Content-Type: text/plain; charset="Windows-1252" Content-Transfer-Encoding: quoted-printable AH! Okay. I would agree that a build crash without an actionable error is s= omething we want to fix. - Bret From: Lin, Derek (HPS SW) Sent: Tuesday, April 7, 2020 7:40 PM To: devel@edk2.groups.io; Bret Barkelew Subject: RE: [edk2-devel] Basetools fail without error message due to PCD = type differs in architecture. Hi Bret, Thanks for your feedback. I=92m very sorry that I didn=92t describe it cle= arly. The Basetools just =93crash=94 in the configuration. The Basetools crash w= ithout any information, then build stops. I=92ve looked into the BaseTools code, the python code crash, during the g= eneration of PCD database for IA32. I=92m not sure if the PCD spec allows different PCD type in different arch= itectures. If the spec allows it, the build needs to be fixed and make sure it works.= I remember the dynamic PCD database carry from PEI to DXE. And I don=92t t= hink that work if PEI is FixedAtBuild while DXE is Dyanamic. In my opinion, this configuration should come to a build failure. And defi= ne the error in PCD spec. Thanks, Derek From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Bret= Barkelew via groups.io Sent: Wednesday, April 8, 2020 5:48 AM To: devel@edk2.groups.io; Lin, Derek (HPS SW) Subject: Re: [EXTERNAL] [edk2-devel] Basetools fail without error message = due to PCD type differs in architecture. This seems like a legitimate configuration. I don=92t know that it should = be considered an error. A warning, maybe. This is also what the build report is perfect for finding. Did you check t= he PCD build report when debugging? - Bret From: Lin, Derek (HPS SW) via groups.io Sent: Sunday, April 5, 2020 7:55 PM To: devel@edk2.groups.io Subject: [EXTERNAL] [edk2-devel] Basetools fail without error message due = to PCD type differs in architecture. We've found a Basetool issue. When a PCD is FixAtBuild at IA32 and Dynamic is at X64, there is no error = message in this situation and it's very difficult to find the root cause. Here's example to see the issue: https://github.com/howdou/edk2/commit/333c5f33adc652308355ec100f7f385d5523= 92fa I think it is not allowed to have FixAtBuild at IA32 and Dynamic at X64. B= aseTool should have an error message for this. --_000_CY4PR21MB0743989EE27B2EA630EE4187EFC00CY4PR21MB0743namp_ Content-Type: text/html; charset="Windows-1252" Content-Transfer-Encoding: quoted-printable

AH! Okay. I would agree that a build crash without = an actionable error is something we want to fix.

 

- Bret

 

From: Lin, Derek (HPS SW)
Sent: Tuesday, April 7, 2020 7:40 PM
To: devel@edk2.groups.io; Bret Barkelew
Subject: RE: [edk2-devel] Basetools fail without error message due = to PCD type differs in architecture.

 

Hi Bret,

 

Thanks for your feedb= ack. I=92m very sorry that I didn=92t describe it clearly.

 

The Basetools just = =93crash=94 in the configuration. The Basetools crash without any informat= ion, then build stops.

I=92ve looked into th= e BaseTools code, the python code crash, during the generation of PCD datab= ase for IA32.

 

 

I=92m not sure if the= PCD spec allows different PCD type in different architectures.<= /span>

If the spec allows it= , the build needs to be fixed and make sure it works. I remember the dynami= c PCD database carry from PEI to DXE. And I don=92t think that work if PEI = is FixedAtBuild while DXE is Dyanamic.

 

In my opinion, this c= onfiguration should come to a build failure. And define the error in PCD sp= ec.

 

Thanks,

Derek

 

From: devel@edk2.groups.io [mailto:devel@edk= 2.groups.io] On Behalf Of Bret Barkelew via groups.io
Sent: Wednesday, April 8, 2020 5:48 AM
To: devel@edk2.groups.io; Lin, Derek (HPS SW) <derek.lin2@hpe.co= m>
Subject: Re: [EXTERNAL] [edk2-devel] Basetools fail without error m= essage due to PCD type differs in architecture.

 

This seems like a legitimate configuration. I don= =92t know that it should be considered an error. A warning, maybe.

 

This is also what the build report is perfect for f= inding. Did you check the PCD build report when debugging?

 

- Bret

 

 

We've found a Basetool issue. 
When a PCD is FixAtBuild at IA32 and Dynamic is at X64, there is no error = message in this situation and it's very difficult to find the root cause.
Here's example to see the issue:
https://github.com/ho= wdou/edk2/commit/333c5f33adc652308355ec100f7f385d552392fa

I think it is not allowed to have FixAtBuild at IA32 and Dynamic at X64. B= aseTool should have an error message for this.

 

 

--_000_CY4PR21MB0743989EE27B2EA630EE4187EFC00CY4PR21MB0743namp_-- --_004_CY4PR21MB0743989EE27B2EA630EE4187EFC00CY4PR21MB0743namp_ Content-Type: image/png; name="ED2113367E40488891C9418E77E74F95.png" Content-Description: ED2113367E40488891C9418E77E74F95.png Content-Disposition: inline; filename="ED2113367E40488891C9418E77E74F95.png"; size=156; creation-date="Wed, 08 Apr 2020 04:11:49 GMT"; modification-date="Wed, 08 Apr 2020 04:11:49 GMT" Content-ID: Content-Transfer-Encoding: base64 iVBORw0KGgoAAAANSUhEUgAAAsQAAAADCAYAAACUJxmmAAAAAXNSR0IArs4c6QAAAARnQU1BAACx jwv8YQUAAAAJcEhZcwAADsMAAA7DAcdvqGQAAAAxSURBVGhD7dYBDQAwDMOw8ic1BEdwMJ90GrWl cEjm3CdJkiS19oc4AADQyhADANArWRCzbWwv0sXDAAAAAElFTkSuQmCC --_004_CY4PR21MB0743989EE27B2EA630EE4187EFC00CY4PR21MB0743namp_--