From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boaz Harrosh Subject: Re: [PATCH 8/8] xfs: add initial DAX support Date: Wed, 25 Mar 2015 11:14:17 +0200 Message-ID: <55127C69.2000207@plexistor.com> References: <1427194266-2885-1-git-send-email-david@fromorbit.com> <1427194266-2885-9-git-send-email-david@fromorbit.com> <55115E20.7090105@plexistor.com> <20150324212550.GU28621@dastard> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: xfs@oss.sgi.com, linux-fsdevel@vger.kernel.org, willy@linux.intel.com, jack@suse.cz To: Dave Chinner , Boaz Harrosh Return-path: Received: from mail-wi0-f176.google.com ([209.85.212.176]:36796 "EHLO mail-wi0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932107AbbCYJOU (ORCPT ); Wed, 25 Mar 2015 05:14:20 -0400 Received: by wibg7 with SMTP id g7so101489346wib.1 for ; Wed, 25 Mar 2015 02:14:19 -0700 (PDT) In-Reply-To: <20150324212550.GU28621@dastard> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: 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