Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1473520

Re: [PATCH V7 5/5] perf tools: adding sink configuration for cs_etm PMU

Show all headers | View raw

On 31 August 2016 at 03:37, Suzuki K Poulose <Suzuki.Poulose@arm.com> wrote:
> On 30/08/16 17:19, Mathieu Poirier wrote:
>>
>> Using the PMU::set_drv_config() callback to enable the CoreSight
>> sink that will be used for the trace session.
>
>
>> +int cs_etm_set_drv_config(struct perf_evsel_config_term *term)
>> +{
>> +       int ret;
>> +       char enable_sink[ENABLE_SINK_MAX];
>> +
>> +       snprintf(enable_sink, ENABLE_SINK_MAX, "%s/%s",
>> +                term->val.drv_cfg, "enable_sink");
>> +
>> +       ret = cs_device__print_file(enable_sink, "%d", 1);
>> +       if (ret < 0)
>> +               return ret;
>> +
>> +       return 0;
>> +}
>
>
>
> Don't we have to disable the sink at the end of the session ? How is that
> taken care of ? Did I miss that ?
>

Correct - the sink has to be disabled once it is no longer needed.  It
is a little tricky to do and I haven't decided on the best way to
proceed.  Fortunately that aspect doesn't affect this patchset.

Mathieu

> Suzuki

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH V7 5/5] perf tools: adding sink configuration for cs_etm PMU Mathieu Poirier <mathieu.poirier@linaro.org> - 2016-08-30 18:20 +0200
  Re: [PATCH V7 5/5] perf tools: adding sink configuration for cs_etm  PMU Suzuki K Poulose <Suzuki.Poulose@arm.com> - 2016-08-31 11:40 +0200
    Re: [PATCH V7 5/5] perf tools: adding sink configuration for cs_etm PMU Mathieu Poirier <mathieu.poirier@linaro.org> - 2016-08-31 16:20 +0200
      Re: [PATCH V7 5/5] perf tools: adding sink configuration for cs_etm  PMU Suzuki K Poulose <Suzuki.Poulose@arm.com> - 2016-09-01 11:10 +0200
        Re: [PATCH V7 5/5] perf tools: adding sink configuration for cs_etm PMU Mathieu Poirier <mathieu.poirier@linaro.org> - 2016-09-01 17:20 +0200

csiph-web