xfs
[Top] [All Lists]

Re: [PATCH 3/3] xfs: introduce per-inode DAX enablement

To: Dan Williams <dan.j.williams@xxxxxxxxx>
Subject: Re: [PATCH 3/3] xfs: introduce per-inode DAX enablement
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Sat, 30 Jan 2016 09:20:06 +1100
Cc: XFS Developers <xfs@xxxxxxxxxxx>, linux-fsdevel <linux-fsdevel@xxxxxxxxxxxxxxx>, ext4@xxxxxxxxxxxxxxx, Matthew Wilcox <willy@xxxxxxxxxxxxxxx>
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <CAPcyv4iXg6pD4p2MTFA3hwUc6eTK=Lpw1nDuchAs1ZQWS05MtA@xxxxxxxxxxxxxx>
References: <1451886892-15548-1-git-send-email-david@xxxxxxxxxxxxx> <1451886892-15548-4-git-send-email-david@xxxxxxxxxxxxx> <CAA9_cmdAYzf3DpjPRZWikNgmJT_sdDACXa_znz6j0oqmmRdLOA@xxxxxxxxxxxxxx> <20160121215820.GA6033@dastard> <CAPcyv4iXg6pD4p2MTFA3hwUc6eTK=Lpw1nDuchAs1ZQWS05MtA@xxxxxxxxxxxxxx>
User-agent: Mutt/1.5.21 (2010-09-15)
On Thu, Jan 21, 2016 at 02:53:06PM -0800, Dan Williams wrote:
> On Thu, Jan 21, 2016 at 1:58 PM, Dave Chinner <david@xxxxxxxxxxxxx> wrote:
> > On Thu, Jan 21, 2016 at 08:37:11AM -0800, Dan Williams wrote:
> >> On Sun, Jan 3, 2016 at 9:54 PM, Dave Chinner <david@xxxxxxxxxxxxx> wrote:
> > i.e. I've already got a couple of fixes we need to add to this - the
> > DAX flag is only valid on CRC enabled filesystems,
> 
> I assume for torn-write protection?  The CRC limitation makes sense,
> but we theoretically could get the same effect by using a separate
> logdev that does not tear writes, right?

No, it requires the CRC enabled format because there isn't any
flag space left in the v2 inode and so we need v3 inodes to store
the flag.

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx

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