Acked-by: Alex Wang <[email protected]>

On Tue, Apr 29, 2014 at 2:44 PM, Ben Pfaff <[email protected]> wrote:

> The main thread has the empty string as its name, but that's not a good
> log string.
>
> Without this patch we can get log message like
>     blocked 1000 ms waiting for  to quiesce
> from ovsrcu_synchronize().
>
> Signed-off-by: Ben Pfaff <[email protected]>
> ---
>  lib/ovs-rcu.c |    4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/lib/ovs-rcu.c b/lib/ovs-rcu.c
> index 8a12564..7aed6db 100644
> --- a/lib/ovs-rcu.c
> +++ b/lib/ovs-rcu.c
> @@ -71,11 +71,13 @@ ovsrcu_perthread_get(void)
>
>      perthread = pthread_getspecific(perthread_key);
>      if (!perthread) {
> +        const char *name = get_subprogram_name();
> +
>          perthread = xmalloc(sizeof *perthread);
>          ovs_mutex_init(&perthread->mutex);
>          perthread->seqno = seq_read(global_seqno);
>          perthread->cbset = NULL;
> -        ovs_strlcpy(perthread->name, get_subprogram_name(),
> +        ovs_strlcpy(perthread->name, name[0] ? name : "main",
>                      sizeof perthread->name);
>
>          ovs_mutex_lock(&ovsrcu_threads_mutex);
> --
> 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