From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerin Jacob Subject: Re: [PATCH v2] doc: announce kvargs API change Date: Fri, 23 Nov 2018 16:31:11 +0000 Message-ID: <20181123163058.GA11828@jerin> References: <20181121154545.14261-1-thomas@monjalon.net> <20181122103223.22837-1-thomas@monjalon.net> <20181122132212.brq3ifximbtu5c5d@platinum> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Cc: Olivier Matz , Thomas Monjalon , "dev@dpdk.org" To: Ferruh Yigit Return-path: Received: from NAM02-SN1-obe.outbound.protection.outlook.com (mail-eopbgr770084.outbound.protection.outlook.com [40.107.77.84]) by dpdk.org (Postfix) with ESMTP id 6960F1B555 for ; Fri, 23 Nov 2018 17:31:13 +0100 (CET) In-Reply-To: Content-Language: en-US Content-ID: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" -----Original Message----- > Date: Thu, 22 Nov 2018 17:08:56 +0000 > From: Ferruh Yigit > To: Olivier Matz , Thomas Monjalon > > CC: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2] doc: announce kvargs API change > User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 > Thunderbird/60.3.1 >=20 >=20 > On 11/22/2018 1:22 PM, Olivier Matz wrote: > > On Thu, Nov 22, 2018 at 11:32:23AM +0100, Thomas Monjalon wrote: > >> After processing a kvlist in rte_kvargs_process(), > >> it may be needed to loop again over kvlist in order to know > >> whether the key is matched or not. > >> In order to simplify implementation of kvargs checks, > >> a new pointer parameter may be used to get the match count. > >> > >> The change of the function prototype would be as below: > >> > >> int > >> rte_kvargs_process(const struct rte_kvargs *kvlist, > >> const char *key_match, > >> + int *match_count, > >> arg_handler_t handler, > >> void *opaque_arg) > >> > >> Signed-off-by: Thomas Monjalon > > > > Maybe "unsigned int" instead of "int". > > > > Apart from this, > > Acked-by: Olivier Matz > > >=20 > Acked-by: Ferruh Yigit Acked-by: Jerin Jacob