From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jerin Jacob <jerin.jacob@caviumnetworks.com>
Subject: Re: [PATCH v2] doc: update ethdev APIs to return named
 opaque type
Date: Sat, 10 Feb 2018 17:32:30 +0530
Message-ID: <20180210120229.GB7298@jerin>
References: <20180209101119.69860-1-ferruh.yigit@intel.com>
 <20180209101818.70041-1-ferruh.yigit@intel.com>
 <2601191342CEEE43887BDE71AB977258905710B0@irsmsx105.ger.corp.intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: "Yigit, Ferruh" <ferruh.yigit@intel.com>,
 Neil Horman <nhorman@tuxdriver.com>,
 "Mcnamara, John" <john.mcnamara@intel.com>,
 "Kovacevic, Marko" <marko.kovacevic@intel.com>,
 "dev@dpdk.org" <dev@dpdk.org>,
 Stephen Hemminger <stephen@networkplumber.org>,
 "Richardson, Bruce" <bruce.richardson@intel.com>,
 Thomas Monjalon <thomas@monjalon.net>
To: "Ananyev, Konstantin" <konstantin.ananyev@intel.com>
Return-path: <dev-bounces@dpdk.org>
Received: from NAM01-SN1-obe.outbound.protection.outlook.com
 (mail-sn1nam01on0052.outbound.protection.outlook.com [104.47.32.52])
 by dpdk.org (Postfix) with ESMTP id 74B091B6F1
 for <dev@dpdk.org>; Sat, 10 Feb 2018 13:02:56 +0100 (CET)
Content-Disposition: inline
In-Reply-To: <2601191342CEEE43887BDE71AB977258905710B0@irsmsx105.ger.corp.intel.com>
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://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: <https://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

-----Original Message-----
> Date: Fri, 9 Feb 2018 13:22:06 +0000
> From: "Ananyev, Konstantin" <konstantin.ananyev@intel.com>
> To: "Yigit, Ferruh" <ferruh.yigit@intel.com>, Neil Horman
>  <nhorman@tuxdriver.com>, "Mcnamara, John" <john.mcnamara@intel.com>,
>  "Kovacevic, Marko" <marko.kovacevic@intel.com>
> CC: "dev@dpdk.org" <dev@dpdk.org>, Stephen Hemminger
>  <stephen@networkplumber.org>, "Richardson, Bruce"
>  <bruce.richardson@intel.com>, Thomas Monjalon <thomas@monjalon.net>
> Subject: Re: [dpdk-dev] [PATCH v2] doc: update ethdev APIs to return named
>  opaque type
> 
> 
> 
> > -----Original Message-----
> > From: Yigit, Ferruh
> > Sent: Friday, February 9, 2018 10:18 AM
> > To: Neil Horman <nhorman@tuxdriver.com>; Mcnamara, John <john.mcnamara@intel.com>; Kovacevic, Marko
> > <marko.kovacevic@intel.com>
> > Cc: dev@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>; Ananyev, Konstantin <konstantin.ananyev@intel.com>; Stephen Hemminger
> > <stephen@networkplumber.org>; Richardson, Bruce <bruce.richardson@intel.com>; Thomas Monjalon <thomas@monjalon.net>
> > Subject: [PATCH v2] doc: update ethdev APIs to return named opaque type
> > 
> > Ethdev APIs to add callback return the callback object as "void *",
> > update return type to actual object type
> > "struct rte_eth_rxtx_callback *"
> > 
> > Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> > ---
> > Cc: Konstantin Ananyev <konstantin.ananyev@intel.com>
> > Cc: Stephen Hemminger <stephen@networkplumber.org>
> > Cc: Bruce Richardson <bruce.richardson@intel.com>
> > Cc: Thomas Monjalon <thomas@monjalon.net>
> > ---
> >  doc/guides/rel_notes/deprecation.rst | 7 +++++++
> >  1 file changed, 7 insertions(+)
> > 
> > diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> > index bbd9456a7..5cb5a00d2 100644
> > --- a/doc/guides/rel_notes/deprecation.rst
> > +++ b/doc/guides/rel_notes/deprecation.rst
> > @@ -49,6 +49,13 @@ Deprecation Notices
> >    rte_eth_dev_get_sec_ctx() is using uint8_t for port_id, which should be
> >    uint16_t.
> > 
> > +* ethdev: functions add rx/tx callback will return named opaque type
> > +  rte_eth_add_rx_callback(), rte_eth_add_first_rx_callback() and
> > +  rte_eth_add_tx_callback() functions currently return callback object as
> > +  "void \*" but APIs to delete callbacks get "struct rte_eth_rxtx_callback \*"
> > +  as parameter. For consistency functions adding callback will return
> > +  "struct rte_eth_rxtx_callback \*" instead of "void * ".
> > +
> >  * i40e: The default flexible payload configuration which extracts the first 16
> >    bytes of the payload for RSS will be deprecated starting from 18.02. If
> >    required the previous behavior can be configured using existing flow
> > --
> 
> Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>

Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>

> 
> > 2.14.3
>