[M] Change in coreboot[main]: soc/intel/xeon_sp: Reserve MMIO high range

[Shuo Liu (Code Review)]

Attention is currently required from: Arthur Heymans, Christian Walter, Johnny Lin, Jonathan Zhang, Lean Sheng Tan, Patrick Rudolph, Paul Menzel, Tim Chu.

Shuo Liu has posted comments on this change by Shuo Liu. ( https://review.coreboot.org/c/coreboot/+/83538?usp=email )

Change subject: soc/intel/xeon_sp: Reserve MMIO high range
......................................................................

Patch Set 5:

(3 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/83538/comment/26a0fd93_abd4ff11?usp... :
PS4, Line 13: belonged
> belong*ing*?

Done

https://review.coreboot.org/c/coreboot/+/83538/comment/bbb816ea_9b947c88?usp... :
PS4, Line 16:
> Paste an example of cluttered coverage?

Done

https://review.coreboot.org/c/coreboot/+/83538/comment/5231102d_d7e80f3a?usp... :
PS4, Line 31: tsc: Detected 2000.000 MHz processor
            : last_pfn = 0x2080000 max_arch_pfn = 0x10000000000
            : x86/PAT: Configuration [0-7]: WB  WC  UC- UC  WB  WP  UC- WT
            : WARNING: BIOS bug: CPU MTRRs don't cover all of memory, losing 129024MB of RAM.
            : ------------[ cut here ]------------
            : WARNING: CPU: 0 PID: 0 at arch/x86/kernel/cpu/mtrr/cleanup.c:978 mtrr_trim_uncached_memory+0x2b9/0x2f9
            : ...
            : Call Trace:
            :  ? 0xffffffff8f600000
            :  ? setup_arch+0x4bb/0xaed
            :  ? printk+0x53/0x6a
            :  ? start_kernel+0x55/0x507
            :  ? load_ucode_intel_bsp+0x1c/0x4d
            :  ? secondary_startup_64_no_verify+0xc2/0xcb
            : random: get_random_bytes called from init_oops_id+0x1d/0x2c with crng_init=0
            : ---[ end trace 0e56686fd458f0c5 ]---
            : update e820 for mtrr
            : modified physical RAM map:
            : modified: [mem 0x0000000000000000-0x0000000000000fff] reserved
            : ...
            : modified: [mem 0x00000000ff000000-0x000000207fffffff] reserved
            : last_pfn = 0x6354e max_arch_pfn = 0x10000000000
            : Memory KASLR using RDRAND RDTSC...
            : x2apic: enabled by BIOS, switching to x2apic ops
            : Using GB pages for direct mapping
            : ...
            : Initmem setup node 0 [mem 0x0000000000001000-0x000000006354dfff]
            :   DMA zone: 28769 pages in unavailable ranges
            :   DMA32 zone: 19122 pages in unavailable ranges
            : BUG: unable to handle page fault for address: ff24b56eba60cff8
            : BAD
            : Oops: 0000 [#1] SMP NOPTI
            : CPU: 0 PID: 0 Comm: swapper Tainted: G        W         5.10.50 #2
            : ...
            : Call Trace:
            :  ? set_pte_vaddr_p4d+0x24/0x35
            :  ? __native_set_fixmap+0x21/0x28
            :  ? map_vsyscall+0x35/0x56
            :  ? setup_arch+0xa00/0xaed
            :  ? printk+0x53/0x6a
            :  ? start_kernel+0x55/0x507
            :  ? load_ucode_intel_bsp+0x1c/0x4d
            :  ? secondary_startup_64_no_verify+0xc2/0xcb
            : CR2: ff24b56eba60cff8
            : ---[ end trace 0e56686fd458f0c6 ]---
            : RIP: 0010:fill_pud+0xa/0x62
            : ...
            : Kernel panic - not syncing: Attempted to kill the idle task!
            : ---[ end Kernel panic - not syncing: Attempted to kill the idle task! ]---
> Linux 5.10.50 seems pretty old. Does it happen with current Linux 6. […]

Yes, in 6.10.0, hangs at below sites without further logs.

... version:                5
... bit width:              48
... generic registers:      8
... value mask:             0000ffffffffffff
... max period:             00007fffffffffff
... fixed-purpose events:   3
... event mask:             00000007000000ff
signal: max sigframe size: 1360
Estimated ratio of average max frequency by base frequency (times 1024): 1455
rcu: Hierarchical SRCU implementation.
rcu: 	Max phase no-delay instances is 1000.
smp: Bringing up secondary CPUs ...
smpboot: x86: Booting SMP configuration: