From fb4a8d4472f9140fbac5b3b1ec9cea65a48b24d9 Mon Sep 17 00:00:00 2001
From: Riccardo Tedeschi <58978462+ricted98@users.noreply.github.com>
Date: Sun, 26 Jan 2025 21:51:25 +0100
Subject: [PATCH] Fix missing parametrization in performance counters (#2740)

---
 core/perf_counters.sv | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/core/perf_counters.sv b/core/perf_counters.sv
index fb423b67fc..8ef2faf1e0 100644
--- a/core/perf_counters.sv
+++ b/core/perf_counters.sv
@@ -29,7 +29,7 @@ module perf_counters
     input logic rst_ni,
     input logic debug_mode_i,  // debug mode
     // SRAM like interface
-    input logic [11:0] addr_i,  // read/write address (up to 6 counters possible)
+    input logic [11:0] addr_i,  // read/write address (up to ariane_pkg::MHPMCounterNum counters possible)
     input logic we_i,  // write enable
     input logic [CVA6Cfg.XLEN-1:0] data_i,  // data to write
     output logic [CVA6Cfg.XLEN-1:0] data_o,  // data to read
@@ -68,7 +68,7 @@ module perf_counters
   //internal signal to keep track of exception
   logic read_access_exception, update_access_exception;
 
-  logic events[6:1];
+  logic events[MHPMCounterNum:1];
   //internal signal for  MUX select line input
   logic [4:0] mhpmevent_d[MHPMCounterNum:1];
   logic [4:0] mhpmevent_q[MHPMCounterNum:1];
@@ -146,7 +146,7 @@ module perf_counters
     update_access_exception = 1'b0;
 
     // Increment the non-inhibited counters with active events
-    for (int unsigned i = 1; i <= 6; i++) begin
+    for (int unsigned i = 1; i <= MHPMCounterNum; i++) begin
       if ((!debug_mode_i) && (!we_i)) begin
         if ((events[i]) == 1 && (!mcountinhibit_i[i+2])) begin
           generic_counter_d[i] = generic_counter_q[i] + 1'b1;