Re: [ovs-dev] [PATCH] Revert "pvector: Expose non-concurrent priority vector."

2016-08-10 Thread Jarno Rajahalme
Thanks for the review, pushed to master, Jarno > On Aug 9, 2016, at 2:43 PM, Daniele Di Proietto wrote: > > Simple revert, looks good to me, thanks > > Acked-by: Daniele Di Proietto > > > > 2016-08-09 13:59 GMT-07:00 Jarno Rajahalme >:

Re: [ovs-dev] [PATCH] Revert "pvector: Expose non-concurrent priority vector."

2016-08-09 Thread Daniele Di Proietto
Simple revert, looks good to me, thanks Acked-by: Daniele Di Proietto 2016-08-09 13:59 GMT-07:00 Jarno Rajahalme : > This reverts commit 8bdfe1313894047d44349fa4cf4402970865950f. > > I failed to see that lib/dpif-netdev.c actually needs the concurrency > provided by pvector prior to this chang

[ovs-dev] [PATCH] Revert "pvector: Expose non-concurrent priority vector."

2016-08-09 Thread Jarno Rajahalme
This reverts commit 8bdfe1313894047d44349fa4cf4402970865950f. I failed to see that lib/dpif-netdev.c actually needs the concurrency provided by pvector prior to this change. More specifically, when a subtable is removed, concurrent lookups may skip over another subtable swapped in to the place of