Message ID | 151943301788.29249.13371602951635567379.stgit@dwillia2-desk3.amr.corp.intel.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
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); >
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);
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> --- include/linux/fs.h | 2 ++ 1 file changed, 2 insertions(+)