You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2022/09/23 09:41:13 UTC

[GitHub] [incubator-nuttx] anchao opened a new pull request, #7173: arch/arm: declare vector array default type to read-only

anchao opened a new pull request, #7173:
URL: https://github.com/apache/incubator-nuttx/pull/7173

   
   ## Summary
   
   arch/arm: declare vector array default type to read-only
   
   Reference:
   https://developer.arm.com/documentation/dui0474/m/image-structure-and-generation/section-placement-with-the-linker/section-placement-with-the-first-and-last-attributes
   
   CAUTION:
   FIRST and LAST must not violate the basic attribute sorting order. For example, FIRST RW is placed after any read-only code or read-only data.
   
   arm-none-eabi-readelf -aS arm_vectors.o
   1. Without const: Section Headers:
     [Nr] Name              Type            Addr     Off    Size   ES Flg Lk Inf Al
     [ 3] .vectors          PROGBITS        00000000 000034 00011c 00  WA  0   0  4
   
   2. const symbol:
     [ 3] .vectors          PROGBITS        00000000 000034 00011c 00   A  0   0  4
   
   Regression by:
   
   ```
   | commit 229b57d6cb1c87868e0ca86f6035e56226979fa5
   |
   |     arch/armv[6|7|8]-m: Move _vectors to arm_internal.h to avoid the duplication
   |
   |     and change the type of _vectors from uint32_t to const void *
   |
   |     Signed-off-by: Xiang Xiao <xi...@xiaomi.com>
   ```
   
   Signed-off-by: chao an <an...@xiaomi.com>
   
   
   ## Impact
   
   N/A
   
   ## Testing
   
   CONFIG_ARM_TOOLCHAIN_ARMCLANG
   ./tools/configure.sh  lm3s6965-ek/qemu-flat


-- 
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: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7173: arch/arm: declare vector array default type to read-only

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged PR #7173:
URL: https://github.com/apache/incubator-nuttx/pull/7173


-- 
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: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org