Thanks, I was wondering about that.

On Thu, Aug 22, 2013 at 04:21:56PM -0700, Jarno Rajahalme wrote:
> Please disregard the "1/3" in subject, there is no later parts for
> this patch "series"?
> 
>    Jarno
> 
> On Aug 22, 2013, at 4:01 PM, Jarno Rajahalme <[email protected]> wrote:
> 
> > Signed-off-by: Jarno Rajahalme <[email protected]>
> > ---
> > ofproto/ofproto-dpif-upcall.c |   48 
> > ++++++++++++++++++++---------------------
> > 1 file changed, 24 insertions(+), 24 deletions(-)
> > 
> > diff --git a/ofproto/ofproto-dpif-upcall.c b/ofproto/ofproto-dpif-upcall.c
> > index 111b779..db78af3 100644
> > --- a/ofproto/ofproto-dpif-upcall.c
> > +++ b/ofproto/ofproto-dpif-upcall.c
> > @@ -554,30 +554,30 @@ recv_upcalls(struct udpif *udpif)
> >                     }
> >                 }
> >             }
> > -           hash =  mhash_finish(hash, n_bytes);
> > -
> > -           handler = &udpif->handlers[hash % udpif->n_handlers];
> > -
> > -           ovs_mutex_lock(&handler->mutex);
> > -           if (handler->n_upcalls < MAX_QUEUE_LENGTH) {
> > -               list_push_back(&handler->upcalls, &upcall->list_node);
> > -               handler->n_upcalls++;
> > -               xpthread_cond_signal(&handler->wake_cond);
> > -               ovs_mutex_unlock(&handler->mutex);
> > -               if (!VLOG_DROP_DBG(&rl)) {
> > -                   struct ds ds = DS_EMPTY_INITIALIZER;
> > -
> > -                   odp_flow_key_format(upcall->dpif_upcall.key,
> > -                                       upcall->dpif_upcall.key_len,
> > -                                       &ds);
> > -                   VLOG_DBG("dispatcher: miss enqueue (%s)", ds_cstr(&ds));
> > -                   ds_destroy(&ds);
> > -               }
> > -           } else {
> > -               ovs_mutex_unlock(&handler->mutex);
> > -               COVERAGE_INC(miss_queue_overflow);
> > -               upcall_destroy(upcall);
> > -           }
> > +            hash =  mhash_finish(hash, n_bytes);
> > +
> > +            handler = &udpif->handlers[hash % udpif->n_handlers];
> > +
> > +            ovs_mutex_lock(&handler->mutex);
> > +            if (handler->n_upcalls < MAX_QUEUE_LENGTH) {
> > +                list_push_back(&handler->upcalls, &upcall->list_node);
> > +                handler->n_upcalls++;
> > +                xpthread_cond_signal(&handler->wake_cond);
> > +                ovs_mutex_unlock(&handler->mutex);
> > +                if (!VLOG_DROP_DBG(&rl)) {
> > +                    struct ds ds = DS_EMPTY_INITIALIZER;
> > +
> > +                    odp_flow_key_format(upcall->dpif_upcall.key,
> > +                                        upcall->dpif_upcall.key_len,
> > +                                        &ds);
> > +                    VLOG_DBG("dispatcher: miss enqueue (%s)", 
> > ds_cstr(&ds));
> > +                    ds_destroy(&ds);
> > +                }
> > +            } else {
> > +                ovs_mutex_unlock(&handler->mutex);
> > +                COVERAGE_INC(miss_queue_overflow);
> > +                upcall_destroy(upcall);
> > +            }
> >         } else {
> >             ovs_mutex_lock(&udpif->upcall_mutex);
> >             if (udpif->n_upcalls < MAX_QUEUE_LENGTH) {
> > -- 
> > 1.7.10.4
> > 
> 
> _______________________________________________
> dev mailing list
> [email protected]
> http://openvswitch.org/mailman/listinfo/dev
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to