Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 14 Jul 2022 12:58:32 -0700
From:   Ira Weiny <ira.weiny@intel.com>
To:     Dan Williams <dan.j.williams@intel.com>
CC:     Bjorn Helgaas <bhelgaas@google.com>,
        Jonathan Cameron <Jonathan.Cameron@huawei.com>,
        Lukas Wunner <lukas@wunner.de>,
        "Alison Schofield" <alison.schofield@intel.com>,
        Vishal Verma <vishal.l.verma@intel.com>,
        Dave Jiang <dave.jiang@intel.com>,
        Ben Widawsky <bwidawsk@kernel.org>,
        <linux-kernel@vger.kernel.org>, <linux-cxl@vger.kernel.org>,
        <linux-pci@vger.kernel.org>
Subject: Re: [PATCH V13 9/9] cxl/port: Parse out DSMAS data from CDAT table
Message-ID: <YtB1aE/xUnoeuOtp@iweiny-desk3>
References: <20220705154932.2141021-1-ira.weiny@intel.com>
 <20220705154932.2141021-10-ira.weiny@intel.com>
 <62d045039d44b_16fb9729443@dwillia2-xfh.jf.intel.com.notmuch>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <62d045039d44b_16fb9729443@dwillia2-xfh.jf.intel.com.notmuch>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?7xGjwlxohZDXwlYEI8nhehwucZ6vJQvYI3iNqKSAcV6NDRPQ5oA4hUQ5VSz8?=
 =?us-ascii?Q?Qs8KjATLNa6U+lZ6/M32rBhoQEeTD0cu49ronpR8C/BHBQwKKj9QGuoDDQOX?=
 =?us-ascii?Q?xxSzVJwkKyOyVlbmFbNBi4OUZNdfKEqJjkeiYqoC8gTTJMB6DSltKZZZeFgg?=
 =?us-ascii?Q?Q3ybnIpXz4p2TeEmQ1FdvUGihlcHIwg5cWjzb4hK2qMLCOZWLWA7Ub54Skd8?=
 =?us-ascii?Q?tKYwZYdaePsxCum1Mbeght+WQKDEM5nuGeHQNVAyTA6XLbCdrReuSAKBzYDG?=
 =?us-ascii?Q?WVxyJqWc/6BmLeAJZufvkEIjKQcUpGqtNrU+8GEdKkVEpMP+OdI5x2EGthCU?=
 =?us-ascii?Q?UZVzZedYhFql5+A1KUwtM/20tbX1XCLoszxxrQXp5FocZ0zpyCmegFaiPdGl?=
 =?us-ascii?Q?V8MxrThyFmIT+LHTdxch3/Y0lL0kgrhSk52V2YLjtLKXla/djxOKbOHC6C2E?=
 =?us-ascii?Q?6jJxPM9t+Yvf61VNwpJGXkprOKzzzKGBq3FwICN4rs+QVnq3kb1Bk/AfJKsp?=
 =?us-ascii?Q?rG0jp+/Lh2FZGwko1BmtMKNMd1PtcStEG32uCOZFtDWOa9TuGL8eQK3kTswZ?=
 =?us-ascii?Q?sLjvgcadFGCOsyWy5732memWubrzjuN4GLsY8c5TgG73eWKtdZbvrwpWKS7j?=
 =?us-ascii?Q?1NyaGC/gLlsIJkY/B0ostotQkKpyrC1H9Fn5lSBBDwAXCLYuUjnC7fX1hm9U?=
 =?us-ascii?Q?JbuqFuUNPDSePjw21RT+QI9wRNI35HgD0dURlVcowHUW6Kwt5YCXT4iBjRul?=
 =?us-ascii?Q?+PJRjVuAyT99kEAXjbWgILI2vsfSGbZ2s3PJHyHT+8Ueh9YGmIpQUw/81/Az?=
 =?us-ascii?Q?/l0Rs7wvNMuaJ3BHXjmUR7sEQ9YrqNXM40cauLmgfwMOLOThNaaKVY4sjej+?=
 =?us-ascii?Q?2YPoRov5WBVoTYNdqiqC+g1c3hk60nXiUyUJIg94Q2iVcN6dNCPOCfSzKCAY?=
 =?us-ascii?Q?pLx4h9GYBso3Mg3j3fpJhXVUdeHqVjGoeQv9eMIarhUTyjGugoTdo/ffc9qd?=
 =?us-ascii?Q?jZ6yezKUx5FWVWR+9Y2lQgbnVCkycUwkzh9NaTo++BOtV+yFtbbHKuN+APvf?=
 =?us-ascii?Q?07+V9eMvTfJGH4ZdYpSyo83CyFIlTlIZVdytz+ChvQXKChrYhdcYtsNN0/Dw?=
 =?us-ascii?Q?AHZ8oWJs0JrkvERL7W1fKGHX4xDYxoMxwZy/7MdyNHbaK7b6uXd7euxuCpMM?=
 =?us-ascii?Q?y8VmlcasGsNtTd3Z5kbVVCKG0hK9Vgy5Bawmr0QXbMDZ5V/yCcOw5QF3xC5b?=
 =?us-ascii?Q?GKXenM6oKV+oTF8zVb3kFl3VFAXqrpi+Boj+3gTlv+EiMDFf9vrFPzPnzkbg?=
 =?us-ascii?Q?uXfRIqagFg5nCVYv0G8f5qFPpUGcp6e94ETJ5wYP1JncnQwKVAlQ/CpN/7BZ?=
 =?us-ascii?Q?aueH3mFXfbhp4ULpqtssLo/SiXFeRXjPFRitvF78EpmbAmxYIP2ItdXKIfSB?=
 =?us-ascii?Q?g1ho6B9wOqqOYbmlN4n4cxVpDXnxySNLHmJ6k+MX//oe2TjKG7BmmnWeZsoa?=
 =?us-ascii?Q?fA3SHuBtRaj7nlazaDIKB8EmnJyYO0xZziSv3aPvaUOVQ4qANbhg4Heo7i3z?=
 =?us-ascii?Q?59nvDnFub4az/ZXKPphR66p2HnVX9ql1FofC9xXsR4h7K7smoUxMdDi/+VOx?=
 =?us-ascii?Q?G/tQf8sANeUAIbrVjmNejZ4w+L0aDBS6ykA2ZwbVrASK?=
