> -----Original Message-----
> From: dev <dev-boun...@dpdk.org> On Behalf Of Ananyev, Konstantin
> Sent: Tuesday, October 22, 2019 5:00 PM
> To: 'Honnappa Nagarahalli' <honnappa.nagaraha...@arm.com>; 'Jerin Jacob' 
> <jerinjac...@gmail.com>
> Cc: 'David Christensen' <d...@linux.vnet.ibm.com>; 'olivier.m...@6wind.com' 
> <olivier.m...@6wind.com>; 'sthem...@microsoft.com'
> <sthem...@microsoft.com>; 'jer...@marvell.com' <jer...@marvell.com>; 
> Richardson, Bruce <bruce.richard...@intel.com>;
> 'david.march...@redhat.com' <david.march...@redhat.com>; 
> 'pbhagavat...@marvell.com' <pbhagavat...@marvell.com>;
> 'dev@dpdk.org' <dev@dpdk.org>; 'Dharmik Thakkar' <dharmik.thak...@arm.com>; 
> 'Ruifeng Wang (Arm Technology China)'
> <ruifeng.w...@arm.com>; 'Gavin Hu (Arm Technology China)' <gavin...@arm.com>; 
> 'step...@networkplumber.org'
> <step...@networkplumber.org>; 'nd' <n...@arm.com>; 'nd' <n...@arm.com>
> Subject: Re: [dpdk-dev] [PATCH v4 1/2] lib/ring: apis to support configurable 
> element size
> 
> 
> 
> > > I have applied your
> > > suggestion in 6/6 in v6 along with my corrections. The rte_ring_elem test 
> > > cases are added in 3/6. I have verified that they are running
> > > fine (they are done for 64b alone, will add more). Hopefully, there are 
> > > no more errors.
> 
> Applied v6 and re-run the tests.
> Functional test passes ok on my boxes.
> Pert-tests numbers below.
> As I can see pretty much same pattern as in v5 remains:
> MP/MC on 2 different cores

Forgot to add: for 8 elems, for 32 - new ones always better. 

> and SP/SC single enq/deq
> show lower numbers for _elem_.
> For others _elem_ numbers are about the same or higher.
> Personally, I am ok to go ahead with these changes.
> Konstantin
> 
> A - ring_perf_autotes
> B - ring_perf_elem_autotest
> 
>  ### Testing single element and burst enq/deq ###     A       B
> SP/SC single enq/dequeue:                             8.27    10.94
> MP/MC single enq/dequeue:                             56.11   47.43
> SP/SC burst enq/dequeue (size: 8):                    4.20    3.50
> MP/MC burst enq/dequeue (size: 8):                    9.93    9.29
> SP/SC burst enq/dequeue (size: 32):                   2.93    1.94
> MP/MC burst enq/dequeue (size: 32):                   4.10    3.35
> 
> ### Testing empty dequeue ###
> SC empty dequeue:                                     2.00    3.00
> MC empty dequeue:                                     3.00    2.00
> 
> ### Testing using a single lcore ###
> SP/SC bulk enq/dequeue (size: 8):                     4.06    3.30
> MP/MC bulk enq/dequeue (size: 8):                     9.84    9.28
> SP/SC bulk enq/dequeue (size: 32):                    2.93    1.88
> MP/MC bulk enq/dequeue (size: 32):                    4.10    3.32
> 
> ### Testing using two hyperthreads ###
> SP/SC bulk enq/dequeue (size: 8):                     9.22    8.83
> MP/MC bulk enq/dequeue (size: 8):                     15.73   15.86
> SP/SC bulk enq/dequeue (size: 32):                    5.78    3.83
> MP/MC bulk enq/dequeue (size: 32):                    6.33    4.53
> 
> ### Testing using two physical cores ###
> SP/SC bulk enq/dequeue (size: 8):                     23.78   19.32
> MP/MC bulk enq/dequeue (size: 8):                     68.54   71.97
> SP/SC bulk enq/dequeue (size: 32):                    11.99   10.77
> MP/MC bulk enq/dequeue (size: 32):                    21.96   18.66
> 
> ### Testing using two NUMA nodes ###
> SP/SC bulk enq/dequeue (size: 8):                     50.13   33.92
> MP/MC bulk enq/dequeue (size: 8):                     177.98  195.87
> SP/SC bulk enq/dequeue (size: 32):                    32.98   23.12
> MP/MC bulk enq/dequeue (size: 32):                    55.86   48.76

Reply via email to