On 02/12/2016 10:11 AM, Ard Biesheuvel wrote:
On 12 February 2016 at 17:06, Jeremy Linton [off-list ref] wrote:
(trimming)
quoted
#if defined(CONFIG_DEBUG_ALIGN_RODATA)
-#define ALIGN_DEBUG_RO . = ALIGN(1<<SECTION_SHIFT);
-#define ALIGN_DEBUG_RO_MIN(min) ALIGN_DEBUG_RO
+#if defined(CONFIG_ARM64_64K_PAGES)
+#define ALIGN_DEBUG_RO_MIN(min) . = ALIGN(CONT_SIZE);
+#else
+#define ALIGN_DEBUG_RO_MIN(min) . = ALIGN(SECTION_SIZE);
Doesn't this align to 32 MB on 16k pages kernels?
Yes, I considered whether it was more appropriate to use CONT_SIZE for
16k as well.
Opinions?
quoted
+#endif
#elif defined(CONFIG_DEBUG_RODATA)
-#define ALIGN_DEBUG_RO . = ALIGN(1<<PAGE_SHIFT);
-#define ALIGN_DEBUG_RO_MIN(min) ALIGN_DEBUG_RO
+#define ALIGN_DEBUG_RO_MIN(min) . = ALIGN(PAGE_SIZE);
#else
-#define ALIGN_DEBUG_RO
#define ALIGN_DEBUG_RO_MIN(min) . = ALIGN(min);
#endif
--
2.4.3