Re: [dpdk-dev] [PATCH 1/3] mbuf: embedding timestamp into the packet

2017-01-24 Thread Olivier MATZ
On Thu, 13 Oct 2016 14:35:06 +, Oleg Kuporosov wrote: > The hard requirement of financial services industry is accurate > timestamping aligned with the packet itself. This patch is to satisfy > this requirement: > > - include uint64_t timestamp field into rte_mbuf with minimal impact > to thr

[dpdk-dev] [PATCH 1/3] mbuf: embedding timestamp into the packet

2016-10-25 Thread Pattan, Reshma
Hi, > -Original Message- > From: Oleg Kuporosov [mailto:olegk at mellanox.com] > Sent: Wednesday, October 19, 2016 6:40 PM > To: Pattan, Reshma ; Olivier Matz > > Cc: Richardson, Bruce ; Ananyev, Konstantin > ; dev at dpdk.org > Subject: RE: [dpdk-dev] [PA

[dpdk-dev] [PATCH 1/3] mbuf: embedding timestamp into the packet

2016-10-20 Thread Oleg Kuporosov
Hello Konstantin, > > My vote also would be to have timestamp in the second cache line. > About moving seqn to the 2-nd cache line too - that's probably a fair point. It may impact throughput till ~6% for applications required such embedded Timestamps. > About the rest of the patch: > Do you r

[dpdk-dev] [PATCH 1/3] mbuf: embedding timestamp into the packet

2016-10-20 Thread Jan Blunck
On Thu, Oct 20, 2016 at 4:03 AM, Oleg Kuporosov wrote: > Hello Konstantin, > >> >> My vote also would be to have timestamp in the second cache line. >> About moving seqn to the 2-nd cache line too - that's probably a fair point. > > It may impact throughput till ~6% for applications required such

[dpdk-dev] [PATCH 1/3] mbuf: embedding timestamp into the packet

2016-10-19 Thread Oleg Kuporosov
Hello Reshma > I just read this mail chain, to make every one aware again, I am emphasizing > on the point that I am also adding new "time_arraived" field to mbuf struct as > part of below 17.02 Road map item. Thanks for your work for extending statistics support in DPDK. "time_arrived" points

[dpdk-dev] [PATCH 1/3] mbuf: embedding timestamp into the packet

2016-10-19 Thread Oleg Kuporosov
Hello Oliver Great thanks for your review and con > > - include uint64_t timestamp field into rte_mbuf with minimal impact to > > throughput/latency. Keep it just simple uint64_t in ns (more than 580 > > years) would be enough for immediate needs while using full > > struct timespec with tw

[dpdk-dev] [PATCH 1/3] mbuf: embedding timestamp into the packet

2016-10-19 Thread Ananyev, Konstantin
Hi lads, > > > > On 10/13/2016 04:35 PM, Oleg Kuporosov wrote: > > The hard requirement of financial services industry is accurate > > timestamping aligned with the packet itself. This patch is to satisfy > > this requirement: > > > > - include uint64_t timestamp field into rte_mbuf with minima

[dpdk-dev] [PATCH 1/3] mbuf: embedding timestamp into the packet

2016-10-19 Thread Pattan, Reshma
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier Matz > Sent: Tuesday, October 18, 2016 4:44 PM > To: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 1/3] mbuf: embedding timestamp into the > packet > > > > On 10/13

[dpdk-dev] [PATCH 1/3] mbuf: embedding timestamp into the packet

2016-10-18 Thread Olivier Matz
On 10/13/2016 04:35 PM, Oleg Kuporosov wrote: > The hard requirement of financial services industry is accurate > timestamping aligned with the packet itself. This patch is to satisfy > this requirement: > > - include uint64_t timestamp field into rte_mbuf with minimal impact to > throughput/l

[dpdk-dev] [PATCH 1/3] mbuf: embedding timestamp into the packet

2016-10-13 Thread Oleg Kuporosov
The hard requirement of financial services industry is accurate timestamping aligned with the packet itself. This patch is to satisfy this requirement: - include uint64_t timestamp field into rte_mbuf with minimal impact to throughput/latency. Keep it just simple uint64_t in ns (more than 580