* Stephane Eranian <eran...@google.com> wrote:

> -     if (is_cgroup_event(event))
> +     if (is_cgroup_event(event)) {
>               ctx->nr_cgroups--;
> +             cpuctx = __get_cpu_context(ctx);
> +             /*
> +              * if there are no more cgroup events
> +              * then clear cgrp to avoid stale pointer
> +              * in update_cgrp_time_from_cpuctx()
> +              */
> +             if (!ctx->nr_cgroups)
> +                     cpuctx->cgrp = NULL;
> +     }

The ->cgrp pointer does not exist on !CGROUPS kernels. I suspect the cleanest 
approach would be to make those two cgrp fields available unconditionally in 
struct perf_event.

Thanks,

        Ingo

------------------------------------------------------------------------------
Enable your software for Intel(R) Active Management Technology to meet the
growing manageability and security demands of your customers. Businesses
are taking advantage of Intel(R) vPro (TM) technology - will your software 
be a part of the solution? Download the Intel(R) Manageability Checker 
today! http://p.sf.net/sfu/intel-dev2devmar
_______________________________________________
perfmon2-devel mailing list
perfmon2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/perfmon2-devel

Reply via email to