Hi Ben,

Hoping you will provide me with a patch that will work... Here's the
ofproto.c.rej:

--- ofproto/ofproto.c
+++ ofproto/ofproto.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2009, 2010, 2011, 2012, 2013, 2014, 2015 Nicira, Inc.
+ * Copyright (c) 2009-2015 Nicira, Inc.
  * Copyright (c) 2010 Jean Tourrilhes - HP-Labs.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
@@ -2963,6 +2963,10 @@

             s->table_id = i;
             s->active_count = classifier_count(cls);
<<<<==== cannot find this in my branch (2.3.0)
+            if (i == 0) {
+                s->active_count -= connmgr_count_hidden_rules(
+                    ofproto->connmgr);
+            }
         }
     } else {
         stats = NULL;

s->active_count = classifier_count(cls); <<<<<===== cannot find this
anywhere in my 2.3.0 branch. Please either provide a patch with more
context (so I know where to patch this change) or provide a patch based on
2.3.0 that will work... I am blocked on this....

Thanks

On Mon, Feb 2, 2015 at 12:21 PM, Ashok Chippa <a.n.chi...@gmail.com> wrote:

> Still waiting :) I cannot find the context (around the change) I saw in
> ofproto.c.rej in 2.3.0 ofproto.c. That's the reason for asking you for the
> patch (based on 2.3.0). I do not need the entire patch, just the small hunk
> that failed (in ofproto.c.rej I sent earlier). The faster I get it, the
> faster I can test it :)
>
> Thanks
>
> On Sun, Feb 1, 2015 at 11:05 PM, Ashok Chippa <a.n.chi...@gmail.com>
> wrote:
>
>> Can one of you please apply these diffs to 2.3.0 and provide me with a
>> patch. I am not able to readily figure out where (and how) in ofproto.c
>> this needs to be applied: It will take me more time to figure this out, as
>> I am not the original author of this code...
>>
>> Appreciate your help. I will then patch it and test it. Thanks.
>>
>> Here's ofproto.c.rej:
>>
>>
>>
>> On Sun, Feb 1, 2015 at 3:07 PM, Ashok Chippa <a.n.chi...@gmail.com>
>> wrote:
>>
>>> Thanks, I figure so... Will do and let you know if it worked...
>>>
>>> On Sun, Feb 1, 2015 at 1:55 PM, Ben Pfaff <b...@nicira.com> wrote:
>>>
>>>> On Fri, Jan 30, 2015 at 05:13:42PM -0800, Ashok Chippa wrote:
>>>> > root@ashok-vb:/usr/local/bin# ovs-ofctl del-flows br-int
>>>> > root@ashok-vb:/usr/local/bin# ovs-ofctl dump-tables br-int
>>>> > OFPST_TABLE reply (xid=0x2): 254 tables
>>>> >   0: classifier: wild=0x3fffff, max=1000000, active=10
>>>> >                lookup=0, matched=0
>>>> >   1: table1  : wild=0x3fffff, max=1000000, active=0
>>>>
>>>> If you apply the patch, then it will show active=0.
>>>>
>>>
>>>
>>
>
_______________________________________________
discuss mailing list
discuss@openvswitch.org
http://openvswitch.org/mailman/listinfo/discuss

Reply via email to