X-MS-Exchange-CrossTenant-Network-Message-Id: d1465be7-d9be-4a73-9793-08da65d33ecc
X-MS-Exchange-CrossTenant-AuthSource: DM4PR11MB6311.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Jul 2022 19:58:39.3384
 (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: 1pLRUdrWLifnGlS7IFkrU0seo+1C41YPbZRiDBQxzWrrWtXfMLci9Ot6HWPDurAaFjhfDHu6PzhWCXVN6cshMQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR1101MB2124
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-5.0 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,
        SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Thu, Jul 14, 2022 at 09:32:03AM -0700, Dan Williams wrote:
> ira.weiny@ wrote:
> > From: Ira Weiny <ira.weiny@intel.com>
> > 
> > CXL Ports with memory devices attached need the information from the
> > Device Scoped Memory Affinity Structure (DSMAS).  This information is
> > contained within the CDAT table buffer which is cached in the port
> > device.
> > 
> > If CDAT data is available, parse and cache DSMAS data from the table.
> > Store this data in unmarshaled struct dsmas data structures for ease of
> > use later.  Ensure DSMAS headers are not malicious or ill formed so as
> > to cause buffer overflow errors.
> > 
> 
> Per patch 6 the parsing needs to move to a refactored version of the
> ACPI table parsing,

I'm not seeing that parsing ATM but I'll figure out what you mean.

> but for now it can just be dropped.

Dropped.
Ira