DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH v1] doc: add distributor library API change notice
@ 2017-02-06  8:08 David Hunt
  2017-02-07 10:50 ` Bruce Richardson
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: David Hunt @ 2017-02-06  8:08 UTC (permalink / raw)
  To: dev; +Cc: bruce.richardson, David Hunt

Given that the packet distributor library improvements (1) will
not be in 17.02, I plan on doing some consolidation of the
API for burst operation for 17.05, merging the two api's into
one, with options for single or burst operation.

(1) http://dpdk.org/dev/patchwork/patch/19911/

Signed-off-by: David Hunt <david.hunt@intel.com>
---
 doc/guides/rel_notes/deprecation.rst | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 755dc65..925e156 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -55,6 +55,12 @@ Deprecation Notices
   and will be removed in 17.02.
   It is replaced by ``rte_mempool_generic_get/put`` functions.
 
+* lib: distributor library API will be changed to incorporate a burst-
+  oriented API. This will include a change to ``rte_distributor_create``
+  to specify which type of instance to create (single or burst), and
+  additional calls for ``rte_poll_pkt_burst`` and ``rte_return_pkt_burst``,
+  among others.
+
 * ethdev: the legacy filter API, including
   ``rte_eth_dev_filter_supported()``, ``rte_eth_dev_filter_ctrl()`` as well
   as filter types MACVLAN, ETHERTYPE, FLEXIBLE, SYN, NTUPLE, TUNNEL, FDIR,
-- 
2.7.4

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH v1] doc: add distributor library API change notice
  2017-02-06  8:08 [dpdk-dev] [PATCH v1] doc: add distributor library API change notice David Hunt
@ 2017-02-07 10:50 ` Bruce Richardson
  2017-02-07 17:12 ` Van Haaren, Harry
  2017-02-09 14:20 ` Ferruh Yigit
  2 siblings, 0 replies; 7+ messages in thread
From: Bruce Richardson @ 2017-02-07 10:50 UTC (permalink / raw)
  To: David Hunt; +Cc: dev

On Mon, Feb 06, 2017 at 08:08:29AM +0000, David Hunt wrote:
> Given that the packet distributor library improvements (1) will
> not be in 17.02, I plan on doing some consolidation of the
> API for burst operation for 17.05, merging the two api's into
> one, with options for single or burst operation.
> 

+1 for merging APIs

Acked-by: Bruce Richardson <bruce.richardson@intel.com>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH v1] doc: add distributor library API change notice
  2017-02-06  8:08 [dpdk-dev] [PATCH v1] doc: add distributor library API change notice David Hunt
  2017-02-07 10:50 ` Bruce Richardson
@ 2017-02-07 17:12 ` Van Haaren, Harry
  2017-02-09 14:20 ` Ferruh Yigit
  2 siblings, 0 replies; 7+ messages in thread
From: Van Haaren, Harry @ 2017-02-07 17:12 UTC (permalink / raw)
  To: Hunt, David, dev; +Cc: Richardson, Bruce, Hunt, David

> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of David Hunt
> Sent: Monday, February 6, 2017 8:08 AM
> To: dev@dpdk.org
> Cc: Richardson, Bruce <bruce.richardson@intel.com>; Hunt, David <david.hunt@intel.com>
> Subject: [dpdk-dev] [PATCH v1] doc: add distributor library API change notice
> 
> Given that the packet distributor library improvements (1) will
> not be in 17.02, I plan on doing some consolidation of the
> API for burst operation for 17.05, merging the two api's into
> one, with options for single or burst operation.
> 
> (1) http://dpdk.org/dev/patchwork/patch/19911/
> 
> Signed-off-by: David Hunt <david.hunt@intel.com>

Acked-by: Harry van Haaren <harry.van.haaren@intel.com>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH v1] doc: add distributor library API change notice
  2017-02-06  8:08 [dpdk-dev] [PATCH v1] doc: add distributor library API change notice David Hunt
  2017-02-07 10:50 ` Bruce Richardson
  2017-02-07 17:12 ` Van Haaren, Harry
@ 2017-02-09 14:20 ` Ferruh Yigit
  2017-02-09 17:02   ` Hunt, David
  2 siblings, 1 reply; 7+ messages in thread
From: Ferruh Yigit @ 2017-02-09 14:20 UTC (permalink / raw)
  To: David Hunt, dev; +Cc: bruce.richardson

On 2/6/2017 8:08 AM, David Hunt wrote:
> Given that the packet distributor library improvements (1) will
> not be in 17.02, I plan on doing some consolidation of the
> API for burst operation for 17.05, merging the two api's into
> one, with options for single or burst operation.
> 
> (1) http://dpdk.org/dev/patchwork/patch/19911/
> 
> Signed-off-by: David Hunt <david.hunt@intel.com>
> ---
>  doc/guides/rel_notes/deprecation.rst | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index 755dc65..925e156 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -55,6 +55,12 @@ Deprecation Notices
>    and will be removed in 17.02.
>    It is replaced by ``rte_mempool_generic_get/put`` functions.
>  
> +* lib: distributor library API will be changed to incorporate a burst-
> +  oriented API. This will include a change to ``rte_distributor_create``
> +  to specify which type of instance to create (single or burst), and
> +  additional calls for ``rte_poll_pkt_burst`` and ``rte_return_pkt_burst``,
> +  among others.

Should new APIs (rte_poll_pkt_burst & rte_return_pkt_burst) have
"rte_distributor_" name space? Apart from this:

Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH v1] doc: add distributor library API change notice
  2017-02-09 14:20 ` Ferruh Yigit
