Re: [PATCH v4 10/12] filesystem-dax: Introduce dax_lock_page()

2018-07-04 Thread Dan Williams
On Tue, Jun 12, 2018 at 11:07 AM, Ross Zwisler wrote: > On Mon, Jun 11, 2018 at 05:41:46PM +0200, Jan Kara wrote: >> On Fri 08-06-18 16:51:14, Dan Williams wrote: >> > In preparation for implementing support for memory poison (media error) >> > handling via dax mappings, implement a lock_page()

Re: [PATCH v4 10/12] filesystem-dax: Introduce dax_lock_page()

2018-07-04 Thread Dan Williams
On Tue, Jun 12, 2018 at 11:15 AM, Ross Zwisler wrote: > On Fri, Jun 08, 2018 at 04:51:14PM -0700, Dan Williams wrote: >> In preparation for implementing support for memory poison (media error) >> handling via dax mappings, implement a lock_page() equivalent. Poison >> error handling requires rmap

Re: [PATCH v4 10/12] filesystem-dax: Introduce dax_lock_page()

2018-06-12 Thread Ross Zwisler
On Fri, Jun 08, 2018 at 04:51:14PM -0700, Dan Williams wrote: > In preparation for implementing support for memory poison (media error) > handling via dax mappings, implement a lock_page() equivalent. Poison > error handling requires rmap and needs guarantees that the page->mapping > association

Re: [PATCH v4 10/12] filesystem-dax: Introduce dax_lock_page()

2018-06-12 Thread Ross Zwisler
On Mon, Jun 11, 2018 at 05:41:46PM +0200, Jan Kara wrote: > On Fri 08-06-18 16:51:14, Dan Williams wrote: > > In preparation for implementing support for memory poison (media error) > > handling via dax mappings, implement a lock_page() equivalent. Poison > > error handling requires rmap and needs

Re: [PATCH v4 10/12] filesystem-dax: Introduce dax_lock_page()

2018-06-11 Thread Dan Williams
On Mon, Jun 11, 2018 at 8:41 AM, Jan Kara wrote: > On Fri 08-06-18 16:51:14, Dan Williams wrote: >> In preparation for implementing support for memory poison (media error) >> handling via dax mappings, implement a lock_page() equivalent. Poison >> error handling requires rmap and needs guarantees

Re: [PATCH v4 10/12] filesystem-dax: Introduce dax_lock_page()

2018-06-11 Thread Jan Kara
On Fri 08-06-18 16:51:14, Dan Williams wrote: > In preparation for implementing support for memory poison (media error) > handling via dax mappings, implement a lock_page() equivalent. Poison > error handling requires rmap and needs guarantees that the page->mapping > association is maintained /

[PATCH v4 10/12] filesystem-dax: Introduce dax_lock_page()

2018-06-08 Thread Dan Williams
In preparation for implementing support for memory poison (media error) handling via dax mappings, implement a lock_page() equivalent. Poison error handling requires rmap and needs guarantees that the page->mapping association is maintained / valid (inode not freed) for the duration of the lookup.