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

Reply via email to