arch: riscv: Use CONFIG_PMP_SLOTS for M-mode stack guard arrays

The arrays `m_mode_pmpaddr_regs` and `m_mode_pmpcfg_regs` within the
`_thread_arch` struct, used for the M-mode stack guard (when
CONFIG_PMP_STACK_GUARD is enabled), were previously sized using the
hardcoded PMP_M_MODE_SLOTS macro (defined as 8).

This commit changes the array sizing to use the Kconfig option
CONFIG_PMP_SLOTS. This option reflects the total number of Physical
Memory Protection (PMP) slots available and configured for the target
hardware.

Using CONFIG_PMP_SLOTS ensures these arrays are dimensioned according
 to the system's actual capabilities, providing better flexibility and
correctness over a fixed size.

Signed-off-by: Firas Sammoura <[email protected]>

Author: fsammoura1980

arch/riscv/core/pmp.c

@@ -539,8 +539,8 @@ void z_riscv_pmp_stackguard_enable(struct k_thread *thread)
 void z_riscv_pmp_stackguard_disable(void)
 {
 
-	unsigned long pmp_addr[PMP_M_MODE_SLOTS];
-	unsigned long pmp_cfg[PMP_M_MODE_SLOTS / sizeof(unsigned long)];
+	unsigned long pmp_addr[CONFIG_PMP_SLOTS];
+	unsigned long pmp_cfg[CONFIG_PMP_SLOTS / sizeof(unsigned long)];
 	unsigned int index = global_pmp_end_index;
 
 	/* Retrieve the pmpaddr value matching the last global PMP slot. */

include/zephyr/arch/riscv/thread.h

@@ -63,8 +63,6 @@ struct z_riscv_fp_context {
 };
 typedef struct z_riscv_fp_context z_riscv_fp_context_t;
 
-#define PMP_M_MODE_SLOTS 8	/* 8 is plenty enough for m-mode */
-
 struct _thread_arch {
 #ifdef CONFIG_FPU_SHARING
 	struct z_riscv_fp_context saved_fp_context;
@@ -81,8 +79,8 @@ struct _thread_arch {
 #endif
 #ifdef CONFIG_PMP_STACK_GUARD
 	unsigned int m_mode_pmp_end_index;
-	unsigned long m_mode_pmpaddr_regs[PMP_M_MODE_SLOTS];
-	unsigned long m_mode_pmpcfg_regs[PMP_M_MODE_SLOTS / sizeof(unsigned long)];
+	unsigned long m_mode_pmpaddr_regs[CONFIG_PMP_SLOTS];
+	unsigned long m_mode_pmpcfg_regs[CONFIG_PMP_SLOTS / sizeof(unsigned long)];
 #endif
 };