xfs
[Top] [All Lists]

Re: [PATCH 8/8] xfs: add initial DAX support

To: Dave Chinner <david@xxxxxxxxxxxxx>, Boaz Harrosh <boaz@xxxxxxxxxxxxx>
Subject: Re: [PATCH 8/8] xfs: add initial DAX support
From: Boaz Harrosh <boaz@xxxxxxxxxxxxx>
Date: Wed, 25 Mar 2015 11:14:17 +0200
Cc: xfs@xxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, willy@xxxxxxxxxxxxxxx, jack@xxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20150324212550.GU28621@dastard>
References: <1427194266-2885-1-git-send-email-david@xxxxxxxxxxxxx> <1427194266-2885-9-git-send-email-david@xxxxxxxxxxxxx> <55115E20.7090105@xxxxxxxxxxxxx> <20150324212550.GU28621@dastard>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0
On 03/24/2015 11:25 PM, Dave Chinner wrote:
<>
> 
> No, I don't see any reason for a mount flag for this, because we
> do not want to be stuck relying on a mount option forever. Once
> This code has been shaken out, I fully intend DAX to be turned on
> automatically for any device that supports, and for it to be turned
> on and off on a per-inode basis through on-disk inode flags. i.e.
> the mount option is really a temporary solution and I don't want to
> code in any dependencies on DAX being mount wide existing.
> 

OK I dropped this flag. I will just check for the direct_access
vector. I'll need a wrapper that defines to nothing in the
!CONFIG_FS_DAX case

> Cheers,
> Dave.
> 

Thanks for your help
Boaz

<Prev in Thread] Current Thread [Next in Thread>