From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <david.hunt@intel.com>
Received: from mga03.intel.com (mga03.intel.com [134.134.136.65])
 by dpdk.org (Postfix) with ESMTP id 71C002C22
 for <dev@dpdk.org>; Wed,  1 Jun 2016 11:39:51 +0200 (CEST)
Received: from orsmga001.jf.intel.com ([10.7.209.18])
 by orsmga103.jf.intel.com with ESMTP; 01 Jun 2016 02:39:50 -0700
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.26,400,1459839600"; d="scan'208";a="966506383"
Received: from dhunt5-mobl.ger.corp.intel.com (HELO [10.237.220.80])
 ([10.237.220.80])
 by orsmga001.jf.intel.com with ESMTP; 01 Jun 2016 02:39:50 -0700
To: Olivier MATZ <olivier.matz@6wind.com>,
 Jan Viktorin <viktorin@rehivetech.com>
References: <1463665501-18325-2-git-send-email-david.hunt@intel.com>
 <20160523143511.7d30699b@pcviktorin.fit.vutbr.cz>
 <574D54D6.1080409@intel.com>
 <20160531140652.018a03de@pcviktorin.fit.vutbr.cz>
 <574D95E9.4020504@intel.com> <574DF6DB.4050905@6wind.com>
Cc: dev@dpdk.org, yuanhan.liu@linux.intel.com, pmatilai@redhat.com,
 jerin.jacob@caviumnetworks.com
From: "Hunt, David" <david.hunt@intel.com>
Message-ID: <574EAD64.9030506@intel.com>
Date: Wed, 1 Jun 2016 10:39:48 +0100
User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:38.0) Gecko/20100101
 Thunderbird/38.6.0
MIME-Version: 1.0
In-Reply-To: <574DF6DB.4050905@6wind.com>
Content-Type: text/plain; charset=windows-1252; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [dpdk-dev] [dpdk-dev,v5,1/3] mempool: support external handler
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: patches and discussions about DPDK <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Wed, 01 Jun 2016 09:39:51 -0000



On 5/31/2016 9:40 PM, Olivier MATZ wrote:

[...]

>>>>>> +/** Structure defining a mempool handler. */
>>>>> Later in the text, I suggested to rename rte_mempool_handler to
>>>>> rte_mempool_ops.
>>>>> I believe that it explains the purpose of this struct better. It
>>>>> would improve
>>>>> consistency in function names (the *_ext_* mark is very strange and
>>>>> inconsistent).
>>>> I agree. I've gone through all the code and renamed to
>>>> rte_mempool_handler_ops.
>>> Ok. I meant rte_mempool_ops because I find the word "handler" to be
>>> redundant.
>> I prefer the use of the word handler, unless others also have opinions
>> either way?
> Well, I think rte_mempool_ops is clear enough, and shorter,
> so I'd vote for it.
>

OK, I've just changed it. It will be in next revision. :)

Regards,
Dave.