[ovs-dev] [PATCH] seq: Initialize 'ovsthread_id' member of waiters in seq_wait__().

Andy Zhou azhou at nicira.com
Fri Aug 16 18:12:34 UTC 2013


Looks good. Thanks.

Acked-by: Andy Zhou <azhou at nicira.com>



On Fri, Aug 16, 2013 at 11:00 AM, Ben Pfaff <blp at nicira.com> wrote:

> Found by valgrind.
>
> Signed-off-by: Ben Pfaff <blp at nicira.com>
> ---
>  lib/seq.c |    1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/lib/seq.c b/lib/seq.c
> index 36e5065..ed205a1 100644
> --- a/lib/seq.c
> +++ b/lib/seq.c
> @@ -148,6 +148,7 @@ seq_wait__(struct seq *seq, uint64_t value)
>      waiter = xmalloc(sizeof *waiter);
>      waiter->seq = seq;
>      hmap_insert(&seq->waiters, &waiter->hmap_node, hash);
> +    waiter->ovsthread_id = id;
>      waiter->value = value;
>      waiter->thread = seq_thread_get();
>      list_push_back(&waiter->thread->waiters, &waiter->list_node);
> --
> 1.7.10.4
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20130816/a5ff7b2a/attachment-0003.html>


More information about the dev mailing list