From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qv1-f47.google.com (mail-qv1-f47.google.com [209.85.219.47]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 638EB194C6A for ; Fri, 7 Feb 2025 21:24:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.47 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738963455; cv=none; b=nvSiHISbtOHPjDD5ZBPW+uCOXM18LM04ODhF95NpBmy0RnKoJr9LzGEg+tuGOrpS8rqRMXR94YDoqvEESrCDI/f6vqSZPrJ+F1Tqd6w22Bq/EsQ9KEP7L+6Keg5BIPALo383nOvDDTSpz7PWpWh1liBgKOxvlI04Hd5n42qbSuk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738963455; c=relaxed/simple; bh=Jh8bJM2/fI9Ng6LWK2hdPmWaTdsJAUhglOnt064rwcQ=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=UW6n4TgP+hJGhoXQV1IwnFg3AsNs7U4N+aMSnFBARpcewE3o4+5C7CX6cvm+1vgmrEwlcdrzKywGXcjsJEv0r8L0WvMBiNYUmHzQFNAeZRlnzMbJzsx/J+t8vKCGH2dOjzLawYWA4Vmxn+GZZEobm0aBDyfOFpFpuOKMbtFm/do= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=gourry.net; spf=pass smtp.mailfrom=gourry.net; dkim=pass (2048-bit key) header.d=gourry.net header.i=@gourry.net header.b=odQmWd+W; arc=none smtp.client-ip=209.85.219.47 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=gourry.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gourry.net Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gourry.net header.i=@gourry.net header.b="odQmWd+W" Received: by mail-qv1-f47.google.com with SMTP id 6a1803df08f44-6e440e64249so22716576d6.3 for ; Fri, 07 Feb 2025 13:24:13 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gourry.net; s=google; t=1738963452; x=1739568252; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=gYyXNF2oTW8ZaEdbEsNiifpIRvkBhrKm317n8H4kIGo=; b=odQmWd+W3Dxj5N8vfZrbfR3yoHcsLCQQVyMysetv8CBtNjrxqb6yahA7njybQEUmW/ V5OLzcbtDzFyZHaAGTk5ZK+eJnhskKVum8kqsW0d9ZbVb62fKSkAC80cliIFfxHdQGwH pBaHTMHmOX6bxkZpd+GQxlbLz/ZUsuR3t7SoUu/bbifRhJ8bAQe+myf7VQy2MszwtEgF +Wy/yYfg4DCjkKsuq7FhBc4KIolDF+FXnqBT4b2iOQoJ44ADjXp4D6N8SuIFuWT264qn BizcX4ML/XDzQ1BnlLbFp8988RIZX+5HyD2QurnYQFKgxN7tXtSDXALSoVtdAvaS6Aav 1NEw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1738963452; x=1739568252; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=gYyXNF2oTW8ZaEdbEsNiifpIRvkBhrKm317n8H4kIGo=; b=JVR8ztMbzlzpHNNZ2OCnH4vkGT2TBFfdEBBGvcF+/NSb5M0waZ1SS9Q3jY21E9mJ8G Ys8IeJMoFu+ds55H2UrfwFjHxhC7saV7AuK4mAQyXqHdFQaM14r33FvNwb6H1QqLVo24 eC7joq/J34PfAjA6oKqJBcvX3/1PRUeHJ8Lu8gyHiwTlE1/VWH3gRVUyc3CEhOQZCwCG kjM65TksOuNnENP6zR+F0mAJ9F8e9AqlGAQ0VM2bDXa+2aZHaLOT8R6YuRo49Li4mCeZ 1vRSmcmqfVkTfVSOEmEwgbdtgbkjr9XxRZ7XwZEaEwYt9lOBbdrIKp/guLrH6Bzvs/rV 14PA== X-Forwarded-Encrypted: i=1; AJvYcCXftpmLU/y1DNTDeuWap83juHW/QsKfWhmU7EJ5YidiV30ZH81gV1Zg0ld7KjpbsOvWX0mw/e2Ck84=@vger.kernel.org X-Gm-Message-State: AOJu0YzJa872f1OBKdDA8HARB2d7nCuqrex0t+tDG7MRbwjrt+me3qgw BbhCkViuXo1/hNU9bmJlsRp/hKJOctesVHqYdmXcRpXICVo2PHgppcWhYywyDVPjprMFuPdd392 e X-Gm-Gg: ASbGncsVkLCVjp57T9BuV/v/HV85o3KLbFAc9a82JWTrBWOwMq4VJt4NPcxDeJ0GbsY wwsRZKaJqWvybSu5IBpOFTN6EZZ6vNYRg5jgWPJGhNF9r4Dps8aEQ/1jhsIhlQt04pZWff0lCB+ v2dbLnZKH5P8079vBDeZbqjyP52D/kY51IRYVipp+TDYMyzltH8R3W0XlEtcCZqKw4w2BRCCLXx vyE5mEQahd8lP7rdGPZwzujqat3Wl3iTD5M0a3y/zOwJuu+BksNyqOXVXjQzTiTc1KsbpQ6HYtM Nc/CcOZE2u9QvnVDZsAwfHYWpZ2159Hni3NIEL8p7gUUfAt519v6vL3pM8TpZ7pU54R8I/bdFg= = X-Google-Smtp-Source: AGHT+IFobMYtRD+ts0IjDmZUcfPs9gYsgQs0G1siAa2ZlSeZodwmtBL8KT228UzP8KiW1YKKcAkJYA== X-Received: by 2002:ad4:5bc4:0:b0:6e4:41b3:4983 with SMTP id 6a1803df08f44-6e4456dd202mr74890986d6.35.1738963452178; Fri, 07 Feb 2025 13:24:12 -0800 (PST) Received: from gourry-fedora-PF4VCD3F (pool-173-79-56-208.washdc.fios.verizon.net. [173.79.56.208]) by smtp.gmail.com with ESMTPSA id 6a1803df08f44-6e44f4835e6sm4336616d6.111.2025.02.07.13.24.10 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 07 Feb 2025 13:24:11 -0800 (PST) Date: Fri, 7 Feb 2025 16:24:09 -0500 From: Gregory Price To: Robert Richter Cc: Alison Schofield , Vishal Verma , Ira Weiny , Dan Williams , Jonathan Cameron , Dave Jiang , Davidlohr Bueso , linux-cxl@vger.kernel.org, linux-kernel@vger.kernel.org, "Fabio M. De Francesco" , Terry Bowman Subject: Re: [PATCH v2 00/18] cxl: Address translation support, part 1: Cleanups and refactoring Message-ID: References: <20250207153753.418849-1-rrichter@amd.com> Precedence: bulk X-Mailing-List: linux-cxl@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20250207153753.418849-1-rrichter@amd.com> On Fri, Feb 07, 2025 at 04:37:35PM +0100, Robert Richter wrote: > This series is the first part of adding support for CXL address > translation. It contains cleanup and code refactoring in preparation > of the actual implementation that will be sent in part 2. Cleanup and > code refactoring have been split in a separate series to reduce the > number of patches of the series. Even without address translation on > top this rework improves esp. the region code, cleans it up, > simplifies it and adds debugging messages to better analyze region > creation failures: > > Content of patches: > > * Patches 1: Remove else after return. > > * Patches 2-4: Cleanups and comments around cxl_hdm_decode_init(). > > * Patches 5, 6, 13: Adding and modifying helper functions. > > * Patches 7-12, 14: Refactoring of endpoint decoder setup. > > * Patches 15-18: Adding and modifying debug messages. > > Most of the patches were part of my first submission of v1 [1], some > of them were already reviewed. > > [1] https://lore.kernel.org/linux-cxl/20250107141015.3367194-1-rrichter@amd.com/ > For entire series: Tested-by: Gregory Price Tested individually and w/ a rebase of the full translation patch set from [1] on a Zen5 platform with PRM and translation. Will test again with part-2 as I expect some changes.