Ouss4 commented on a change in pull request #2014:
URL: https://github.com/apache/incubator-nuttx/pull/2014#discussion_r509330177
##########
File path: arch/xtensa/src/common/xtensa_int_handlers.S
##########
@@ -68,6 +68,19 @@
#include "chip_macros.h"
#include "xtensa_timer.h"
+#if !defined(CONFIG_SMP) && CONFIG_ARCH_INTERRUPTSTACK > 15
+ .data
+ .align 16
Review comment:
Yes, it means 16-byte alignement. (ref.
https://sourceware.org/binutils/docs-2.25/as/Align.html)
Here is the relevant text:
> The way the required alignment is specified varies from system to system.
For ....and xtensa, the first expression is the alignment request in bytes.
There is also a clearer example
[here](https://github.com/jcmvbkbc/linux-xtensa/blob/f75aef392f869018f78cfedf3c320a6b3fcfda6b/arch/xtensa/kernel/vectors.S#L663).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
