On Fri, May 19, 2023 at 08:40:20AM +0100, David Howells wrote: > +#ifdef CONFIG_FS_DAX > + if (IS_DAX(in->f_mapping->host)) No need for the ifdef. IS_DAX is compile-time false if CONFIG_FS_DAX is not set. A comment on why we're doing this in the code would probably be useful as well.