Received-SPF: pass (google.com: domain of linux-kernel+bounces-128829-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Date: Tue, 2 Apr 2024 16:27:03 -0700
From: Ira Weiny <ira.weiny@intel.com>
To: Dave Jiang <dave.jiang@intel.com>, Ira Weiny <ira.weiny@intel.com>,
	Davidlohr Bueso <dave@stgolabs.net>
CC: Fan Ni <fan.ni@samsung.com>, Jonathan Cameron
	<Jonathan.Cameron@huawei.com>, Navneet Singh <navneet.singh@intel.com>, "Dan
 Williams" <dan.j.williams@intel.com>, Alison Schofield
	<alison.schofield@intel.com>, Vishal Verma <vishal.l.verma@intel.com>,
	<linux-btrfs@vger.kernel.org>, <linux-cxl@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 02/26] cxl/core: Separate region mode from decoder mode
Message-ID: <660c9447838db_8a7d0294d6@iweiny-mobl.notmuch>
References: <20240324-dcd-type2-upstream-v1-0-b7b00d623625@intel.com>
 <20240324-dcd-type2-upstream-v1-2-b7b00d623625@intel.com>
 <yd6mynddwk755ypomyspzxfsqwm7j5g47yfmbw2yfwoadpnnqy@7g3ktxrwkeau>
 <6604fe9e18eeb_20890294a1@iweiny-mobl.notmuch>
 <b6ff085c-8cdc-4867-83fd-9566d59d5946@intel.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <b6ff085c-8cdc-4867-83fd-9566d59d5946@intel.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?amOXZkjN55d3jMSc32tprPALzW6vgy0fc4rRf29sOKyg4Z2qt4hQjb2QYc0X?=
 =?us-ascii?Q?3Sb+MWMUJ/vW+VZroQc70DN78Ez5fAfZVIcC2fiilcnL/WzxL4x2fqTlSxX3?=
 =?us-ascii?Q?9e4CBM5TqvJLD5AZKIbNkkbpVVtsCuboabm39AoK3k8E0zH2HHM1UEX7Bvy2?=
 =?us-ascii?Q?/Vtwmc/Dq8AW1JLQRlFiEOxhCz1Vq9Jm+jFYHEqaDeH20rrnnmDDNqtHK4rW?=
 =?us-ascii?Q?JupT6fFvkEi0EcBsNVcbJB92Iz6XYEfLxmbYvZ+QFNs64LnXppyzUD7V3PxS?=
 =?us-ascii?Q?YisgxF9ldpjMWrKc9FXJertsX2DADcnEG6D/XMarGaUJDnp0TOSS6ANFHVsd?=
 =?us-ascii?Q?aBZGT3qAs0lMKHviQi7iyZVqhpf06OVuW9SWznltjhc2Tyw9L/Xn87CkipKt?=
 =?us-ascii?Q?YAS6UHF7m/Ngx0xOYGKtTxJvYLSt3kUB72UTtrkpye7Uvc5yVGzUeSGEYDuM?=
 =?us-ascii?Q?bmEpiwFEXk68H9s2ChL8KS2Qgh11QfJu/aYZ7rF5rg6/pJJHO78P5OGAyfU5?=
 =?us-ascii?Q?tyegYqpWs25zr8BfMSephEqawK8NoBjAPhvDs4Nd7p0TB5ntb4ZUrnId7L1U?=
 =?us-ascii?Q?jsGiDsRt1uN9/dHQZejdMQ7H9f+NHMqLd8qRSMwjTGpaIp0uEFlJB3PEes0Z?=
 =?us-ascii?Q?o4HgOUHBBs90XJzXyosrGNLDIcSRGg3FQ7Li+dPn7Aha6yR+mcIWP6Ay2oOs?=
 =?us-ascii?Q?J5ZlkyDcljdyOmrb/l9WYM27tnBveb2UvMenCrALLeQ4OYow/CPPenEBT1tx?=
 =?us-ascii?Q?uLXl4rGz4ZPEuD8B7FMTnGRSeMzbhodZq96TSlVmYZd6fTryqgoMTR4fbXiU?=
 =?us-ascii?Q?nCO5sCsVS698bYHr/TekP3ffYK6vGpzRWzoiXjogYMMNm1WL5SsX2opsYWcW?=
 =?us-ascii?Q?cvxyOR/Zfa8xn3G09gMY+w3sKcsg6zTy+DzuyoGAG8z6b0gb7Ecd+LNfOTcw?=
 =?us-ascii?Q?UfXdwMwt3Igp3FNP3TH78tAvRrYIOCtEJz4NKn8lMSK6M0PvJULzlQNjzJkT?=
 =?us-ascii?Q?JX4jgombv4TVD8TRDquAvN9zmSqJUixTPzLGzLhlnMOYWzJoU/Zn2aShpfHP?=
 =?us-ascii?Q?AvQ8C8ay0t58p+Yb/K/xGc31sxQbZhJxvkT4R4o7uY2LIV6bHDA99StnraY4?=
 =?us-ascii?Q?ZIrh1MnEvH9HeBmjcHA2iOwDg84ZSsgvMjkVPFTkAJ2l7OwL/RFpA6w4Yoqd?=
 =?us-ascii?Q?cP0HorsyZglyObUM+13AC9BYl3n0ClaJhe0Y14QqGEum3c7/uZbOCer6/nDb?=
 =?us-ascii?Q?XTjsGOmJ1yPN/SYlMnRru0Q5wBVcR9VZobuRuxUUuywrYgyemcdin7ev4R1h?=
 =?us-ascii?Q?Im/VdGjf05GxgdLGAPqfNN8wuyL7Jumx1W/SUffIHoJlk2c6w3iGvP3tq7NK?=
 =?us-ascii?Q?zvPOnS62tS3pgD6i7FX0E4PXmpkiAfaJ6J/J/JpieaUmoTvtnKt6NAuD21Lk?=
 =?us-ascii?Q?9nrwtCDLFhgatezM6DCyWYIb+Y3ZVQFuviMBrvUmd0AWHSJjrK4iow5xbK6l?=
 =?us-ascii?Q?wxr+L2/4TvPAJcnSIeN15elTLwGuh6cQ46EN4+sis4XoARMhkdsd4rhWMYvy?=
 =?us-ascii?Q?i+jA74Pg4vhoaMP4sMTol+DADq6tVBKOmpFb0NSu?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 058536d6-caf3-414c-863d-08dc536c68e9
