From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qk1-f178.google.com (mail-qk1-f178.google.com [209.85.222.178]) (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 8B14C1FCFFC for ; Mon, 23 Mar 2026 05:37:53 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.222.178 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774244274; cv=none; b=InEUDyq7KWoxoTIAqgGhx7DPROhNBBHz3GS5JoBIjvMnZORwFKCFV3f6RbNFnITZje9w16YToq6EGs/mLPBqZxdSKp1GmVx0aMIWhhcmGF7DkpRWoU1MP/WnqxyCtnEwufw+qjlonSRcX1JKQwKDWXybX/cNkYOikl5fr6vgwZ0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774244274; c=relaxed/simple; bh=wDaqZ2jF94kJR9bG7QnZEQCKVo6+1vG1RJvhGhdVIQQ=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=W444HKsSzb6AxxGIar9BMUpDnVtRnFzN8EcEThddTHc9lSxBvrjkCTMpHQGSi8nevP5/UVm58WQRL8EZX9U5QwM4YK0odvHB55fgXZQUF8nfL485Szn2H+tlQHoZFo9sZO03yEjzqQNeOsBahC8ZIilnq/LVniVil6rGZrCvsDI= 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=XIBtvL9H; arc=none smtp.client-ip=209.85.222.178 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="XIBtvL9H" Received: by mail-qk1-f178.google.com with SMTP id af79cd13be357-8cb40149037so424506685a.2 for ; Sun, 22 Mar 2026 22:37:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gourry.net; s=google; t=1774244272; x=1774849072; 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=0eznnwyONrV2zfnlYhXqFCw+ScBb85TQDOYN9JM+ZwY=; b=XIBtvL9HNzFL0OlqmOkpWQ96jOsDt7VXJGWhyMc6hWIENDxkIVBAJWXUqM+z5InrX6 JYDuOngNAQLeHISu1Gqd6sgQ5EtFjkzoEhbVZCHsTbiJIe+g3TAIrdM7TPSNdVRhzH5N 9oRIbEu00/jZcR0iY1+Iyvhe6QRuFIPUL/ykl4m6LWefWyY1q6ZgexyGhN4AthOrp/zq hYMLyJ6L6xcTqw7ledYjMKpzrLDDCeW40NTnNsZyv9Ws6CIX11AsAw01PpzCyrG4+RlR IT8MUXyzvC38b9HgJxmtuhyjZAs46OaaifdjyhiEMLHPACv62zp/cnt48dRhFAbZtb+G d1cA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1774244272; x=1774849072; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=0eznnwyONrV2zfnlYhXqFCw+ScBb85TQDOYN9JM+ZwY=; b=ZfFv9IYGUNk6k8Wus+vOBZPEgOstZwBRXlL5I3zK7KvExG+fszycYyl5UYnhJNkzUF /dcXDwNo7SPVmWrjm+TypKRWEi2H9ZhxTJZUQEaWrcJjjNzqXVyJk9tKVsQtxGJ1JAwL 6szYqQ9IDWks0rFYgcUN5yoR9r6mDYKbnGRxosG51rlhK6a8kXyenEc78U13gj4coU6b BGOugJ3QEKuYZLNi2oMmJXaFpQ4EP/iQzXq+Mpmzpcy/pXUPvW+IkU81qWN1dCfcFd5B rdkXy73P3tDEieMnC2RyZAWefNfgX7Zr/NzB7XBid12aKXu/o1H7uvXeulPn6SJEeC2J hP/Q== X-Forwarded-Encrypted: i=1; AJvYcCUIxY2udLXyumXnvEkxyF9ZMxieKGTBIaZ94iD6zG4NLrpUUq4F4IQwEuhCLXJXvDnbcyrTIeBMBUUlFMQ=@vger.kernel.org X-Gm-Message-State: AOJu0Yxcv12yufpfNzGdef5fq223XsYYPonz10cSBEvGI5wdw82U+gqd 423RgvgG2pZm5BA7o5JJ/obtjXhYM9+d18T8P234jFVlEL0AO+9torOz6vSOD2VgO5k= X-Gm-Gg: ATEYQzySAP6b2qn7vzH3flClgUuLqfDsY9fC0AQenwOnhrG6f4EBrQNSFtCynvbmylc BSJ4E9PE0w/j6KMMTGo0Obye9XXUoJ9ipdElOT6PeFEJA3G7RSI5lEWEybmU6s8nfABLq9TqVxx XmGLLaqmF43AB6CP7+GfTG8wsv/oxOD2TwNn43gFQRXAHpn6299rt7t7hJj2UyzwlsJXzd5yaas BX3udDeVs19YTpddBP7bWkdWQxKxSgid3/nU3EZzgc6JugAhZbnZumPduQ/NMHOL4ar8KZg/426 X4X3cZG8g+s2F8Y61RI28++UUWNUvUpnjyBPizib1kwreoRITdzpPxrum0D9PHguMKYKcywFZZy PkyJFSI+PNpOHKJfyPw40y+OWY4lm35j/aK5hxxoUwW3PCeQDfUryBjnyKvnSa/pJAf3noF60fb S8dwsAVWVrr3y8Ln8TEBQTVZo7QLhug/Y5Sj7HkBnk71xuA7IvrliKa4pf11t116vpBDEnjqkjE m9CXyQSCw== X-Received: by 2002:a05:620a:25cb:b0:8c9:f8e5:9f12 with SMTP id af79cd13be357-8cfc80bb85bmr1589794785a.57.1774244272450; Sun, 22 Mar 2026 22:37:52 -0700 (PDT) Received: from gourry-fedora-PF4VCD3F (pool-96-255-20-138.washdc.ftas.verizon.net. [96.255.20.138]) by smtp.gmail.com with ESMTPSA id af79cd13be357-8cfc90b49d4sm696195785a.33.2026.03.22.22.37.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 22 Mar 2026 22:37:51 -0700 (PDT) Date: Mon, 23 Mar 2026 01:37:49 -0400 From: Gregory Price To: Alison Schofield Cc: linux-cxl@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-team@meta.com, dave@stgolabs.net, jonathan.cameron@huawei.com, dave.jiang@intel.com, vishal.l.verma@intel.com, ira.weiny@intel.com, dan.j.williams@intel.com Subject: Re: [PATCH v4 1/3] cxl/core/region: move pmem region driver logic into region_pmem.c Message-ID: References: <20260322131638.3636725-1-gourry@gourry.net> <20260322131638.3636725-2-gourry@gourry.net> Precedence: bulk X-Mailing-List: linux-kernel@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: On Sun, Mar 22, 2026 at 07:37:28PM -0700, Alison Schofield wrote: > On Sun, Mar 22, 2026 at 09:16:36AM -0400, Gregory Price wrote: > > core/region.c is overloaded with per-region control logic (pmem, dax, > > sysram, etc). Move the pmem region driver logic from region.c into > > region_pmem.c make it clear that this code only applies to pmem regions. > > > > No functional changes. > > This always adds region_pmem.o regardless of CONFIG_CXL_PMEM. > Are you heading towards conditionally compiling, where region_pmem.o is > omitted if !CONFIG_CXL_PMEM? > > Similar question for region_dax.o > I hadn't considered going that far, but it doesn't seem unreasonable? It just seemed natural to break out the code this way, but I don't know if adding more Kconfig cruft is worth it just to save some .o's and a smaller amount of code. The recent pmem/nvdimm Kconfig change does kind of show that we might have some bloat for some deployments which might not care about pmem - for example. ~Gregory