On Thu, Sep 23, 2021 at 05:25:22PM +0800, Jeffle Xu wrote:
> This is in prep for following per-file DAX checking.
>
> Signed-off-by: Jeffle Xu <jefflexu@xxxxxxxxxxxxxxxxx>
> ---
> fs/fuse/dax.c | 10 +++++++++-
> 1 file changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/fs/fuse/dax.c b/fs/fuse/dax.c
> index 281d79f8b3d3..28db96ea23e2 100644
> --- a/fs/fuse/dax.c
> +++ b/fs/fuse/dax.c
> @@ -1330,11 +1330,19 @@ static const struct address_space_operations fuse_dax_file_aops = {
> .invalidatepage = noop_invalidatepage,
> };
>
> -void fuse_dax_inode_init(struct inode *inode)
> +static bool fuse_should_enable_dax(struct inode *inode)
> {
> struct fuse_conn *fc = get_fuse_conn(inode);
>
> if (!fc->dax)
> + return false;
> +
> + return true;
> +}
It probably is easier to read if we flip the logic.
{
if (fc->dax)
return true;
return false;
}
> +
> +void fuse_dax_inode_init(struct inode *inode)
> +{
> + if (!fuse_should_enable_dax(inode))
> return;
>
> inode->i_flags |= S_DAX;
> --
> 2.27.0
>
