Received-SPF: pass (google.com: domain of linux-kernel+bounces-157427-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Date: Wed, 24 Apr 2024 10:58:25 -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: <66294841a62a9_e1f58294e8@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?tp4OJ2NztcGD9d1o7EWdRSZL03WTm4bR3wYFkip3e4FGmNG4enYZluiT5wG5?=
 =?us-ascii?Q?8JYYPrAKTOyoqvWg9CLWrDgk1fR/jvbwIJxzYhP/2GpkDQSORR2j+6PwtAhF?=
 =?us-ascii?Q?IkaP7StXN2x+Z+V4dqbolRyhSSe8MdNSqk2+Gg/zxbDscK+xbMPrbeGI/x9C?=
 =?us-ascii?Q?DOjGTEG4NT5jMt9q6UAttbWTK0d5n5SiNE7KVwFGpmCHUyRlld6Rh83fngeL?=
 =?us-ascii?Q?i2EA+mQf4BmHsWXV/2NNHRKERNA89EpxiSZW5JlcRpe2kZfUmfSslwQ4tTwp?=
 =?us-ascii?Q?/pE/BiyyGhp/VbmgzK9vta8zUxU9sW+G+flwfP4IDT8wkJX3ByVMz+UBWYXv?=
 =?us-ascii?Q?1xCeYAqJZCS2U8IVbiqa1EWwt4W5sQW/kOe+NE31FTecnNMNrsECSdX2/iWP?=
 =?us-ascii?Q?gDOyDgDok6zPq5/yS8j1nWn9t8ELc+Br0u/XY3YdVzfvaNuM6pxxak+oy22A?=
 =?us-ascii?Q?wVIiWTXyIckRN3nQrCaGB28w5RFEEEy6JjTiFpbQRWQqnxoSVvcYs5GoGENN?=
 =?us-ascii?Q?+g1bmFDTSbD52RKScUvGJjsKsd96sbikyC+x3nA1Q16WS/aBujaD1UghRPsc?=
 =?us-ascii?Q?ioAYjcto8x609PljDrhZgA/4SjEvUMESDXUDb9t1c0LepZFLGWVOQ9EkSyyO?=
 =?us-ascii?Q?xqDAEdjetMl69hK+z3Yz5s6I9YLdAvEIUbjFTInjMaPoOzmNtgE8nJbKNOcr?=
 =?us-ascii?Q?43KqZOpEIRsNp8ncrt+EohTmls77uESh8vA6gP1nVkvqA08ydQa69I4PM66d?=
 =?us-ascii?Q?ctYaOlRs986Y2OyqjKGskHuxACTdGBJea36vvVlWGMppTsQ/t/RcjdvvkzQR?=
 =?us-ascii?Q?YT8h+jF/Wzle0yuYi7Yl6kqJPnbRhc2gkzwzeciWzOKqN7Y+2v868NUkuR6o?=
 =?us-ascii?Q?tXYbQbNedlX6BeZKSmQfHczg40EX5hINhce4lMNPu5XTyE9quHIZxr9vGN9P?=
 =?us-ascii?Q?D/FcLjbOM0nf3jc10i/SANSARupa9QNvSg/kLIvf0AM9LR1apUuAFGaZOy4y?=
 =?us-ascii?Q?GvHsbOUVUv2oj37vMHhIXULmZGvYIlw58TqiY/c4ulzujRb4xG4POf1UfhUE?=
 =?us-ascii?Q?Ru6jTKwS32PtzX7uS4Wx5dhuodm9YzSMDAMRBb60NIKjiOP6FMeb0VLLdfqL?=
 =?us-ascii?Q?OyU0Dh0WKmgDKFAmhev3SOZTcmz6xzJQ5Nn3KtnscR0F7oI2HXSQvGKcOsp1?=
 =?us-ascii?Q?cTTTaTR8jmk01P42Bv7EKNyuK7KzrUvsSx76qQ/ZQlGDWZR7wp1givNZ7EN4?=
 =?us-ascii?Q?yCual5xXR/KXhHJWxToR6xgI++E8GogbB5S43RjtCfD0TCCxB/b/ZPqoyV7x?=
 =?us-ascii?Q?jwhaPrqwRX8I0geoOQ3eq23SK00OB7jfziX9wu2rYfimCo5838PXYrNM/zgp?=
 =?us-ascii?Q?uu742SyzptbuC3Z8siMOiuwMGfCOoEZ+hWhzaa0nQ2A6nwdXpAyu8kknQEXj?=
 =?us-ascii?Q?QPe4M7dVXHs/rTWtrZ7M29wo0hhdxMlrBhJ2/cmGkb8sBzYAypDkg5CroO8A?=
 =?us-ascii?Q?DdcJpnb8OgG845iekWba3Jo1CvjFjZNhsEwaj8o5o4jXSa4UAEf+B3qNM+rp?=
 =?us-ascii?Q?YUM5jyvkslWR9tN3FUIrF4+BpO6rV+rR7ySGl0ba?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 506a4262-20da-4177-10b3-08dc64882500
X-MS-Exchange-CrossTenant-AuthSource: SA1PR11MB6733.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Apr 2024 17:58:28.6445
 (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: bp827Xu2jmlFT+E2Z2WDOHjVi8TBEqJeOEZQB293SLNjAbWHcSaIwhQJR+CvAEEfsR8bLy9zYu10PGOOW+TwwQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR11MB4709
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?

No I don't think so.
Ira

> > 
> > Ira