From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from NAM11-CO1-obe.outbound.protection.outlook.com (NAM11-CO1-obe.outbound.protection.outlook.com [40.107.220.108]) by mx.groups.io with SMTP id smtpd.web11.22769.1585168465839476138 for ; Wed, 25 Mar 2020 13:34:26 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@microsoft.com header.s=selector2 header.b=NDggd57p; spf=pass (domain: microsoft.com, ip: 40.107.220.108, mailfrom: bret.barkelew@microsoft.com) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=XjUqCcNq6eleCs0rbCdv2juTafHXg3pZB80G0mL/uTOvq0QotFh8hHk6cki7xpUpxCnEYmoZ5s+xvBt8NkTlCACmu2BF+zOvn4lCSyjIF/naII9Y6zHn+LK1j07x7e/oUWhWxxKYWnoDbqaMqE/0pWuH9QYaUQQvVEQChvg42dUjqQ+61/w6bMucviYBkYuB4abBfXRbearBMdNQSNIB/INIRDN7FRcIkOw6dBea1Kj/cFtiwjPOpI55oRQeqxzTPC1KblSZlrOLGhW3lzitJWJnSylevNoGMuDBJGtcJvIY+QbsZ+lQUn4evBUXd5OZzwUumQ5WvTiF6ZDHGOeodw== 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=7ZEyKG8z1446l8C/j++sc4CiMhYMdHqgXMO/5SkkVdY=; b=S9P2DVm6Or0xwj0C1G9WJB+PQ+/Icf1Xv39fVOTczDzlJosQNLIJSIurB17LL5edFXoDePPe2xhD24vE59u/lrokNsK520rgKUjZYDOJVyVXC+KxuHdGPZrt9G/wQW5FFQR/9knWpl153OcqbUksrbYIbaFikHUvKjiNiqDdVHrERxPd+7EAR7JatixO16ZdDbo0n1Wz6BslTL3I1sIb2kUCi9/62FYU8MQNdkyKsxcMGkCtSgXR4k8zTj0srSw+OIJKZlZ5BgAmEoGqtmBxh5UFRAzjrnnAlT+qaDS+O9Z0r1N6xMdf7bIndQoqOejYgoqa3+glMFEvx1xhuB0Maw== 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=7ZEyKG8z1446l8C/j++sc4CiMhYMdHqgXMO/5SkkVdY=; b=NDggd57ptbBuwfJMnaJIF3mAv2XUqwGWRwiCWWG1/JZjhdFJDXs1nd9JDPs50qYLgojOuyaIzZu+J5+KT5x4Mr+k7QvncgnshTObaeAet8cNLHceglJQaKzlOOrrJpLkVzTZvZ+t8WFcSqj6kwK7tu+RV6n/IvghzRSU4fCEMTQ= Received: from CY4PR21MB0743.namprd21.prod.outlook.com (2603:10b6:903:b2::9) by CY4PR21MB1569.namprd21.prod.outlook.com (2603:10b6:910:8d::26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2878.2; Wed, 25 Mar 2020 20:34:24 +0000 Received: from CY4PR21MB0743.namprd21.prod.outlook.com ([fe80::90d:10d9:c5bc:5318]) by CY4PR21MB0743.namprd21.prod.outlook.com ([fe80::90d:10d9:c5bc:5318%10]) with mapi id 15.20.2878.007; Wed, 25 Mar 2020 20:34:24 +0000 From: "Bret Barkelew" To: "devel@edk2.groups.io" , "ard.biesheuvel@linaro.org" , Laszlo Ersek CC: "Ni, Ray" , "devel@edk2.groups.io" , "tigerliu@zhaoxin.com" , "Kinney, Michael D" , "Leif Lindholm (Nuvia address)" , Jiewen Yao Subject: Re: [EXTERNAL] Re: [edk2-devel] Questions about UEFI MAT / PcdPropertiesTableEnable Thread-Topic: [EXTERNAL] Re: [edk2-devel] Questions about UEFI MAT / PcdPropertiesTableEnable Thread-Index: AQHWAmSvDQ7cAIDZ8k+OSdvWzEiN/KhZh2eAgAABoYCAADuelQ== Date: Wed, 25 Mar 2020 20:34:24 +0000 Message-ID: References: <986b51e441804c3ba288a0af210d0f4f@zhaoxin.com> <4c1161fc-2f1a-3a18-fe7e-9a395a5a532c@redhat.com> <734D49CCEBEEF84792F5B80ED585239D5C4B26AD@SHSMSX104.ccr.corp.intel.com> , In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: 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-03-25T20:34:00.9563165Z;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: 31c8f84e-3b59-411f-6822-08d7d0fbe80c x-ms-traffictypediagnostic: CY4PR21MB1569:|CY4PR21MB1569: x-ld-processed: 72f988bf-86f1-41af-91ab-2d7cd011db47,ExtAddr x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:9508; x-forefront-prvs: 0353563E2B 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)(396003)(366004)(376002)(346002)(136003)(39860400002)(4326008)(26005)(55016002)(81166006)(66946007)(186003)(8990500004)(66446008)(64756008)(66556008)(66476007)(2906002)(81156014)(33656002)(8676002)(8936002)(9686003)(10290500003)(53546011)(52536014)(5660300002)(478600001)(7696005)(316002)(6506007)(54906003)(110136005)(76116006)(71200400001)(86362001)(91956017)(966005);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: Ac2XHtyVgbM9CReggzV9JSJaFs5UEOyKKuUgOF7/YXmmvzuEbBJjBBRHwEEhionmHRxPLl1t3hZHa9D27wswxhTAAUuTvQJAv95CXJLb/atuWXCHz5ICppZ/q1+ZCJ+zjUelCGyhJPQ27OH9frHWnidFSKitqVnPzai4B962ZBFtWLc3E6bm5BsfRajXOqZuXYFj04cHyfV07jLg8oEMi/OAVHENanwtJNahakWJoZHdg+vtyikGmE/xeWCj6vSq/zxxmjuIw7GWQP13kR34M3+QiWAAIyoQIUh8Dh6WZoLkAQ5lWFIV+wtGsVkLJzlZAoABDb0jKRs/Dz2qfmyPzXTUH/g9LjnyyPFHfStZaX9958u4nKSCPKAvndLfA6/1oMMS0TVbXYmrWHc4bKIjAP9YrIe/DxMSkNCTlF5bMRtsTp/o0uMTshGv0bgpv2lW4Avh2zuNAriusnQns3Mje72a/u/T+bpo/TgF4fhtm0LwBu8+x+BldcMFNowp/yj2iYqXBPY5aqo3DvrLVzdRfQ== x-ms-exchange-antispam-messagedata: rnjteQzh8VA1bJED+UqAfscWUqbLSvtEsyS/nWdRQ+W+QM4422LidpBfwzo0TUpcqMxQIFwxuH3OXioTupv4kw1QglBrrWkeW5dnBwH/FrPeyBsd/eLuV9bPrudSpU/OTKLedyh53W6Xbh9du8Gv8A== x-ms-exchange-transport-forked: True MIME-Version: 1.0 X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-Network-Message-Id: 31c8f84e-3b59-411f-6822-08d7d0fbe80c X-MS-Exchange-CrossTenant-originalarrivaltime: 25 Mar 2020 20:34:24.3760 (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: iqDXxvhAZSP8wa3RMi1RI87THETf32DX5UMRt+266vuZy3gsXkzkOVWPlgyBXqdkb5HGQrHwdwN2y/f28W6D0w== X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR21MB1569 Content-Language: en-US Content-Type: multipart/alternative; boundary="_000_CY4PR21MB0743267C90E80B7A4D6EB03EEFCE0CY4PR21MB0743namp_" --_000_CY4PR21MB0743267C90E80B7A4D6EB03EEFCE0CY4PR21MB0743namp_ Content-Type: text/plain; charset="Windows-1252" Content-Transfer-Encoding: quoted-printable Agreed. Let=92s please do what=92s needed to put dropping this table on the= roadmap. - Bret ________________________________ From: devel@edk2.groups.io on behalf of Ard Biesheu= vel via Groups.Io Sent: Wednesday, March 25, 2020 10:00:39 AM To: Laszlo Ersek Cc: Ni, Ray ; devel@edk2.groups.io ; tigerliu@zhaoxin.com ; Kinney, Michael D ; Leif Lindholm (Nuvia address) ; Jie= wen Yao Subject: [EXTERNAL] Re: [edk2-devel] Questions about UEFI MAT / PcdPropert= iesTableEnable On Wed, 25 Mar 2020 at 17:55, Laszlo Ersek wrote: > > On 03/25/20 06:17, Ni, Ray wrote: > >> > >> The properties table should not be used. It has been superseded by th= e memory attributes table, per spec. > >> > >> In edk2, the properties table is controlled by the PCD, regardless of= the memory attributes table. > >> > >> In edk2, the memory attributes table is always produced, regardless o= f the properties table. > >> > >> Please see the discussion under: > >> > >> [edk2] [patch 0/7] Add UEFI2.6 MemoryAttributesTable support. > >> https://nam06.safelinks.protection.outlook.com/?url=3Dhttp%3A%2F%2Fmi= d.mail-archive.com%2F1454069539-4056-1-git-send-email-jiewen.yao%40intel.co= m&data=3D02%7C01%7Cbret.barkelew%40microsoft.com%7C46138683ba554847c600= 08d7d0de1753%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C63720752459998854= 5&sdata=3D7M4nzzz%2B8UZ2kb4LHyP%2BfBOWFrv5xFuAT19pdAn3kEE%3D&reserv= ed=3D0 > >> > > ... > > > Do you think we could remove properties table? > > Yes, that's exactly what Ard requested, as soon as Jiewen posted the MAT > series. Back then, Jiewen said that some production OSes were still > using the properties table, and would need time to migrate to MAT. > > The agreement -- four years ago! -- seemed to be that the UEFI spec > should drop the properties table definition in some time, and then edk2 > could remove the reference implementation too. > > See the attached discussion. > > Given that the properties table had been deprecated in the UEFI spec > even in Feb 2016, I think it's now high time to remove it altogether > (both spec and edk2). > > > The existence of both is confusing. > > Yes, very much. > Agreed. We should at least log this as a task, and if anyone has the bandwidth to do the work, plan it for the next stable tag. --_000_CY4PR21MB0743267C90E80B7A4D6EB03EEFCE0CY4PR21MB0743namp_ Content-Type: text/html; charset="Windows-1252" Content-Transfer-Encoding: quoted-printable

Agreed. Let=92s please do what=92s needed to put dr= opping this table on the roadmap.

 

- Bret

 

From: devel@edk2.groups.io= <devel@edk2.groups.io> on behalf of Ard Biesheuvel via Groups.Io <= ;ard.biesheuvel=3Dlinaro.org@groups.io>
Sent: Wednesday, March 25, 2020 10:00:39 AM
To: Laszlo Ersek <lersek@redhat.com>
Cc: Ni, Ray <ray.ni@intel.com>; devel@edk2.groups.io <deve= l@edk2.groups.io>; tigerliu@zhaoxin.com <tigerliu@zhaoxin.com>; Ki= nney, Michael D <michael.d.kinney@intel.com>; Leif Lindholm (Nuvia ad= dress) <leif@nuviainc.com>; Jiewen Yao <jiewen.yao@intel.com> Subject: [EXTERNAL] Re: [edk2-devel] Questions about UEFI MAT / Pcd= PropertiesTableEnable
 
On Wed, 25 Mar 2020 at 17:55, Laszlo Ersek <le= rsek@redhat.com> wrote:
>
> On 03/25/20 06:17, Ni, Ray wrote:
> >>
> >> The properties table should not be used. It has been superse= ded by the memory attributes table, per spec.
> >>
> >> In edk2, the properties table is controlled by the PCD, rega= rdless of the memory attributes table.
> >>
> >> In edk2, the memory attributes table is always produced, reg= ardless of the properties table.
> >>
> >> Please see the discussion under:
> >>
> >> [edk2] [patch 0/7] Add UEFI2.6 MemoryAttributesTable support= .
> >> https://nam06.safelinks.protection.outlook.com/?url=3Dhttp%3A%2F%2Fmid.mai= l-archive.com%2F1454069539-4056-1-git-send-email-jiewen.yao%40intel.com&= ;amp;data=3D02%7C01%7Cbret.barkelew%40microsoft.com%7C46138683ba554847c6000= 8d7d0de1753%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637207524599988545= &amp;sdata=3D7M4nzzz%2B8UZ2kb4LHyP%2BfBOWFrv5xFuAT19pdAn3kEE%3D&amp= ;reserved=3D0
> >>
> >
...
>
> > Do you think we could remove properties table?
>
> Yes, that's exactly what Ard requested, as soon as Jiewen posted the = MAT
> series. Back then, Jiewen said that some production OSes were still > using the properties table, and would need time to migrate to MAT. >
> The agreement -- four years ago! -- seemed to be that the UEFI spec > should drop the properties table definition in some time, and then ed= k2
> could remove the reference implementation too.
>
> See the attached discussion.
>
> Given that the properties table had been deprecated in the UEFI spec<= br> > even in Feb 2016, I think it's now high time to remove it altogether<= br> > (both spec and edk2).
>
> > The existence of both is confusing.
>
> Yes, very much.
>

Agreed. We should at least log this as a task, and if anyone has the
bandwidth to do the work, plan it for the next stable tag.



--_000_CY4PR21MB0743267C90E80B7A4D6EB03EEFCE0CY4PR21MB0743namp_--