From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=192.55.52.115; helo=mga14.intel.com; envelope-from=ruiyu.ni@intel.com; receiver=edk2-devel@lists.01.org Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id DAFCA2034D801 for ; Mon, 6 Nov 2017 21:32:04 -0800 (PST) Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 06 Nov 2017 21:36:03 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.44,356,1505804400"; d="scan'208";a="172911775" Received: from fmsmsx105.amr.corp.intel.com ([10.18.124.203]) by fmsmga006.fm.intel.com with ESMTP; 06 Nov 2017 21:36:03 -0800 Received: from fmsmsx124.amr.corp.intel.com (10.18.125.39) by FMSMSX105.amr.corp.intel.com (10.18.124.203) with Microsoft SMTP Server (TLS) id 14.3.319.2; Mon, 6 Nov 2017 21:36:02 -0800 Received: from shsmsx152.ccr.corp.intel.com (10.239.6.52) by fmsmsx124.amr.corp.intel.com (10.18.125.39) with Microsoft SMTP Server (TLS) id 14.3.319.2; Mon, 6 Nov 2017 21:36:02 -0800 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.152]) by SHSMSX152.ccr.corp.intel.com ([169.254.6.93]) with mapi id 14.03.0319.002; Tue, 7 Nov 2017 13:36:00 +0800 From: "Ni, Ruiyu" To: Laszlo Ersek , "Shi, Steven" CC: "Zeng, Star" , "edk2-devel@lists.01.org" Thread-Topic: [PATCH] PcAtChipsetPkg/IsaAcpiDxe: Restore PCI attributes correctly Thread-Index: AQHTV1ZG5ASih8DvG0y8WvqBX5YMH6MIZbCg Date: Tue, 7 Nov 2017 05:35:59 +0000 Message-ID: <734D49CCEBEEF84792F5B80ED585239D5BAB5822@SHSMSX104.ccr.corp.intel.com> References: <20171103082836.125696-1-ruiyu.ni@intel.com> <6925f265-26e3-90fb-10f4-f794a9c925e5@redhat.com> In-Reply-To: <6925f265-26e3-90fb-10f4-f794a9c925e5@redhat.com> Accept-Language: en-US, zh-CN X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [PATCH] PcAtChipsetPkg/IsaAcpiDxe: Restore PCI attributes correctly X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 07 Nov 2017 05:32:05 -0000 Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 TGFzemxvLA0KU3VyZSBJIHdpbGwgYWRkIHRoZSBCdWd6aWxsYSB1cmwgaW4gdGhlIGNvbW1pdCBt ZXNzYWdlLg0KDQpTdGV2ZW4sDQpDb3VsZCB5b3UgcGxlYXNlIGNoZWNrIHdoZXRoZXIgdGhpcyBw YXRjaCBjYW4gZml4IHlvdXIgInJlY29ubmVjdCAtciIgaGFuZz8NCg0KVGhhbmtzL1JheQ0KDQo+ IC0tLS0tT3JpZ2luYWwgTWVzc2FnZS0tLS0tDQo+IEZyb206IExhc3psbyBFcnNlayBbbWFpbHRv OmxlcnNla0ByZWRoYXQuY29tXQ0KPiBTZW50OiBUdWVzZGF5LCBOb3ZlbWJlciA3LCAyMDE3IDc6 MjMgQU0NCj4gVG86IE5pLCBSdWl5dSA8cnVpeXUubmlAaW50ZWwuY29tPjsgZWRrMi1kZXZlbEBs aXN0cy4wMS5vcmcNCj4gQ2M6IFplbmcsIFN0YXIgPHN0YXIuemVuZ0BpbnRlbC5jb20+OyBTaGks IFN0ZXZlbiA8c3RldmVuLnNoaUBpbnRlbC5jb20+DQo+IFN1YmplY3Q6IFJlOiBbUEFUQ0hdIFBj QXRDaGlwc2V0UGtnL0lzYUFjcGlEeGU6IFJlc3RvcmUgUENJIGF0dHJpYnV0ZXMNCj4gY29ycmVj dGx5DQo+IA0KPiBIaSBSYXksDQo+IA0KPiBPbiAxMS8wMy8xNyAwOToyOCwgUnVpeXUgTmkgd3Jv dGU6DQo+ID4gVGhlIG9yaWdpbmFsIGNvZGUgZW5hYmxlcyBzb21lIEJJVHMgaW4gUENJIGF0dHJp YnV0ZXMgaW4gU3RhcnQoKSwgYnV0DQo+ID4gd3JvbmdseSB0byBkaXNhYmxlIHRoZXNlIEJJVHMg aW4gU3RvcCgpLg0KPiA+DQo+ID4gVGhlIGNvcnJlY3QgYmVoYXZpb3IgaXMgdG8gc2F2ZSB0aGUg b3JpZ2luYWwgUENJIGF0dHJpYnV0ZXMgYmVmb3JlDQo+ID4gZW5hYmxlcyBzb21lIEJJVHMgaW4g U3RhcnQoKSwgYW5kIHJlc3RvcmUgdG8gb3JpZ2luYWwgdmFsdWUgaW4gU3RvcCgpLg0KPiA+DQo+ ID4gQ29udHJpYnV0ZWQtdW5kZXI6IFRpYW5vQ29yZSBDb250cmlidXRpb24gQWdyZWVtZW50IDEu MQ0KPiA+IFNpZ25lZC1vZmYtYnk6IFJ1aXl1IE5pIDxydWl5dS5uaUBpbnRlbC5jb20+DQo+ID4g Q2M6IFN0YXIgWmVuZyA8c3Rhci56ZW5nQGludGVsLmNvbT4NCj4gPiBDYzogTGFzemxvIEVyc2Vr IDxsZXJzZWtAcmVkaGF0LmNvbT4NCj4gPiAtLS0NCj4gPiAgUGNBdENoaXBzZXRQa2cvSXNhQWNw aUR4ZS9QY2F0SXNhQWNwaS5jIHwgNDQNCj4gPiArKysrKysrKysrKysrKysrKy0tLS0tLS0tLS0t LS0tLS0NCj4gPiBQY0F0Q2hpcHNldFBrZy9Jc2FBY3BpRHhlL1BjYXRJc2FBY3BpLmggfCAgMyAr Ky0NCj4gPiAgMiBmaWxlcyBjaGFuZ2VkLCAyNSBpbnNlcnRpb25zKCspLCAyMiBkZWxldGlvbnMo LSkNCj4gDQo+IElzIHRoaXMgZm9yIDxodHRwczovL2J1Z3ppbGxhLnRpYW5vY29yZS5vcmcvc2hv d19idWcuY2dpP2lkPTQwNT4/DQo+IA0KPiBJZiBzbywgY2FuIHlvdSBwbGVhc2UgYWRkIHRoZSBy ZWZlcmVuY2UgdG8gdGhlIGNvbW1pdCBtZXNzYWdlPw0KPiANCj4gQWxzbywgSSB0aGluayB3ZSBz aG91bGQgYXNrIFN0ZXZlbiB0byB0ZXN0IHRoZSBwYXRjaC4gKENDJ2QuKQ0KPiANCj4gSSdsbCB0 cnkgdG8gY29tbWVudCBtb3JlIGxhdGVyLg0KPiANCj4gVGhhbmtzDQo+IExhc3psbw0KPiANCj4g DQo+ID4NCj4gPiBkaWZmIC0tZ2l0IGEvUGNBdENoaXBzZXRQa2cvSXNhQWNwaUR4ZS9QY2F0SXNh QWNwaS5jDQo+ID4gYi9QY0F0Q2hpcHNldFBrZy9Jc2FBY3BpRHhlL1BjYXRJc2FBY3BpLmMNCj4g PiBpbmRleCAzMjM4MWIxMTJkLi42MGQyZmI1YTViIDEwMDY0NA0KPiA+IC0tLSBhL1BjQXRDaGlw c2V0UGtnL0lzYUFjcGlEeGUvUGNhdElzYUFjcGkuYw0KPiA+ICsrKyBiL1BjQXRDaGlwc2V0UGtn L0lzYUFjcGlEeGUvUGNhdElzYUFjcGkuYw0KPiA+IEBAIC0xNzIsNiArMTcyLDcgQEAgUGNhdElz YUFjcGlEcml2ZXJCaW5kaW5nU3RhcnQgKA0KPiA+ICAgIEVGSV9QQ0lfSU9fUFJPVE9DT0wgICpQ Y2lJbzsNCj4gPiAgICBQQ0FUX0lTQV9BQ1BJX0RFViAgICAqUGNhdElzYUFjcGlEZXY7DQo+ID4g ICAgVUlOVDY0ICAgICAgICAgICAgICAgU3VwcG9ydHM7DQo+ID4gKyAgVUlOVDY0ICAgICAgICAg ICAgICAgT3JpZ2luYWxBdHRyaWJ1dGVzOw0KPiA+ICAgIEJPT0xFQU4gICAgICAgICAgICAgIEVu YWJsZWQ7DQo+ID4NCj4gPiAgICBFbmFibGVkID0gRkFMU0U7DQo+ID4gQEAgLTIxMCw5ICsyMTEs MTggQEAgUGNhdElzYUFjcGlEcml2ZXJCaW5kaW5nU3RhcnQgKA0KPiA+ICAgIGlmIChTdXBwb3J0 cyA9PSAwIHx8IFN1cHBvcnRzID09IChFRklfUENJX0lPX0FUVFJJQlVURV9JU0FfSU8gfA0KPiBF RklfUENJX0lPX0FUVFJJQlVURV9JU0FfSU9fMTYpKSB7DQo+ID4gICAgICBTdGF0dXMgPSBFRklf VU5TVVBQT1JURUQ7DQo+ID4gICAgICBnb3RvIERvbmU7DQo+ID4gLSAgfQ0KPiA+ICsgIH0NCj4g PiArDQo+ID4gKyAgU3RhdHVzID0gUGNpSW8tPkF0dHJpYnV0ZXMgKA0KPiA+ICsgICAgICAgICAg ICAgICAgICAgIFBjaUlvLA0KPiA+ICsgICAgICAgICAgICAgICAgICAgIEVmaVBjaUlvQXR0cmli dXRlT3BlcmF0aW9uR2V0LA0KPiA+ICsgICAgICAgICAgICAgICAgICAgIDAsDQo+ID4gKyAgICAg ICAgICAgICAgICAgICAgJk9yaWdpbmFsQXR0cmlidXRlcw0KPiA+ICsgICAgICAgICAgICAgICAg ICAgICk7DQo+ID4gKyAgaWYgKEVGSV9FUlJPUiAoU3RhdHVzKSkgew0KPiA+ICsgICAgZ290byBE b25lOw0KPiA+ICsgIH0NCj4gPg0KPiA+IC0gIEVuYWJsZWQgPSBUUlVFOw0KPiA+ICAgIFN0YXR1 cyA9IFBjaUlvLT5BdHRyaWJ1dGVzICgNCj4gPiAgICAgICAgICAgICAgICAgICAgICBQY2lJbywN Cj4gPiAgICAgICAgICAgICAgICAgICAgICBFZmlQY2lJb0F0dHJpYnV0ZU9wZXJhdGlvbkVuYWJs ZSwgQEAgLTIyMiw3DQo+ID4gKzIzMiw4IEBAIFBjYXRJc2FBY3BpRHJpdmVyQmluZGluZ1N0YXJ0 ICgNCj4gPiAgICBpZiAoRUZJX0VSUk9SIChTdGF0dXMpKSB7DQo+ID4gICAgICBnb3RvIERvbmU7 DQo+ID4gICAgfQ0KPiA+IC0NCj4gPiArDQo+ID4gKyAgRW5hYmxlZCA9IFRSVUU7DQo+ID4gICAg Ly8NCj4gPiAgICAvLyBBbGxvY2F0ZSBtZW1vcnkgZm9yIHRoZSBQQ0FUIElTQSBBQ1BJIERldmlj ZSBzdHJ1Y3R1cmUNCj4gPiAgICAvLw0KPiA+IEBAIC0yMzksOSArMjUwLDEwIEBAIFBjYXRJc2FB Y3BpRHJpdmVyQmluZGluZ1N0YXJ0ICgNCj4gPiAgICAvLw0KPiA+ICAgIC8vIEluaXRpYWxpemUg dGhlIFBDQVQgSVNBIEFDUEkgRGV2aWNlIHN0cnVjdHVyZQ0KPiA+ICAgIC8vDQo+ID4gLSAgUGNh dElzYUFjcGlEZXYtPlNpZ25hdHVyZSA9IFBDQVRfSVNBX0FDUElfREVWX1NJR05BVFVSRTsNCj4g PiAtICBQY2F0SXNhQWNwaURldi0+SGFuZGxlICAgID0gQ29udHJvbGxlcjsNCj4gPiAtICBQY2F0 SXNhQWNwaURldi0+UGNpSW8gICAgID0gUGNpSW87DQo+ID4gKyAgUGNhdElzYUFjcGlEZXYtPlNp Z25hdHVyZSAgICAgICAgID0gUENBVF9JU0FfQUNQSV9ERVZfU0lHTkFUVVJFOw0KPiA+ICsgIFBj YXRJc2FBY3BpRGV2LT5IYW5kbGUgICAgICAgICAgICA9IENvbnRyb2xsZXI7DQo+ID4gKyAgUGNh dElzYUFjcGlEZXYtPlBjaUlvICAgICAgICAgICAgID0gUGNpSW87DQo+ID4gKyAgUGNhdElzYUFj cGlEZXYtPk9yaWdpbmFsQXR0cmlidXRlID0gT3JpZ2luYWxBdHRyaWJ1dGVzOw0KPiA+DQo+ID4g ICAgLy8NCj4gPiAgICAvLyBJbml0aWFsaXplIFBjYXRJc2FBY3BpRGV2aWNlTGlzdCBAQCAtMjc0 LDggKzI4Niw4IEBAIERvbmU6DQo+ID4gICAgICBpZiAoUGNpSW8gIT0gTlVMTCAmJiBFbmFibGVk KSB7DQo+ID4gICAgICAgIFBjaUlvLT5BdHRyaWJ1dGVzICgNCj4gPiAgICAgICAgICAgICAgICAg UGNpSW8sDQo+ID4gLSAgICAgICAgICAgICAgIEVmaVBjaUlvQXR0cmlidXRlT3BlcmF0aW9uRGlz YWJsZSwNCj4gPiAtICAgICAgICAgICAgICAgRUZJX1BDSV9ERVZJQ0VfRU5BQkxFIHwgU3VwcG9y dHMgfA0KPiBFRklfUENJX0lPX0FUVFJJQlVURV9JU0FfTU9USEVSQk9BUkRfSU8sDQo+ID4gKyAg ICAgICAgICAgICAgIEVmaVBjaUlvQXR0cmlidXRlT3BlcmF0aW9uU2V0LA0KPiA+ICsgICAgICAg ICAgICAgICBPcmlnaW5hbEF0dHJpYnV0ZXMsDQo+ID4gICAgICAgICAgICAgICAgIE5VTEwNCj4g PiAgICAgICAgICAgICAgICAgKTsNCj4gPiAgICAgIH0NCj4gPiBAQCAtMzIxLDcgKzMzMyw2IEBA IFBjYXRJc2FBY3BpRHJpdmVyQmluZGluZ1N0b3AgKA0KPiA+ICAgIEVGSV9TVEFUVVMgICAgICAg ICAgICAgU3RhdHVzOw0KPiA+ICAgIEVGSV9JU0FfQUNQSV9QUk9UT0NPTCAgKklzYUFjcGk7DQo+ ID4gICAgUENBVF9JU0FfQUNQSV9ERVYgICAgICAqUGNhdElzYUFjcGlEZXY7DQo+ID4gLSAgVUlO VDY0ICAgICAgICAgICAgICAgICBTdXBwb3J0czsNCj4gPg0KPiA+ICAgIC8vDQo+ID4gICAgLy8g R2V0IHRoZSBJU0EgQUNQSSBQcm90b2NvbCBJbnRlcmZhY2UgQEAgLTM0OCwyMyArMzU5LDE0IEBA DQo+ID4gUGNhdElzYUFjcGlEcml2ZXJCaW5kaW5nU3RvcCAoDQo+ID4gICAgLy8NCj4gPiAgICBT dGF0dXMgPSBQY2F0SXNhQWNwaURldi0+UGNpSW8tPkF0dHJpYnV0ZXMgKA0KPiA+ICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICBQY2F0SXNhQWNwaURldi0+UGNpSW8sDQo+ID4g LSAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIEVmaVBjaUlvQXR0cmlidXRlT3Bl cmF0aW9uU3VwcG9ydGVkLA0KPiA+IC0gICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAwLA0KPiA+IC0gICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAmU3VwcG9ydHMN Cj4gPiArICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgRWZpUGNpSW9BdHRyaWJ1 dGVPcGVyYXRpb25TZXQsDQo+ID4gKyAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg IFBjYXRJc2FBY3BpRGV2LT5PcmlnaW5hbEF0dHJpYnV0ZSwNCj4gPiArICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgMA0KPiA+ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICApOw0KPiA+ICAgIGlmIChFRklfRVJST1IgKFN0YXR1cykpIHsNCj4gPiAgICAgIHJl dHVybiBTdGF0dXM7DQo+ID4gICAgfQ0KPiA+DQo+ID4gLSAgU3VwcG9ydHMgJj0gKFVJTlQ2NCkg KEVGSV9QQ0lfSU9fQVRUUklCVVRFX0lTQV9JTyB8DQo+ID4gRUZJX1BDSV9JT19BVFRSSUJVVEVf SVNBX0lPXzE2KTsNCj4gPiAtDQo+ID4gLSAgUGNhdElzYUFjcGlEZXYtPlBjaUlvLT5BdHRyaWJ1 dGVzICgNCj4gPiAtICAgICAgICAgICAgICAgICAgICAgICAgICAgUGNhdElzYUFjcGlEZXYtPlBj aUlvLA0KPiA+IC0gICAgICAgICAgICAgICAgICAgICAgICAgICBFZmlQY2lJb0F0dHJpYnV0ZU9w ZXJhdGlvbkRpc2FibGUsDQo+ID4gLSAgICAgICAgICAgICAgICAgICAgICAgICAgIEVGSV9QQ0lf REVWSUNFX0VOQUJMRSB8IFN1cHBvcnRzIHwNCj4gRUZJX1BDSV9JT19BVFRSSUJVVEVfSVNBX01P VEhFUkJPQVJEX0lPLA0KPiA+IC0gICAgICAgICAgICAgICAgICAgICAgICAgICBOVUxMDQo+ID4g LSAgICAgICAgICAgICAgICAgICAgICAgICAgICk7DQo+ID4gLQ0KPiA+ICAgIC8vDQo+ID4gICAg Ly8gVW5pbnN0YWxsIHByb3RvY29sIGludGVyZmFjZTogRUZJX0lTQV9BQ1BJX1BST1RPQ09MDQo+ ID4gICAgLy8NCj4gPiBkaWZmIC0tZ2l0IGEvUGNBdENoaXBzZXRQa2cvSXNhQWNwaUR4ZS9QY2F0 SXNhQWNwaS5oDQo+ID4gYi9QY0F0Q2hpcHNldFBrZy9Jc2FBY3BpRHhlL1BjYXRJc2FBY3BpLmgN Cj4gPiBpbmRleCAwNjcxMTI3NjQ0Li4zYWQzYTNmMzEzIDEwMDY0NA0KPiA+IC0tLSBhL1BjQXRD aGlwc2V0UGtnL0lzYUFjcGlEeGUvUGNhdElzYUFjcGkuaA0KPiA+ICsrKyBiL1BjQXRDaGlwc2V0 UGtnL0lzYUFjcGlEeGUvUGNhdElzYUFjcGkuaA0KPiA+IEBAIC0xLDcgKzEsNyBAQA0KPiA+ICAv KiogQGZpbGUNCj4gPiAgICBFRkkgUENBVCBJU0EgQUNQSSBEcml2ZXIgZm9yIGEgR2VuZXJpYyBQ QyBQbGF0Zm9ybQ0KPiA+DQo+ID4gLUNvcHlyaWdodCAoYykgMjAwNiAtIDIwMTEsIEludGVsIENv cnBvcmF0aW9uLiBBbGwgcmlnaHRzDQo+ID4gcmVzZXJ2ZWQuPEJSPg0KPiA+ICtDb3B5cmlnaHQg KGMpIDIwMDYgLSAyMDE3LCBJbnRlbCBDb3Jwb3JhdGlvbi4gQWxsIHJpZ2h0cw0KPiA+ICtyZXNl cnZlZC48QlI+DQo+ID4gIFRoaXMgcHJvZ3JhbSBhbmQgdGhlIGFjY29tcGFueWluZyBtYXRlcmlh bHMNCj4gPiAgYXJlIGxpY2Vuc2VkIGFuZCBtYWRlIGF2YWlsYWJsZSB1bmRlciB0aGUgdGVybXMg YW5kIGNvbmRpdGlvbnMgb2YgdGhlIEJTRA0KPiBMaWNlbnNlDQo+ID4gIHdoaWNoIGFjY29tcGFu aWVzIHRoaXMgZGlzdHJpYnV0aW9uLiAgVGhlIGZ1bGwgdGV4dCBvZiB0aGUgbGljZW5zZSBtYXkg YmUNCj4gZm91bmQgYXQNCj4gPiBAQCAtNDMsNiArNDMsNyBAQCB0eXBlZGVmIHN0cnVjdCB7DQo+ ID4gICAgRUZJX0hBTkRMRSAgICAgICAgICAgICBIYW5kbGU7DQo+ID4gICAgRUZJX0lTQV9BQ1BJ X1BST1RPQ09MICBJc2FBY3BpOw0KPiA+ICAgIEVGSV9QQ0lfSU9fUFJPVE9DT0wgICAgKlBjaUlv Ow0KPiA+ICsgIFVJTlQ2NCAgICAgICAgICAgICAgICAgT3JpZ2luYWxBdHRyaWJ1dGU7DQo+ID4g IH0gUENBVF9JU0FfQUNQSV9ERVY7DQo+ID4NCj4gPiAgI2RlZmluZSBQQ0FUX0lTQV9BQ1BJX0RF Vl9GUk9NX1RISVMoYSkgQkFTRV9DUihhLA0KPiBQQ0FUX0lTQV9BQ1BJX0RFViwNCj4gPiBJc2FB Y3BpKQ0KPiA+DQoNCg==