Message ID | 20230602-dax-region-put-v1-0-d8668f335d45@intel.com (mailing list archive) |
---|---|
Headers | show
Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5D7E61C06 for <nvdimm@lists.linux.dev>; Sat, 3 Jun 2023 02:09:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1685758169; x=1717294169; h=from:subject:date:message-id:mime-version: content-transfer-encoding:to:cc; bh=H0AeSXOoLm6d+sBF/QPZvpAaFBwHsrKQzLumiIp5WZ4=; b=RILLJsQ87KYfLcEP7avfhkEg/4P/xPZE8oK+LS73Y4Iq5DHrWLqN64OS 9CpE7hKn2TtawkFtDYChQBBE+8V303d0FBvNcrZZkI3roH+DBzUtPSt8a I2pHl/PTNoq+XM3CiMrzVCtwucKlpJfrUGlEhWDxS330t29QAjHN+6YEM rILTOVWGnmjMmXILF3XkmFhjMMlPT8yWIbREWJNhYCZOY+WcSjCst2vM2 D0d8OpbOre2tzh6qLIK09Nm/jO+TdZBzIefV6ImKMBtvmJ90YWxwdJPP+ 57BVMxBzLyzkbvhe2HHlxQAJQC6CuTsLPTtaHZn8f4EdIm1wzcTd31PY0 Q==; X-IronPort-AV: E=McAfee;i="6600,9927,10729"; a="340649432" X-IronPort-AV: E=Sophos;i="6.00,214,1681196400"; d="scan'208";a="340649432" Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Jun 2023 19:09:28 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10729"; a="852354408" X-IronPort-AV: E=Sophos;i="6.00,214,1681196400"; d="scan'208";a="852354408" Received: from iweiny-mobl.amr.corp.intel.com (HELO localhost) ([10.212.97.230]) by fmsmga001-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Jun 2023 19:09:26 -0700 From: Ira Weiny <ira.weiny@intel.com> Subject: [PATCH RFC 0/4] dax: Clean up dax_region references Date: Fri, 02 Jun 2023 19:09:20 -0700 Message-Id: <20230602-dax-region-put-v1-0-d8668f335d45@intel.com> Precedence: bulk X-Mailing-List: nvdimm@lists.linux.dev List-Id: <nvdimm.lists.linux.dev> List-Subscribe: <mailto:nvdimm+subscribe@lists.linux.dev> List-Unsubscribe: <mailto:nvdimm+unsubscribe@lists.linux.dev> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-B4-Tracking: v=1; b=H4sIANCgemQC/x2NQQqDQAxFryJZNzATodpuCz2A29JFZkx1Fp1KY osg3r2jy8f7j7+CiSYxuFYrqPySpU8u4E8VxJHzIJj6wkCOand2hD0vqDKUGU7fGRuK1F583cT goUSBTTAo5zju2ZttFt3FpPJKy/H0gO5+g+e2/QFT0YkefgAAAA== To: Dan Williams <dan.j.williams@intel.com>, Vishal Verma <vishal.l.verma@intel.com>, Dave Jiang <dave.jiang@intel.com>, Andrew Morton <akpm@linux-foundation.org>, Joao Martins <joao.m.martins@oracle.com>, Jonathan Cameron <Jonathan.Cameron@huawei.com> Cc: Yongqiang Liu <liuyongqiang13@huawei.com>, Paul Cassella <cassella@hpe.com>, linux-kernel@vger.kernel.org, nvdimm@lists.linux.dev, linux-cxl@vger.kernel.org, Ira Weiny <ira.weiny@intel.com> X-Mailer: b4 0.13-dev-9a8cd X-Developer-Signature: v=1; a=ed25519-sha256; t=1685758165; l=1350; i=ira.weiny@intel.com; s=20221211; h=from:subject:message-id; bh=H0AeSXOoLm6d+sBF/QPZvpAaFBwHsrKQzLumiIp5WZ4=; b=PjRZrjIiB58OFu6WAQdcDL3Xejhg8ixrdE5f89yqGxPlQfRinmET22nVsPyNarG24GyR2xccf dB0c8u+VKbMANqBhmPQqFxwwEe0qVe05gadf2596QxTAPXXTClSoZD8 X-Developer-Key: i=ira.weiny@intel.com; a=ed25519; pk=noldbkG+Wp1qXRrrkfY1QJpDf7QsOEthbOT7vm0PqsE= |
Series | dax: Clean up dax_region references | expand |
In[*] Yongqiang Liu presented a fix to the reference counting associated with the dax_region in hmem. At the time it was thought the patch was unnecessary because dax_region_unregister() call would properly handle reference counting. Upon closer inspection Paul noted that this was not the case. In fact Yongqiang's patch was correct but there were other issues as well. This series includes Yongqiang's patch and breaks up additional fixes which can be backported if necessary followed by a final patch which simplifies the reference counting. [*] https://lore.kernel.org/all/20221203095858.612027-1-liuyongqiang13@huawei.com/ Signed-off-by: Ira Weiny <ira.weiny@intel.com> --- Ira Weiny (3): dax/bus: Fix leaked reference in alloc_dax_region() dax/cxl: Fix refcount leak in cxl_dax_region_probe() dax/bus: Remove unnecessary reference in alloc_dax_region() Yongqiang Liu (1): dax/hmem: Fix refcount leak in dax_hmem_probe() drivers/dax/bus.c | 6 +++++- drivers/dax/cxl.c | 7 +------ drivers/dax/hmem/hmem.c | 7 +------ drivers/dax/pmem.c | 8 +------- 4 files changed, 8 insertions(+), 20 deletions(-) --- base-commit: 921bdc72a0d68977092d6a64855a1b8967acc1d9 change-id: 20230602-dax-region-put-72c289137cb1 Best regards,