lkml.org 
[lkml]   [2017]   [Oct]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v3 00/13] dax: fix dma vs truncate and remove 'page-less' support
    On Fri, Oct 27, 2017 at 01:42:16PM +0200, Dan Williams wrote:
    > [replying from my phone, please forgive formatting]
    >
    > On Friday, October 27, 2017, Dave Chinner <david@fromorbit.com> wrote:
    >
    >
    > > > Here are the two primary patches in
    > > > the series, do you think the extent-busy approach would be cleaner?
    > >
    > > The XFS_DAXDMA....
    > >
    > > $DEITY that patch is so ugly I can't even bring myself to type it.
    >
    >
    > Right, and so is the problem it's trying to solve. So where do you want to
    > go from here?
    >
    > I could go back to the FL_ALLOCATED approach, but use page idle callbacks
    > instead of polling for the lease end notification. Or do we want to try
    > busy extents? My concern with busy extents is that it requires more per-fs
    > code.

    I don't care if it takes more per-fs code to solve the problem -
    dumping butt-ugly, nasty locking crap into filesystems that
    filesystem developers are completely unable to test is about the
    worst possible solution you can come up with.

    Cheers,

    Dave.
    --
    Dave Chinner
    david@fromorbit.com

    \
     
     \ /
      Last update: 2017-10-29 22:53    [W:2.779 / U:0.100 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site