mbox series

[v4,0/2] Mesh PMU: Add tertiary match group support

Message ID 20240618005056.3092866-1-ilkka@os.amperecomputing.com (mailing list archive)
Headers show
Series Mesh PMU: Add tertiary match group support | expand

Message

Ilkka Koskinen June 18, 2024, 12:50 a.m. UTC 
v4:	* No changes, only rebased on top of v6.10-rc4

v3:
	* Replaced wp_cfg in arm_cmn_hw_event with wp_idx that keeps
	  track, whether the event uses 0 or 1 index for the given direction
	* Cleaned and simplified allocation/claiming of wp config
	* arm_cmn_val_add_event() can't and won't fail anymore
	* Separated wp_combine from wp[] in event validation phase
	* use memset()/sizeof() when clearing an event
	* Still kept wp config allocator in separate functions - at least
	  for now
	* https://lore.kernel.org/all/20240329013215.169345-1-ilkka@os.amperecomputing.com/

v2:
        * Wp config registers are allocated on node basis, instead
	  of using the same one globally
	* Use bitmap in the event structure to keep track on the
	  wp config registers assigned to the specific event.
	* The bitmap tracks only either UP (wp conf 0&1) or DOWN
	  (wp conf 2&3) registers.
	* Dropped the second patch
	  ("The patch set v2 is now in the internal mailing list")
	  as perf/sysfs doesn't really support items with the same
	  name even if visibility would handle them.
	* Addressed a bunch of other comments by the author
	* https://lore.kernel.org/all/20240307230929.6233-2-ilkka@os.amperecomputing.com/

v1:
	* https://lore.kernel.org/all/20240126221215.1537377-1-ilkka@os.amperecomputing.com/

Ilkka Koskinen (2):
  perf/arm-cmn: Decouple wp_config registers from filter group number
  perf/arm-cmn: Enable support for tertiary match group

 drivers/perf/arm-cmn.c | 116 +++++++++++++++++++++++++++++++++--------
 1 file changed, 93 insertions(+), 23 deletions(-)

Comments

Will Deacon July 1, 2024, 3:35 p.m. UTC | #1 
On Mon, 17 Jun 2024 17:50:54 -0700, Ilkka Koskinen wrote:
> v4:	* No changes, only rebased on top of v6.10-rc4
> 
> v3:
> 	* Replaced wp_cfg in arm_cmn_hw_event with wp_idx that keeps
> 	  track, whether the event uses 0 or 1 index for the given direction
> 	* Cleaned and simplified allocation/claiming of wp config
> 	* arm_cmn_val_add_event() can't and won't fail anymore
> 	* Separated wp_combine from wp[] in event validation phase
> 	* use memset()/sizeof() when clearing an event
> 	* Still kept wp config allocator in separate functions - at least
> 	  for now
> 	* https://lore.kernel.org/all/20240329013215.169345-1-ilkka@os.amperecomputing.com/
> 
> [...]

Applied to will (for-next/perf), thanks!

[1/2] perf/arm-cmn: Decouple wp_config registers from filter group number
      https://git.kernel.org/will/c/4a112585ebe8
[2/2] perf/arm-cmn: Enable support for tertiary match group
      https://git.kernel.org/will/c/f9a7a91f6406

Cheers,