x86: Use defines for the cache flags
authorSimon Glass <sjg@chromium.org>
Sat, 15 Mar 2025 14:25:32 +0000 (14:25 +0000)
committerTom Rini <trini@konsulko.com>
Thu, 3 Apr 2025 17:41:55 +0000 (11:41 -0600)
Use some named flags when setting up the cache, so it is easier to see
what is going on.

Signed-off-by: Simon Glass <sjg@chromium.org>
arch/x86/cpu/i386/cpu.c

index 3bbad3b..845e00c 100644 (file)
@@ -592,6 +592,13 @@ int cpu_has_64bit(void)
 #define PAGETABLE_BASE         0x80000
 #define PAGETABLE_SIZE         (6 * 4096)
 
+#define _PRES BIT(0)   /* present */
+#define _RW   BIT(1)   /* write allowed */
+#define _US   BIT(2)   /* user-access allowed */
+#define _A    BIT(5)   /* has been accessed */
+#define _D    BIT(6)   /* has been written to */
+#define _PS   BIT(7)   /* indicates 2MB page size here */
+
 /**
  * build_pagetable() - build a flat 4GiB page table structure for 64-bti mode
  *
@@ -604,15 +611,17 @@ static void build_pagetable(uint32_t *pgtable)
        memset(pgtable, '\0', PAGETABLE_SIZE);
 
        /* Level 4 needs a single entry */
-       pgtable[0] = (ulong)&pgtable[1024] + 7;
+       pgtable[0] = (ulong)&pgtable[1024] + _PRES + _RW + _US + _A;
 
        /* Level 3 has one 64-bit entry for each GiB of memory */
        for (i = 0; i < 4; i++)
-               pgtable[1024 + i * 2] = (ulong)&pgtable[2048] + 0x1000 * i + 7;
+               pgtable[1024 + i * 2] = (ulong)&pgtable[2048] + 0x1000 * i +
+                       _PRES + _RW + _US + _A;
 
        /* Level 2 has 2048 64-bit entries, each repesenting 2MiB */
        for (i = 0; i < 2048; i++)
-               pgtable[2048 + i * 2] = 0x183 + (i << 21UL);
+               pgtable[2048 + i * 2] = _PRES + _RW + _US + _PS + _A +  _D +
+                                        (i << 21UL);
 }
 
 int cpu_jump_to_64bit(ulong setup_base, ulong target)