[PATCH v2 3/6] arm64/perf: Filter common events based on PMCEIDn_EL0

[mark.rutland@arm.com (Mark Rutland)]

HI,

On Thu, Mar 24, 2016 at 05:52:37AM -0700, Ashok Kumar wrote:
> The complete common architectural and micro-architectural
> event number structure is filtered based on PMCEIDn_EL0 and
> copied to a new structure which is exposed to /sys
>
> The function which derives event bitmap from PMCEIDn_EL0 is
> executed in the cpus, which has the pmu being initialized,
> for heterogeneous pmu support.

I would prefer it we could instead filter the list at run time by
implementing attribute_group::is_visible() for the events attribute
group, and share a common set of attributes and attr groups. That would
avoid the filtering, copying, and assocaited memory allocation.

e.g. have an array of:

struct event_attribute {
	struct attribute attr;
	int pmceid_idx;
};

Then cache the pmceid value at probe time, and have and something like:

umode_t event_attr_is_visible(struct kobject *kobj, struct attribute *attr, int)
{
	struct arm_pmu *arm_pmu;
	struct event_attribute *e_attr;

	arm_pmu = pmu_kobj_to_armpmu(kobj);
	e_attr = container_of(attr, struct event_attr, attr);
	
	if (test_bit(e_attr->pmceid_idx, &arm_pmu->cached_pmceid))
		return 0444;

	return 0;
}

Thanks,
Mark.