Received-SPF: pass (google.com: domain of linux-kernel+bounces-110011-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Date: Thu, 21 Mar 2024 13:09:08 +0100
From: Robert Richter <rrichter@amd.com>
To: Dan Williams <dan.j.williams@intel.com>
Cc: "Rafael J. Wysocki" <rafael@kernel.org>,
	Alison Schofield <alison.schofield@intel.com>,
	Dave Hansen <dave.hansen@linux.intel.com>,
	linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-cxl@vger.kernel.org, Derick Marks <derick.w.marks@intel.com>,
	Len Brown <lenb@kernel.org>
Subject: Re: [PATCH] cxl: Fix use of phys_to_target_node() outside of init
 section
Message-ID: <ZfwjZMJMNUpWXZN8@rric.localdomain>
References: <20240318210904.2188120-1-rrichter@amd.com>
 <20240318210904.2188120-2-rrichter@amd.com>
 <65f8b191c0422_aa222941b@dwillia2-mobl3.amr.corp.intel.com.notmuch>
 <Zfl9Efxe7DwuU_i3@rric.localdomain>
 <65fa2c219548f_aa2229499@dwillia2-mobl3.amr.corp.intel.com.notmuch>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <65fa2c219548f_aa2229499@dwillia2-mobl3.amr.corp.intel.com.notmuch>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?o/IqBz3iH8m59jzBje4GlfIy60V0slJpbH1NwFLVSrl3Syux9Hka8yL0+Ndt?=
 =?us-ascii?Q?JLLSqlIPAGnqdFpXqBgrZu7Vz7tsNptLX3gu3NOXOnO0ah3oSPDTjjHNN7xO?=
 =?us-ascii?Q?gA+/7yA79JiuZlWVXmxENHszf17Oh1heaZ1HB+KoLlejAAoqDIUc7T9fwyE1?=
 =?us-ascii?Q?a6iYVnzu4ZFjYeYz2xU+JsxwpHBUn8WPgH56VMafCa1fMWuG8HyPlcNTwR+8?=
 =?us-ascii?Q?zq322rhA9X6oEh/wKR7daL+TSoFwvBbmyMLg4DExgnoBYcWdGz5SYLLQjR9B?=
 =?us-ascii?Q?09DB60ViFTQAiTS9mYrk2g0ZZ5Tr5J/N1G+UHaNDDJzTa3hveKwXRZKCTRHr?=
 =?us-ascii?Q?NSBX1DREM4Ik2RTiMx8E+jpWgosdYxF/bv2XsPSeLvsdD5Za5KdhatxSUIF8?=
 =?us-ascii?Q?ye++LyUsiro5s1/NyLvyaKfnkbHvEqiY/0zVS73XbtRI15OQCHm8ARfzQ1M3?=
 =?us-ascii?Q?beOxnDcBwx42wqlaSE5zGg0M3/l60TsOhEeYbEh76SBNBRdkMKVAOXpS0pxj?=
 =?us-ascii?Q?f/dEgctiPUuZCvcL6ice5j0HXqGNdRCTDrfVKYctfebRQEuCyrTp2UJunsF7?=
 =?us-ascii?Q?NH0mUDPE/ygkwNLFhH/j3MNc7TriS99bjV9xz1ExQycj7uOdf5VxZm4cjIZ4?=
 =?us-ascii?Q?OAp1S6gT0DTZc1rK4lf/iBYunPzmtB+ZjQOo7EJp17JUURlb3FHc7FGkck8L?=
 =?us-ascii?Q?mkb4MYagYbrIIFNspY9DEyDNNWui0eQQH7hiSjJ3TiWRCOOdQiIjP0Y9XwMZ?=
 =?us-ascii?Q?sMTT91pETpRnktg+n4wyJBYD36O+XJcms0t6mZ0X4WXmF73WfQuKtR4COqBt?=
 =?us-ascii?Q?SCa5k/EUcrzmVdkzpf1JyNvf95KJeEPPCj2R46/e61hfb2wc2mSP8UFpdpaJ?=
 =?us-ascii?Q?XqrOelyt6402mpeb3QbCrwOj8AsQMsz5oLYiqNm5hdV5gwq7R7k52uMNvXoU?=
 =?us-ascii?Q?A0CSVXGD2KuV8/L0qAc3tJ78DWx8zCjL22QH+L+Zvlkoyd6f7qOxlBJwtjf0?=
 =?us-ascii?Q?SiPpGyExDQZ76C2xjl3gwm0QjhX6U50lJPBqs0RFNIC37WQSgAetQEb/Uc+8?=
 =?us-ascii?Q?NSCD85W92uKoTAsIw2RKwMp5zKWhhL3urnsndODcA95J6y82Hnm0Usej8Y38?=
 =?us-ascii?Q?ZUYvvj5Zy1XWgB5e+tSkCn84cVM6q+OUKMpQqOUxzbm1W41djPXQc1h2riag?=
 =?us-ascii?Q?6NX5h3IoXW6rnYxuUaTMTG7SNWBJcQ29ar2tR5yts2W1EevCxtSZ/mVMWKd2?=
 =?us-ascii?Q?l51h4eATCWMclX8AwubHl3LUVBcONUOJH0sRmEFToOqt2j9PZXqeN1MRtU6+?=
 =?us-ascii?Q?9czhWQ7iL1XPVVOXYXnWh0erfV0E9DjoJkpdfjlrXtIOmuFzfRTbaLbqG6Ne?=
 =?us-ascii?Q?sBkfVqiEYlL1YVs1jMZ6ThBrfc90Cw9BKkUhrQ3UCRuAyRqbDXgn7qqrnVtk?=
 =?us-ascii?Q?MyDEc5DXjAxUXcDmcSYrDX8niaejEuACIVOz7u4CQRf2t0ANCYcbcyT+APrk?=
 =?us-ascii?Q?OXQf48QPAF3KYBNjnCp1EUxI6b433yklwG6mpC3GAvhYskowGe/Pc/8ixFA2?=
 =?us-ascii?Q?CYB72py0+G+Mn6ZztvrYxt5bsry0098RWsZ/Aq6b?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f2d24bc5-0eef-4aa8-7927-08dc499fb9f6
X-MS-Exchange-CrossTenant-AuthSource: CYYPR12MB8750.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Mar 2024 12:09:14.9480
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: Y41Sf8aV9rXDeJfSuyX2dy/forSTkEWtNMRwEarORvrZm5i7AFsIkr8saOk1VQjSGtLRlkrbNM+kZSy6YsfS9A==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR12MB6468

On 19.03.24 17:21:53, Dan Williams wrote:
> Robert Richter wrote:
> > Hi Dan,
> > 
> > patch below. I have not included it into v2 of the SRAT/CEDT changes
> > as it is cxl specific and can be applied separately.
> > 
> > Thanks,
> > 
> > -Robert
> > 
> > 
> > On 18.03.24 14:26:41, Dan Williams wrote:
> > > It should also be the case that cxl_acpi needs this:
> > > 
> > > diff --git a/drivers/cxl/Kconfig b/drivers/cxl/Kconfig
> > > index 67998dbd1d46..1bf25185c35b 100644
> > > --- a/drivers/cxl/Kconfig
> > > +++ b/drivers/cxl/Kconfig
> > > @@ -6,6 +6,7 @@ menuconfig CXL_BUS
> > >         select FW_UPLOAD
> > >         select PCI_DOE
> > >         select FIRMWARE_TABLE
> > > +       select NUMA_KEEP_MEMINFO if NUMA
> > >         help
> > >           CXL is a bus that is electrically compatible with PCI Express, but
> > >           layers three protocols on that signalling (CXL.io, CXL.cache, and
> > 
> > From be5b495980bae41d879909212db02dac0fba978e Mon Sep 17 00:00:00 2001
> 
> Hi Robert,
> 
> When you send inline patches like this can you remember to include a
> scissors line? That way tools like "b4 am" automatically know where to
> trim things. So add a line like the following:
> 
> -- >8 --
> 
> ...see "git mailinfo --help" for details.

Thanks for the inside on your patch processing. Will use that in the
future.

> 
> Also note that if you reply with an updated patch in a series include
> the "vX NN/MM" suffix, like "Subject: [PATCH v3 2/3] ..." so that b4 am
> knows to perform a "partial reroll".

This patch is in addition to the other SRAT patches and can be applied
directly to the cxl tree. That is why there is no version update here.
But I replied to this series for reference. I saw the b4 shazam
--no-parent option, would that help here?

Thanks,

-Robert

> 
> > From: Robert Richter <rrichter@amd.com>
> > Date: Tue, 19 Mar 2024 09:28:33 +0100
> > Subject: [PATCH] cxl: Fix use of phys_to_target_node() outside of init section
> > 
> > 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 to be set. Enable the option for the
> > driver accordingly.
> > 
> > Suggested-by: Dan Williams <dan.j.williams@intel.com>
> > Signed-off-by: Robert Richter <rrichter@amd.com>
> > ---
> >  drivers/cxl/Kconfig | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/cxl/Kconfig b/drivers/cxl/Kconfig
> > index 67998dbd1d46..6140b3529a29 100644
> > --- a/drivers/cxl/Kconfig
> > +++ b/drivers/cxl/Kconfig
> > @@ -6,6 +6,7 @@ menuconfig CXL_BUS
> >  	select FW_UPLOAD
> >  	select PCI_DOE
> >  	select FIRMWARE_TABLE
> > +	select NUMA_KEEP_MEMINFO if (NUMA && X86)
> >  	help
> >  	  CXL is a bus that is electrically compatible with PCI Express, but
> >  	  layers three protocols on that signalling (CXL.io, CXL.cache, and
> > -- 
> > 2.39.2
> > 
> 
>