Thanks a lot for the info.

For you review, I still have one issue (buried in my emails):

"""

> +/* Index counter used to compute the moving average array's index. */
> > +static unsigned int idx_count = 0;
>
> It looks to me like idx_count should be protected by coverage_mutex.



I'm a bit confused here. Want to ask, that there will only be one thread
going through the loop in ovs-vswitchd.c (which calls coverage_run()),
right? If so, idx_count will only be accessed by that thread. Then do we
still need to protect it?

"""
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to