> -----Original Message-----
> From: Pavan Nikhilesh [mailto:[email protected]]
> Sent: Tuesday, April 3, 2018 10:05 AM
> To: [email protected];
> [email protected]; Carrillo, Erik G
> <[email protected]>
> Cc: [email protected]; Pavan Nikhilesh <[email protected]>
> Subject: [dpdk-dev] [PATCH v3 05/12] event/octeontx: add event timer stats
> get and reset
> 
> Add functions to get and reset event timer adapter stats.
> 
> Signed-off-by: Pavan Nikhilesh <[email protected]>
> ---

<...snipped...>

> --- a/drivers/event/octeontx/timvf_evdev.h
> +++ b/drivers/event/octeontx/timvf_evdev.h
> @@ -145,6 +145,7 @@ struct timvf_meta {
>       struct tim_mem_bucket *bkt;
>       void *chunk_pool;
>       uint64_t tck_int;
> +     volatile uint64_t tim_arm_cnt;
 
It seems like the count could be incorrect if multiple threads arm timers 
concurrently.  Is that the case?

Thanks,
Gabriel

>  };
> 
>  struct timvf_ring {
> --
> 2.16.3

Reply via email to