Received-SPF: pass (google.com: domain of linux-kernel+bounces-164499-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: Tue, 30 Apr 2024 10:14:02 -0700
From: Dan Williams <dan.j.williams@intel.com>
To: Robert Richter <rrichter@amd.com>, "Rafael J. Wysocki" <rafael@kernel.org>
CC: 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>, Robert Richter <rrichter@amd.com>, Len Brown
	<lenb@kernel.org>
Subject: Re: [PATCH v6 6/7] ACPI/NUMA: Add log messages for memory ranges
 found in CEDT
Message-ID: <663126da389b7_10c2129445@dwillia2-mobl3.amr.corp.intel.com.notmuch>
References: <20240430092200.2335887-1-rrichter@amd.com>
 <20240430092200.2335887-7-rrichter@amd.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20240430092200.2335887-7-rrichter@amd.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?2ZkARnB0rx4ZTRcpm1uBoS9H6Puooo2GllqKoaW1FTyd5pH2+ybd8JpMbrcf?=
 =?us-ascii?Q?q00a10DcBFNTxQXS4r/BvmZ9YbQcl9f1FdOdKCP7v3pva7rb73imkTl2Plck?=
 =?us-ascii?Q?v4spX7/OphNgVXPUu+7zwIwTMZm8W1CVUcIlVWMrPTAqdAR+eZJff8POioXJ?=
 =?us-ascii?Q?8RvI2kIDfd8IWelEd5lcQcGEdIyPDpItfb0ecRb1ItT5U6zejZNJfgJca/67?=
 =?us-ascii?Q?iwGo2rYrW12CYEOqvMwXpJ8QC+KEDSwIi2U5g2bizD8LX6e49fWGUrZjy9qu?=
 =?us-ascii?Q?3RkdEC5+IRc8FJl1xa8pdQ94jE3XZzjSEIkUPDr71W0y/vC+REem0Lxthumx?=
 =?us-ascii?Q?0guSqnWUuWSinV21KktZkSSS29f30QuzA+ph48cs7WmwmER478hkm4u+etOD?=
 =?us-ascii?Q?YjcR7EnDCEfAr2nppSNy30fY7+v8jJBvgLgI4oEL0p32iOv/rC8JV1jdBOag?=
 =?us-ascii?Q?kMPAYaQPZgH1tzj0hRMx76/hBRqZ4pruYOz0O+yxzpW/UbIQBZJpOJKFwGdA?=
 =?us-ascii?Q?4eIM9srjHHQSw9ykQdGvEYabcRcBCvhpMdUAZEgpXAxvp7hRe/G1EhoceLeK?=
 =?us-ascii?Q?D3Gd5uTI8e13UYaKH9mrXYkL0hbFfkfriCxAPC6hjBXOvLJyWd8xxoSUxwYv?=
 =?us-ascii?Q?+AjwsJkaSTG6rPYwDU7+da1wphqmwGRaLCqgl1Og1qE0pOP1y1wAYJ/jCJ63?=
 =?us-ascii?Q?Vijeg52zG3rtQYVDTTTU1aLdPalGv4vKAp6KOxLs3jeHky0unEtC/lT6Vqt9?=
 =?us-ascii?Q?ReqTqn2kB1Nh8tVUlN6re8eckOpfJwNoUMdxdTbgRbp/9yeiz+XuGUXetZdm?=
 =?us-ascii?Q?yr1V8JhUGr1dokwcSMw5l4FKpHqCt08z7vgfdy1yb6QcMnzRbNHsofb7Tnef?=
 =?us-ascii?Q?4qFTHGekUJu2chyT8sQ3fwXmiS0Z+UBq7jQnh2js/p0jSwAd7OUYxZEhm3pE?=
 =?us-ascii?Q?hVdYxyWBhBw5I2iBG6UuFQP0p6QpfxbfDKnguv3CW3FaXDfBB6sKzqCxZxc9?=
 =?us-ascii?Q?8O54NYYIbHFs3qbXK9CRe1yo6qA31FRpc8RiUWWFrNNo1DB0vDFKiD3QYPXg?=
 =?us-ascii?Q?cgGNcJ86vfnnl39DSwDz2fqCcCXH8PLqY9Fp6YSaSNC8A950WUTUOwK2fDt/?=
 =?us-ascii?Q?aGwpdQZX3y3V+/KzW5pzTHHEK2YAnN1yLITRVwIx2QogMXCcfYGfR03URy6I?=
 =?us-ascii?Q?1gifShlgLzcYPZhgerk0sEC9pKNWNBb5RRURjMRF979JF9D5gyvBo+dMU9GY?=
 =?us-ascii?Q?JfivHM1+SOACgTNmvgpjuXNWwy3okURQ0IHewtG3fyu/EIv/9InqmNw7D8pr?=
 =?us-ascii?Q?Citb8tPhI9KYSxetvJqpWLuJi1gGs5/bsqoC+5oEcszcAp1myo8g1OtosPj0?=
 =?us-ascii?Q?oTH9YqiP/vQw1F0PLFSwTiWr0lH0kPHNu8/jvyXz4GDJ3xFRPeJAAZ6XgdC/?=
 =?us-ascii?Q?u99Ft9w41BOlIGHkY5vYx1lkgouBU12roVCGDVhmm1eCLB5xHzkwt/XYpVLB?=
 =?us-ascii?Q?yakYQzebqhZxso99ssE6teSwjEQcoFfOHHiDu2HhZ96JjFTiGFR81790oETY?=
 =?us-ascii?Q?KzzO2gkypdTx37LxH3gkCk2yr+oO7jfvL4xrWDU1QGs+0EAoxYpr93+3Kj3P?=
 =?us-ascii?Q?9w=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: f1e57431-65d6-4dac-3919-08dc6938f030
X-MS-Exchange-CrossTenant-AuthSource: PH8PR11MB8107.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Apr 2024 17:14:04.9940
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: A42qgUZ/G0l1EAEKagSOeEkAdUnK7fWQKzW9DOuzw01lYyRPOC5B72RXBZj7a+OiocmBB55MOgEsRTcPb5xIv73p36k5/b3lhW9kxFd6iVY=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR11MB6896
X-OriginatorOrg: intel.com

Robert Richter wrote:
> Adding a pr_info() when successfully adding a CFMWS memory range.
> 
> Suggested-by: Alison Schofield <alison.schofield@intel.com>
> Signed-off-by: Robert Richter <rrichter@amd.com>
> ---
>  drivers/acpi/numa/srat.c | 11 ++++++++++-
>  1 file changed, 10 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/acpi/numa/srat.c b/drivers/acpi/numa/srat.c
> index 76b39a6d3aef..34ecf2dc912f 100644
> --- a/drivers/acpi/numa/srat.c
> +++ b/drivers/acpi/numa/srat.c
> @@ -339,8 +339,12 @@ static int __init acpi_parse_cfmws(union acpi_subtable_headers *header,
>  	 * window.
>  	 */
>  	modified = numa_fill_memblks(start, end);
> -	if (modified != NUMA_NO_MEMBLK)
> +	if (modified != NUMA_NO_MEMBLK) {
> +		if (modified)
> +			pr_info("CEDT: memblk extended [mem %#010Lx-%#010Lx]\n",
> +				(unsigned long long) start, (unsigned long long) end - 1);

This one still feels too verbose to me, can it not be gleaned from other
startup messages?