diff --git a/lld/ELF/Writer.cpp b/lld/ELF/Writer.cpp index 0f53b7f488bd5e..649958b7479863 100644 --- a/lld/ELF/Writer.cpp +++ b/lld/ELF/Writer.cpp @@ -2638,10 +2638,11 @@ template void Writer::setPhdrs(Partition &part) { if (p->p_type == PT_GNU_RELRO) { p->p_align = 1; - // musl/glibc/FreeBSD ld.so rounds the size down, so we need to round up - // to protect the last page. - p->p_memsz = - alignTo(p->p_offset + p->p_memsz, config->maxPageSize) - p->p_offset; + // musl/glibc ld.so rounds the size down, so we need to round up + // to protect the last page. This is a no-op on FreeBSD which always + // rounds up. + p->p_memsz = alignTo(p->p_offset + p->p_memsz, config->commonPageSize) - + p->p_offset; } } } diff --git a/lld/docs/ReleaseNotes.rst b/lld/docs/ReleaseNotes.rst index 645721fbb80389..b3751e53db55c2 100644 --- a/lld/docs/ReleaseNotes.rst +++ b/lld/docs/ReleaseNotes.rst @@ -29,10 +29,6 @@ ELF Improvements * ``-z pack-relative-relocs`` is now available to support ``DT_RELR`` for glibc 2.36+. (`D120701 `_) * ``--no-fortran-common`` (pre 12.0.0 behavior) is now the default. -* The end of ``PT_GNU_RELRO`` is now aligned by ``max-page-size`` instead of ``common-page-size``. - This matches GNU ld from 2.39 onwards. If the system page size is larger than ``common-page-size``, - the previous choice may make a partial page not protected by RELRO. - (`D125410 `_) Breaking changes ---------------- diff --git a/lld/test/ELF/aarch64-relro.s b/lld/test/ELF/aarch64-relro.s index 4223156df996ee..4047382c8aa1bd 100644 --- a/lld/test/ELF/aarch64-relro.s +++ b/lld/test/ELF/aarch64-relro.s @@ -8,7 +8,7 @@ # CHECK-NEXT: VirtualAddress: 0x220190 # CHECK-NEXT: PhysicalAddress: # CHECK-NEXT: FileSize: -# CHECK-NEXT: MemSize: 65136 +# CHECK-NEXT: MemSize: 3696 .section .data.rel.ro,"aw",%progbits .byte 1 diff --git a/lld/test/ELF/basic-ppc64.s b/lld/test/ELF/basic-ppc64.s index b3740230c41ed4..4032ceca614fb5 100644 --- a/lld/test/ELF/basic-ppc64.s +++ b/lld/test/ELF/basic-ppc64.s @@ -336,7 +336,7 @@ // CHECK-NEXT: VirtualAddress: 0x20238 // CHECK-NEXT: PhysicalAddress: 0x20238 // CHECK-NEXT: FileSize: 96 -// CHECK-NEXT: MemSize: 64968 +// CHECK-NEXT: MemSize: 3528 // CHECK-NEXT: Flags [ (0x4) // CHECK-NEXT: PF_R (0x4) // CHECK-NEXT: ]