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

Groups > linux.kernel > #1479854

Re: [RFCv4 1/7] drivers/perf: arm_pmu: add common attr group fields

Show all headers | View raw

On Thu, Sep 08, 2016 at 11:21:46AM +0100, Mark Rutland wrote:
> In preparation for adding common attribute groups, add an array of
> attribute group pointers to arm_pmu, which will be used if the
> backend hasn't already set pmu::attr_groups.
> 
> Subsequent patches will move backends over to using these, before adding
> common fields.
> 
> Signed-off-by: Mark Rutland <mark.rutland@arm.com>
> Cc: Will Deacon <will.deacon@arm.com>
> ---
>  drivers/perf/arm_pmu.c       | 3 +++
>  include/linux/perf/arm_pmu.h | 9 ++++++++-
>  2 files changed, 11 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/perf/arm_pmu.c b/drivers/perf/arm_pmu.c
> index f5e1008..145caf4 100644
> --- a/drivers/perf/arm_pmu.c
> +++ b/drivers/perf/arm_pmu.c
> @@ -1039,6 +1039,9 @@ int arm_pmu_device_probe(struct platform_device *pdev,
>  		goto out_free;
>  	}
>  
> +	if (!pmu->pmu.attr_groups)
> +		pmu->pmu.attr_groups = pmu->attr_groups;
> +
>  	ret = cpu_pmu_init(pmu);
>  	if (ret)
>  		goto out_free;
> diff --git a/include/linux/perf/arm_pmu.h b/include/linux/perf/arm_pmu.h
> index e188438..8030814 100644
> --- a/include/linux/perf/arm_pmu.h
> +++ b/include/linux/perf/arm_pmu.h
> @@ -14,7 +14,7 @@
>  
>  #include <linux/interrupt.h>
>  #include <linux/perf_event.h>
> -
> +#include <linux/sysfs.h>
>  #include <asm/cputype.h>
>  
>  /*
> @@ -77,6 +77,12 @@ struct pmu_hw_events {
>  	struct arm_pmu		*percpu_pmu;
>  };
>  
> +enum armpmu_attr_groups {
> +	ARMPMU_ATTR_GROUP_EVENTS,
> +	ARMPMU_ATTR_GROUP_FORMATS,
> +	ARMPMU_NR_ATTR_GROUPS
> +};
> +
>  struct arm_pmu {
>  	struct pmu	pmu;
>  	cpumask_t	active_irqs;
> @@ -111,6 +117,7 @@ struct arm_pmu {
>  	struct pmu_hw_events	__percpu *hw_events;
>  	struct list_head	entry;
>  	struct notifier_block	cpu_pm_nb;
> +	const struct attribute_group *attr_groups[ARMPMU_NR_ATTR_GROUPS + 1];

Is the '+ 1' because the array has to be NULL terminated? Probably worth
a comment to disuade people from sending "obvious" fixes.

Will

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

Thread

[RFCv4 0/7]  arm_pmu/perf tools: play nicely with CPU PMU cpumasks Mark Rutland <mark.rutland@arm.com> - 2016-09-08 12:30 +0200
  [RFCv4 7/7] perf: util: support alternative sysfs cpumask Mark Rutland <mark.rutland@arm.com> - 2016-09-08 12:30 +0200
    [tip:perf/core] perf pmu: Support alternative sysfs cpumask tip-bot for Mark Rutland <tipbot@zytor.com> - 2016-09-09 08:00 +0200
  [RFCv4 5/7] drivers/perf: arm_pmu: expose a cpumask in sysfs Mark Rutland <mark.rutland@arm.com> - 2016-09-08 12:30 +0200
    Re: [RFCv4 5/7] drivers/perf: arm_pmu: expose a cpumask in sysfs Will Deacon <will.deacon@arm.com> - 2016-09-09 12:30 +0200
      Re: [RFCv4 5/7] drivers/perf: arm_pmu: expose a cpumask in sysfs Mark Rutland <mark.rutland@arm.com> - 2016-09-09 13:10 +0200
  [RFCv4 2/7] arm64: perf: move to common attr_group fields Mark Rutland <mark.rutland@arm.com> - 2016-09-08 12:30 +0200
  [RFCv4 4/7] drivers/perf: arm_pmu: only use common attr_groups Mark Rutland <mark.rutland@arm.com> - 2016-09-08 12:30 +0200
  [RFCv4 3/7] arm: perf: move to common attr_group fields Mark Rutland <mark.rutland@arm.com> - 2016-09-08 12:30 +0200
  [RFCv4 6/7] perf: util: only open events on CPUs an evsel permits Mark Rutland <mark.rutland@arm.com> - 2016-09-08 12:30 +0200
    [tip:perf/core] perf evlist: Only open events on CPUs an evsel  permits tip-bot for Mark Rutland <tipbot@zytor.com> - 2016-09-09 08:00 +0200
  [RFCv4 1/7] drivers/perf: arm_pmu: add common attr group fields Mark Rutland <mark.rutland@arm.com> - 2016-09-08 12:30 +0200
    Re: [RFCv4 1/7] drivers/perf: arm_pmu: add common attr group fields Will Deacon <will.deacon@arm.com> - 2016-09-09 12:30 +0200
      Re: [RFCv4 1/7] drivers/perf: arm_pmu: add common attr group fields Mark Rutland <mark.rutland@arm.com> - 2016-09-09 13:10 +0200
  Re: [RFCv4 0/7]  arm_pmu/perf tools: play nicely with CPU PMU  cpumasks Will Deacon <will.deacon@arm.com> - 2016-09-08 12:40 +0200
    Re: [RFCv4 0/7]  arm_pmu/perf tools: play nicely with CPU PMU  cpumasks Mark Rutland <mark.rutland@arm.com> - 2016-09-08 13:30 +0200
  Re: [RFCv4 0/7]  arm_pmu/perf tools: play nicely with CPU PMU  cpumasks Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-09-08 18:30 +0200
    Re: [RFCv4 0/7]  arm_pmu/perf tools: play nicely with CPU PMU  cpumasks Peter Zijlstra <peterz@infradead.org> - 2016-09-08 20:20 +0200
      Re: [RFCv4 0/7]  arm_pmu/perf tools: play nicely with CPU PMU  cpumasks Will Deacon <will.deacon@arm.com> - 2016-09-09 11:40 +0200

csiph-web