Messages in this thread | | | Date | Mon, 26 Feb 2018 11:08:32 +0100 | From | Jan Kara <> | Subject | Re: [PATCH v3 5/6] dax: short circuit vma_is_fsdax() in the CONFIG_FS_DAX=n case |
| |
On Fri 23-02-18 16:43:37, Dan Williams wrote: > Do not bother looking up the file type in the case when Filesystem-DAX > is disabled at build time. > > Cc: Alexander Viro <viro@zeniv.linux.org.uk> > Cc: linux-fsdevel@vger.kernel.org > Cc: Christoph Hellwig <hch@lst.de> > Cc: Jan Kara <jack@suse.cz> > Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Looks good. You can add:
Reviewed-by: Jan Kara <jack@suse.cz>
Honza
> --- > include/linux/fs.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/include/linux/fs.h b/include/linux/fs.h > index 7418341578a3..c97fc4dbaae1 100644 > --- a/include/linux/fs.h > +++ b/include/linux/fs.h > @@ -3197,6 +3197,8 @@ static inline bool vma_is_fsdax(struct vm_area_struct *vma) > > if (!vma->vm_file) > return false; > + if (!IS_ENABLED(CONFIG_FS_DAX)) > + return false; > if (!vma_is_dax(vma)) > return false; > inode = file_inode(vma->vm_file); > -- Jan Kara <jack@suse.com> SUSE Labs, CR
| |