[dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api

2016-03-09 Thread Kulasek, TomaszX
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Wednesday, March 9, 2016 18:07 > To: Kulasek, TomaszX > Cc: dev at dpdk.org; Ananyev, Konstantin > Subject: Re: [dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api >

[dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api

2016-03-09 Thread Thomas Monjalon
2016-03-09 16:35, Kulasek, TomaszX: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > > +void > > > +rte_eth_count_unsent_packet_callback(struct rte_mbuf **pkts, uint16_t > > unsent, > > > + void *userdata); > > > > What about rte_eth_tx_buffer_default_callback as name? >

[dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api

2016-03-09 Thread Thomas Monjalon
2016-03-09 16:17, Ananyev, Konstantin: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > 2016-03-09 15:42, Ananyev, Konstantin: > > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > > > 2016-03-09 15:23, Ananyev, Konstantin: > > > > > > > > > > > > 2016-03-09 13:36,

[dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api

2016-03-09 Thread Thomas Monjalon
2016-03-09 15:42, Ananyev, Konstantin: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > 2016-03-09 15:23, Ananyev, Konstantin: > > > > > > > > 2016-03-09 13:36, Ananyev, Konstantin: > > > > > > > + if (to_send == 0) > > > > > > > + return 0; > > > > > > > > > > > > Why

[dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api

2016-03-09 Thread Thomas Monjalon
2016-03-09 15:32, Kulasek, TomaszX: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > 2016-03-09 15:23, Ananyev, Konstantin: > > > > > > > > 2016-03-09 13:36, Ananyev, Konstantin: > > > > > > > + if (to_send == 0) > > > > > > > + return 0; > > > > > > > > > > > > Why

[dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api

2016-03-09 Thread Kulasek, TomaszX
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Tuesday, March 8, 2016 23:52 > To: Kulasek, TomaszX > Cc: dev at dpdk.org; Ananyev, Konstantin > Subject: Re: [dpdk-dev] [PATCH v2 1/2] ethdev: add buffe

[dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api

2016-03-09 Thread Thomas Monjalon
2016-03-09 15:23, Ananyev, Konstantin: > > > > 2016-03-09 13:36, Ananyev, Konstantin: > > > > > + if (to_send == 0) > > > > > + return 0; > > > > > > > > Why this check is done in the lib? > > > > What is the performance gain if we are idle? > > > > It can be done outside if needed. >

[dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api

2016-03-09 Thread Ananyev, Konstantin
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Wednesday, March 09, 2016 3:52 PM > To: Ananyev, Konstantin > Cc: Kulasek, TomaszX; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api >

[dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api

2016-03-09 Thread Ananyev, Konstantin
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Wednesday, March 09, 2016 3:27 PM > To: Ananyev, Konstantin > Cc: Kulasek, TomaszX; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api >

[dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api

2016-03-09 Thread Kulasek, TomaszX
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Wednesday, March 9, 2016 16:27 > To: Ananyev, Konstantin > Cc: Kulasek, TomaszX ; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api >

[dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api

2016-03-09 Thread Thomas Monjalon
2016-03-09 13:36, Ananyev, Konstantin: > > > + if (to_send == 0) > > > + return 0; > > > > Why this check is done in the lib? > > What is the performance gain if we are idle? > > It can be done outside if needed. > > Yes, that could be done outside, but if user has to do it anyway, >

[dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api

2016-03-09 Thread Ananyev, Konstantin
> > 2016-03-09 13:36, Ananyev, Konstantin: > > > > + if (to_send == 0) > > > > + return 0; > > > > > > Why this check is done in the lib? > > > What is the performance gain if we are idle? > > > It can be done outside if needed. > > > > Yes, that could be done outside, but if user has

[dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api

2016-03-09 Thread Ananyev, Konstantin
Hi Thomas, > > Hi, > > It is an overlay on the tx burst API. > Probably it doesn't hurt to add it but we have to be really cautious > with the API definition to try keeping it stable in the future. > > 2016-02-24 18:08, Tomasz Kulasek: > > +/** > > + * Structure used to buffer packets for

[dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api

2016-03-08 Thread Thomas Monjalon
Hi, It is an overlay on the tx burst API. Probably it doesn't hurt to add it but we have to be really cautious with the API definition to try keeping it stable in the future. 2016-02-24 18:08, Tomasz Kulasek: > +/** > + * Structure used to buffer packets for future TX > + * Used by APIs

[dpdk-dev] [PATCH v2 1/2] ethdev: add buffered tx api

2016-02-24 Thread Tomasz Kulasek
Many sample apps include internal buffering for single-packet-at-a-time operation. Since this is such a common paradigm, this functionality is better suited to being implemented in the ethdev API. The new APIs in the ethdev library are: * rte_eth_tx_buffer_init - initialize buffer *