Received-SPF: pass (google.com: domain of linux-kernel+bounces-157846-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Wed, 24 Apr 2024 16:11:42 -0700
From: Davidlohr Bueso <dave@stgolabs.net>
To: Robert Richter <rrichter@amd.com>
Cc: Jonathan Cameron <jonathan.cameron@huawei.com>,
	Dave Jiang <dave.jiang@intel.com>,
	Alison Schofield <alison.schofield@intel.com>,
	Vishal Verma <vishal.l.verma@intel.com>,
	Ira Weiny <ira.weiny@intel.com>,
	Dan Williams <dan.j.williams@intel.com>, linux-cxl@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3] cxl: Fix use of phys_to_target_node() for x86
Message-ID: <20240424231142.rytjmxfassu44vln@offworld>
References: <20240424154756.2152614-1-rrichter@amd.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Disposition: inline
In-Reply-To: <20240424154756.2152614-1-rrichter@amd.com>
User-Agent: NeoMutt/20220429

On Wed, 24 Apr 2024, Robert Richter wrote:

>The CXL driver uses both functions phys_to_target_node() and
>memory_add_physaddr_to_nid(). The x86 architecture relies on the
>NUMA_KEEP_MEMINFO kernel option enabled for both functions to work
>correct. Update Kconfig to make sure the option is always enabled for
>the driver.
>
>Suggested-by: Dan Williams <dan.j.williams@intel.com>
>Link: http://lore.kernel.org/r/65f8b191c0422_aa222941b@dwillia2-mobl3.amr.corp.intel.com.notmuch
>Reviewed-by: Ira Weiny <ira.weiny@intel.com>
>Reviewed-by: Dan Williams <dan.j.williams@intel.com>
>Signed-off-by: Robert Richter <rrichter@amd.com>

Reviewed-by: Davidlohr Bueso <dave@stgolabs.net>