Re: [ovs-dev] [PATCH] dpif-netdev: Fix improper use of CMAP_FOR_EACH.

2016-02-02 Thread Daniele Di Proietto
On 02/02/2016 10:39, "Jarno Rajahalme" wrote: >With the small nits below: > >Acked-by: Jarno Rajahalme Thanks Jarno and Ilya for the reviews! I applied this on master and branch-2.5 > >> On Jan 26, 2016, at 9:13 PM, Daniele Di Proietto >> wrote: >> >> It is ok to iterate a cmap with CMAP_F

Re: [ovs-dev] [PATCH] dpif-netdev: Fix improper use of CMAP_FOR_EACH.

2016-02-02 Thread Jarno Rajahalme
With the small nits below: Acked-by: Jarno Rajahalme > On Jan 26, 2016, at 9:13 PM, Daniele Di Proietto > wrote: > > It is ok to iterate a cmap with CMAP_FOR_EACH and remove elements with > cmap_remove(), but having quiescent states inside the loop might create > problems, since some of the p