X-MS-Exchange-CrossTenant-AuthSource: SA1PR11MB6733.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Apr 2024 23:27:06.3077
 (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: w9QYrOwh5w3lWdug4fiWjh2zcObZQCAZ1GtZH5JMXNBgdl9uLLlODadk8C3cQtPZAiBgOzyTEIz0g99EYZwu8A==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR11MB6477
X-OriginatorOrg: intel.com

Dave Jiang wrote:
> 
> 
> On 3/27/24 10:22 PM, Ira Weiny wrote:
> > Davidlohr Bueso wrote:
> >> On Sun, 24 Mar 2024, ira.weiny@intel.com wrote:
> >>
> >>> From: Navneet Singh <navneet.singh@intel.com>
> >>>
> >>> Until now region modes and decoder modes were equivalent in that they
> >>> were either PMEM or RAM.  With the upcoming addition of Dynamic Capacity
> >>> regions (which will represent an array of device regions [better named
> >>> partitions] the index of which could be different on different
> >>> interleaved devices), the mode of an endpoint decoder and a region will
> >>> no longer be equivalent.
> >>>
> >>> Define a new region mode enumeration and adjust the code for it.
> >>
> >> Could this could also be picked up regardless of dcd?
> > 
> > It could but there is no need for it without DCD.
> > 
> > I will work on re-ordering the cleanups if Dave will agree to take them
> > early.
> 
> There's no reason for the change unless it comes with DCD right? And probably no urgent need to taking it ahead then?
> > 

I think I just replied for a 2nd time to this...  yea.

LOL  I have to do better...

Ira