[ovs-dev] [RFC 03/14] revalidator: Use 'cmap' for storing ukeys.

[Ben Pfaff]

On Thu, Aug 21, 2014 at 05:41:58PM +1200, Joe Stringer wrote:
> Signed-off-by: Joe Stringer <joestringer at nicira.com>

As of this patch, there isn't an advantage to using a cmap yet, right?

Here, I'd be inclined to add the ovsrcu_quiesce() call outside the
'if' statement, unconditionally, because otherwise it's just a matter
of luck whether the number of ops happened to be a multiple of
REVALIDATE_MAX_BATCH:
> @@ -1582,6 +1587,7 @@ revalidate(struct revalidator *revalidator)
>  
>          if (n_ops) {
>              push_dump_ops__(udpif, ops, n_ops);
> +            ovsrcu_quiesce();
>          }
>      }
>      dpif_flow_dump_thread_destroy(dump_thread);

Acked-by: Ben Pfaff <blp at nicira.com>