Thx, applied, Nothing better than applying 100th patch on Friday,~
At this rate, if everyone stop applying patches, it would take me 8 yrs to match your number of commits, and 40+yrs to match Ben's record. ;D On Fri, Apr 18, 2014 at 5:10 PM, Alex Wang <al...@nicira.com> wrote: > > > > On Fri, Apr 18, 2014 at 4:51 PM, Ethan Jackson <et...@nicira.com> 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 <et...@nicira.com> > >
_______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev