On Fri, Apr 18, 2014 at 4:51 PM, Ethan Jackson <[email protected]> wrote:
> > + * - "flow_dumper" thread that reads the kernel flow table and
> dispatches
>
> Lost the initial "A" here.
Thx,
> struct upcall {
> > - struct list list_node; /* For queuing upcalls. */
> > + bool is_valid; /* If the upcall can be used. */
> > struct flow_miss *flow_miss; /* This upcall's flow_miss. */
>
> Looks like is_valid isn't necessary anymore.
>
Yeah, I also found it. Will remove it,
> I was going to suggest some other major changes, but I think it
> probably makes more sense to do it as a separate patch. Go ahead and
> merge.
>
> Acked-by: Ethan Jackson <[email protected]>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
