From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id D07DE237 for ; Sat, 2 Dec 2017 00:51:53 +0100 (CET) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga104.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 01 Dec 2017 15:51:52 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.45,347,1508828400"; d="scan'208";a="8185702" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.241.225.115]) ([10.241.225.115]) by FMSMGA003.fm.intel.com with ESMTP; 01 Dec 2017 15:51:52 -0800 To: Bruce Richardson , "Ananyev, Konstantin" Cc: Thomas Monjalon , "dev@dpdk.org" , "vladz@cloudius-systems.com" References: <20171201022957.64329-1-ferruh.yigit@intel.com> <20171201022957.64329-7-ferruh.yigit@intel.com> <20171201103308.GB10384@bricha3-MOBL3.ger.corp.intel.com> <2601191342CEEE43887BDE71AB9772585FAC3718@irsmsx105.ger.corp.intel.com> <20171201131702.GA2552@bricha3-MOBL3.ger.corp.intel.com> From: Ferruh Yigit Message-ID: <964acb9f-28ee-9c69-fd02-de7d3f09e429@intel.com> Date: Fri, 1 Dec 2017 15:51:52 -0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 MIME-Version: 1.0 In-Reply-To: <20171201131702.GA2552@bricha3-MOBL3.ger.corp.intel.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH 7/7] ethdev: use opaque user callback object 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: , X-List-Received-Date: Fri, 01 Dec 2017 23:51:54 -0000 On 12/1/2017 5:17 AM, Bruce Richardson wrote: > On Fri, Dec 01, 2017 at 11:22:12AM +0000, Ananyev, Konstantin wrote: >> >> >>> -----Original Message----- >>> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Bruce Richardson >>> Sent: Friday, December 1, 2017 10:33 AM >>> To: Yigit, Ferruh >>> Cc: Thomas Monjalon ; dev@dpdk.org; vladz@cloudius-systems.com >>> Subject: Re: [dpdk-dev] [PATCH 7/7] ethdev: use opaque user callback object >>> >>> On Fri, Dec 01, 2017 at 02:29:57AM +0000, Ferruh Yigit wrote: >>>> "struct rte_eth_rxtx_callback" is defined as internal data structure but >>>> used in public APIs. >>>> >>>> Checking the API documentation shows that intention was using this >>>> object as opaque object. Data structure only used in delete APIs which >>>> doesn't require to know the internals of the data structure. >>>> >>>> Converting callback parameter in API to void pointer should not require >>>> any modification in user application because this data structure was >>>> already marked as internal and only should be used as pointer in >>>> application. >>>> >>>> Signed-off-by: Ferruh Yigit >>>> --- >>> >>> I disagree on this patch. The structure itself is not exposed, only the >>> name, since it is only passed around as a pointer, so there is no need >>> to change the parameters to void pointer. It's a named opaque type. >> >> Personally I think it would be better to do visa-versa: >> make rte_eth_add_(rx|tx)_callback() to return struct rte_eth_rxtx_callback * >> instead of void *. >> Konstantin >> > I didn't realise that it did, so definite +1 to that suggestion. No issue on having a named opaque type, but unfortunately struct is exposed because of inline functions again. It has been moved into rte_ethdev_core.h but accessible by applications. And since intention is an opaque type, because of "void *" return types, I thought it is better to hide type completely so that application can't access details.