Received-SPF: pass (google.com: domain of linux-kernel+bounces-171224-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Precedence: bulk
MIME-Version: 1.0
References: <20240502131012.2385725-1-rrichter@amd.com>
In-Reply-To: <20240502131012.2385725-1-rrichter@amd.com>
From: "Rafael J. Wysocki" <rafael@kernel.org>
Date: Tue, 7 May 2024 13:48:49 +0200
Message-ID: <CAJZ5v0hHqbFyCARK9+OegRjdFNpq+nX+QMb+gF1w9333DOXe0g@mail.gmail.com>
Subject: Re: [PATCH v7 0/4] SRAT/CEDT fixes and updates
To: Robert Richter <rrichter@amd.com>
Cc: "Rafael J. Wysocki" <rafael@kernel.org>, Dave Hansen <dave.hansen@linux.intel.com>, 
	Dan Williams <dan.j.williams@intel.com>, Alison Schofield <alison.schofield@intel.com>, 
	linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, 
	linux-cxl@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Thu, May 2, 2024 at 3:10=E2=80=AFPM Robert Richter <rrichter@amd.com> wr=
ote:
>
> 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!