From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=134.134.136.65; helo=mga03.intel.com; envelope-from=star.zeng@intel.com; receiver=edk2-devel@lists.01.org Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) (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 0CCDC202E5E52 for ; Mon, 6 Nov 2017 17:35:58 -0800 (PST) Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 06 Nov 2017 17:39:57 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.44,355,1505804400"; d="scan'208";a="172336382" Received: from fmsmsx107.amr.corp.intel.com ([10.18.124.205]) by fmsmga005.fm.intel.com with ESMTP; 06 Nov 2017 17:39:57 -0800 Received: from shsmsx104.ccr.corp.intel.com (10.239.4.70) by fmsmsx107.amr.corp.intel.com (10.18.124.205) with Microsoft SMTP Server (TLS) id 14.3.319.2; Mon, 6 Nov 2017 17:39:57 -0800 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.175]) by SHSMSX104.ccr.corp.intel.com ([169.254.5.152]) with mapi id 14.03.0319.002; Tue, 7 Nov 2017 09:39:39 +0800 From: "Zeng, Star" To: Laszlo Ersek , "edk2-devel@lists.01.org" CC: Julien Grall , "Ni, Ruiyu" , "Zeng, Star" Thread-Topic: [PATCH] MdeModulePkg SerialDxe: Handle Timeout change more robustly Thread-Index: AQHTV0hJR97PvhMuzUeYRG7j0eUvZ6MII3DQ Date: Tue, 7 Nov 2017 01:39:39 +0000 Message-ID: <0C09AFA07DD0434D9E2A0C6AEB0483103B9B23DD@shsmsx102.ccr.corp.intel.com> References: <1509586896-56444-1-git-send-email-star.zeng@intel.com> In-Reply-To: Accept-Language: zh-CN, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [PATCH] MdeModulePkg SerialDxe: Handle Timeout change more robustly 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 01:35:59 -0000 Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 TGFzemxvLA0KDQpJIGFncmVlIGl0IGNvdWxkIGJlIG1vcmUgcmlnb3JvdXMgdG8gY29tcGFyZSBh Z2FpbnN0IHRoZSBvcmlnaW5hbCBwYXJhbWV0ZXJzLg0KUGxlYXNlIGNoZWNrIHRoZSBWMiBwYXRj aCBhdCBodHRwczovL2xpc3RzLjAxLm9yZy9waXBlcm1haWwvZWRrMi1kZXZlbC8yMDE3LU5vdmVt YmVyLzAxNjk2OC5odG1sLg0KDQpKdWxpZW4sDQoNClBsZWFzZSBoZWxwIHRha2UgdGhlIHRlc3Qg b24geW91ciBjYXNlIHdpdGggdGhlIFYyIHBhdGNoLg0KDQpUaGFua3MsDQpTdGFyDQotLS0tLU9y aWdpbmFsIE1lc3NhZ2UtLS0tLQ0KRnJvbTogTGFzemxvIEVyc2VrIFttYWlsdG86bGVyc2VrQHJl ZGhhdC5jb21dIA0KU2VudDogVHVlc2RheSwgTm92ZW1iZXIgNywgMjAxNyA1OjQzIEFNDQpUbzog WmVuZywgU3RhciA8c3Rhci56ZW5nQGludGVsLmNvbT47IGVkazItZGV2ZWxAbGlzdHMuMDEub3Jn DQpDYzogSnVsaWVuIEdyYWxsIDxqdWxpZW4uZ3JhbGxAbGluYXJvLm9yZz47IE5pLCBSdWl5dSA8 cnVpeXUubmlAaW50ZWwuY29tPg0KU3ViamVjdDogUmU6IFtQQVRDSF0gTWRlTW9kdWxlUGtnIFNl cmlhbER4ZTogSGFuZGxlIFRpbWVvdXQgY2hhbmdlIG1vcmUgcm9idXN0bHkNCg0KU3RhciwNCg0K T24gMTEvMDIvMTcgMDI6NDEsIFN0YXIgWmVuZyB3cm90ZToNCj4gaHR0cHM6Ly9saXN0cy4wMS5v cmcvcGlwZXJtYWlsL2VkazItZGV2ZWwvMjAxNy1PY3RvYmVyLzAxNjQ3OS5odG1sDQo+IHJlcG9y dGVkICJYZW4gQ29uc29sZSBpbnB1dCB2ZXJ5IHNsb3cgaW4gcmVjZW50IFVFRkkiIHRoYXQgYXBw ZWFycyANCj4gYWZ0ZXIgNGNmM2YzN2M4N2JhMWY5ZDU4MDcyNDQ0YmQ3MzVlNDBlNDc3OWU3MCAi TWRlTW9kdWxlUGtnDQo+IFNlcmlhbER4ZTogUHJvY2VzcyB0aW1lb3V0IGNvbnNpc3RlbnRseSBp biBTZXJpYWxSZWFkIi4NCj4gDQo+IEp1bGllbiBkaWQgbW9yZSBkZWJ1Z2dpbmcgYW5kIGZpbmQg b3V0IHRoZSBmb2xsb3dpbmcgaXMgaGFwcGVuaW5nIGluIA0KPiBUZXJtaW5hbENvbkluVGltZXJI YW5kbGVyIChNZGVNb2R1bGVQa2cvVW5pdmVyc2FsL0NvbnNvbGUvVGVybWluYWxEeGUpDQo+IHdo ZW4gYSBjaGFyYWN0ZXIgaXMgcmVjZWl2ZWQ6DQo+IDEpIEdldENvbnRyb2wgd2lsbCByZXR1cm4g RUZJX1NFUklBTF9JTlBVVF9CVUZGRVJfRU1QVFkgdW5zZXQNCj4gICA9PiBFbnRlcmluZyBpbiB0 aGUgbG9vcCB0byBmZXRjaCBjaGFyYWN0ZXIgZnJvbSB0aGUgc2VyaWFsDQo+IDIpIEdldE9uZUtl eUZyb21TZXJpYWwoKQ0KPiAgID0+IFJldHVybiBkaXJlY3RseSB3aXRoIHRoZSBjaGFyYWN0ZXIg cmVhZA0KPiAzKSBMb29waW5nIGFzIHRoZSBmaWZvIGlzIG5vdCBmdWxsIGFuZCBubyBlcnJvcg0K PiA0KSBHZXRPbmVLZXlGcm9tU2VyaWFsKCkgLT4gU2VyaWFsUmVhZCgpDQo+ICAgPT4gTm8gbW9y ZSBjaGFyYWN0ZXIgc28gU2VyaWFsUG9ydFBvbGwoKSB3aWxsIHJldHVybiBGQUxTRSBhbmQgbG9v cA0KPiAgICAgIHVudGlsIHRpbWVvdXQNCj4gICA9PiBSZXR1cm4gRUZJX1RJTUVPVVQNCj4gNSkg RXhpdGluZyB0aGUgbG9vcCBmcm9tIFRlcm1pbmFsQ29uSW5UaW1lckhhbmRsZXINCj4gNikgQ2hh cmFjdGVycyBhcmUgcHJpbnRlZA0KPiANCj4gQWZ0ZXIgc29tZSBpbnZlc3RpZ2F0aW9uLCBJIGZv dW5kIGl0IGlzIHJlbGF0ZWQgdG8gdGhlIFRpbWVvdXQgdmFsdWUuDQo+IA0KPiBUaGUgVGltZW91 dCBpcyAxMDAwMDAwICgxcykgYnkgZGVmYXVsdCB0byBmb2xsb3cgVUVGSSBzcGVjLg0KPiBBbmQg dGhlIFRlcm1pbmFsIGRyaXZlciB3aWxsIHJlY2FsY3VsYXRlIGFuZCBzZXQgdGhlIFRpbWVvdXQg dmFsdWUgDQo+IGJhc2VkIG9uIHRoZSBwcm9wZXJ0aWVzIG9mIFVBUlQgaW4gVGVybWluYWxEcml2 ZXJCaW5kaW5nU3RhcnQoKS8gDQo+IFRlcm1pbmFsQ29uSW5UaW1lckhhbmRsZXIoKS4NCj4gDQo+ ICAgU2VyaWFsSW5UaW1lT3V0ID0gMDsNCj4gICBpZiAoTW9kZS0+QmF1ZFJhdGUgIT0gMCkgew0K PiAgICAgLy8NCj4gICAgIC8vIEFjY29yZGluZyB0byBCQVVEIHJhdGUgdG8gY2FsY3VsYXRlIHRo ZSB0aW1lb3V0IHZhbHVlLg0KPiAgICAgLy8NCj4gICAgIFNlcmlhbEluVGltZU91dCA9ICgxICsg TW9kZS0+RGF0YUJpdHMgKyBNb2RlLT5TdG9wQml0cykgKg0KPiAgICAgICAgICAgICAgICAgICAg ICAgMiAqIDEwMDAwMDAgLyAoVUlOVE4pIE1vZGUtPkJhdWRSYXRlOw0KPiAgIH0NCj4gDQo+IEZv ciBleGFtcGxlLCBiYXNlZCBvbiB0aGUgUENEIHZhbHVlcyBvZiBQY2RVYXJ0RGVmYXVsdEJhdWRS YXRlLCANCj4gUGNkVWFydERlZmF1bHREYXRhQml0cyBhbmQgUGNkVWFydERlZmF1bHRTdG9wQml0 cywgU2VyaWFsSW5UaW1lT3V0ID0NCj4gKDEgKyA4ICArIDEpICogMiAqIDEwMDAwMDAgLyAoVUlO VE4pIDExNTIwMCA9IDE3MyAodXMpLg0KPiANCj4gV2hlbiBTZXJpYWxEeGUgaXMgdXNlZCwNCj4g VGVybWluYWxEcml2ZXJCaW5kaW5nU3RhcnQoKS9UZXJtaW5hbENvbkluVGltZXJIYW5kbGVyKCkg LT4NCj4gICBTZXJpYWxJby0+U2V0QXR0cmlidXRlcygpIC0+DQo+ICAgICBTZXJpYWxTZXRBdHRy aWJ1dGVzKCkgLT4NCj4gICAgICAgU2VyaWFsUG9ydFNldEF0dHJpYnV0ZXMoKQ0KPiANCj4gU29t ZSBpbXBsZW1lbnRhdGlvbnMgb2YgU2VyaWFsUG9ydFNldEF0dHJpYnV0ZXMoKSBjb3VsZCBoYW5k bGUgdGhlIA0KPiBpbnB1dCBwYXJhbWV0ZXJzIGFuZCByZXR1cm4gUkVUVVJOX1NVQ0NFU1MsIGZv ciBleGFtcGxlIA0KPiBCYXNlU2VyaWFsUG9ydExpYjE2NTUwLCB0aGVuIFRpbWVvdXQgdmFsdWUg d2lsbCBiZSBjaGFuZ2VkIHRvIDE3MyANCj4gKHVzKSwgbm8gInNsb3cgZG93biIgd2lsbCBiZSBv YnNlcnZlZC4NCj4gQnV0IHNvbWUgaW1wbGVtZW50YXRpb25zIG9mIFNlcmlhbFBvcnRTZXRBdHRy aWJ1dGVzKCkganVzdCByZXR1cm4gDQo+IFJFVFVSTl9VTlNVUFBPUlRFRCwgZm9yIGV4YW1wbGUg WGVuQ29uc29sZVNlcmlhbFBvcnRMaWIsIHRoZW4gVGltZW91dCANCj4gdmFsdWUgd2lsbCBiZSBu b3QgY2hhbmdlZCBhbmQga2VwdCAxMDAwMDAwICgxcyksICJzbG93IGRvd24iIHdpbGwgYmUgDQo+ IG9ic2VydmVkLg0KPiANCj4gU2VyaWFsUG9ydExpYiBpbnN0YW5jZSBjYW4gYmUgZW5oYW5jZWQg dG8gMS4gSGFuZGxlIHRoZSBpbnB1dCANCj4gcGFyYW1ldGVycyBhbmQgcmV0dXJuIHN0YXR1cyBh Y2NvcmRpbmdseSBpbnN0ZWFkIG9mIGp1c3QgcmV0dXJuaW5nIA0KPiBSRVRVUk5fVU5TVVBQT1JU RUQgaW4gU2VyaWFsUG9ydFNldEF0dHJpYnV0ZXMoKS4NCj4gMi4gSnVzdCByZXR1cm4gUkVUVVJO X1NVQ0NFU1MgaW5zdGVhZCBvZiBSRVRVUk5fVU5TVVBQT1JURUQgaW4NCj4gU2VyaWFsUG9ydFNl dEF0dHJpYnV0ZXMoKSBpZiB0aGUgaW5zdGFuY2UgZG9lcyBub3QgY2FyZSB0aGUgaW5wdXQgDQo+ IHBhcmFtZXRlcnMgYXQgYWxsLg0KPiANCj4gQW5kIFNlcmlhbER4ZSBjYW4gYWxzbyBiZSBlbmhh bmNlZCBsaWtlIHRoaXMgcGF0Y2ggdG8gYmUgbW9yZSByb2J1c3QgDQo+IHRvIGhhbmRsZSBUaW1l b3V0IGNoYW5nZS4NCj4gDQo+IENjOiBKdWxpZW4gR3JhbGwgPGp1bGllbi5ncmFsbEBsaW5hcm8u b3JnPg0KPiBDYzogTGFzemxvIEVyc2VrIDxsZXJzZWtAcmVkaGF0LmNvbT4NCj4gQ2M6IFJ1aXl1 IE5pIDxydWl5dS5uaUBpbnRlbC5jb20+DQo+IENvbnRyaWJ1dGVkLXVuZGVyOiBUaWFub0NvcmUg Q29udHJpYnV0aW9uIEFncmVlbWVudCAxLjENCj4gU2lnbmVkLW9mZi1ieTogU3RhciBaZW5nIDxz dGFyLnplbmdAaW50ZWwuY29tPg0KPiAtLS0NCj4gIE1kZU1vZHVsZVBrZy9Vbml2ZXJzYWwvU2Vy aWFsRHhlL1NlcmlhbElvLmMgfCAxNiArKysrKysrKysrKysrKystDQo+ICAxIGZpbGUgY2hhbmdl ZCwgMTUgaW5zZXJ0aW9ucygrKSwgMSBkZWxldGlvbigtKQ0KPiANCj4gZGlmZiAtLWdpdCBhL01k ZU1vZHVsZVBrZy9Vbml2ZXJzYWwvU2VyaWFsRHhlL1NlcmlhbElvLmMgDQo+IGIvTWRlTW9kdWxl UGtnL1VuaXZlcnNhbC9TZXJpYWxEeGUvU2VyaWFsSW8uYw0KPiBpbmRleCBlYmNkOTI3MjYzMTQu LjA2MGVhNTZjMmIxYSAxMDA2NDQNCj4gLS0tIGEvTWRlTW9kdWxlUGtnL1VuaXZlcnNhbC9TZXJp YWxEeGUvU2VyaWFsSW8uYw0KPiArKysgYi9NZGVNb2R1bGVQa2cvVW5pdmVyc2FsL1NlcmlhbER4 ZS9TZXJpYWxJby5jDQo+IEBAIC0yODUsNyArMjg1LDIxIEBAIFNlcmlhbFNldEF0dHJpYnV0ZXMg KA0KPiAgDQo+ICAgIFN0YXR1cyA9IFNlcmlhbFBvcnRTZXRBdHRyaWJ1dGVzICgmQmF1ZFJhdGUs ICZSZWNlaXZlRmlmb0RlcHRoLCAmVGltZW91dCwgJlBhcml0eSwgJkRhdGFCaXRzLCAmU3RvcEJp dHMpOw0KPiAgICBpZiAoRUZJX0VSUk9SIChTdGF0dXMpKSB7DQo+IC0gICAgcmV0dXJuIFN0YXR1 czsNCj4gKyAgICAvLw0KPiArICAgIC8vIElmIGl0IGlzIGp1c3QgdG8gc2V0IFRpbWVvdXQgdmFs dWUgYW5kIHVuc3VwcG9ydGVkIGlzIHJldHVybmVkLA0KPiArICAgIC8vIGRvIG5vdCByZXR1cm4g ZXJyb3IuDQo+ICsgICAgLy8NCj4gKyAgICBpZiAoKFN0YXR1cyA9PSBFRklfVU5TVVBQT1JURUQp ICYmDQo+ICsgICAgICAgIChUaGlzLT5Nb2RlLT5UaW1lb3V0ICAgICAgICAgICE9IFRpbWVvdXQp ICYmDQo+ICsgICAgICAgIChUaGlzLT5Nb2RlLT5SZWNlaXZlRmlmb0RlcHRoID09IFJlY2VpdmVG aWZvRGVwdGgpICYmDQo+ICsgICAgICAgIChUaGlzLT5Nb2RlLT5CYXVkUmF0ZSAgICAgICAgID09 IEJhdWRSYXRlKSAmJg0KPiArICAgICAgICAoVGhpcy0+TW9kZS0+RGF0YUJpdHMgICAgICAgICA9 PSAoVUlOVDMyKSBEYXRhQml0cykgJiYNCj4gKyAgICAgICAgKFRoaXMtPk1vZGUtPlBhcml0eSAg ICAgICAgICAgPT0gKFVJTlQzMikgUGFyaXR5KSAmJg0KPiArICAgICAgICAoVGhpcy0+TW9kZS0+ U3RvcEJpdHMgICAgICAgICA9PSAoVUlOVDMyKSBTdG9wQml0cykpIHsNCj4gKyAgICAgIFN0YXR1 cyA9IEVGSV9TVUNDRVNTOw0KPiArICAgIH0gZWxzZSB7DQo+ICsgICAgICByZXR1cm4gU3RhdHVz Ow0KPiArICAgIH0NCj4gICAgfQ0KPiAgDQo+ICAgIC8vDQo+IA0KDQppcyB0aGUgU2VyaWFsUG9y dFNldEF0dHJpYnV0ZXMoKSBsaWJyYXJ5IEFQSSBhbGxvd2VkIHRvIG92ZXJ3cml0ZSAtLSBwb3Nz aWJseSBldmVuIHdpdGggZ2FyYmFnZSAtLSB0aGUgSU4gT1VUIHBhcmFtZXRlcnMgaWYgaXQgZmFp bHM/DQpQcmFjdGljZSBpcyBpbmNvbnNpc3RlbnQgb24gdGhpczsgc29tZSBsaWJyYXJ5IEFQSXMg ZXhwbGljaXRseSAidGFpbnQiDQpvdXRwdXQgcGFyYW1ldGVycyBvbiBlcnJvciwgd2hpbGUgb3Ro ZXJzIGV4cGxpY2l0bHkgInByZXNlcnZlIg0KaW5wdXQvb3V0cHV0IHBhcmFtZXRlcnMgb24gZXJy b3IuIFlldCBvdGhlcnMgKGEgZmV3IGV4Y2VwdGlvbmFsIEFQSXMpIG91dHB1dCB2YWxpZCAvIHNl bnNpYmxlIHZhbHVlcyAqZXZlbiogb24gZXJyb3IuIFRoaXMgQVBJIGluIHBhcnRpY3VsYXIgcHJv bWlzZXMgbmVpdGhlciAoaW4gdGhlIGxpYiBjbGFzcyBoZWFkZXIpLCBzbyBJIGRvbid0IGtub3cu DQoNCklmIGl0IGlzIG5vdCBtdWNoIGJ1cmRlbiwgSSdkIHByZWZlciBjb21wYXJpc29ucyBhZ2Fp bnN0IHRoZSBvcmlnaW5hbCBwYXJhbWV0ZXJzLCBub3QgYWdhaW5zdCB0aG9zZSB0aGF0IHdlcmUg cG9zc2libHkgbW9kaWZpZWQgYnkgU2VyaWFsUG9ydFNldEF0dHJpYnV0ZXMoKSwgYmVmb3JlIGl0 IHJldHVybmVkIEVGSV9VTlNVUFBPUlRFRC4NCg0KSSdsbCBsZWF2ZSBpdCB1cCB0byB5b3UgdG8g ZGVjaWRlLg0KDQpSZXZpZXdlZC1ieTogTGFzemxvIEVyc2VrIDxsZXJzZWtAcmVkaGF0LmNvbT4N Cg0KSnVsaWVuLCB3aWxsIHlvdSB0ZXN0IHRoaXM/DQoNClRoYW5rcyENCkxhc3psbw0K