[v7,0/4] SRAT/CEDT fixes and updates

Message

Robert Richter May 2, 2024, 1:10 p.m. UTC

Some fixes and updates for SRAT/CEDT parsing code. Patches can be
applied individually and are independent.

First patch fixes a page fault during boot (fix as suggested by Dan).

Patches 2 to 4 remove architectural code no longer needed.

Changelog:

v7:
 * added Reviewed-by tags
 * dropped printout patches (#5 to #7)

v6:
 * rebased onto cxl/fixes
 * fixed 0day build errors in patch #1:
   https://github.com/intel-lab-lkp/linux/commits/Robert-Richter/x86-numa-Fix-SRAT-lookup-of-CFMWS-ranges-with-numa_fill_memblks/20240429-205337

v5:
 * dropped: "x86/numa: Fix SRAT lookup of CFMWS ranges with
   numa_fill_memblks()"
 * added: "ACPI/NUMA: Return memblk modification state from
   numa_fill_memblks()"
 * conditionally print CEDT extended memblks

v4:
 * updated SOB chains and desription
 * added patch "x86/numa: Remove numa_fill_memblks() from sparsemem.h
   using __weak"
 * Reordered patches to move CEDT table printout as an option at the
   end
 * split print table patch and added: "ACPI/NUMA: Add log messages for
   memory ranges found in CEDT"

v3:
 * Rebased onto v6.9-rc1
 * Fixing x86 build error in sparsemem.h [Dan/Alison]
 * Added CEDT node info [Alison]
 * Use pr_debug() for table output [Dan]
 * Refactoring split in 3 patches [Dan]
 * Fixed performance regression introduced [kbot]
 * Fixed checkpatch issues [Dan]

Robert Richter (4):
  x86/numa: Fix SRAT lookup of CFMWS ranges with numa_fill_memblks()
  ACPI/NUMA: Remove architecture dependent remainings
  ACPI/NUMA: Squash acpi_numa_slit_init() into acpi_parse_slit()
  ACPI/NUMA: Squash acpi_numa_memory_affinity_init() into
    acpi_parse_memory_affinity()

 arch/x86/include/asm/sparsemem.h |  2 -
 arch/x86/mm/numa.c               |  4 +-
 drivers/acpi/numa/srat.c         | 82 +++++++++++++-------------------
 include/linux/acpi.h             |  5 --
 include/linux/numa.h             |  7 +--
 5 files changed, 35 insertions(+), 65 deletions(-)


base-commit: 5d211c7090590033581175d6405ae40917ca3a06

Comments

Rafael J. Wysocki May 7, 2024, 11:48 a.m. UTC | #1

On Thu, May 2, 2024 at 3:10 PM Robert Richter <[e-mail]> wrote:
>
> Some fixes and updates for SRAT/CEDT parsing code. Patches can be
> applied individually and are independent.
>
> First patch fixes a page fault during boot (fix as suggested by Dan).
>
> Patches 2 to 4 remove architectural code no longer needed.
>
> Changelog:
>
> v7:
>  * added Reviewed-by tags
>  * dropped printout patches (#5 to #7)
>
> v6:
>  * rebased onto cxl/fixes
>  * fixed 0day build errors in patch #1:
>    https://github.com/intel-lab-lkp/linux/commits/Robert-Richter/x86-numa-Fix-SRAT-lookup-of-CFMWS-ranges-with-numa_fill_memblks/20240429-205337
>
> v5:
>  * dropped: "x86/numa: Fix SRAT lookup of CFMWS ranges with
>    numa_fill_memblks()"
>  * added: "ACPI/NUMA: Return memblk modification state from
>    numa_fill_memblks()"
>  * conditionally print CEDT extended memblks
>
> v4:
>  * updated SOB chains and desription
>  * added patch "x86/numa: Remove numa_fill_memblks() from sparsemem.h
>    using __weak"
>  * Reordered patches to move CEDT table printout as an option at the
>    end
>  * split print table patch and added: "ACPI/NUMA: Add log messages for
>    memory ranges found in CEDT"
>
> v3:
>  * Rebased onto v6.9-rc1
>  * Fixing x86 build error in sparsemem.h [Dan/Alison]
>  * Added CEDT node info [Alison]
>  * Use pr_debug() for table output [Dan]
>  * Refactoring split in 3 patches [Dan]
>  * Fixed performance regression introduced [kbot]
>  * Fixed checkpatch issues [Dan]
>
> Robert Richter (4):
>   x86/numa: Fix SRAT lookup of CFMWS ranges with numa_fill_memblks()
>   ACPI/NUMA: Remove architecture dependent remainings
>   ACPI/NUMA: Squash acpi_numa_slit_init() into acpi_parse_slit()
>   ACPI/NUMA: Squash acpi_numa_memory_affinity_init() into
>     acpi_parse_memory_affinity()
>
>  arch/x86/include/asm/sparsemem.h |  2 -
>  arch/x86/mm/numa.c               |  4 +-
>  drivers/acpi/numa/srat.c         | 82 +++++++++++++-------------------
>  include/linux/acpi.h             |  5 --
>  include/linux/numa.h             |  7 +--
>  5 files changed, 35 insertions(+), 65 deletions(-)
>
>
> base-commit: 5d211c7090590033581175d6405ae40917ca3a06
> --

Whole series applied as 6.10 material, thanks!