From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by dpdk.space (Postfix) with ESMTP id BBE99A0096 for ; Thu, 9 May 2019 01:01:29 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 7D06F37AF; Thu, 9 May 2019 01:01:29 +0200 (CEST) Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id 8ED2B3772 for ; Thu, 9 May 2019 01:01:27 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 08 May 2019 16:01:26 -0700 X-ExtLoop1: 1 Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by fmsmga008.fm.intel.com with ESMTP; 08 May 2019 16:01:26 -0700 Received: from FMSMSX109.amr.corp.intel.com (10.18.116.9) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.408.0; Wed, 8 May 2019 16:01:26 -0700 Received: from fmsmsx115.amr.corp.intel.com ([169.254.4.68]) by FMSMSX109.amr.corp.intel.com ([169.254.15.198]) with mapi id 14.03.0415.000; Wed, 8 May 2019 16:01:25 -0700 From: "Carrillo, Erik G" To: "Burakov, Anatoly" , "rsanford@akamai.com" , "thomas@monjalon.net" CC: "dev@dpdk.org" Thread-Topic: [PATCH v2] timer: fix resource leak in finalize Thread-Index: AQHVBMSSHkt2fsGJK02BXBn6e5VAxqZgLxXwgAEywgCAAHVn0A== Date: Wed, 8 May 2019 23:01:25 +0000 Message-ID: References: <1556737217-24338-1-git-send-email-erik.g.carrillo@intel.com> <1556924082-22535-1-git-send-email-erik.g.carrillo@intel.com> <7baed0b9-432f-be86-5e39-68035bc309a4@intel.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.0.600.7 dlp-reaction: no-action x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiNjlkZDJlYTEtYmU1ZS00NzJkLTk2NmItNzU4MGNhOGRjZDFiIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiemlZXC95Tkd6SU04OWJsUWl2YTlvMGZPVWU1WHJXSEZ1NXY1VGt0UjdicmRSTldTUWpEcUlxRDd5XC9EOUpUWDNPIn0= x-ctpclassification: CTP_NT x-originating-ip: [10.1.200.108] Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: base64 MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v2] timer: fix resource leak in finalize X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Message-ID: <20190508230125.hnFuKsMZ-Nb5E8To5wZzI68aLPpsbCfikhKo2hvlry0@z> DQoNCj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gRnJvbTogQnVyYWtvdiwgQW5hdG9s eQ0KPiBTZW50OiBXZWRuZXNkYXksIE1heSA4LCAyMDE5IDM6NTAgQU0NCj4gVG86IENhcnJpbGxv LCBFcmlrIEcgPGVyaWsuZy5jYXJyaWxsb0BpbnRlbC5jb20+OyByc2FuZm9yZEBha2FtYWkuY29t Ow0KPiB0aG9tYXNAbW9uamFsb24ubmV0DQo+IENjOiBkZXZAZHBkay5vcmcNCj4gU3ViamVjdDog UmU6IFtQQVRDSCB2Ml0gdGltZXI6IGZpeCByZXNvdXJjZSBsZWFrIGluIGZpbmFsaXplDQo+IA0K PiBPbiAwNy1NYXktMTkgMTE6MDQgUE0sIENhcnJpbGxvLCBFcmlrIEcgd3JvdGU6DQo+ID4gSGkg QW5hdG9seSwNCj4gPg0KPiA+IFRoYW5rcyBmb3IgdGhlIHJldmlldy4gIENvbW1lbnRzIGluLWxp bmU6DQo+ID4NCj4gPiA8Li4uc25pcHBlZC4uLj4NCj4gPg0KPiA+Pj4gICAgI2RlZmluZSBSVEVf TUFYX0RBVEFfRUxTIDY0DQo+ID4+PiArc3RhdGljIGNvbnN0IHN0cnVjdCBydGVfbWVtem9uZSAq cnRlX3RpbWVyX2RhdGFfbXo7IHN0YXRpYw0KPiA+Pj4gK3J0ZV9hdG9taWMxNl90ICpydGVfdGlt ZXJfbXpfcmVmY250Ow0KPiA+Pj4gICAgc3RhdGljIHN0cnVjdCBydGVfdGltZXJfZGF0YSAqcnRl X3RpbWVyX2RhdGFfYXJyOw0KPiA+Pj4gICAgc3RhdGljIGNvbnN0IHVpbnQzMl90IGRlZmF1bHRf ZGF0YV9pZDsNCj4gPj4+ICAgIHN0YXRpYyB1aW50MzJfdCBydGVfdGltZXJfc3Vic3lzdGVtX2lu aXRpYWxpemVkOyBAQCAtMTU1LDYgKzE1Nyw3DQo+ID4+PiBAQA0KPiA+Pj4gcnRlX3RpbWVyX3N1 YnN5c3RlbV9pbml0X3YxOTA1KHZvaWQpDQo+ID4+PiAgICAJc3RydWN0IHJ0ZV90aW1lcl9kYXRh ICpkYXRhOw0KPiA+Pj4gICAgCWludCBpLCBsY29yZV9pZDsNCj4gPj4+ICAgIAlzdGF0aWMgY29u c3QgY2hhciAqbXpfbmFtZSA9ICJydGVfdGltZXJfbXoiOw0KPiA+Pj4gKwlzaXplX3QgZGF0YV9h cnJfc2l6ZSA9IFJURV9NQVhfREFUQV9FTFMgKg0KPiA+Pj4gK3NpemVvZigqcnRlX3RpbWVyX2Rh dGFfYXJyKTsNCj4gPj4NCj4gPj4gbml0cGlja2luZywgYnV0Li4uIGNvbnN0Pw0KPiA+Pg0KPiA+ DQo+ID4gTm8gcHJvYmxlbSAtIEknbGwgbWFrZSB0aGlzIGNoYW5nZSBpZiB0aGlzIGxpbmUgcGVy c2lzdHMgaW50byB0aGUgbmV4dCB2ZXJzaW9uLg0KPiA+DQo+ID4gPC4uLnNuaXBwZWQuLi4+DQo+ ID4NCj4gPj4+DQo+ID4+PiBAQCAtMjA1LDggKzIxNiwxMSBAQA0KPiA+PiBCSU5EX0RFRkFVTFRf U1lNQk9MKHJ0ZV90aW1lcl9zdWJzeXN0ZW1faW5pdCwgX3YxOTA1LCAxOS4wNSk7DQo+ID4+PiAg ICB2b2lkIF9fcnRlX2V4cGVyaW1lbnRhbA0KPiA+Pj4gICAgcnRlX3RpbWVyX3N1YnN5c3RlbV9m aW5hbGl6ZSh2b2lkKQ0KPiA+Pj4gICAgew0KPiA+Pj4gLQlpZiAocnRlX3RpbWVyX2RhdGFfYXJy KQ0KPiA+Pj4gLQkJcnRlX2ZyZWUocnRlX3RpbWVyX2RhdGFfYXJyKTsNCj4gPj4+ICsJaWYgKCFy dGVfdGltZXJfc3Vic3lzdGVtX2luaXRpYWxpemVkKQ0KPiA+Pj4gKwkJcmV0dXJuOw0KPiA+Pj4g Kw0KPiA+Pj4gKwlpZiAocnRlX2F0b21pYzE2X2RlY19hbmRfdGVzdChydGVfdGltZXJfbXpfcmVm Y250KSkNCj4gPj4+ICsJCXJ0ZV9tZW16b25lX2ZyZWUocnRlX3RpbWVyX2RhdGFfbXopOw0KPiA+ Pg0KPiA+PiBJIHRoaW5rIHRoZXJlJ3MgYSByYWNlIGhlcmUuIFlvdSBtYXkgZ2V0IHByZWVtcHRl ZCBhZnRlciB0ZXN0IGJ1dA0KPiA+PiBiZWZvcmUgZnJlZSwgd2hlcmUgYW5vdGhlciBzZWNvbmRh cnkgY291bGQgaW5pdGlhbGl6ZS4gQXMgZmFyIGFzIGkNCj4gPj4ga25vdywgd2UgYWxzbw0KPiA+ DQo+ID4gSW5kZWVkLCB0aGFua3MgZm9yIGNhdGNoaW5nIHRoaXMuDQo+ID4NCj4gPj4gc3VwcG9y dCBhIGNhc2Ugd2hlbiBzZWNvbmRhcnkgaW5pdGlhbGl6ZXMgYWZ0ZXIgcHJpbWFyeSBzdG9wcyBy dW5uaW5nLg0KPiA+Pg0KPiA+PiBMZXQncyBldmVuIHN1cHBvc2UgdGhhdCB3ZSBhbGxvdyBzZWNv bmRhcnkgcHJvY2Vzc2VzIHRvIGluaXRpYWxpemUNCj4gPj4gdGhlIHRpbWVyIHN1YnN5c3RlbSBi eSByZXNlcnZpbmcgbWVtem9uZSBhbmQgY2hlY2tpbmcgcnRlX2Vycm5vLiBZb3UNCj4gPj4gd291 bGQgc3RpbGwgaGF2ZSBhIGNoYW5jZSBvZiB0d28gaW5pdC9kZWluaXQgY29uZmxpY3RpbmcsIGJl Y2F1c2UNCj4gPj4gdGhlcmUncyBhIGhvbGUgYmV0d2VlbiBtZW16b25lIGFsbG9jYXRpb24gYW5k IGF0b21pYyBpbmNyZW1lbnQuDQo+ID4+DQo+ID4+IEkgZG9uJ3QgdGhpbmsgdGhpcyByYWNlIGNh biBiZSByZXNvbHZlZCBpbiBhIHNhZmUgd2F5LCBzbyB3ZSBtaWdodA0KPiA+PiBqdXN0IGhhdmUg dG8gc2V0dGxlIGZvciBhIG1lbW9yeSBsZWFrLg0KPiA+Pg0KPiA+DQo+ID4gSSBkb24ndCBzZWUg YSBzb2x1dGlvbiBoZXJlIGN1cnJlbnRseSBlaXRoZXIuICBJJ2xsIGxvb2sgYXQgcmVtb3ZpbmcN Cj4gPiB0aGUgbWVtem9uZV9mcmVlKCkgY2FsbCBhbmQgcG9zc2libHkgdGhlDQo+ID4gcnRlX3Rp bWVyX3N1YnN5c3RlbV9maW5hbGl6ZSgpIEFQSSwgc2luY2UgaXQgc2VlbXMgbGlrZSB0aGVyZSdz IG5vIHJlYXNvbg0KPiBmb3IgaXQgdG8gZXhpc3QgaWYgaXQgY2FuJ3QgZnJlZSB0aGUgYWxsb2Nh dGlvbnMuDQo+IA0KPiBJIHdvbmRlciBpZiB0aGVyZSBhcmUgb3RoZXIgcGxhY2VzIGluIERQREsg d2hlcmUgdGhpcyBwYXR0ZXJuIGlzIHVzZWQuDQo+IA0KPiBUZWNobmljYWxseSwgdGhpcyBraW5k IG9mIHRoaW5nIC9jb3VsZC8gYmUgcmVzb2x2ZWQgYnkgaGF2aW5nIHNvbWV0aGluZyBpbiBvdXIN Cj4gbXVsdGlwcm9jZXNzIHNoYXJlZCBtZW1vcnkgb3V0c2lkZSBvZiBEUERLIGhlYXAuIEkuZS4g c3RvcmUgc29tZXRoaW5nIGluDQo+IHJ0ZV9lYWxfbWVtY29uZmlnIGxpa2Ugc29tZSBvdGhlciB0 aGluZ3MgZG8uIFRoaXMgY2hhbmdlLCBob3dldmVyLCB3b3VsZA0KPiByZXF1aXJlIGFuIEFCSSBi cmVhaywgc28gd2hpbGUgY2hhbmdpbmcgdGhpcyBwYXJ0aWN1bGFyIEFQSSB3b24ndCBuZWVkIGEN Cj4gZGVwcmVjYXRpb24gbm90aWNlLCB0aGUgY2hhbmdlIGl0c2VsZiB3b3VsZC4NCg0KSSBsaWtl IHRoaXMgaWRlYTsgdGhhbmtzIGZvciB0aGUgc3VnZ2VzdGlvbi4gIEkgd2VudCBhaGVhZCBhbmQg c3VibWl0dGVkIGEgbmV3IHZlcnNpb24gDQpvZiB0aGlzIHBhdGNoIHdpdGggdGhpcyBhcHByb2Fj aC4gIEl0J3MgZGVwZW5kZW50IG9uIG9uZSBvdGhlciBuZXcgcGF0Y2ggdGhhdCBhbGxvd3MNCnNl Y29uZGFyeSBwcm9jZXNzZXMgdG8gcmVzZXJ2ZSB0aGUgbWVtem9uZS4gIEkgYWxzbyBzdWJtaXR0 ZWQgYSBkZXByZWNhdGlvbg0Kbm90aWNlIGZvciB0aGUgQUJJIGJyZWFrIGZvciB0aGUgY2hhbmdl IHRvIHJ0ZV9tZW1fY29uZmlnLg0KDQpUaG9tYXMsIEkgc3VzcGVjdCB0aGF0IEkgc2hvdWxkIG1h cmsgdjMgb2YgdGhpcyBwYXRjaCBhcyAiZGVmZXJyZWQiLCBzbyBJJ3ZlIGdvbmUgYWhlYWQgYW5k DQpkb25lIHRoYXQsIGJ1dCBsZXQgbWUga25vdyBpZiB0aGF0J3Mgd3JvbmcuDQoNClRoYW5rcywN CkVyaWsNCg0KPiANCj4gPg0KPiA+IFJlZ2FyZHMsDQo+ID4gRXJpaw0KPiA+DQo+ID4+Pg0KPiA+ Pj4gICAgCXJ0ZV90aW1lcl9zdWJzeXN0ZW1faW5pdGlhbGl6ZWQgPSAwOw0KPiA+Pj4gICAgfQ0K PiA+Pj4NCj4gPj4NCj4gPj4NCj4gPj4gLS0NCj4gPj4gVGhhbmtzLA0KPiA+PiBBbmF0b2x5DQo+ IA0KPiANCj4gLS0NCj4gVGhhbmtzLA0KPiBBbmF0b2x5DQo=