@ 2017-02-09 17:02   ` Hunt, David
  2017-02-09 17:49     ` Thomas Monjalon
  0 siblings, 1 reply; 7+ messages in thread
From: Hunt, David @ 2017-02-09 17:02 UTC (permalink / raw)
  To: Ferruh Yigit, Thomas Monjalon; +Cc: dev, bruce.richardson


On 9/2/2017 2:20 PM, Ferruh Yigit wrote:
> On 2/6/2017 8:08 AM, David Hunt wrote:
>> Given that the packet distributor library improvements (1) will
>> not be in 17.02, I plan on doing some consolidation of the
>> API for burst operation for 17.05, merging the two api's into
>> one, with options for single or burst operation.
>>
>> (1) http://dpdk.org/dev/patchwork/patch/19911/
>>
>> Signed-off-by: David Hunt <david.hunt@intel.com>
>> ---
>>   doc/guides/rel_notes/deprecation.rst | 6 ++++++
>>   1 file changed, 6 insertions(+)
>>
>> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
>> index 755dc65..925e156 100644
>> --- a/doc/guides/rel_notes/deprecation.rst
>> +++ b/doc/guides/rel_notes/deprecation.rst
>> @@ -55,6 +55,12 @@ Deprecation Notices
>>     and will be removed in 17.02.
>>     It is replaced by ``rte_mempool_generic_get/put`` functions.
>>   
>> +* lib: distributor library API will be changed to incorporate a burst-
>> +  oriented API. This will include a change to ``rte_distributor_create``
>> +  to specify which type of instance to create (single or burst), and
>> +  additional calls for ``rte_poll_pkt_burst`` and ``rte_return_pkt_burst``,
>> +  among others.
> Should new APIs (rte_poll_pkt_burst & rte_return_pkt_burst) have
> "rte_distributor_" name space? Apart from this:
>
> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
>

Ferruh,
     Thanks for the third Ack.

Thomas,
    Would you prefer me to re-spin the patch after inserting 
"_distributor" into
the two function names, or would you be so good as to do it during the 
merge?

Regards,
Dave.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH v1] doc: add distributor library API change notice
  2017-02-09 17:02   ` Hunt, David
@ 2017-02-09 17:49     ` Thomas Monjalon
  2017-02-13 16:29       ` Thomas Monjalon
  0 siblings, 1 reply; 7+ messages in thread
From: Thomas Monjalon @ 2017-02-09 17:49 UTC (permalink / raw)
  To: Hunt, David; +Cc: Ferruh Yigit, dev, bruce.richardson

2017-02-09 17:02, Hunt, David:
> On 9/2/2017 2:20 PM, Ferruh Yigit wrote:
> > On 2/6/2017 8:08 AM, David Hunt wrote:
> >> +* lib: distributor library API will be changed to incorporate a burst-
> >> +  oriented API. This will include a change to ``rte_distributor_create``
> >> +  to specify which type of instance to create (single or burst), and
> >> +  additional calls for ``rte_poll_pkt_burst`` and ``rte_return_pkt_burst``,
> >> +  among others.
> > Should new APIs (rte_poll_pkt_burst & rte_return_pkt_burst) have
> > "rte_distributor_" name space? Apart from this:
> >
> > Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
> 
> Ferruh,
>      Thanks for the third Ack.
> 
> Thomas,
>     Would you prefer me to re-spin the patch after inserting 
> "_distributor" into
> the two function names, or would you be so good as to do it during the 
> merge?

I can do it Dave :)

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH v1] doc: add distributor library API change notice
  2017-02-09 17:49     ` Thomas Monjalon
@ 2017-02-13 16:29       ` Thomas Monjalon
  0 siblings, 0 replies; 7+ messages in thread
From: Thomas Monjalon @ 2017-02-13 16:29 UTC (permalink / raw)
  To: Hunt, David; +Cc: Ferruh Yigit, dev, bruce.richardson

2017-02-09 18:49, Thomas Monjalon:
> 2017-02-09 17:02, Hunt, David:
> > On 9/2/2017 2:20 PM, Ferruh Yigit wrote:
> > > On 2/6/2017 8:08 AM, David Hunt wrote:
> > >> +* lib: distributor library API will be changed to incorporate a burst-
> > >> +  oriented API. This will include a change to ``rte_distributor_create``
> > >> +  to specify which type of instance to create (single or burst), and
> > >> +  additional calls for ``rte_poll_pkt_burst`` and ``rte_return_pkt_burst``,
> > >> +  among others.
> > > Should new APIs (rte_poll_pkt_burst & rte_return_pkt_burst) have
> > > "rte_distributor_" name space? Apart from this:
> > >
> > > Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
> > 
> > Ferruh,
> >      Thanks for the third Ack.
> > 
> > Thomas,
> >     Would you prefer me to re-spin the patch after inserting 
> > "_distributor" into
> > the two function names, or would you be so good as to do it during the 
> > merge?
> 
> I can do it Dave :)

Applied

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2017-02-13 16:29 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-06  8:08 [dpdk-dev] [PATCH v1] doc: add distributor library API change notice David Hunt
2017-02-07 10:50 ` Bruce Richardson
2017-02-07 17:12 ` Van Haaren, Harry
2017-02-09 14:20 ` Ferruh Yigit
2017-02-09 17:02   ` Hunt, David
2017-02-09 17:49     ` Thomas Monjalon
2017-02-13 16:29       ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).