Re: [PATCH v4 06/12] dax: Check for data cache aliasing at runtime

To: Dan Williams <dan.j.williams@xxxxxxxxx>, Arnd Bergmann <arnd@xxxxxxxx>, Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: [PATCH v4 06/12] dax: Check for data cache aliasing at runtime
From: Mathieu Desnoyers <mathieu.desnoyers@xxxxxxxxxxxx>
Date: Thu, 8 Feb 2024 17:10:59 -0500
Cc: linux-kernel@xxxxxxxxxxxxxxx, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>, Vishal Verma <vishal.l.verma@xxxxxxxxx>, Dave Jiang <dave.jiang@xxxxxxxxx>, Matthew Wilcox <willy@xxxxxxxxxxxxx>, Russell King <linux@xxxxxxxxxxxxxxx>, linux-arch@xxxxxxxxxxxxxxx, linux-cxl@xxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, linux-mm@xxxxxxxxx, linux-xfs@xxxxxxxxxxxxxxx, dm-devel@xxxxxxxxxxxxxxx, nvdimm@xxxxxxxxxxxxxxx, linux-s390@xxxxxxxxxxxxxxx
In-reply-to: <65c54a13c52e_afa429444@dwillia2-xfh.jf.intel.com.notmuch>
References: <20240208184913.484340-1-mathieu.desnoyers@efficios.com> <20240208184913.484340-7-mathieu.desnoyers@efficios.com> <65c54a13c52e_afa429444@dwillia2-xfh.jf.intel.com.notmuch>
User-agent: Mozilla Thunderbird

On 2024-02-08 16:39, Dan Williams wrote:
[...]


So per other feedback on earlier patches, I think this hunk deserves to
be moved to its own patch earlier in the series as a standalone fixup.


Done.


Rest of this patch looks good to me.


Adding your Acked-by to what is left of this patch if OK with you.

Thanks,

Mathieu

--
Mathieu Desnoyers
EfficiOS Inc.
https://www.efficios.com

Follow-Ups:
- Re: [PATCH v4 06/12] dax: Check for data cache aliasing at runtime
  - From: Dan Williams

References:
- [PATCH v4 00/12] Introduce cpu_dcache_is_aliasing() to fix DAX regression
  - From: Mathieu Desnoyers
- [PATCH v4 06/12] dax: Check for data cache aliasing at runtime
  - From: Mathieu Desnoyers
- RE: [PATCH v4 06/12] dax: Check for data cache aliasing at runtime
  - From: Dan Williams