From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Wiles, Keith" Subject: Re: [EXT] Re: [PATCH v2 2/2] eal: roundup tsc frequency when estimating it Date: Sat, 16 Mar 2019 17:18:27 +0000 Message-ID: <96FB992B-82D4-4B5D-97B1-5E255DD0621F@intel.com> References: <20181129083138.23029-1-pbhagavatula@caviumnetworks.com> <20190316070302.32432-1-pbhagavatula@marvell.com> <20190316070302.32432-2-pbhagavatula@marvell.com> <86480C64-4ECD-4F4D-968E-5716D02D7D83@intel.com> <3e56d8dbbddfde0c48a26c6257af9c43a42f99dd.camel@marvell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Cc: "stephen@networkplumber.org" , "thomas@monjalon.net" , "dev@dpdk.org" , Jerin Jacob Kollanukkaran To: Pavan Nikhilesh Bhagavatula Return-path: Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by dpdk.org (Postfix) with ESMTP id F1B7311A4 for ; Sat, 16 Mar 2019 18:18:29 +0100 (CET) In-Reply-To: <3e56d8dbbddfde0c48a26c6257af9c43a42f99dd.camel@marvell.com> Content-Language: en-US Content-ID: <969F83D32CC41F488590B2AC3A2D3F2F@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" DQoNCj4gT24gTWFyIDE2LCAyMDE5LCBhdCAxMDowNiBBTSwgUGF2YW4gTmlraGlsZXNoIEJoYWdh dmF0dWxhIDxwYmhhZ2F2YXR1bGFAbWFydmVsbC5jb20+IHdyb3RlOg0KPiANCj4gT24gU2F0LCAy MDE5LTAzLTE2IGF0IDE0OjQyICswMDAwLCBXaWxlcywgS2VpdGggd3JvdGU6DQo+Pj4gT24gTWFy IDE2LCAyMDE5LCBhdCAyOjAzIEFNLCBQYXZhbiBOaWtoaWxlc2ggQmhhZ2F2YXR1bGEgPA0KPj4+ IHBiaGFnYXZhdHVsYUBtYXJ2ZWxsLmNvbT4gd3JvdGU6DQo+Pj4gDQo+Pj4gRnJvbTogUGF2YW4g TmlraGlsZXNoIDxwYmhhZ2F2YXR1bGFAbWFydmVsbC5jb20+DQo+Pj4gDQo+Pj4gV2hlbiBlc3Rp bWF0aW5nIHRzYyBmcmVxdWVuY3kgdXNpbmcgc2xlZXAvZ2V0dGltZSByb3VuZCBpdCB1cCB0bw0K Pj4+IHRoZQ0KPj4+IG5lYXJlc3QgbXVsdGlwbGUgb2YgMTBNaHogZm9yIG1vcmUgYWNjdXJhY3ku DQoNCkhvdyBkb2VzIHJvdW5kaW5nIHVwIHRoZSBUU0MgdmFsdWUgYmVjb21lIG1vcmUgYWNjdXJh dGUsIElmIHRoZSB2YWx1ZSBpcyAxIGN5Y2xlcyBtb3JlIHRoZW4gaXQgc2hvdWxkIGJlIHRoZW4g eW91ciBtYWNybyB3b3VsZCByb3VuZCBkb3duIGFuZCBpZiBpdCBpcyAxIGN5Y2xlIGdyZWF0ZXIg dGhhbiAxRTcgaXQgd291bGQgcm91bmQgdXAuDQo+Pj4gDQo+Pj4gU2lnbmVkLW9mZi1ieTogUGF2 YW4gTmlraGlsZXNoIDxwYmhhZ2F2YXR1bGFAbWFydmVsbC5jb20+DQo+Pj4gLS0tDQo+Pj4gVXNl ZnVsIGluIGNhc2Ugb2YgQVJNNjQgaWYgd2UgZW5hYmxlIFJURV9BUk1fRUFMX1JEVFNDX1VTRV9Q TVUsDQo+Pj4gZ2V0X3RzY19mcmVxX2FyY2goKSB3aWxsIHJldHVybiAwIGFzIHRoZXJlIGlzIG5v IGluc3RydWN0aW9uIHRvDQo+Pj4gZGV0ZXJtaW5lDQo+Pj4gdGhlIGNsayBvZiBQTVUgYW5kIGVh bCBmYWxscyBiYWNrIHRvIHNsZWVwKDEpLg0KPj4+IA0KPj4+IGxpYi9saWJydGVfZWFsL2NvbW1v bi9lYWxfY29tbW9uX3RpbWVyLmMgfCA0ICsrLS0NCj4+PiBsaWIvbGlicnRlX2VhbC9saW51eGFw cC9lYWwvZWFsX3RpbWVyLmMgIHwgMiArLQ0KPj4+IDIgZmlsZXMgY2hhbmdlZCwgMyBpbnNlcnRp b25zKCspLCAzIGRlbGV0aW9ucygtKQ0KDQpJdCBhcHBlYXJzIHlvdSBkaWQgbm90IHVzZSB0aGUg aGVhZCBvZiB0aGUgbWFzdGVyIGFzIGxpbnV4YXBwIGlzIG5vdyBqdXN0IGxpbnV4IGFuZCBmcmVl YnNkYXBwIGlzIGZyZWVic2QuIFlvdSBuZWVkIHRvIHJlYmFzZSB0byB0aGUgaGVhZCBvZiBtYXN0 ZXIgYW5kIHNlbmQgYSBuZXcgdmVyc2lvbi4NCj4+PiANCj4+PiBkaWZmIC0tZ2l0IGEvbGliL2xp YnJ0ZV9lYWwvY29tbW9uL2VhbF9jb21tb25fdGltZXIuYw0KPj4+IGIvbGliL2xpYnJ0ZV9lYWwv Y29tbW9uL2VhbF9jb21tb25fdGltZXIuYw0KPj4+IGluZGV4IGRjZjI2YmZlYS4uMTM1OGJiZWQw IDEwMDY0NA0KPj4+IC0tLSBhL2xpYi9saWJydGVfZWFsL2NvbW1vbi9lYWxfY29tbW9uX3RpbWVy LmMNCj4+PiArKysgYi9saWIvbGlicnRlX2VhbC9jb21tb24vZWFsX2NvbW1vbl90aW1lci5jDQo+ Pj4gQEAgLTY5LDcgKzY5LDcgQEAgZXN0aW1hdGVfdHNjX2ZyZXEodm9pZCkNCj4+PiAJLyogYXNz dW1lIHRoYXQgdGhlIHNsZWVwKDEpIHdpbGwgc2xlZXAgZm9yIDEgc2Vjb25kICovDQo+Pj4gCXVp bnQ2NF90IHN0YXJ0ID0gcnRlX3JkdHNjKCk7DQo+Pj4gCXNsZWVwKDEpOw0KPj4+IC0JcmV0dXJu IHJ0ZV9yZHRzYygpIC0gc3RhcnQ7DQo+Pj4gKwlyZXR1cm4gUlRFX0FMSUdOX01VTF9ORUFSKHJ0 ZV9yZHRzYygpIC0gc3RhcnQsIDFFNyk7DQoNClRoZSAxRTcgaXMgYSBtYWdpYyBudW1iZXIgY29u dmVydCB0aGlzIHRvIGEgbWVhbmluZ2Z1bCBkZWZpbmUuDQo+Pj4gfQ0KPj4+IA0KPj4+IHZvaWQN Cj4+PiBAQCAtODMsNyArODMsNyBAQCBzZXRfdHNjX2ZyZXEodm9pZCkNCj4+PiAJaWYgKCFmcmVx KQ0KPj4+IAkJZnJlcSA9IGVzdGltYXRlX3RzY19mcmVxKCk7DQo+Pj4gDQo+Pj4gLQlSVEVfTE9H KERFQlVHLCBFQUwsICJUU0MgZnJlcXVlbmN5IGlzIH4lIiBQUkl1NjQgIiBLSHpcbiIsIGZyZXEN Cj4+PiAvIDEwMDApOw0KPj4+ICsJUlRFX0xPRyhJTkZPLCBFQUwsICJUU0MgZnJlcXVlbmN5IGlz IH4lIiBQUkl1NjQgIiBIelxuIiwgZnJlcSk7DQo+Pj4gCWVhbF90c2NfcmVzb2x1dGlvbl9oeiA9 IGZyZXE7DQo+IA0KPiBJIG1pc3NlZCB0aGlzIGxvZyB3aWxsIHJlbW92ZSBpdCBpbiB0aGUgbmV4 dCB2ZXJzaW9uLg0KPiANCj4+PiB9DQo+Pj4gDQo+Pj4gZGlmZiAtLWdpdCBhL2xpYi9saWJydGVf ZWFsL2xpbnV4YXBwL2VhbC9lYWxfdGltZXIuYw0KPj4+IGIvbGliL2xpYnJ0ZV9lYWwvbGludXhh cHAvZWFsL2VhbF90aW1lci5jDQo+Pj4gaW5kZXggYmM4ZjA1MTk5Li44NjRkNmVmMjkgMTAwNjQ0 DQo+Pj4gLS0tIGEvbGliL2xpYnJ0ZV9lYWwvbGludXhhcHAvZWFsL2VhbF90aW1lci5jDQo+Pj4g KysrIGIvbGliL2xpYnJ0ZV9lYWwvbGludXhhcHAvZWFsL2VhbF90aW1lci5jDQo+Pj4gQEAgLTI0 OCw3ICsyNDgsNyBAQCBnZXRfdHNjX2ZyZXEodm9pZCkNCj4+PiANCj4+PiAJCWRvdWJsZSBzZWNz ID0gKGRvdWJsZSlucy9OU19QRVJfU0VDOw0KPj4+IAkJdHNjX2h6ID0gKHVpbnQ2NF90KSgoZW5k IC0gc3RhcnQpL3NlY3MpOw0KPj4+IC0JCXJldHVybiB0c2NfaHo7DQo+Pj4gKwkJcmV0dXJuIFJU RV9BTElHTl9NVUxfTkVBUih0c2NfaHosIDFFNyk7DQo+PiANCj4+IE1heWJlIEkgbWlzc2VkIGFu IGVtYWlsIGFib3V0IHRoaXMsIGJ1dCB3aHkgd291bGQgSSB3YW50IHRoZSBUU0MgaHoNCj4+IHJv dW5kZWQgaGVyZT8gSSBkbyBub3QgbWluZCB0aGUgbWFjcm8ganVzdCB0aGUgZmFjdCB0aGF0IHdl IGFyZQ0KPj4gY2hhbmdpbmcgVFNDIGh6IHZhbHVlLiBJZiB0aGUgVFNDIHZhbHVlIGlzIHdyb25n IHRoZW4gd2UgbmVlZCB0byBmaXgNCj4+IHRoZSB2YWx1ZSwgYnV0IEkgZG8gbm90IHNlZSBpdCBi ZWluZyB3cm9uZyBoZXJlLg0KPiANCj4gU2luY2UgaW4gdGhpcyBmdW5jdGlvbiBuYW5vc2xlZXAg bWlnaHQgbm90IGJlIGN5Y2xlIGFjY3VyYXRlIHdlIG5lZWQgdG8NCj4gcm91bmQgaXQgdXAuDQo+ IA0KPiBQbGVhc2Ugbm90ZSB0aGF0IGVzdGltYXRpb24gb25seSBhcHBsaWVzIHdoZW4gIGdldF90 c2NfZnJlcV9hcmNoKCkNCj4gZmFpbHMuIGkuZSB0aGVyZSBpcyBubyBDUFUgaW5zdHJ1Y3Rpb24g dGhhdCBzcGVjaWZpZXMgdGhlIGN5Yy9zZWMuDQo+IA0KPiBBcyBJIG1lbnRpb25lZCBpbiB0aGUg cGF0Y2ggbm90ZXMNCj4gIlVzZWZ1bCBpbiBjYXNlIG9mIEFSTTY0IGlmIHdlIGVuYWJsZSBSVEVf QVJNX0VBTF9SRFRTQ19VU0VfUE1VLA0KPiBnZXRfdHNjX2ZyZXFfYXJjaCgpIHdpbGwgcmV0dXJu IDAgYXMgdGhlcmUgaXMgbm8gaW5zdHJ1Y3Rpb24gdG8NCj4gZGV0ZXJtaW5lIHRoZSBjbG9jayBv ZiBQTVUgYW5kIGVhbCBmYWxscyBiYWNrIHRvIHNsZWVwKDEpL25hbm9zbGVlcC7igJ0gDQoNCk9L LCBJIGxvb2tlZCBhdCB0aGUgY2hhbmdlcyBhbmQgdGhlIGNvZGUgZm9yIHNldHRpbmcgdGhlIFRT QyBhZ2Fpbi4gSSB3b3VsZCBoYXZlIG5vdCBoYW5kbGVkIHRoZSBzZXR0aW5nIG9mIFRTQyBpbiB0 aGUgd2F5IGl0IHdhcyBkb25lLCBidXQgdGhhdCBpcyBub3QgeW91ciBwcm9ibGVtLiBJIGFncmVl IHRoZSBjaGFuZ2VzIGRvIGxvb2sgb2ssIHRoZSBvbmx5IHByb2JsZW0gSSBoYXZlIGlzIHRoZSBu ZXcgbWFjcm8gd2lsbCByb3VuZHVwIG9yIGRvd24gZGVwZW5kaW5nIG9uIHRoZSB2YWx1ZS4gSW4g eW91ciBzdGF0ZW1lbnQgeW91IGFyZSB3YW50aW5nIHRvIHJvdW5kdXAgdGhlIHZhbHVlcy4NCg0K SWYgdGhlIHNsZWVwL25hbm9zbGVlcCBpcyBsZXNzIHRoYW4gYSBzZWNvbmQgZm9yIHNvbWUgcmVh c29uLCB0aGVuIHlvdXIgbWFjcm8gd291bGQgcm91bmQgaXQgZG93biBpcyB0aGF0IHdoYXQgd2Ug d2FudGVkPyBJIGd1ZXNzIG15IHBvaW50IGlzIHlvdSBhcmUgYXNzdW1pbmcgdGhlIFRTQyBjYWxj dWxhdGlvbiB3aWxsIGFsd2F5cyBiZSBsZXNzIHRoYW4gYSBzZWNvbmQgKHdpdGggc2xlZXApIGFu ZCB0aGUgbWFjcm8gd291bGQgcm91bmQgdXAgZGVwZW5kaW5nIG9uIHRoZSB2YWx1ZSBjYWxjdWxh dGVkIHVzaW5nIHRoZSBzbGVlcC9uYW5vc2xlZXAuDQoNCkkgd2FzIHBsYXlpbmcgd2l0aCB0aGVz ZSBNVUwgbWFjcm9zIGFuZCBJIGFtIG5vdCBzdXJlIHRoZXkgZG8gd2hhdCB3ZSBleHBlY3QgaW4g dGhlIGNhc2Ugb2YgdGhlIG11bHRpcGxlIHZhbHVlIGlzIG11Y2ggY2xvc2VyIHRvIHRoZSB2YWx1 ZSBwYXNzZWQuDQoNCklmIHdlIGhhdmUgYSB2ID0gMTAwMDEgYW5kIG11bHRpcGxlIHRvIDEwMDAg d2UgaGF2ZSB0aGUgZm9sbG93aW5nOg0KDQpSVEVfQUxJR05fTVVMX0NFSUwoMTAwMDEsIDEwMDAp DQoJKDEwMDAxICsgKDEwMDAgLSAxKSkgLyAoMTAwMCAqIDEwMDApDQoJKDEwMDAxICsgOTk5KSAg ICAgICAgLyAxMDAwMDAwDQoJMjAwMDAgICAgICAgICAgICAgICAgLyAxMDAwMDAwDQpSZXN1bHQ6 IDANCg0KUlRFX0FMSUdOX01VTF9GTE9PUigxMDAwMSwgMTAwMCkNCgkoMTAwMDEgLyAoMTAwMCAq IDEwMDApKQ0KCSgxMDAwMSAvIDEwMDAwMDApDQpSZXN1bHQ6IDANCg0KVW5sZXNzIEkgYW0gd3Jv bmcgaGVyZSB0aGUgdmFsdWUgdiBtdXN0IGJlIG92ZXIgYSAxLDAwMCwwMDAgdG8gbWFrZSB0aGVz ZSBtYWNyb3Mgd29yayBvciB0aGUgdmFsdWUgdiB0byBiZSBncmVhdGVyIHRoYW4gKG11bCAqIG11 bCkgaW4gYWxsIGNhc2VzIG9yIHplcm8gaXMgdGhlIHJlc3VsdC4gSXQgbWF5IHdvcmsgZm9yIHRo ZSBUU0MgdmFsdWVzIGFzIHdlIGFyZSB1c2luZyBhIHNtYWxsIG11bCB2YWx1ZSBjb21wYXJlZCB0 byB0aGUgVFNDIHZhbHVlLiBJZiBEUERLIHdhcyBwb3J0ZWQgdG8gYSBzbG93ZXIgbWFjaGluZSBp dCBjb3VsZCBiZSBhbHNvIHplcm8uDQoNCkkgdGhpbmsgd2UgbmVlZCB0byBmaXggdGhlIG1hY3Jv cyBhbmQgcmV0aGluayBob3cgVFNDIGlzIHNldCBoZXJlLg0KDQo+IA0KPj4+IAl9DQo+Pj4gI2Vu ZGlmDQo+Pj4gCXJldHVybiAwOw0KPj4+IC0tDQo+Pj4gMi4yMS4wDQo+Pj4gDQo+PiANCj4+IFJl Z2FyZHMsDQo+PiBLZWl0aA0KDQpSZWdhcmRzLA0KS2VpdGgNCg0K