On Fri, Mar 01, 2019 at 02:49:54PM +0100, Jan Kara wrote:
> pfn = dax_to_pfn(entry);
> - size = PAGE_SIZE << dax_entry_order(entry);
> + count = 1UL << dax_entry_order(entry);
> + index = xas->xa_index &~ (count - 1);
Hum, why do you compute 'index' here when you actually never use it? The
whole patch looks fishy since it is effectively a noop AFAICT...
*facepalm*. I lost the change to the next line while I was backing out
some extraneous changes.
> dax_entry_mkclean(mapping, xas->xa_index, pfn);
Replacement patch sent.