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.136; helo=mga12.intel.com; envelope-from=hong-chihx.hsueh@intel.com; receiver=edk2-devel@lists.01.org Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) (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 2E75C211B76D0 for ; Mon, 28 Jan 2019 15:40:31 -0800 (PST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 28 Jan 2019 15:40:30 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.56,535,1539673200"; d="scan'208";a="133906014" Received: from fmsmsx107.amr.corp.intel.com ([10.18.124.205]) by orsmga001.jf.intel.com with ESMTP; 28 Jan 2019 15:40:30 -0800 Received: from fmsmsx126.amr.corp.intel.com (10.18.125.43) by fmsmsx107.amr.corp.intel.com (10.18.124.205) with Microsoft SMTP Server (TLS) id 14.3.408.0; Mon, 28 Jan 2019 15:40:30 -0800 Received: from fmsmsx107.amr.corp.intel.com ([169.254.6.186]) by FMSMSX126.amr.corp.intel.com ([169.254.1.70]) with mapi id 14.03.0415.000; Mon, 28 Jan 2019 15:40:30 -0800 From: "Hsueh, Hong-chihX" To: Laszlo Ersek , "edk2-devel@lists.01.org" CC: "Kinney, Michael D" , "Gao, Liming" , "Bi, Dandan" Thread-Topic: [PATCH] MdePkg/BasePeCoffLib: skip runtime relocation if relocation info is invalid. Thread-Index: AQHUt1cs1lQUBHYIX0Wt3wnGh5MdVKXFUgVw Date: Mon, 28 Jan 2019 23:40:29 +0000 Message-ID: <1F8AF101AC6B5D47B4348ACCED214431CA59DADB@fmsmsx107.amr.corp.intel.com> References: <20190128184047.20792-1-hong-chihx.hsueh@intel.com> <0313d273-69f2-af63-bbff-8d561aaf8bbd@redhat.com> In-Reply-To: <0313d273-69f2-af63-bbff-8d561aaf8bbd@redhat.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.0.400.15 dlp-reaction: no-action x-ctpclassification: CTP_NT x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiZjUzN2U3NGUtZDVkMy00YTExLWI0ODUtNmE2NDlhNGZmZTc2IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiaVhXUWg0a01CZTV2TEQydVZWZGJDUGpuTHZzNEFpWlVaMTNMRGkwRzhLckJ4aTVSWTVJbW1TM3BUdnVUMVJIRSJ9 x-originating-ip: [10.1.200.106] MIME-Version: 1.0 Subject: Re: [PATCH] MdePkg/BasePeCoffLib: skip runtime relocation if relocation info is invalid. X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 28 Jan 2019 23:40:32 -0000 Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 PiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiBGcm9tOiBMYXN6bG8gRXJzZWsgW21haWx0 bzpsZXJzZWtAcmVkaGF0LmNvbV0NCj4gU2VudDogTW9uZGF5LCBKYW51YXJ5IDI4LCAyMDE5IDI6 MTcgUE0NCj4gVG86IEhzdWVoLCBIb25nLWNoaWhYIDxob25nLWNoaWh4LmhzdWVoQGludGVsLmNv bT47IGVkazItZGV2ZWxAbGlzdHMuMDEub3JnDQo+IENjOiBLaW5uZXksIE1pY2hhZWwgRCA8bWlj aGFlbC5kLmtpbm5leUBpbnRlbC5jb20+OyBHYW8sIExpbWluZw0KPiA8bGltaW5nLmdhb0BpbnRl bC5jb20+OyBCaSwgRGFuZGFuIDxkYW5kYW4uYmlAaW50ZWwuY29tPg0KPiBTdWJqZWN0OiBSZTog W1BBVENIXSBNZGVQa2cvQmFzZVBlQ29mZkxpYjogc2tpcCBydW50aW1lIHJlbG9jYXRpb24gaWYN Cj4gcmVsb2NhdGlvbiBpbmZvIGlzIGludmFsaWQuDQo+IA0KPiBPbiAwMS8yOC8xOSAxOTo0MCwg TmVvIEhzdWVoIHdyb3RlOg0KPiA+IFNraXAgcnVudGltZSByZWxvY2F0aW9uIGZvciBQRSBpbWFn ZXMgdGhhdCBwcm92aWRlIGludmFsaWQgcmVsb2NhdGlvbg0KPiA+IGluZm9tYXRpb24NCj4gPiAo ZXg6IFJlbG9jRGlyLT5TaXplID0gMCkgdG8gZml4IGEgaGFuZyBvYnNlcnZlZCB3aGlsZSBib290 aW5nIFdpbmRvd3MuDQo+ID4NCj4gPiBDb250cmlidXRlZC11bmRlcjogVGlhbm9Db3JlIENvbnRy aWJ1dGlvbiBBZ3JlZW1lbnQgMS4xDQo+ID4NCj4gPiBTaWduZWQtb2ZmLWJ5OiBOZW8gSHN1ZWgg PGhvbmctY2hpaHguaHN1ZWhAaW50ZWwuY29tPg0KPiA+IENjOiBNaWNoYWVsIEQgS2lubmV5IDxt aWNoYWVsLmQua2lubmV5QGludGVsLmNvbT4NCj4gPiBDYzogTGltaW5nIEdhbyA8bGltaW5nLmdh b0BpbnRlbC5jb20+DQo+ID4gQ2M6IERhbmRhbiBCaSA8ZGFuZGFuLmJpQGludGVsLmNvbT4NCj4g PiBDYzogTGFzemxvIEVyc2VrIDxsZXJzZWtAcmVkaGF0LmNvbT4NCj4gPiAtLS0NCj4gPiAgTWRl UGtnL0xpYnJhcnkvQmFzZVBlQ29mZkxpYi9CYXNlUGVDb2ZmLmMgfCA2ICsrKysrKw0KPiA+ICAx IGZpbGUgY2hhbmdlZCwgNiBpbnNlcnRpb25zKCspDQo+ID4NCj4gPiBkaWZmIC0tZ2l0IGEvTWRl UGtnL0xpYnJhcnkvQmFzZVBlQ29mZkxpYi9CYXNlUGVDb2ZmLmMNCj4gPiBiL01kZVBrZy9MaWJy YXJ5L0Jhc2VQZUNvZmZMaWIvQmFzZVBlQ29mZi5jDQo+ID4gaW5kZXggMWJkMDc5YWQ2YS4uZjAx YzY5MWRlYSAxMDA2NDQNCj4gPiAtLS0gYS9NZGVQa2cvTGlicmFyeS9CYXNlUGVDb2ZmTGliL0Jh c2VQZUNvZmYuYw0KPiA+ICsrKyBiL01kZVBrZy9MaWJyYXJ5L0Jhc2VQZUNvZmZMaWIvQmFzZVBl Q29mZi5jDQo+ID4gQEAgLTE3NDYsNiArMTc0NiwxMiBAQCBQZUNvZmZMb2FkZXJSZWxvY2F0ZUlt YWdlRm9yUnVudGltZSAoDQo+ID4gICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBSZWxvY0Rpci0+VmlydHVh bEFkZHJlc3MgKw0KPiBSZWxvY0Rpci0+U2l6ZSAtIDEsDQo+ID4gICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAwDQo+ID4NCj4gPiApOw0KPiA+ICsgICAgaWYgKFJlbG9jQmFzZSA9PSBOVUxMIHx8IFJlbG9j QmFzZUVuZCA9PSBOVUxMIHx8IFJlbG9jQmFzZUVuZCA8DQo+IFJlbG9jQmFzZSkgew0KPiA+ICsg ICAgICAvLw0KPiA+ICsgICAgICAvLyByZWxvY2F0aW9uIGJsb2NrIGlzIG5vdCB2YWxpZCwganVz dCByZXR1cm4NCj4gPiArICAgICAgLy8NCj4gPiArICAgICAgcmV0dXJuOw0KPiA+ICsgICAgfQ0K PiA+ICAgIH0gZWxzZSB7DQo+ID4gICAgICAvLw0KPiA+ICAgICAgLy8gQ2Fubm90IGZpbmQgcmVs b2NhdGlvbnMsIGNhbm5vdCBjb250aW51ZSB0byByZWxvY2F0ZSB0aGUgaW1hZ2UsIEFTU0VSVA0K PiBmb3IgdGhpcyBpbnZhbGlkIGltYWdlLg0KPiA+DQo+IA0KPiBUaGFuayB5b3UgZm9yIHRoZSB1 cGRhdGUuDQo+IA0KPiAuLi4gT3JpZ2luYWxseSBJIG1lYW50IHRvIHJlc3BvbmQgd2l0aCBhbiBB Y2tlZC1ieSAocHVyZWx5IGZyb20gYSBmb3JtYWwgcG9pbnQtDQo+IG9mLXZpZXcpOyBob3dldmVy IEkgZmlndXJlZCB0aGUgcGF0Y2ggd2Fzbid0IGxhcmdlIGFuZCBJIGNvdWxkIGNoZWNrIGl0IGZv ciBhDQo+IFJldmlld2VkLWJ5IGFzIHdlbGwuDQo+IA0KPiBJJ20gbm90aWNpbmcgdGhlIGNvbXBh cmlzb24gKFJlbG9jQmFzZUVuZCA8IFJlbG9jQmFzZSkgaXMgc3VwcG9zZWQgdG8gY2F0Y2gNCj4g aW52YWxpZCByZWxvY2F0aW9uIGluZm8uIFRoZXNlIHZhcmlhYmxlcyBhcmUgcG9pbnRlcnMsIGRl Y2xhcmVkIGFzDQo+IGZvbGxvd3M6DQo+IA0KPiAgIEVGSV9JTUFHRV9CQVNFX1JFTE9DQVRJT04g ICAgICAgICAgICAgKlJlbG9jQmFzZTsNCj4gICBFRklfSU1BR0VfQkFTRV9SRUxPQ0FUSU9OICAg ICAgICAgICAgICpSZWxvY0Jhc2VFbmQ7DQo+IA0KPiBBY2NvcmRpbmcgdG8gdGhlIEMgc3RhbmRh cmQsIHRoZSByZWxhdGlvbmFsIG9wZXJhdG9ycyBjYW4gb25seSBiZSBhcHBsaWVkIHRvIGENCj4g cGFpciBvZiBwb2ludGVycyBpZiBlYWNoIG9mIHRob3NlIHBvaW50cyBpbnRvIHRoZSBzYW1lIGFy cmF5LCBvciBvbmUgcGFzdCB0aGUgbGFzdA0KPiBlbGVtZW50LiBJbiB0aGlzIGNhc2UsIGdpdmVu IHRoYXQgeW91IGludGVuZCB0byBjYXRjaCBpbnZhbGlkIHJlbG9jYXRpb24gaW5mbywNCj4gdGhh dCdzIGV4YWN0bHkgKm5vdCogdGhlIGNhc2UuIEluIG90aGVyIHdvcmRzLCBpbiB0aGUgb25seSBj YXNlIHdoZW4gdGhlDQo+IHJlbGF0aW9uYWwgb3BlcmF0b3Igd291bGQgZXZhbHVhdGUgdG8gdHJ1 ZSwgaXQgd291bGQgYWxzbyBpbnZva2UgdW5kZWZpbmVkDQo+IGJlaGF2aW9yLiBGdXJ0aGVybW9y ZSwgdGhlIGJ5dGUgZGlzdGFuY2UgYmV0d2VlbiB0aGUgcG9pbnRlZC10by1vYmplY3RzIG1pZ2h0 DQo+IG5vdCBldmVuIGJlIGEgd2hvbGUgbXVsdGlwbGUgb2Ygc2l6ZW9mIChFRklfSU1BR0VfQkFT RV9SRUxPQ0FUSU9OKS4NCj4gDQo+IE5vcm1hbGx5IEkgd291bGQgc3VnZ2VzdCBjaGFuZ2luZyB0 aGUgcmV0dXJuIHR5cGUgb2YNCj4gUGVDb2ZmTG9hZGVySW1hZ2VBZGRyZXNzKCkgdG8gVUlOVE4g LS0gdGhhdCB3b3VsZCBiZSBmaXR0aW5nIGJlY2F1c2UgdGhlDQo+IGludGVybmFsIGNvbXB1dGF0 aW9uIGlzIGFscmVhZHkgcGVyZm9ybWVkIGluIFVJTlROLCBhbmQgb25seSBjYXN0IHRvDQo+IChD SEFSOCAqKSBhcyBsYXN0IHN0ZXAuIFRoaXMgd2F5IHdlIGNvdWxkIG1vdmUgdGhlIGNhc3QgdG8g dGhlIGNhbGxlcnMsIGFuZA0KPiBwZXJmb3JtIHRoZSBzYW5pdHkgY2hlY2tzIGJlZm9yZSB0aGUg Y29udmVyc2lvbiB0byAoVk9JRCopIChvciB0byBvdGhlciBwb2ludGVyDQo+IHR5cGVzKS4NCj4g DQo+IEkgZG8gc2VlIHRoZSBmdW5jdGlvbiBpcyBjYWxsZWQgZnJvbSBtYW55IHBsYWNlcywgc28g dGhpcyBjaGFuZ2UgbWlnaHQgYmUgdG9vDQo+IGNvc3RseS4gQ2FuIHdlIGF0IGxlYXN0IHdyaXRl IGluIHRoaXMgcGF0Y2gsDQo+IA0KPiAgIGlmIChSZWxvY0Jhc2UgPT0gTlVMTCB8fA0KPiAgICAg ICBSZWxvY0Jhc2VFbmQgPT0gTlVMTCB8fA0KPiAgICAgICAoVUlOVE4pUmVsb2NCYXNlRW5kIDwg KFVJTlROKVJlbG9jQmFzZSB8fA0KPiAgICAgICAoKChVSU5UTilSZWxvY0Jhc2VFbmQgLSAoVUlO VE4pUmVsb2NCYXNlKSAlDQo+ICAgICAgICBzaXplb2YgKEVGSV9JTUFHRV9CQVNFX1JFTE9DQVRJ T04pICE9IDApKSB7DQo+ICAgICByZXR1cm47DQo+ICAgfQ0KPiANCj4gPw0KPiANCj4gUGVyaGFw cyB3ZSBzaG91bGQgZXZlbiBleHRyYWN0IHRoaXMgbG9naWMgdG8gYSBoZWxwZXIgZnVuY3Rpb24s IGJlY2F1c2UgSSBzZWUNCj4gYW5vdGhlciBzcG90IHdpdGggdGhlIHNhbWUgY29uZGl0aW9uLiBU aGF0J3MgaW4gUGVDb2ZmTG9hZGVyUmVsb2NhdGVJbWFnZSgpLA0KPiBmcm9tIHRoZSB0b3Agb2Yg Y29tbWl0IGE4ZDhkNDMwNTEwZCAoIlN1cHBvcnQgbG9hZCA2NCBiaXQgaW1hZ2UgZnJvbSAzMiBi aXQNCj4gY29yZS4gQWRkIG1vcmUgZW5oYW5jZW1lbnQgdG8gY2hlY2sgaW52YWxpZCBQRSBmb3Jt YXQuIiwgMjAxNC0wMy0yNSkuDQo+IA0KPiBJJ20gc29ycnkgdGhhdCBJIGRpZG4ndCBtYW5hZ2Ug dG8gbWFrZSB0aGVzZSBzdWdnZXN0aW9ucyB1bmRlciB0aGUgdjEgcG9zdGluZy4NCj4gDQo+IFRo YW5rcywNCj4gTGFzemxvDQoNCkhpIExhc3psbywNClRoYW5rIHlvdS4gSSBhZ3JlZSB0aGUgcG9p bnRlciBjb21wYXJpc29uIGlzIG5vdCBvcHRpbWFsIGVzcGVjaWFsbHkgaW4gdGhpcyBjYXNlLg0K SG93ZXZlciBJIGRpZG4ndCBhZGQgbXVsdGlwbGUgb2Ygc2l6ZSAoRUZJX0lNQUdFX0JBU0VfUkVM T0NBVElPTikgY2hlY2sgYmVjYXVzZSBmcm9tIHRoZSBjb21taXQgZWI3NmI3NjIsIHdlIGFjdHVh bGx5IGNoZWNrIHRoZSBhZGRyZXNzIHJhbmdlIGJldHdlZW4gQmFzZSB0byBSZWxvY0Rpci0+U2l6 ZSAtIDEuDQoNCkhlcmUgaXMgdGhlIHVwZGF0ZWQgcGF0Y2ggOg0KaHR0cHM6Ly9saXN0cy4wMS5v cmcvcGlwZXJtYWlsL2VkazItZGV2ZWwvMjAxOS1KYW51YXJ5LzAzNTgxMC5odG1sDQoNClJlZ2Fy ZHMsDQpOZW8NCg==