gustavonihei commented on code in PR #6162:
URL: https://github.com/apache/incubator-nuttx/pull/6162#discussion_r859828302


##########
arch/risc-v/src/common/riscv_pmp.c:
##########
@@ -600,9 +600,14 @@ int riscv_config_pmp_region(uintptr_t region, uintptr_t 
attr,
 #   error "XLEN of risc-v not supported"
 # endif
 
-  /* fence is needed when page-based virtual memory is implemented */
+#ifdef CONFIG_ARCH_HAVE_S_MODE

Review Comment:
   > Volume II: RISC-V Privileged Architectures V1.10 table Table 3.2: Encoding 
of Extensions field in misa: This table conveniently lists how the capabilities 
are encoded.
   
   Wow, thanks for pointing this, I wasn't aware of this table. I only 
considered the table from the **Preface** of the Volume I



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to