On Sun, Sep 10, 2017 at 07:23:18PM -0700, kan.li...@intel.com wrote:

SNIP

> +     pthread_mutex_unlock(&thread->namespaces_lock);
> +
>       return 0;
>  }
>  
> -void thread__namespaces_id(const struct thread *thread,
> +void thread__namespaces_id(struct thread *thread,
>                          u64 *dev, u64 *ino)
>  {
>       struct namespaces *ns;
>  
> +     pthread_mutex_lock(&thread->namespaces_lock);
>       ns = thread__namespaces(thread);

isn't it just thread__namespaces that needs this lock?

if that's the case we don't need the change for __hists__add_entry
in previous patch

jirka

Reply via email to