Re: [PATCH v10 5/9] fsdax: Introduce dax_load_page()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Shiyang Ruan <ruansy.fnst@xxxxxxxxxxx>
Subject: Re: [PATCH v10 5/9] fsdax: Introduce dax_load_page()
From: Dan Williams <dan.j.williams@xxxxxxxxx>
Date: Tue, 15 Feb 2022 17:34:12 -0800
Cc: Linux Kernel Mailing List <linux-kernel@xxxxxxxxxxxxxxx>, linux-xfs <linux-xfs@xxxxxxxxxxxxxxx>, Linux NVDIMM <nvdimm@xxxxxxxxxxxxxxx>, Linux MM <linux-mm@xxxxxxxxx>, linux-fsdevel <linux-fsdevel@xxxxxxxxxxxxxxx>, "Darrick J. Wong" <djwong@xxxxxxxxxx>, david <david@xxxxxxxxxxxxx>, Christoph Hellwig <hch@xxxxxxxxxxxxx>, Jane Chu <jane.chu@xxxxxxxxxx>, Christoph Hellwig <hch@xxxxxx>
In-reply-to: <20220127124058.1172422-6-ruansy.fnst@fujitsu.com>

On Thu, Jan 27, 2022 at 4:41 AM Shiyang Ruan <ruansy.fnst@xxxxxxxxxxx> wrote:
>
> The current dax_lock_page() locks dax entry by obtaining mapping and
> index in page.  To support 1-to-N RMAP in NVDIMM, we need a new function
> to lock a specific dax entry

I do not see a call to dax_lock_entry() in this function, what keeps
this lookup valid after xas_unlock_irq()?

Follow-Ups:
- Re: [PATCH v10 5/9] fsdax: Introduce dax_load_page()
  - From: Shiyang Ruan

References:
- [PATCH v10 0/9] fsdax: introduce fs query to support reflink
  - From: Shiyang Ruan
- [PATCH v10 5/9] fsdax: Introduce dax_load_page()
  - From: Shiyang Ruan

Prev by Date: Re: [PATCH] init: remove unused names parameter of split_fs_names()
Next by Date: Re: [PATCH v10 6/9] mm: move pgoff_address() to vma_pgoff_address()
Previous by thread: [PATCH v10 5/9] fsdax: Introduce dax_load_page()
Next by thread: Re: [PATCH v10 5/9] fsdax: Introduce dax_load_page()
Index(es):
- Date
- Thread

[Index of Archives] [Linux Ext4 Filesystem] [Union Filesystem] [Filesystem Testing] [Ceph Users] [Ecryptfs] [NTFS 3] [AutoFS] [Kernel Newbies] [Share Photos] [Security] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux Cachefs] [Reiser Filesystem] [Linux RAID] [NTFS 3] [Samba] [Device Mapper] [CEPH Development]