linux-kernel - [PATCH RFC 0/4] dax: Clean up dax

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-Id: <20230602-dax-region-put-v1-0-d8668f335d45@intel.com>
Date:   Fri, 02 Jun 2023 19:09:20 -0700
From:   Ira Weiny <ira.weiny@...el.com>
To:     Dan Williams <dan.j.williams@...el.com>,
        Vishal Verma <vishal.l.verma@...el.com>,
        Dave Jiang <dave.jiang@...el.com>,
        Andrew Morton <akpm@...ux-foundation.org>,
        Joao Martins <joao.m.martins@...cle.com>,
        Jonathan Cameron <Jonathan.Cameron@...wei.com>
Cc:     Yongqiang Liu <liuyongqiang13@...wei.com>,
        Paul Cassella <cassella@....com>, linux-kernel@...r.kernel.org,
        nvdimm@...ts.linux.dev, linux-cxl@...r.kernel.org,
        Ira Weiny <ira.weiny@...el.com>
Subject: [PATCH RFC 0/4] dax: Clean up dax_region references

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@...el.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,
-- 
Ira Weiny <ira.weiny@...el.com>