Received-SPF: pass (google.com: domain of linux-kernel+bounces-106727-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: Mon, 18 Mar 2024 22:55:35 +0100
From: Robert Richter <rrichter@amd.com>
To: Dan Williams <dan.j.williams@intel.com>
Cc: "Rafael J. Wysocki" <rafael@kernel.org>,
	Dave Hansen <dave.hansen@linux.intel.com>,
	Alison Schofield <alison.schofield@intel.com>,
	linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-cxl@vger.kernel.org, Len Brown <lenb@kernel.org>
Subject: Re: [PATCH 2/3] ACPI/NUMA: Print CXL Early Discovery Table (CEDT)
Message-ID: <Zfi4V1g8kvcgUx44@rric.localdomain>
References: <20240318210904.2188120-1-rrichter@amd.com>
 <20240318210904.2188120-4-rrichter@amd.com>
 <65f8b263bd757_aa2229444@dwillia2-mobl3.amr.corp.intel.com.notmuch>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <65f8b263bd757_aa2229444@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?2NoEQECxZ8YiWjdhKHR4NsZpiiXm4q657TXx+nNwIP7YZd/ST6FNskr0aJWw?=
 =?us-ascii?Q?BeEDZdMgP3C1Xa38mjl8jIL7QeLBrjhxri1FarOS4Y5mTL6losJor6z4KZqj?=
 =?us-ascii?Q?D8M7bBbcWJgzA86NRnYsJVd+S17mt4H+vaI9h95LdCzQesurPyNJrCVsrNtx?=
 =?us-ascii?Q?40Zd27qU8Co3SlpsY7lfO5I0LmhtR/Boe7IlRhztI/oS+GRt1laqgoBOlfNv?=
 =?us-ascii?Q?SjBT3cD5/I7+p0IGIsSg8YJZObxRdEEKOBFQiup4JbcsFi7c9P7jBHAxe/al?=
 =?us-ascii?Q?3AvrZIJkw+rau7AmBzPfqmI19scJ9iDtZ2KecQII1uFlAc0OM8pJ66c/jYlH?=
 =?us-ascii?Q?rgK5OTcL2+pqT2nyYz3uGWrDNdAaXNj9HJIr43brHL0dfWXezhz2/xFliHyv?=
 =?us-ascii?Q?wbdnoBHszX18L/aYtkEasphd2Aqmx6WLfPoqwhxVozYRhsO2bZnM7Rz15X2/?=
 =?us-ascii?Q?RLpUu8tCey3GxNJMahnInGytxV9btcc6yAnegf3LrZSDns7tJU+6pNXpgt3j?=
 =?us-ascii?Q?L0RdK0vjbSZdepNc20XGI0OujNy1S1BI5C+WXevijXSapXlvyvLsskGAfpD1?=
 =?us-ascii?Q?TBQqNwlRLe3a8ArsXd0qU/4rrOUXliqIUvi/a6VLKUUEBbcGrESv97dQuHk1?=
 =?us-ascii?Q?2J1dXhGaHD8vLzQSR01AwbOK9KWS9Krf6N5/i+mOLXEq9QYI56FgxfGP0BS3?=
 =?us-ascii?Q?q0uX/lVNr1Z76AeRxnMmO/fyFFYAW+2FtLQEY/9SsMFKANGoEMs5kMwhzOZ7?=
 =?us-ascii?Q?ezgS06n9zYLsthcSkrqVr2TeZgg1CxC/XTGrswnnPm6DjLvZj5Lbmsp8RXDo?=
 =?us-ascii?Q?3aVBsdVQ9BlOYKRmjCjoaQhCrTVlAgtzvGyIOZdMqCEIpm8P/Pls04quF+HX?=
 =?us-ascii?Q?mt8PQVHjeqeU/lpQQMUytitEHv+usqFZTMuHRXbtKOlalSZDkjYKMp6aRiIL?=
 =?us-ascii?Q?mMZHYnBart2lvtxzsqaZEE64ccujRDC+mav/wgkFCHndrrKKS6lVC06DLCRq?=
 =?us-ascii?Q?lu1L4tBfB5obD0rqiwQHBeBdRFWvst9kXyDJeUgIoj8OF07dZbz6s7a8xYD5?=
 =?us-ascii?Q?46NxeW/h/pxEr7sPhgVnMLiieb5T/ol/z+oisYIi/vfPDRPsNTv8P8zTBPvU?=
 =?us-ascii?Q?XP7QBSXD7BtF199jRm7UlH9yn38eLZ/6ABYohgmPW3QuXcUPIF/vqIxBIdoD?=
 =?us-ascii?Q?zdJs8B88dmTQ+kHFwMybSIh8qVTmv2JhoKCqU/JzBveXxwt83CR0+0R/ze/+?=
 =?us-ascii?Q?XkSWdfgNG6TNl+++8F4CebjeKN19oPluypBuX/6PTKrHf5pfuCU5Wqi5ZMb5?=
 =?us-ascii?Q?2LAGU9jrOkqJg/PvoBdP1sU73txngg1iv0AuHSg0rjBo6IgPpf9TrwI+PZz6?=
 =?us-ascii?Q?eRgPISdxG6hgb4wUSGGD0xb87BypN0KxvAU2nf1bnM+fsfmLjuyOR1LuycQ8?=
 =?us-ascii?Q?7UaOitz3Qu52cH3tmviu/mvWprcGXc1mkPAyhe69Vmnc7h30ohF6qX8z+0BU?=
 =?us-ascii?Q?Vg+wweM1uOKOZJx9gBuNDNZ2Z+VY7RQLxnSx/J/ExNqIMO3EEFSvRpsX9sx4?=
 =?us-ascii?Q?RXCcEu0UxiyGtXH8jLt3uPZdvaBkTCT6PmqCqKuF?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b4869d91-168a-4bad-f428-08dc4796271c
X-MS-Exchange-CrossTenant-AuthSource: CYYPR12MB8750.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Mar 2024 21:55:40.7437
 (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: 461MJEQJygeW3RA2C2pZKKeTGjlvZ2OcvnB0cQhBjpv038ePM2TWrAu3QqAv99sqyX83OyGGzOdZEOE7O4zpxQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR12MB7119

On 18.03.24 14:30:11, Dan Williams wrote:
> Robert Richter wrote:
> > The CEDT contains similar entries as the SRAT. For diagnostic reasons
> > print the CEDT same style as the SRAT.
> 
> I will defer to Rafael here, but with acpica-tools, cxl-cli, and
> /proc/iomem is there much value to adding this to the boot-up logs by
> default?

Compared to the number of SRAT printks there are just a few CEDT
entries. I have added it as CEDT and SRAT entries have the similar
effect to NUMA, so diagnostics should be at the same level.

Thanks,

-Robert