[xen stable-4.19] x86/mm: Fix IS_ALIGNED() check in IS_LnE_ALIGNED()

[patchbot@xxxxxxx]

commit 4d91aaf3494ed7ef25c12564b3b3a4f8a933bc26
Author:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
AuthorDate: Thu Mar 20 13:21:12 2025 +0100
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Thu Mar 20 13:21:12 2025 +0100

    x86/mm: Fix IS_ALIGNED() check in IS_LnE_ALIGNED()
    
    The current CI failures turn out to be a latent bug triggered by a narrow 
set
    of properties of the initrd and the host memory map, which CI encountered by
    chance.
    
    One step during boot involves constructing directmap mappings for modules.
    With some probing at the point of creation, it is observed that there's a 4k
    mapping missing towards the end of the initrd.
    
      (XEN) === Mapped Mod1 [0000000394001000, 00000003be1ff6dc] to Directmap
      (XEN) Probing paddr 394001000, va ffff830394001000
      (XEN) Probing paddr 3be1ff6db, va ffff8303be1ff6db
      (XEN) Probing paddr 3bdffffff, va ffff8303bdffffff
      (XEN) Probing paddr 3be001000, va ffff8303be001000
      (XEN) Probing paddr 3be000000, va ffff8303be000000
      (XEN) Early fatal page fault at e008:ffff82d04032014c 
(cr2=ffff8303be000000, ec=0000)
    
    The conditions for this bug appear to be map_pages_to_xen() call with a 
start
    address of exactly 4k beyond a 2M boundary, some number of full 2M pages, 
then
    a tail needing 4k pages.
    
    Anyway, the condition for spotting superpage boundaries in 
map_pages_to_xen()
    is wrong.  The IS_ALIGNED() macro expects a power of two for the alignment
    argument, and subtracts 1 itself.
    
    Fixing this causes the failing case to now boot.
    
    Fixes: 97fb6fcf26e8 ("x86/mm: introduce helpers to detect super page 
alignment")
    Debugged-by: Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>
    Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    Tested-by: Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>
    Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
    master commit: b07c7d63f9b587e4df5d71f6da9eaa433512c974
    master date: 2025-03-19 14:53:28 +0000
---
 xen/arch/x86/mm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xen/arch/x86/mm.c b/xen/arch/x86/mm.c
index 2d9c7a5316..c3e15a029b 100644
--- a/xen/arch/x86/mm.c
+++ b/xen/arch/x86/mm.c
@@ -5236,7 +5236,7 @@ int map_pages_to_xen(
                                                                 \
     ASSERT(!mfn_eq(m_, INVALID_MFN));                           \
     IS_ALIGNED(PFN_DOWN(v) | mfn_x(m_),                         \
-               (1UL << (PAGETABLE_ORDER * ((n) - 1))) - 1);     \
+               1UL << (PAGETABLE_ORDER * ((n) - 1)));           \
 })
 #define IS_L2E_ALIGNED(v, m) IS_LnE_ALIGNED(v, m, 2)
 #define IS_L3E_ALIGNED(v, m) IS_LnE_ALIGNED(v, m, 3)
--
generated by git-patchbot for /home/xen/git/xen.git#stable-4.19